mbox series

[v3,0/2] ceph: fix cephfs rsync kworker high load issue

Message ID 20220223015934.37379-1-xiubli@redhat.com (mailing list archive)
Headers show
Series ceph: fix cephfs rsync kworker high load issue | expand

Message

Xiubo Li Feb. 23, 2022, 1:59 a.m. UTC
From: Xiubo Li <xiubli@redhat.com>

V3:
- switch to use the kmem_cache_zalloc() to zero the memory.
- rebase to the latest code in testing branch.

V2:
- allocate the capsnap memory ourside of ceph_queue_cap_snap() from
Jeff's advice.
- fix the code style and logs to make the logs to be more readable

Xiubo Li (2):
  ceph: allocate capsnap memory outside of ceph_queue_cap_snap()
  ceph: misc fix for code style and logs

 fs/ceph/snap.c | 168 ++++++++++++++++++++++++++-----------------------
 1 file changed, 90 insertions(+), 78 deletions(-)

Comments

Jeff Layton Feb. 23, 2022, 12:49 p.m. UTC | #1
On Wed, 2022-02-23 at 09:59 +0800, xiubli@redhat.com wrote:
> From: Xiubo Li <xiubli@redhat.com>
> 
> V3:
> - switch to use the kmem_cache_zalloc() to zero the memory.
> - rebase to the latest code in testing branch.
> 
> V2:
> - allocate the capsnap memory ourside of ceph_queue_cap_snap() from
> Jeff's advice.
> - fix the code style and logs to make the logs to be more readable
> 
> Xiubo Li (2):
>   ceph: allocate capsnap memory outside of ceph_queue_cap_snap()
>   ceph: misc fix for code style and logs
> 
>  fs/ceph/snap.c | 168 ++++++++++++++++++++++++++-----------------------
>  1 file changed, 90 insertions(+), 78 deletions(-)
> 

LGTM!

Reviewed-by: Jeff Layton <jlayton@kernel.org>