Re: [RESEND PATCH] PCI: hotplug: check the return of hotplug bridge

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Aug 31, 2024 at 10:28:21AM -0300, Guilherme Giacomo Simoes wrote:
> In some pci drivers, when the pci bridge is added if the process return an error,
> the drivers don't check this and continue your execute normally.
> Then, this patch change this drivers for check return of pci_hp_add_bridge(), and
> if has an error, then the drivers call goto lable , free your mutex and return the
> error for your caller.
> 
> Signed-off-by: Guilherme Giacomo Simoes <trintaeoitogc@xxxxxxxxx>

Did you see Lukas' response at
https://lore.kernel.org/r/ZsAzM8K9PnN5jxR9@xxxxxxxxx ?

AFAICT the discussion he pointed to is very relevant and needs to be
considered before we apply a patch like this.

If you have addressed those comments, please describe how you
addressed them in the commit log, and include links to Nam Cao's work
and cc them as well.

Bjorn




[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux