On Mon, Apr 27, 2009 at 01:09:57PM -0400, Takahiro Yasui wrote:
> @@ -2633,9 +2633,12 @@ scsi_internal_device_unblock(struct scsi
> unsigned long flags;
>
> /*
> - * Try to transition the scsi device to SDEV_RUNNING
> - * and goose the device queue if successful.
> + * Try to transition the scsi device to SDEV_RUNNING if it is
> + * SDEV_BLOCK and goose the device queue if successful.
I think the code looks good, but the edit to the comment dilutes its
flavour somewhat. How about just moving the comment down below the
check, and then you don't need to edit the comment at all?
> */
> + if (sdev->sdev_state != SDEV_BLOCK)
> + return 0;
> +
> err = scsi_device_set_state(sdev, SDEV_RUNNING);
> if (err) {
> err = scsi_device_set_state(sdev, SDEV_CREATED);
>
>
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
Matthew Wilcox Intel Open Source Technology Centre
"Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours. We can't possibly take such
a retrograde step."
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
