> In the pci_acpi_scan_root() function, if the PCI bus creation fails,
failed?
> the allocated memory should be released to avoid memory occupation.
Do you propose to complete the exception handling?
How do you think about to add any tags (like “Fixes” and “Cc”) accordingly?
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.15-rc4#n145
See also:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.15-rc4#n94
…
> +++ b/drivers/pci/pci-acpi.c
…
> @@ -1710,6 +1708,11 @@ struct pci_bus *pci_acpi_scan_root(struct acpi_pci_root *root)
> pcie_bus_configure_settings(child);
>
> return bus;
> +
> +cleanup_exit:
How do you think about to use the label “free_root_ops”?
> + kfree(root_ops);
I suggest to use another label “free_ri” so that a bit of duplicate exception handling code
can be avoided from a previous if branch.
> + kfree(ri);
> + return NULL;
> }
…
How do you think about to benefit any more from the application of the attribute “__free”?
https://elixir.bootlin.com/linux/v6.15-rc4/source/include/linux/slab.h#L476
Regards,
Markus
