Neil Thanks for the insight, much appreciated. I've tried what you suggested and still get stuck. >:losetup /dev/loop0 /tmp/foo/sdb1 >:losetup /dev/loop1 /tmp/foo/sdc1 >:losetup /dev/loop2 /tmp/foo/sdd1 >:losetup /dev/loop3 /tmp/foo/sde1 >:losetup /dev/loop4 /tmp/foo/sdf1 >:losetup /dev/loop5 /tmp/foo/sdg1 >:losetup /dev/loop6 /tmp/foo/sdh1 >:mdadm --assemble --force --update=revert-reshape --invalid-backup /dev/md127 /dev/loop0 /dev/loop1 /dev/loop2 /dev/loop3 /dev/loop4 /dev/loop5 /dev/loop6 mdadm: /dev/md127: Need a backup file to complete reshape of this array. mdadm: Please provided one with "--backup-file=..." mdadm: (Don't specify --update=revert-reshape again, that part succeeded.) As you can see it 'seems' to have accepted the revert command, but even though I've told it the backup is invalid it's still insisting on the backup being made available. Any further thoughts or insights would be gratefully received. On 9 March 2016 at 00:23, NeilBrown <nfbrown@xxxxxxxxxx> wrote: > On Wed, Mar 02 2016, Another Sillyname wrote: > >> I have a 30TB RAID6 array using 7 x 6TB drives that I wanted to >> migrate to RAID5 to take one of the drives offline and use in a new >> array for a migration. >> >> sudo mdadm --grow /dev/md127 --level=raid5 --raid-device=6 >> --backup-file=mdadm_backupfile > > First observation: Don't use --backup-file unless mdadm tell you that > you have to. New mdadm on new kernel with newly create arrays don't > need a backup file at all. Your array is sufficiently newly created and > I think your mdadm/kernel are new enough too. Note in the --examine output: > >> Unused Space : before=262056 sectors, after=143 sectors > > This means there is (nearly) 128M of free space in the start of each > device. md can perform the reshape by copying a few chunks down into > this space, then the next few chunks into the space just freed, then the > next few chunks ... and so on. No backup file needed. That is > providing the chunk size is quite a bit smaller than the space, and your > 512K chunk size certainly is. > > A reshape which increases the size of the array needs 'before' space, a > reshape which decreases the size of the array needs 'after' space. A > reshape which doesn't change the size of the array (like yours) can use > either. > >> >> I watched this using cat /proc/mdstat and even after an hour the >> percentage of the reshape was still 0.0%. > > A more useful number to watch is the (xxx/yyy) after the percentage. > The first number should change at least every few seconds. > >> >> Reboot..... >> >> Array will not come back online at all. >> >> Bring the server up without the array trying to automount. >> >> cat /proc/mdstat shows the array offline. >> >> Personalities : >> md127 : inactive sdf1[2](S) sde1[3](S) sdg1[0](S) sdb1[8](S) >> sdh1[7](S) sdc1[1](S) sdd1[6](S) >> 41022733300 blocks super 1.2 >> >> unused devices: <none> >> >> Try to reassemble the array. >> >>>sudo mdadm --assemble /dev/md127 /dev/sdb1 /dev/sdc1 /dev/sdd1 /dev/sde1 /dev/sdf1 /dev/sdg1 /dev/sdh1 >> mdadm: /dev/sdg1 is busy - skipping >> mdadm: /dev/sdh1 is busy - skipping >> mdadm: Merging with already-assembled /dev/md/server187.internallan.com:1 > > It looks like you are getting races with udev. mdadm is detecting the > race and says that it is "Merging" rather than creating a separate array > but still the result isn't very useful... > > > When you run "mdadm --assemble /dev/md127 ...." mdadm notices that /dev/md127 > already exists but isn't active, so it stops it properly so that all the > devices become available to be assembled. > As the devices become available they tell udev "Hey, I've changed > status" and udev says "Hey, you look like part of an md array, let's put > you back together".... or something like that. I might have the details > a little wrong - it is a while since I looked at this. > Anyway it seems that udev called "mdadm -I" to put some of the devices > together so they were busy when your "mdadm --assemble" looked at them. > > >> mdadm: Failed to restore critical section for reshape, sorry. >> Possibly you needed to specify the --backup-file >> >> >> Have no idea where the server187 stuff has come from. > > That is in the 'Name' field in the metadata, which must have been put > there when the array was created >> Name : server187.internallan.com:1 >> Creation Time : Sun May 10 14:47:51 2015 > > It is possible to change it after-the-fact, but unlikely unless someone > explicitly tried. > I doesn't really matter how it got there as all the devices are the > same. > When "mdadm -I /dev/sdb1" etc is run by udev, mdadm needs to deduce a > name for the array. It looks in the Name filed and creates > > /dev/md/server187.internallan.com:1 > >> >> stop the array. >> >>>sudo mdadm --stop /dev/md127 >> >> try to re-assemble >> >>>sudo mdadm --assemble /dev/md127 /dev/sdb1 /dev/sdc1 /dev/sdd1 /dev/sde1 /dev/sdf1 /dev/sdg1 /dev/sdh1 >> >> mdadm: Failed to restore critical section for reshape, sorry. >> Possibly you needed to specify the --backup-file >> >> >> try to re-assemble using the backup file >> >>>sudo mdadm --assemble /dev/md127 /dev/sdb1 /dev/sdc1 /dev/sdd1 /dev/sde1 /dev/sdf1 /dev/sdg1 /dev/sdh1 --backup-file=mdadm_backupfile >> >> mdadm: Failed to restore critical section for reshape, sorry. > > As you have noted else where, the backup file contains nothing useful. > That is causing the problem. > > When an in-place reshape like yours (not changing the size of the array, > just changing the configuration) starts the sequence is something like: > > - make sure reshape doesn't progress at all (set md/sync_max to zero) > - tell the kernel about the new shape of the array > - start the reshape (this won't make any progress, but will update the > metadata) > Start: > - suspend user-space writes to the next few stripes > - read the next few stripes and write to the backup file > - tell the kernel that it is allowed to progress to the end of those > 'few stripes' > - wait for the kernel to do that > - invalidate the backup > - resume user-space writes to those next few stripes > - goto Start > > (the process is actually 'double-buffered' so it is more complex, but > this gives the idea close enough) > > If the system crashes or is shut down, on restart the kernel cannot know > if the "next few stripes" started reshaping or not, so it depends on > mdadm to load the backup file, check if there is valid data, and write > it out. > > I suspect that part of the problem is that mdadm --grow doesn't initialize the > backup file in quite the right way, so when mdadm --assemble looks at it > it doesn't see "Nothing has been written yet" but instead sees > "confusion" and gives up. > > If you --stop and then run the same --assemble command, including the > --backup, but this time add --invalid-backup (a bit like Wol > suggested) it should assemble and restart the reshape. --invalid-backup > tells mdadm "I know the backup file is invalid, I know that means there > could be inconsistent data which won't be restored, but I know what is > going on and I'm willing to take that risk. Just don't restore anything, > it'll be find. Really". > > I don't actually recommend doing that though. > > It would be better to revert the current reshape and start again with no > --backup file. This will use the new mechanism of changing the "Data > Offset" which is easier to work with and should be faster. > > If you have the very latest mdadm (3.4) you can add > --update=revert-reshape together with --invalid-backup and in your case > this will cancel the reshape and let you start again. > > You can test this out fairly safely if you want to. > > mkdir /tmp/foo > mdadm --dump /tmp/foo /dev/.... list of all devices in the array > > This will create sparse files in /tmp/foo containing just the md > metadata from those devices. Use "losetup /dev/loop0 /tmp/foo/sdb1" etc > to create loop-back device for all those files (there are multiple hard > links to each file - just choose 1 each). > Then you can experiment with mdadm on those /dev/loopXX files to see > what happens. > > Once you have the array reverted, you can start a new --grow, but don't > specify a --backup file. That should DoTheRightThing. > > This still leaves the question of why it didn't start a reshape in the > first place. If someone would like to experiment (probably with > loop-back files) and produce a test case that reliably (or even just > occasionally) hangs, then I'm happy to have a look at it. > > It also doesn't answer the question of why mdadm doesn't create the > backup file in a format that it knows is safe to ignore. Maybe someone > could look into that. > > > Good luck :-) > > NeilBrown -- To unsubscribe from this list: send the line "unsubscribe linux-raid" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
