Re: [PATCH] cifs: reinstate the forcegid option

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 2012-10-04 at 18:46 +0530, Suresh Jayaraman wrote:
> On 10/04/2012 01:32 AM, Jeff Layton wrote:
> > Apparently this was lost when we converted to the standard option
> > parser.
> > 
> > Cc: Sachin Prabhu <sprabhu@xxxxxxxxxx>
> > Reported-by: Gregory Lee Bartholomew <gregory.lee.bartholomew@xxxxxxxxx>
> > Signed-off-by: Jeff Layton <jlayton@xxxxxxxxxx>
> > ---
> 
> Oops, atleast a couple of options have been reported missing and fixed
> so far. Time to audit the new parsing code so that we fix them
> proactively before we get reports from users..?
> 

I am taking another look at the code to compare the options supported
against the old parser with the new parser. Some gotchas which tripped
me last time were checks for shortened options which also matched the
longer versions of the mount options.

We should also start thinking about paring down the number of mount
options which in my opinion are on the high side.

Sachin Prabhu

--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux