Re: Error handling on FC devices

[Kipp Aldrich <kippa@tributary.com>]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 12/03/2012 04:52 PM, Elliott, Robert (Server Storage) wrote:
> 
>> Well, looking at QLogic and Emulex both emulate a bus reset with a
>> loop over each target and invoke a target reset there.
>> I somewhat fail to see the rationale behind it, other than emulating
>> the bus reset behaviour on SPI.
>> Given that the original target reset already failed (otherwise we
>> wouldn't be doing a bus reset), I doubt a _second_ target reset
>> will lead to a different result.
>>
>> So invoking REMOVE I_T NEXUS here can only improve matters :-)
>>
>> I'm all for renaming bus_reset, though :-)
>>
> 
> Since modern storage fabrics don't really have a "bus" to reset any more, a bolder approach of getting rid of bus_reset might be warranted.
> 

This is exactly what we do now. We modify drivers to disable bus_resets. It is dangerous, and disruptive, to reliable data storage,
especially on SANs. At least make it optional to enable it, disable it by default. Bus resets can destroy or lose customer data, for
goodness sake. Any commercial vendor using linux with these drivers that doesn't disable bus resets is risking their customer's data.

Really, how early '80s to have bus_reset. Time to move on.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.16 (GNU/Linux)

iEYEARECAAYFAlC+HUcACgkQeGqeAAwqIi147wCcDHEMmyr9m+umifJAbEDGns04
B4gAniGcrjM110cbX9/Ki/3e+jZO4d1o
=IRk4
-----END PGP SIGNATURE-----