xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Keir Fraser <keir.xen@gmail.com>
To: Ben Guthro <ben@guthro.net>, Jan Beulich <JBeulich@suse.com>
Cc: xen-devel <xen-devel@lists.xen.org>,
	john.baboval@citrix.com, Thomas Goetz <thomas.goetz@citrix.com>,
	Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Subject: Re: Xen4.2 S3 regression?
Date: Thu, 20 Sep 2012 07:24:58 +0100	[thread overview]
Message-ID: <CC80754A.3F386%keir.xen@gmail.com> (raw)
In-Reply-To: <CC80728B.3F362%keir.xen@gmail.com>

On 20/09/2012 07:13, "Keir Fraser" <keir.xen@gmail.com> wrote:

> CPU#1 got stuck in loop in cpu_init() as it appears to be Œalready
> initialised¹ in cpu_initialized bitmap. CPU#0 detects it is stuck and carries
> on, but the resume code assumes all CPUs are brought back online and crashes
> later.
> 
> I wonder how long this has been broken. I recall reworking the CPU bringup
> code a lot early during 4.1.0 development... And I didn¹t test S3.
> 
>  -- Keir

However, I did test CPU hotplug a lot, and S3 uses the hotplug logic to take
down and bring up CPUs. So I don't think I can have broken this.

Are you able to hotplug physical CPUs from dom0 using the
tools/misc/xen-hptool utility? If not, at least this might be a friendlier
test method and environment than a full S3.

 -- Keir

