lkml.org 
[lkml]   [2005]   [Oct]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [RFC] atomic create+open
From
Date
on den 12.10.2005 Klokka 13:01 (+0200) skreiv Miklos Szeredi:

> However I don't really like that the filesystem is reentered from
> lookup_instantiate_filp() via __dentry_open() and ->open(). Is this
> necessary?

If filesystems need to be able to change the value of f_mapping, then
yes, however if none of the potential users of lookup_instantiate_filp()
care about doing so, then we can get rid of it.
I don't care either way since we will not be supporting non-intent based
opens for NFSv4.

> I see you've fixed the O_TRUNC problem. The accmode==3 case is still
> slightly broken, since now the file is being opened in read-write mode
> instead of no-read-no-write mode. This probably won't break anything
> too badly though.

It is non-portable and it was never supported on NFSv4 anyway. If
someone cares, they can fix it, but I don't see much need.

> Equivalent and simpler:
>
> flags = nd->intent.open.flags - 1;
>
> Note, that the access bits of intent.open.flags will never both be
> zero, so this is safe.

Agreed.

Cheers,
Trond

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-10-12 15:25    [W:0.367 / U:0.380 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site