public inbox for cluster-cvs@sourceware.org
help / color / mirror / Atom feed
* RHEL5 - dm-log-clustered.ko: Fix bug 464481 - 'block_on_error' flag should not be required
@ 2008-09-29 17:11 Jonathan Brassow
  0 siblings, 0 replies; only message in thread
From: Jonathan Brassow @ 2008-09-29 17:11 UTC (permalink / raw)
  To: cluster-cvs-relay

Gitweb:        http://git.fedorahosted.org/git/cluster.git?p=cluster.git;a=commitdiff;h=a5af1f999f1b36a46bcd83f71856852be47a85ac
Commit:        a5af1f999f1b36a46bcd83f71856852be47a85ac
Parent:        c387199bca6c04c79b1224638a8474e9bec53dbb
Author:        Jonathan Brassow <jbrassow@redhat.com>
AuthorDate:    Mon Sep 29 09:44:39 2008 -0500
Committer:     Jonathan Brassow <jbrassow@redhat.com>
CommitterDate: Mon Sep 29 09:47:20 2008 -0500

dm-log-clustered.ko: Fix bug 464481 - 'block_on_error' flag should not be required

Requiring 'block_on_error' flag to the log constructor table breaks pvmove.
---
 cmirror-kernel/src/dm-clog.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/cmirror-kernel/src/dm-clog.c b/cmirror-kernel/src/dm-clog.c
index 1e2b80e..f21823e 100644
--- a/cmirror-kernel/src/dm-clog.c
+++ b/cmirror-kernel/src/dm-clog.c
@@ -172,7 +172,7 @@ static int cluster_ctr(struct dm_dirty_log *log, struct dm_target *ti,
  * @argv
  *
  * argv contains:
- *   <region_size> <uuid> [[no]sync] <block_on_error>
+ *   <region_size> <uuid> [[no]sync] [block_on_error]
  *
  * Returns: 0 on success, -XXX on failure
  */
@@ -180,7 +180,7 @@ static int cluster_core_ctr(struct dm_dirty_log *log, struct dm_target *ti,
 			    unsigned int argc, char **argv)
 {
 	int i, r;
-	if ((argc < 3) || (argc > 4)) {
+	if ((argc < 2) || (argc > 4)) {
 		DMERR("Too %s arguments to clustered-core mirror log type.",
 		      (argc < 2) ? "few" : "many");
 		DMERR("  %d arguments supplied:", argc);
@@ -203,7 +203,7 @@ static int cluster_core_ctr(struct dm_dirty_log *log, struct dm_target *ti,
  * @argv
  *
  * argv contains:
- *   <disk> <region_size> <uuid> [[no]sync] <block_on_error>
+ *   <disk> <region_size> <uuid> [[no]sync] [block_on_error]
  *
  * Returns: 0 on success, -XXX on failure
  */
@@ -213,7 +213,7 @@ static int cluster_disk_ctr(struct dm_dirty_log *log, struct dm_target *ti,
 	int r, i;
 	struct dm_dev *dev;
 
-	if ((argc < 4) || (argc > 5)) {
+	if ((argc < 3) || (argc > 5)) {
 		DMERR("Too %s arguments to clustered-disk mirror log type.",
 		      (argc < 3) ? "few" : "many");
 		DMERR("  %d arguments supplied:", argc);


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2008-09-29 14:48 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-09-29 17:11 RHEL5 - dm-log-clustered.ko: Fix bug 464481 - 'block_on_error' flag should not be required Jonathan Brassow

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).