1*86d7f5d3SJohn Marino /* $NetBSD: merge.c,v 1.1.1.1 2008/12/22 00:18:07 haad Exp $ */
2*86d7f5d3SJohn Marino
3*86d7f5d3SJohn Marino /*
4*86d7f5d3SJohn Marino * Copyright (C) 2001-2004 Sistina Software, Inc. All rights reserved.
5*86d7f5d3SJohn Marino * Copyright (C) 2004-2007 Red Hat, Inc. All rights reserved.
6*86d7f5d3SJohn Marino *
7*86d7f5d3SJohn Marino * This file is part of LVM2.
8*86d7f5d3SJohn Marino *
9*86d7f5d3SJohn Marino * This copyrighted material is made available to anyone wishing to use,
10*86d7f5d3SJohn Marino * modify, copy, or redistribute it subject to the terms and conditions
11*86d7f5d3SJohn Marino * of the GNU Lesser General Public License v.2.1.
12*86d7f5d3SJohn Marino *
13*86d7f5d3SJohn Marino * You should have received a copy of the GNU Lesser General Public License
14*86d7f5d3SJohn Marino * along with this program; if not, write to the Free Software Foundation,
15*86d7f5d3SJohn Marino * Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
16*86d7f5d3SJohn Marino */
17*86d7f5d3SJohn Marino
18*86d7f5d3SJohn Marino #include "lib.h"
19*86d7f5d3SJohn Marino #include "metadata.h"
20*86d7f5d3SJohn Marino #include "toolcontext.h"
21*86d7f5d3SJohn Marino #include "lv_alloc.h"
22*86d7f5d3SJohn Marino #include "pv_alloc.h"
23*86d7f5d3SJohn Marino #include "str_list.h"
24*86d7f5d3SJohn Marino #include "segtype.h"
25*86d7f5d3SJohn Marino
26*86d7f5d3SJohn Marino /*
27*86d7f5d3SJohn Marino * Attempt to merge two adjacent segments.
28*86d7f5d3SJohn Marino * Currently only supports striped segments on AREA_PV.
29*86d7f5d3SJohn Marino * Returns success if successful, in which case 'first'
30*86d7f5d3SJohn Marino * gets adjusted to contain both areas.
31*86d7f5d3SJohn Marino */
_merge(struct lv_segment * first,struct lv_segment * second)32*86d7f5d3SJohn Marino static int _merge(struct lv_segment *first, struct lv_segment *second)
33*86d7f5d3SJohn Marino {
34*86d7f5d3SJohn Marino if (!first || !second || first->segtype != second->segtype ||
35*86d7f5d3SJohn Marino !first->segtype->ops->merge_segments) return 0;
36*86d7f5d3SJohn Marino
37*86d7f5d3SJohn Marino return first->segtype->ops->merge_segments(first, second);
38*86d7f5d3SJohn Marino }
39*86d7f5d3SJohn Marino
lv_merge_segments(struct logical_volume * lv)40*86d7f5d3SJohn Marino int lv_merge_segments(struct logical_volume *lv)
41*86d7f5d3SJohn Marino {
42*86d7f5d3SJohn Marino struct dm_list *segh, *t;
43*86d7f5d3SJohn Marino struct lv_segment *current, *prev = NULL;
44*86d7f5d3SJohn Marino
45*86d7f5d3SJohn Marino if (lv->status & LOCKED || lv->status & PVMOVE)
46*86d7f5d3SJohn Marino return 1;
47*86d7f5d3SJohn Marino
48*86d7f5d3SJohn Marino dm_list_iterate_safe(segh, t, &lv->segments) {
49*86d7f5d3SJohn Marino current = dm_list_item(segh, struct lv_segment);
50*86d7f5d3SJohn Marino
51*86d7f5d3SJohn Marino if (_merge(prev, current))
52*86d7f5d3SJohn Marino dm_list_del(¤t->list);
53*86d7f5d3SJohn Marino else
54*86d7f5d3SJohn Marino prev = current;
55*86d7f5d3SJohn Marino }
56*86d7f5d3SJohn Marino
57*86d7f5d3SJohn Marino return 1;
58*86d7f5d3SJohn Marino }
59*86d7f5d3SJohn Marino
60*86d7f5d3SJohn Marino /*
61*86d7f5d3SJohn Marino * Verify that an LV's segments are consecutive, complete and don't overlap.
62*86d7f5d3SJohn Marino */
check_lv_segments(struct logical_volume * lv,int complete_vg)63*86d7f5d3SJohn Marino int check_lv_segments(struct logical_volume *lv, int complete_vg)
64*86d7f5d3SJohn Marino {
65*86d7f5d3SJohn Marino struct lv_segment *seg, *seg2;
66*86d7f5d3SJohn Marino uint32_t le = 0;
67*86d7f5d3SJohn Marino unsigned seg_count = 0, seg_found;
68*86d7f5d3SJohn Marino int r = 1;
69*86d7f5d3SJohn Marino uint32_t area_multiplier, s;
70*86d7f5d3SJohn Marino struct seg_list *sl;
71*86d7f5d3SJohn Marino
72*86d7f5d3SJohn Marino dm_list_iterate_items(seg, &lv->segments) {
73*86d7f5d3SJohn Marino seg_count++;
74*86d7f5d3SJohn Marino if (seg->le != le) {
75*86d7f5d3SJohn Marino log_error("LV %s invalid: segment %u should begin at "
76*86d7f5d3SJohn Marino "LE %" PRIu32 " (found %" PRIu32 ").",
77*86d7f5d3SJohn Marino lv->name, seg_count, le, seg->le);
78*86d7f5d3SJohn Marino r = 0;
79*86d7f5d3SJohn Marino }
80*86d7f5d3SJohn Marino
81*86d7f5d3SJohn Marino area_multiplier = segtype_is_striped(seg->segtype) ?
82*86d7f5d3SJohn Marino seg->area_count : 1;
83*86d7f5d3SJohn Marino
84*86d7f5d3SJohn Marino if (seg->area_len * area_multiplier != seg->len) {
85*86d7f5d3SJohn Marino log_error("LV %s: segment %u has inconsistent "
86*86d7f5d3SJohn Marino "area_len %u",
87*86d7f5d3SJohn Marino lv->name, seg_count, seg->area_len);
88*86d7f5d3SJohn Marino r = 0;
89*86d7f5d3SJohn Marino }
90*86d7f5d3SJohn Marino
91*86d7f5d3SJohn Marino if (complete_vg && seg->log_lv) {
92*86d7f5d3SJohn Marino if (!seg_is_mirrored(seg)) {
93*86d7f5d3SJohn Marino log_error("LV %s: segment %u has log LV but "
94*86d7f5d3SJohn Marino "is not mirrored",
95*86d7f5d3SJohn Marino lv->name, seg_count);
96*86d7f5d3SJohn Marino r = 0;
97*86d7f5d3SJohn Marino }
98*86d7f5d3SJohn Marino
99*86d7f5d3SJohn Marino if (!(seg->log_lv->status & MIRROR_LOG)) {
100*86d7f5d3SJohn Marino log_error("LV %s: segment %u log LV %s is not "
101*86d7f5d3SJohn Marino "a mirror log",
102*86d7f5d3SJohn Marino lv->name, seg_count, seg->log_lv->name);
103*86d7f5d3SJohn Marino r = 0;
104*86d7f5d3SJohn Marino }
105*86d7f5d3SJohn Marino
106*86d7f5d3SJohn Marino if (!(seg2 = first_seg(seg->log_lv)) ||
107*86d7f5d3SJohn Marino find_mirror_seg(seg2) != seg) {
108*86d7f5d3SJohn Marino log_error("LV %s: segment %u log LV does not "
109*86d7f5d3SJohn Marino "point back to mirror segment",
110*86d7f5d3SJohn Marino lv->name, seg_count);
111*86d7f5d3SJohn Marino r = 0;
112*86d7f5d3SJohn Marino }
113*86d7f5d3SJohn Marino }
114*86d7f5d3SJohn Marino
115*86d7f5d3SJohn Marino if (complete_vg && seg->status & MIRROR_IMAGE) {
116*86d7f5d3SJohn Marino if (!find_mirror_seg(seg) ||
117*86d7f5d3SJohn Marino !seg_is_mirrored(find_mirror_seg(seg))) {
118*86d7f5d3SJohn Marino log_error("LV %s: segment %u mirror image "
119*86d7f5d3SJohn Marino "is not mirrored",
120*86d7f5d3SJohn Marino lv->name, seg_count);
121*86d7f5d3SJohn Marino r = 0;
122*86d7f5d3SJohn Marino }
123*86d7f5d3SJohn Marino }
124*86d7f5d3SJohn Marino
125*86d7f5d3SJohn Marino if (seg_is_snapshot(seg)) {
126*86d7f5d3SJohn Marino if (seg->cow && seg->cow == seg->origin) {
127*86d7f5d3SJohn Marino log_error("LV %s: segment %u has same LV %s for "
128*86d7f5d3SJohn Marino "both origin and snapshot",
129*86d7f5d3SJohn Marino lv->name, seg_count, seg->cow->name);
130*86d7f5d3SJohn Marino r = 0;
131*86d7f5d3SJohn Marino }
132*86d7f5d3SJohn Marino }
133*86d7f5d3SJohn Marino
134*86d7f5d3SJohn Marino for (s = 0; s < seg->area_count; s++) {
135*86d7f5d3SJohn Marino if (seg_type(seg, s) == AREA_UNASSIGNED) {
136*86d7f5d3SJohn Marino log_error("LV %s: segment %u has unassigned "
137*86d7f5d3SJohn Marino "area %u.",
138*86d7f5d3SJohn Marino lv->name, seg_count, s);
139*86d7f5d3SJohn Marino r = 0;
140*86d7f5d3SJohn Marino } else if (seg_type(seg, s) == AREA_PV) {
141*86d7f5d3SJohn Marino if (!seg_pvseg(seg, s) ||
142*86d7f5d3SJohn Marino seg_pvseg(seg, s)->lvseg != seg ||
143*86d7f5d3SJohn Marino seg_pvseg(seg, s)->lv_area != s) {
144*86d7f5d3SJohn Marino log_error("LV %s: segment %u has "
145*86d7f5d3SJohn Marino "inconsistent PV area %u",
146*86d7f5d3SJohn Marino lv->name, seg_count, s);
147*86d7f5d3SJohn Marino r = 0;
148*86d7f5d3SJohn Marino }
149*86d7f5d3SJohn Marino } else {
150*86d7f5d3SJohn Marino if (!seg_lv(seg, s) ||
151*86d7f5d3SJohn Marino seg_lv(seg, s)->vg != lv->vg ||
152*86d7f5d3SJohn Marino seg_lv(seg, s) == lv) {
153*86d7f5d3SJohn Marino log_error("LV %s: segment %u has "
154*86d7f5d3SJohn Marino "inconsistent LV area %u",
155*86d7f5d3SJohn Marino lv->name, seg_count, s);
156*86d7f5d3SJohn Marino r = 0;
157*86d7f5d3SJohn Marino }
158*86d7f5d3SJohn Marino
159*86d7f5d3SJohn Marino if (complete_vg && seg_lv(seg, s) &&
160*86d7f5d3SJohn Marino (seg_lv(seg, s)->status & MIRROR_IMAGE) &&
161*86d7f5d3SJohn Marino (!(seg2 = find_seg_by_le(seg_lv(seg, s),
162*86d7f5d3SJohn Marino seg_le(seg, s))) ||
163*86d7f5d3SJohn Marino find_mirror_seg(seg2) != seg)) {
164*86d7f5d3SJohn Marino log_error("LV %s: segment %u mirror "
165*86d7f5d3SJohn Marino "image %u missing mirror ptr",
166*86d7f5d3SJohn Marino lv->name, seg_count, s);
167*86d7f5d3SJohn Marino r = 0;
168*86d7f5d3SJohn Marino }
169*86d7f5d3SJohn Marino
170*86d7f5d3SJohn Marino /* FIXME I don't think this ever holds?
171*86d7f5d3SJohn Marino if (seg_le(seg, s) != le) {
172*86d7f5d3SJohn Marino log_error("LV %s: segment %u has "
173*86d7f5d3SJohn Marino "inconsistent LV area %u "
174*86d7f5d3SJohn Marino "size",
175*86d7f5d3SJohn Marino lv->name, seg_count, s);
176*86d7f5d3SJohn Marino r = 0;
177*86d7f5d3SJohn Marino }
178*86d7f5d3SJohn Marino */
179*86d7f5d3SJohn Marino seg_found = 0;
180*86d7f5d3SJohn Marino dm_list_iterate_items(sl, &seg_lv(seg, s)->segs_using_this_lv)
181*86d7f5d3SJohn Marino if (sl->seg == seg)
182*86d7f5d3SJohn Marino seg_found++;
183*86d7f5d3SJohn Marino if (!seg_found) {
184*86d7f5d3SJohn Marino log_error("LV %s segment %d uses LV %s,"
185*86d7f5d3SJohn Marino " but missing ptr from %s to %s",
186*86d7f5d3SJohn Marino lv->name, seg_count,
187*86d7f5d3SJohn Marino seg_lv(seg, s)->name,
188*86d7f5d3SJohn Marino seg_lv(seg, s)->name, lv->name);
189*86d7f5d3SJohn Marino r = 0;
190*86d7f5d3SJohn Marino } else if (seg_found > 1) {
191*86d7f5d3SJohn Marino log_error("LV %s has duplicated links "
192*86d7f5d3SJohn Marino "to LV %s segment %d",
193*86d7f5d3SJohn Marino seg_lv(seg, s)->name,
194*86d7f5d3SJohn Marino lv->name, seg_count);
195*86d7f5d3SJohn Marino r = 0;
196*86d7f5d3SJohn Marino }
197*86d7f5d3SJohn Marino }
198*86d7f5d3SJohn Marino }
199*86d7f5d3SJohn Marino
200*86d7f5d3SJohn Marino le += seg->len;
201*86d7f5d3SJohn Marino }
202*86d7f5d3SJohn Marino
203*86d7f5d3SJohn Marino dm_list_iterate_items(sl, &lv->segs_using_this_lv) {
204*86d7f5d3SJohn Marino seg = sl->seg;
205*86d7f5d3SJohn Marino seg_found = 0;
206*86d7f5d3SJohn Marino for (s = 0; s < seg->area_count; s++) {
207*86d7f5d3SJohn Marino if (seg_type(seg, s) != AREA_LV)
208*86d7f5d3SJohn Marino continue;
209*86d7f5d3SJohn Marino if (lv == seg_lv(seg, s))
210*86d7f5d3SJohn Marino seg_found++;
211*86d7f5d3SJohn Marino }
212*86d7f5d3SJohn Marino if (seg->log_lv == lv)
213*86d7f5d3SJohn Marino seg_found++;
214*86d7f5d3SJohn Marino if (!seg_found) {
215*86d7f5d3SJohn Marino log_error("LV %s is used by LV %s:%" PRIu32 "-%" PRIu32
216*86d7f5d3SJohn Marino ", but missing ptr from %s to %s",
217*86d7f5d3SJohn Marino lv->name, seg->lv->name, seg->le,
218*86d7f5d3SJohn Marino seg->le + seg->len - 1,
219*86d7f5d3SJohn Marino seg->lv->name, lv->name);
220*86d7f5d3SJohn Marino r = 0;
221*86d7f5d3SJohn Marino } else if (seg_found != sl->count) {
222*86d7f5d3SJohn Marino log_error("Reference count mismatch: LV %s has %d "
223*86d7f5d3SJohn Marino "links to LV %s:%" PRIu32 "-%" PRIu32
224*86d7f5d3SJohn Marino ", which has %d links",
225*86d7f5d3SJohn Marino lv->name, sl->count, seg->lv->name, seg->le,
226*86d7f5d3SJohn Marino seg->le + seg->len - 1, seg_found);
227*86d7f5d3SJohn Marino r = 0;
228*86d7f5d3SJohn Marino }
229*86d7f5d3SJohn Marino
230*86d7f5d3SJohn Marino seg_found = 0;
231*86d7f5d3SJohn Marino dm_list_iterate_items(seg2, &seg->lv->segments)
232*86d7f5d3SJohn Marino if (sl->seg == seg2) {
233*86d7f5d3SJohn Marino seg_found++;
234*86d7f5d3SJohn Marino break;
235*86d7f5d3SJohn Marino }
236*86d7f5d3SJohn Marino if (!seg_found) {
237*86d7f5d3SJohn Marino log_error("LV segment %s:%" PRIu32 "-%" PRIu32
238*86d7f5d3SJohn Marino "is incorrectly listed as being used by LV %s",
239*86d7f5d3SJohn Marino seg->lv->name, seg->le, seg->le + seg->len - 1,
240*86d7f5d3SJohn Marino lv->name);
241*86d7f5d3SJohn Marino r = 0;
242*86d7f5d3SJohn Marino }
243*86d7f5d3SJohn Marino }
244*86d7f5d3SJohn Marino
245*86d7f5d3SJohn Marino if (le != lv->le_count) {
246*86d7f5d3SJohn Marino log_error("LV %s: inconsistent LE count %u != %u",
247*86d7f5d3SJohn Marino lv->name, le, lv->le_count);
248*86d7f5d3SJohn Marino r = 0;
249*86d7f5d3SJohn Marino }
250*86d7f5d3SJohn Marino
251*86d7f5d3SJohn Marino return r;
252*86d7f5d3SJohn Marino }
253*86d7f5d3SJohn Marino
254*86d7f5d3SJohn Marino /*
255*86d7f5d3SJohn Marino * Split the supplied segment at the supplied logical extent
256*86d7f5d3SJohn Marino * NB Use LE numbering that works across stripes PV1: 0,2,4 PV2: 1,3,5 etc.
257*86d7f5d3SJohn Marino */
_lv_split_segment(struct logical_volume * lv,struct lv_segment * seg,uint32_t le)258*86d7f5d3SJohn Marino static int _lv_split_segment(struct logical_volume *lv, struct lv_segment *seg,
259*86d7f5d3SJohn Marino uint32_t le)
260*86d7f5d3SJohn Marino {
261*86d7f5d3SJohn Marino struct lv_segment *split_seg;
262*86d7f5d3SJohn Marino uint32_t s;
263*86d7f5d3SJohn Marino uint32_t offset = le - seg->le;
264*86d7f5d3SJohn Marino uint32_t area_offset;
265*86d7f5d3SJohn Marino
266*86d7f5d3SJohn Marino if (!seg_can_split(seg)) {
267*86d7f5d3SJohn Marino log_error("Unable to split the %s segment at LE %" PRIu32
268*86d7f5d3SJohn Marino " in LV %s", seg->segtype->name, le, lv->name);
269*86d7f5d3SJohn Marino return 0;
270*86d7f5d3SJohn Marino }
271*86d7f5d3SJohn Marino
272*86d7f5d3SJohn Marino /* Clone the existing segment */
273*86d7f5d3SJohn Marino if (!(split_seg = alloc_lv_segment(lv->vg->cmd->mem, seg->segtype,
274*86d7f5d3SJohn Marino seg->lv, seg->le, seg->len,
275*86d7f5d3SJohn Marino seg->status, seg->stripe_size,
276*86d7f5d3SJohn Marino seg->log_lv,
277*86d7f5d3SJohn Marino seg->area_count, seg->area_len,
278*86d7f5d3SJohn Marino seg->chunk_size, seg->region_size,
279*86d7f5d3SJohn Marino seg->extents_copied))) {
280*86d7f5d3SJohn Marino log_error("Couldn't allocate cloned LV segment.");
281*86d7f5d3SJohn Marino return 0;
282*86d7f5d3SJohn Marino }
283*86d7f5d3SJohn Marino
284*86d7f5d3SJohn Marino if (!str_list_dup(lv->vg->cmd->mem, &split_seg->tags, &seg->tags)) {
285*86d7f5d3SJohn Marino log_error("LV segment tags duplication failed");
286*86d7f5d3SJohn Marino return 0;
287*86d7f5d3SJohn Marino }
288*86d7f5d3SJohn Marino
289*86d7f5d3SJohn Marino /* In case of a striped segment, the offset has to be / stripes */
290*86d7f5d3SJohn Marino area_offset = offset;
291*86d7f5d3SJohn Marino if (seg_is_striped(seg))
292*86d7f5d3SJohn Marino area_offset /= seg->area_count;
293*86d7f5d3SJohn Marino
294*86d7f5d3SJohn Marino split_seg->area_len -= area_offset;
295*86d7f5d3SJohn Marino seg->area_len = area_offset;
296*86d7f5d3SJohn Marino
297*86d7f5d3SJohn Marino split_seg->len -= offset;
298*86d7f5d3SJohn Marino seg->len = offset;
299*86d7f5d3SJohn Marino
300*86d7f5d3SJohn Marino split_seg->le = seg->le + seg->len;
301*86d7f5d3SJohn Marino
302*86d7f5d3SJohn Marino /* Adjust the PV mapping */
303*86d7f5d3SJohn Marino for (s = 0; s < seg->area_count; s++) {
304*86d7f5d3SJohn Marino seg_type(split_seg, s) = seg_type(seg, s);
305*86d7f5d3SJohn Marino
306*86d7f5d3SJohn Marino /* Split area at the offset */
307*86d7f5d3SJohn Marino switch (seg_type(seg, s)) {
308*86d7f5d3SJohn Marino case AREA_LV:
309*86d7f5d3SJohn Marino if (!set_lv_segment_area_lv(split_seg, s, seg_lv(seg, s),
310*86d7f5d3SJohn Marino seg_le(seg, s) + seg->area_len, 0))
311*86d7f5d3SJohn Marino return_0;
312*86d7f5d3SJohn Marino log_debug("Split %s:%u[%u] at %u: %s LE %u", lv->name,
313*86d7f5d3SJohn Marino seg->le, s, le, seg_lv(seg, s)->name,
314*86d7f5d3SJohn Marino seg_le(split_seg, s));
315*86d7f5d3SJohn Marino break;
316*86d7f5d3SJohn Marino
317*86d7f5d3SJohn Marino case AREA_PV:
318*86d7f5d3SJohn Marino if (!(seg_pvseg(split_seg, s) =
319*86d7f5d3SJohn Marino assign_peg_to_lvseg(seg_pv(seg, s),
320*86d7f5d3SJohn Marino seg_pe(seg, s) +
321*86d7f5d3SJohn Marino seg->area_len,
322*86d7f5d3SJohn Marino seg_pvseg(seg, s)->len -
323*86d7f5d3SJohn Marino seg->area_len,
324*86d7f5d3SJohn Marino split_seg, s)))
325*86d7f5d3SJohn Marino return_0;
326*86d7f5d3SJohn Marino log_debug("Split %s:%u[%u] at %u: %s PE %u", lv->name,
327*86d7f5d3SJohn Marino seg->le, s, le,
328*86d7f5d3SJohn Marino dev_name(seg_dev(seg, s)),
329*86d7f5d3SJohn Marino seg_pe(split_seg, s));
330*86d7f5d3SJohn Marino break;
331*86d7f5d3SJohn Marino
332*86d7f5d3SJohn Marino case AREA_UNASSIGNED:
333*86d7f5d3SJohn Marino log_error("Unassigned area %u found in segment", s);
334*86d7f5d3SJohn Marino return 0;
335*86d7f5d3SJohn Marino }
336*86d7f5d3SJohn Marino }
337*86d7f5d3SJohn Marino
338*86d7f5d3SJohn Marino /* Add split off segment to the list _after_ the original one */
339*86d7f5d3SJohn Marino dm_list_add_h(&seg->list, &split_seg->list);
340*86d7f5d3SJohn Marino
341*86d7f5d3SJohn Marino return 1;
342*86d7f5d3SJohn Marino }
343*86d7f5d3SJohn Marino
344*86d7f5d3SJohn Marino /*
345*86d7f5d3SJohn Marino * Ensure there's a segment boundary at the given logical extent
346*86d7f5d3SJohn Marino */
lv_split_segment(struct logical_volume * lv,uint32_t le)347*86d7f5d3SJohn Marino int lv_split_segment(struct logical_volume *lv, uint32_t le)
348*86d7f5d3SJohn Marino {
349*86d7f5d3SJohn Marino struct lv_segment *seg;
350*86d7f5d3SJohn Marino
351*86d7f5d3SJohn Marino if (!(seg = find_seg_by_le(lv, le))) {
352*86d7f5d3SJohn Marino log_error("Segment with extent %" PRIu32 " in LV %s not found",
353*86d7f5d3SJohn Marino le, lv->name);
354*86d7f5d3SJohn Marino return 0;
355*86d7f5d3SJohn Marino }
356*86d7f5d3SJohn Marino
357*86d7f5d3SJohn Marino /* This is a segment start already */
358*86d7f5d3SJohn Marino if (le == seg->le)
359*86d7f5d3SJohn Marino return 1;
360*86d7f5d3SJohn Marino
361*86d7f5d3SJohn Marino if (!_lv_split_segment(lv, seg, le))
362*86d7f5d3SJohn Marino return_0;
363*86d7f5d3SJohn Marino
364*86d7f5d3SJohn Marino if (!vg_validate(lv->vg))
365*86d7f5d3SJohn Marino return_0;
366*86d7f5d3SJohn Marino
367*86d7f5d3SJohn Marino return 1;
368*86d7f5d3SJohn Marino }
369