[PATCH 0/3] erofs: random decompression cleanups

Gao Xiang posted 3 patches 3 years, 11 months ago
fs/erofs/zdata.c | 165 +++++++++++++++++++----------------------------
fs/erofs/zdata.h |  50 +++++++-------
2 files changed, 88 insertions(+), 127 deletions(-)
[PATCH 0/3] erofs: random decompression cleanups
Posted by Gao Xiang 3 years, 11 months ago
From: Gao Xiang <hsiangkao@linux.alibaba.com>

Hi folks,

Now I'm working on cleanuping decompression code and doing some
folio adaption for the next 5.20 cycle, see:

https://git.kernel.org/pub/scm/linux/kernel/git/xiang/linux.git/log/?h=erofs/folios

This cleanup work completely gets rid of PageError usage finally
and tries to prepare for introducing rolling hashing for EROFS
since EROFS supports compressing variable-sized data instead of
fixed-sized clusters.

Therefore, EROFS can support rolling hash easily and our mechanism
can make full use of filesystem interfaces (byte-addressed) naturally.

Before that, I'd like to submit some trivial cleanups in advance for
the 5.19 cycle. All patches are without any logical change, so I can
have a more recent codebase for the next rework cycle.

Thanks,
Gao Xiang

Gao Xiang (3):
  erofs: get rid of `struct z_erofs_collection'
  erofs: get rid of label `restart_now'
  erofs: simplify z_erofs_pcluster_readmore()

 fs/erofs/zdata.c | 165 +++++++++++++++++++----------------------------
 fs/erofs/zdata.h |  50 +++++++-------
 2 files changed, 88 insertions(+), 127 deletions(-)

-- 
2.30.2
Re: [PATCH 0/3] erofs: random decompression cleanups
Posted by Chao Yu 3 years, 11 months ago
Acked-by: Chao Yu <chao@kernel.org>

Thanks,

On 2022/5/29 13:54, Gao Xiang wrote:
> From: Gao Xiang <hsiangkao@linux.alibaba.com>
> 
> Hi folks,
> 
> Now I'm working on cleanuping decompression code and doing some
> folio adaption for the next 5.20 cycle, see:
> 
> https://git.kernel.org/pub/scm/linux/kernel/git/xiang/linux.git/log/?h=erofs/folios
> 
> This cleanup work completely gets rid of PageError usage finally
> and tries to prepare for introducing rolling hashing for EROFS
> since EROFS supports compressing variable-sized data instead of
> fixed-sized clusters.
> 
> Therefore, EROFS can support rolling hash easily and our mechanism
> can make full use of filesystem interfaces (byte-addressed) naturally.
> 
> Before that, I'd like to submit some trivial cleanups in advance for
> the 5.19 cycle. All patches are without any logical change, so I can
> have a more recent codebase for the next rework cycle.
> 
> Thanks,
> Gao Xiang
> 
> Gao Xiang (3):
>    erofs: get rid of `struct z_erofs_collection'
>    erofs: get rid of label `restart_now'
>    erofs: simplify z_erofs_pcluster_readmore()
> 
>   fs/erofs/zdata.c | 165 +++++++++++++++++++----------------------------
>   fs/erofs/zdata.h |  50 +++++++-------
>   2 files changed, 88 insertions(+), 127 deletions(-)
>
Re: [PATCH 0/3] erofs: random decompression cleanups
Posted by Gao Xiang 3 years, 11 months ago
On Tue, May 31, 2022 at 02:45:38PM +0800, Chao Yu wrote:
> Acked-by: Chao Yu <chao@kernel.org>

Thanks Chao! It'd be much helpful for me if we apply these cleanups
so I can have a new folio rework+cleanup+rolling hash 5.20 cycle.

Thanks,
Gao Xiang

> 
> Thanks,
> 
> On 2022/5/29 13:54, Gao Xiang wrote:
> > From: Gao Xiang <hsiangkao@linux.alibaba.com>
> > 
> > Hi folks,
> > 
> > Now I'm working on cleanuping decompression code and doing some
> > folio adaption for the next 5.20 cycle, see:
> > 
> > https://git.kernel.org/pub/scm/linux/kernel/git/xiang/linux.git/log/?h=erofs/folios
> > 
> > This cleanup work completely gets rid of PageError usage finally
> > and tries to prepare for introducing rolling hashing for EROFS
> > since EROFS supports compressing variable-sized data instead of
> > fixed-sized clusters.
> > 
> > Therefore, EROFS can support rolling hash easily and our mechanism
> > can make full use of filesystem interfaces (byte-addressed) naturally.
> > 
> > Before that, I'd like to submit some trivial cleanups in advance for
> > the 5.19 cycle. All patches are without any logical change, so I can
> > have a more recent codebase for the next rework cycle.
> > 
> > Thanks,
> > Gao Xiang
> > 
> > Gao Xiang (3):
> >    erofs: get rid of `struct z_erofs_collection'
> >    erofs: get rid of label `restart_now'
> >    erofs: simplify z_erofs_pcluster_readmore()
> > 
> >   fs/erofs/zdata.c | 165 +++++++++++++++++++----------------------------
> >   fs/erofs/zdata.h |  50 +++++++-------
> >   2 files changed, 88 insertions(+), 127 deletions(-)
> >