* Re: [PATCH RESEND] csiostor:Fix error handling in the function csio_device_reset
[not found] <1442355274-22410-1-git-send-email-xerofoify@gmail.com>
@ 2015-09-16 6:49 ` Hannes Reinecke
0 siblings, 0 replies; only message in thread
From: Hannes Reinecke @ 2015-09-16 6:49 UTC (permalink / raw)
To: Nicholas Krause, JBottomley
Cc: michaelc, davem, anish, martin.petersen, hariprasad, linux-scsi,
linux-kernel
On 09/16/2015 12:14 AM, Nicholas Krause wrote:
> This fixes error handling in the function csio_device_reset to
> check the return value of aftering the function csio_hw_reset
> to check if it returned a error code and if so unlock the irq
> spinlock for the hardware plus return the hardware error code
> immediately.
>
> Signed-off-by: Nicholas Krause <xerofoify@gmail.com>
> ---
> drivers/scsi/csiostor/csio_scsi.c | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/scsi/csiostor/csio_scsi.c b/drivers/scsi/csiostor/csio_scsi.c
> index 2c4562d..faea4e7 100644
> --- a/drivers/scsi/csiostor/csio_scsi.c
> +++ b/drivers/scsi/csiostor/csio_scsi.c
> @@ -1378,6 +1378,7 @@ csio_device_reset(struct device *dev,
> {
> struct csio_lnode *ln = shost_priv(class_to_shost(dev));
> struct csio_hw *hw = csio_lnode_to_hw(ln);
> + int ret;
>
> if (*buf != '1')
> return -EINVAL;
> @@ -1389,7 +1390,11 @@ csio_device_reset(struct device *dev,
> csio_lnodes_block_request(hw);
>
> spin_lock_irq(&hw->lock);
> - csio_hw_reset(hw);
> + ret = csio_hw_reset(hw);
> + if (ret) {
> + spin_unlock_irq(&hw->lock);
> + return ret;
> + }
> spin_unlock_irq(&hw->lock);
>
> /* Unblock upper IOs */
>
No sure if that is correct; wouldn't the device stay blocked
if csio_hw_reset() fails?
Cheers,
Hannes
--
Dr. Hannes Reinecke zSeries & Storage
hare@suse.de +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: J. Hawn, J. Guild, F. Imendörffer, HRB 16746 (AG Nürnberg)
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2015-09-16 6:49 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1442355274-22410-1-git-send-email-xerofoify@gmail.com>
2015-09-16 6:49 ` [PATCH RESEND] csiostor:Fix error handling in the function csio_device_reset Hannes Reinecke
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).