diff options
| author |   Chen, Kenneth W <kenneth.w.chen@intel.com> | 2005-10-13 21:49:29 +0200 |
|---|---|---|
| committer |   Jens Axboe <axboe@nelson.home.kernel.dk> | 2005-10-28 08:15:43 +0200 |
| commit | b2982649ce38293b14684b26bcda20cfc54164e6 (patch) | |
| tree | 8208c0cd33f2565f0f193d0afab282a7d0778382 /drivers/block/ll_rw_blk.c | |
| parent | [patch] remove gendisk->stamp_idle field (diff) | |
| download | linux-dev-b2982649ce38293b14684b26bcda20cfc54164e6.tar.xz linux-dev-b2982649ce38293b14684b26bcda20cfc54164e6.zip | |
Following the same idea, it occurs to me that we should only update
disk stat when "now" is different from disk->stamp. Otherwise, we
are again needlessly adding zero to the stats.
Signed-off-by: Ken Chen <kenneth.w.chen@intel.com>
Signed-off-by: Jens Axboe <axboe@suse.de>
Diffstat (limited to 'drivers/block/ll_rw_blk.c')
| -rw-r--r-- | drivers/block/ll_rw_blk.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/block/ll_rw_blk.c b/drivers/block/ll_rw_blk.c index c42071fd2e95..4e2b1b06b411 100644 --- a/drivers/block/ll_rw_blk.c +++ b/drivers/block/ll_rw_blk.c @@ -2433,6 +2433,9 @@ void disk_round_stats(struct gendisk *disk) { unsigned long now = jiffies; + if (now == disk->stamp) + return; + if (disk->in_flight) { __disk_stat_add(disk, time_in_queue, disk->in_flight * (now - disk->stamp)); |