linux-scsi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] hpsa: return 0 from driver probe function on success, not 1
@ 2013-10-29 18:39 Stephen M. Cameron
  0 siblings, 0 replies; 3+ messages in thread
From: Stephen M. Cameron @ 2013-10-29 18:39 UTC (permalink / raw)
  To: james.bottomley; +Cc: stephenmcameron, mikem, thenzl, linux-scsi, scott.teel

From: Stephen M. Cameron <scameron@beardog.cce.hp.com>

A return value of 1 is interpreted as an error

Signed-off-by: Stephen M. Cameron <scameron@beardog.cce.hp.com>
---
 drivers/scsi/hpsa.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
index 891c86b..3c97974 100644
--- a/drivers/scsi/hpsa.c
+++ b/drivers/scsi/hpsa.c
@@ -4926,7 +4926,7 @@ reinit_after_soft_reset:
 	hpsa_hba_inquiry(h);
 	hpsa_register_scsi(h);	/* hook ourselves into SCSI subsystem */
 	start_controller_lockup_detector(h);
-	return 1;
+	return 0;
 
 clean4:
 	hpsa_free_sg_chain_blocks(h);


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [PATCH] hpsa: return 0 from driver probe function on success, not 1
@ 2013-11-01 16:02 Stephen M. Cameron
  2013-11-01 16:02 ` Stephen M. Cameron
  0 siblings, 1 reply; 3+ messages in thread
From: Stephen M. Cameron @ 2013-11-01 16:02 UTC (permalink / raw)
  To: james.bottomley
  Cc: linux-scsi, stephenmcameron, thenzl, scott.teel, akpm, mikem

Just resending the patch with a better change log message (as requested
by Andrew Morton) and cc'ing stable@vger.kernel.org, (as I originally
had intended.)

---

Stephen M. Cameron (1):
      hpsa: return 0 from driver probe function on success, not 1


 drivers/scsi/hpsa.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

-- 
-- steve

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH] hpsa: return 0 from driver probe function on success, not 1
  2013-11-01 16:02 Stephen M. Cameron
@ 2013-11-01 16:02 ` Stephen M. Cameron
  0 siblings, 0 replies; 3+ messages in thread
From: Stephen M. Cameron @ 2013-11-01 16:02 UTC (permalink / raw)
  To: james.bottomley
  Cc: linux-scsi, stephenmcameron, thenzl, scott.teel, akpm, mikem

From: Stephen M. Cameron <scameron@beardog.cce.hp.com>

A return value of 1 is interpreted as an error.  See pci_driver.
in local_pci_probe().  If you're wondering how this ever could
have worked, it's because it used to be the case that only return
values less than zero were interpreted as failure.  But even in
the current kernel if the driver registers its various entry
points with the kernel, and then returns a value which is
interpreted as failure, those registrations aren't undone, so
the driver still mostly works.  However, the driver's remove
function wouldn't be called on rmmod, and pci power management
functions wouldn't work.  In the case of Smart Array, since it
has a battery backed cache (or else no cache) even if the driver
is not shut down properly as long as there is no outstanding
i/o, nothing too bad happens, which is why it took so long to
notice.

Requesting backport to stable because the change to pci-driver.c
which requires driver probe functions to return 0 occurred between
2.6.35 and 2.6.36 (the pci power management breakage) and again
between 3.7 and 3.8 (pci_dev->driver getting set to NULL in
local_pci_probe() preventing driver remove function from being
called on rmmod.)

Signed-off-by: Stephen M. Cameron <scameron@beardog.cce.hp.com>
Cc: stable@vger.kernel.org
---
 drivers/scsi/hpsa.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
index 891c86b..3c97974 100644
--- a/drivers/scsi/hpsa.c
+++ b/drivers/scsi/hpsa.c
@@ -4926,7 +4926,7 @@ reinit_after_soft_reset:
 	hpsa_hba_inquiry(h);
 	hpsa_register_scsi(h);	/* hook ourselves into SCSI subsystem */
 	start_controller_lockup_detector(h);
-	return 1;
+	return 0;
 
 clean4:
 	hpsa_free_sg_chain_blocks(h);


^ permalink raw reply related	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-11-01 16:02 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-10-29 18:39 [PATCH] hpsa: return 0 from driver probe function on success, not 1 Stephen M. Cameron
  -- strict thread matches above, loose matches on Subject: below --
2013-11-01 16:02 Stephen M. Cameron
2013-11-01 16:02 ` Stephen M. Cameron

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 NNTP newsgroup(s).