All of lore.kernel.org
 help / color / mirror / Atom feed
From: jbrassow@sourceware.org <jbrassow@sourceware.org>
To: lvm-devel@redhat.com
Subject: LVM2 ./WHATS_NEW_DM lib/mirror/mirrored.c
Date: 13 Aug 2009 19:36:05 -0000	[thread overview]
Message-ID: <20090813193605.11299.qmail@sourceware.org> (raw)

CVSROOT:	/cvs/lvm2
Module name:	LVM2
Changes by:	jbrassow at sourceware.org	2009-08-13 19:36:04

Modified files:
	.              : WHATS_NEW_DM 
	lib/mirror     : mirrored.c 

Log message:
	mirror table generating code:  Properly handle 'block_on_errors' and 'cluster' features
	
	The device-mapper mirror CTR table has been changing over time.  This has
	now been corrected to handle the old and new methods for invoking the
	'block_on_errors' and 'cluster' features.  (The code that does this was
	accidentally committed in the previous check-in.  This check-in finishes
	the job.)

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/WHATS_NEW_DM.diff?cvsroot=lvm2&r1=1.293&r2=1.294
http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/lib/mirror/mirrored.c.diff?cvsroot=lvm2&r1=1.62&r2=1.63

--- LVM2/WHATS_NEW_DM	2009/08/06 19:32:26	1.293
+++ LVM2/WHATS_NEW_DM	2009/08/13 19:36:04	1.294
@@ -1,5 +1,6 @@
 Version 1.02.37 -
 =================================
+  Fix mirror table CTR code to handle 'block_on_error' and 'cluster' features
 
 Version 1.02.36 - 6th August 2009
 =================================
--- LVM2/lib/mirror/mirrored.c	2009/07/09 11:29:00	1.62
+++ LVM2/lib/mirror/mirrored.c	2009/08/13 19:36:04	1.63
@@ -362,14 +362,22 @@
 		_mirrored_present = target_present(cmd, "mirror", 1);
 
 		/*
-		 * block_on_error available with mirror target >= 1.1 and <= 1.11
+		 * block_on_error available as "block_on_error" log
+		 * argument with mirror target >= 1.1 and <= 1.11
 		 * or with 1.0 in RHEL4U3 driver >= 4.5
+		 *
+		 * block_on_error available as "handle_errors" mirror
+		 * argument with mirror target >= 1.12.
+		 *
+		 * libdm-deptree.c is smart enough to handle the differences
+		 * between block_on_error and handle_errors for all
+		 * mirror target versions >= 1.1
 		 */
 		/* FIXME Move this into libdevmapper */
 
 		if (target_version("mirror", &maj, &min, &patchlevel) &&
 		    maj == 1 &&
-		    ((min >= 1 && min <= 11) ||
+		    ((min >= 1) ||
 		     (min == 0 && driver_version(vsn, sizeof(vsn)) &&
 		      sscanf(vsn, "%u.%u.%u", &maj2, &min2, &patchlevel2) == 3 &&
 		      maj2 == 4 && min2 == 5 && patchlevel2 == 0)))	/* RHEL4U3 */



                 reply	other threads:[~2009-08-13 19:36 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20090813193605.11299.qmail@sourceware.org \
    --to=jbrassow@sourceware.org \
    --cc=lvm-devel@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.