netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Niklas Söderlund" <niklas.soderlund@ragnatech.se>
To: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>,
	Linux-Renesas <linux-renesas-soc@vger.kernel.org>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>
Subject: Re: ravb WoL interrupt flood
Date: Mon, 12 Jun 2017 13:29:51 +0200	[thread overview]
Message-ID: <20170612112951.GG17461@bigcity.dyn.berto.se> (raw)
In-Reply-To: <20170530102924.GB7484@bigcity.dyn.berto.se>

Hi Geert,

On 2017-05-30 12:29:24 +0200, Niklas Söderlund wrote:
> Hi Geert,
> 
> Thanks for the report. I did run a 200+ loop suspend/resume without 
> issues so this error is probably hard to reproduce or something changed 
> since v4.11-rc1 which if IIRC was the base I used to do the tests.
> 
> All I can think of is that if the interface was down at the time the 
> system was suspend. There is a bug in  the driver which is in 
> renesas-drivers related to that. But given that you woken up by WoL that 
> is probably not the case :-)
> 
> I will do more tests and see if I can reproduce this once I'm back in 
> Stockholm.

I have now done over a 500 suspend/resume loops and not been able to see 
this :-(

I have run it with and without your '[PATCH/RFC] clk: renesas: cpg-mssr: 
Restore module clock registers during resume' patch but no noticeable 
difference.

I had a small cleanup in my patch which would differ from the one you 
use. The diff is only in effect when resuming from s2ram so it should 
have no effect here but still it is a change. My plan is to submit the 
version with this fix for tomorrows renesas-drivers and then rerun my 
tests using that. If I still can't reproduce it then I think I will 
submit a v2 of my WoL patch to the ML, do this sounds like an OK plan to 
you?

> 
> 
> On 2017-05-30 11:23:23 +0200, Geert Uytterhoeven wrote:
> > Hi Niklas,
> > 
> > I just got a flood of messages after WoL from s2idle on r8a7795/salvator-x:
> > 
> > [  802.341060] ravb e6800000.ethernet eth0: ignoring interrupt, rx
> > status 0x00000001, rx mask 0x00000001,
> > [  802.350369] ravb e6800000.ethernet eth0:                     tx
> > status 0x00000000, tx mask 0x00000001.
> > [  802.359684] ravb e6800000.ethernet eth0: ignoring interrupt, rx
> > status 0x00000001, rx mask 0x00000001,
> > [  802.368993] ravb e6800000.ethernet eth0:                     tx
> > status 0x00000000, tx mask 0x00000001.
> > [  802.378308] ravb e6800000.ethernet eth0: ignoring interrupt, rx
> > status 0x00000001, rx mask 0x00000001,
> > [  802.387616] ravb e6800000.ethernet eth0:                     tx
> > status 0x00000000, tx mask 0x00000001.
> > [  802.396931] ravb e6800000.ethernet eth0: ignoring interrupt, rx
> > status 0x00000001, rx mask 0x00000001,
> > [  802.406240] ravb e6800000.ethernet eth0:                     tx
> > status 0x00000000, tx mask 0x00000001.
> > ...
> > 
> > Unfortunately I cannot reproduce it.
> > 
> > Gr{oetje,eeting}s,
> > 
> >                         Geert
> > 
> > --
> > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
> > 
> > In personal conversations with technical people, I call myself a hacker. But
> > when I'm talking to journalists I just say "programmer" or something like that.
> >                                 -- Linus Torvalds
> 
> -- 
> Regards,
> Niklas Söderlund

-- 
Regards,
Niklas Söderlund

      reply	other threads:[~2017-06-12 11:29 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-05-30  9:23 ravb WoL interrupt flood Geert Uytterhoeven
2017-05-30 10:29 ` Niklas Söderlund
2017-06-12 11:29   ` Niklas Söderlund [this message]

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=20170612112951.GG17461@bigcity.dyn.berto.se \
    --to=niklas.soderlund@ragnatech.se \
    --cc=geert@linux-m68k.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=sergei.shtylyov@cogentembedded.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;
as well as URLs for NNTP newsgroup(s).