public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH v3 00/31] libsas and pm8001 fixes
@ 2022-02-14  2:17 Damien Le Moal
  2022-02-14  2:17 ` [PATCH v3 01/31] scsi: libsas: Fix sas_ata_qc_issue() handling of NCQ NON DATA commands Damien Le Moal
                   ` (32 more replies)
  0 siblings, 33 replies; 61+ messages in thread
From: Damien Le Moal @ 2022-02-14  2:17 UTC (permalink / raw)
  To: linux-scsi, Martin K . Petersen, John Garry, Xiang Chen,
	Jason Yan, Luo Jiaxing

This first part of this series (patches 1 to 24) fixes handling of NCQ
NON DATA commands in libsas and many bugs in the pm8001 driver.

The fixes for the pm8001 driver include:
* Suppression of all sparse warnings, fixing along the way many le32
  handling bugs for big-endian architectures
* Fix handling of NCQ NON DATA commands
* Fix of tag values handling (0 *is* a valid tag value)
* Fix many tag iand memory leaks in error path
* Fix NCQ error recovery (abort all task execution) that was causing a
  crash

The second part of the series (patches 25 to 31) iadd a small cleanup of
libsas code and many simplifications of the pm8001 driver code.

With these fixes, libzbc test suite passes all test case. This test
suite was used with an SMR drive for testing because it generates many
NCQ NON DATA commands (for zone management commands) and also generates
many NCQ command errors to check ASC/ASCQ returned by the device. With
the test suite, the error recovery path was extensively exercised. The
same tests were also executed with a SAS SMR drives to exercise the
error path.

The patches are based on the 5.18/scsi-staging tree.

Changes from v2:
* Reorganized the series: fixes first, cleanups second.
* Added more bug/leaks fix patches
* Addressed Gary's comment for the ccb alloc helper (patch 28)
* Rebased (and tested) all patches on 5.18/scsi-staging

Changes from v1:
* Added reviewed-by tags
* Addressed Christoph's comments on patch 4 and 8
* Added patches 21 and 22 to fix 2 additional problems found while
  preparing this v2 series
* Added patch 23 and 24 to cleanup the code further.

Damien Le Moal (31):
  scsi: libsas: Fix sas_ata_qc_issue() handling of NCQ NON DATA commands
  scsi: pm8001: Fix __iomem pointer use in pm8001_phy_control()
  scsi: pm8001: Fix pm8001_update_flash() local variable type
  scsi: pm8001: Fix command initialization in pm80XX_send_read_log()
  scsi: pm8001: Fix pm80xx_pci_mem_copy() interface
  scsi: pm8001: Fix command initialization in pm8001_chip_ssp_tm_req()
  scsi: pm8001: Fix payload initialization in pm80xx_set_thermal_config()
  scsi: pm8001: Fix le32 values handling in pm80xx_set_sas_protocol_timer_config()
  scsi: pm8001: Fix payload initialization in pm80xx_encrypt_update()
  scsi: pm8001: Fix le32 values handling in pm80xx_chip_ssp_io_req()
  scsi: pm8001: Fix le32 values handling in pm80xx_chip_sata_req()
  scsi: pm8001: Fix use of struct set_phy_profile_req fields
  scsi: pm8001: Remove local variable in pm8001_pci_resume()
  scsi: pm8001: Fix NCQ NON DATA command task initialization
  scsi: pm8001: Fix NCQ NON DATA command completion handling
  scsi: pm8001: Fix abort all task initialization
  scsi: pm8001: Fix pm8001_tag_alloc() failures handling
  scsi: pm8001: Fix pm80xx_chip_phy_ctl_req()
  scsi: pm8001: Fix pm8001_mpi_task_abort_resp()
  scsi: pm8001: Fix tag values handling
  scsi: pm8001: Fix task leak in pm8001_send_abort_all()
  scsi: pm8001: Fix tag leaks on error
  scsi: pm8001: fix memory leak in pm8001_chip_fw_flash_update_req()
  scsi: pm8001: Fix process_one_iomb() kdoc comment
  scsi: libsas: Simplify sas_ata_qc_issue() detection of NCQ commands
  scsi: pm8001: Simplify pm8001_get_ncq_tag()
  scsi: pm8001: Cleanup pm8001_queue_command()
  scsi: pm8001: Introduce ccb alloc/free helpers
  scsi: pm8001: Simplify pm8001_mpi_build_cmd() interface
  scsi: pm8001: Simplify pm8001_task_exec()
  scsi: pm8001: Simplify pm8001_ccb_task_free()

 drivers/scsi/libsas/sas_ata.c     |  11 +-
 drivers/scsi/pm8001/pm8001_ctl.c  |   5 +-
 drivers/scsi/pm8001/pm8001_hwi.c  | 450 ++++++++++++-----------------
 drivers/scsi/pm8001/pm8001_init.c |  11 +-
 drivers/scsi/pm8001/pm8001_sas.c  | 297 +++++++++----------
 drivers/scsi/pm8001/pm8001_sas.h  |  64 ++++-
 drivers/scsi/pm8001/pm80xx_hwi.c  | 458 ++++++++++++++----------------
 drivers/scsi/pm8001/pm80xx_hwi.h  |   2 +-
 8 files changed, 605 insertions(+), 693 deletions(-)

