lkml.org 
[lkml]   [2016]   [Jan]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] fuse: implement cuse mmap
On Tue, 5 Jan 2016 21:21:16 +0100 Luca Risolia <luca.risolia@studio.unibo.it> wrote:

> I was wondering if there's a chance to have this patch merged anytime
> soon. Note that FUSE maintainer left off sometime ago and there's no one
> taking care of kernel patches at the moment.
>
> Please let Jader and me know if there are any problems.

I didn't know that FUSE maintainership had altered.

At -rc8 isn't a convenient time - please resend after 4.5-rc1. I of
course *could* merge the patch now and hold it off for the 4.6 merge
window, but...

Please write a much more complete changelog. Much. What are the
objectives, why do we need this feature. How did you choose to
implement the feature. Are there alternative ways of implementing it
and if so, why did you choose this way, etc.

Also, the code is poorly commented. Please fix that, concentrating
less on *what* the code is doing and more on *why* it is doing it.
Tell the reader what you're trying to achieve. Tell the reader those
things which aren't discernible from the code itself.

Documenting the data structures is particularly important. In kernel
code, if that is done well enough then the code itself tends to become
rather obvious. Like, what's going on with this vm_original_ops thing?
hm, that's a bit of a head-scratcher at present.

Thanks.


\
 
 \ /
  Last update: 2016-01-05 22:41    [W:0.406 / U:0.104 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site