lkml.org 
[lkml]   [2001]   [Sep]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [SOLVED + PATCH]: documented Oops running big-endian reiserfs on parisc architecture
On Mon, Sep 03, 2001 at 12:34:37AM +0200, Richard Zidlicky wrote:
> On Sun, Sep 02, 2001 at 07:57:17PM +0200, thunder7@xs4all.nl wrote:
> >
> > --- linux/include/linux/reiserfs_fs.h Sun Sep 2 21:54:25 2001
> > +++ linux-new/include/linux/reiserfs_fs.h Sun Sep 2 20:47:27 2001
> > @@ -924,7 +924,7 @@
> > #define DEH_Visible 2
> >
> > /* 64 bit systems (and the S/390) need to be aligned explicitly -jdm */
> > -#if BITS_PER_LONG == 64 || defined(__s390__)
> > +#if BITS_PER_LONG == 64 || defined(__s390__) || defined(__hppa__)
> > # define ADDR_UNALIGNED_BITS (3)
> > #endif
>
> couldn't reiserfs use asm/unaligned.h like anyone else?
> Seems at least sparc and mips may need the same treatment.

I'll be the first to admit that having the #if followed by arch defines
really sucks, and that asm/unaligned.h would be much cleaner if for no
other reason than avoiding an unnecessary gotcha for someone porting a new
arch.

Unfortunately, this doesn't appear to work in all cases. I know from
testing that the S/390 case will fail using the {get,put)_unaligned macros,
since they are identical to a normal, unassisted assignment.

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.

I'm open to suggestions.

-Jeff

--
Jeff Mahoney | "Bill Gates is a monocle and a Persian cat away
jeffm@suse.com | from being the villain in a James Bond movie."
jeffm@csh.rit.edu | -- Dennis Miller
-
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/

\
 
 \ /
  Last update: 2005-03-22 13:02    [W:0.025 / U:0.544 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site