diff options
author | Jens Axboe <axboe@suse.de> | 2005-11-01 03:26:16 -0500 |
---|---|---|
committer | Jens Axboe <axboe@suse.de> | 2005-11-01 03:26:16 -0500 |
commit | a362357b6cd62643d4dda3b152639303d78473da (patch) | |
tree | fe4ce823e638ded151edcb142f28a240860f0d33 /drivers/md/raid6main.c | |
parent | d72d904a5367ad4ca3f2c9a2ce8c3a68f0b28bf0 (diff) |
[BLOCK] Unify the seperate read/write io stat fields into arrays
Instead of having ->read_sectors and ->write_sectors, combine the two
into ->sectors[2] and similar for the other fields. This saves a branch
several places in the io path, since we don't have to care for what the
actual io direction is. On my x86-64 box, that's 200 bytes less text in
just the core (not counting the various drivers).
Signed-off-by: Jens Axboe <axboe@suse.de>
Diffstat (limited to 'drivers/md/raid6main.c')
-rw-r--r-- | drivers/md/raid6main.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/drivers/md/raid6main.c b/drivers/md/raid6main.c index 267eb1430c83..6437a95ffc1c 100644 --- a/drivers/md/raid6main.c +++ b/drivers/md/raid6main.c | |||
@@ -1621,6 +1621,7 @@ static int make_request (request_queue_t *q, struct bio * bi) | |||
1621 | sector_t new_sector; | 1621 | sector_t new_sector; |
1622 | sector_t logical_sector, last_sector; | 1622 | sector_t logical_sector, last_sector; |
1623 | struct stripe_head *sh; | 1623 | struct stripe_head *sh; |
1624 | const int rw = bio_data_dir(bi); | ||
1624 | 1625 | ||
1625 | if (unlikely(bio_barrier(bi))) { | 1626 | if (unlikely(bio_barrier(bi))) { |
1626 | bio_endio(bi, bi->bi_size, -EOPNOTSUPP); | 1627 | bio_endio(bi, bi->bi_size, -EOPNOTSUPP); |
@@ -1629,13 +1630,8 @@ static int make_request (request_queue_t *q, struct bio * bi) | |||
1629 | 1630 | ||
1630 | md_write_start(mddev, bi); | 1631 | md_write_start(mddev, bi); |
1631 | 1632 | ||
1632 | if (bio_data_dir(bi)==WRITE) { | 1633 | disk_stat_inc(mddev->gendisk, ios[rw]); |
1633 | disk_stat_inc(mddev->gendisk, writes); | 1634 | disk_stat_add(mddev->gendisk, sectors[rw], bio_sectors(bi)); |
1634 | disk_stat_add(mddev->gendisk, write_sectors, bio_sectors(bi)); | ||
1635 | } else { | ||
1636 | disk_stat_inc(mddev->gendisk, reads); | ||
1637 | disk_stat_add(mddev->gendisk, read_sectors, bio_sectors(bi)); | ||
1638 | } | ||
1639 | 1635 | ||
1640 | logical_sector = bi->bi_sector & ~((sector_t)STRIPE_SECTORS-1); | 1636 | logical_sector = bi->bi_sector & ~((sector_t)STRIPE_SECTORS-1); |
1641 | last_sector = bi->bi_sector + (bi->bi_size>>9); | 1637 | last_sector = bi->bi_sector + (bi->bi_size>>9); |
@@ -1682,7 +1678,7 @@ static int make_request (request_queue_t *q, struct bio * bi) | |||
1682 | if (--bi->bi_phys_segments == 0) { | 1678 | if (--bi->bi_phys_segments == 0) { |
1683 | int bytes = bi->bi_size; | 1679 | int bytes = bi->bi_size; |
1684 | 1680 | ||
1685 | if ( bio_data_dir(bi) == WRITE ) | 1681 | if (rw == WRITE ) |
1686 | md_write_end(mddev); | 1682 | md_write_end(mddev); |
1687 | bi->bi_size = 0; | 1683 | bi->bi_size = 0; |
1688 | bi->bi_end_io(bi, bytes, 0); | 1684 | bi->bi_end_io(bi, bytes, 0); |