[lkml]   [2005]   [Sep]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: List of things requested by lkml for reiser4 inclusion (to review)
    Christoph Hellwig wrote:

    > 6. remove type safe lists and type safe hash queues.
    >> not done, it is not clear that the person asking for this represents a unified consensus of lkml. Other persons instead asked that it just be moved out of reiser4 code into the generic kernel code, which implies they did not object to it. There are many who like being type safe. Akpm, what do you yourself think?
    >I think quite a few people complained ;-)
    I remember that you complained.

    > It's huge CPP abuse
    can you define what that means? and how abuse differs from cleverness?
    This code was not my idea, but it seemed more cleverness than abuse to
    me when I read it.

    > which we
    >generally don't want - at least for the hash case we already had something
    >similar (linux/ghash.h and got rid of it). That beeing said a generic hash
    >abstraction without too much CPP abuse might be really useful, but the list
    >code should certainly go.
    >>8. Remove all assertions because they clutter the code and make it hard to read
    >> We think this person was not an experienced security specialist,
    >please stop attacking people personally all the time.
    It is not an attack, fewer than 1% of programmers are security
    specialists, and I go on to say that until I was educated by some such
    specialists, I would have agreed with him, so I don't think you can view
    it as an attack. I hardly expect most people to hang out with the DoD.

    > You're certainly not
    >what I'd call an experienced security specialist either, but fortunately that
    >doesn't matter for this case at all.
    >Removing all assertations certainly doesn't make sense, we have them all
    >over the tree. Whether your own assert macros makes sense is a different
    >question, but given that you use something similar in reiser3 and lots
    >of other drivers have their own things built around BUG() I won't complain
    >to loudly.
    >What should go on the other hand are useless assertation, like one that
    >asserts that something is non-NULL just before dereferencing it - the
    >latter gives a backtrace just as nice.
    I'll be happy to accept point of code improvements.

    >As additional requirements please give people time to actually audit the
    9 months time given so far....

    > I've started but it's quite a pain with all the plugin
    >indirections right now.
    Here I sympathize with you. If anyone has ideas for how to comment it,
    etc., to make it trivial to follow the indirections, I am all ears. It
    annoys me also.

    >One major item I found is that you're using your own code for the
    >read/write file operations,
    As I remember, there were things that did not code right using the
    generic code, and I told the guys to code it right. I no longer
    remember what those things were, quite honestly, but I remember they
    affected performance. The whole idea of vfs operations is that each
    file system gets to do them its own way, and please remember that we
    code for more than just use on Linux.

    > which not only duplicates core code but
    >also lacks features (vectored I/O, AIO, direct I/O) from the common code
    in time....

    >and is totally buggy (it has no chance on working on architectures that
    >have completely separate address spaces for user/kernel like s390, please
    >check how they define PAGE_OFFSET).
    zam, please look at that on Tuesday.

    > Depending on your requirements you
    >might not use the complete generic code, but you should at least use
    >the most important fragments, ala XFS or ocfs, and improve them where
    Oh we do try to do that where it does what we need. However, you should
    keep in mind that reiser4 is not made just for Linux. Our code needs to
    be at least moderately self-contained.

    >A very annoying small thing that comes to mind is the usage of
    >reiser4_internal. Please remove it, all but exported symbol are
    Here I mostly agree. Edward, can you work on a patch to fix it? Thanks.

    Christoph, thanks kindly for the donation of your time to improving our
    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: 2005-09-09 23:24    [W:0.027 / U:15.228 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site