On Wed, Aug 6, 2008 at 1:03 PM, Francois Romieu <romieu@fr.zoreil.com> wrote:
quoted text
> John P Poet <jppoet@gmail.com> :
> [...]
>> Thank you for the response.  That DOES seem to fix the cold-boot issue.
>>
>> Unfortunately, it does not fix the timeout problem.  I am still
>> experiencing transmittion problems:
>
> Does the patch below (on top of the current pile) make a difference ?
>
> diff --git a/drivers/net/r8169.c b/drivers/net/r8169.c
> index c421bf6..348846f 100644
> --- a/drivers/net/r8169.c
> +++ b/drivers/net/r8169.c
> @@ -874,12 +874,9 @@ static int rtl8169_set_speed_xmii(struct net_device *dev,
>
>        auto_nego |= ADVERTISE_PAUSE_CAP | ADVERTISE_PAUSE_ASYM;
>
> -       if ((tp->mac_version == RTL_GIGA_MAC_VER_12) ||
> -           (tp->mac_version == RTL_GIGA_MAC_VER_17)) {
> -               /* Vendor specific (0x1f) and reserved (0x0e) MII registers. */
> -               mdio_write(ioaddr, 0x1f, 0x0000);
> -               mdio_write(ioaddr, 0x0e, 0x0000);
> -       }
> +       /* Vendor specific (0x1f) and reserved (0x0e) MII registers. */
> +       mdio_write(ioaddr, 0x1f, 0x0000);
> +       mdio_write(ioaddr, 0x0e, 0x0000);
>
>        tp->phy_auto_nego_reg = auto_nego;
>        tp->phy_1000_ctrl_reg = giga_ctrl;
> --
> Ueimor
>

That does not make any difference.

I will say, that the problem seems to be reduced by this patch (or the
0002-r8169-avoid-thrashing-PCI-conf-space-above-RTL_GIGA.patch).

Before, I was getting failures two or three times an hour.  With
either of these patches applied, the failure rate has dropped to one
every two to four hours.

Thanks for trying.

John
-- 
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html