From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Subject: Re: [patch 5/5] block: use a driver-specific handler for the "inflight" value Date: Thu, 8 Nov 2018 06:52:40 -0800 Message-ID: <20181108145240.GA20087@infradead.org> References: <20181106213858.391264280@debian-a64.vm> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <20181106213858.391264280@debian-a64.vm> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dm-devel-bounces@redhat.com Errors-To: dm-devel-bounces@redhat.com To: Mikulas Patocka Cc: Mike Snitzer , dm-devel@redhat.com, "Alasdair G. Kergon" List-Id: dm-devel.ids On Tue, Nov 06, 2018 at 10:35:03PM +0100, Mikulas Patocka wrote: > Device mapper was converted to percpu inflight counters. In order to > display the correct values in the "inflight" sysfs file, we need a custom > callback that sums the percpu counters. The attribute that calls this is per-partition, while your method is per-queue, so there is some impedence mismatch here. Is there any way you could look into just making the generic code use percpu counters? Also please cc linux-block on patches that touch the generic block code.