Date: Sun, 21 Feb 2010 22:46:14 +0100 From: Rafal Jaworowski <raj@semihalf.com> To: Doug Barton <dougb@FreeBSD.org> Cc: freebsd-arm@FreeBSD.org, Olivier Houchard <cognet@ci0.org> Subject: Re: bind on arm Message-ID: <EF0BCCA4-011C-4705-AE02-FC85B6419BDE@semihalf.com> In-Reply-To: <4B819F21.70907@FreeBSD.org> References: <20100221152824.GA58060@ci0.org> <4B819F21.70907@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 2010-02-21, at 22:01, Doug Barton wrote: > On 02/21/10 07:28, Olivier Houchard wrote: >> Hi Doug, >>=20 >> I'd like to get the attached patch committed. It fixes the bind arm = atomic >> stuff in -CURRENT and RELENG_8. Some times ago, the RAS address was = changed, >> but it was hardcoded in the bind code and the change was never = reflected. >> The patch uses a macro, so that even if it happens again it won't be = a problem. >=20 > First question (and I think the answer is yes, but I need to > double-check) is it the consensus of the ARM gurus that this is the > right solution? Second question, is this solution something that I can > send upstream, both in the sense that I have permission to do so, and > that it would be generally applicable to ARM on other OSs? I believe the desired direction here is to make the cmpxchg atomic op a = CPU-level routine, rather than it being part of the application code. = I'm not sure what is a wider context of such a change for BIND or = whether this code is FreeBSD-only, but I thought the posted changes were = just a quick fix to verify that we have a problem around RAS, and = there's more insight needed before considering it a final fix. Rafal
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?EF0BCCA4-011C-4705-AE02-FC85B6419BDE>