[lkml]   [2005]   [Sep]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: I request inclusion of reiser4 in the mainline kernel
    more trivial review comments ontop of the previous one, after looking
    at things:

    - please never use list_for_each in new code but list_for_each_entry
    - never use kernel_thread in new code but kthread_*
    - do_sendfile duplicates the common sendfile code. why aren't you
    using the generic code?
    - there's tons of really useless assertation of the category
    discussed in the last thread
    - there's tons of deep pagecache messing in there. normally this
    shouldn't be a filesystem, and if this breaks because of VM changes you'll
    have to fix it, don't complain..
    - you still do your plugin mess in ->readpage. honsetly could you
    please explain why mpage_readpage{,s} don't work for you?
    - (issues with the read/write path already addresses in the previous thread)
    - looking at ->d_count in ->release is wrong
    - still has security plugin stuff that duplicates LSM
    - why do underlying attributes change when VFS inode doesn't change?
    if not please rip out most of getattr_common
    - link_common S_ISDIR doesn't make sense, VFS takes care of it
    - please use the generic_readlink infrastructure

    additinoal comment is that the code is very messy, very different
    from normal kernel style, full of indirections and thus hard to read.
    real review will take some time.
    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-16 19:43    [W:0.019 / U:101.304 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site