CP-35551: remove blktap device handling

Signed-off-by: Mark Syms <[email protected]>

Author: MarkSymsCtx

control/tap-ctl-allocate.c

@@ -48,7 +48,7 @@
 #include <linux/major.h>
 
 #include "tap-ctl.h"
-#include "blktap2.h"
+#include "blktap.h"
 
 static int
 tap_ctl_prepare_directory(const char *dir)
@@ -71,7 +71,7 @@ tap_ctl_prepare_directory(const char *dir)
 		if (ptr)
 			*ptr = '\0';
 
-		err = mkdir(name, 0755);
+		err = mkdir(name, 0700);
 		if (err && errno != EEXIST) {
 			PERROR("mkdir %s", name);
 			err = errno;
@@ -94,165 +94,37 @@ tap_ctl_prepare_directory(const char *dir)
 }
 
 static int
-tap_ctl_make_device(const char *devname, const int major,
-		    const int minor, const int perm)
+tap_ctl_check_environment(void)
 {
 	int err;
-	char *copy, *dir;
-
-	copy = strdup(devname);
-	if (!copy)
-		return ENOMEM;
-
-	dir = dirname(copy);
-
-	err = tap_ctl_prepare_directory(dir);
-	free(copy);
-
-	if (err)
-		return err;
-
-	if (unlink(devname)) {
-		err = errno;
-		if (err != ENOENT) {
-			PERROR("unlink %s", devname);
-			EPRINTF("Unlink failed with %d\n", err);
-			return err;
-		}
-	}
 
-	err = mknod(devname, perm, makedev(major, minor));
+	err = tap_ctl_prepare_directory(BLKTAP2_CONTROL_DIR);
 	if (err) {
-		err = errno;
-		PERROR("mknod %s", devname);
-		EPRINTF("Mknod failed with %d\n", err);
+		EPRINTF("Prepare %s directory failed %d",
+				BLKTAP2_CONTROL_DIR, err);
 		return err;
 	}
 
-	return 0;
-}
-
-static int
-tap_ctl_check_environment(void)
-{
-	FILE *f;
-	int err, minor;
-	char name[257];
-
-	err = tap_ctl_prepare_directory(BLKTAP2_CONTROL_DIR);
+	err = tap_ctl_prepare_directory(BLKTAP2_NP_RUN_DIR);
 	if (err) {
-		EPRINTF("Prepare directory failed %d", err);
+		EPRINTF("Prepare %s directory failed %d",
+				BLKTAP2_NP_RUN_DIR, err);
 		return err;
 	}
 
-	f = fopen("/proc/misc", "r");
-	if (!f) {
-		EPRINTF("failed to open /proc/misc: %d\n", errno);
-		return errno;
-	}
-	/* There is not a lot we can do about an error returned
-	 * from flock() so don't check */
-	flock(fileno(f), LOCK_EX);
-
-	/* Note err is 0 owing to tap_ctl_prepare_directory() above */
-	if (!access(BLKTAP2_CONTROL_DEVICE, R_OK | W_OK))
-		goto out;
-
-	memset(name, 0, sizeof(name));
-
-	while (fscanf(f, "%d %256s", &minor, name) == 2)
-		if (!strcmp(name, BLKTAP2_CONTROL_NAME)) {
-			err = tap_ctl_make_device(BLKTAP2_CONTROL_DEVICE,
-						  MISC_MAJOR,
-						  minor, S_IFCHR | 0600);
-			if (err)
-				EPRINTF("tap_ctl_make_device failed\n");
-			goto out;
-		}
-
-	err = ENOSYS;
-	EPRINTF("didn't find %s in /proc/misc\n", BLKTAP2_CONTROL_NAME);
-
-out:
-	flock(fileno(f), LOCK_UN);
-	fclose(f);
 	return err;
 }
 
 static int
 tap_ctl_allocate_device(int *minor, char **devname)
 {
-	char *name;
-	int fd, err;
-	struct blktap2_handle handle;
-	char free_devname = 0;
-
 	*minor = -1;
 	if (!devname)
 		return EINVAL;
 
-	fd = open(BLKTAP2_CONTROL_DEVICE, O_RDONLY);
-	if (fd == -1) {
-		EPRINTF("failed to open control device: %d\n", errno);
-		return errno;
-	}
-
-	err = ioctl(fd, BLKTAP2_IOCTL_ALLOC_TAP, &handle);
-	close(fd);
-	if (err == -1) {
-		EPRINTF("failed to allocate new device: %d\n", errno);
-		return errno;
-	}
-
-	err = asprintf(&name, "%s%d", BLKTAP2_RING_DEVICE, handle.minor);
-	if (err == -1) {
-		err = ENOMEM;
-		goto fail;
-	}
-
-	err = tap_ctl_make_device(name, handle.ring,
-				  handle.minor, S_IFCHR | 0600);
-	free(name);
-	if (err) {
-		EPRINTF("creating ring device for %d failed: %d\n",
-			handle.minor, err);
-		goto fail;
-	}
-
-	if (*devname)
-		name = *devname;
-	else {
-		err = asprintf(&name, "%s%d",
-			       BLKTAP2_IO_DEVICE, handle.minor);
-		if (err == -1) {
-			err = ENOMEM;
-			goto fail;
-		}
-		*devname = name;
-		free_devname = 1;
-	}
-
-	err = tap_ctl_make_device(name, handle.device,
-				  handle.minor, S_IFBLK | 0600);
-	if (err) {
-		EPRINTF("creating IO device for %d failed: %d\n",
-			handle.minor, err);
-		goto fail;
-	}
-
-	DBG("new interface: ring: %u, device: %u, minor: %u\n",
-	    handle.ring, handle.device, handle.minor);
-
-	*minor = handle.minor;
+	/* TO-DO: get this from a file based resource */
+	*minor = 1;
 	return 0;
-
-fail:
-	if (free_devname) {
-		free(*devname);
-		*devname = 0;
-	}
-	tap_ctl_free(handle.minor);
-	return err;
 }
 
 int