> On 19/09/2012 22:07, "Ben Guthro" <ben@guthro.net> wrote:
> 
>> No hardware debugger just yet - but I've moved to another machine (Lenovo
>> T400 laptop) - and am now seeing the following stack trace when I resume
>> (this is using the tip of the 4.2-testing tree)
>> 
>> It looks like either the vcpu, or the runstate is NULL, at this point in the
>> resume process...
>> 
>> 
>> (XEN) Finishing wakeup from ACPI S3 state.
>> (XEN) Enabling non-boot CPUs  ...
>> (XEN) CPU#1 already initialized!
>> (XEN) Stuck ??
>> (XEN) Error taking CPU1 up: -5
>> [   38.570054] ACPI: Low-level resume complete
>> [   38.570054] PM: Restoring platform NVS memory
>> [   38.570054] Enabling non-boot CPUs ...
>> (XEN) ----[ Xen-4.2.1-pre  x86_64  debug=n  Tainted:    C ]----
>> (XEN) CPU:    0
>> (XEN) RIP:    e008:[<ffff82c480120585>] vcpu_runstate_get+0xe5/0x130
>> (XEN) RFLAGS: 0000000000010006   CONTEXT: hypervisor
>> (XEN) rax: 00007d3b7fd17180   rbx: ffff8300bd2fe000   rcx: 0000000000000000
>> (XEN) rdx: ffff08003fc8bd80   rsi: ffff82c48029fe28   rdi: ffff8300bd2fe000
>> (XEN) rbp: ffff82c48029fe28   rsp: ffff82c48029fdf8   r8:  0000000000000008
>> (XEN) r9:  00000000000001c0   r10: ffff82c48021f4a0   r11: 0000000000000282
>> (XEN) r12: ffff82c4802e8ee0   r13: ffff880039762da0   r14: ffff82c4802d3140
>> (XEN) r15: fffffffffffffff2   cr0: 000000008005003b   cr4: 00000000000026f0
>> (XEN) cr3: 0000000139ee4000   cr2: 0000000000000060
>> (XEN) ds: 0000   es: 0000   fs: 0000   gs: 0000   ss: e010   cs: e008
>> (XEN) Xen stack trace from rsp=ffff82c48029fdf8:
>> (XEN)    ffff8300bd2fe000 ffff82c48029ff18 ffff880037481d40 ffff880039762da0
>> (XEN)    0000000000000001 ffff82c480157df4 0000000000000070 ffff82f6016db300
>> (XEN)    00000000000b6d98 ffff8301355d8000 0000000000000070 ffff82c4801702ab
>> (XEN)    ffff88003fc8bd80 0000000000000000 0000000000000020 ffff8300bd2fe000
>> (XEN)    ffff8301355d8000 ffff880037481d40 ffff880039762da0 0000000000000001
>> (XEN)    0000000000000003 ffff82c4801058df ffff82c48029ff18 ffff82c48011462e
>> (XEN)    0000000000000000 0000000000000000 0000000400000004 ffff82c48029ff18
>> (XEN)    0000000000000010 ffff8300bd6a0000 ffff8800374819a8 ffff8300bd6a0000
>> (XEN)    ffff880037481d48 0000000000000001 ffff880039762da0 ffff82c480214288
>> (XEN)    0000000000000003 0000000000000001 ffff880039762da0 0000000000000001
>> (XEN)    ffff880037481d48 0000000000000001 0000000000000282 ffff880002dc4240
>> (XEN)    00000000000001c0 00000000000001c0 0000000000000018 ffffffff8100130a
>> (XEN)    ffff880037481d40 0000000000000001 0000000000000005 0000010000000000
>> (XEN)    ffffffff8100130a 000000000000e033 0000000000000282 ffff880037481d20
>> (XEN)    000000000000e02b 0000000000000000 0000000000000000 0000000000000000
>> (XEN)    0000000000000000 0000000000000000 ffff8300bd6a0000 0000000000000000
>> (XEN)    0000000000000000
>> (XEN) Xen call trace:
>> (XEN)    [<ffff82c480120585>] vcpu_runstate_get+0xe5/0x130
>> (XEN)    [<ffff82c480157df4>] arch_do_vcpu_op+0x134/0x5d0
>> (XEN)    [<ffff82c4801702ab>] do_update_descriptor+0x1db/0x220
>> (XEN)    [<ffff82c4801058df>] do_vcpu_op+0x6f/0x4a0
>> (XEN)    [<ffff82c48011462e>] do_multicall+0x13e/0x330
>> (XEN)    [<ffff82c480214288>] syscall_enter+0x88/0x8d
>> (XEN)    
>> (XEN) Pagetable walk from 0000000000000060:
>> (XEN)  L4[0x000] = 00000001004a5067 0000000000038c9d
>> (XEN)  L3[0x000] = 000000013a703067 0000000000003094
>> (XEN)  L2[0x000] = 0000000000000000 ffffffffffffffff 
>> (XEN) 
>> (XEN) ****************************************
>> (XEN) Panic on CPU 0:
>> (XEN) FATAL PAGE FAULT
>> (XEN) [error_code=0000]
>> (XEN) Faulting linear address: 0000000000000060
>> (XEN) ****************************************
>> (XEN) 
>> (XEN) Reboot in five seconds...
>> 
>> 
>> On Fri, Sep 7, 2012 at 12:06 PM, Ben Guthro <ben@guthro.net> wrote:
>>> I'll work on getting a JTAG, ICE, or something else - it is on an
>>> Intel SDP - so it should have the ports for it.
>>> 
>>> My current suspicion on this is that the hardware registers are not
>>> being programmed the same way as they were in 4.0.x
>>> (Since the "pulsing power button LED" on the laptops, and the behavior
>>> of the Desktop SDP are now similar)
>>> 
>>> Once again - I don't have a lot of evidence to back this up - however,
>>> if I ifdef out the register writes that actually start the low level
>>> suspend - in
>>> xen/arch/x86/acpi/power.c  acpi_enter_sleep_state() - the rest of the
>>> suspend process completes as though the machine suspended, and then
>>> immediately resumed.
>>> 
>>> In this case - the system seems to be functioning properly.
>>> 
>>> 
>>> 
>>> 
>>> 
>>> Hack to prevent low level S3 attached.
>>> 
>>> 
>>> 
>>> On Fri, Sep 7, 2012 at 8:18 AM, Jan Beulich <JBeulich@suse.com> wrote:
>>>>>>> On 07.09.12 at 13:51, Ben Guthro <ben@guthro.net> wrote:
>>>>> However, when I run with console=none, the observed behavior is very
>>>>> different.
>>>>> The system seems to go to sleep successfully - but when I press the
>>>>> power button to wake it up - the power comes on - the fans spin up -
>>>>> but the system is unresponsive.
>>>>> No video
>>>>> No network
>>>>> keyboard LEDs (Caps,Numlock) do not light up.
>>>>> 
>>>>> 
>>>>> Alternate debugging strategies welcome.
>>>> 
>>>> I'm afraid other than being lucky to spot something via code
>>>> inspection, the only alternative is an ITP/ICE. Maybe Intel folks
>>>> could help out debugging this if it's reproducible for them.
>>>> 
>>>> Jan
>>>> 
>> 
>> 
>> 
>> _______________________________________________
>> Xen-devel mailing list
>> Xen-devel@lists.xen.org
>> http://lists.xen.org/xen-devel
> 

  reply	other threads:[~2012-09-20  6:24 UTC|newest]

