Re: [PATCH 7/8] scsi: Add 'eh_deadline' to limit SCSI EH runtime

[Hannes Reinecke <hare@suse.de>]

On 02/28/2014 08:01 AM, Ren Mingxin wrote:
> Hi, Hannes:
> 
> On 10/23/2013 04:51 PM, Hannes Reinecke wrote:
>> This patchs adds an 'eh_deadline' sysfs attribute to the scsi
>> host which limits the overall runtime of the SCSI EH.
> 
> As you known, adding to scsi host means such interface has also been
> added to the SATA and USB controllers. But to users, I think it is
> possible that there are 3 confusing points below:
> 
> 1) There should not be this sysfs interface for SATA controllers, for
>    such interface will not work under SATA's own EH policy;
> 2) There should not be this sysfs interface for USB controllers,
>    because probably they will not consider EH recovery to USB ones;
> 3) They are not willing to affect SATA/USB controllers(even if their
>    sysfs interfaces) while setting global interafce by assigning scsi
>    module parameter.
> 
> I was thinking how to mask SATA/USB controllers, but havn't a perfect
> solution so far, for it seems that it is not clever enough to mask
> them in each controller driver. Do you have any idea about above?
> 
Yes, I have.

I've send a patchset titled 'Display EVPD pages in sysfs', which
introduces the ability to mask individual (device) attributes.
The same mechanism can easily be used for host attributes.

James B indicated that those patches might be okay for inclusion,
but so far nothing has happened with them.
I'll ping him again.

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)
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html