md: Fix - again - partition detection when array becomes active
authorNeilBrown <neilb@suse.de>
Thu, 24 Feb 2011 06:26:41 +0000 (17:26 +1100)
committerAK <andi@firstfloor.org>
Thu, 31 Mar 2011 18:58:54 +0000 (11:58 -0700)
commitecbde8e81e12c89a4d19ba4c66afa4f8784187d3
tree38ee9254fdfa44bfac1b1ca930b4c43f7dad383b
parentb6c33e339d01d1132887d822555697b0b4bbded9
md: Fix - again - partition detection when array becomes active

[ upstream commit f0b4f7e2f29af678bd9af43422c537dcb6008603 ]

Revert
    b821eaa572fd737faaf6928ba046e571526c36c6
and
    f3b99be19ded511a1bf05a148276239d9f13eefa

When I wrote the first of these I had a wrong idea about the
lifetime of 'struct block_device'.  It can disappear at any time that
the block device is not open if it falls out of the inode cache.

So relying on the 'size' recorded with it to detect when the
device size has changed and so we need to revalidate, is wrong.

Rather, we really do need the 'changed' attribute stored directly in
the mddev and set/tested as appropriate.

Without this patch, a sequence of:
   mknod / open / close / unlink

(which can cause a block_device to be created and then destroyed)
will result in a rescan of the partition table and consequence removal
and addition of partitions.
Several of these in a row can get udev racing to create and unlink and
other code can get confused.

With the patch, the rescan is only performed when needed and so there
are no races.

This is suitable for any stable kernel from 2.6.35.

Reported-by: "Wojcik, Krzysztof" <krzysztof.wojcik@intel.com>
Signed-off-by: NeilBrown <neilb@suse.de>
Signed-off-by: Andi Kleen <ak@linux.intel.com>
Cc: stable@kernel.org
drivers/md/md.c
drivers/md/md.h