Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965004AbWJWSjY (ORCPT ); Mon, 23 Oct 2006 14:39:24 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S965009AbWJWSjY (ORCPT ); Mon, 23 Oct 2006 14:39:24 -0400 Received: from iriserv.iradimed.com ([69.44.168.233]:6939 "EHLO iradimed.com") by vger.kernel.org with ESMTP id S965004AbWJWSjX (ORCPT ); Mon, 23 Oct 2006 14:39:23 -0400 Message-ID: <453D0C62.4030601@cfl.rr.com> Date: Mon, 23 Oct 2006 14:39:30 -0400 From: Phillip Susi User-Agent: Thunderbird 1.5.0.7 (Windows/20060909) MIME-Version: 1.0 To: Martin Peschke CC: Andrew Morton , linux-kernel@vger.kernel.org Subject: Re: [Patch 0/5] I/O statistics through request queues References: <1161435423.3054.111.camel@dyn-9-152-230-71.boeblingen.de.ibm.com> In-Reply-To: <1161435423.3054.111.camel@dyn-9-152-230-71.boeblingen.de.ibm.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-OriginalArrivalTime: 23 Oct 2006 18:39:30.0282 (UTC) FILETIME=[936124A0:01C6F6D2] X-TM-AS-Product-Ver: SMEX-7.2.0.1122-3.6.1039-14768.003 X-TM-AS-Result: No--17.611700-5.000000-31 Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2649 Lines: 59 Would this be a good candidate to implement using kprobes? I was under the impression that basing instrumentation on kprobes would be a good thing since you can load the instrumentation code only when needed, then unload it. Martin Peschke wrote: > This patch set makes the block layer maintain statistics for request > queues. Resulting data closely resembles the actual I/O traffic to a > device, as the block layer takes hints from block device drivers when a > request is being issued as well as when it is about to complete. > > It is crucial (for us) to be able to look at such kernel level data in > case of customer situations. It allows us to determine what kind of > requests might be involved in performance situations. This information > helps to understand whether one faces a device issue or a Linux issue. > Not being able to tap into performance data is regarded as a big minus > by some enterprise customers, who are reluctant to use Linux SCSI > support or Linux. > > Statistics data includes: > - request sizes (read + write), > - residual bytes of partially completed requests (read + write), > - request latencies (read + write), > - request retries (read + write), > - request concurrency, > > For sample data please have a look at the SCSI stack patch or the DASD > driver patch respectively. This data is only gathered if statistics have > been enabled by users at run time (default is off). > > The advantage of instrumenting request queues is that we can cover a > broad range of devices, including SCSI tape devices. > Having the block layer maintain such statistics on behalf of drivers > provides for comparability through a common set of statistics. > > A previous approach was to put all the code into the SCSI stack: > http://marc.theaimsgroup.com/?l=linux-kernel&m=115928678420835&w=2 > I gathered from feedback that moving that stuff to the block layer > might be preferable. > > These patches use the statistics component described in > Documentation/statistics.txt. > > Patches are against 2.6.19-rc2-mm2. > > [Patch 1/5] I/O statistics through request queues: timeval_to_us() > [Patch 2/5] I/O statistics through request queues: queue instrumentation > [Patch 3/5] I/O statistics through request queues: small SCSI cleanup > [Patch 4/5] I/O statistics through request queues: SCSI > [Patch 5/5] I/O statistics through request queues: DASD > > Martin > - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/