> -----Original Message-----
> From: Hannes Reinecke [mailto:h...@suse.de]
> Sent: Thursday, February 23, 2017 4:55 AM
> To: Martin K. Petersen <martin.peter...@oracle.com>
> Cc: Christoph Hellwig <h...@lst.de>; James Bottomley
> <james.bottom...@hansenpartnership.com>; linux-scsi@vger.kernel.org;
> Hannes Reinecke <h...@suse.de>; Don Brace
> <don.br...@microsemi.com>; Hannes Reinecke <h...@suse.com>
> Subject: [PATCHv2] hpsa: expose enclosures
> 
> EXTERNAL EMAIL
> 
> 
> Some servers have a built-in enclosure which will show up on the
> same bus as the internal physical devices. This patch fixes the
> driver to expose them.
> 
> Cc: Don Brace <don.br...@microsemi.com>
> Signed-off-by: Hannes Reinecke <h...@suse.com>

Masking various SES targets is a conscious choice, not a design oversight.
So, I have to decline this patch.,

Thanks,
Don Brace
ESC - Smart Storage
Microsemi Corporation

> ---
>  drivers/scsi/hpsa.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
> index 524a0c7..a77ed5a 100644
> --- a/drivers/scsi/hpsa.c
> +++ b/drivers/scsi/hpsa.c
> @@ -4407,7 +4407,10 @@ static void hpsa_update_scsi_devices(struct
> ctlr_info *h)
>                  * Expose all devices except for physical devices that
>                  * are masked.
>                  */
> -               if (MASKED_DEVICE(lunaddrbytes) && this_device-
> >physical_device)
> +               if (this_device->devtype == TYPE_ENCLOSURE)
> +                       this_device->expose_device = 1;
> +               else if (MASKED_DEVICE(lunaddrbytes) &&
> +                        this_device->physical_device)
>                         this_device->expose_device = 0;
>                 else
>                         this_device->expose_device = 1;
> --
> 1.8.5.6

Reply via email to