kernel.git/net/9p, branch linux-4.1.y

net/9p: Switch to wait_event_killable()

2017-12-08T23:01:08Z

[ Upstream commit 9523feac272ccad2ad8186ba4fcc89103754de52 ] Because userspace gets Very Unhappy when calls like stat() and execve() return -EINTR on 9p filesystem mounts. For instance, when bash is looking in PATH for things to execute and some SIGCHLD interrupts stat(), bash can throw a spurious 'command not found' since it doesn't retry the stat(). In practice, hitting the problem is rare and needs a really slow/bogged down 9p server. Cc: stable@vger.kernel.org Signed-off-by: Tuomas Tynkkynen Signed-off-by: Al Viro Signed-off-by: Sasha Levin

p9_client_readdir() fix

2017-05-17T19:08:23Z

[ Upstream commit 71d6ad08379304128e4bdfaf0b4185d54375423e ] Don't assume that server is sane and won't return more data than asked for. Cc: stable@vger.kernel.org Signed-off-by: Al Viro Signed-off-by: Sasha Levin

9p: ensure err is initialized to 0 in p9_client_read/write

2015-09-13T16:07:50Z

commit 999b8b88c6060adf7a9b7907740ae86ace65291e upstream. Some use of those functions were providing unitialized values to those functions. Notably, when reading 0 bytes from an empty file on a 9P filesystem, the return code of read() was not 0. Tested with this simple program: #include #include #include #include #include int main(int argc, const char **argv) { assert(argc == 2); char buffer[256]; int fd = open(argv[1], O_RDONLY|O_NOCTTY); assert(fd >= 0); assert(read(fd, buffer, 0) == 0); return 0; } Signed-off-by: Vincent Bernat Signed-off-by: Al Viro Signed-off-by: Greg Kroah-Hartman

p9_client_write(): avoid double p9_free_req()

2015-08-03T16:29:18Z

commit 67e808fbb0404a12d9b9830a44bbb48d447d8bc9 upstream. Braino in "9p: switch p9_client_write() to passing it struct iov_iter *"; if response is impossible to parse and we discard the request, get the out of the loop right there. Signed-off-by: Al Viro Signed-off-by: Greg Kroah-Hartman

9p: forgetting to cancel request on interrupted zero-copy RPC

2015-08-03T16:29:18Z

commit a84b69cb6e0a41e86bc593904faa6def3b957343 upstream. If we'd already sent a request and decide to abort it, we *must* issue TFLUSH properly and not just blindly reuse the tag, or we'll get seriously screwed when response eventually arrives and we confuse it for response to later request that had reused the same tag. Signed-off-by: Al Viro Signed-off-by: Greg Kroah-Hartman

Merge tag 'for-linus-4.1-merge-window' of git://git.kernel.org/pub/scm/linux/kernel/git/ericvh/v9fs

2015-04-18T21:45:30Z

Pull 9pfs updates from Eric Van Hensbergen: "Some accumulated cleanup patches for kerneldoc and unused variables as well as some lock bug fixes and adding privateport option for RDMA" * tag 'for-linus-4.1-merge-window' of git://git.kernel.org/pub/scm/linux/kernel/git/ericvh/v9fs: net/9p: add a privport option for RDMA transport. fs/9p: Initialize status in v9fs_file_do_lock. net/9p: Initialize opts->privport as it should be. net/9p: use memcpy() instead of snprintf() in p9_mount_tag_show() 9p: use unsigned integers for nwqid/count 9p: do not crash on unknown lock status code 9p: fix error handling in v9fs_file_do_lock 9p: remove unused variable in p9_fd_create() 9p: kerneldoc warning fixes

net/9p: remove (now-)unused helpers

2015-04-12T02:28:29Z

Signed-off-by: Al Viro

p9_client_attach(): set fid->uid correctly

2015-04-12T02:28:28Z

it's almost always equal to current_fsuid(), but there's an exception - if the first writeback fid is opened by non-root *and* that happens before root has done any lookups in /, we end up doing attach for root. The current code leaves the resulting FID owned by root from the server POV and by non-root from the client one. Unfortunately, it means that e.g. massive dcache eviction will leave that user buggered - they'll end up redoing walks from / *and* picking that FID every time. As soon as they try to create something, the things will get nasty. Signed-off-by: Al Viro

9p: switch p9_client_read() to passing struct iov_iter *

2015-04-12T02:28:27Z

... and make it loop Signed-off-by: Al Viro

9p: switch p9_client_write() to passing it struct iov_iter *

2015-04-12T02:28:25Z

... and make it loop until it's done Signed-off-by: Al Viro