Thread (1 message) 1 message, 1 author, 2021-08-16

Re: [syzbot] general protection fault in wb_timer_fn

From: Christoph Hellwig <hch@lst.de>
Date: 2021-08-16 09:33:41
Also in: lkml

On Mon, Aug 16, 2021 at 05:10:41PM +0800, Hillf Danton wrote:
quoted hunk
Remove and free all qos callbacks added, with cb->timer deleted in
blk_stat_remove_callback().

only for thoughts.
+++ x/block/blk-sysfs.c
@@ -800,9 +800,7 @@ static void blk_release_queue(struct kob
 
 	might_sleep();
 
-	if (test_bit(QUEUE_FLAG_POLL_STATS, &q->queue_flags))
-		blk_stat_remove_callback(q, q->poll_cb);
-	blk_stat_free_callback(q->poll_cb);
+	rq_qos_exit(q);
rq_qos_exit is already called in blk_cleanup_queue, and the blk-mq
pollig doesn't even use the qos framework.  So I'm not sure what this
is supposed to help.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help