linux-stable.git/fs/ncpfs, branch linux-3.14.y

move d_rcu from overlapping d_child to overlapping d_alias

2015-04-29T08:31:54+00:00

commit 946e51f2bf37f1656916eb75bd0742ba33983c28 upstream.

move d_rcu from overlapping d_child to overlapping d_alias

Signed-off-by: Al Viro 
Cc: Ben Hutchings 
[hujianyang: Backported to 3.14 refer to the work of Ben Hutchings in 3.2:
 - Apply name changes in all the different places we use d_alias and d_child
 - Move the WARN_ON() in __d_free() to d_free() as we don't have dentry_free()]
Signed-off-by: hujianyang 
Signed-off-by: Greg Kroah-Hartman

ncpfs: return proper error from NCP_IOC_SETROOT ioctl

2015-01-08T18:00:50+00:00

commit a682e9c28cac152e6e54c39efcf046e0c8cfcf63 upstream.

If some error happens in NCP_IOC_SETROOT ioctl, the appropriate error
return value is then (in most cases) just overwritten before we return.
This can result in reporting success to userspace although error happened.

This bug was introduced by commit 2e54eb96e2c8 ("BKL: Remove BKL from
ncpfs").  Propagate the errors correctly.

Coverity id: 1226925.

Fixes: 2e54eb96e2c80 ("BKL: Remove BKL from ncpfs")
Signed-off-by: Jan Kara 
Cc: Petr Vandrovec 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds 
Signed-off-by: Greg Kroah-Hartman

ncpfs: rcu-delay unload_nls() and freeing ncp_server

2013-10-25T03:43:28+00:00

makes ->d_hash() and ->d_compare() safety in RCU mode independent
from vfsmount_lock.

Signed-off-by: Al Viro

ncpfs: switch to %p[dD]

2013-10-25T03:34:52+00:00

Signed-off-by: Al Viro

ncpfs: fix error return code in ncp_parse_options()

2013-07-08T09:36:43+00:00

Fix to return -EINVAL from the option parse error handling
case instead of 0, as done elsewhere in this function.

Signed-off-by: Wei Yongjun 
Signed-off-by: Al Viro

ncpfs: use vma_pages() to replace (vm_end - vm_start) >> PAGE_SHIFT

2013-07-03T23:07:26+00:00

(*->vm_end - *->vm_start) >> PAGE_SHIFT operation is implemented
as a inline funcion vma_pages() in linux/mm.h, so using it.

Signed-off-by: Libin 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

Don't pass inode to ->d_hash() and ->d_compare()

2013-06-29T08:57:36+00:00

Instances either don't look at it at all (the majority of cases) or
only want it to find the superblock (which can be had as dentry->d_sb).
A few cases that want more are actually safe with dentry->d_inode -
the only precaution needed is the check that it hadn't been replaced with
NULL by rmdir() or by overwriting rename(), which case should be simply
treated as cache miss.

Signed-off-by: Linus Torvalds 
Signed-off-by: Al Viro

kill find_inode_number()

2013-06-29T08:57:20+00:00

the only remaining caller (in ncpfs) is guaranteed to return 0 -
we only hit it if we'd just checked that there's no dentry with
such name.

Signed-off-by: Al Viro

ncpfs: don't bother with EBUSY on removal of busy directories

2013-06-29T08:57:16+00:00

Signed-off-by: Al Viro

[readdir] convert ncpfs

2013-06-29T08:56:57+00:00

Signed-off-by: Al Viro