[PATCH 3/4] PCI: hotplug: document unchecked return value of pci_hp_add_bridge()

From: Nam Cao
Date: Fri May 03 2024 - 15:23:50 EST


Some hotplug drivers do not check the returned value of
pci_hp_add_bridge(). This may be probmatic if the driver proceed while
pci_hp_add_bridge() fails.

Signed-off-by: Nam Cao <namcao@xxxxxxxxxxxxx>
---
drivers/pci/hotplug/TODO | 6 ++++++
1 file changed, 6 insertions(+)

diff --git a/drivers/pci/hotplug/TODO b/drivers/pci/hotplug/TODO
index fdb8dd6ea24d..f0a1746c3c88 100644
--- a/drivers/pci/hotplug/TODO
+++ b/drivers/pci/hotplug/TODO
@@ -6,6 +6,8 @@ cpcihp:
->set_power callbacks in struct cpci_hp_controller_ops. Why were they
introduced? Can they be removed from the struct?

+* Returned code from pci_hp_add_bridge() is not checked.
+
cpqphp:

* The driver spawns a kthread cpqhp_event_thread() which is woken by the
@@ -16,6 +18,8 @@ cpqphp:
* A large portion of cpqphp_ctrl.c and cpqphp_pci.c concerns resource
management. Doesn't this duplicate functionality in the core?

+* Returned code from pci_hp_add_bridge() is not checked.
+
ibmphp:

* Implementations of hotplug_slot_ops callbacks such as get_adapter_present()
@@ -43,6 +47,8 @@ ibmphp:
* A large portion of ibmphp_res.c and ibmphp_pci.c concerns resource
management. Doesn't this duplicate functionality in the core?

+* Returned code from pci_hp_add_bridge() is not checked.
+
sgi_hotplug:

* Several functions access the pci_slot member in struct hotplug_slot even
--
2.39.2