[linux-cifs-client] Re: [PATCH 1/7] [CIFS] add local struct inode pointer to cifs_setattr

simo idra at samba.org
Sat May 10 00:47:57 GMT 2008


On Fri, 2008-05-09 at 19:01 -0400, Jeff Layton wrote:
> On Fri, 9 May 2008 17:29:12 -0500
> "Steve French" <smfrench at gmail.com> wrote:
> 
> > Merged patches 1, 6 and 7 from this series.
> > 
> > For the others since they change current behavior and would break some
> > apps (including connectathon test cases) would like the behavior e.g.
> > to be configurable by mount option
> > 
> 
> Thanks...
> 
> How about this? We make the new behavior the default, and add a
> mount option ("fakeperm" or something) to allow for the legacy
> behavior where permissions can be set in memory only.
> 
> Sound reasonable? If so, I'll get to work on a respin of this set.

Although I do not like an option that screws up permissions, I think
this is a good enough plan.

Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer <simo at samba.org>
Senior Software Engineer at Red Hat Inc. <ssorce at redhat.com>



More information about the linux-cifs-client mailing list