* [GIT PULL] f2fs update for 4.16-rc1 @ 2018-01-30 22:32 ` Jaegeuk Kim 0 siblings, 0 replies; 19+ messages in thread From: Jaegeuk Kim @ 2018-01-30 22:32 UTC (permalink / raw) To: Linus Torvalds; +Cc: Linux F2FS Dev Mailing List, Linux Kernel Mailing List Hi Linux, Could you please consider the below pull request? Thanks, The following changes since commit ace52288edf0cb5e7a52b681f057f86224c49c27: Merge tag 'for-linus-20171218' of git://git.infradead.org/linux-mtd (2017-12-18 16:18:01 -0800) are available in the Git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git tags/f2fs-for-4.16-rc1 for you to fetch changes up to 1c1d35df71104c76a4a2e25862926f22c334c9d6: f2fs: support inode creation time (2018-01-25 14:10:39 -0800) ---------------------------------------------------------------- f2fs-for-4.16-rc1 In this round, we've followed up to support some generic features such as cgroup, block reservation, linking fscrypt_ops, delivering write_hints, and some ioctls. And, we could fix some corner cases in terms of power-cut recovery and subtle deadlocks. Enhancement: - bitmap operations to handle NAT blocks - readahead to improve readdir speed - switch to use fscrypt_* - apply write hints for direct IO - add reserve_root=%u,resuid=%u,resgid=%u to reserve blocks for root/uid/gid - modify b_avail and b_free to consider root reserved blocks - support cgroup writeback - support FIEMAP_FLAG_XATTR for fibmap - add F2FS_IOC_PRECACHE_EXTENTS to pre-cache extents - add F2FS_IOC_{GET/SET}_PIN_FILE to pin LBAs for data blocks - support inode creation time Bug fix: - sysfile-based quota operations - memory footprint accounting - allow to write data on partial preallocation case - fix deadlock case on fallocate - fix to handle fill_super errors - fix missing inode updates of fsync'ed file - recover renamed file which was fsycn'ed before - drop inmemory pages in corner error case - keep last_disk_size correctly - recover missing i_inline flags during roll-forward Various clean-up patches were added as well. ---------------------------------------------------------------- Chao Yu (32): f2fs: reserve nid resource for quota sysfile f2fs: remove unneeded memory footprint accounting f2fs: fix lock dependency in between dio_rwsem & i_mmap_sem f2fs: inject fault to kzalloc f2fs: inject fault to kvmalloc f2fs: spread f2fs_k{m,z}alloc f2fs: fix error handling in fill_super f2fs: clean up hash codes f2fs: clean up f2fs_map_blocks f2fs: don't return value in truncate_data_blocks_range f2fs: fix potential hangtask in f2fs_trace_pid f2fs: continue to do direct IO if we only preallocate partial blocks f2fs: clean up unneeded declaration f2fs: avoid high cpu usage in discard thread f2fs: remove unused pend_list_tag f2fs: fix to caclulate required free section correctly f2fs: fix to cover f2fs_inline_data_fiemap with inode_lock f2fs: support FIEMAP_FLAG_XATTR f2fs: support F2FS_IOC_PRECACHE_EXTENTS f2fs: speed up defragment on sparse file f2fs: fix to drop all inmem pages correctly f2fs: clean up error path of fill_super f2fs: kill F2FS_INLINE_XATTR_ADDRS for cleanup f2fs: fix to update last_disk_size correctly f2fs: split need_inplace_update f2fs: hanlde error case in f2fs_ioc_shutdown f2fs: stop gc/discard thread after fs shutdown f2fs: drop page cache after fs shutdown f2fs: use GFP_F2FS_ZERO for cleanup f2fs: clean up duplicated assignment in init_discard_policy f2fs: stop issuing discard if fs is readonly f2fs: support inode creation time Daeho Jeong (3): f2fs: handle newly created page when revoking inmem pages f2fs: prevent newly created inode from being dirtied incorrectly f2fs: correct removexattr behavior for null valued extended attribute Elena Reshetova (1): posix_acl: convert posix_acl.a_refcount from atomic_t to refcount_t Eric Biggers (5): f2fs: switch to fscrypt_file_open() f2fs: switch to fscrypt_prepare_link() f2fs: switch to fscrypt_prepare_rename() f2fs: switch to fscrypt_prepare_lookup() f2fs: switch to fscrypt_prepare_setattr() Hyunchul Lee (1): f2fs: apply write hints to select the type of segment for direct write Jaegeuk Kim (12): f2fs: return error during fill_super f2fs: recover directory operations by fsync f2fs: fix missing error number for xattr operation f2fs: skip stop_checkpoint for user data writes f2fs: enable quota at remount from r to w f2fs: show precise # of blocks that user/root can use f2fs: add reserved blocks for root user f2fs: add resgid and resuid to reserve root blocks f2fs: add an ioctl to disable GC for specific file f2fs: allow quota to use reserved blocks f2fs: recover some i_inline flags f2fs: allow to recover node blocks given updated checkpoint LiFan (4): f2fs: fix concurrent problem for updating free bitmap f2fs: remove an excess variable f2fs: remove a redundant conditional expression f2fs: use unlikely for release case Sheng Yong (4): f2fs: introduce sysfs readdir_ra to readahead inode block in readdir f2fs: still write data if preallocate only partial blocks f2fs: remove unused parameter f2fs: avoid hungtask when GC encrypted block if io_bits is set Wei Yongjun (1): f2fs: make local functions static Yufen Yu (1): f2fs: implement cgroup writeback support Yunlei He (6): f2fs: no need to read nat block if nat_block_bitmap is set f2fs: no need return value in restore summary process f2fs: fix an error case of missing update inode page f2fs: update inode info to inode page for new file f2fs: check node page again in write end io f2fs: rebuild sit page from sit info in mem Yunlong Song (1): f2fs: check segment type in __f2fs_replace_block Zhikang Zhang (1): f2fs: remove repeated f2fs_bug_on Documentation/ABI/testing/sysfs-fs-f2fs | 6 + fs/f2fs/acl.c | 2 +- fs/f2fs/checkpoint.c | 10 +- fs/f2fs/data.c | 299 +++++++++++++++++++++++++++----- fs/f2fs/debug.c | 12 +- fs/f2fs/dir.c | 6 + fs/f2fs/f2fs.h | 201 +++++++++++++++------ fs/f2fs/file.c | 248 ++++++++++++++++++++------ fs/f2fs/gc.c | 18 +- fs/f2fs/gc.h | 2 + fs/f2fs/inode.c | 34 +++- fs/f2fs/namei.c | 67 +++---- fs/f2fs/node.c | 149 ++++++++-------- fs/f2fs/node.h | 4 + fs/f2fs/recovery.c | 27 ++- fs/f2fs/segment.c | 129 +++++++------- fs/f2fs/segment.h | 92 +++++----- fs/f2fs/super.c | 142 ++++++++++++--- fs/f2fs/sysfs.c | 14 +- fs/f2fs/trace.c | 12 +- fs/f2fs/xattr.c | 12 +- fs/posix_acl.c | 6 +- include/linux/f2fs_fs.h | 14 +- include/linux/posix_acl.h | 7 +- include/trace/events/f2fs.h | 3 +- 25 files changed, 1064 insertions(+), 452 deletions(-) ^ permalink raw reply [flat|nested] 19+ messages in thread
* [GIT PULL] f2fs update for 4.16-rc1 @ 2018-01-30 22:32 ` Jaegeuk Kim 0 siblings, 0 replies; 19+ messages in thread From: Jaegeuk Kim @ 2018-01-30 22:32 UTC (permalink / raw) To: Linus Torvalds; +Cc: Linux Kernel Mailing List, Linux F2FS Dev Mailing List Hi Linux, Could you please consider the below pull request? Thanks, The following changes since commit ace52288edf0cb5e7a52b681f057f86224c49c27: Merge tag 'for-linus-20171218' of git://git.infradead.org/linux-mtd (2017-12-18 16:18:01 -0800) are available in the Git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git tags/f2fs-for-4.16-rc1 for you to fetch changes up to 1c1d35df71104c76a4a2e25862926f22c334c9d6: f2fs: support inode creation time (2018-01-25 14:10:39 -0800) ---------------------------------------------------------------- f2fs-for-4.16-rc1 In this round, we've followed up to support some generic features such as cgroup, block reservation, linking fscrypt_ops, delivering write_hints, and some ioctls. And, we could fix some corner cases in terms of power-cut recovery and subtle deadlocks. Enhancement: - bitmap operations to handle NAT blocks - readahead to improve readdir speed - switch to use fscrypt_* - apply write hints for direct IO - add reserve_root=%u,resuid=%u,resgid=%u to reserve blocks for root/uid/gid - modify b_avail and b_free to consider root reserved blocks - support cgroup writeback - support FIEMAP_FLAG_XATTR for fibmap - add F2FS_IOC_PRECACHE_EXTENTS to pre-cache extents - add F2FS_IOC_{GET/SET}_PIN_FILE to pin LBAs for data blocks - support inode creation time Bug fix: - sysfile-based quota operations - memory footprint accounting - allow to write data on partial preallocation case - fix deadlock case on fallocate - fix to handle fill_super errors - fix missing inode updates of fsync'ed file - recover renamed file which was fsycn'ed before - drop inmemory pages in corner error case - keep last_disk_size correctly - recover missing i_inline flags during roll-forward Various clean-up patches were added as well. ---------------------------------------------------------------- Chao Yu (32): f2fs: reserve nid resource for quota sysfile f2fs: remove unneeded memory footprint accounting f2fs: fix lock dependency in between dio_rwsem & i_mmap_sem f2fs: inject fault to kzalloc f2fs: inject fault to kvmalloc f2fs: spread f2fs_k{m,z}alloc f2fs: fix error handling in fill_super f2fs: clean up hash codes f2fs: clean up f2fs_map_blocks f2fs: don't return value in truncate_data_blocks_range f2fs: fix potential hangtask in f2fs_trace_pid f2fs: continue to do direct IO if we only preallocate partial blocks f2fs: clean up unneeded declaration f2fs: avoid high cpu usage in discard thread f2fs: remove unused pend_list_tag f2fs: fix to caclulate required free section correctly f2fs: fix to cover f2fs_inline_data_fiemap with inode_lock f2fs: support FIEMAP_FLAG_XATTR f2fs: support F2FS_IOC_PRECACHE_EXTENTS f2fs: speed up defragment on sparse file f2fs: fix to drop all inmem pages correctly f2fs: clean up error path of fill_super f2fs: kill F2FS_INLINE_XATTR_ADDRS for cleanup f2fs: fix to update last_disk_size correctly f2fs: split need_inplace_update f2fs: hanlde error case in f2fs_ioc_shutdown f2fs: stop gc/discard thread after fs shutdown f2fs: drop page cache after fs shutdown f2fs: use GFP_F2FS_ZERO for cleanup f2fs: clean up duplicated assignment in init_discard_policy f2fs: stop issuing discard if fs is readonly f2fs: support inode creation time Daeho Jeong (3): f2fs: handle newly created page when revoking inmem pages f2fs: prevent newly created inode from being dirtied incorrectly f2fs: correct removexattr behavior for null valued extended attribute Elena Reshetova (1): posix_acl: convert posix_acl.a_refcount from atomic_t to refcount_t Eric Biggers (5): f2fs: switch to fscrypt_file_open() f2fs: switch to fscrypt_prepare_link() f2fs: switch to fscrypt_prepare_rename() f2fs: switch to fscrypt_prepare_lookup() f2fs: switch to fscrypt_prepare_setattr() Hyunchul Lee (1): f2fs: apply write hints to select the type of segment for direct write Jaegeuk Kim (12): f2fs: return error during fill_super f2fs: recover directory operations by fsync f2fs: fix missing error number for xattr operation f2fs: skip stop_checkpoint for user data writes f2fs: enable quota at remount from r to w f2fs: show precise # of blocks that user/root can use f2fs: add reserved blocks for root user f2fs: add resgid and resuid to reserve root blocks f2fs: add an ioctl to disable GC for specific file f2fs: allow quota to use reserved blocks f2fs: recover some i_inline flags f2fs: allow to recover node blocks given updated checkpoint LiFan (4): f2fs: fix concurrent problem for updating free bitmap f2fs: remove an excess variable f2fs: remove a redundant conditional expression f2fs: use unlikely for release case Sheng Yong (4): f2fs: introduce sysfs readdir_ra to readahead inode block in readdir f2fs: still write data if preallocate only partial blocks f2fs: remove unused parameter f2fs: avoid hungtask when GC encrypted block if io_bits is set Wei Yongjun (1): f2fs: make local functions static Yufen Yu (1): f2fs: implement cgroup writeback support Yunlei He (6): f2fs: no need to read nat block if nat_block_bitmap is set f2fs: no need return value in restore summary process f2fs: fix an error case of missing update inode page f2fs: update inode info to inode page for new file f2fs: check node page again in write end io f2fs: rebuild sit page from sit info in mem Yunlong Song (1): f2fs: check segment type in __f2fs_replace_block Zhikang Zhang (1): f2fs: remove repeated f2fs_bug_on Documentation/ABI/testing/sysfs-fs-f2fs | 6 + fs/f2fs/acl.c | 2 +- fs/f2fs/checkpoint.c | 10 +- fs/f2fs/data.c | 299 +++++++++++++++++++++++++++----- fs/f2fs/debug.c | 12 +- fs/f2fs/dir.c | 6 + fs/f2fs/f2fs.h | 201 +++++++++++++++------ fs/f2fs/file.c | 248 ++++++++++++++++++++------ fs/f2fs/gc.c | 18 +- fs/f2fs/gc.h | 2 + fs/f2fs/inode.c | 34 +++- fs/f2fs/namei.c | 67 +++---- fs/f2fs/node.c | 149 ++++++++-------- fs/f2fs/node.h | 4 + fs/f2fs/recovery.c | 27 ++- fs/f2fs/segment.c | 129 +++++++------- fs/f2fs/segment.h | 92 +++++----- fs/f2fs/super.c | 142 ++++++++++++--- fs/f2fs/sysfs.c | 14 +- fs/f2fs/trace.c | 12 +- fs/f2fs/xattr.c | 12 +- fs/posix_acl.c | 6 +- include/linux/f2fs_fs.h | 14 +- include/linux/posix_acl.h | 7 +- include/trace/events/f2fs.h | 3 +- 25 files changed, 1064 insertions(+), 452 deletions(-) ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [GIT PULL] f2fs update for 4.16-rc1 2018-01-30 22:32 ` Jaegeuk Kim (?) @ 2018-01-30 22:54 ` Stephen Rothwell 2018-01-31 0:25 ` Jaegeuk Kim -1 siblings, 1 reply; 19+ messages in thread From: Stephen Rothwell @ 2018-01-30 22:54 UTC (permalink / raw) To: Jaegeuk Kim Cc: Linus Torvalds, Linux F2FS Dev Mailing List, Linux Kernel Mailing List, Chris Mason, Josef Bacik, Yufen Yu, Tejun Heo Hi all, On Tue, 30 Jan 2018 14:32:20 -0800 Jaegeuk Kim <jaegeuk@kernel.org> wrote: > > Could you please consider the below pull request? > > Thanks, > > The following changes since commit ace52288edf0cb5e7a52b681f057f86224c49c27: > > Merge tag 'for-linus-20171218' of git://git.infradead.org/linux-mtd (2017-12-18 16:18:01 -0800) > > are available in the Git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git tags/f2fs-for-4.16-rc1 > > for you to fetch changes up to 1c1d35df71104c76a4a2e25862926f22c334c9d6: > > f2fs: support inode creation time (2018-01-25 14:10:39 -0800) This has an interesting conflict with the btrfs tree: ----------------------------------------------------------------------- After merging the f2fs tree, today's linux-next build (x86_64 allmodconfig) failed like this: /home/sfr/next/next/fs/f2fs/super.c: In function 'f2fs_fill_super': /home/sfr/next/next/fs/f2fs/super.c:2563:18: error: 'SB_I_CGROUPWB' undeclared (first use in this function); did you mean 'S_CGROUPWB'? sb->s_iflags |= SB_I_CGROUPWB; ^~~~~~~~~~~~~ S_CGROUPWB Caused by commit 578c647879f7 ("f2fs: implement cgroup writeback support") interacting with commit 34934189a3dd ("cgroup, writeback: replace SB_I_CGROUPWB with per-inode S_CGROUPWB" from the btrfs tree. I have reverted the f2fs tree commit for today, but a better resolution would be appreciated. ----------------------------------------------------------------------- I have seen no response to the above email and have been reverting that commit since Jan 18. -- Cheers, Stephen Rothwell ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [GIT PULL] f2fs update for 4.16-rc1 2018-01-30 22:54 ` Stephen Rothwell @ 2018-01-31 0:25 ` Jaegeuk Kim 0 siblings, 0 replies; 19+ messages in thread From: Jaegeuk Kim @ 2018-01-31 0:25 UTC (permalink / raw) To: Stephen Rothwell Cc: Linus Torvalds, Linux F2FS Dev Mailing List, Linux Kernel Mailing List, Chris Mason, Josef Bacik, Yufen Yu, Tejun Heo On 01/31, Stephen Rothwell wrote: > Hi all, > > On Tue, 30 Jan 2018 14:32:20 -0800 Jaegeuk Kim <jaegeuk@kernel.org> wrote: > > > > Could you please consider the below pull request? > > > > Thanks, > > > > The following changes since commit ace52288edf0cb5e7a52b681f057f86224c49c27: > > > > Merge tag 'for-linus-20171218' of git://git.infradead.org/linux-mtd (2017-12-18 16:18:01 -0800) > > > > are available in the Git repository at: > > > > git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git tags/f2fs-for-4.16-rc1 > > > > for you to fetch changes up to 1c1d35df71104c76a4a2e25862926f22c334c9d6: > > > > f2fs: support inode creation time (2018-01-25 14:10:39 -0800) > > This has an interesting conflict with the btrfs tree: > > ----------------------------------------------------------------------- > After merging the f2fs tree, today's linux-next build (x86_64 > allmodconfig) failed like this: > > /home/sfr/next/next/fs/f2fs/super.c: In function 'f2fs_fill_super': > /home/sfr/next/next/fs/f2fs/super.c:2563:18: error: 'SB_I_CGROUPWB' undeclared (first use in this function); did you mean 'S_CGROUPWB'? > sb->s_iflags |= SB_I_CGROUPWB; > ^~~~~~~~~~~~~ > S_CGROUPWB > > Caused by commit > > 578c647879f7 ("f2fs: implement cgroup writeback support") > > interacting with commit > > 34934189a3dd ("cgroup, writeback: replace SB_I_CGROUPWB with per-inode S_CGROUPWB" > > from the btrfs tree. > > I have reverted the f2fs tree commit for today, but a better resolution > would be appreciated. > ----------------------------------------------------------------------- > > I have seen no response to the above email and have been reverting that > commit since Jan 18. I think this can be addressed by: diff --git a/fs/f2fs/inode.c b/fs/f2fs/inode.c index 89c838bfb067..c743b73976d0 100644 --- a/fs/f2fs/inode.c +++ b/fs/f2fs/inode.c @@ -48,9 +48,10 @@ void f2fs_set_inode_flags(struct inode *inode) new_fl |= S_DIRSYNC; if (f2fs_encrypted_inode(inode)) new_fl |= S_ENCRYPTED; + new_fl |= S_CGROUPWB; inode_set_flags(inode, new_fl, S_SYNC|S_APPEND|S_IMMUTABLE|S_NOATIME|S_DIRSYNC| - S_ENCRYPTED); + S_ENCRYPTED|S_CGROUPWB); } static void __get_inode_rdev(struct inode *inode, struct f2fs_inode *ri) diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c index 8173ae688814..ebcd58476fc7 100644 --- a/fs/f2fs/super.c +++ b/fs/f2fs/super.c @@ -2560,7 +2560,6 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent) sb->s_flags = (sb->s_flags & ~SB_POSIXACL) | (test_opt(sbi, POSIX_ACL) ? SB_POSIXACL : 0); memcpy(&sb->s_uuid, raw_super->uuid, sizeof(raw_super->uuid)); - sb->s_iflags |= SB_I_CGROUPWB; /* init f2fs-specific super block info */ sbi->valid_super_block = valid_super_block; ^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [GIT PULL] f2fs update for 4.16-rc1 @ 2018-01-31 0:25 ` Jaegeuk Kim 0 siblings, 0 replies; 19+ messages in thread From: Jaegeuk Kim @ 2018-01-31 0:25 UTC (permalink / raw) To: Stephen Rothwell Cc: Yufen Yu, Josef Bacik, Linux Kernel Mailing List, Linux F2FS Dev Mailing List, Chris Mason, Tejun Heo, Linus Torvalds On 01/31, Stephen Rothwell wrote: > Hi all, > > On Tue, 30 Jan 2018 14:32:20 -0800 Jaegeuk Kim <jaegeuk@kernel.org> wrote: > > > > Could you please consider the below pull request? > > > > Thanks, > > > > The following changes since commit ace52288edf0cb5e7a52b681f057f86224c49c27: > > > > Merge tag 'for-linus-20171218' of git://git.infradead.org/linux-mtd (2017-12-18 16:18:01 -0800) > > > > are available in the Git repository at: > > > > git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git tags/f2fs-for-4.16-rc1 > > > > for you to fetch changes up to 1c1d35df71104c76a4a2e25862926f22c334c9d6: > > > > f2fs: support inode creation time (2018-01-25 14:10:39 -0800) > > This has an interesting conflict with the btrfs tree: > > ----------------------------------------------------------------------- > After merging the f2fs tree, today's linux-next build (x86_64 > allmodconfig) failed like this: > > /home/sfr/next/next/fs/f2fs/super.c: In function 'f2fs_fill_super': > /home/sfr/next/next/fs/f2fs/super.c:2563:18: error: 'SB_I_CGROUPWB' undeclared (first use in this function); did you mean 'S_CGROUPWB'? > sb->s_iflags |= SB_I_CGROUPWB; > ^~~~~~~~~~~~~ > S_CGROUPWB > > Caused by commit > > 578c647879f7 ("f2fs: implement cgroup writeback support") > > interacting with commit > > 34934189a3dd ("cgroup, writeback: replace SB_I_CGROUPWB with per-inode S_CGROUPWB" > > from the btrfs tree. > > I have reverted the f2fs tree commit for today, but a better resolution > would be appreciated. > ----------------------------------------------------------------------- > > I have seen no response to the above email and have been reverting that > commit since Jan 18. I think this can be addressed by: diff --git a/fs/f2fs/inode.c b/fs/f2fs/inode.c index 89c838bfb067..c743b73976d0 100644 --- a/fs/f2fs/inode.c +++ b/fs/f2fs/inode.c @@ -48,9 +48,10 @@ void f2fs_set_inode_flags(struct inode *inode) new_fl |= S_DIRSYNC; if (f2fs_encrypted_inode(inode)) new_fl |= S_ENCRYPTED; + new_fl |= S_CGROUPWB; inode_set_flags(inode, new_fl, S_SYNC|S_APPEND|S_IMMUTABLE|S_NOATIME|S_DIRSYNC| - S_ENCRYPTED); + S_ENCRYPTED|S_CGROUPWB); } static void __get_inode_rdev(struct inode *inode, struct f2fs_inode *ri) diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c index 8173ae688814..ebcd58476fc7 100644 --- a/fs/f2fs/super.c +++ b/fs/f2fs/super.c @@ -2560,7 +2560,6 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent) sb->s_flags = (sb->s_flags & ~SB_POSIXACL) | (test_opt(sbi, POSIX_ACL) ? SB_POSIXACL : 0); memcpy(&sb->s_uuid, raw_super->uuid, sizeof(raw_super->uuid)); - sb->s_iflags |= SB_I_CGROUPWB; /* init f2fs-specific super block info */ sbi->valid_super_block = valid_super_block; ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot ^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [GIT PULL] f2fs update for 4.16-rc1 2018-01-31 0:25 ` Jaegeuk Kim (?) @ 2018-01-31 22:25 ` Stephen Rothwell 2018-03-13 23:17 ` Stephen Rothwell -1 siblings, 1 reply; 19+ messages in thread From: Stephen Rothwell @ 2018-01-31 22:25 UTC (permalink / raw) To: Jaegeuk Kim Cc: Linus Torvalds, Linux F2FS Dev Mailing List, Linux Kernel Mailing List, Chris Mason, Josef Bacik, Yufen Yu, Tejun Heo Hi Jaegeuk, On Tue, 30 Jan 2018 16:25:35 -0800 Jaegeuk Kim <jaegeuk@kernel.org> wrote: > > On 01/31, Stephen Rothwell wrote: > > > > On Tue, 30 Jan 2018 14:32:20 -0800 Jaegeuk Kim <jaegeuk@kernel.org> wrote: > > > > > > Could you please consider the below pull request? > > > > > > Thanks, > > > > > > The following changes since commit ace52288edf0cb5e7a52b681f057f86224c49c27: > > > > > > Merge tag 'for-linus-20171218' of git://git.infradead.org/linux-mtd (2017-12-18 16:18:01 -0800) > > > > > > are available in the Git repository at: > > > > > > git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git tags/f2fs-for-4.16-rc1 > > > > > > for you to fetch changes up to 1c1d35df71104c76a4a2e25862926f22c334c9d6: > > > > > > f2fs: support inode creation time (2018-01-25 14:10:39 -0800) > > > > This has an interesting conflict with the btrfs tree: > > > > ----------------------------------------------------------------------- > > After merging the f2fs tree, today's linux-next build (x86_64 > > allmodconfig) failed like this: > > > > /home/sfr/next/next/fs/f2fs/super.c: In function 'f2fs_fill_super': > > /home/sfr/next/next/fs/f2fs/super.c:2563:18: error: 'SB_I_CGROUPWB' undeclared (first use in this function); did you mean 'S_CGROUPWB'? > > sb->s_iflags |= SB_I_CGROUPWB; > > ^~~~~~~~~~~~~ > > S_CGROUPWB > > > > Caused by commit > > > > 578c647879f7 ("f2fs: implement cgroup writeback support") > > > > interacting with commit > > > > 34934189a3dd ("cgroup, writeback: replace SB_I_CGROUPWB with per-inode S_CGROUPWB" > > > > from the btrfs tree. > > > > I have reverted the f2fs tree commit for today, but a better resolution > > would be appreciated. > > ----------------------------------------------------------------------- > > > > I have seen no response to the above email and have been reverting that > > commit since Jan 18. > > I think this can be addressed by: > > diff --git a/fs/f2fs/inode.c b/fs/f2fs/inode.c > index 89c838bfb067..c743b73976d0 100644 > --- a/fs/f2fs/inode.c > +++ b/fs/f2fs/inode.c > @@ -48,9 +48,10 @@ void f2fs_set_inode_flags(struct inode *inode) > new_fl |= S_DIRSYNC; > if (f2fs_encrypted_inode(inode)) > new_fl |= S_ENCRYPTED; > + new_fl |= S_CGROUPWB; > inode_set_flags(inode, new_fl, > S_SYNC|S_APPEND|S_IMMUTABLE|S_NOATIME|S_DIRSYNC| > - S_ENCRYPTED); > + S_ENCRYPTED|S_CGROUPWB); > } > > static void __get_inode_rdev(struct inode *inode, struct f2fs_inode *ri) > diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c > index 8173ae688814..ebcd58476fc7 100644 > --- a/fs/f2fs/super.c > +++ b/fs/f2fs/super.c > @@ -2560,7 +2560,6 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent) > sb->s_flags = (sb->s_flags & ~SB_POSIXACL) | > (test_opt(sbi, POSIX_ACL) ? SB_POSIXACL : 0); > memcpy(&sb->s_uuid, raw_super->uuid, sizeof(raw_super->uuid)); > - sb->s_iflags |= SB_I_CGROUPWB; > > /* init f2fs-specific super block info */ > sbi->valid_super_block = valid_super_block; > I have applied that (by hand) as a merge fix patch for the merge of teh btrfs tree for today. Thanks for the resolution. -- Cheers, Stephen Rothwell ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [GIT PULL] f2fs update for 4.16-rc1 2018-01-31 22:25 ` Stephen Rothwell @ 2018-03-13 23:17 ` Stephen Rothwell 2018-04-03 2:29 ` Stephen Rothwell 0 siblings, 1 reply; 19+ messages in thread From: Stephen Rothwell @ 2018-03-13 23:17 UTC (permalink / raw) To: Chris Mason, Josef Bacik Cc: Jaegeuk Kim, Linus Torvalds, Linux F2FS Dev Mailing List, Linux Kernel Mailing List, Yufen Yu, Tejun Heo, David Sterba [-- Attachment #1: Type: text/plain, Size: 5317 bytes --] Hi all, On Thu, 1 Feb 2018 09:25:51 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote: > > On Tue, 30 Jan 2018 16:25:35 -0800 Jaegeuk Kim <jaegeuk@kernel.org> wrote: > > > > On 01/31, Stephen Rothwell wrote: > > > > > > On Tue, 30 Jan 2018 14:32:20 -0800 Jaegeuk Kim <jaegeuk@kernel.org> wrote: > > > > > > > > Could you please consider the below pull request? > > > > > > > > Thanks, > > > > > > > > The following changes since commit ace52288edf0cb5e7a52b681f057f86224c49c27: > > > > > > > > Merge tag 'for-linus-20171218' of git://git.infradead.org/linux-mtd (2017-12-18 16:18:01 -0800) > > > > > > > > are available in the Git repository at: > > > > > > > > git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git tags/f2fs-for-4.16-rc1 > > > > > > > > for you to fetch changes up to 1c1d35df71104c76a4a2e25862926f22c334c9d6: > > > > > > > > f2fs: support inode creation time (2018-01-25 14:10:39 -0800) > > > > > > This has an interesting conflict with the btrfs tree: > > > > > > ----------------------------------------------------------------------- > > > After merging the f2fs tree, today's linux-next build (x86_64 > > > allmodconfig) failed like this: > > > > > > /home/sfr/next/next/fs/f2fs/super.c: In function 'f2fs_fill_super': > > > /home/sfr/next/next/fs/f2fs/super.c:2563:18: error: 'SB_I_CGROUPWB' undeclared (first use in this function); did you mean 'S_CGROUPWB'? > > > sb->s_iflags |= SB_I_CGROUPWB; > > > ^~~~~~~~~~~~~ > > > S_CGROUPWB > > > > > > Caused by commit > > > > > > 578c647879f7 ("f2fs: implement cgroup writeback support") > > > > > > interacting with commit > > > > > > 34934189a3dd ("cgroup, writeback: replace SB_I_CGROUPWB with per-inode S_CGROUPWB" > > > > > > from the btrfs tree. > > > > > > I have reverted the f2fs tree commit for today, but a better resolution > > > would be appreciated. > > > ----------------------------------------------------------------------- > > > > > > I have seen no response to the above email and have been reverting that > > > commit since Jan 18. > > > > I think this can be addressed by: > > > > diff --git a/fs/f2fs/inode.c b/fs/f2fs/inode.c > > index 89c838bfb067..c743b73976d0 100644 > > --- a/fs/f2fs/inode.c > > +++ b/fs/f2fs/inode.c > > @@ -48,9 +48,10 @@ void f2fs_set_inode_flags(struct inode *inode) > > new_fl |= S_DIRSYNC; > > if (f2fs_encrypted_inode(inode)) > > new_fl |= S_ENCRYPTED; > > + new_fl |= S_CGROUPWB; > > inode_set_flags(inode, new_fl, > > S_SYNC|S_APPEND|S_IMMUTABLE|S_NOATIME|S_DIRSYNC| > > - S_ENCRYPTED); > > + S_ENCRYPTED|S_CGROUPWB); > > } > > > > static void __get_inode_rdev(struct inode *inode, struct f2fs_inode *ri) > > diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c > > index 8173ae688814..ebcd58476fc7 100644 > > --- a/fs/f2fs/super.c > > +++ b/fs/f2fs/super.c > > @@ -2560,7 +2560,6 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent) > > sb->s_flags = (sb->s_flags & ~SB_POSIXACL) | > > (test_opt(sbi, POSIX_ACL) ? SB_POSIXACL : 0); > > memcpy(&sb->s_uuid, raw_super->uuid, sizeof(raw_super->uuid)); > > - sb->s_iflags |= SB_I_CGROUPWB; > > > > /* init f2fs-specific super block info */ > > sbi->valid_super_block = valid_super_block; > > > > I have applied that (by hand) as a merge fix patch for the merge of teh > btrfs tree for today. Thanks for the resolution. I am still applying this to the merge of the btrfs tree every day ... Commit 578c647879f7 ("f2fs: implement cgroup writeback support") was merged into Linus' tree on Jan 31. Here is the actual merge fix patch: From: Stephen Rothwell <sfr@canb.auug.org.au> Date: Thu, 1 Feb 2018 09:21:06 +1100 Subject: [PATCH] f2fs: fixup for cgroup/writeback change Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au> --- fs/f2fs/inode.c | 3 ++- fs/f2fs/super.c | 1 - 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/f2fs/inode.c b/fs/f2fs/inode.c index 205add3d0f3a..f34652c4288f 100644 --- a/fs/f2fs/inode.c +++ b/fs/f2fs/inode.c @@ -48,9 +48,10 @@ void f2fs_set_inode_flags(struct inode *inode) new_fl |= S_DIRSYNC; if (f2fs_encrypted_inode(inode)) new_fl |= S_ENCRYPTED; + new_fl |= S_CGROUPWB; inode_set_flags(inode, new_fl, S_SYNC|S_APPEND|S_IMMUTABLE|S_NOATIME|S_DIRSYNC| - S_ENCRYPTED); + S_ENCRYPTED|S_CGROUPWB); } static void __get_inode_rdev(struct inode *inode, struct f2fs_inode *ri) diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c index 8173ae688814..ebcd58476fc7 100644 --- a/fs/f2fs/super.c +++ b/fs/f2fs/super.c @@ -2560,7 +2560,6 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent) sb->s_flags = (sb->s_flags & ~SB_POSIXACL) | (test_opt(sbi, POSIX_ACL) ? SB_POSIXACL : 0); memcpy(&sb->s_uuid, raw_super->uuid, sizeof(raw_super->uuid)); - sb->s_iflags |= SB_I_CGROUPWB; /* init f2fs-specific super block info */ sbi->valid_super_block = valid_super_block; -- 2.15.1 -- Cheers, Stephen Rothwell [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [GIT PULL] f2fs update for 4.16-rc1 2018-03-13 23:17 ` Stephen Rothwell @ 2018-04-03 2:29 ` Stephen Rothwell 2018-04-03 14:20 ` Tejun Heo ` (2 more replies) 0 siblings, 3 replies; 19+ messages in thread From: Stephen Rothwell @ 2018-04-03 2:29 UTC (permalink / raw) To: Chris Mason, Josef Bacik Cc: Jaegeuk Kim, Linus Torvalds, Linux F2FS Dev Mailing List, Linux Kernel Mailing List, Yufen Yu, Tejun Heo, David Sterba [-- Attachment #1: Type: text/plain, Size: 5698 bytes --] Hi all, Ping? On Wed, 14 Mar 2018 10:17:04 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote: > > On Thu, 1 Feb 2018 09:25:51 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote: > > > > On Tue, 30 Jan 2018 16:25:35 -0800 Jaegeuk Kim <jaegeuk@kernel.org> wrote: > > > > > > On 01/31, Stephen Rothwell wrote: > > > > > > > > On Tue, 30 Jan 2018 14:32:20 -0800 Jaegeuk Kim <jaegeuk@kernel.org> wrote: > > > > > > > > > > Could you please consider the below pull request? > > > > > > > > > > Thanks, > > > > > > > > > > The following changes since commit ace52288edf0cb5e7a52b681f057f86224c49c27: > > > > > > > > > > Merge tag 'for-linus-20171218' of git://git.infradead.org/linux-mtd (2017-12-18 16:18:01 -0800) > > > > > > > > > > are available in the Git repository at: > > > > > > > > > > git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git tags/f2fs-for-4.16-rc1 > > > > > > > > > > for you to fetch changes up to 1c1d35df71104c76a4a2e25862926f22c334c9d6: > > > > > > > > > > f2fs: support inode creation time (2018-01-25 14:10:39 -0800) > > > > > > > > This has an interesting conflict with the btrfs tree: > > > > > > > > ----------------------------------------------------------------------- > > > > After merging the f2fs tree, today's linux-next build (x86_64 > > > > allmodconfig) failed like this: > > > > > > > > /home/sfr/next/next/fs/f2fs/super.c: In function 'f2fs_fill_super': > > > > /home/sfr/next/next/fs/f2fs/super.c:2563:18: error: 'SB_I_CGROUPWB' undeclared (first use in this function); did you mean 'S_CGROUPWB'? > > > > sb->s_iflags |= SB_I_CGROUPWB; > > > > ^~~~~~~~~~~~~ > > > > S_CGROUPWB > > > > > > > > Caused by commit > > > > > > > > 578c647879f7 ("f2fs: implement cgroup writeback support") > > > > > > > > interacting with commit > > > > > > > > 34934189a3dd ("cgroup, writeback: replace SB_I_CGROUPWB with per-inode S_CGROUPWB" > > > > > > > > from the btrfs tree. > > > > > > > > I have reverted the f2fs tree commit for today, but a better resolution > > > > would be appreciated. > > > > ----------------------------------------------------------------------- > > > > > > > > I have seen no response to the above email and have been reverting that > > > > commit since Jan 18. > > > > > > I think this can be addressed by: > > > > > > diff --git a/fs/f2fs/inode.c b/fs/f2fs/inode.c > > > index 89c838bfb067..c743b73976d0 100644 > > > --- a/fs/f2fs/inode.c > > > +++ b/fs/f2fs/inode.c > > > @@ -48,9 +48,10 @@ void f2fs_set_inode_flags(struct inode *inode) > > > new_fl |= S_DIRSYNC; > > > if (f2fs_encrypted_inode(inode)) > > > new_fl |= S_ENCRYPTED; > > > + new_fl |= S_CGROUPWB; > > > inode_set_flags(inode, new_fl, > > > S_SYNC|S_APPEND|S_IMMUTABLE|S_NOATIME|S_DIRSYNC| > > > - S_ENCRYPTED); > > > + S_ENCRYPTED|S_CGROUPWB); > > > } > > > > > > static void __get_inode_rdev(struct inode *inode, struct f2fs_inode *ri) > > > diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c > > > index 8173ae688814..ebcd58476fc7 100644 > > > --- a/fs/f2fs/super.c > > > +++ b/fs/f2fs/super.c > > > @@ -2560,7 +2560,6 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent) > > > sb->s_flags = (sb->s_flags & ~SB_POSIXACL) | > > > (test_opt(sbi, POSIX_ACL) ? SB_POSIXACL : 0); > > > memcpy(&sb->s_uuid, raw_super->uuid, sizeof(raw_super->uuid)); > > > - sb->s_iflags |= SB_I_CGROUPWB; > > > > > > /* init f2fs-specific super block info */ > > > sbi->valid_super_block = valid_super_block; > > > > > > > I have applied that (by hand) as a merge fix patch for the merge of teh > > btrfs tree for today. Thanks for the resolution. > > I am still applying this to the merge of the btrfs tree every day ... > > Commit > 578c647879f7 ("f2fs: implement cgroup writeback support") > was merged into Linus' tree on Jan 31. > > Here is the actual merge fix patch: > > From: Stephen Rothwell <sfr@canb.auug.org.au> > Date: Thu, 1 Feb 2018 09:21:06 +1100 > Subject: [PATCH] f2fs: fixup for cgroup/writeback change > > Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au> > --- > fs/f2fs/inode.c | 3 ++- > fs/f2fs/super.c | 1 - > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/fs/f2fs/inode.c b/fs/f2fs/inode.c > index 205add3d0f3a..f34652c4288f 100644 > --- a/fs/f2fs/inode.c > +++ b/fs/f2fs/inode.c > @@ -48,9 +48,10 @@ void f2fs_set_inode_flags(struct inode *inode) > new_fl |= S_DIRSYNC; > if (f2fs_encrypted_inode(inode)) > new_fl |= S_ENCRYPTED; > + new_fl |= S_CGROUPWB; > inode_set_flags(inode, new_fl, > S_SYNC|S_APPEND|S_IMMUTABLE|S_NOATIME|S_DIRSYNC| > - S_ENCRYPTED); > + S_ENCRYPTED|S_CGROUPWB); > } > > static void __get_inode_rdev(struct inode *inode, struct f2fs_inode *ri) > diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c > index 8173ae688814..ebcd58476fc7 100644 > --- a/fs/f2fs/super.c > +++ b/fs/f2fs/super.c > @@ -2560,7 +2560,6 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent) > sb->s_flags = (sb->s_flags & ~SB_POSIXACL) | > (test_opt(sbi, POSIX_ACL) ? SB_POSIXACL : 0); > memcpy(&sb->s_uuid, raw_super->uuid, sizeof(raw_super->uuid)); > - sb->s_iflags |= SB_I_CGROUPWB; > > /* init f2fs-specific super block info */ > sbi->valid_super_block = valid_super_block; > -- > 2.15.1 -- Cheers, Stephen Rothwell [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [GIT PULL] f2fs update for 4.16-rc1 2018-04-03 2:29 ` Stephen Rothwell @ 2018-04-03 14:20 ` Tejun Heo 2018-05-02 22:42 ` Stephen Rothwell 2018-06-18 3:38 ` Stephen Rothwell 2 siblings, 0 replies; 19+ messages in thread From: Tejun Heo @ 2018-04-03 14:20 UTC (permalink / raw) To: Stephen Rothwell Cc: Chris Mason, Josef Bacik, Jaegeuk Kim, Linus Torvalds, Linux F2FS Dev Mailing List, Linux Kernel Mailing List, Yufen Yu, David Sterba Hello, Stephen. On Tue, Apr 03, 2018 at 12:29:19PM +1000, Stephen Rothwell wrote: > > I am still applying this to the merge of the btrfs tree every day ... > > > > Commit > > 578c647879f7 ("f2fs: implement cgroup writeback support") > > was merged into Linus' tree on Jan 31. > > > > Here is the actual merge fix patch: Yeah, the merge fix looks correct to me. Depending on the merge order, it can go either after the btrfs merge or the f2fs merge, I guess. Stephen, beyond appyling the merge fix afterwards, is there somethine else you are concerned about? Thanks. -- tejun ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [GIT PULL] f2fs update for 4.16-rc1 @ 2018-04-03 14:20 ` Tejun Heo 0 siblings, 0 replies; 19+ messages in thread From: Tejun Heo @ 2018-04-03 14:20 UTC (permalink / raw) To: Stephen Rothwell Cc: Yufen Yu, Josef Bacik, David Sterba, Linux Kernel Mailing List, Linux F2FS Dev Mailing List, Chris Mason, Jaegeuk Kim, Linus Torvalds Hello, Stephen. On Tue, Apr 03, 2018 at 12:29:19PM +1000, Stephen Rothwell wrote: > > I am still applying this to the merge of the btrfs tree every day ... > > > > Commit > > 578c647879f7 ("f2fs: implement cgroup writeback support") > > was merged into Linus' tree on Jan 31. > > > > Here is the actual merge fix patch: Yeah, the merge fix looks correct to me. Depending on the merge order, it can go either after the btrfs merge or the f2fs merge, I guess. Stephen, beyond appyling the merge fix afterwards, is there somethine else you are concerned about? Thanks. -- tejun ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [GIT PULL] f2fs update for 4.16-rc1 2018-04-03 14:20 ` Tejun Heo (?) @ 2018-04-03 14:45 ` Stephen Rothwell -1 siblings, 0 replies; 19+ messages in thread From: Stephen Rothwell @ 2018-04-03 14:45 UTC (permalink / raw) To: Tejun Heo Cc: Chris Mason, Josef Bacik, Jaegeuk Kim, Linus Torvalds, Linux F2FS Dev Mailing List, Linux Kernel Mailing List, Yufen Yu, David Sterba [-- Attachment #1: Type: text/plain, Size: 1190 bytes --] Hi Tejun, On Tue, 3 Apr 2018 07:20:29 -0700 Tejun Heo <tj@kernel.org> wrote: > > Hello, Stephen. > > On Tue, Apr 03, 2018 at 12:29:19PM +1000, Stephen Rothwell wrote: > > > I am still applying this to the merge of the btrfs tree every day ... > > > > > > Commit > > > 578c647879f7 ("f2fs: implement cgroup writeback support") > > > was merged into Linus' tree on Jan 31. > > > > > > Here is the actual merge fix patch: > > Yeah, the merge fix looks correct to me. Depending on the merge > order, it can go either after the btrfs merge or the f2fs merge, I > guess. Stephen, beyond appyling the merge fix afterwards, is there > somethine else you are concerned about? The f2fs part of the conflict was merged into Linus' tree on Jan 31, and I have been applying the conflict resolution patch for a whole release cycle i.e. the btrfs tree part of the conflict did not get merged during the last merge window, and has not been updated since then - in fact the btrfs tree (git://git.kernel.org/pub/scm/linux/kernel/git/mason/linux-btrfs.git#next) has not been updated or merged into Linus' tree since last December. :-( -- Cheers, Stephen Rothwell [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [GIT PULL] f2fs update for 4.16-rc1 2018-04-03 2:29 ` Stephen Rothwell 2018-04-03 14:20 ` Tejun Heo @ 2018-05-02 22:42 ` Stephen Rothwell 2018-05-03 9:49 ` David Sterba 2018-06-18 3:38 ` Stephen Rothwell 2 siblings, 1 reply; 19+ messages in thread From: Stephen Rothwell @ 2018-05-02 22:42 UTC (permalink / raw) To: Chris Mason, Josef Bacik, David Sterba Cc: Jaegeuk Kim, Linux Kernel Mailing List, Yufen Yu, Tejun Heo [-- Attachment #1: Type: text/plain, Size: 6184 bytes --] Hi all, On Tue, 3 Apr 2018 12:29:19 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote: > > Ping? Ping again ... So has Chris gone missing? Or should I just remove the btrfs tree from linux-next? > On Wed, 14 Mar 2018 10:17:04 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote: > > > > On Thu, 1 Feb 2018 09:25:51 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote: > > > > > > On Tue, 30 Jan 2018 16:25:35 -0800 Jaegeuk Kim <jaegeuk@kernel.org> wrote: > > > > > > > > On 01/31, Stephen Rothwell wrote: > > > > > > > > > > On Tue, 30 Jan 2018 14:32:20 -0800 Jaegeuk Kim <jaegeuk@kernel.org> wrote: > > > > > > > > > > > > Could you please consider the below pull request? > > > > > > > > > > > > Thanks, > > > > > > > > > > > > The following changes since commit ace52288edf0cb5e7a52b681f057f86224c49c27: > > > > > > > > > > > > Merge tag 'for-linus-20171218' of git://git.infradead.org/linux-mtd (2017-12-18 16:18:01 -0800) > > > > > > > > > > > > are available in the Git repository at: > > > > > > > > > > > > git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git tags/f2fs-for-4.16-rc1 > > > > > > > > > > > > for you to fetch changes up to 1c1d35df71104c76a4a2e25862926f22c334c9d6: > > > > > > > > > > > > f2fs: support inode creation time (2018-01-25 14:10:39 -0800) > > > > > > > > > > This has an interesting conflict with the btrfs tree: > > > > > > > > > > ----------------------------------------------------------------------- > > > > > After merging the f2fs tree, today's linux-next build (x86_64 > > > > > allmodconfig) failed like this: > > > > > > > > > > /home/sfr/next/next/fs/f2fs/super.c: In function 'f2fs_fill_super': > > > > > /home/sfr/next/next/fs/f2fs/super.c:2563:18: error: 'SB_I_CGROUPWB' undeclared (first use in this function); did you mean 'S_CGROUPWB'? > > > > > sb->s_iflags |= SB_I_CGROUPWB; > > > > > ^~~~~~~~~~~~~ > > > > > S_CGROUPWB > > > > > > > > > > Caused by commit > > > > > > > > > > 578c647879f7 ("f2fs: implement cgroup writeback support") > > > > > > > > > > interacting with commit > > > > > > > > > > 34934189a3dd ("cgroup, writeback: replace SB_I_CGROUPWB with per-inode S_CGROUPWB" > > > > > > > > > > from the btrfs tree. > > > > > > > > > > I have reverted the f2fs tree commit for today, but a better resolution > > > > > would be appreciated. > > > > > ----------------------------------------------------------------------- > > > > > > > > > > I have seen no response to the above email and have been reverting that > > > > > commit since Jan 18. > > > > > > > > I think this can be addressed by: > > > > > > > > diff --git a/fs/f2fs/inode.c b/fs/f2fs/inode.c > > > > index 89c838bfb067..c743b73976d0 100644 > > > > --- a/fs/f2fs/inode.c > > > > +++ b/fs/f2fs/inode.c > > > > @@ -48,9 +48,10 @@ void f2fs_set_inode_flags(struct inode *inode) > > > > new_fl |= S_DIRSYNC; > > > > if (f2fs_encrypted_inode(inode)) > > > > new_fl |= S_ENCRYPTED; > > > > + new_fl |= S_CGROUPWB; > > > > inode_set_flags(inode, new_fl, > > > > S_SYNC|S_APPEND|S_IMMUTABLE|S_NOATIME|S_DIRSYNC| > > > > - S_ENCRYPTED); > > > > + S_ENCRYPTED|S_CGROUPWB); > > > > } > > > > > > > > static void __get_inode_rdev(struct inode *inode, struct f2fs_inode *ri) > > > > diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c > > > > index 8173ae688814..ebcd58476fc7 100644 > > > > --- a/fs/f2fs/super.c > > > > +++ b/fs/f2fs/super.c > > > > @@ -2560,7 +2560,6 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent) > > > > sb->s_flags = (sb->s_flags & ~SB_POSIXACL) | > > > > (test_opt(sbi, POSIX_ACL) ? SB_POSIXACL : 0); > > > > memcpy(&sb->s_uuid, raw_super->uuid, sizeof(raw_super->uuid)); > > > > - sb->s_iflags |= SB_I_CGROUPWB; > > > > > > > > /* init f2fs-specific super block info */ > > > > sbi->valid_super_block = valid_super_block; > > > > > > > > > > I have applied that (by hand) as a merge fix patch for the merge of teh > > > btrfs tree for today. Thanks for the resolution. > > > > I am still applying this to the merge of the btrfs tree every day ... > > > > Commit > > 578c647879f7 ("f2fs: implement cgroup writeback support") > > was merged into Linus' tree on Jan 31. > > > > Here is the actual merge fix patch: > > > > From: Stephen Rothwell <sfr@canb.auug.org.au> > > Date: Thu, 1 Feb 2018 09:21:06 +1100 > > Subject: [PATCH] f2fs: fixup for cgroup/writeback change > > > > Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au> > > --- > > fs/f2fs/inode.c | 3 ++- > > fs/f2fs/super.c | 1 - > > 2 files changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/fs/f2fs/inode.c b/fs/f2fs/inode.c > > index 205add3d0f3a..f34652c4288f 100644 > > --- a/fs/f2fs/inode.c > > +++ b/fs/f2fs/inode.c > > @@ -48,9 +48,10 @@ void f2fs_set_inode_flags(struct inode *inode) > > new_fl |= S_DIRSYNC; > > if (f2fs_encrypted_inode(inode)) > > new_fl |= S_ENCRYPTED; > > + new_fl |= S_CGROUPWB; > > inode_set_flags(inode, new_fl, > > S_SYNC|S_APPEND|S_IMMUTABLE|S_NOATIME|S_DIRSYNC| > > - S_ENCRYPTED); > > + S_ENCRYPTED|S_CGROUPWB); > > } > > > > static void __get_inode_rdev(struct inode *inode, struct f2fs_inode *ri) > > diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c > > index 8173ae688814..ebcd58476fc7 100644 > > --- a/fs/f2fs/super.c > > +++ b/fs/f2fs/super.c > > @@ -2560,7 +2560,6 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent) > > sb->s_flags = (sb->s_flags & ~SB_POSIXACL) | > > (test_opt(sbi, POSIX_ACL) ? SB_POSIXACL : 0); > > memcpy(&sb->s_uuid, raw_super->uuid, sizeof(raw_super->uuid)); > > - sb->s_iflags |= SB_I_CGROUPWB; > > > > /* init f2fs-specific super block info */ > > sbi->valid_super_block = valid_super_block; > > -- > > 2.15.1 -- Cheers, Stephen Rothwell [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [GIT PULL] f2fs update for 4.16-rc1 2018-05-02 22:42 ` Stephen Rothwell @ 2018-05-03 9:49 ` David Sterba 0 siblings, 0 replies; 19+ messages in thread From: David Sterba @ 2018-05-03 9:49 UTC (permalink / raw) To: Stephen Rothwell Cc: Chris Mason, Josef Bacik, Jaegeuk Kim, Linux Kernel Mailing List, Yufen Yu, Tejun Heo On Thu, May 03, 2018 at 08:42:59AM +1000, Stephen Rothwell wrote: > Hi all, > > On Tue, 3 Apr 2018 12:29:19 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote: > > > > Ping? > > Ping again ... > > So has Chris gone missing? Or should I just remove the btrfs tree from linux-next? As far as btrfs code changes are concerned, they go through my tree. The other conflicting changes are some cgroup tweaks that have impact on btrfs but do not affect the development. ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [GIT PULL] f2fs update for 4.16-rc1 2018-04-03 2:29 ` Stephen Rothwell @ 2018-06-18 3:38 ` Stephen Rothwell 2018-05-02 22:42 ` Stephen Rothwell 2018-06-18 3:38 ` Stephen Rothwell 2 siblings, 0 replies; 19+ messages in thread From: Stephen Rothwell @ 2018-06-18 3:38 UTC (permalink / raw) To: Chris Mason, Josef Bacik Cc: Jaegeuk Kim, Linus Torvalds, Linux F2FS Dev Mailing List, Linux Kernel Mailing List, Yufen Yu, Tejun Heo, David Sterba [-- Attachment #1: Type: text/plain, Size: 6259 bytes --] Hi Chris, On Tue, 3 Apr 2018 12:29:19 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote: > > Ping? > > On Wed, 14 Mar 2018 10:17:04 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote: > > > > On Thu, 1 Feb 2018 09:25:51 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote: > > > > > > On Tue, 30 Jan 2018 16:25:35 -0800 Jaegeuk Kim <jaegeuk@kernel.org> wrote: > > > > > > > > On 01/31, Stephen Rothwell wrote: > > > > > > > > > > On Tue, 30 Jan 2018 14:32:20 -0800 Jaegeuk Kim <jaegeuk@kernel.org> wrote: > > > > > > > > > > > > Could you please consider the below pull request? > > > > > > > > > > > > Thanks, > > > > > > > > > > > > The following changes since commit ace52288edf0cb5e7a52b681f057f86224c49c27: > > > > > > > > > > > > Merge tag 'for-linus-20171218' of git://git.infradead.org/linux-mtd (2017-12-18 16:18:01 -0800) > > > > > > > > > > > > are available in the Git repository at: > > > > > > > > > > > > git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git tags/f2fs-for-4.16-rc1 > > > > > > > > > > > > for you to fetch changes up to 1c1d35df71104c76a4a2e25862926f22c334c9d6: > > > > > > > > > > > > f2fs: support inode creation time (2018-01-25 14:10:39 -0800) > > > > > > > > > > This has an interesting conflict with the btrfs tree: > > > > > > > > > > ----------------------------------------------------------------------- > > > > > After merging the f2fs tree, today's linux-next build (x86_64 > > > > > allmodconfig) failed like this: > > > > > > > > > > /home/sfr/next/next/fs/f2fs/super.c: In function 'f2fs_fill_super': > > > > > /home/sfr/next/next/fs/f2fs/super.c:2563:18: error: 'SB_I_CGROUPWB' undeclared (first use in this function); did you mean 'S_CGROUPWB'? > > > > > sb->s_iflags |= SB_I_CGROUPWB; > > > > > ^~~~~~~~~~~~~ > > > > > S_CGROUPWB > > > > > > > > > > Caused by commit > > > > > > > > > > 578c647879f7 ("f2fs: implement cgroup writeback support") > > > > > > > > > > interacting with commit > > > > > > > > > > 34934189a3dd ("cgroup, writeback: replace SB_I_CGROUPWB with per-inode S_CGROUPWB" > > > > > > > > > > from the btrfs tree. > > > > > > > > > > I have reverted the f2fs tree commit for today, but a better resolution > > > > > would be appreciated. > > > > > ----------------------------------------------------------------------- > > > > > > > > > > I have seen no response to the above email and have been reverting that > > > > > commit since Jan 18. > > > > > > > > I think this can be addressed by: > > > > > > > > diff --git a/fs/f2fs/inode.c b/fs/f2fs/inode.c > > > > index 89c838bfb067..c743b73976d0 100644 > > > > --- a/fs/f2fs/inode.c > > > > +++ b/fs/f2fs/inode.c > > > > @@ -48,9 +48,10 @@ void f2fs_set_inode_flags(struct inode *inode) > > > > new_fl |= S_DIRSYNC; > > > > if (f2fs_encrypted_inode(inode)) > > > > new_fl |= S_ENCRYPTED; > > > > + new_fl |= S_CGROUPWB; > > > > inode_set_flags(inode, new_fl, > > > > S_SYNC|S_APPEND|S_IMMUTABLE|S_NOATIME|S_DIRSYNC| > > > > - S_ENCRYPTED); > > > > + S_ENCRYPTED|S_CGROUPWB); > > > > } > > > > > > > > static void __get_inode_rdev(struct inode *inode, struct f2fs_inode *ri) > > > > diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c > > > > index 8173ae688814..ebcd58476fc7 100644 > > > > --- a/fs/f2fs/super.c > > > > +++ b/fs/f2fs/super.c > > > > @@ -2560,7 +2560,6 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent) > > > > sb->s_flags = (sb->s_flags & ~SB_POSIXACL) | > > > > (test_opt(sbi, POSIX_ACL) ? SB_POSIXACL : 0); > > > > memcpy(&sb->s_uuid, raw_super->uuid, sizeof(raw_super->uuid)); > > > > - sb->s_iflags |= SB_I_CGROUPWB; > > > > > > > > /* init f2fs-specific super block info */ > > > > sbi->valid_super_block = valid_super_block; > > > > > > > > > > I have applied that (by hand) as a merge fix patch for the merge of teh > > > btrfs tree for today. Thanks for the resolution. > > > > I am still applying this to the merge of the btrfs tree every day ... > > > > Commit > > 578c647879f7 ("f2fs: implement cgroup writeback support") > > was merged into Linus' tree on Jan 31. > > > > Here is the actual merge fix patch: > > > > From: Stephen Rothwell <sfr@canb.auug.org.au> > > Date: Thu, 1 Feb 2018 09:21:06 +1100 > > Subject: [PATCH] f2fs: fixup for cgroup/writeback change > > > > Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au> > > --- > > fs/f2fs/inode.c | 3 ++- > > fs/f2fs/super.c | 1 - > > 2 files changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/fs/f2fs/inode.c b/fs/f2fs/inode.c > > index 205add3d0f3a..f34652c4288f 100644 > > --- a/fs/f2fs/inode.c > > +++ b/fs/f2fs/inode.c > > @@ -48,9 +48,10 @@ void f2fs_set_inode_flags(struct inode *inode) > > new_fl |= S_DIRSYNC; > > if (f2fs_encrypted_inode(inode)) > > new_fl |= S_ENCRYPTED; > > + new_fl |= S_CGROUPWB; > > inode_set_flags(inode, new_fl, > > S_SYNC|S_APPEND|S_IMMUTABLE|S_NOATIME|S_DIRSYNC| > > - S_ENCRYPTED); > > + S_ENCRYPTED|S_CGROUPWB); > > } > > > > static void __get_inode_rdev(struct inode *inode, struct f2fs_inode *ri) > > diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c > > index 8173ae688814..ebcd58476fc7 100644 > > --- a/fs/f2fs/super.c > > +++ b/fs/f2fs/super.c > > @@ -2560,7 +2560,6 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent) > > sb->s_flags = (sb->s_flags & ~SB_POSIXACL) | > > (test_opt(sbi, POSIX_ACL) ? SB_POSIXACL : 0); > > memcpy(&sb->s_uuid, raw_super->uuid, sizeof(raw_super->uuid)); > > - sb->s_iflags |= SB_I_CGROUPWB; > > > > /* init f2fs-specific super block info */ > > sbi->valid_super_block = valid_super_block; > > -- > > 2.15.1 Another merge window has passed and I am still holding this tree and its merge fixes ... Tomorrow I will drop the btrfs tree and rename the btrfs-kdave tree to btrfs. OK? -- Cheers, Stephen Rothwell [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [GIT PULL] f2fs update for 4.16-rc1 @ 2018-06-18 3:38 ` Stephen Rothwell 0 siblings, 0 replies; 19+ messages in thread From: Stephen Rothwell @ 2018-06-18 3:38 UTC (permalink / raw) To: Chris Mason, Josef Bacik Cc: Jaegeuk Kim, Linus Torvalds, Linux F2FS Dev Mailing List, Linux Kernel Mailing List, Yufen Yu, Tejun Heo, David Sterba [-- Attachment #1: Type: text/plain, Size: 6259 bytes --] Hi Chris, On Tue, 3 Apr 2018 12:29:19 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote: > > Ping? > > On Wed, 14 Mar 2018 10:17:04 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote: > > > > On Thu, 1 Feb 2018 09:25:51 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote: > > > > > > On Tue, 30 Jan 2018 16:25:35 -0800 Jaegeuk Kim <jaegeuk@kernel.org> wrote: > > > > > > > > On 01/31, Stephen Rothwell wrote: > > > > > > > > > > On Tue, 30 Jan 2018 14:32:20 -0800 Jaegeuk Kim <jaegeuk@kernel.org> wrote: > > > > > > > > > > > > Could you please consider the below pull request? > > > > > > > > > > > > Thanks, > > > > > > > > > > > > The following changes since commit ace52288edf0cb5e7a52b681f057f86224c49c27: > > > > > > > > > > > > Merge tag 'for-linus-20171218' of git://git.infradead.org/linux-mtd (2017-12-18 16:18:01 -0800) > > > > > > > > > > > > are available in the Git repository at: > > > > > > > > > > > > git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git tags/f2fs-for-4.16-rc1 > > > > > > > > > > > > for you to fetch changes up to 1c1d35df71104c76a4a2e25862926f22c334c9d6: > > > > > > > > > > > > f2fs: support inode creation time (2018-01-25 14:10:39 -0800) > > > > > > > > > > This has an interesting conflict with the btrfs tree: > > > > > > > > > > ----------------------------------------------------------------------- > > > > > After merging the f2fs tree, today's linux-next build (x86_64 > > > > > allmodconfig) failed like this: > > > > > > > > > > /home/sfr/next/next/fs/f2fs/super.c: In function 'f2fs_fill_super': > > > > > /home/sfr/next/next/fs/f2fs/super.c:2563:18: error: 'SB_I_CGROUPWB' undeclared (first use in this function); did you mean 'S_CGROUPWB'? > > > > > sb->s_iflags |= SB_I_CGROUPWB; > > > > > ^~~~~~~~~~~~~ > > > > > S_CGROUPWB > > > > > > > > > > Caused by commit > > > > > > > > > > 578c647879f7 ("f2fs: implement cgroup writeback support") > > > > > > > > > > interacting with commit > > > > > > > > > > 34934189a3dd ("cgroup, writeback: replace SB_I_CGROUPWB with per-inode S_CGROUPWB" > > > > > > > > > > from the btrfs tree. > > > > > > > > > > I have reverted the f2fs tree commit for today, but a better resolution > > > > > would be appreciated. > > > > > ----------------------------------------------------------------------- > > > > > > > > > > I have seen no response to the above email and have been reverting that > > > > > commit since Jan 18. > > > > > > > > I think this can be addressed by: > > > > > > > > diff --git a/fs/f2fs/inode.c b/fs/f2fs/inode.c > > > > index 89c838bfb067..c743b73976d0 100644 > > > > --- a/fs/f2fs/inode.c > > > > +++ b/fs/f2fs/inode.c > > > > @@ -48,9 +48,10 @@ void f2fs_set_inode_flags(struct inode *inode) > > > > new_fl |= S_DIRSYNC; > > > > if (f2fs_encrypted_inode(inode)) > > > > new_fl |= S_ENCRYPTED; > > > > + new_fl |= S_CGROUPWB; > > > > inode_set_flags(inode, new_fl, > > > > S_SYNC|S_APPEND|S_IMMUTABLE|S_NOATIME|S_DIRSYNC| > > > > - S_ENCRYPTED); > > > > + S_ENCRYPTED|S_CGROUPWB); > > > > } > > > > > > > > static void __get_inode_rdev(struct inode *inode, struct f2fs_inode *ri) > > > > diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c > > > > index 8173ae688814..ebcd58476fc7 100644 > > > > --- a/fs/f2fs/super.c > > > > +++ b/fs/f2fs/super.c > > > > @@ -2560,7 +2560,6 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent) > > > > sb->s_flags = (sb->s_flags & ~SB_POSIXACL) | > > > > (test_opt(sbi, POSIX_ACL) ? SB_POSIXACL : 0); > > > > memcpy(&sb->s_uuid, raw_super->uuid, sizeof(raw_super->uuid)); > > > > - sb->s_iflags |= SB_I_CGROUPWB; > > > > > > > > /* init f2fs-specific super block info */ > > > > sbi->valid_super_block = valid_super_block; > > > > > > > > > > I have applied that (by hand) as a merge fix patch for the merge of teh > > > btrfs tree for today. Thanks for the resolution. > > > > I am still applying this to the merge of the btrfs tree every day ... > > > > Commit > > 578c647879f7 ("f2fs: implement cgroup writeback support") > > was merged into Linus' tree on Jan 31. > > > > Here is the actual merge fix patch: > > > > From: Stephen Rothwell <sfr@canb.auug.org.au> > > Date: Thu, 1 Feb 2018 09:21:06 +1100 > > Subject: [PATCH] f2fs: fixup for cgroup/writeback change > > > > Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au> > > --- > > fs/f2fs/inode.c | 3 ++- > > fs/f2fs/super.c | 1 - > > 2 files changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/fs/f2fs/inode.c b/fs/f2fs/inode.c > > index 205add3d0f3a..f34652c4288f 100644 > > --- a/fs/f2fs/inode.c > > +++ b/fs/f2fs/inode.c > > @@ -48,9 +48,10 @@ void f2fs_set_inode_flags(struct inode *inode) > > new_fl |= S_DIRSYNC; > > if (f2fs_encrypted_inode(inode)) > > new_fl |= S_ENCRYPTED; > > + new_fl |= S_CGROUPWB; > > inode_set_flags(inode, new_fl, > > S_SYNC|S_APPEND|S_IMMUTABLE|S_NOATIME|S_DIRSYNC| > > - S_ENCRYPTED); > > + S_ENCRYPTED|S_CGROUPWB); > > } > > > > static void __get_inode_rdev(struct inode *inode, struct f2fs_inode *ri) > > diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c > > index 8173ae688814..ebcd58476fc7 100644 > > --- a/fs/f2fs/super.c > > +++ b/fs/f2fs/super.c > > @@ -2560,7 +2560,6 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent) > > sb->s_flags = (sb->s_flags & ~SB_POSIXACL) | > > (test_opt(sbi, POSIX_ACL) ? SB_POSIXACL : 0); > > memcpy(&sb->s_uuid, raw_super->uuid, sizeof(raw_super->uuid)); > > - sb->s_iflags |= SB_I_CGROUPWB; > > > > /* init f2fs-specific super block info */ > > sbi->valid_super_block = valid_super_block; > > -- > > 2.15.1 Another merge window has passed and I am still holding this tree and its merge fixes ... Tomorrow I will drop the btrfs tree and rename the btrfs-kdave tree to btrfs. OK? -- Cheers, Stephen Rothwell [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [GIT PULL] f2fs update for 4.16-rc1 2018-06-18 3:38 ` Stephen Rothwell @ 2018-06-18 18:05 ` Chris Mason -1 siblings, 0 replies; 19+ messages in thread From: Chris Mason @ 2018-06-18 18:05 UTC (permalink / raw) To: Stephen Rothwell Cc: Josef Bacik, Jaegeuk Kim, Linus Torvalds, Linux F2FS Dev Mailing List, Linux Kernel Mailing List, Yufen Yu, Tejun Heo, David Sterba On 17 Jun 2018, at 23:38, Stephen Rothwell wrote: > Hi Chris, > > > Another merge window has passed and I am still holding this tree and > its > merge fixes ... Tomorrow I will drop the btrfs tree and rename the > btrfs-kdave tree to btrfs. OK? Really sorry for the hassle, the f2fs subject line got this filtered into a different inbox. Tejun and Josef are working out these btrfs cgroup fixes, so I've pushed v4.17 to my -next tree for now. Dave has been doing the Btrfs merge window for some time, so focusing linux-next on his tree is a better fit. But either way, I've fixed my my email filters so you're back at the top where you belong. -chris ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [GIT PULL] f2fs update for 4.16-rc1 @ 2018-06-18 18:05 ` Chris Mason 0 siblings, 0 replies; 19+ messages in thread From: Chris Mason @ 2018-06-18 18:05 UTC (permalink / raw) To: Stephen Rothwell Cc: Yufen Yu, Josef Bacik, David Sterba, Linux Kernel Mailing List, Linux F2FS Dev Mailing List, Tejun Heo, Jaegeuk Kim, Linus Torvalds On 17 Jun 2018, at 23:38, Stephen Rothwell wrote: > Hi Chris, > > > Another merge window has passed and I am still holding this tree and > its > merge fixes ... Tomorrow I will drop the btrfs tree and rename the > btrfs-kdave tree to btrfs. OK? Really sorry for the hassle, the f2fs subject line got this filtered into a different inbox. Tejun and Josef are working out these btrfs cgroup fixes, so I've pushed v4.17 to my -next tree for now. Dave has been doing the Btrfs merge window for some time, so focusing linux-next on his tree is a better fit. But either way, I've fixed my my email filters so you're back at the top where you belong. -chris ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [GIT PULL] f2fs update for 4.16-rc1 2018-06-18 18:05 ` Chris Mason @ 2018-06-20 23:26 ` Stephen Rothwell -1 siblings, 0 replies; 19+ messages in thread From: Stephen Rothwell @ 2018-06-20 23:26 UTC (permalink / raw) To: Chris Mason Cc: Josef Bacik, Jaegeuk Kim, Linus Torvalds, Linux F2FS Dev Mailing List, Linux Kernel Mailing List, Yufen Yu, Tejun Heo, David Sterba [-- Attachment #1: Type: text/plain, Size: 627 bytes --] Hi Chris, On Mon, 18 Jun 2018 14:05:32 -0400 Chris Mason <clm@fb.com> wrote: > > Really sorry for the hassle, the f2fs subject line got this filtered > into a different inbox. Tejun and Josef are working out these btrfs Ah, that explains it. > cgroup fixes, so I've pushed v4.17 to my -next tree for now. Thanks. > Dave has been doing the Btrfs merge window for some time, so focusing > linux-next on his tree is a better fit. But either way, I've fixed my > my email filters so you're back at the top where you belong. I'll just leave things as they are for now. -- Cheers, Stephen Rothwell [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [GIT PULL] f2fs update for 4.16-rc1 @ 2018-06-20 23:26 ` Stephen Rothwell 0 siblings, 0 replies; 19+ messages in thread From: Stephen Rothwell @ 2018-06-20 23:26 UTC (permalink / raw) To: Chris Mason Cc: Josef Bacik, Jaegeuk Kim, Linus Torvalds, Linux F2FS Dev Mailing List, Linux Kernel Mailing List, Yufen Yu, Tejun Heo, David Sterba [-- Attachment #1: Type: text/plain, Size: 627 bytes --] Hi Chris, On Mon, 18 Jun 2018 14:05:32 -0400 Chris Mason <clm@fb.com> wrote: > > Really sorry for the hassle, the f2fs subject line got this filtered > into a different inbox. Tejun and Josef are working out these btrfs Ah, that explains it. > cgroup fixes, so I've pushed v4.17 to my -next tree for now. Thanks. > Dave has been doing the Btrfs merge window for some time, so focusing > linux-next on his tree is a better fit. But either way, I've fixed my > my email filters so you're back at the top where you belong. I'll just leave things as they are for now. -- Cheers, Stephen Rothwell [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 19+ messages in thread
end of thread, other threads:[~2018-06-20 23:26 UTC | newest] Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-01-30 22:32 [GIT PULL] f2fs update for 4.16-rc1 Jaegeuk Kim 2018-01-30 22:32 ` Jaegeuk Kim 2018-01-30 22:54 ` Stephen Rothwell 2018-01-31 0:25 ` Jaegeuk Kim 2018-01-31 0:25 ` Jaegeuk Kim 2018-01-31 22:25 ` Stephen Rothwell 2018-03-13 23:17 ` Stephen Rothwell 2018-04-03 2:29 ` Stephen Rothwell 2018-04-03 14:20 ` Tejun Heo 2018-04-03 14:20 ` Tejun Heo 2018-04-03 14:45 ` Stephen Rothwell 2018-05-02 22:42 ` Stephen Rothwell 2018-05-03 9:49 ` David Sterba 2018-06-18 3:38 ` Stephen Rothwell 2018-06-18 3:38 ` Stephen Rothwell 2018-06-18 18:05 ` Chris Mason 2018-06-18 18:05 ` Chris Mason 2018-06-20 23:26 ` Stephen Rothwell 2018-06-20 23:26 ` Stephen Rothwell
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.