Re: [PATCH] fix: correct metadata's update communication

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 18 Nov 2011 15:34:33 +0100 Lukasz Dorau <lukasz.dorau@xxxxxxxxx>
wrote:

> The problem occurs when array under migration is assembled incrementally.
> st->update_tail is not initialized in function
> assemble_container_content() and during reshape
> the checkpoint information in metadata is not being updated.
> 
> The value of st->update_tail is now initialized in function
> assemble_container_content() and during reshape the checkpoint
> information in metadata is being updated correctly on all disks.
> 
> Signed-off-by: Lukasz Dorau <lukasz.dorau@xxxxxxxxx>
> ---
>  Assemble.c |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
> 
> diff --git a/Assemble.c b/Assemble.c
> index 3c8e74d..74708d1 100644
> --- a/Assemble.c
> +++ b/Assemble.c
> @@ -1585,6 +1585,9 @@ int assemble_container_content(struct supertype *st, int mdfd,
>  				if (!mdmon_running(st->container_dev))
>  					start_mdmon(st->container_dev);
>  				ping_monitor_by_id(st->container_dev);
> +				if (mdmon_running(st->container_dev) &&
> +						st->update_tail == NULL)
> +					st->update_tail = &st->updates;
>  			}
>  
>  			err = Grow_continue(mdfd, st, content, backup_file,


This and the previous patch ( [PATCH] imsm: fix: correct printing value of
blocks per migration unit) applied - thanks.

NeilBrown

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux RAID Wiki]     [ATA RAID]     [Linux SCSI Target Infrastructure]     [Linux Block]     [Linux IDE]     [Linux SCSI]     [Linux Hams]     [Device Mapper]     [Device Mapper Cryptographics]     [Kernel]     [Linux Admin]     [Linux Net]     [GFS]     [RPM]     [git]     [Yosemite Forum]


  Powered by Linux