diff options
| author | Tao Ma <boyu.mt@taobao.com> | 2011-03-03 10:53:20 -0500 |
|---|---|---|
| committer | Jens Axboe <jaxboe@fusionio.com> | 2011-03-03 10:53:20 -0500 |
| commit | 2d3a8497f8cc5aca14b722cd37d51f6c15ff9f74 (patch) | |
| tree | 2e0013fdc1eec486a425e666a885b04a1c37befb /include/linux/blktrace_api.h | |
| parent | 255bb490c8c27eed484d538efe6ef6a7473bd3f6 (diff) | |
| download | kernel_goldelico_gta04-2d3a8497f8cc5aca14b722cd37d51f6c15ff9f74.zip kernel_goldelico_gta04-2d3a8497f8cc5aca14b722cd37d51f6c15ff9f74.tar.gz kernel_goldelico_gta04-2d3a8497f8cc5aca14b722cd37d51f6c15ff9f74.tar.bz2 | |
blktrace: Remove blk_fill_rwbs_rq.
If we enable trace events to trace block actions, We use
blk_fill_rwbs_rq to analyze the corresponding actions
in request's cmd_flags, but we only choose the minor 2 bits
from it, so most of other flags(e.g, REQ_SYNC) are missing.
For example, with a sync write we get:
write_test-2409 [001] 160.013869: block_rq_insert: 3,64 W 0 () 258135 + =
8 [write_test]
Since now we have integrated the flags of both bio and request,
it is safe to pass rq->cmd_flags directly to blk_fill_rwbs and
blk_fill_rwbs_rq isn't needed any more.
With this patch, after a sync write we get:
write_test-2417 [000] 226.603878: block_rq_insert: 3,64 WS 0 () 258135 +=
8 [write_test]
Signed-off-by: Tao Ma <boyu.mt@taobao.com>
Acked-by: Jeff Moyer <jmoyer@redhat.com>
Signed-off-by: Jens Axboe <jaxboe@fusionio.com>
Diffstat (limited to 'include/linux/blktrace_api.h')
| -rw-r--r-- | include/linux/blktrace_api.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/include/linux/blktrace_api.h b/include/linux/blktrace_api.h index 3395cf7..b22fb0d 100644 --- a/include/linux/blktrace_api.h +++ b/include/linux/blktrace_api.h @@ -245,7 +245,6 @@ static inline int blk_cmd_buf_len(struct request *rq) extern void blk_dump_cmd(char *buf, struct request *rq); extern void blk_fill_rwbs(char *rwbs, u32 rw, int bytes); -extern void blk_fill_rwbs_rq(char *rwbs, struct request *rq); #endif /* CONFIG_EVENT_TRACING && CONFIG_BLOCK */ |