Re: [PATCH v3 2/2] staging: erofs: fix i_blocks calculation

2019-05-27 Thread Chao Yu
On 2019/5/28 11:19, Gao Xiang wrote: > For compressed files, i_blocks should not be calculated > by using i_size. use i_u.compressed_blocks instead. > > In addition, i_blocks was miscalculated for non-compressed > files previously, fix it as well. > > Signed-off-by: Gao Xiang Reviewed-by: Chao

Re: [PATCH v3 1/2] staging: erofs: support statx

2019-05-27 Thread Chao Yu
On 2019/5/28 11:19, Gao Xiang wrote: > statx() has already been supported in commit a528d35e8bfc > ("statx: Add a system call to make enhanced file info available"), > user programs can get more useful attributes. > > Signed-off-by: Gao Xiang Reviewed-by: Chao Yu Thanks,

Re: [PATCH v2 2/2] staging: erofs: fix i_blocks calculation

2019-05-27 Thread Gao Xiang
On 2019/5/28 11:02, Chao Yu wrote: > On 2019/5/28 10:36, Gao Xiang wrote: >> For compressed files, i_blocks should not be calculated >> by using i_size. i_u.compressed_blocks is used instead. >> >> In addition, i_blocks was miscalculated for non-compressed >> files previously, fix it as well.

[PATCH v3 2/2] staging: erofs: fix i_blocks calculation

2019-05-27 Thread Gao Xiang
For compressed files, i_blocks should not be calculated by using i_size. use i_u.compressed_blocks instead. In addition, i_blocks was miscalculated for non-compressed files previously, fix it as well. Signed-off-by: Gao Xiang --- change log v3: - wrap i.u.compressed_blocks with le32_to_cpu()

[PATCH v3 1/2] staging: erofs: support statx

2019-05-27 Thread Gao Xiang
statx() has already been supported in commit a528d35e8bfc ("statx: Add a system call to make enhanced file info available"), user programs can get more useful attributes. Signed-off-by: Gao Xiang --- change log v3: - update attributes_mask to indicate bits which erofs support as Chao pointed

Re: [PATCH v2 2/2] staging: erofs: fix i_blocks calculation

2019-05-27 Thread Chao Yu
On 2019/5/28 10:36, Gao Xiang wrote: > For compressed files, i_blocks should not be calculated > by using i_size. i_u.compressed_blocks is used instead. > > In addition, i_blocks was miscalculated for non-compressed > files previously, fix it as well. > > Signed-off-by: Gao Xiang > --- > change

Re: [PATCH 1/2] staging: erofs: support statx

2019-05-27 Thread Gao Xiang
Hi Chao, On 2019/5/28 10:44, Chao Yu wrote: > On 2019/5/28 10:31, Gao Xiang wrote: >> statx() has already been supported in commit a528d35e8bfc >> ("statx: Add a system call to make enhanced file info available"), >> user programs can get more useful attributes. >> >> Signed-off-by: Gao Xiang >>

Re: [PATCH 1/2] staging: erofs: support statx

2019-05-27 Thread Chao Yu
On 2019/5/28 10:31, Gao Xiang wrote: > statx() has already been supported in commit a528d35e8bfc > ("statx: Add a system call to make enhanced file info available"), > user programs can get more useful attributes. > > Signed-off-by: Gao Xiang > --- > drivers/staging/erofs/inode.c| 18

[PATCH v2 2/2] staging: erofs: fix i_blocks calculation

2019-05-27 Thread Gao Xiang
For compressed files, i_blocks should not be calculated by using i_size. i_u.compressed_blocks is used instead. In addition, i_blocks was miscalculated for non-compressed files previously, fix it as well. Signed-off-by: Gao Xiang --- change log v2: - fix description in commit message - fix to

[PATCH 2/2] staging: erofs: fix i_blocks calculation

2019-05-27 Thread Gao Xiang
For compressed files, i_blocks should not be calculated by using i_size. i_u.compressed_blocks is used instead. In addition, i_blocks is miscalculated for non-compressed files previously, fix it as well. Signed-off-by: Gao Xiang --- drivers/staging/erofs/inode.c | 14 -- 1 file

[PATCH 1/2] staging: erofs: support statx

2019-05-27 Thread Gao Xiang
statx() has already been supported in commit a528d35e8bfc ("statx: Add a system call to make enhanced file info available"), user programs can get more useful attributes. Signed-off-by: Gao Xiang --- drivers/staging/erofs/inode.c| 18 ++ drivers/staging/erofs/internal.h | 2

Re: [PATCH 2/3] mm: remove cleancache.c

2019-05-27 Thread Chao Yu
On 2019/5/27 18:32, Juergen Gross wrote: > With the removal of tmem and xen-selfballoon the only user of > cleancache is gone. Remove it, too. > > Signed-off-by: Juergen Gross > --- > Documentation/vm/cleancache.rst | 296 > Documentation/vm/frontswap.rst

Re: [PATCH 2/3] mm: remove cleancache.c

2019-05-27 Thread Joseph Qi
On 19/5/27 18:32, Juergen Gross wrote: > With the removal of tmem and xen-selfballoon the only user of > cleancache is gone. Remove it, too. > > Signed-off-by: Juergen Gross > --- > Documentation/vm/cleancache.rst | 296 > Documentation/vm/frontswap.rst

Re: [PATCH 2/3] mm: remove cleancache.c

2019-05-27 Thread David Sterba
On Mon, May 27, 2019 at 12:32:06PM +0200, Juergen Gross wrote: > With the removal of tmem and xen-selfballoon the only user of > cleancache is gone. Remove it, too. > > Signed-off-by: Juergen Gross > --- > Documentation/vm/cleancache.rst | 296 >

[PATCH 0/3] remove tmem and code depending on it

2019-05-27 Thread Juergen Gross
Tmem has been an experimental Xen feature which has been dropped recently due to security problems and lack of maintainership. So it is time now to drop it in Linux kernel, too. Juergen Gross (3): xen: remove tmem driver mm: remove cleancache.c mm: remove tmem specifics from frontswap

[PATCH 2/3] mm: remove cleancache.c

2019-05-27 Thread Juergen Gross
With the removal of tmem and xen-selfballoon the only user of cleancache is gone. Remove it, too. Signed-off-by: Juergen Gross --- Documentation/vm/cleancache.rst | 296 Documentation/vm/frontswap.rst | 10 +- Documentation/vm/index.rst | 1 -