LinuxPPC-Dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: "sjoyeau@wanadoo.fr" <sjoyeau@wanadoo.fr>
To: Scott Coulter <scott.coulter@cyclone.com>
Cc: linuxppc-dev@ozlabs.org
Subject: Re: Gianfar tx-babbling-errors
Date: Thu, 19 Feb 2009 17:48:03 +0100	[thread overview]
Message-ID: <d194b8ce0902190848hb53166cj32e7d0bd65021d2e@mail.gmail.com> (raw)
In-Reply-To: <43EB80E07C42E1408726E4905FB96B04C07700@CYBORG3.cyclone.com>

[-- Attachment #1: Type: text/plain, Size: 1650 bytes --]

Hi Scott,

Your issue may come from data setup (or corruption) instead of code path:
babbling error may occurs when a TSEC TX descriptor hasn't its "last frame"
bit set or when the data length is greated than max frame length.

--
sj

2009/2/19 Scott Coulter <scott.coulter@cyclone.com>

>
>
> Kumar,
>
> >
> > can't think of any.  How about adding a BUG_ON() in the tx path to see
> > if the buffer size > MTU and re-run your tests.
> >
>
> So, here are the checks I've tried in gfar_start_xmit():
>
> BUG_ON(skb->len > DEFAULT_RX_BUFFER_SIZE)
>
> BUG_ON(skb->len > priv->regs->maxfrm)
>
> Neither produces a bug check yet ethtool reports non-zero
> tx-babbling-errors.  The last check appears to be the definition of
> tx-babbling-errors.  Is there a transmit path that I have missed?
>
> Scott
>
> ___________________________________________________________________
>
>  Scott N. Coulter
>  Senior Software Engineer
>
>  Cyclone Microsystems
>  370 James Street              Phone:  203.786.5536 ext. 118
>  New Haven, CT 06513-3051      Email:  scott.coulter@cyclone.com
>  U.S.A.                        Web:    http://www.cyclone.com
> ___________________________________________________________________
>
> _______________________________________________
> Linuxppc-dev mailing list
> Linuxppc-dev@ozlabs.org
> https://ozlabs.org/mailman/listinfo/linuxppc-dev
>



-- 
------------------
Sylvain JOYEAU
Freelance Engineer
Software RT-OS R&D
sylvain.joyeau@gmail.com
Tél: +33-(0)667 477 052
"A good idea is one side of the coin. The other side is the practical
usefulness". J. Liedke.

[-- Attachment #2: Type: text/html, Size: 2679 bytes --]

  parent reply	other threads:[~2009-02-19 16:48 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-18 16:16 Gianfar tx-babbling-errors Scott Coulter
2009-02-18 17:03 ` Kumar Gala
2009-02-18 17:22   ` Scott Coulter
2009-02-18 17:29     ` Kumar Gala
2009-02-18 20:45       ` Scott Coulter
2009-02-19 16:17       ` Scott Coulter
2009-02-19 16:29         ` Kumar Gala
2009-02-19 16:38           ` Scott Coulter
2009-02-19 16:48         ` sjoyeau [this message]
2009-02-19 17:03           ` Kumar Gala
2009-02-19 17:29             ` Scott Coulter
2009-02-20 19:16 ` Haruki Dai-R35557
2009-02-20 20:59   ` Scott Coulter
2009-02-20 21:32     ` Haruki Dai-R35557
2009-02-24 16:58       ` Scott Coulter
2009-03-03 16:33       ` Scott Coulter

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=d194b8ce0902190848hb53166cj32e7d0bd65021d2e@mail.gmail.com \
    --to=sjoyeau@wanadoo.fr \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=scott.coulter@cyclone.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