| Message ID | 20181121024400.4346-4-devel@etsukata.com |
|---|---|
| State | New, archived |
| Headers | show |
| Series | fs: fix race between llseek SEEK_END and write | expand |
On Wed, Nov 21, 2018 at 11:43:59AM +0900, Eiichi Tsukata wrote: > This patch itself seems to be just a cleanup but with the > commit b25bd1d9fd87 ("vfs: fix race between llseek SEEK_END and write") > it fixes race. Please move this patch to the beginning of the series and replace the commit log with something like the one below. Note that your commit id is different from the one that will appear once applied upstream, so the aboe isn't too helpful. --- f2fs: use generic_file_llseek f2fs always passes inode->i_sb->s_maxbytes to generic_file_llseek_size, and thus should simply use generic_file_llseek. For now this is a just a cleanup, but it will allow f2fs to pick up a race fix in generic_file_llseek for free.
2018年11月21日(水) 18:23 Christoph Hellwig <hch@infradead.org>: > > On Wed, Nov 21, 2018 at 11:43:59AM +0900, Eiichi Tsukata wrote: > > This patch itself seems to be just a cleanup but with the > > commit b25bd1d9fd87 ("vfs: fix race between llseek SEEK_END and write") > > it fixes race. > > Please move this patch to the beginning of the series and replace > the commit log with something like the one below. Note that your > commit id is different from the one that will appear once applied > upstream, so the aboe isn't too helpful. > > --- > f2fs: use generic_file_llseek > > f2fs always passes inode->i_sb->s_maxbytes to generic_file_llseek_size, > and thus should simply use generic_file_llseek. For now this is a just > a cleanup, but it will allow f2fs to pick up a race fix in > generic_file_llseek for free. Thanks, I'll fix it in v2.
diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c index 88b124677189..14a923607eff 100644 --- a/fs/f2fs/file.c +++ b/fs/f2fs/file.c @@ -447,15 +447,11 @@ static loff_t f2fs_seek_block(struct file *file, loff_t offset, int whence) static loff_t f2fs_llseek(struct file *file, loff_t offset, int whence) { - struct inode *inode = file->f_mapping->host; - loff_t maxbytes = inode->i_sb->s_maxbytes; - switch (whence) { case SEEK_SET: case SEEK_CUR: case SEEK_END: - return generic_file_llseek_size(file, offset, whence, - maxbytes, i_size_read(inode)); + return generic_file_llseek(file, offset, whence); case SEEK_DATA: case SEEK_HOLE: if (offset < 0)
This patch itself seems to be just a cleanup but with the commit b25bd1d9fd87 ("vfs: fix race between llseek SEEK_END and write") it fixes race. Signed-off-by: Eiichi Tsukata <devel@etsukata.com> --- fs/f2fs/file.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-)