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>
This commit is contained in:
parent
20e5c81fcf
commit
b2982649ce
1 changed files with 3 additions and 0 deletions
|
@ -2433,6 +2433,9 @@ void disk_round_stats(struct gendisk *disk)
|
||||||
{
|
{
|
||||||
unsigned long now = jiffies;
|
unsigned long now = jiffies;
|
||||||
|
|
||||||
|
if (now == disk->stamp)
|
||||||
|
return;
|
||||||
|
|
||||||
if (disk->in_flight) {
|
if (disk->in_flight) {
|
||||||
__disk_stat_add(disk, time_in_queue,
|
__disk_stat_add(disk, time_in_queue,
|
||||||
disk->in_flight * (now - disk->stamp));
|
disk->in_flight * (now - disk->stamp));
|
||||||
|
|
Loading…
Reference in a new issue