mbox series

[0/2] Fix IOMMU page fault on report zones

Message ID 20240719073913.179559-1-dlemoal@kernel.org
Headers show
Series Fix IOMMU page fault on report zones | expand

Message

Damien Le Moal July 19, 2024, 7:39 a.m. UTC 
A couple of patches to avoid IOMMU page faults error when executing a
report zones command on ATA SMR drives connected to mpt3sas or mpi3mr
SAS HBAs. These page faults happen only with AMD hosts and are not
triggered with Intel machines.

Damien Le Moal (2):
  scsi: mpi3mr: Avoid IOMMU page faults on report zones
  scsi: mpt3sas: Avoid IOMMU page faults on report zones

 drivers/scsi/mpi3mr/mpi3mr_os.c     | 11 +++++++++++
 drivers/scsi/mpt3sas/mpt3sas_base.c | 20 ++++++++++++++++++--
 2 files changed, 29 insertions(+), 2 deletions(-)

Comments

Christoph Hellwig July 19, 2024, 7:46 a.m. UTC | #1 
Pretty sad that people think they can get away with this, but as we'll
have to deal with it:

Reviewed-by: Christoph Hellwig <hch@lst.de>
Johannes Thumshirn July 19, 2024, 12:49 p.m. UTC | #2 
On 19.07.24 09:39, Damien Le Moal wrote:
> A couple of patches to avoid IOMMU page faults error when executing a
> report zones command on ATA SMR drives connected to mpt3sas or mpi3mr
> SAS HBAs. These page faults happen only with AMD hosts and are not
> triggered with Intel machines.
> 
> Damien Le Moal (2):
>    scsi: mpi3mr: Avoid IOMMU page faults on report zones
>    scsi: mpt3sas: Avoid IOMMU page faults on report zones
> 
>   drivers/scsi/mpi3mr/mpi3mr_os.c     | 11 +++++++++++
>   drivers/scsi/mpt3sas/mpt3sas_base.c | 20 ++++++++++++++++++--
>   2 files changed, 29 insertions(+), 2 deletions(-)
> 

Looks good,
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
Martin K. Petersen July 23, 2024, 12:37 a.m. UTC | #3 
Damien,

> A couple of patches to avoid IOMMU page faults error when executing a
> report zones command on ATA SMR drives connected to mpt3sas or mpi3mr
> SAS HBAs. These page faults happen only with AMD hosts and are not
> triggered with Intel machines.

Applied to 6.11/scsi-staging, thanks!