The Linux Kernel Mailing List
 help / color / mirror / Atom feed
From: Francois Romieu <romieu@fr.zoreil.com>
To: Martin Michlmayr <tbm@cyrius.com>
Cc: Lennert Buytenhek <buytenh@wantstofly.org>,
	Riku Voipio <riku.voipio@iki.fi>,
	linux-kernel@vger.kernel.org
Subject: Re: r8169 on n2100 (was Re: r8169 mac address change (was Re: [0/3] 2.6.19-rc2: known regressions))
Date: Sat, 25 Nov 2006 17:02:20 +0100	[thread overview]
Message-ID: <20061125160220.GA13480@electric-eye.fr.zoreil.com> (raw)
In-Reply-To: <20061125145226.GA526@deprecation.cyrius.com>

Martin Michlmayr <tbm@cyrius.com> :
[...]
> Do you think there'll be a better fix in the future ?

It's the best trade-off that I can figure but there are surely more
knowledgeable people than me. The patch does not completely disable
the reporting of serious PCI errors. If the user knows that it is
otherwise safe, he can disable it: the error will then be reported
only once. I must confess that the history of the 8169 PCI errors is
not crystal clear.

> Do you believe that the boot loader on the N2100 doesn't
> initialize Ethernet properly or that this is a generic problem on iop
> or with this particular RTL chip?  We have fairly good contacts with
> the company producing the N2100 so if it's the former it could
> probably be fixed. (Altough I'm not sure this is the case given that
> Realtek's driver works).

Yes, switching from MM register accesses has been reported to fix/hide
the issue but it's a sledgehammer which does not tell what is going on
(side note: Realtek's driver does not enable the SYSErr irq).

So far I can only tell that 1) the 8169 reports a data parity error on
almost each received packet when it is not silenced 2) the error can
be ignored. If there really is an error and the chipset automatically
retries the transaction, one should expect some loss of efficiency but
it will not necessarily be easy to notice through software.

If I had unlimited resources/time/$$$, I would plug a PCI bus analyzer
and check what is going on. :o)

-- 
Ueimor

  reply	other threads:[~2006-11-25 16:05 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20061107115940.GA23954@unjust.cyrius.com>
2006-11-08 20:35 ` r8169 mac address change (was Re: [0/3] 2.6.19-rc2: known regressions)] Riku Voipio
2006-11-09 22:13   ` Francois Romieu
2006-11-09 23:14     ` Lennert Buytenhek
2006-11-10 18:59       ` Francois Romieu
2006-11-21 10:24         ` Martin Michlmayr
2006-11-21 20:45           ` Francois Romieu
2006-11-22 23:16             ` r8169 on n2100 (was Re: r8169 mac address change (was Re: [0/3] 2.6.19-rc2: known regressions)) Francois Romieu
2006-11-25 14:52               ` Martin Michlmayr
2006-11-25 16:02                 ` Francois Romieu [this message]
2006-12-15 13:27               ` Lennert Buytenhek
2006-12-15 20:15                 ` Francois Romieu
2006-12-15 21:03                   ` Lennert Buytenhek
2006-12-15 21:14                     ` Russell King
2006-12-16 23:09                       ` Lennert Buytenhek
2006-12-16 23:31                         ` Francois Romieu
2006-12-16 23:52                           ` Lennert Buytenhek
     [not found]                             ` <20061217195635.GA10181@kos.to>
2006-12-17 19:28                             ` Martin Michlmayr
2006-12-17 21:02                               ` Riku Voipio
2006-12-17 21:13                                 ` Lennert Buytenhek
2006-12-17 21:40                                   ` Riku Voipio
2006-12-17 21:48                                   ` Francois Romieu
2006-12-17 22:10                                     ` Martin Michlmayr
2006-12-16  0:54                     ` Francois Romieu
2006-12-16  2:09                       ` Lennert Buytenhek
2006-12-16 21:58                         ` Francois Romieu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20061125160220.GA13480@electric-eye.fr.zoreil.com \
    --to=romieu@fr.zoreil.com \
    --cc=buytenh@wantstofly.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=riku.voipio@iki.fi \
    --cc=tbm@cyrius.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox