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