Commit bc1a7d6a authored Jul 09, 2015 by Andy Grover Committed by Nicholas Bellinger Jul 23, 2015

target: Indicate success if writing 0 to pi_prot_type

See https://bugzilla.redhat.com/show_bug.cgi?id=1240687



Returning 0 from a configfs store function results in infinite retries.

Reported-by: Yanko Kaneti <yaneti@declera.com>
Signed-off-by: Andy Grover <agrover@redhat.com>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>

parent fd4e1393

drivers/target/target_core_configfs.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -747,7 +747,7 @@ static ssize_t store_pi_prot_type(struct se_dev_attrib *da,
		if (!dev->transport->init_prot \|\| !dev->transport->free_prot) {
		/* 0 is only allowed value for non-supporting backends */
		if (flag == 0)
		return 0;
		return count;

		pr_err("DIF protection not supported by backend: %s\n",
		dev->transport->name);

Admin message