From: NeilBrown <neilb@suse.de>
Date: Wed, 21 May 2014 04:03:48 +0000 (+1000)
Subject: DDF: remove some pointless code in validate_geometry
X-Git-Tag: mdadm-3.3.1~15
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=2ecda5a3fa0382c615aebe11111c671357137378;p=thirdparty%2Fmdadm.git

DDF: remove some pointless code in validate_geometry

I'm not sure what this was supposed to do, but it isn't needed
as creating on a container and on individual devices (in a container)
work fine already.

Signed-off-by: NeilBrown <neilb@suse.de>
---

diff --git a/super-ddf.c b/super-ddf.c
index 6368bc43..f9121995 100644
--- a/super-ddf.c
+++ b/super-ddf.c
@@ -3435,16 +3435,8 @@ static int validate_geometry_ddf(struct supertype *st,
 	 */
 	fd = open(dev, O_RDONLY|O_EXCL, 0);
 	if (fd >= 0) {
-		sra = sysfs_read(fd, NULL, GET_VERSION);
 		close(fd);
-		if (sra && sra->array.major_version == -1 &&
-		    strcmp(sra->text_version, "ddf") == 0) {
-			/* load super */
-			/* find space for 'n' devices. */
-			/* remember the devices */
-			/* Somehow return the fact that we have enough */
-		}
-
+		/* Just a bare device, no good to us */
 		if (verbose)
 			pr_err("ddf: Cannot create this array "
 			       "on device %s - a container is required.\n",