[lkml]   [2007]   [May]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] ubi: kill homegrown endian macros
    On Thu, 17 May 2007 18:09:50 +0300 Artem Bityutskiy <> wrote:

    > On Thu, 2007-05-17 at 16:56 +0200, Christoph Hellwig wrote:
    > > On Thu, May 17, 2007 at 05:50:43PM +0300, Artem Bityutskiy wrote:
    > > > Christoph,
    > > >
    > > > On Thu, 2007-05-17 at 16:32 +0200, Christoph Hellwig wrote:
    > > > > Kill ubis homegrown endianess handling crap and replace it with
    > > > > the normal kernel endianess handling.
    > > >
    > > > Err,__be32 and the company are just sparse things, while I have compiler
    > > > checks with my struct ubi32_t and friends. JFFS2 also uses the same
    > > > technique. Why do you force me to rely on sparse instead instead of
    > > > compiler?
    > >
    > > Yes. Like all other code in the kernel aswell.
    > Andrew, may I please have your ack that I absolutely have to use __be32
    > instead of my own types since Christoph tends to provide no explanation
    > to his requests.

    umm.. I'd say what you've done in there is an improvement to the exiting
    stuff: getting gcc to check it is better than having to use sparse.

    I'd have expected gcc to generate poorer code with your approach but I'm
    showing zero text size changes from Christoph's patch (gcc-4.1 and

    So I wouldn't be averse to creating a new, generic, kernel-wide alternative
    to the existing __be32/__le32/etc code. It is an improvement.

    But I don't think we want driver-private implementations.

    We could conceivably simply switch the existing stuff to use structs, but
    quite a lot of code assumes that cpu_to_foo32(0) == 0 and just does
    open-coded assigments of zero. They'd need fixups.
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2007-05-17 19:37    [W:0.020 / U:11.620 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site