linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: w@1wt.eu (Willy Tarreau)
To: linux-arm-kernel@lists.infradead.org
Subject: Issue found in Armada 370: "No buffer space available" error during continuous ping
Date: Mon, 21 Jul 2014 07:44:05 +0200	[thread overview]
Message-ID: <20140721054405.GK21834@1wt.eu> (raw)
In-Reply-To: <CAB8gEUvwPJUhDN-=tugTyKe93SdNwqNaGtKMF1DRCb8ZNncmmg@mail.gmail.com>

Hi Maggie,

On Sun, Jul 20, 2014 at 07:45:13PM -0700, Maggie Mae Roxas wrote:
> Hi Willy,
> Good day.
> 
> BTW, here are some answers to your questions.
> 
> > In fact I don't know if you're running your own board or a "standard"
> one (a mirabox or any NAS board).
> We are using a "customized" one, not a "standard" one.
> We based the design on Armada 370 RD Evaluation Board, but we used
> Marvell 88E1512 as Ethernet PHY and Marvell 88F6707 as processor
> instead of the ones in the Armada 370 RD (I think it uses Marvell
> 88E1310 as Ethernet PHY and Marvell 88F6W11 as processor).

OK. For information, the mirabox uses a 88F6707 and a 88E1510 for
the phy. So it's very close to what you have.

> > Maggie, do you know if it is possible that for any reason your board
> would not deliver an IRQ on Tx completion ? That could explain things.
> > You can easily test reverting commit 4f3a4f701b just in case.
> > If that's the case, then the next step will be to figure out how it is possible
> that IRQs are disabled!
> After reverting 4f3a4f701b, as I reported, issue does not happen anymore.

As you said that you both applied cd71e2 and reverted 4f3a4f, could you
please confirm that with cd71 applied only it was not enough ? I'm
finding it really strange, because as you use the same CPU as the
mirabox, I'm seeing no reason why the IRQ wouldn't work, and since
you're using a slightly different phy from us, the first patch which
changes the the RGMII configuration (cd71e2) would be a more likely
candidate.

> Please let me know how to "figure out how it is possible that IRQs are
> disabled".

Checking /proc/interrupts when you're sending some traffic should show
that the IRQ is increasing from time to time.

> Also, what is the impact if I use this combination?

First you're not using a mainline kernel which means that you'll always
be bothered. Second, removing support for the Tx IRQ means that your
Tx traffic can become very slow (typically 134 Mbps instead of 987 for
unidirectional traffic), which can be a problem if your board is used
as a router for example. If you're building a NAS, you'll have less
impact. Third, considering that other boards work without applying
these changes, it might be possible that there's an issue on your
board, and maybe detecting it early would allow you to fix it for all
future batches, and maybe only apply these patches for the few very
first ones.

Regards,
Willy

  reply	other threads:[~2014-07-21  5:44 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-08  2:20 Issue found in Armada 370: "No buffer space available" error during continuous ping Maggie Mae Roxas
2014-07-08  2:27 ` Maggie Mae Roxas
2014-07-08  8:21   ` Thomas Petazzoni
2014-07-09  6:35     ` Maggie Mae Roxas
2014-07-14  3:55       ` Maggie Mae Roxas
2014-07-15 12:24       ` Thomas Petazzoni
2014-07-15 12:43         ` Willy Tarreau
2014-07-17  5:37           ` Maggie Mae Roxas
2014-07-17  8:15             ` Willy Tarreau
2014-07-21  1:57               ` Maggie Mae Roxas
2014-07-21  2:45                 ` Maggie Mae Roxas
2014-07-21  5:44                   ` Willy Tarreau [this message]
2014-07-21  6:33                     ` Maggie Mae Roxas
2014-07-21  7:03                       ` Willy Tarreau
2014-07-23  2:24                         ` Maggie Mae Roxas
2014-07-23  6:16                           ` Willy Tarreau
2014-07-24  7:24                             ` Maggie Mae Roxas
2014-12-01  6:35                               ` Maggie Mae Roxas
     [not found]                               ` <CAB8gEUtgo-8nets3tRtqiZ8qRx+SyCq2d8v05scavWNwE5TNXg@mail.gmail.com>
2014-12-01  7:28                                 ` Willy Tarreau
2014-12-01  8:27                                   ` Maggie Mae Roxas
2014-12-01  9:28                                     ` Willy Tarreau
2014-12-01  9:32                                       ` Thomas Petazzoni
2014-12-01  9:58                                         ` Willy Tarreau
2014-12-01 10:15                                           ` Maggie Mae Roxas
2014-12-02  4:09                                             ` Maggie Mae Roxas
2014-12-02  6:56                                               ` Willy Tarreau
2014-12-02  7:04                                                 ` Maggie Mae Roxas

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=20140721054405.GK21834@1wt.eu \
    --to=w@1wt.eu \
    --cc=linux-arm-kernel@lists.infradead.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).