Re: [PATCH -next v3 1/2] erofs: get rid of erofs_fs_context

2024-04-28 Thread Chao Yu
On 2024/4/19 20:36, Baokun Li wrote: Instead of allocating the erofs_sb_info in fill_super() allocate it during erofs_init_fs_context() and ensure that erofs can always have the info available during erofs_kill_sb(). After this erofs_fs_context is no longer needed, replace ctx with sbi, no

Re: [PATCH -next v3 1/2] erofs: get rid of erofs_fs_context

2024-04-28 Thread Gao Xiang
On 2024/4/19 20:36, Baokun Li wrote: Instead of allocating the erofs_sb_info in fill_super() allocate it during erofs_init_fs_context() and ensure that erofs can always have the info available during erofs_kill_sb(). After this erofs_fs_context is no longer needed, replace ctx with sbi, no

Re: [PATCH -next v3 1/2] erofs: get rid of erofs_fs_context

2024-04-23 Thread Gao Xiang
On 2024/4/19 20:36, Baokun Li wrote: Instead of allocating the erofs_sb_info in fill_super() allocate it during erofs_init_fs_context() and ensure that erofs can always have the info available during erofs_kill_sb(). After this erofs_fs_context is no longer needed, replace ctx with sbi, no

Re: [PATCH -next v3 1/2] erofs: get rid of erofs_fs_context

2024-04-22 Thread Jingbo Xu
On 4/22/24 7:31 PM, Baokun Li wrote: > Hi Jingbo, > > On 2024/4/22 18:25, Jingbo Xu wrote: >> >> On 4/19/24 8:36 PM, Baokun Li wrote: >> >>> @@ -761,12 +747,15 @@ static void erofs_free_dev_context(struct >>> erofs_dev_context *devs) >>>     static void erofs_fc_free(struct fs_context *fc) >>>

Re: [PATCH -next v3 1/2] erofs: get rid of erofs_fs_context

2024-04-22 Thread Baokun Li via Linux-erofs
Hi Jingbo, On 2024/4/22 18:25, Jingbo Xu wrote: On 4/19/24 8:36 PM, Baokun Li wrote: @@ -761,12 +747,15 @@ static void erofs_free_dev_context(struct erofs_dev_context *devs) static void erofs_fc_free(struct fs_context *fc) { - struct erofs_fs_context *ctx = fc->fs_private; +

Re: [PATCH -next v3 1/2] erofs: get rid of erofs_fs_context

2024-04-22 Thread Jingbo Xu
On 4/19/24 8:36 PM, Baokun Li wrote: > @@ -761,12 +747,15 @@ static void erofs_free_dev_context(struct > erofs_dev_context *devs) > > static void erofs_fc_free(struct fs_context *fc) > { > - struct erofs_fs_context *ctx = fc->fs_private; > + struct erofs_sb_info *sbi =

[PATCH -next v3 1/2] erofs: get rid of erofs_fs_context

2024-04-19 Thread Baokun Li via Linux-erofs
Instead of allocating the erofs_sb_info in fill_super() allocate it during erofs_init_fs_context() and ensure that erofs can always have the info available during erofs_kill_sb(). After this erofs_fs_context is no longer needed, replace ctx with sbi, no functional changes. Suggested-by: Jingbo Xu