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>
Subject: Re: Xen4.2 S3 regression?
Date: Mon, 24 Sep 2012 21:30:16 +0100	[thread overview]
Message-ID: <CC868168.3FD29%keir.xen@gmail.com> (raw)
In-Reply-To: <CAOvdn6UCF2A3yP1cqWn=dxwr0ayRbr+rg+xj8META9Dm0+CoNw@mail.gmail.com>


[-- Attachment #1.1: Type: text/plain, Size: 5967 bytes --]

Do a debug build so the backtrace can be trusted. It¹s a NULL pointer
dereference so shouldn¹t be too tricky to make some headway on this one.
Easier than the previous bug. :)

 -- Keir

On 24/09/2012 20:02, "Ben Guthro" <ben@guthro.net> wrote:

> Well...knock one bug down - and another crops up.
> 
> It appears that dom0_vcpu_pin is incompatible with S3.
> I'll start digging into why, but if you have any thoughts from the stack
> below, I'd welcome any pointers.
> 
> /btg
> 
> 
> (XEN) Preparing system for ACPI S3 state.
> (XEN) Disabling non-boot CPUs ...
> (XEN) Entering ACPI S3 state.
> (XEN) mce_intel.c:1239: MCA Capability: BCAST 1 SER 0 CMCI 0 firstbank 1
> extended MCE MSR 0
> (XEN) CMCI: CPU0 has no CMCI support
> (XEN) CPU0: Thermal monitoring enabled (TM2)
> (XEN) Finishing wakeup from ACPI S3 state.
> (XEN) Enabling non-boot CPUs  ...
> (XEN) Booting processor 1/1 eip 8a000
> (XEN) Initializing CPU#1
> (XEN) CPU: L1 I cache: 32K, L1 D cache: 32K
> (XEN) CPU: L2 cache: 3072K
> (XEN) CPU: Physical Processor ID: 0
> (XEN) CPU: Processor Core ID: 1
> (XEN) CMCI: CPU1 has no CMCI support
> (XEN) CPU1: Thermal monitoring enabled (TM2)
> (XEN) CPU1: Intel(R) Core(TM)2 Duo CPU     P8400  @ 2.26GHz stepping 06
> (XEN) microcode: CPU1 updated from revision 0x60c to 0x60f, date = 2010-09-29 
> [   60.100054] ACPI: Low-level resume complete
> [   60.100054] PM: Restoring platform NVS memory
> [   60.100054] Enabling non-boot CPUs ...
> [   60.100054] installing Xen timer for CPU 1
> [   60.100054] cpu 1 spinlock event irq 279
> (XEN) ----[ Xen-4.2.1-pre  x86_64  debug=n  Tainted:    C ]----
> (XEN) CPU:    1
> (XEN) RIP:    e008:[<ffff82c480121562>] vcpu_migrate+0x172/0x360
> (XEN) RFLAGS: 0000000000010096   CONTEXT: hypervisor
> (XEN) rax: 00007d3b7fd17180   rbx: ffff82c4802e8ee0   rcx: ffff82c4802e8ee0
> (XEN) rdx: ffff83013a3c5068   rsi: 0000000000000004   rdi: ffff8301300b7d68
> (XEN) rbp: 0000000000000001   rsp: ffff8301300b7e28   r8:  0000000000000000
> (XEN) r9:  000000000000003e   r10: 000000000000003e   r11: 0000000000000246
> (XEN) r12: ffff83013a3c5068   r13: ffff83013a3c5068   r14: ffff82c4802d3140
> (XEN) r15: 0000000000000001   cr0: 000000008005003b   cr4: 00000000000026f0
> (XEN) cr3: 0000000131a05000   cr2: 0000000000000060
> (XEN) ds: 002b   es: 002b   fs: 0000   gs: 0000   ss: e010   cs: e008
> (XEN) Xen stack trace from rsp=ffff8301300b7e28:
> (XEN)    ffff82c4802d3140 ffff83013a3c5068 0000000000000246 0000000000000004
> (XEN)    ffff8300bd2fe000 ffff82c4802e8ee0 00000004012d3140 ffff82c4802e8ee0
> (XEN)    ffff88003fc8e820 ffff8300bd2fe000 ffff8301355d8000 0000000000000000
> (XEN)    0000000000000000 0000000000000000 ffff88003fc8e820 ffff82c480105a50
> (XEN)    0000000000000000 ffff82c4801805ec 0000060f00000000 ffff82c480184f16
> (XEN)    0000000000000032 78a20f6e65780b0f ffff88003976fdc8 ffff8300bd2fe000
> (XEN)    ffff88003976fe50 ffff8300bd2fe000 ffff88003976fda0 0000000000000001
> (XEN)    0000000000000000 ffff82c480214288 ffff88003fc8e820 0000000000000000
> (XEN)    0000000000000000 0000000000000001 ffff88003976fda0 ffff88003fc8bdc0
> (XEN)    0000000000000246 ffff88003976fe60 00000000ffffffff 0000000000000000
> (XEN)    0000000000000018 ffffffff8100130a 0000000000000000 0000000000000001
> (XEN)    0000000000000007 0000010000000000 ffffffff8100130a 000000000000e033
> (XEN)    0000000000000246 ffff88003976fd88 000000000000e02b d43d5f3fedaef5e7
> (XEN)    d3b2ddaeed5038ff 270adb813ad76c9b ddfd6ff5f85e6775 b5881cbf00000001
> (XEN)    ffff8300bd2fe000 0000003cba0dc180 0a109ac649c118a1
> (XEN) Xen call trace:
> (XEN)    [<ffff82c480121562>] vcpu_migrate+0x172/0x360
> (XEN)    [<ffff82c480105a50>] do_vcpu_op+0x1e0/0x4a0
> (XEN)    [<ffff82c4801805ec>] do_invalid_op+0x19c/0x3f0
> (XEN)    [<ffff82c480184f16>] copy_from_user+0x26/0x90
> (XEN)    [<ffff82c480214288>] syscall_enter+0x88/0x8d
> (XEN)    
> (XEN) Pagetable walk from 0000000000000060:
> (XEN)  L4[0x000] = 0000000000000000 ffffffffffffffff
> (XEN) 
> (XEN) ****************************************
> (XEN) Panic on CPU 1:
> (XEN) FATAL PAGE FAULT
> (XEN) [error_code=0000]
> (XEN) Faulting linear address: 0000000000000060
> (XEN) ****************************************
> (XEN) 
> (XEN) Reboot in five seconds...
> 
> 
> On Mon, Sep 24, 2012 at 10:28 AM, Jan Beulich <JBeulich@suse.com> wrote:
>>>>> >>> On 24.09.12 at 16:16, Ben Guthro <ben@guthro.net> wrote:
>>> > Would you prefer a separate [PATCH] email for this fix, or will you apply
>>> > it as-is?
>> 
>> I'll put something together - the most important thing here obviously
>> is having a proper description. Plus I'd like to slightly extend this and
>> have acpi_dead_idle() actually use default_dead_idle(), just to have
>> things consolidated in one place. I assume I can put your S-o-b on
>> what you sent...
>> 
>> Jan
>> 
>>> > On Mon, Sep 24, 2012 at 10:10 AM, Jan Beulich <JBeulich@suse.com> wrote:
>>> >
>>>>>>> >> >>> On 24.09.12 at 15:56, Ben Guthro <ben@guthro.net> wrote:
>>>>> >> > On Mon, Sep 24, 2012 at 9:34 AM, Jan Beulich <JBeulich@suse.com>
>>>>> wrote:
>>>>>> >> >> ...; the interesting ones are
>>>>>> >> >> - at the end of xen/arch/x86/acpu/cpu_idle.c:acpi_dead_idle()
>>>>>> >> >> - xen/arch/x86/domain.c:default_dead_idle()
>>>>> >> >
>>>>> >> >
>>>>> >> > Thanks! This fixes the issue on this machine!
>>>> >>
>>>> >> Hooray!
>>>> >>
>>>>> >> > Is this a reasonable long-term solution - or are there reasons not to
>>>>> >> > call wbinvd() here?
>>>> >>
>>>> >> That's a perfectly valid adjustment (see my earlier reply where
>>>> >> I originally suggested it and explained why it may be necessary).
>>>> >>
>>>> >> Jan
>>>> >>
>>>> >>
>> 
>> 
>> 
> 
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel


[-- Attachment #1.2: Type: text/html, Size: 7367 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

  reply	other threads:[~2012-09-24 20:30 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
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 [this message]
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=CC868168.3FD29%keir.xen@gmail.com \
    --to=keir.xen@gmail.com \
    --cc=JBeulich@suse.com \
    --cc=ben@guthro.net \
    --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).