Re: [PATCH 2/2] scsi: storvsc: Add support for FC lightweight host.

From: Dan Carpenter
Date: Sun Jan 22 2017 - 14:26:12 EST


On Sun, Jan 22, 2017 at 01:51:13PM -0500, Cathy Avery wrote:
> I'm sorry. In my zeal to push out this patch I have done a poor job
> of communication on a number of levels.
>
> The first patch which deals with the fc transport changes will not
> set the scsi_transport_template.eh_timed_out function directly
> during lightweight fc_attach_transport(). It instead relies on
> whatever was indicated as the scsi_host_template timeout handler
> during inscsi_times_out() scsi_error.c.
>
> So yes in a sense it is related but now I believe I understand your
> point. Perhaps this would fall more under the heading of post
> fc_transport implementation storvsc cleanup necessitating its own
> patch.
>
> I will break it out in the next go round.

Ah... Actually it probably belongs in this patch, it just wasn't
obvious to me how they were related (I'm a total newbie with this code
so that's probably part of the confusion). Since you're resend the code
anyway, could you just add this information to the commit message?

regards,
dan carpenter