On 19.02.21 17:15:50, Krzysztof Wilczyński wrote:

Hi Robert,

[...]

quoted

Obiously this is meant here:

	if (!pci_is_managed(dev))

[...]

A question to improve my understanding for future reference.  Was the
previous approach of checking for "enabled" flag from struct pci_devres
was not a good choice here?

Initially this was meant to just show the idea.

After careful review I don't see this additional check is required as
once the pci dev is managed, it will be always released with
pcim_release().

-Robert