Re: [PATCH -tip] x86: reboot_fixup_32.c add missing pci_dev_put

From: Ingo Molnar
Date: Wed Dec 02 2009 - 05:10:10 EST



* Xiaotian Feng <dfeng@xxxxxxxxxx> wrote:

> pci_get_device increments reference count to the found pci device,
> that should be decremented using pci_dev_put.

> +++ b/arch/x86/kernel/reboot_fixups_32.c
> @@ -80,6 +80,7 @@ void mach_reboot_fixups(void)
> continue;
>
> cur->reboot_fixup(dev);
> + pci_dev_put(dev);

Does this matter in practice? We are about to reboot. (but if it's
cleaner to do it like this or if there's some real bug fixed by this
then please spell it out in the changelog.)

Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/