xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Atom2 <ariel.atom2@web2web.at>
To: Andrew Cooper <andrew.cooper3@citrix.com>
Cc: xen-devel <xen-devel@lists.xenproject.org>,
	Ian Campbell <ian.campbell@citrix.com>,
	Jan Beulich <JBeulich@suse.com>
Subject: Re: Powerdown problem on XEN | ACPI S5
Date: Wed, 14 Aug 2013 19:00:39 +0200	[thread overview]
Message-ID: <520BB7B7.1000108@web2web.at> (raw)
In-Reply-To: <520B8D63.6040903@citrix.com>

[-- Attachment #1: Type: text/plain, Size: 3558 bytes --]

Hi Andrew,
please see my inline comments further below.

And many thanks to all of you for your support so far!

Am 14.08.13 16:00, schrieb Andrew Cooper:
> On 14/08/13 14:52, Atom2 wrote:
>> Hi Jan,
>> thanks for reply. You are obviously right that the first thing
>> device_power_down does, is console_suspend(). I don't know why that
>> escaped my eyes when I originally searched the file ...
>>
>> Anyways, I have now disabled console_suspend() and also added a few
>> more lines to the code with printk statements indicating up to which
>> point the system had gone (without errors). With hindsight I guess the
>> new printk() statements might not have been required as now, with the
>> console still active, a panic message pops up at the end, resulting in
>> rebooting the system:
>>
>> (XEN) Disabling non-boot CPUs ...
>> (XEN) Entering ACPI S5 state.
>> (XEN) After local_irq_save
>> (XEN) After spin_debug_disable
>> (XEN) After time_suspend
>> (XEN) After li8259_suspend
>> (XEN) After ioapic_suspend
>> (XEN) DMAR_IQA_REG = 80d87c002
>> (XEN) DMAR_IQH_REG = 120
>> (XEN) DMAR_IQT_REG = 140
>> (XEN)
>> (XEN) ****************************************
>> (XEN) Panic on CPU 0:
>> (XEN) queue invalidate wait descriptor was not executed
>> (XEN) ****************************************
>> (XEN)
>> (XEN) Reboot in five seconds...
>> (XEN) Resetting with ACPI MEMORY or I/O RESET_REG.
>>
>> NOTE: All the messages starting with "(XEN) After" are from my changes
>> to the code; the rest is as is - except me commenting out
>> console_suspend() in power.c. I hope that helps in resolving the issue
>> and the panic is not just the result of a knock-on effect from
>> commenting out console_suspend() earlier.
>
> Huh - I thought I had fixed this issue already.
>
> Can you confirm exactly which version of Xen you are using (including
> changeset), and perhaps compile in this patch:
The version I am using is 4.2.2 from the gentoo ebuild. Interesting 
enough, the log in the consoles states
(XEN) Latest ChangeSet: unavailable
I don't know what to make out of this - or is there any other way to 
figure out, what you are after?

The alternative would be to apply the WARN() to the 4.3.0 version I have 
downloaded yesterday from xenbits at 
http://www.xenproject.org/downloads/xen-archives/supported-xen-43-series/xen-430/287-xen-430-2/file.html 
(FYI: the reboot also happened there). If that helps, I'll rerun it on 
the 4.3.0 version. So far I have used the gentoo version as this allows 
me to stay within the portage system.
>
> diff --git a/xen/drivers/passthrough/vtd/qinval.c
> b/xen/drivers/passthrough/vtd/qinval.c
> index 6a410d8..d023b26 100644
> --- a/xen/drivers/passthrough/vtd/qinval.c
> +++ b/xen/drivers/passthrough/vtd/qinval.c
> @@ -220,6 +220,7 @@ static int queue_invalidate_wait(struct iommu *iommu,
>               if ( NOW() > (start_time + DMAR_OPERATION_TIMEOUT) )
>               {
>                   print_qi_regs(iommu);
> +                WARN();
>                   panic("queue invalidate wait descriptor was not
> executed\n");
>               }
>               cpu_relax();
I have manually applied the patch - which was just an added
	WARN();
inbetween if I read the patch correctly; the rest was already there in 
4.2.2 (and also 4.3.0 - I checked its source as well). I have attached 
the serial log from my 4.2.2 run to prevent line-wrap. I hope that helps.

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

[-- Attachment #2: XEN console log --]
[-- Type: text/plain, Size: 3453 bytes --]

(XEN) Entering ACPI S5 state.
(XEN) After local_irq_save
(XEN) After spin_debug_disable
(XEN) After time_suspend
(XEN) After li8259_suspend
(XEN) After ioapic_suspend
(XEN) DMAR_IQA_REG = 80d87c002
(XEN) DMAR_IQH_REG = 120
(XEN) DMAR_IQT_REG = 140
(XEN) Xen WARN at qinval.c:223
(XEN) ----[ Xen-4.2.2  x86_64  debug=n  Not tainted ]----
(XEN) CPU:    0
(XEN) RIP:    e008:[<ffff82c48014543c>] invalidate_sync+0x22c/0x280
(XEN) RFLAGS: 0000000000010082   CONTEXT: hypervisor
(XEN) rax: ffff82c4802f14a0   rbx: ffff83080d8855b0   rcx: 0000000000000000
(XEN) rdx: ffff82c4802a7f18   rsi: 000000000000000a   rdi: ffff82c48025a640
(XEN) rbp: 0000000b2ae4ae9f   rsp: ffff82c4802a7d20   r8:  0000000000000004
(XEN) r9:  0000000000000002   r10: 0000000000000003   r11: 0000000000000400
(XEN) r12: ffff83080d8855e4   r13: ffff83080d88562c   r14: 0000000000000002
(XEN) r15: 0000000000000002   cr0: 000000008005003b   cr4: 00000000000426f0
(XEN) cr3: 00000000db6b5000   cr2: ffff88001a89ea48
(XEN) ds: 0000   es: 0000   fs: 0000   gs: 0000   ss: e010   cs: e008
(XEN) Xen stack trace from rsp=ffff82c4802a7d20:
(XEN)    0000000000000002 000000080d87c002 0000000000000082 000000000d87c002
(XEN)    0000000000000086 ffff83080d8855b0 0000000000000000 1000000000000000
(XEN)    0000000000000000 0000000000000000 0000000000000000 ffff82c480145546
(XEN)    0000000000000000 0000000000000000 0000000800000004 ffff83080d8855b0
(XEN)    ffff83080d885530 ffff83080d885640 ffff82c48024bbe0 0000000000000000
(XEN)    ffff82c4802f14e0 ffff82c4801413ee 0000000000000000 0000000000000086
(XEN)    ffff82c4802f14a0 0000000000000005 0000000000000000 0000000000000000
(XEN)    ffff82c4802db740 ffff82c480141e05 0000000000000082 0000000000000086
(XEN)    ffff82c4802a7f18 ffff82c4802f14a0 0000000000000005 0000000000000000
(XEN)    0000000000000000 ffff82c4802db740 ffff82c4802f14e0 ffff82c48019a245
(XEN)    ffff82c4802a7f18 00000000ffffffff 0000000000000286 ffff8300dc697000
(XEN)    ffff8300dc697000 ffff830669301fc0 ffff8300dc697000 ffff82c4802f1488
(XEN)    ffff82c4802db740 ffff82c4802db740 ffff82c4802f14e0 ffff82c4801054ae
(XEN)    ffff8300dc697180 0000000000000000 ffff82c4802f1590 ffff82c480125670
(XEN)    ffff82c4802f1570 ffff82c480125878 ffff82c4802a7f18 ffff8300dc697000
(XEN)    00000000ffffffff ffff82c480156933 ffff8300dcda6000 0000000000000000
(XEN)    0000000000000003 0000000000000005 0000000000002003 ffffffff815adf58
(XEN)    0000000000000000 0000000000000246 0000000000000370 0000000000000005
(XEN)    0000000000000000 0000000000000000 ffffffff810010ea 0000000000002003
(XEN)    0000000000003c03 ffff880019dd1d18 0000010000000000 ffffffff810010ea
(XEN) Xen call trace:
(XEN)    [<ffff82c48014543c>] invalidate_sync+0x22c/0x280
(XEN)    [<ffff82c480145546>] flush_iotlb_qi+0xb6/0x100
(XEN)    [<ffff82c4801413ee>] iommu_flush_all+0x8e/0xf0
(XEN)    [<ffff82c480141e05>] vtd_suspend+0x55/0x110
(XEN)    [<ffff82c48019a245>] enter_state_helper+0x235/0x430
(XEN)    [<ffff82c4801054ae>] continue_hypercall_tasklet_handler+0xee/0x100
(XEN)    [<ffff82c480125670>] do_tasklet_work+0x60/0xc0
(XEN)    [<ffff82c480125878>] do_tasklet+0x78/0xb0
(XEN)    [<ffff82c480156933>] idle_loop+0x23/0x50
(XEN)
(XEN)
(XEN) ****************************************
(XEN) Panic on CPU 0:
(XEN) queue invalidate wait descriptor was not executed
(XEN) ****************************************
(XEN)
(XEN) Reboot in five seconds...
(XEN) Resetting with ACPI MEMORY or I/O RESET_REG.

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

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

  reply	other threads:[~2013-08-14 17:00 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-14  8:48 Powerdown problem on XEN | ACPI S5 Atom2
2013-08-14 10:30 ` Jan Beulich
2013-08-14 13:52   ` Atom2
2013-08-14 14:00     ` Andrew Cooper
2013-08-14 17:00       ` Atom2 [this message]
2013-08-14 17:30         ` Andrew Cooper
2013-08-14 18:40           ` Atom2
2013-08-14 19:10             ` Atom2
2013-08-14 19:18               ` Andrew Cooper
2013-08-14 19:39                 ` Atom2
2013-08-14 20:18                   ` Andrew Cooper
2013-08-14 20:24                     ` Atom2
2013-08-14 20:30                       ` Atom2
2013-08-14 20:34                       ` Ben Guthro
2013-08-14 20:37                         ` Konrad Rzeszutek Wilk
2013-08-14 21:56                           ` Atom2
2013-08-15  1:58                             ` Ben Guthro
2013-08-15 19:28                               ` Atom2
2013-08-15 20:26                                 ` Konrad Rzeszutek Wilk
2013-08-15 21:39                                   ` Atom2
2013-08-16 12:24                                     ` Konrad Rzeszutek Wilk
2013-12-11 21:52                                     ` Konrad Rzeszutek Wilk
2013-08-15 13:40                             ` Konrad Rzeszutek Wilk
2013-08-14 20:38                       ` Andrew Cooper
2013-08-14 20:54                         ` Atom2
2013-08-14 21:11                           ` Andrew Cooper
2013-08-15  8:12                           ` Jan Beulich
2013-08-15  8:16                             ` Atom2

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=520BB7B7.1000108@web2web.at \
    --to=ariel.atom2@web2web.at \
    --cc=JBeulich@suse.com \
    --cc=andrew.cooper3@citrix.com \
    --cc=ian.campbell@citrix.com \
    --cc=xen-devel@lists.xenproject.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).