Re: [SOLVED + PATCH]: documented Oops running big-endian reiserfs on parisc architecture

Andi Kleen (ak@suse.de)
Tue, 4 Sep 2001 12:25:24 +0200


On Tue, Sep 04, 2001 at 03:04:54AM -0700, David S. Miller wrote:
> From: Andi Kleen <ak@suse.de>
> Date: 04 Sep 2001 11:44:30 +0200
>
> Jeff Mahoney <jeffm@suse.com> writes:
>
> > I did kick around the idea of making those macros the default accessors for
> > the deh_state member (which is the only place they're used), but it unfairly
> > penalizes arches that don't need them.
>
> On archs that don't need them {get,put}_unaligned should be just normal
> assignments. They are certainly on i386.
>
> I can also almost guarentee you that the x86 will sometimes not
> execute these bitops atomically on SMP.

It's not needed when you have another lock to protect and don't have
interrupt threads.

iirc near all of reiserfs still runs under the BKL. BKL/schedule are full
memory barriers.

> What's more, you will have less QA'ing to do, since this code will
> always be in use and thus tested.

I think for a fs it is best to go with bigger locks and avoid the issue
of atomic operations completely.

-Andi
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/