[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4F0E4333.6070607@lge.com>
Date: Thu, 12 Jan 2012 11:19:31 +0900
From: Namhyung Kim <namhyung.kim@....com>
To: Tejun Heo <tj@...nel.org>
CC: Namhyung Kim <namhyung@...il.com>, axboe@...nel.dk,
mingo@...hat.com, rostedt@...dmis.org, fweisbec@...il.com,
teravest@...gle.com, slavapestov@...gle.com, ctalbott@...gle.com,
dhsharp@...gle.com, linux-kernel@...r.kernel.org,
winget@...gle.com, Chanho Park <chanho61.park@...sung.com>
Subject: Re: [PATCH RESEND 9/9] block, trace: implement ioblame - IO tracer
with origin tracking
2012-01-12 10:44 AM, Tejun Heo wrote:
> On Wed, Jan 11, 2012 at 5:41 PM, Namhyung Kim<namhyung.kim@....com> wrote:
>> OK, but that can be controlled from userspace, I guess? Do we have to care
>> about that?
>
> Otherwise, it's gonna be a pretty silly interface. If you want to
> watch the TP, you have to restart the whole thing which also implies
> intent number would change. And there still is the problem of losing
> new intent events due to overflow. It just isn't the right interface
> for the task.
>
I understood it's unreliable. BTW I think that sane userland tool must
start both of ioblame_io and ioblame_create_intent, say, TPs at once -
prior to triggering ioblame/enable, obviously. The ioblame/intents can
be used as a backup just in case.
Thanks,
Namhyung Kim
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists