Comment 155 for bug 160413

Hi there :D

Actually, I haven't tested the suspend, because I have no swap / no resume partition configured ... I will try to use a regular file or create the partition and let You know ..... but I suppose I won't be able to wake my computer up after the ifdown+suspend ...

And ... YES ...
I was really thinking about the following changes several times :)

why 2.6.30.1 contains :
>static struct pci_driver rtl8169_pci_driver = {
> .name = MODULENAME,
> .id_table = rtl8169_pci_tbl,
> .probe = rtl8169_init_one,
> .remove = __devexit_p(rtl8169_remove_one),
>#ifdef CONFIG_PM
> .suspend = rtl8169_suspend,
> .resume = rtl8169_resume,
> .shutdown = rtl_shutdown,
>#endif
>};
(and the suspend was called from the rtl_shutdown ...)

and 2.6.31.rc3 contains just :
>static struct pci_driver rtl8169_pci_driver = {
> .name = MODULENAME,
> .id_table = rtl8169_pci_tbl,
> .probe = rtl8169_init_one,
> .remove = __devexit_p(rtl8169_remove_one),
> .shutdown = rtl_shutdown,
> .driver.pm = RTL8169_PM_OPS,
>};
(and there's no suspend defined here... )
?

I suppose You know why :)

BR, J.