[PATCH v6 0/4] scsi: libsas: Fix the failure of adding phy with zero-address to port

Xingui Yang posted 4 patches 1 year, 9 months ago
drivers/scsi/libsas/sas_expander.c | 38 +++++++++++++++++++++++-------
drivers/scsi/libsas/sas_internal.h | 15 ------------
2 files changed, 30 insertions(+), 23 deletions(-)
[PATCH v6 0/4] scsi: libsas: Fix the failure of adding phy with zero-address to port
Posted by Xingui Yang 1 year, 9 months ago
This series is to solve the problem of a BUG() when adding phy with zero
address to a new port.

v5 -> v6
1. rename sas_add_parent_port() to sas_ex_add_parent_port() based on
   John's suggestion.
2. Distill port settings into a single patch.
3. Update comments information.

v4 -> v5
1. Add new helper sas_port_add_ex_phy() based on John's suggestion.
2. Move sas_add_parent_port() to sas_expander.c based on John's suggestion.
3. Update the comments.

v3 -> v4:
1. Update patch title and comments based on John's suggestion.

v2 -> v3:
1. Set ex_dev->parent_port to NULL when the number of PHYs of the parent
   port becomes 0.
2. Update the comments.

v1 -> v2:
1. Set ex_phy->port with parent_port when ex_phy is added to the parent
   port.
2. Set ex_phy to NULL when free expander.
3. Update the comments.

Xingui Yang (4):
  scsi: libsas: Add helper for port add ex_phy
  scsi: libsas: Move sas_add_parent_port() to sas_expander.c
  scsi: libsas: Set port when ex_phy is added or deleted
  scsi: libsas: Fix the failure of adding phy with zero-address to port

 drivers/scsi/libsas/sas_expander.c | 38 +++++++++++++++++++++++-------
 drivers/scsi/libsas/sas_internal.h | 15 ------------
 2 files changed, 30 insertions(+), 23 deletions(-)

-- 
2.17.1
Re: [PATCH v6 0/4] scsi: libsas: Fix the failure of adding phy with zero-address to port
Posted by Martin K. Petersen 1 year, 7 months ago
On Tue, 12 Mar 2024 14:10:59 +0000, Xingui Yang wrote:

> This series is to solve the problem of a BUG() when adding phy with zero
> address to a new port.
> 
> v5 -> v6
> 1. rename sas_add_parent_port() to sas_ex_add_parent_port() based on
>    John's suggestion.
> 2. Distill port settings into a single patch.
> 3. Update comments information.
> 
> [...]

Applied to 6.10/scsi-queue, thanks!

[1/4] scsi: libsas: Add helper for port add ex_phy
      https://git.kernel.org/mkp/scsi/c/888ea1b12b06
[2/4] scsi: libsas: Move sas_add_parent_port() to sas_expander.c
      https://git.kernel.org/mkp/scsi/c/48032c0be6c7
[3/4] scsi: libsas: Set port when ex_phy is added or deleted
      https://git.kernel.org/mkp/scsi/c/7a165a81d55f
[4/4] scsi: libsas: Fix the failure of adding phy with zero-address to port
      https://git.kernel.org/mkp/scsi/c/06036a0a5db3

-- 
Martin K. Petersen	Oracle Linux Engineering
Re: [PATCH v6 0/4] scsi: libsas: Fix the failure of adding phy with zero-address to port
Posted by Martin K. Petersen 1 year, 8 months ago
Xingui,

> This series is to solve the problem of a BUG() when adding phy with zero
> address to a new port.

Applied to 6.10/scsi-staging, thanks!

-- 
Martin K. Petersen	Oracle Linux Engineering