From: David Miller <davem@davemloft.net>
To: avorontsov@mvista.com
Cc: linuxppc-dev@ozlabs.org, afleming@freescale.com,
Sandeep.Kumar@freescale.com, manfred.rudigier@omicron.at,
netdev@vger.kernel.org
Subject: Re: [PATCH v2 2/3] gianfar: Implement workaround for eTSEC76 erratum
Date: Wed, 30 Jun 2010 11:37:19 -0700 (PDT) [thread overview]
Message-ID: <20100630.113719.128591285.davem@davemloft.net> (raw)
In-Reply-To: <20100630163913.GB23337@oksana.dev.rtsoft.ru>
From: Anton Vorontsov <avorontsov@mvista.com>
Date: Wed, 30 Jun 2010 20:39:13 +0400
> MPC8313ECE says:
>
> "For TOE=1 huge or jumbo frames, the data required to generate the
> checksum may exceed the 2500-byte threshold beyond which the controller
> constrains itself to one memory fetch every 256 eTSEC system clocks.
>
> This throttling threshold is supposed to trigger only when the
> controller has sufficient data to keep transmit active for the duration
> of the memory fetches. The state machine handling this threshold,
> however, fails to take large TOE frames into account. As a result,
> TOE=1 frames larger than 2500 bytes often see excess delays before start
> of transmission."
>
> This patch implements the workaround as suggested by the errata
> document, i.e.:
>
> "Limit TOE=1 frames to less than 2500 bytes to avoid excess delays due to
> memory throttling.
> When using packets larger than 2700 bytes, it is recommended to turn TOE
> off."
>
> To be sure, we limit the TOE frames to 2500 bytes, and do software
> checksumming instead.
>
> Signed-off-by: Anton Vorontsov <avorontsov@mvista.com>
Applied.
next prev parent reply other threads:[~2010-06-30 18:37 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-29 20:59 [PATCH 1/3] gianfar: Implement workaround for eTSEC74 erratum Anton Vorontsov
2010-06-29 22:16 ` David Miller
2010-06-30 16:38 ` Anton Vorontsov
2010-06-30 16:39 ` [PATCH v2 " Anton Vorontsov
2010-06-30 18:37 ` David Miller
2010-06-30 16:39 ` [PATCH v2 2/3] gianfar: Implement workaround for eTSEC76 erratum Anton Vorontsov
2010-06-30 18:37 ` David Miller [this message]
2010-06-30 16:39 ` [PATCH v2 3/3] gianfar: Implement workaround for eTSEC-A002 erratum Anton Vorontsov
2010-06-30 18:37 ` David Miller
2010-06-30 18:37 ` [PATCH 1/3] gianfar: Implement workaround for eTSEC74 erratum David Miller
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=20100630.113719.128591285.davem@davemloft.net \
--to=davem@davemloft.net \
--cc=Sandeep.Kumar@freescale.com \
--cc=afleming@freescale.com \
--cc=avorontsov@mvista.com \
--cc=linuxppc-dev@ozlabs.org \
--cc=manfred.rudigier@omicron.at \
--cc=netdev@vger.kernel.org \
/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;
as well as URLs for NNTP newsgroup(s).