Thread overview: 134+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-07 15:04 Xen4.2 S3 regression? Ben Guthro
2012-08-07 16:21 ` Ben Guthro
2012-08-07 16:33   ` Konrad Rzeszutek Wilk
2012-08-07 16:48     ` Ben Guthro
2012-08-07 20:14       ` Ben Guthro
2012-08-08  8:35         ` Jan Beulich
2012-08-08 10:39           ` Ben Guthro
2012-08-09 15:21             ` Ben Guthro
2012-08-09 15:37               ` Jan Beulich
2012-08-09 15:46                 ` Ben Guthro
2012-08-09 15:51                   ` Jan Beulich
2012-08-09 16:09                     ` Ben Guthro
2012-08-10  6:50                       ` Jan Beulich
2012-08-10 19:15                         ` Ben Guthro
2012-08-14 17:31                           ` Ben Guthro
2012-08-15  8:11                             ` Jan Beulich
2012-08-15 10:32                               ` Ben Guthro
2012-08-15 12:32                                 ` Ben Guthro
2012-08-15 12:58                                   ` Jan Beulich
2012-08-15 13:11                                     ` Ben Guthro
2012-08-15 14:50                                       ` Jan Beulich
2012-08-15 14:58                                         ` Ben Guthro
2012-08-15 15:00                                           ` Andrew Cooper
2012-08-15 15:06                                           ` Jan Beulich
2012-08-15 15:16                                             ` Ben Guthro
2012-08-16  8:31                                       ` Jan Beulich
2012-08-16 10:37                                         ` Ben Guthro
2012-08-16 11:07                                           ` Jan Beulich
2012-08-16 11:56                                             ` Ben Guthro
2012-08-17 10:22                                               ` Ben Guthro
2012-08-17 10:40                                                 ` Jan Beulich
2012-08-23 18:03                                                   ` Ben Guthro
2012-08-23 18:37                                                     ` Andrew Cooper
2012-08-24 22:11                                                       ` Jan Beulich
2012-08-24 22:55                                                     ` Jan Beulich
2012-08-25  0:48                                                       ` Ben Guthro
2012-09-03  9:31                                               ` Jan Beulich
2012-09-04 12:27                                                 ` Ben Guthro
2012-09-04 12:49                                                   ` Ben Guthro
2012-09-04 14:26                                                     ` Jan Beulich
2012-09-04 14:28                                                       ` Ben Guthro
2012-09-04 14:36                                                         ` Konrad Rzeszutek Wilk
2012-09-04 15:02                                                         ` Jan Beulich
2012-09-06 10:22                                                   ` Jan Beulich
2012-09-06 11:48                                                     ` Ben Guthro
2012-09-06 11:51                                                       ` Ben Guthro
2012-09-06 13:05                                                       ` Konrad Rzeszutek Wilk
2012-09-06 13:27                                                         ` Ben Guthro
2012-09-06 13:36                                                           ` Ben Guthro
2012-09-06 16:42                                                       ` Ben Guthro
2012-09-07  8:38                                                         ` Jan Beulich
2012-09-07 10:37                                                           ` Ben Guthro
2012-09-07 11:15                                                             ` Jan Beulich
2012-09-07 11:51                                                               ` Ben Guthro
2012-09-07 12:18                                                                 ` Jan Beulich
2012-09-07 16:06                                                                   ` Ben Guthro
2012-09-19 21:07                                                                     ` Ben Guthro
2012-09-20  6:13                                                                       ` Keir Fraser
2012-09-20  6:24                                                                         ` Keir Fraser [this message]
2012-09-20  8:03                                                                         ` Jan Beulich
2012-09-20  8:14                                                                           ` Keir Fraser
2012-09-20 12:56                                                                           ` Ben Guthro
2012-09-20 13:07                                                                             ` Keir Fraser
2012-09-20 20:30                                                                             ` Ben Guthro
2012-09-21  6:34                                                                               ` Keir Fraser
2012-09-21  6:47                                                                               ` Jan Beulich
2012-09-21 18:20                                                                                 ` Ben Guthro
2012-09-21 18:42                                                                                   ` Keir Fraser
2012-09-24 11:22                                                                                     ` Jan Beulich
2012-09-24 11:25                                                                                       ` Ben Guthro
2012-09-24 11:45                                                                                         ` Jan Beulich
2012-09-24 11:54                                                                                           ` Ben Guthro
2012-09-24 12:05                                                                                             ` Jan Beulich
2012-09-24 12:24                                                                                               ` Ben Guthro
2012-09-24 12:32                                                                                                 ` Jan Beulich
     [not found]                                                                                                   ` <CAOvdn6UMHmPWqedYE9GQQMDaM4oiHLDSn9ZzSgJjGf89g1DgTw@mail.gmail.com>
     [not found]                                                                                                     ` <50607D70020000780009D5C3@nat28.tlf.novell.com>
     [not found]                                                                                                       ` <CAOvdn6XL9ebp2oUV0XEXk_WdU3-=YAj+xfz6AMLDBpVThH3Xvw@mail.gmail.com>
2012-09-24 14:10                                                                                                         ` Jan Beulich
2012-09-24 14:16                                                                                                           ` Ben Guthro
2012-09-24 14:28                                                                                                             ` Jan Beulich
2012-09-24 19:02                                                                                                               ` Ben Guthro
2012-09-24 20:30                                                                                                                 ` Keir Fraser
2012-09-24 20:46                                                                                                                   ` Ben Guthro
2012-09-24 21:12                                                                                                                     ` Ben Guthro
2012-09-25  7:00                                                                                                                       ` Jan Beulich
2012-09-25 11:56                                                                                                                         ` Ben Guthro
2012-09-25 14:22                                                                                                                           ` Ben Guthro
2012-09-25 14:53                                                                                                                             ` Keir Fraser
2012-09-25 15:10                                                                                                                             ` Jan Beulich
2012-09-25 15:45                                                                                                                               ` Ben Guthro
2012-09-25 15:52                                                                                                                                 ` Keir Fraser
2012-09-26 11:49                                                                                                                                 ` Jan Beulich
2012-09-26 10:43                                                                                                                           ` Jan Beulich
2012-09-26 10:47                                                                                                                             ` Ben Guthro
2012-09-26 18:21                                                                                                                             ` Ben Guthro
2012-09-27  7:38                                                                                                                               ` Jan Beulich
2012-09-27  7:46                                                                                                                                 ` Keir Fraser
2012-09-27 12:12                                                                                                                                 ` Ben Guthro
2012-09-27 13:41                                                                                                                                   ` Jan Beulich
2012-09-27 15:25                                                                                                                                   ` Jan Beulich
2012-09-27 15:32                                                                                                                                     ` Ben Guthro
2012-09-27 15:59                                                                                                                                       ` [PATCH] x86/ucode: fix Intel case of resume handling on boot CPU Jan Beulich
2012-09-27 16:06                                                                                                                                         ` Keir Fraser
2012-09-24 14:32                                                                                                             ` Xen4.2 S3 regression? Keir Fraser
2012-09-24 12:22                                                                                             ` Pasi Kärkkäinen
2012-09-24 12:27                                                                                               ` Ben Guthro
2012-09-24 12:37                                                                                                 ` Javier Marcet
2012-09-24 14:04                                                                                                   ` Konrad Rzeszutek Wilk
2012-09-24 15:08                                                                                                     ` Javier Marcet
2012-09-24 21:36                                                                                                     ` Javier Marcet
2012-09-25 14:06                                                                                                       ` Konrad Rzeszutek Wilk
2012-09-25 14:47                                                                                                         ` Javier Marcet
2012-09-25 15:21                                                                                                           ` Jan Beulich
2012-09-25 15:23                                                                                                             ` Javier Marcet
2012-09-25 19:55                                                                                                             ` Javier Marcet
2012-09-25 19:57                                                                                                               ` Ben Guthro
2012-09-25 20:08                                                                                                                 ` Javier Marcet
2012-09-26  7:17                                                                                                               ` Jan Beulich
2012-09-26  7:59                                                                                                                 ` Javier Marcet
2012-09-26 12:43                                                                                                                   ` Konrad Rzeszutek Wilk
2012-09-26 14:14                                                                                                                     ` Javier Marcet
2012-09-26 14:26                                                                                                                       ` Ben Guthro
2012-09-26 14:40                                                                                                                         ` Javier Marcet
2012-09-26  8:05                                                                                                                 ` Javier Marcet
2012-09-24 12:37                                                                                                 ` Jan Beulich
2012-09-24 14:02                                                                                             ` Konrad Rzeszutek Wilk
2012-09-20  7:17                                                                       ` Jan Beulich
     [not found]                           ` <CAAnFQG-u1VUDgn11ZW0=UaYC4MvUtxxq8ZjjUOrNpXTSUWP41Q@mail.gmail.com>
     [not found]                             ` <CAOvdn6VuD_5Mhd9wvOskfZWfCBjr2nT5LppDxyY5S-5LhGhSvA@mail.gmail.com>
     [not found]                               ` <CAAnFQG_hMNvwM9Z3XPGR590=Gifos-kOftqjLFUX4YFW6tTTgg@mail.gmail.com>
     [not found]                                 ` <CAOvdn6UzdzO_sM6f9coN2udQ6eUC5=Sty-NgC7+yf3XMawF-0A@mail.gmail.com>
2012-09-04 15:31                                   ` Javier Marcet
  -- strict thread matches above, loose matches on Subject: below --
2012-08-23 18:54 Andrew Cooper
2012-08-23 19:06 ` Ben Guthro
2012-08-23 19:26   ` Ben Guthro
2012-08-23 19:38   ` Andrew Cooper
2012-08-23 20:38     ` Ben Guthro
2012-08-24 15:10       ` Ben Guthro
2012-08-24 22:16         ` Jan Beulich
     [not found]           ` <CAOvdn6U1touhawCb2GvgVQZqxhWn9CRw6-wkqdxk=uOTq015OA@mail.gmail.com>
2012-09-06  9:24             ` Jan Beulich

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=CC80754A.3F386%keir.xen@gmail.com \
    --to=keir.xen@gmail.com \
    --cc=JBeulich@suse.com \
    --cc=ben@guthro.net \
    --cc=john.baboval@citrix.com \
    --cc=konrad.wilk@oracle.com \
    --cc=thomas.goetz@citrix.com \
    --cc=xen-devel@lists.xen.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).