Re: [patch] cifs: accidentally creating empty files

[Dan Carpenter <dan.carpenter@oracle.com>]

On Wed, Oct 19, 2011 at 09:14:37PM -0500, Steve French wrote:
> Doesn't this force the create to happen later - rather than
> at lookup time where it belongs?
> 
> if the issue is just noperm ... we should let this through if the user
> has local permissions.  Doubling the cost of a file create to Samba
> seems like a bad idea (ie doing aQueryPathInfo AND an NTCreateX
> doubles the roundrip, doubles the load on the server etc.) - the whole
> point of this is to let us do an "atomic" open or create operation
> and not have to split it into multiple requests.  In any case why would
> we do the open and then follow it with a create?
> 
> Can we fix this to (at least) narrow the performance penalty.
> 

Yes, it does add another back and forth to file create...  It's hard/
impossible to check the permissions first and then decide whether to
pass the O_CREAT flag.  Maybe we could add an if statement to check
whether noperm was used and the noperm version could use the create
on lookup.  It's obviously not pretty but it might be worth it for
speedup.

regards,
dan carpenter