bind on arm

Rafal Jaworowski raj at semihalf.com
Sun Feb 21 21:46:17 UTC 2010


On 2010-02-21, at 22:01, Doug Barton wrote:

> On 02/21/10 07:28, Olivier Houchard wrote:
>> Hi Doug,
>> 
>> 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.
> 
> 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



More information about the freebsd-arm mailing list