[fuse-discuss] Code for review: read call implementation

Mark Phalan Mark.Phalan at Sun.COM
Thu Jun 28 05:30:40 PDT 2007


On Fri, 2007-06-01 at 00:08 -0500, subra056 at umn.edu wrote:
> Hi all,
> 
> I have added "read" and related calls to fuse kernel module. The webrev of 
> the changes can be found here: http://www-users.cs.umn.edu/~subram/webrev/ 
> "Sdiffs" link might give a useful view of the changes. Depending on the 
> review comments, I would expect to commit these changes in a week or two.
> 
> Regards,
> Sunil

So I finally finished looking at your changes.. They look pretty good to
me!

I just have one very minor nit:

fuse.h:91, 180
Missing space after '}'

Unfortunately I'm not 100% up to speed on how the fuse read
implementation should work so I think Sarah should definitely take a
look at this.

Cheers,

-Mark

> _______________________________________________
> fuse-discuss mailing list
> fuse-discuss at opensolaris.org
> http://opensolaris.org/mailman/listinfo/fuse-discuss




More information about the fuse-discuss mailing list