* [PATCH] blkparse: Fix blktrace output pipe broken in the new kernel.
@ 2011-01-11 6:48 Tao Ma
2011-01-11 7:36 ` [PATCH] blkparse: Fix blktrace output pipe broken in the new Jens Axboe
2011-01-11 8:12 ` Tao Ma
0 siblings, 2 replies; 3+ messages in thread
From: Tao Ma @ 2011-01-11 6:48 UTC (permalink / raw)
To: linux-btrace
From: Tao Ma <boyu.mt@taobao.com>
With the newest kernel(say 2.6.37, some older one should also have the
similar problem), some cfq messages are added to blktrace, so it makes
the old blkparse broken.
See a simple example:
1. blktrace /dev/sdb -o -|blkparse -i -
2. Run the following command(/dev/sdb1 is mounted at /mnt/test_dir):
dd if=/mnt/test_dir/test of=/dev/null bs=4k count=1 iflag=direct
There are only 2 lines of output there:
8,16 0 1 0.000000000 13183 A R 114759 + 8 <- (8,17) 114696
8,16 0 2 0.000000491 13183 Q R 114759 + 8 [dd]
And even we run a command line like:
for((i=0;i<100;i++))do dd if=/mnt/ocfs2/test of=/dev/null bs=4k count=1 iflag=direct;done
We are only given the same 2 lines of output.
While the really one should look like:
8,16 0 1 0.000000000 13319 A R 114759 + 8 <- (8,17) 114696
8,16 0 2 0.000000376 13319 Q R 114759 + 8 [dd]
8,16 0 0 0.000005931 0 m N cfq13319 alloced
8,16 0 3 0.000006259 13319 G R 114759 + 8 [dd]
8,16 0 4 0.000007143 13319 P N [dd]
8,16 0 5 0.000007817 13319 I R 114759 + 8 [dd]
8,16 0 0 0.000008491 0 m N cfq13319 insert_request
8,16 0 0 0.000009029 0 m N cfq13319 add_to_rr
...
The main reason is that in show_entries_rb, we test sequences every time,
but actually with some messages like cfq, the sequence number is always
0 which makes the old sequence check refuses all the logs after it.
So only check/store sequence number if it isn't a message.
Cc: Jens Axboe <axboe@kernel.dk>
Signed-off-by: Tao Ma <boyu.mt@taobao.com>
---
blkparse.c | 6 ++++--
1 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/blkparse.c b/blkparse.c
index da34c14..c911185 100644
--- a/blkparse.c
+++ b/blkparse.c
@@ -2065,7 +2065,8 @@ static void show_entries_rb(int force)
break;
}
- if (check_sequence(pdi, t, force))
+ if (!(bit->action = BLK_TN_MESSAGE) &&
+ check_sequence(pdi, t, force))
break;
if (!force && bit->time > last_allowed_time)
@@ -2076,7 +2077,8 @@ static void show_entries_rb(int force)
if (!pci || pci->cpu != bit->cpu)
pci = get_cpu_info(pdi, bit->cpu);
- pci->last_sequence = bit->sequence;
+ if (!(bit->action = BLK_TN_MESSAGE))
+ pci->last_sequence = bit->sequence;
pci->nelems++;
--
1.6.3.GIT
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] blkparse: Fix blktrace output pipe broken in the new
2011-01-11 6:48 [PATCH] blkparse: Fix blktrace output pipe broken in the new kernel Tao Ma
@ 2011-01-11 7:36 ` Jens Axboe
2011-01-11 8:12 ` Tao Ma
1 sibling, 0 replies; 3+ messages in thread
From: Jens Axboe @ 2011-01-11 7:36 UTC (permalink / raw)
To: linux-btrace
On 2011-01-11 07:48, Tao Ma wrote:
> From: Tao Ma <boyu.mt@taobao.com>
>
> With the newest kernel(say 2.6.37, some older one should also have the
> similar problem), some cfq messages are added to blktrace, so it makes
> the old blkparse broken.
>
> See a simple example:
> 1. blktrace /dev/sdb -o -|blkparse -i -
> 2. Run the following command(/dev/sdb1 is mounted at /mnt/test_dir):
> dd if=/mnt/test_dir/test of=/dev/null bs=4k count=1 iflag=direct
>
> There are only 2 lines of output there:
> 8,16 0 1 0.000000000 13183 A R 114759 + 8 <- (8,17) 114696
> 8,16 0 2 0.000000491 13183 Q R 114759 + 8 [dd]
>
> And even we run a command line like:
> for((i=0;i<100;i++))do dd if=/mnt/ocfs2/test of=/dev/null bs=4k count=1 iflag=direct;done
> We are only given the same 2 lines of output.
>
> While the really one should look like:
> 8,16 0 1 0.000000000 13319 A R 114759 + 8 <- (8,17) 114696
> 8,16 0 2 0.000000376 13319 Q R 114759 + 8 [dd]
> 8,16 0 0 0.000005931 0 m N cfq13319 alloced
> 8,16 0 3 0.000006259 13319 G R 114759 + 8 [dd]
> 8,16 0 4 0.000007143 13319 P N [dd]
> 8,16 0 5 0.000007817 13319 I R 114759 + 8 [dd]
> 8,16 0 0 0.000008491 0 m N cfq13319 insert_request
> 8,16 0 0 0.000009029 0 m N cfq13319 add_to_rr
> ...
>
> The main reason is that in show_entries_rb, we test sequences every time,
> but actually with some messages like cfq, the sequence number is always
> 0 which makes the old sequence check refuses all the logs after it.
> So only check/store sequence number if it isn't a message.
Thanks for the patch with the nice and detailed description, I have
applied it.
But I'm curious, this is a 2.6.37 regression?
--
Jens Axboe
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] blkparse: Fix blktrace output pipe broken in the new
2011-01-11 6:48 [PATCH] blkparse: Fix blktrace output pipe broken in the new kernel Tao Ma
2011-01-11 7:36 ` [PATCH] blkparse: Fix blktrace output pipe broken in the new Jens Axboe
@ 2011-01-11 8:12 ` Tao Ma
1 sibling, 0 replies; 3+ messages in thread
From: Tao Ma @ 2011-01-11 8:12 UTC (permalink / raw)
To: linux-btrace
On 01/11/2011 03:36 PM, Jens Axboe wrote:
> On 2011-01-11 07:48, Tao Ma wrote:
>> From: Tao Ma<boyu.mt@taobao.com>
>>
>> With the newest kernel(say 2.6.37, some older one should also have the
>> similar problem), some cfq messages are added to blktrace, so it makes
>> the old blkparse broken.
>>
>> See a simple example:
>> 1. blktrace /dev/sdb -o -|blkparse -i -
>> 2. Run the following command(/dev/sdb1 is mounted at /mnt/test_dir):
>> dd if=/mnt/test_dir/test of=/dev/null bs=4k count=1 iflag=direct
>>
>> There are only 2 lines of output there:
>> 8,16 0 1 0.000000000 13183 A R 114759 + 8<- (8,17) 114696
>> 8,16 0 2 0.000000491 13183 Q R 114759 + 8 [dd]
>>
>> And even we run a command line like:
>> for((i=0;i<100;i++))do dd if=/mnt/ocfs2/test of=/dev/null bs=4k count=1 iflag=direct;done
>> We are only given the same 2 lines of output.
>>
>> While the really one should look like:
>> 8,16 0 1 0.000000000 13319 A R 114759 + 8<- (8,17) 114696
>> 8,16 0 2 0.000000376 13319 Q R 114759 + 8 [dd]
>> 8,16 0 0 0.000005931 0 m N cfq13319 alloced
>> 8,16 0 3 0.000006259 13319 G R 114759 + 8 [dd]
>> 8,16 0 4 0.000007143 13319 P N [dd]
>> 8,16 0 5 0.000007817 13319 I R 114759 + 8 [dd]
>> 8,16 0 0 0.000008491 0 m N cfq13319 insert_request
>> 8,16 0 0 0.000009029 0 m N cfq13319 add_to_rr
>> ...
>>
>> The main reason is that in show_entries_rb, we test sequences every time,
>> but actually with some messages like cfq, the sequence number is always
>> 0 which makes the old sequence check refuses all the logs after it.
>> So only check/store sequence number if it isn't a message.
>
> Thanks for the patch with the nice and detailed description, I have
> applied it.
cool.
>
> But I'm curious, this is a 2.6.37 regression?
Oh, I don't have time to look back the git to see whether it is a
regression or not. ;)
But it should be a compatible problem in blkparse and it should be a
problem when we adds cfq tracing to the kernel, a very long time ago I
guess. I just found this recently when I wanted to know more internals
about the blktrace.
Regards,
Tao
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-01-11 8:12 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-01-11 6:48 [PATCH] blkparse: Fix blktrace output pipe broken in the new kernel Tao Ma
2011-01-11 7:36 ` [PATCH] blkparse: Fix blktrace output pipe broken in the new Jens Axboe
2011-01-11 8:12 ` Tao Ma
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.