Commit e80d1c80 authored by Vivek Goyal's avatar Vivek Goyal Committed by Mike Snitzer
Browse files

dm: do not override error code returned from dm_get_device() 



Some of the device mapper targets override the error code returned by
dm_get_device() and return either -EINVAL or -ENXIO.  There is nothing
gained by this override.  It is better to propagate the returned error
code unchanged to caller.

This work was motivated by hitting an issue where the underlying device
was busy but -EINVAL was being returned.  After this change we get
-EBUSY instead and it is easier to figure out the problem.

Signed-off-by: default avatarVivek Goyal <vgoyal@redhat.com>
Signed-off-by: default avatarMike Snitzer <snitzer@redhat.com>
parent ab37844d

drivers/md/dm-crypt.c

+3 −1
Original line number Diff line number Diff line
@@ -1811,11 +1811,13 @@ static int crypt_ctr(struct dm_target *ti, unsigned int argc, char **argv) 
	}

	cc->iv_offset = tmpll;



	if (dm_get_device(ti, argv[3], dm_table_get_mode(ti->table), &cc->dev)) {

	ret = dm_get_device(ti, argv[3], dm_table_get_mode(ti->table), &cc->dev);

	if (ret) {

		ti->error = "Device lookup failed";

		goto bad;

	}



	ret = -EINVAL;

	if (sscanf(argv[4], "%llu%c", &tmpll, &dummy) != 1) {

		ti->error = "Invalid device sector";

		goto bad;

drivers/md/dm-delay.c

+11 −5
Original line number Diff line number Diff line
@@ -129,6 +129,7 @@ static int delay_ctr(struct dm_target *ti, unsigned int argc, char **argv) 
	struct delay_c *dc;

	unsigned long long tmpll;

	char dummy;

	int ret;



	if (argc != 3 && argc != 6) {

		ti->error = "requires exactly 3 or 6 arguments";
@@ -143,6 +144,7 @@ static int delay_ctr(struct dm_target *ti, unsigned int argc, char **argv) 


	dc->reads = dc->writes = 0;



	ret = -EINVAL;

	if (sscanf(argv[1], "%llu%c", &tmpll, &dummy) != 1) {

		ti->error = "Invalid device sector";

		goto bad;
@@ -154,12 +156,14 @@ static int delay_ctr(struct dm_target *ti, unsigned int argc, char **argv) 
		goto bad;

	}



	if (dm_get_device(ti, argv[0], dm_table_get_mode(ti->table),

			  &dc->dev_read)) {

	ret = dm_get_device(ti, argv[0], dm_table_get_mode(ti->table),

			    &dc->dev_read);

	if (ret) {

		ti->error = "Device lookup failed";

		goto bad;

	}



	ret = -EINVAL;

	dc->dev_write = NULL;

	if (argc == 3)

		goto out;
@@ -175,13 +179,15 @@ static int delay_ctr(struct dm_target *ti, unsigned int argc, char **argv) 
		goto bad_dev_read;

	}



	if (dm_get_device(ti, argv[3], dm_table_get_mode(ti->table),

			  &dc->dev_write)) {

	ret = dm_get_device(ti, argv[3], dm_table_get_mode(ti->table),

			    &dc->dev_write);

	if (ret) {

		ti->error = "Write device lookup failed";

		goto bad_dev_read;

	}



out:

	ret = -EINVAL;

	dc->kdelayd_wq = alloc_workqueue("kdelayd", WQ_MEM_RECLAIM, 0);

	if (!dc->kdelayd_wq) {

		DMERR("Couldn't start kdelayd");
@@ -208,7 +214,7 @@ bad_dev_read: 
	dm_put_device(ti, dc->dev_read);

bad:

	kfree(dc);

	return -EINVAL;

	return ret;

}



static void delay_dtr(struct dm_target *ti)

drivers/md/dm-flakey.c

+4 −2
Original line number Diff line number Diff line
@@ -183,6 +183,7 @@ static int flakey_ctr(struct dm_target *ti, unsigned int argc, char **argv) 


	devname = dm_shift_arg(&as);



	r = -EINVAL;

	if (sscanf(dm_shift_arg(&as), "%llu%c", &tmpll, &dummy) != 1) {

		ti->error = "Invalid device sector";

		goto bad;
@@ -211,7 +212,8 @@ static int flakey_ctr(struct dm_target *ti, unsigned int argc, char **argv) 
	if (r)

		goto bad;



	if (dm_get_device(ti, devname, dm_table_get_mode(ti->table), &fc->dev)) {

	r = dm_get_device(ti, devname, dm_table_get_mode(ti->table), &fc->dev);

	if (r) {

		ti->error = "Device lookup failed";

		goto bad;

	}
@@ -224,7 +226,7 @@ static int flakey_ctr(struct dm_target *ti, unsigned int argc, char **argv) 


bad:

	kfree(fc);

	return -EINVAL;

	return r;

}



static void flakey_dtr(struct dm_target *ti)

drivers/md/dm-linear.c

+5 −2
Original line number Diff line number Diff line
@@ -30,6 +30,7 @@ static int linear_ctr(struct dm_target *ti, unsigned int argc, char **argv) 
	struct linear_c *lc;

	unsigned long long tmp;

	char dummy;

	int ret;



	if (argc != 2) {

		ti->error = "Invalid argument count";
@@ -42,13 +43,15 @@ static int linear_ctr(struct dm_target *ti, unsigned int argc, char **argv) 
		return -ENOMEM;

	}



	ret = -EINVAL;

	if (sscanf(argv[1], "%llu%c", &tmp, &dummy) != 1) {

		ti->error = "dm-linear: Invalid device sector";

		goto bad;

	}

	lc->start = tmp;



	if (dm_get_device(ti, argv[0], dm_table_get_mode(ti->table), &lc->dev)) {

	ret = dm_get_device(ti, argv[0], dm_table_get_mode(ti->table), &lc->dev);

	if (ret) {

		ti->error = "dm-linear: Device lookup failed";

		goto bad;

	}
@@ -61,7 +64,7 @@ static int linear_ctr(struct dm_target *ti, unsigned int argc, char **argv) 


      bad:

	kfree(lc);

	return -EINVAL;

	return ret;

}



static void linear_dtr(struct dm_target *ti)

drivers/md/dm-log-writes.c

+8 −3
Original line number Diff line number Diff line
@@ -420,6 +420,7 @@ static int log_writes_ctr(struct dm_target *ti, unsigned int argc, char **argv) 
	struct log_writes_c *lc;

	struct dm_arg_set as;

	const char *devname, *logdevname;

	int ret;



	as.argc = argc;

	as.argv = argv;
@@ -443,18 +444,22 @@ static int log_writes_ctr(struct dm_target *ti, unsigned int argc, char **argv) 
	atomic_set(&lc->pending_blocks, 0);



	devname = dm_shift_arg(&as);

	if (dm_get_device(ti, devname, dm_table_get_mode(ti->table), &lc->dev)) {

	ret = dm_get_device(ti, devname, dm_table_get_mode(ti->table), &lc->dev);

	if (ret) {

		ti->error = "Device lookup failed";

		goto bad;

	}



	logdevname = dm_shift_arg(&as);

	if (dm_get_device(ti, logdevname, dm_table_get_mode(ti->table), &lc->logdev)) {

	ret = dm_get_device(ti, logdevname, dm_table_get_mode(ti->table),

			    &lc->logdev);

	if (ret) {

		ti->error = "Log device lookup failed";

		dm_put_device(ti, lc->dev);

		goto bad;

	}



	ret = -EINVAL;

	lc->log_kthread = kthread_run(log_writes_kthread, lc, "log-write");

	if (!lc->log_kthread) {

		ti->error = "Couldn't alloc kthread";
@@ -479,7 +484,7 @@ static int log_writes_ctr(struct dm_target *ti, unsigned int argc, char **argv) 


bad:

	kfree(lc);

	return -EINVAL;

	return ret;

}



static int log_mark(struct log_writes_c *lc, char *data)

CRA Git | Maintained and supported by SUSTech CRA and CCSE