Re: [PATCH RFC] multipath-tools: add cciss_tur checker to CCISS arrays

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 2018-07-05 at 20:56 +0200, Xose Vazquez Perez wrote:
> In cf87a17ee840aafbbc6166f663963e5213573de7 Hannes says:
>    
>   The 'normal' SG_IO tur checker has some issues with cciss, so
>   we'd better use a dedicated path checker here.
> 
> CC: Hannes Reinecke <hare@xxxxxxx>
> Cc: Christophe Varoqui <christophe.varoqui@xxxxxxxxxxx>
> Cc: DM-DEV ML <dm-devel@xxxxxxxxxx>
> Signed-off-by: Xose Vazquez Perez <xose.vazquez@xxxxxxxxx>
> ---
> 
> cciss(block) driver was replaced by hpsa(scsi) in
> 253d2464df446456c0bba5ed4137a7be0b278aa8 the last year.
> 
> Should cciss be removed from multipath-tools?
> 
> ---
>  libmultipath/hwtable.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
> index d3a8d9b..c460d29 100644
> --- a/libmultipath/hwtable.c
> +++ b/libmultipath/hwtable.c
> @@ -199,11 +199,12 @@ static struct hwentry default_hw[] = {
>  		.prio_name     = PRIO_ALUA,
>  	},
>  	{
> -		/* Smart Array */
> +		/* Smart Array CCISS */
>  		.vendor        = "HP",
>  		.product       = "LOGICAL VOLUME",
>  		.pgpolicy      = MULTIBUS,
>  		.no_path_retry = 12,
> +		.checker_name  = CCISS_TUR,
>  	},

This won't work, because these volumes are detected as "HP" "LOGICAL
VOLUME", no matter whether they're attached to the cciss or the hpsa
driver.

Martin

-- 
Dr. Martin Wilck <mwilck@xxxxxxxx>, Tel. +49 (0)911 74053 2107
SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)

--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel




[Index of Archives]     [DM Crypt]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Packaging]     [Fedora SELinux]     [Yosemite Discussion]     [KDE Users]     [Fedora Docs]

  Powered by Linux