Whamcloud - gitweb
blkid: remove whole-disk entries from cache when partitions are found
authorEric Sandeen <sandeen@redhat.com>
Thu, 23 Apr 2009 03:51:51 +0000 (22:51 -0500)
committerTheodore Ts'o <tytso@mit.edu>
Thu, 23 Apr 2009 02:52:40 +0000 (22:52 -0400)
commit4e60e06847c781efffc8c43342b7756bbe5cff45
tree6d50cc96f77a6e4de6ad3482242dd0d2956dccbc
parentde7a86e23c6113f386dc3500304cd8a37b91bfa8
blkid: remove whole-disk entries from cache when partitions are found

We can get into a situation in blkid where whole disks remain
in the cache, even though partitions are found.  For labels
such as sun disklabels which may have the first partition
beginning at sector 0, this is even somewhat likely.

1) create a sun disklabel w/partitions
2) mkfs the first partition (at sector 0)
3) remove the partition table
4) run blkid - this finds the fs on the whole disk, places in cache
5) recreate the partition table
6) run blkid - this finds the partition, places in cache

And now we have both /dev/sda and /dev/sda1 in cache.

There are heuristics in probe_all to avoid putting the whole disk
in cache if it has partitions, but there is nothing to remove the
whole-disk entry in the above case.  I think the below patch
suffices, although I haven't quite convinced myself that setting
the lens[which]=0; is the right logic for that bit of state...

Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
lib/blkid/devname.c