1
0
mirror of git://sourceware.org/git/lvm2.git synced 2025-01-03 05:18:29 +03:00

pvresize: add missing return

Log error path missed return 0.
Also fix some unneded bactraces (since log_error already shows
position).
This commit is contained in:
Zdenek Kabelac 2018-06-05 21:21:26 +02:00
parent 0c62ae3f89
commit fb171edd45
2 changed files with 4 additions and 2 deletions

View File

@ -1,5 +1,6 @@
Version 3.0.0 Version 3.0.0
============= =============
Fix check if resized PV can also fit metadata area.
Avoid showing internal error in lvs output or pvmoved LVs. Avoid showing internal error in lvs output or pvmoved LVs.
Remove clvmd Remove clvmd
Remove lvmlib (api) Remove lvmlib (api)

View File

@ -566,6 +566,7 @@ static int _pv_resize(struct physical_volume *pv, struct volume_group *vg, uint6
log_error("Size must exceed physical extent start " log_error("Size must exceed physical extent start "
"of %" PRIu64 " sectors on PV %s.", "of %" PRIu64 " sectors on PV %s.",
pv_pe_start(pv), pv_dev_name(pv)); pv_pe_start(pv), pv_dev_name(pv));
return 0;
} }
old_pe_count = pv->pe_count; old_pe_count = pv->pe_count;
@ -645,7 +646,7 @@ int pv_resize_single(struct cmd_context *cmd,
pv_name, display_size(cmd, new_size), pv_name, display_size(cmd, new_size),
display_size(cmd, size)) == 'n') { display_size(cmd, size)) == 'n') {
log_error("Physical Volume %s not resized.", pv_name); log_error("Physical Volume %s not resized.", pv_name);
goto_out; goto out;
} }
} else if (new_size < size) } else if (new_size < size)
@ -653,7 +654,7 @@ int pv_resize_single(struct cmd_context *cmd,
pv_name, display_size(cmd, new_size), pv_name, display_size(cmd, new_size),
display_size(cmd, size)) == 'n') { display_size(cmd, size)) == 'n') {
log_error("Physical Volume %s not resized.", pv_name); log_error("Physical Volume %s not resized.", pv_name);
goto_out; goto out;
} }
if (new_size == size) if (new_size == size)