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
F120056396: D50153.diff
Sat, Jun 14, 4:14 PM
F119979799: D50153.diff
Fri, Jun 13, 11:20 PM
Unknown Object (File)
Tue, Jun 3, 11:29 AM
Unknown Object (File)
Mon, Jun 2, 7:15 PM
Unknown Object (File)
Thu, May 22, 12:48 PM
Unknown Object (File)
Tue, May 20, 7:49 PM
Unknown Object (File)
Fri, May 16, 6:59 PM
Unknown Object (File)
May 11 2025, 8:20 AM
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