[lvm-devel] LVM2 lib/metadata/thin_manip.c libdm/libdm-dep ...
zkabelac at sourceware.org
zkabelac at sourceware.org
Wed Oct 19 16:42:15 UTC 2011
CVSROOT: /cvs/lvm2
Module name: LVM2
Changes by: zkabelac at sourceware.org 2011-10-19 16:42:14
Modified files:
lib/metadata : thin_manip.c
libdm : libdm-deptree.c
Log message:
Update error message
Drop INTERNAL_ERROR from public API functions.
Improve some messages.
Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/lib/metadata/thin_manip.c.diff?cvsroot=lvm2&r1=1.11&r2=1.12
http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/libdm/libdm-deptree.c.diff?cvsroot=lvm2&r1=1.131&r2=1.132
--- LVM2/lib/metadata/thin_manip.c 2011/10/19 16:39:09 1.11
+++ LVM2/lib/metadata/thin_manip.c 2011/10/19 16:42:14 1.12
@@ -126,7 +126,7 @@
tmsg->u.delete_id = delete_id;
break;
default:
- log_error(INTERNAL_ERROR "Unsupported message type %d", type);
+ log_error(INTERNAL_ERROR "Unsupported message type %u.", type);
return 0;
}
--- LVM2/libdm/libdm-deptree.c 2011/10/19 16:40:59 1.131
+++ LVM2/libdm/libdm-deptree.c 2011/10/19 16:42:14 1.132
@@ -2880,13 +2880,13 @@
struct thin_message *tm;
if (node->props.segment_count != 1) {
- log_error(INTERNAL_ERROR "Attempt to use non thin pool segment.");
+ log_error("Thin pool node must have only one segment.");
return 0;
}
seg = dm_list_item(dm_list_last(&node->props.segs), struct load_segment);
if (seg->type != SEG_THIN_POOL) {
- log_error(INTERNAL_ERROR "Attempt to use non thin pool segment %s.",
+ log_error("Thin pool node has segment type %s.",
dm_segtypes[seg->type].target);
return 0;
}
@@ -2898,9 +2898,9 @@
switch (message->type) {
case DM_THIN_MESSAGE_CREATE_SNAP:
- /* Origin MUST be suspend! */
+ /* If the thin origin is active, it must be suspend first! */
if (message->u.m_create_snap.device_id == message->u.m_create_snap.origin_id) {
- log_error("Same origin used for thin snapshot.");
+ log_error("Cannot use same device id for origin and its snapshot.");
return 0;
}
if (!_thin_validate_device_id(message->u.m_create_snap.device_id) ||
More information about the lvm-devel
mailing list