summaryrefslogtreecommitdiff
path: root/fs/coda
diff options
context:
space:
mode:
authorDominik Brodowski <linux@dominikbrodowski.net>2020-01-01 20:05:03 +0100
committerLinus Torvalds <torvalds@linux-foundation.org>2020-01-02 16:15:33 -0800
commit74f1a299107b9e1a563831a4ba85f769ab577164 (patch)
tree847a00cb533bea4d8af5e451315ed60215548d19 /fs/coda
parent738d2902773e30939a982c8df7a7f94293659810 (diff)
Revert "fs: remove ksys_dup()"
This reverts commit 8243186f0cc7 ("fs: remove ksys_dup()") and the subsequent fix for it in commit 2d3145f8d280 ("early init: fix error handling when opening /dev/console"). Trying to use filp_open() and f_dupfd() instead of pseudo-syscalls caused more trouble than what is worth it: it requires accessing vfs internals and it turns out there were other bugs in it too. In particular, the file reference counting was wrong - because unlike the original "open+2*dup" sequence it used "filp_open+3*f_dupfd" and thus had an extra leaked file reference. That in turn then caused odd problems with Androidx86 long after boot becaue of how the extra reference to the console kept the session active even after all file descriptors had been closed. Reported-by: youling 257 <youling257@gmail.com> Cc: Arvind Sankar <nivedita@alum.mit.edu> Cc: Al Viro <viro@zeniv.linux.org.uk> Signed-off-by: Dominik Brodowski <linux@dominikbrodowski.net> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/coda')
0 files changed, 0 insertions, 0 deletions