<div dir="ltr">I agree with Hannes on this.<div><br></div><div>The default values are not meant to provide optimal performance ... they are meant to provide an accessible LUN whatever the hardware and on the broader range of Linux kernels. Vendors are responsible to provide their array tuning as hwtable entries. And the size or factorization of the hwtable is not an issue.</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 11, 2016 at 7:58 AM, Hannes Reinecke <span dir="ltr"><<a href="mailto:hare@suse.de" target="_blank">hare@suse.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 08/10/2016 06:20 PM, Xose Vazquez Perez wrote:<br>
> service-time is more advanced, and it works better in asymmetric and<br>
> asymmetric environments.<br>
><br>
> Cc: Hannes Reinecke <<a href="mailto:hare@suse.de">hare@suse.de</a>><br>
> Cc: Benjamin Marzinski <<a href="mailto:bmarzins@redhat.com">bmarzins@redhat.com</a>><br>
> Cc: Christophe Varoqui <<a href="mailto:christophe.varoqui@opensvc.com">christophe.varoqui@opensvc.<wbr>com</a>><br>
> Cc: device-mapper development <<a href="mailto:dm-devel@redhat.com">dm-devel@redhat.com</a>><br>
> Signed-off-by: Xose Vazquez Perez <<a href="mailto:xose.vazquez@gmail.com">xose.vazquez@gmail.com</a>><br>
> ---<br>
>  libmultipath/hwtable.c | 2 --<br>
>  1 file changed, 2 deletions(-)<br>
><br>
> diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c<br>
> index bc2d407..edfd8fe 100644<br>
> --- a/libmultipath/hwtable.c<br>
> +++ b/libmultipath/hwtable.c<br>
> @@ -198,7 +198,6 @@ static struct hwentry default_hw[] = {<br>
>       {<br>
>               .vendor        = "XtremIO",<br>
>               .product       = "XtremApp",<br>
> -             .selector      = "queue-length 0",<br>
>               .pgpolicy      = MULTIBUS,<br>
>               .pgfailback    = FAILBACK_UNDEF,<br>
>       },<br>
> @@ -823,7 +822,6 @@ static struct hwentry default_hw[] = {<br>
>       {<br>
>               .vendor        = "PURE",<br>
>               .product       = "FlashArray",<br>
> -             .selector      = "queue-length 0",<br>
>               .pgpolicy      = MULTIBUS,<br>
>               .pgfailback    = FAILBACK_UNDEF,<br>
>               .fast_io_fail  = 10,<br>
><br>
</div></div>I would _not_ do this, unless tested on the real hardware.<br>
Just claiming 'the other one is more advanced' is not enough here.<br>
<br>
NACK.<br>
<br>
Cheers,<br>
<br>
Hannes<br>
<span class="HOEnZb"><font color="#888888">--<br>
Dr. Hannes Reinecke                Teamlead Storage & Networking<br>
<a href="mailto:hare@suse.de">hare@suse.de</a>                                   <a href="tel:%2B49%20911%2074053%20688" value="+4991174053688">+49 911 74053 688</a><br>
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg<br>
GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton<br>
HRB 21284 (AG Nürnberg)<br>
</font></span></blockquote></div><br></div>