All of lore.kernel.org
 help / color / mirror / Atom feed
From: andrew@lunn.ch (Andrew Lunn)
To: linux-arm-kernel@lists.infradead.org
Subject: [Patch v2 RfT] ARM: Kirkwood: Fix DT based DSA.
Date: Wed, 10 Sep 2014 16:08:48 +0200	[thread overview]
Message-ID: <20140910140848.GC13342@lunn.ch> (raw)
In-Reply-To: <20140909171742.GT30828@titan.lakedaemon.net>

On Tue, Sep 09, 2014 at 01:17:42PM -0400, Jason Cooper wrote:
> > I've tested v2 of the patch and it worked fine.

Hi Eugene, thanks for testing.
> 
> Ok, I'll consider that a Tested-by...
> 
> > Though, I had a couple of comments:
> > 
> > 1) I have concerns regarding naming of the boards.
> > Since distros use those names to identify boards, we should use
> > shorter and consistent naming.

I agree with Jason here. The filenames don't matter, its the
description in the compatible string which people should be looking
at. And it seems Marvell own naming of these boards is not
particularly good either.

> > other too).  With 3.17rc2 I am not able to get more then ~50Mbps while
> > same board with in same setup running v3.2 or v3.10 easily pass
> > 650Mbps mark.
> 
> Is this with/without this particular patch?  Or is this a separate
> issue?

I've not looked at this yet, but i guess it is related to TSO and the
stack trace we are seeing. It is independent of this patch. Without
this patch, networking is totally borked. With this patch, it somewhat
works, but i guess with packet loss, causing the low TCP bandwidth.

Once i'm back home i will take a look at this.

> Yes, which will be near the end of the window for getting this in.  I'd
> like to have everything hammered out _before_ he returns so he can Ack a
> (potentially) new version that I can apply.

If Eugene wants to produce a version with slightly modified model and
compatible strings, i can Ack it during the next couple of days. I
should have internet access. If not, please take the patch as is, with
the Tested-by added.

    Thanks
	Andrew

  reply	other threads:[~2014-09-10 14:08 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-01 17:35 Andrew Lunn
2014-09-09 16:06 ` Jason Cooper
2014-09-09 16:44   ` Eugene Sanivsky
2014-09-09 17:17     ` Jason Cooper
2014-09-10 14:08       ` Andrew Lunn [this message]
2014-09-12 21:37         ` Andrew Lunn
2014-09-13 21:06 ` Jason Cooper

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=20140910140848.GC13342@lunn.ch \
    --to=andrew@lunn.ch \
    --cc=linux-arm-kernel@lists.infradead.org \
    --subject='Re: [Patch v2 RfT] ARM: Kirkwood: Fix DT based DSA.' \
    /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

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.