Re: [f2fs-dev] [PATCH] f2fs: change maximum zstd compression buffer size

2020-05-07 Thread Chao Yu
On 2020/5/6 22:56, Jaegeuk Kim wrote: > On 05/06, Chao Yu wrote: >> On 2020/5/6 7:05, Jaegeuk Kim wrote: >>> On 05/05, Chao Yu wrote: On 2020-5-4 22:30, Jaegeuk Kim wrote: > From: Daeho Jeong > > Current zstd compression buffer size is one page and header size less > than

Re: [f2fs-dev] [PATCH] f2fs: change maximum zstd compression buffer size

2020-05-06 Thread Chao Yu
On 2020/5/6 22:56, Jaegeuk Kim wrote: > On 05/06, Chao Yu wrote: >> On 2020/5/6 7:05, Jaegeuk Kim wrote: >>> On 05/05, Chao Yu wrote: On 2020-5-4 22:30, Jaegeuk Kim wrote: > From: Daeho Jeong > > Current zstd compression buffer size is one page and header size less > than

Re: [f2fs-dev] [PATCH] f2fs: change maximum zstd compression buffer size

2020-05-06 Thread Jaegeuk Kim
On 05/06, Chao Yu wrote: > On 2020/5/6 7:05, Jaegeuk Kim wrote: > > On 05/05, Chao Yu wrote: > >> On 2020-5-4 22:30, Jaegeuk Kim wrote: > >>> From: Daeho Jeong > >>> > >>> Current zstd compression buffer size is one page and header size less > >>> than cluster size. By this, zstd compression

Re: [f2fs-dev] [PATCH] f2fs: change maximum zstd compression buffer size

2020-05-06 Thread Chao Yu
On 2020/5/6 7:05, Jaegeuk Kim wrote: > On 05/05, Chao Yu wrote: >> On 2020-5-4 22:30, Jaegeuk Kim wrote: >>> From: Daeho Jeong >>> >>> Current zstd compression buffer size is one page and header size less >>> than cluster size. By this, zstd compression always succeeds even if >>> the real

Re: [f2fs-dev] [PATCH] f2fs: change maximum zstd compression buffer size

2020-05-05 Thread Jaegeuk Kim
On 05/05, Chao Yu wrote: > On 2020-5-4 22:30, Jaegeuk Kim wrote: > > From: Daeho Jeong > > > > Current zstd compression buffer size is one page and header size less > > than cluster size. By this, zstd compression always succeeds even if > > the real compression data is failed to fit into the

Re: [f2fs-dev] [PATCH] f2fs: change maximum zstd compression buffer size

2020-05-04 Thread Chao Yu
On 2020-5-5 9:52, Chao Yu wrote: On 2020-5-4 22:30, Jaegeuk Kim wrote: From: Daeho Jeong Current zstd compression buffer size is one page and header size less than cluster size. By this, zstd compression always succeeds even if the real compression data is failed to fit into the buffer size,

Re: [f2fs-dev] [PATCH] f2fs: change maximum zstd compression buffer size

2020-05-04 Thread Chao Yu
On 2020-5-4 22:30, Jaegeuk Kim wrote: From: Daeho Jeong Current zstd compression buffer size is one page and header size less than cluster size. By this, zstd compression always succeeds even if the real compression data is failed to fit into the buffer size, and eventually reading the cluster