Re: [PATCH] Replace XFS bit functions with Linux functions

Russell Cattelan
2007-10-03T18:20:23Z

[-- Attachment #1: Type: text/plain, Size: 1653 bytes --]

Andi Kleen wrote:
>> I would like to keep thing abstracted enough such that it is won't be to
>> difficult keep to map to the FreeBSD bit functions.
>>
>
> It's no different to map linux bit functions to FreeBSD than mapping
> XFS bit functions to FreeBSD. Besides I expect it has fls already --
> that is actually a BSDism.
>
>
Yes but for the most part the common code does not play favorites
to linux or bsd or X in terms of which platform is favored. (with some
execptions).
I have been working getting the freebsd xfs code based updated and I've
come
across calls to spin_lock that may need to redone. I need to look at
the XFS code
paths more to figure out which freebsd lock is needed.
For the most part freebsd tries not to use mutex spin_lock (disable
interrupts and spin) since it is
more expensive that just mtx_lock (do not disable interrupts and spin
for awhile then sleep)
and unless there is a good reason to disable interrupts they try
avoiding doing so.

Playing favorites is kinda what has fueled this whole "ported half of
irix to linux" FUD
that has dogged xfs forever.
So for the same reason that people complained about mapping irix calls
to linux calls
I don't like the argument that linux calls should just be mapped to bsd
calls.

And I agree with Dave the -1 adjustments are bit hard to follow and would
be better encapsulated in a inline call where it can be documented.

Having the xfs_ call in the common code would also allow the fbsd port
to catch up with
this optimization as some point vs having to back stitch the current
code into common layer
and eventually having to realign.

> -Andi
>
>

[-- Attachment #2: cattelan.vcf --]
[-- Type: text/x-vcard, Size: 131 bytes --]

begin:vcard
fn:Russell Cattelan
n:Cattelan;Russell
email;internet:cattelan@thebarn.com
x-mozilla-html:FALSE
version:2.1
end:vcard