public inbox for lvm2-cvs@sourceware.org
help / color / mirror / Atom feed
* LVM2 lib/metadata/thin_manip.c libdm/libdm-dep ...
@ 2011-10-30 22:52 zkabelac
0 siblings, 0 replies; 2+ messages in thread
From: zkabelac @ 2011-10-30 22:52 UTC (permalink / raw)
To: lvm-devel, lvm2-cvs
CVSROOT: /cvs/lvm2
Module name: LVM2
Changes by: zkabelac@sourceware.org 2011-10-30 22:52:08
Modified files:
lib/metadata : thin_manip.c
libdm : libdm-deptree.c
Log message:
Thin fix compile warns
Test for dm_snprintf < 0.
Add header for moved backup.
Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/lib/metadata/thin_manip.c.diff?cvsroot=lvm2&r1=1.16&r2=1.17
http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/libdm/libdm-deptree.c.diff?cvsroot=lvm2&r1=1.141&r2=1.142
--- LVM2/lib/metadata/thin_manip.c 2011/10/30 22:01:39 1.16
+++ LVM2/lib/metadata/thin_manip.c 2011/10/30 22:52:08 1.17
@@ -18,6 +18,7 @@
#include "metadata.h"
#include "segtype.h"
#include "lv_alloc.h"
+#include "archiver.h"
int attach_pool_metadata_lv(struct lv_segment *seg, struct logical_volume *pool_metadata_lv)
{
--- LVM2/libdm/libdm-deptree.c 2011/10/30 22:04:57 1.141
+++ LVM2/libdm/libdm-deptree.c 2011/10/30 22:52:08 1.142
@@ -1294,9 +1294,11 @@
m->u.m_set_transaction_id.current_id,
m->u.m_set_transaction_id.new_id);
break;
+ default:
+ r = -1;
}
- if (!r) {
+ if (r < 0) {
log_error("Failed to prepare message.");
return 0;
}
^ permalink raw reply [flat|nested] 2+ messages in thread
* LVM2 lib/metadata/thin_manip.c libdm/libdm-dep ...
@ 2011-10-19 16:42 zkabelac
0 siblings, 0 replies; 2+ messages in thread
From: zkabelac @ 2011-10-19 16:42 UTC (permalink / raw)
To: lvm-devel, lvm2-cvs
CVSROOT: /cvs/lvm2
Module name: LVM2
Changes by: zkabelac@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) ||
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-10-30 22:52 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-10-30 22:52 LVM2 lib/metadata/thin_manip.c libdm/libdm-dep zkabelac
-- strict thread matches above, loose matches on Subject: below --
2011-10-19 16:42 zkabelac
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).