[lvm-devel] [PATCH 1 of 3] Handle failures of mirror devices while under snapshot origin (bug 613829)
Jonathan Brassow
jbrassow at redhat.com
Mon Aug 16 19:20:40 UTC 2010
On Aug 16, 2010, at 10:29 AM, Alasdair G Kergon wrote:
> On Sun, Aug 15, 2010 at 05:28:45PM -0500, Jon Brassow wrote:
>> Index: LVM2/lib/activate/activate.c
>> ===================================================================
>> --- LVM2.orig/lib/activate/activate.c
>> +++ LVM2/lib/activate/activate.c
>> @@ -772,7 +772,8 @@ int monitor_dev_for_events(struct cmd_co
>> if (!monitor_dev_for_events(cmd, dm_list_struct_base(snh,
>> struct lv_segment, origin_list)->cow, monitor))
>> r = 0;
>> - return r;
>> + if (!lv_is_mirrored(lv))
>> + return r;
>> }
>
> Please update the comment above this function too.
done
>
>> Index: LVM2/lib/mirror/mirrored.c
>> ===================================================================
>> --- LVM2.orig/lib/mirror/mirrored.c
>> +++ LVM2/lib/mirror/mirrored.c
>> @@ -631,18 +631,16 @@ static int _target_set_events(struct lv_
>> int evmask __attribute__((unused)), int set)
>
>> - if (!(uuid = build_dm_uuid(vg->cmd->mem, lv->lvid.s, NULL)))
>> + if (!(uuid = build_dm_uuid(vg->cmd->mem, lv->lvid.s, suffix)))
>> return_0;
>
> Does the other instance of build_dm_uuid() in _target_monitored()
> need a similar change?
yes, I think you are right.
I'll send updated patch for this.
brassow
More information about the lvm-devel
mailing list