[dm-devel] [PATCH] multipath-tools: reorder vendors in hwtable
Martin Wilck
mwilck at suse.de
Thu Dec 20 23:51:06 UTC 2018
On Wed, 2018-12-19 at 22:23 +0100, Xose Vazquez Perez wrote:
> Xio was acquired by Violin, and add FlashSystem 9100 to Storwize in
> comments.
>
> Cc: Christophe Varoqui <christophe.varoqui at opensvc.com>
> Cc: DM-DEVEL ML <dm-devel at redhat.com>
> Signed-off-by: Xose Vazquez Perez <xose.vazquez at gmail.com>
> ---
> libmultipath/hwtable.c | 50 ++++++++++++++++++++------------------
> ----
> 1 file changed, 24 insertions(+), 26 deletions(-)
>
How important is it to reflect this kind of company-A-owns-company-B
information in multipath-tools source code? IMO it makes it harder to
track code changes, for no obvious technical reason.
Regards
Martin
> diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
> index d3a8d9b..3eeadde 100644
> --- a/libmultipath/hwtable.c
> +++ b/libmultipath/hwtable.c
> @@ -604,7 +604,7 @@ static struct hwentry default_hw[] = {
> .pgpolicy = MULTIBUS,
> },
> {
> - /* Storwize family / SAN Volume Controller / Flex
> System V7000 / FlashSystem V840/V9000 */
> + /* Storwize family / SAN Volume Controller / Flex
> System V7000 / FlashSystem V840/V9000/9100 */
> .vendor = "IBM",
> .product = "^2145",
> .no_path_retry = NO_PATH_RETRY_QUEUE,
> @@ -1099,31 +1099,7 @@ static struct hwentry default_hw[] = {
> .no_path_retry = 30,
> },
> /*
> - * Xiotech
> - */
> - {
> - /* Intelligent Storage Elements family */
> - .vendor = "(XIOTECH|XIOtech)",
> - .product = "ISE",
> - .pgpolicy = MULTIBUS,
> - .no_path_retry = 12,
> - },
> - {
> - /* iglu blaze family */
> - .vendor = "(XIOTECH|XIOtech)",
> - .product = "IGLU DISK",
> - .pgpolicy = MULTIBUS,
> - .no_path_retry = 30,
> - },
> - {
> - /* Magnitude family */
> - .vendor = "(XIOTECH|XIOtech)",
> - .product = "Magnitude",
> - .pgpolicy = MULTIBUS,
> - .no_path_retry = 30,
> - },
> - /*
> - * Violin Memory
> + * Violin Systems
> */
> {
> /* 3000 / 6000 Series */
> @@ -1148,6 +1124,28 @@ static struct hwentry default_hw[] = {
> .product = "CONCERTO ARRAY",
> .pgpolicy = MULTIBUS,
> .no_path_retry = 30,
> + },
> + /* Xiotech */
> + {
> + /* Intelligent Storage Elements family */
> + .vendor = "(XIOTECH|XIOtech)",
> + .product = "ISE",
> + .pgpolicy = MULTIBUS,
> + .no_path_retry = 12,
> + },
> + {
> + /* iglu blaze family */
> + .vendor = "(XIOTECH|XIOtech)",
> + .product = "IGLU DISK",
> + .pgpolicy = MULTIBUS,
> + .no_path_retry = 30,
> + },
> + {
> + /* Magnitude family */
> + .vendor = "(XIOTECH|XIOtech)",
> + .product = "Magnitude",
> + .pgpolicy = MULTIBUS,
> + .no_path_retry = 30,
> },
> /*
> * Promise Technology
More information about the dm-devel
mailing list