16 Apr, 2015
18 commits
-
... except where that code acts as a filesystem driver, rather than
working with dentries given to it.Signed-off-by: David Howells
Signed-off-by: Al Viro -
most of the ->d_inode uses there refer to the same inode IO would
go to, i.e. d_backing_inode()Signed-off-by: David Howells
Signed-off-by: Al Viro -
socket inodes and sunrpc filesystems - inodes owned by that code
Signed-off-by: David Howells
Signed-off-by: Al Viro -
places where we are dealing with S_ISSOCK file creation/lookups.
Signed-off-by: David Howells
Signed-off-by: Al Viro -
relayfs and tracefs are dealing with inodes of their own;
those two act as filesystem driversSigned-off-by: David Howells
Signed-off-by: Al Viro -
Signed-off-by: David Howells
Signed-off-by: Al Viro -
Fix up some ->d_inode accesses in the chelsio driver.
(1) FILE_DATA() should just be replaced with file_inode().
(2) set_debugfs_file_size() should be removed and debugfs_create_file_size()
should be used to create the file.Signed-off-by: David Howells
Signed-off-by: Al Viro -
Cachefiles should perform fs modifications (eg. vfs_unlink()) on the top layer
only and should not attempt to alter the lower layer.Signed-off-by: David Howells
Signed-off-by: Al Viro -
AF_UNIX sockets should call mknod on the top layer only and should not attempt
to modify the lower layer in a layered filesystem such as overlayfs.Signed-off-by: David Howells
Signed-off-by: Al Viro -
Signed-off-by: Christoph Hellwig
Signed-off-by: Al Viro -
Fix inconsistent use of file_inode() vs file->f_path.dentry->d_inode.
Reported-by: Dan Carpenter
Signed-off-by: David Howells
Signed-off-by: Al Viro -
Make pathwalk use d_is_reg() rather than S_ISREG() to determine whether to
honour O_TRUNC. Since this occurs after complete_walk(), the dentry type
field cannot change and the inode pointer cannot change as we hold a ref on
the dentry, so this should be safe.Signed-off-by: David Howells
Signed-off-by: Al Viro -
Fix up debugfs to use d_is_dir(dentry) in place of
S_ISDIR(dentry->d_inode->i_mode).Signed-off-by: David Howells
Signed-off-by: Al Viro -
Where we have:
if (!dentry->d_inode || d_is_negative(dentry)) {
type constructions in pathwalk we should be able to eliminate the check of
d_inode and rely solely on the result of d_is_negative() or d_is_positive().What we do have to take care to do is to read d_inode after calling a
d_is_xxx() typecheck function to get the barriering right.Signed-off-by: David Howells
Signed-off-by: Al Viro -
Don't use d_inode as a variable name as it now masks a function name.
Signed-off-by: David Howells
Signed-off-by: Al Viro -
Impose ordering on accesses of d_inode and d_flags to avoid the need to do
this:if (!dentry->d_inode || d_is_negative(dentry)) {
when this:
if (d_is_negative(dentry)) {
should suffice.
This check is especially problematic if a dentry can have its type field set
to something other than DENTRY_MISS_TYPE when d_inode is NULL (as in
unionmount).What we really need to do is stick a write barrier between setting d_inode and
setting d_flags and a read barrier between reading d_flags and reading
d_inode.Signed-off-by: David Howells
Signed-off-by: Al Viro -
Supply two functions to test whether a filesystem's own dentries are positive
or negative (d_really_is_positive() and d_really_is_negative()).The problem is that the DCACHE_ENTRY_TYPE field of dentry->d_flags may be
overridden by the union part of a layered filesystem and isn't thus
necessarily indicative of the type of dentry.Normally, this would involve a negative dentry (ie. ->d_inode == NULL) having
->d_layer.lower pointed to a lower layer dentry, DCACHE_PINNING_LOWER set and
the DCACHE_ENTRY_TYPE field set to something other than DCACHE_MISS_TYPE - but
it could also involve, say, a DCACHE_SPECIAL_TYPE being overridden to
DCACHE_WHITEOUT_TYPE if a 0,0 chardev is detected in the top layer.However, inside a filesystem, when that fs is looking at its own dentries, it
probably wants to know if they are really negative or not - and doesn't care
about the fallthrough bits used by the union.To this end, a filesystem should normally use d_really_is_positive/negative()
when looking at its own dentries rather than d_is_positive/negative() and
should use d_inode() to get at the inode.Anyone looking at someone else's dentries (this includes pathwalk) should use
d_is_xxx() and d_backing_inode().Signed-off-by: David Howells
Signed-off-by: Al Viro -
Signed-off-by: Al Viro
12 Apr, 2015
22 commits
-
we can do that now - all we need is to clear IOCB_DIRECT from ->ki_flags in
"can't do dio" case.Signed-off-by: Al Viro
-
... avoiding write_iter/fcntl races.
Signed-off-by: Al Viro
-
... returning -E... upon error and amount of data left in iter after
(possible) truncation upon success. Note, that normal case gives
a non-zero (positive) return value, so any tests for != 0 _must_ be
updated.Signed-off-by: Al Viro
Conflicts:
fs/ext4/file.c -
Alignment checks for dio depend upon the range truncation done by
generic_write_checks(). They can be done as soon as we got ocfs2_rw_lock()
and that actually makes ocfs2_prepare_inode_for_write() simpler.The only thing to watch out for is restoring the original count
in "unlock and redo without dio" case. Position doesn't need to be
restored, since we change it only in O_APPEND case and in that case it
will be reassigned anyway.Signed-off-by: Al Viro
-
it's &iocb->ki_pos; no need to obfuscate.
Signed-off-by: Al Viro
-
it's easier to do generic_write_checks() first
Signed-off-by: Al Viro
-
already done by caller. We used to call __fuse_direct_write(), which
called generic_write_checks(); now the former got expanded, bringing
the latter to the surface. It used to be called all along and calling
it from there had been wrong all along...Signed-off-by: Al Viro
-
simpler that way...
Signed-off-by: Al Viro
-
Signed-off-by: Al Viro
-
all remaining callers are passing 0; some just obscure that fact.
Signed-off-by: Al Viro
-
Signed-off-by: Al Viro
-
Signed-off-by: Al Viro
-
A side effect worth noting: in O_APPEND case we set ->ki_pos early,
so if it turns out to be an error or a zero-length write, we'll
end up with ->ki_pos modified. Safe, since all callers never
look at the ->ki_pos after the call of __generic_file_write_iter()
returning non-positive, all the way to caller of ->write_iter() and
those discard ->ki_pos when getting that.Signed-off-by: Al Viro
-
Signed-off-by: Al Viro
-
Signed-off-by: Al Viro
-
That allows ->write_iter() instances much more convenient life wrt
iocb->ki_pos (and fixes several filesystems with borderline POSIX
violations when zero-length write succeeds and changes the current
position).Signed-off-by: Al Viro
-
Now that no one is using rw, remove it completely.
Signed-off-by: Omar Sandoval
Signed-off-by: Al Viro -
The rw parameter to direct_IO is redundant with iov_iter->type, and
treated slightly differently just about everywhere it's used: some users
do rw & WRITE, and others do rw == WRITE where they should be doing a
bitwise check. Simplify this with the new iov_iter_rw() helper, which
always returns either READ or WRITE.Signed-off-by: Omar Sandoval
Signed-off-by: Al Viro -
And use iov_iter_rw() instead.
Signed-off-by: Omar Sandoval
Signed-off-by: Al Viro -
Most filesystems call through to these at some point, so we'll start
here.Signed-off-by: Omar Sandoval
Signed-off-by: Al Viro -
Get either READ or WRITE out of iter->type.
Signed-off-by: Omar Sandoval
Signed-off-by: Al Viro
