diff options
| author | Kent Overstreet <koverstreet@google.com> | 2013-02-05 18:19:29 -0500 |
|---|---|---|
| committer | Kent Overstreet <koverstreet@google.com> | 2013-03-23 17:15:30 -0400 |
| commit | aa8b57aa3d1c06ca53312294ee6dfc767ee3ddb3 (patch) | |
| tree | a5921d44756d32e0555a8ae284db3c16a8386860 /include/trace | |
| parent | f73a1c7d117d07a96d89475066188a2b79e53c48 (diff) | |
block: Use bio_sectors() more consistently
Bunch of places in the code weren't using it where they could be -
this'll reduce the size of the patch that puts bi_sector/bi_size/bi_idx
into a struct bvec_iter.
Signed-off-by: Kent Overstreet <koverstreet@google.com>
CC: Jens Axboe <axboe@kernel.dk>
CC: "Ed L. Cashin" <ecashin@coraid.com>
CC: Nick Piggin <npiggin@kernel.dk>
CC: Jiri Kosina <jkosina@suse.cz>
CC: Jim Paris <jim@jtan.com>
CC: Geoff Levand <geoff@infradead.org>
CC: Alasdair Kergon <agk@redhat.com>
CC: dm-devel@redhat.com
CC: Neil Brown <neilb@suse.de>
CC: Steven Rostedt <rostedt@goodmis.org>
Acked-by: Ed Cashin <ecashin@coraid.com>
Diffstat (limited to 'include/trace')
| -rw-r--r-- | include/trace/events/block.h | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/include/trace/events/block.h b/include/trace/events/block.h index 9961726523d0..5a28843725df 100644 --- a/include/trace/events/block.h +++ b/include/trace/events/block.h | |||
| @@ -244,7 +244,7 @@ TRACE_EVENT(block_bio_bounce, | |||
| 244 | __entry->dev = bio->bi_bdev ? | 244 | __entry->dev = bio->bi_bdev ? |
| 245 | bio->bi_bdev->bd_dev : 0; | 245 | bio->bi_bdev->bd_dev : 0; |
| 246 | __entry->sector = bio->bi_sector; | 246 | __entry->sector = bio->bi_sector; |
| 247 | __entry->nr_sector = bio->bi_size >> 9; | 247 | __entry->nr_sector = bio_sectors(bio); |
| 248 | blk_fill_rwbs(__entry->rwbs, bio->bi_rw, bio->bi_size); | 248 | blk_fill_rwbs(__entry->rwbs, bio->bi_rw, bio->bi_size); |
| 249 | memcpy(__entry->comm, current->comm, TASK_COMM_LEN); | 249 | memcpy(__entry->comm, current->comm, TASK_COMM_LEN); |
| 250 | ), | 250 | ), |
| @@ -281,7 +281,7 @@ TRACE_EVENT(block_bio_complete, | |||
| 281 | __entry->dev = bio->bi_bdev ? | 281 | __entry->dev = bio->bi_bdev ? |
| 282 | bio->bi_bdev->bd_dev : 0; | 282 | bio->bi_bdev->bd_dev : 0; |
| 283 | __entry->sector = bio->bi_sector; | 283 | __entry->sector = bio->bi_sector; |
| 284 | __entry->nr_sector = bio->bi_size >> 9; | 284 | __entry->nr_sector = bio_sectors(bio); |
| 285 | __entry->error = error; | 285 | __entry->error = error; |
| 286 | blk_fill_rwbs(__entry->rwbs, bio->bi_rw, bio->bi_size); | 286 | blk_fill_rwbs(__entry->rwbs, bio->bi_rw, bio->bi_size); |
| 287 | ), | 287 | ), |
| @@ -309,7 +309,7 @@ DECLARE_EVENT_CLASS(block_bio_merge, | |||
| 309 | TP_fast_assign( | 309 | TP_fast_assign( |
| 310 | __entry->dev = bio->bi_bdev->bd_dev; | 310 | __entry->dev = bio->bi_bdev->bd_dev; |
| 311 | __entry->sector = bio->bi_sector; | 311 | __entry->sector = bio->bi_sector; |
| 312 | __entry->nr_sector = bio->bi_size >> 9; | 312 | __entry->nr_sector = bio_sectors(bio); |
| 313 | blk_fill_rwbs(__entry->rwbs, bio->bi_rw, bio->bi_size); | 313 | blk_fill_rwbs(__entry->rwbs, bio->bi_rw, bio->bi_size); |
| 314 | memcpy(__entry->comm, current->comm, TASK_COMM_LEN); | 314 | memcpy(__entry->comm, current->comm, TASK_COMM_LEN); |
| 315 | ), | 315 | ), |
| @@ -376,7 +376,7 @@ TRACE_EVENT(block_bio_queue, | |||
| 376 | TP_fast_assign( | 376 | TP_fast_assign( |
| 377 | __entry->dev = bio->bi_bdev->bd_dev; | 377 | __entry->dev = bio->bi_bdev->bd_dev; |
| 378 | __entry->sector = bio->bi_sector; | 378 | __entry->sector = bio->bi_sector; |
| 379 | __entry->nr_sector = bio->bi_size >> 9; | 379 | __entry->nr_sector = bio_sectors(bio); |
| 380 | blk_fill_rwbs(__entry->rwbs, bio->bi_rw, bio->bi_size); | 380 | blk_fill_rwbs(__entry->rwbs, bio->bi_rw, bio->bi_size); |
| 381 | memcpy(__entry->comm, current->comm, TASK_COMM_LEN); | 381 | memcpy(__entry->comm, current->comm, TASK_COMM_LEN); |
| 382 | ), | 382 | ), |
| @@ -404,7 +404,7 @@ DECLARE_EVENT_CLASS(block_get_rq, | |||
| 404 | TP_fast_assign( | 404 | TP_fast_assign( |
| 405 | __entry->dev = bio ? bio->bi_bdev->bd_dev : 0; | 405 | __entry->dev = bio ? bio->bi_bdev->bd_dev : 0; |
| 406 | __entry->sector = bio ? bio->bi_sector : 0; | 406 | __entry->sector = bio ? bio->bi_sector : 0; |
| 407 | __entry->nr_sector = bio ? bio->bi_size >> 9 : 0; | 407 | __entry->nr_sector = bio ? bio_sectors(bio) : 0; |
| 408 | blk_fill_rwbs(__entry->rwbs, | 408 | blk_fill_rwbs(__entry->rwbs, |
| 409 | bio ? bio->bi_rw : 0, __entry->nr_sector); | 409 | bio ? bio->bi_rw : 0, __entry->nr_sector); |
| 410 | memcpy(__entry->comm, current->comm, TASK_COMM_LEN); | 410 | memcpy(__entry->comm, current->comm, TASK_COMM_LEN); |
| @@ -580,7 +580,7 @@ TRACE_EVENT(block_bio_remap, | |||
| 580 | TP_fast_assign( | 580 | TP_fast_assign( |
| 581 | __entry->dev = bio->bi_bdev->bd_dev; | 581 | __entry->dev = bio->bi_bdev->bd_dev; |
| 582 | __entry->sector = bio->bi_sector; | 582 | __entry->sector = bio->bi_sector; |
| 583 | __entry->nr_sector = bio->bi_size >> 9; | 583 | __entry->nr_sector = bio_sectors(bio); |
| 584 | __entry->old_dev = dev; | 584 | __entry->old_dev = dev; |
| 585 | __entry->old_sector = from; | 585 | __entry->old_sector = from; |
| 586 | blk_fill_rwbs(__entry->rwbs, bio->bi_rw, bio->bi_size); | 586 | blk_fill_rwbs(__entry->rwbs, bio->bi_rw, bio->bi_size); |
