From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeff Moyer Subject: Re: [patch] nd_blk,nd_pmem,nd_btt: add endio blktrace events Date: Wed, 16 Nov 2016 09:06:11 -0500 Message-ID: References: <20161109191715.GA20314@infradead.org> <20161109193441.GA17343@infradead.org> <20161110193333.GA10865@infradead.org> <20161116045657.GA30049@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain Return-path: In-Reply-To: <20161116045657.GA30049@linux.intel.com> (Ross Zwisler's message of "Tue, 15 Nov 2016 21:56:57 -0700") Sender: linux-block-owner@vger.kernel.org To: Ross Zwisler Cc: Christoph Hellwig , linux-block@vger.kernel.org, linux-nvdimm@ml01.01.org, axboe@kernel.dk List-Id: linux-nvdimm@lists.01.org Ross Zwisler writes: > FWIW I think BRD has this same issue where we get block_bio_queue tracepoint > events but not block_bio_complete. Solving this in bio_endio() would fix that > driver as well. Yeah, there are several other drivers that will benefit. > Where does the Q (bio enqueue), I (req insert), etc. naming show up? Looking > at a tracepoint trace in perf I don't see that naming. Is that just a short > hand used between developers, or is it something else? That shorthand appears in blkparse and btt output (tools that are part of blktrace). Cheers, Jeff