Re: [f2fs-dev] [PATCH 2/4] btrfs: Use module_subinit{_noexit} and module_subeixt helper macros

2024-07-23 Thread Youling Tang
On 24/07/2024 06:24, kernel test robot wrote: Hi Youling, kernel test robot noticed the following build warnings: [auto build test WARNING on kdave/for-next] [also build test WARNING on linus/master next-20240723] [cannot apply to jaegeuk-f2fs/dev-test jaegeuk-f2fs/dev soc/for-next v6.10] [If

[f2fs-dev] Patchwork summary for: f2fs

2024-07-23 Thread patchwork-bot+f2fs
Hello: The following patches were marked "accepted", because they were applied to jaegeuk/f2fs.git (dev): Series: [f2fs-dev,1/3] fs: Export in_group_or_capable() Submitter: Youling Tang Committer: Christian Brauner Patchwork: https://patchwork.kernel.org/project/f2fs/list/?series=863678

Re: [f2fs-dev] [PATCH 1/3] fs: Export in_group_or_capable()

2024-07-23 Thread patchwork-bot+f2fs
Hello: This series was applied to jaegeuk/f2fs.git (dev) by Christian Brauner : On Thu, 20 Jun 2024 11:23:33 +0800 you wrote: > From: Youling Tang > > Export in_group_or_capable() as a VFS helper function. > > Signed-off-by: Youling Tang > --- > fs/attr.c | 2 -- > fs/inode.c

Re: [f2fs-dev] [PATCH v18 0/7] Case insensitive cleanup for ext4/f2fs

2024-07-23 Thread patchwork-bot+f2fs
Hello: This series was applied to jaegeuk/f2fs.git (dev) by Christian Brauner : On Thu, 6 Jun 2024 10:33:46 +0300 you wrote: > Hello, > > I am trying to respin the series here : > https://www.spinics.net/lists/linux-ext4/msg85081.html > > I resent some of the v9 patches and got some reviews fr

Re: [f2fs-dev] [RESEND PATCH v9 0/3] Introduce case-insensitive string comparison helper

2024-07-23 Thread patchwork-bot+f2fs
Hello: This series was applied to jaegeuk/f2fs.git (dev) by Christian Brauner : On Thu, 8 Feb 2024 08:43:31 +0200 you wrote: > Hello, > > I am trying to respin the series here : > https://www.spinics.net/lists/linux-ext4/msg85081.html > > To make it easier to apply I split it into smaller chun

Re: [f2fs-dev] [RESEND PATCH v9 0/2] Minor cleanup for case insensitive path

2024-07-23 Thread patchwork-bot+f2fs
Hello: This series was applied to jaegeuk/f2fs.git (dev) by Christian Brauner : On Wed, 7 Feb 2024 08:43:00 +0200 you wrote: > Hello, > > I am trying to respin the series here : > https://www.spinics.net/lists/linux-ext4/msg85081.html > > To make it easier to apply I split it into smaller chun

Re: [f2fs-dev] [PATCH v16 0/9] Cache insensitive cleanup for ext4/f2fs

2024-07-23 Thread patchwork-bot+f2fs
Hello: This series was applied to jaegeuk/f2fs.git (dev) by Christian Brauner : On Fri, 5 Apr 2024 15:13:23 +0300 you wrote: > Hello, > > I am trying to respin the series here : > https://www.spinics.net/lists/linux-ext4/msg85081.html > > I resent some of the v9 patches and got some reviews fr

Re: [f2fs-dev] [PATCH v10 0/8]

2024-07-23 Thread patchwork-bot+f2fs
Hello: This series was applied to jaegeuk/f2fs.git (dev) by Christian Brauner : On Thu, 15 Feb 2024 06:26:46 +0200 you wrote: > Hello, > > I am trying to respin the series here : > https://www.spinics.net/lists/linux-ext4/msg85081.html > > I resent some of the v9 patches and got some reviews fr

Re: [f2fs-dev] [GIT PULL] f2fs update for 6.11-rc1

2024-07-23 Thread patchwork-bot+f2fs
Hello: This pull request was applied to jaegeuk/f2fs.git (dev) by Linus Torvalds : On Tue, 23 Jul 2024 20:58:12 + you wrote: > Hi Linus, > > Could you please consider this pull request? > > Thanks, > > The following changes since commit 2ef5971ff345d3c000873725db555085e0131961: > > [...]

Re: [f2fs-dev] [PATCH 0/6] Remove page_mapping()

2024-07-23 Thread patchwork-bot+f2fs
Hello: This series was applied to jaegeuk/f2fs.git (dev) by Andrew Morton : On Tue, 23 Apr 2024 23:55:31 +0100 you wrote: > There are only a few users left. Convert them all to either call > folio_mapping() or just use folio->mapping directly. > > Matthew Wilcox (Oracle) (6): > fscrypt: Conve

Re: [f2fs-dev] [PATCH 4/4] f2fs: Use module_{subinit, subeixt} helper macros

2024-07-23 Thread Youling Tang
On 24/07/2024 02:51, kernel test robot wrote: Hi Youling, kernel test robot noticed the following build warnings: [auto build test WARNING on kdave/for-next] [also build test WARNING on linus/master next-20240723] [cannot apply to jaegeuk-f2fs/dev-test jaegeuk-f2fs/dev soc/for-next v6.10] [If

Re: [f2fs-dev] [PATCH 1/4] module: Add module_subinit{_noexit} and module_subeixt helper macros

2024-07-23 Thread Youling Tang
Hi, Christoph On 23/07/2024 22:33, Christoph Hellwig wrote: On Tue, Jul 23, 2024 at 04:32:36PM +0800, Youling Tang wrote: Providing module_subinit{_noexit} and module_subeixt helps macros ensure that modules init/exit match their order, while also simplifying the code. The three macros are def

Re: [f2fs-dev] [PATCH 1/4] module: Add module_subinit{_noexit} and module_subeixt helper macros

2024-07-23 Thread Youling Tang
Hi, Mika On 23/07/2024 17:58, Mika Penttilä wrote: On 7/23/24 11:32, Youling Tang wrote: From: Youling Tang In theory init/exit should match their sequence, thus normally they should look like this: -+ init_A();| init_B();

[f2fs-dev] [PATCH v3] dump.f2fs: Dump symlinks as symlinks

2024-07-23 Thread Daniel Rosenberg via Linux-f2fs-devel
Previously, dumped symlinks would always create regular files instead. This allows symlinks to be dumped as symlinks with the -L option. The i_name field's name may not be the same as the actual name from the dirent, so we use the dirent name when available. Currently hardlinks aren't detected, s

Re: [f2fs-dev] [GIT PULL] f2fs update for 6.11-rc1

2024-07-23 Thread pr-tracker-bot
The pull request you sent on Tue, 23 Jul 2024 20:58:12 +: > git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git > tags/f2fs-for-6.11-rc1 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/5ad7ff8738b8bd238ca899df08badb1f61bcc39e Thank you! -- Deet-doot-d

Re: [f2fs-dev] [PATCH 2/4] btrfs: Use module_subinit{_noexit} and module_subeixt helper macros

2024-07-23 Thread kernel test robot
Hi Youling, kernel test robot noticed the following build warnings: [auto build test WARNING on kdave/for-next] [also build test WARNING on linus/master next-20240723] [cannot apply to jaegeuk-f2fs/dev-test jaegeuk-f2fs/dev soc/for-next v6.10] [If your patch is applied to the wrong git tree

Re: [f2fs-dev] [PATCH 4/4] f2fs: Use module_{subinit, subeixt} helper macros

2024-07-23 Thread kernel test robot
Hi Youling, kernel test robot noticed the following build warnings: [auto build test WARNING on kdave/for-next] [also build test WARNING on linus/master next-20240723] [cannot apply to jaegeuk-f2fs/dev-test jaegeuk-f2fs/dev soc/for-next v6.10] [If your patch is applied to the wrong git tree

[f2fs-dev] [GIT PULL] f2fs update for 6.11-rc1

2024-07-23 Thread Jaegeuk Kim
Hi Linus, Could you please consider this pull request? Thanks, The following changes since commit 2ef5971ff345d3c000873725db555085e0131961: Merge tag 'vfs-6.10-rc4.fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/vfs/vfs (2024-06-11 12:04:21 -0700) are available in the Git repositor

Re: [f2fs-dev] [PATCH 4/4] f2fs: Use module_{subinit, subeixt} helper macros

2024-07-23 Thread kernel test robot
Hi Youling, kernel test robot noticed the following build warnings: [auto build test WARNING on kdave/for-next] [also build test WARNING on linus/master next-20240723] [cannot apply to jaegeuk-f2fs/dev-test jaegeuk-f2fs/dev soc/for-next v6.10] [If your patch is applied to the wrong git tree

Re: [f2fs-dev] [PATCH] dump.f2fs: Dump symlinks as symlinks

2024-07-23 Thread Chao Yu
On 2024/7/16 9:57, Daniel Rosenberg wrote: Previously, dumped symlinks would always create regular files instead. This allows symlinks to be dumped as symlinks with the -L option. The i_name field's name may not be the same as the actual name from the dirent, so we use the dirent name when avail

Re: [f2fs-dev] [PATCH 1/4] module: Add module_subinit{_noexit} and module_subeixt helper macros

2024-07-23 Thread Christoph Hellwig
On Tue, Jul 23, 2024 at 04:32:36PM +0800, Youling Tang wrote: > Providing module_subinit{_noexit} and module_subeixt helps macros ensure > that modules init/exit match their order, while also simplifying the code. > > The three macros are defined as follows: > - module_subinit(initfn, exitfn,rollb

Re: [f2fs-dev] [PATCH 0/3] Add {init, exit}_sequence_fs() helper function

2024-07-23 Thread Christoph Hellwig
On Tue, Jul 23, 2024 at 04:44:14PM +0800, Youling Tang wrote: > Thanks for your suggestion,  I re-implemented it using section mode, > and the new patch set [1] has been sent. Nice! I'll review it. ___ Linux-f2fs-devel mailing list Linux-f2fs-devel@l

Re: [f2fs-dev] [PATCH 1/4] module: Add module_subinit{_noexit} and module_subeixt helper macros

2024-07-23 Thread Mika Penttilä
On 7/23/24 11:32, Youling Tang wrote: > From: Youling Tang > > In theory init/exit should match their sequence, thus normally they should > look like this: > -+ > init_A();| > init_B();| > init_C();|

Re: [f2fs-dev] [PATCH 0/3] Add {init, exit}_sequence_fs() helper function

2024-07-23 Thread Youling Tang
Hi, Christoph On 11/07/2024 16:26, Christoph Hellwig wrote: Can we please stop this boilerplate code an instead our __init/__exit sections to supper multiple entires per module. This should be mostly trivial, except that we'd probably want a single macro that has the init and exit calls so that

[f2fs-dev] [PATCH 3/4] ext4: Use module_{subinit, subexit} helper macros

2024-07-23 Thread Youling Tang
From: Youling Tang Use module_{subinit, subinit} to ensure that modules init and exit are in sequence and to simplify the code. Signed-off-by: Youling Tang --- fs/ext4/super.c | 115 ++-- 1 file changed, 33 insertions(+), 82 deletions(-) diff --git

[f2fs-dev] [PATCH 2/4] btrfs: Use module_subinit{_noexit} and module_subeixt helper macros

2024-07-23 Thread Youling Tang
From: Youling Tang Use module_{subinit, subinit} to ensure that modules init and exit are in sequence and to simplify the code. Signed-off-by: Youling Tang --- fs/btrfs/super.c | 123 +-- 1 file changed, 23 insertions(+), 100 deletions(-) diff --git

[f2fs-dev] [PATCH 4/4] f2fs: Use module_{subinit, subeixt} helper macros

2024-07-23 Thread Youling Tang
From: Youling Tang Use module_{subinit, subinit} to ensure that modules init and exit are in sequence and to simplify the code. Signed-off-by: Youling Tang --- fs/f2fs/debug.c | 3 +- fs/f2fs/f2fs.h | 4 +- fs/f2fs/super.c | 139 +++- 3 files ch

[f2fs-dev] [PATCH 0/4] Add module_subinit{_noexit} and module_subeixt helper macros

2024-07-23 Thread Youling Tang
This series provides the module_subinit{_noexit} and module_subeixt helper macros and applies to btrfs, ext4 and f2fs. See link [1] for the previous discussion process. [1]: https://lore.kernel.org/all/20240711074859.366088-1-youling.t...@linux.dev/ Youling Tang (4): module: Add module_subini

[f2fs-dev] [PATCH 1/4] module: Add module_subinit{_noexit} and module_subeixt helper macros

2024-07-23 Thread Youling Tang
From: Youling Tang In theory init/exit should match their sequence, thus normally they should look like this: -+ init_A();| init_B();| init_C();| | exit_C();