Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754541AbZCYN4v (ORCPT ); Wed, 25 Mar 2009 09:56:51 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752585AbZCYN4m (ORCPT ); Wed, 25 Mar 2009 09:56:42 -0400 Received: from brick.kernel.dk ([93.163.65.50]:35497 "EHLO kernel.dk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751323AbZCYN4m (ORCPT ); Wed, 25 Mar 2009 09:56:42 -0400 Date: Wed, 25 Mar 2009 14:56:39 +0100 From: Jens Axboe To: Arnaldo Carvalho de Melo Cc: Ingo Molnar , Li Zefan , Steven Rostedt , Frederic Weisbecker , LKML Subject: Re: [PATCH 3/3] blktrace: fix the original blktrace Message-ID: <20090325135639.GU27476@kernel.dk> References: <49C9F700.9070609@cn.fujitsu.com> <49C9F796.9040503@cn.fujitsu.com> <20090325101451.GK2341@elte.hu> <20090325101747.GL27476@kernel.dk> <20090325114720.GA9762@elte.hu> <20090325135142.GD10928@ghostprotocols.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20090325135142.GD10928@ghostprotocols.net> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2845 Lines: 65 On Wed, Mar 25 2009, Arnaldo Carvalho de Melo wrote: > Em Wed, Mar 25, 2009 at 12:47:20PM +0100, Ingo Molnar escreveu: > > > > * Jens Axboe wrote: > > > > > On Wed, Mar 25 2009, Ingo Molnar wrote: > > > > > > > > * Li Zefan wrote: > > > > > > > > > Currently the original blktrace, which is using relay and is used > > > > > via ioctl, is broken. You can use ftrace to see the output of > > > > > blktrace, but user-space blktrace is unusable. > > > > > > > > good catch, thanks Li! > > > > > > > > Jens, Arnaldo, do these fixes look good to you too? > > > > > > Look fine, but I'm very worried about the state of the current > > > code. I mean, this last round of fixes got the timestamp correct > > > and made blktrace work again. [...] > > > > Correct. I tested it on a 16way box (see the blkparse output below). > > If you can see any sort of anomaly please let us know so we can fix > > it. > > > > > [...] Those are something that should have been caught even before > > > the whole thing was posted, let alone merged. When do you plan on > > > pushing this upstream? Looks like 2.6.31 to me, it's clearly not > > > ready for 2.6.30 by any stretch. > > > > Well, apparently nobody tried ioctl based blktrace+blkparse on -tip > > or linux-next in the past ~1 month. The relayfs portion was (meant > > to be) kept largely untouched but this bug still crept in. > > > > Li started looking into it and found the bug. I tested the ftrace > > plugin side regularly, but you are right that this bug took too long > > to find (over a month) - from now on i'll check the ioctl side more > > regularly too, for all tracing and relayfs changes as well. > > I got sidetracked by other stuff and didn't managed to continue working > on it, but at the same time testing by somebody else is needed. There is > no bug-free code, let alone in the first iteration, even less for > something I wasn't familiar with and depended on people like Li to do a > careful review. > > Li fixed bugs and continued the work, getting things I wanted working > and made the initial steps to get done, like being able to just fed > trace_pipe into blkparse and get the same result as btrace. > > So I apologise for not having continued working on it more often to > catch myself the bugs I introduced, and give a big thank you to Li for > the thorough review, bugfixes and improvements. Dont apologise, there's nothing wrong with that. The only thing wrong would be for the unfinished code to get merged, when it clearly wasn't ready. -- Jens Axboe -- 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/