diff options
author | Chen, Kenneth W <kenneth.w.chen@intel.com> | 2005-10-13 21:48:42 +0200 |
---|---|---|
committer | Jens Axboe <axboe@nelson.home.kernel.dk> | 2005-10-28 08:15:30 +0200 |
commit | 20e5c81fcff89535dced2ed71cf24c6c648ff40e (patch) | |
tree | c19edf4ce636f0e4713e39dd918c7d3772cda64a /fs | |
parent | 741b2252a5e14d6c60a913c77a6099abe73a854a (diff) | |
download | op-kernel-dev-20e5c81fcff89535dced2ed71cf24c6c648ff40e.zip op-kernel-dev-20e5c81fcff89535dced2ed71cf24c6c648ff40e.tar.gz |
[patch] remove gendisk->stamp_idle field
struct gendisk has these two fields: stamp, stamp_idle. Update to
stamp_idle is always in sync with stamp and they are always the same.
Therefore, it does not add any value in having two fields tracking
same timestamp. Suggest to remove it.
Also, we should only update gendisk stats with non-zero value.
Advantage is that we don't have to needlessly calculate memory address,
and then add zero to the content.
Signed-off-by: Ken Chen <kenneth.w.chen@intel.com>
Signed-off-by: Jens Axboe <axboe@suse.de>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/partitions/check.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/partitions/check.c b/fs/partitions/check.c index 77e178f..1e84864 100644 --- a/fs/partitions/check.c +++ b/fs/partitions/check.c @@ -430,7 +430,7 @@ void del_gendisk(struct gendisk *disk) disk->flags &= ~GENHD_FL_UP; unlink_gendisk(disk); disk_stat_set_all(disk, 0); - disk->stamp = disk->stamp_idle = 0; + disk->stamp = 0; devfs_remove_disk(disk); |