On Wed, Dec 02 2015, Song Liu <songliubraving@xxxxxx> wrote: > These patches try to recreate missing/faulty journal in mdadm. Since > the array is readonly with journal missing, it is necessary to > restart the array after recreating journal. > > Song Liu (3): > add sysfs_array_state to struct mdinfo > [mdadm] recreate journal in mdadm > make .spec file work I didn't get that third patch??? The patches generally look good, but it bothers me that you have to stop and restart the array for the added journal to work. Can we just teach the kernel to accept a new journal on an active array, and then use that functionality in mdadm? That is certainly what I want in the longer term and I'm not sure it makes sense to have a half-way "--add-journal" that doesn't do the whole job. Thanks, NeilBrown > > Assemble.c | 8 +++++++- > Manage.c | 32 +++++++++++++++++++++++++++++++- > ReadMe.c | 3 ++- > mdadm.c | 8 ++++++++ > mdadm.h | 5 +++++ > mdadm.spec | 8 ++++---- > super1.c | 3 ++- > sysfs.c | 7 +++++++ > udev-md-raid-assembly.rules | 2 +- > 9 files changed, 67 insertions(+), 9 deletions(-) > > -- > 2.4.6
Attachment:
signature.asc
Description: PGP signature
