From: Arto Vuori <avuori@ssh.com>
To: Jean-Denis Boyer <jdboyer@mediatrix.com>
Cc: "'Ricardo Scop'" <scop@digitel.com.br>,
linuxppc-embedded@lists.linuxppc.org,
Dan Malek <dan@embeddededge.com>,
andy_lowe@mvista.com
Subject: Re: Kernel oops while routing
Date: Fri, 07 Dec 2001 17:56:48 +0200 [thread overview]
Message-ID: <3C10E6C0.923FA543@ssh.com> (raw)
In-Reply-To: F1BED55F35F4D3118C0F00E0295CFF4DFF9E7F@mail.mediatrix.com
Jean-Denis Boyer wrote:
> IMHO, I could suggest an easier patch, that would result in modifying only
> one line of code, without changing the 'tx_full' logic. In function
> fcc_enet_start_xmit, instead of checking the ready bit (which is bad), we
> could only check if cur_tx has reached dirty_tx, and then call
> netif_stop_queue. Does it make sense?
I have fixed our version of the driver with very similar modifications.
Since that it has not leaked any memory while it has been under testing
for several months.
>
> BTW, I worked hard last week in debugging the fcc_enet driver. It was not
> handling correctly some transmission errors, resulting in the transmitter
> completely stopping, without restarting. This is related to an errata
> (CPM37) from Motorola about the 8260, concerning the way of restarting the
> transmitter. If someone is interested, I can release a patch for that.
>
I have noticed the same problem on old kernel versions. I think i
mentioned about it on the mailing list and i understood that it had been
already fixed on later versions, but i didn't check the status of the
latest driver version.
- Arto
--
Arto Vuori
email: avuori@ssh.com
mobile: +358 40 754 5223
** Sent via the linuxppc-embedded mail list. See http://lists.linuxppc.org/
next prev parent reply other threads:[~2001-12-07 15:56 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-12-05 18:01 Kernel oops while routing Jean-Denis Boyer
2001-12-05 18:21 ` Peter Desnoyers
2001-12-05 18:33 ` Dan Malek
2001-12-05 22:41 ` Re[2]: " Ricardo Scop
2001-12-07 15:56 ` Arto Vuori [this message]
-- strict thread matches above, loose matches on Subject: below --
2001-12-05 18:38 Jean-Denis Boyer
2001-11-26 16:54 Ricardo Scop
2001-11-29 22:27 ` Ricardo Scop
2001-11-29 22:42 ` Dan Malek
2001-12-05 3:24 ` Re[2]: " Ricardo Scop
2001-12-05 17:56 ` Dan Malek
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=3C10E6C0.923FA543@ssh.com \
--to=avuori@ssh.com \
--cc=andy_lowe@mvista.com \
--cc=dan@embeddededge.com \
--cc=jdboyer@mediatrix.com \
--cc=linuxppc-embedded@lists.linuxppc.org \
--cc=scop@digitel.com.br \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.