Page MenuHomeFreeBSD

LinuxKPI: pci: fix two errors in lkpi_pci_get_device()
ClosedPublic

Authored by bz on May 4 2025, 10:03 PM.
Tags
None
Referenced Files
F156611497: D50153.id.diff
Fri, May 15, 2:50 AM
Unknown Object (File)
Mon, May 11, 7:38 PM
Unknown Object (File)
Mon, May 11, 7:38 PM
Unknown Object (File)
Mon, May 11, 7:29 PM
Unknown Object (File)
Mon, Apr 27, 7:27 PM
Unknown Object (File)
Mon, Apr 27, 12:19 PM
Unknown Object (File)
Thu, Apr 23, 9:31 PM
Unknown Object (File)
Tue, Apr 21, 12:51 PM
Subscribers

Details

Summary

On any result we need to acquire a reference. pci_dev_get() deals with
a NULL argument so we can simply pass what we "found" at the end.
This will avoid reference count problems.

If the iteration on the linux list in lkpi_pci_get_device() does
not find a match pdev will still be defined at the end of the
loop but not pointing to a valid pdev.
Store the found entry in a 2nd variable which otherwise will be NULL.
This will avoid random panics, usually in sysfs_remove_dir() when
the reference gets released.

Found during mt76 bringup.

Sponsored by: The FreeBSD Foundation
MFC after: 3 days
Fixes: 8f61992d7cc11

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable