mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Octavio Alvarez <octallk1@alvarezp.org>
To: Thomas Gleixner <tglx@linutronix.de>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
	LKML <linux-kernel@vger.kernel.org>,
	x86@kernel.org, Ingo Molnar <mingo@redhat.com>,
	Borislav Petkov <bp@alien8.de>, "H. Peter Anvin" <hpa@zytor.com>,
	Marc Zyngier <marc.zyngier@arm.com>,
	"Rafael J. Wysocki" <rafael.j.wysocki@intel.com>,
	"Steven Rostedt (VMware)" <rostedt@goodmis.org>,
	Jiang Biao <jiang.biao2@zte.com.cn>,
	Yi Wang <wang.yi59@zte.com.cn>,
	Dou Liyang <douly.fnst@cn.fujitsu.com>,
	Nicolai Stange <nstange@suse.de>,
	Mirko Lindner <mlindner@marvell.com>,
	Stephen Hemminger <stephen@networkplumber.org>
Subject: Re: PROBLEM: Marvell 88E8040 (sky2) fails after hibernation
Date: Mon, 15 Jul 2019 11:40:31 -0500	[thread overview]
Message-ID: <82fa0f47-ccb9-18fc-e35d-af02df37e3fb@alvarezp.org> (raw)
In-Reply-To: <alpine.DEB.2.21.1906271632300.32342@nanos.tec.linutronix.de>

On 6/27/19 9:38 AM, Thomas Gleixner wrote:
>> I did two tests:
>>
>> If I boot with sky2.disable_msi=1 on the kernel cmdline then the problem goes
>> away (when back from hibernation, the NIC works OK).
>>
>> If I boot regularly (disable_msi not set) and then do modprobe -r sky2;
>> modprobe sky2 disable_msi=1, the problem stays (when back from hibernation,
>> the NIC does not work).
> 
> Interesting. Did you verify that the driver still uses INTx after
> hibernation in /proc/interrupts?
> 
>   cat /proc/interrupts | grep eth0
> 
> The 6st column should show IO-APIC for INTx. If it shows PCI-MSI then
> something went wrong

Hi, Thomas,

If I reboot with sky2.disable_msi=1, then I get IO-APIC and the bug does 
not occur:

  19:          0          0          0          0   IO-APIC  19-fasteoi eth0

However, if I reboot without sky2.disable_msi=1 it properly starts as 
PCI-MSI and then, after re-modprobing it it goes to IO-APIC, but the bug 
occurs anyway:

$ cat /proc/interrupts | grep eth
  27:          0          1          0          0   PCI-MSI 3145728-edge 
      eth0

$ sudo modprobe -r sky2
[sudo] password for alvarezp:

$ sudo modprobe sky2 disable_msi=1

$ # hibernating and coming back hibernation

$ cat /proc/interrupts | grep eth
  19:          0          0          0          0   IO-APIC  19-fasteoi 
  eth0


> Also please check Linus suspicion about the module being reloaded after
> hibernation through some distro magic.

This is not happening. Each time the driver is loaded the message "sky2: 
driver version 1.30" is shown.

I confirm only 1 line for the sky2.disable_msi=1 from kernel boot and 
only 2 lines for re-modprobing.

Best regards,
Octavio.

  reply	other threads:[~2019-07-15 16:40 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <aba1c363-92de-66d7-4aac-b555f398e70a@alvarezp.org>
2019-06-01 13:16 ` Octavio Alvarez
2019-06-13  5:02 ` Octavio Alvarez
2019-06-13 20:45   ` Thomas Gleixner
2019-06-20  1:37     ` Octavio Alvarez
2019-06-22 13:27       ` Thomas Gleixner
2019-06-23 12:54         ` Thomas Gleixner
2019-06-24 15:36           ` Stephen Hemminger
2019-06-24 15:39             ` Thomas Gleixner
2019-06-24 17:08           ` Octavio Alvarez
2019-06-24 19:48             ` Linus Torvalds
2019-06-27 14:38             ` Thomas Gleixner
2019-07-15 16:40               ` Octavio Alvarez [this message]
2019-07-15 19:09                 ` Thomas Gleixner
2019-07-15 20:40                   ` Stephen Hemminger

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=82fa0f47-ccb9-18fc-e35d-af02df37e3fb@alvarezp.org \
    --to=octallk1@alvarezp.org \
    --cc=bp@alien8.de \
    --cc=douly.fnst@cn.fujitsu.com \
    --cc=hpa@zytor.com \
    --cc=jiang.biao2@zte.com.cn \
    --cc=linux-kernel@vger.kernel.org \
    --cc=marc.zyngier@arm.com \
    --cc=mingo@redhat.com \
    --cc=mlindner@marvell.com \
    --cc=nstange@suse.de \
    --cc=rafael.j.wysocki@intel.com \
    --cc=rostedt@goodmis.org \
    --cc=stephen@networkplumber.org \
    --cc=tglx@linutronix.de \
    --cc=torvalds@linux-foundation.org \
    --cc=wang.yi59@zte.com.cn \
    --cc=x86@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

all inboxes | Powered by JetHome®