control/tap-ctl-create.c

@@ -39,7 +39,7 @@
 #include <getopt.h>
 
 #include "tap-ctl.h"
-#include "blktap2.h"
+#include "blktap.h"
 
 int
 tap_ctl_create(const char *params, char **devname, int flags, int parent_minor,

control/tap-ctl-destroy.c

@@ -40,7 +40,7 @@
 #include <getopt.h>
 
 #include "tap-ctl.h"
-#include "blktap2.h"
+#include "blktap.h"
 
 int
 tap_ctl_destroy(const int id, const int minor,

control/tap-ctl-free.c

@@ -41,22 +41,24 @@
 #include <sys/ioctl.h>
 
 #include "tap-ctl.h"
-#include "blktap2.h"
+#include "blktap.h"
 
 int
 tap_ctl_free(const int minor)
 {
-	int fd, err;
+	/* TO-DO: Take the lock and remove the associated marker file */
+	/* int fd, err; */
 
-	fd = open(BLKTAP2_CONTROL_DEVICE, O_RDONLY);
-	if (fd == -1) {
-		EPRINTF("failed to open control device: %d\n", errno);
-		return errno;
-	}
+	/* fd = open(BLKTAP2_CONTROL_DEVICE, O_RDONLY); */
+	/* if (fd == -1) { */
+	/* 	EPRINTF("failed to open control device: %d\n", errno); */
+	/* 	return errno; */
+	/* } */
 
-	err = ioctl(fd, BLKTAP2_IOCTL_FREE_TAP, minor);
-	err = (err == -1) ? -errno : 0;
-	close(fd);
+	/* err = ioctl(fd, BLKTAP2_IOCTL_FREE_TAP, minor); */
+	/* err = (err == -1) ? -errno : 0; */
+	/* close(fd); */
 
-	return err;
+	/* return err; */
+	return 0;
 }

control/tap-ctl-ipc.c

@@ -44,7 +44,7 @@
 #include <sys/socket.h>
 
 #include "tap-ctl.h"
-#include "blktap2.h"
+#include "blktap.h"
 #include "compiler.h"
 #include "util.h"
 

control/tap-ctl-list.c

@@ -40,7 +40,7 @@
 #include <glob.h>
 
 #include "tap-ctl.h"
-#include "blktap2.h"
+#include "blktap.h"
 #include "list.h"
 
 static tap_list_t*

control/tap-ctl-spawn.c

@@ -41,7 +41,7 @@
 #include <sys/wait.h>
 
 #include "tap-ctl.h"
-#include "blktap2.h"
+#include "blktap.h"
 
 static pid_t
 __tap_ctl_spawn(int *readfd)

drivers/Makefile.am

@@ -33,8 +33,6 @@ libtapdisk_la_SOURCES += tapdisk-control.h
 libtapdisk_la_SOURCES += tapdisk-vbd.c
 libtapdisk_la_SOURCES += tapdisk-vbd.h
 libtapdisk_la_SOURCES += linux-blktap.h
-libtapdisk_la_SOURCES += tapdisk-blktap.c
-libtapdisk_la_SOURCES += tapdisk-blktap.h
 libtapdisk_la_SOURCES += tapdisk-protocol-new.h
 libtapdisk_la_SOURCES += tapdisk-nbdserver.c
 libtapdisk_la_SOURCES += tapdisk-nbdserver.h