From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from verein.lst.de ([213.95.11.211]:58262 "EHLO newverein.lst.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751771AbdHJHOJ (ORCPT ); Thu, 10 Aug 2017 03:14:09 -0400 Date: Thu, 10 Aug 2017 09:14:08 +0200 From: Christoph Hellwig To: Bart Van Assche Cc: Jens Axboe , linux-block@vger.kernel.org, Christoph Hellwig , dm-devel@redhat.com, Mike Snitzer , Laurence Oberman , stable@vger.kernel.org Subject: Re: [PATCH] block: Make blk_mq_delay_kick_requeue_list() rerun the queue at a quiet time Message-ID: <20170810071408.GA20981@lst.de> References: <20170809182806.32741-1-bart.vanassche@wdc.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170809182806.32741-1-bart.vanassche@wdc.com> Sender: stable-owner@vger.kernel.org List-ID: On Wed, Aug 09, 2017 at 11:28:06AM -0700, Bart Van Assche wrote: > The blk_mq_delay_kick_requeue_list() function is used by the device > mapper and only by the device mapper to rerun the queue and requeue > list after a delay. This function is called once per request that > gets requeued. Modify this function such that the queue is run once > per path change event instead of once per request that is requeued. Is there a chance we could look into killing this function and moving dm to run it every time we requeue a request like all the other drivers?