mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-09-07 03:18:31 +00:00
scsi: mpi3mr: Fix ATA NCQ priority support
commit90e6f08915
upstream. The function mpi3mr_qcmd() of the mpi3mr driver is able to indicate to the HBA if a read or write command directed at an ATA device should be translated to an NCQ read/write command with the high prioiryt bit set when the request uses the RT priority class and the user has enabled NCQ priority through sysfs. However, unlike the mpt3sas driver, the mpi3mr driver does not define the sas_ncq_prio_supported and sas_ncq_prio_enable sysfs attributes, so the ncq_prio_enable field of struct mpi3mr_sdev_priv_data is never actually set and NCQ Priority cannot ever be used. Fix this by defining these missing atributes to allow a user to check if an ATA device supports NCQ priority and to enable/disable the use of NCQ priority. To do this, lift the function scsih_ncq_prio_supp() out of the mpt3sas driver and make it the generic SCSI SAS transport function sas_ata_ncq_prio_supported(). Nothing in that function is hardware specific, so this function can be used in both the mpt3sas driver and the mpi3mr driver. Reported-by: Scott McCoy <scott.mccoy@wdc.com> Fixes:023ab2a9b4
("scsi: mpi3mr: Add support for queue command processing") Cc: stable@vger.kernel.org Signed-off-by: Damien Le Moal <dlemoal@kernel.org> Link: https://lore.kernel.org/r/20240611083435.92961-1-dlemoal@kernel.org Reviewed-by: Niklas Cassel <cassel@kernel.org> Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
3e9785d3e9
commit
da097dccae
6 changed files with 89 additions and 28 deletions
|
@ -1854,10 +1854,72 @@ persistent_id_show(struct device *dev, struct device_attribute *attr,
|
||||||
}
|
}
|
||||||
static DEVICE_ATTR_RO(persistent_id);
|
static DEVICE_ATTR_RO(persistent_id);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* sas_ncq_prio_supported_show - Indicate if device supports NCQ priority
|
||||||
|
* @dev: pointer to embedded device
|
||||||
|
* @attr: sas_ncq_prio_supported attribute descriptor
|
||||||
|
* @buf: the buffer returned
|
||||||
|
*
|
||||||
|
* A sysfs 'read-only' sdev attribute, only works with SATA devices
|
||||||
|
*/
|
||||||
|
static ssize_t
|
||||||
|
sas_ncq_prio_supported_show(struct device *dev,
|
||||||
|
struct device_attribute *attr, char *buf)
|
||||||
|
{
|
||||||
|
struct scsi_device *sdev = to_scsi_device(dev);
|
||||||
|
|
||||||
|
return sysfs_emit(buf, "%d\n", sas_ata_ncq_prio_supported(sdev));
|
||||||
|
}
|
||||||
|
static DEVICE_ATTR_RO(sas_ncq_prio_supported);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* sas_ncq_prio_enable_show - send prioritized io commands to device
|
||||||
|
* @dev: pointer to embedded device
|
||||||
|
* @attr: sas_ncq_prio_enable attribute descriptor
|
||||||
|
* @buf: the buffer returned
|
||||||
|
*
|
||||||
|
* A sysfs 'read/write' sdev attribute, only works with SATA devices
|
||||||
|
*/
|
||||||
|
static ssize_t
|
||||||
|
sas_ncq_prio_enable_show(struct device *dev,
|
||||||
|
struct device_attribute *attr, char *buf)
|
||||||
|
{
|
||||||
|
struct scsi_device *sdev = to_scsi_device(dev);
|
||||||
|
struct mpi3mr_sdev_priv_data *sdev_priv_data = sdev->hostdata;
|
||||||
|
|
||||||
|
if (!sdev_priv_data)
|
||||||
|
return 0;
|
||||||
|
|
||||||
|
return sysfs_emit(buf, "%d\n", sdev_priv_data->ncq_prio_enable);
|
||||||
|
}
|
||||||
|
|
||||||
|
static ssize_t
|
||||||
|
sas_ncq_prio_enable_store(struct device *dev,
|
||||||
|
struct device_attribute *attr,
|
||||||
|
const char *buf, size_t count)
|
||||||
|
{
|
||||||
|
struct scsi_device *sdev = to_scsi_device(dev);
|
||||||
|
struct mpi3mr_sdev_priv_data *sdev_priv_data = sdev->hostdata;
|
||||||
|
bool ncq_prio_enable = 0;
|
||||||
|
|
||||||
|
if (kstrtobool(buf, &ncq_prio_enable))
|
||||||
|
return -EINVAL;
|
||||||
|
|
||||||
|
if (!sas_ata_ncq_prio_supported(sdev))
|
||||||
|
return -EINVAL;
|
||||||
|
|
||||||
|
sdev_priv_data->ncq_prio_enable = ncq_prio_enable;
|
||||||
|
|
||||||
|
return strlen(buf);
|
||||||
|
}
|
||||||
|
static DEVICE_ATTR_RW(sas_ncq_prio_enable);
|
||||||
|
|
||||||
static struct attribute *mpi3mr_dev_attrs[] = {
|
static struct attribute *mpi3mr_dev_attrs[] = {
|
||||||
&dev_attr_sas_address.attr,
|
&dev_attr_sas_address.attr,
|
||||||
&dev_attr_device_handle.attr,
|
&dev_attr_device_handle.attr,
|
||||||
&dev_attr_persistent_id.attr,
|
&dev_attr_persistent_id.attr,
|
||||||
|
&dev_attr_sas_ncq_prio_supported.attr,
|
||||||
|
&dev_attr_sas_ncq_prio_enable.attr,
|
||||||
NULL,
|
NULL,
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
|
@ -2045,9 +2045,6 @@ void
|
||||||
mpt3sas_setup_direct_io(struct MPT3SAS_ADAPTER *ioc, struct scsi_cmnd *scmd,
|
mpt3sas_setup_direct_io(struct MPT3SAS_ADAPTER *ioc, struct scsi_cmnd *scmd,
|
||||||
struct _raid_device *raid_device, Mpi25SCSIIORequest_t *mpi_request);
|
struct _raid_device *raid_device, Mpi25SCSIIORequest_t *mpi_request);
|
||||||
|
|
||||||
/* NCQ Prio Handling Check */
|
|
||||||
bool scsih_ncq_prio_supp(struct scsi_device *sdev);
|
|
||||||
|
|
||||||
void mpt3sas_setup_debugfs(struct MPT3SAS_ADAPTER *ioc);
|
void mpt3sas_setup_debugfs(struct MPT3SAS_ADAPTER *ioc);
|
||||||
void mpt3sas_destroy_debugfs(struct MPT3SAS_ADAPTER *ioc);
|
void mpt3sas_destroy_debugfs(struct MPT3SAS_ADAPTER *ioc);
|
||||||
void mpt3sas_init_debugfs(void);
|
void mpt3sas_init_debugfs(void);
|
||||||
|
|
|
@ -4034,7 +4034,7 @@ sas_ncq_prio_supported_show(struct device *dev,
|
||||||
{
|
{
|
||||||
struct scsi_device *sdev = to_scsi_device(dev);
|
struct scsi_device *sdev = to_scsi_device(dev);
|
||||||
|
|
||||||
return sysfs_emit(buf, "%d\n", scsih_ncq_prio_supp(sdev));
|
return sysfs_emit(buf, "%d\n", sas_ata_ncq_prio_supported(sdev));
|
||||||
}
|
}
|
||||||
static DEVICE_ATTR_RO(sas_ncq_prio_supported);
|
static DEVICE_ATTR_RO(sas_ncq_prio_supported);
|
||||||
|
|
||||||
|
@ -4069,7 +4069,7 @@ sas_ncq_prio_enable_store(struct device *dev,
|
||||||
if (kstrtobool(buf, &ncq_prio_enable))
|
if (kstrtobool(buf, &ncq_prio_enable))
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
|
|
||||||
if (!scsih_ncq_prio_supp(sdev))
|
if (!sas_ata_ncq_prio_supported(sdev))
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
|
|
||||||
sas_device_priv_data->ncq_prio_enable = ncq_prio_enable;
|
sas_device_priv_data->ncq_prio_enable = ncq_prio_enable;
|
||||||
|
|
|
@ -12590,29 +12590,6 @@ scsih_pci_mmio_enabled(struct pci_dev *pdev)
|
||||||
return PCI_ERS_RESULT_RECOVERED;
|
return PCI_ERS_RESULT_RECOVERED;
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* scsih_ncq_prio_supp - Check for NCQ command priority support
|
|
||||||
* @sdev: scsi device struct
|
|
||||||
*
|
|
||||||
* This is called when a user indicates they would like to enable
|
|
||||||
* ncq command priorities. This works only on SATA devices.
|
|
||||||
*/
|
|
||||||
bool scsih_ncq_prio_supp(struct scsi_device *sdev)
|
|
||||||
{
|
|
||||||
struct scsi_vpd *vpd;
|
|
||||||
bool ncq_prio_supp = false;
|
|
||||||
|
|
||||||
rcu_read_lock();
|
|
||||||
vpd = rcu_dereference(sdev->vpd_pg89);
|
|
||||||
if (!vpd || vpd->len < 214)
|
|
||||||
goto out;
|
|
||||||
|
|
||||||
ncq_prio_supp = (vpd->data[213] >> 4) & 1;
|
|
||||||
out:
|
|
||||||
rcu_read_unlock();
|
|
||||||
|
|
||||||
return ncq_prio_supp;
|
|
||||||
}
|
|
||||||
/*
|
/*
|
||||||
* The pci device ids are defined in mpi/mpi2_cnfg.h.
|
* The pci device ids are defined in mpi/mpi2_cnfg.h.
|
||||||
*/
|
*/
|
||||||
|
|
|
@ -416,6 +416,29 @@ unsigned int sas_is_tlr_enabled(struct scsi_device *sdev)
|
||||||
}
|
}
|
||||||
EXPORT_SYMBOL_GPL(sas_is_tlr_enabled);
|
EXPORT_SYMBOL_GPL(sas_is_tlr_enabled);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* sas_ata_ncq_prio_supported - Check for ATA NCQ command priority support
|
||||||
|
* @sdev: SCSI device
|
||||||
|
*
|
||||||
|
* Check if an ATA device supports NCQ priority using VPD page 89h (ATA
|
||||||
|
* Information). Since this VPD page is implemented only for ATA devices,
|
||||||
|
* this function always returns false for SCSI devices.
|
||||||
|
*/
|
||||||
|
bool sas_ata_ncq_prio_supported(struct scsi_device *sdev)
|
||||||
|
{
|
||||||
|
struct scsi_vpd *vpd;
|
||||||
|
bool ncq_prio_supported = false;
|
||||||
|
|
||||||
|
rcu_read_lock();
|
||||||
|
vpd = rcu_dereference(sdev->vpd_pg89);
|
||||||
|
if (vpd && vpd->len >= 214)
|
||||||
|
ncq_prio_supported = (vpd->data[213] >> 4) & 1;
|
||||||
|
rcu_read_unlock();
|
||||||
|
|
||||||
|
return ncq_prio_supported;
|
||||||
|
}
|
||||||
|
EXPORT_SYMBOL_GPL(sas_ata_ncq_prio_supported);
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* SAS Phy attributes
|
* SAS Phy attributes
|
||||||
*/
|
*/
|
||||||
|
|
|
@ -200,6 +200,8 @@ unsigned int sas_is_tlr_enabled(struct scsi_device *);
|
||||||
void sas_disable_tlr(struct scsi_device *);
|
void sas_disable_tlr(struct scsi_device *);
|
||||||
void sas_enable_tlr(struct scsi_device *);
|
void sas_enable_tlr(struct scsi_device *);
|
||||||
|
|
||||||
|
bool sas_ata_ncq_prio_supported(struct scsi_device *sdev);
|
||||||
|
|
||||||
extern struct sas_rphy *sas_end_device_alloc(struct sas_port *);
|
extern struct sas_rphy *sas_end_device_alloc(struct sas_port *);
|
||||||
extern struct sas_rphy *sas_expander_alloc(struct sas_port *, enum sas_device_type);
|
extern struct sas_rphy *sas_expander_alloc(struct sas_port *, enum sas_device_type);
|
||||||
void sas_rphy_free(struct sas_rphy *);
|
void sas_rphy_free(struct sas_rphy *);
|
||||||
|
|
Loading…
Reference in a new issue