diff options
author | Ulrich Drepper <drepper@redhat.com> | 2007-07-23 18:43:46 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-07-24 12:24:58 -0700 |
commit | 0d786d4a2773f06a791e8c3730d049077fb81df6 (patch) | |
tree | de553420b6b1a70b80034385ff94eb706bac12ab /fs | |
parent | 4f640efb3170dbcf99a37a3cc99060647b95428c (diff) | |
download | kernel_samsung_smdk4412-0d786d4a2773f06a791e8c3730d049077fb81df6.zip kernel_samsung_smdk4412-0d786d4a2773f06a791e8c3730d049077fb81df6.tar.gz kernel_samsung_smdk4412-0d786d4a2773f06a791e8c3730d049077fb81df6.tar.bz2 |
fallocate syscall interface deficiency
The fallocate syscall returns ENOSYS in case the filesystem does not support
the operation and expects the userlevel code to fill in. This is good in
concept.
The problem is that the libc code for old kernels should be able to
distinguish the case where the syscall is not at all available vs not
functioning for a specific mount point. As is this is not possible and we
always have to invoke the syscall even if the kernel doesn't support it.
I suggest the following patch. Using EOPNOTSUPP is IMO the right thing to do.
Cc: Amit Arora <aarora@in.ibm.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/open.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -403,7 +403,7 @@ asmlinkage long sys_fallocate(int fd, int mode, loff_t offset, loff_t len) if (inode->i_op && inode->i_op->fallocate) ret = inode->i_op->fallocate(inode, mode, offset, len); else - ret = -ENOSYS; + ret = -EOPNOTSUPP; out_fput: fput(file); |