-- 
2.34.1


^ permalink raw reply	[flat|nested] 61+ messages in thread

end of thread, other threads:[~2022-02-18 11:21 UTC | newest]

Thread overview: 61+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-02-14  2:17 [PATCH v3 00/31] libsas and pm8001 fixes Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 01/31] scsi: libsas: Fix sas_ata_qc_issue() handling of NCQ NON DATA commands Damien Le Moal
2022-02-14 17:56   ` John Garry
2022-02-14 22:23     ` Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 02/31] scsi: pm8001: Fix __iomem pointer use in pm8001_phy_control() Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 03/31] scsi: pm8001: Fix pm8001_update_flash() local variable type Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 04/31] scsi: pm8001: Fix command initialization in pm80XX_send_read_log() Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 05/31] scsi: pm8001: Fix pm80xx_pci_mem_copy() interface Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 06/31] scsi: pm8001: Fix command initialization in pm8001_chip_ssp_tm_req() Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 07/31] scsi: pm8001: Fix payload initialization in pm80xx_set_thermal_config() Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 08/31] scsi: pm8001: Fix le32 values handling in pm80xx_set_sas_protocol_timer_config() Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 09/31] scsi: pm8001: Fix payload initialization in pm80xx_encrypt_update() Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 10/31] scsi: pm8001: Fix le32 values handling in pm80xx_chip_ssp_io_req() Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 11/31] scsi: pm8001: Fix le32 values handling in pm80xx_chip_sata_req() Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 12/31] scsi: pm8001: Fix use of struct set_phy_profile_req fields Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 13/31] scsi: pm8001: Remove local variable in pm8001_pci_resume() Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 14/31] scsi: pm8001: Fix NCQ NON DATA command task initialization Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 15/31] scsi: pm8001: Fix NCQ NON DATA command completion handling Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 16/31] scsi: pm8001: Fix abort all task initialization Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 17/31] scsi: pm8001: Fix pm8001_tag_alloc() failures handling Damien Le Moal
2022-02-14 18:02   ` John Garry
2022-02-14  2:17 ` [PATCH v3 18/31] scsi: pm8001: Fix pm80xx_chip_phy_ctl_req() Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 19/31] scsi: pm8001: Fix pm8001_mpi_task_abort_resp() Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 20/31] scsi: pm8001: Fix tag values handling Damien Le Moal
2022-02-15 11:09   ` John Garry
2022-02-15 23:44     ` Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 21/31] scsi: pm8001: Fix task leak in pm8001_send_abort_all() Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 22/31] scsi: pm8001: Fix tag leaks on error Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 23/31] scsi: pm8001: fix memory leak in pm8001_chip_fw_flash_update_req() Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 24/31] scsi: pm8001: Fix process_one_iomb() kdoc comment Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 25/31] scsi: libsas: Simplify sas_ata_qc_issue() detection of NCQ commands Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 26/31] scsi: pm8001: Simplify pm8001_get_ncq_tag() Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 27/31] scsi: pm8001: Cleanup pm8001_queue_command() Damien Le Moal
2022-02-15 10:55   ` John Garry
2022-02-16 11:36     ` Damien Le Moal
2022-02-16 11:38       ` John Garry
2022-02-16 11:42         ` Damien Le Moal
2022-02-16 11:50           ` John Garry
2022-02-16 12:05             ` Damien Le Moal
2022-02-16 12:21               ` John Garry
2022-02-17  0:12                 ` Damien Le Moal
2022-02-17  9:23                   ` John Garry
2022-02-17 10:47                     ` Damien Le Moal
2022-02-17 12:49                       ` John Garry
2022-02-18  3:12                         ` Damien Le Moal
2022-02-18 11:21                           ` John Garry
2022-02-17 11:47                     ` Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 28/31] scsi: pm8001: Introduce ccb alloc/free helpers Damien Le Moal
2022-02-15 11:07   ` John Garry
2022-02-15 23:41     ` Damien Le Moal
2022-02-16 11:43     ` Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 29/31] scsi: pm8001: Simplify pm8001_mpi_build_cmd() interface Damien Le Moal
2022-02-14  2:17 ` [PATCH v3 30/31] scsi: pm8001: Simplify pm8001_task_exec() Damien Le Moal
2022-02-15  8:57   ` John Garry
2022-02-14  2:17 ` [PATCH v3 31/31] scsi: pm8001: Simplify pm8001_ccb_task_free() Damien Le Moal
2022-02-14  2:23 ` [PATCH v3 00/31] libsas and pm8001 fixes Damien Le Moal
2022-02-15  3:18   ` Martin K. Petersen
2022-02-15  7:38     ` Damien Le Moal
2022-02-14 18:06 ` John Garry
2022-02-14 22:29   ` Damien Le Moal
2022-02-15  8:16     ` John Garry

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox