linux-kernel - Re: [PATCH v2] block: fix request.queuelist usage in flush

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <171821162990.49689.4088473696825322669.b4-ty@kernel.dk>
Date: Wed, 12 Jun 2024 11:00:29 -0600
From: Jens Axboe <axboe@...nel.dk>
To: ming.lei@...hat.com, hch@....de, f.weber@...xmox.com, 
 bvanassche@....org, Chengming Zhou <chengming.zhou@...ux.dev>
Cc: linux-block@...r.kernel.org, linux-kernel@...r.kernel.org, 
 zhouchengming@...edance.com
Subject: Re: [PATCH v2] block: fix request.queuelist usage in flush


On Sat, 08 Jun 2024 22:31:15 +0800, Chengming Zhou wrote:
> Friedrich Weber reported a kernel crash problem and bisected to commit
> 81ada09cc25e ("blk-flush: reuse rq queuelist in flush state machine").
> 
> The root cause is that we use "list_move_tail(&rq->queuelist, pending)"
> in the PREFLUSH/POSTFLUSH sequences. But rq->queuelist.next == xxx since
> it's popped out from plug->cached_rq in __blk_mq_alloc_requests_batch().
> We don't initialize its queuelist just for this first request, although
> the queuelist of all later popped requests will be initialized.
> 
> [...]

Applied, thanks!

[1/1] block: fix request.queuelist usage in flush
      commit: d0321c812d89c5910d8da8e4b10c891c6b96ff70

Best regards,
-- 
Jens Axboe