[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