Re: [PATCH 0/4] Fix for "sbitmapq" command

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Thank you for the review, Kazu.

On Tue, May 24, 2022 at 4:39 PM HAGIO KAZUHITO(萩尾 一仁) <k-hagio-ab@xxxxxxx> wrote:
Hi Lianbo,

On 2022/05/23 19:04, Lianbo Jiang wrote:
> The sbitmapq command will fail due to the following kernel commit:
>
> [1] commit <c548e62bcf6a> ("scsi: sbitmap: Move allocation hint
>      into sbitmap")
> [2] commit <efe1f3a1d583> ("scsi: sbitmap: Maintain allocation
>      round_robin in sbitmap")
> [3] commit <3301bc53358a> ("lib/sbitmap: kill 'depth' from sbitmap_word")
>
> Without the patchset, the sbitmapq won't work.

Good job!  The patchset looks nice and clean to me, tested ok with
some kernels up to 5.18.  Applied with some small adjustments.
 
No problem. Thanks 

btw, do you want to use the "<commid id>" style in crash's commit log? 
I think it's not so common, so removed the brackets this time.

Oh, I copied the "<commit id>" from another place. But you could feel free to adjust or remove the "<commit id>" from the patch log.

Thanks.
Lianbo


Thanks,
Kazu

>
> Lianbo Jiang (4):
>    sbitmapq: a cleanup in the sbitmapq_init()
>    sbitmapq: fix invalid offset for "sbitmap_queue_alloc_hint" on linux
>      v5.13-rc1
>    sbitmapq: fix invalid offset for "sbitmap_queue_round_robin" on linux
>      v5.13-rc1
>    sbitmapq: fix invalid offset for "sbitmap_word_depth" on linux
>      v5.18-rc1
>
>   defs.h    |  4 ++++
>   sbitmap.c | 70 +++++++++++++++++++++++++++----------------------------
>   symbols.c |  4 ++++
>   3 files changed, 42 insertions(+), 36 deletions(-)
>
--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://listman.redhat.com/mailman/listinfo/crash-utility
Contribution Guidelines: https://github.com/crash-utility/crash/wiki

[Index of Archives]     [Fedora Development]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]

 

Powered by Linux