Re: [PATCH] ses: Fix SAS device detection in enclosure
> "Ewan" == Ewan D Milnewrites: Ewan> The call to scsi_is_sas_rphy() needs to be made on the SAS Ewan> end_device, not on the SCSI device. Applied to 4.10/scsi-fixes. -- Martin K. Petersen Oracle Linux Engineering -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] ses: Fix SAS device detection in enclosure
On Mon, 2017-01-09 at 16:33 -0500, Ewan D. Milne wrote: > From: "Ewan D. Milne"> > The call to scsi_is_sas_rphy() needs to be made on the > SAS end_device, not on the SCSI device. > > Fixes: 835831c57e9b ("ses: use scsi_is_sas_rphy instead of > is_sas_attached") > Reviewed-by: Johannes Thumshirn > Signed-off-by: Ewan D. Milne Yes, looks good to me as well Reviewed-by: James Bottomley -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] ses: Fix SAS device detection in enclosure
On Mon, Jan 16, 2017 at 04:02:24PM -0500, Martin K . Petersen wrote: > > "Ewan" == Ewan D Milnewrites: > > Ewan> The call to scsi_is_sas_rphy() needs to be made on the SAS > Ewan> end_device, not on the SCSI device. > > Ewan> Fixes: 835831c57e9b ("ses: use scsi_is_sas_rphy instead of > Ewan> is_sas_attached") Reviewed-by: Johannes Thumshirn > Ewan> Signed-off-by: Ewan D. Milne > Ewan> --- > Ewan> drivers/scsi/ses.c | 2 +- 1 file changed, 1 insertion(+), 1 > Ewan> deletion(-) > > Ewan> diff --git a/drivers/scsi/ses.c b/drivers/scsi/ses.c index > Ewan> 8c9a35c..50adabb 100644 > Ewan> --- a/drivers/scsi/ses.c > Ewan> +++ b/drivers/scsi/ses.c > Ewan> @@ -587,7 +587,7 @@ static void ses_match_to_enclosure(struct > Ewan> enclosure_device *edev, > > Ewan> ses_enclosure_data_process(edev, > Ewan> to_scsi_device(edev->edev.parent), 0); > > Ewan> - if (scsi_is_sas_rphy(>sdev_gendev)) > Ewan> + if (scsi_is_sas_rphy(sdev->sdev_target->dev.parent)) > Ewan> efd.addr = sas_get_address(sdev); > > Ewan> if (efd.addr) { > Ewan> -- 1.8.3.1 > > James? FWIW I have positive test results from a customer of ours with this as well. Thanks, Johannes -- Johannes Thumshirn Storage jthumsh...@suse.de+49 911 74053 689 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850 -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] ses: Fix SAS device detection in enclosure
> "Ewan" == Ewan D Milnewrites: Ewan> The call to scsi_is_sas_rphy() needs to be made on the SAS Ewan> end_device, not on the SCSI device. Ewan> Fixes: 835831c57e9b ("ses: use scsi_is_sas_rphy instead of Ewan> is_sas_attached") Reviewed-by: Johannes Thumshirn Ewan> Signed-off-by: Ewan D. Milne Ewan> --- Ewan> drivers/scsi/ses.c | 2 +- 1 file changed, 1 insertion(+), 1 Ewan> deletion(-) Ewan> diff --git a/drivers/scsi/ses.c b/drivers/scsi/ses.c index Ewan> 8c9a35c..50adabb 100644 Ewan> --- a/drivers/scsi/ses.c Ewan> +++ b/drivers/scsi/ses.c Ewan> @@ -587,7 +587,7 @@ static void ses_match_to_enclosure(struct Ewan> enclosure_device *edev, Ewan> ses_enclosure_data_process(edev, Ewan> to_scsi_device(edev->edev.parent), 0); Ewan> - if (scsi_is_sas_rphy(>sdev_gendev)) Ewan> + if (scsi_is_sas_rphy(sdev->sdev_target->dev.parent)) Ewan> efd.addr = sas_get_address(sdev); Ewan> if (efd.addr) { Ewan> -- 1.8.3.1 James? -- Martin K. Petersen Oracle Linux Engineering -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html