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
next prev parent 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).