All of lore.kernel.org
 help / color / mirror / Atom feed
From: Suravee Suthikulanit <suravee.suthikulpanit@amd.com>
To: Jan Beulich <JBeulich@suse.com>
Cc: Sander Eikelenboom <linux@eikelenboom.it>,
	xen-devel <xen-devel@lists.xen.org>
Subject: Re: [xen-unstable] Commit 2ca9fbd739b8a72b16dd790d0fff7b75f5488fb8 AMD IOMMU: allocate IRTE entries instead of using a static mapping, makes dom0 boot process stall several times.
Date: Tue, 6 Aug 2013 10:47:04 -0500	[thread overview]
Message-ID: <52011A78.2040701@amd.com> (raw)
In-Reply-To: <51FF74E502000078000E9267@nat28.tlf.novell.com>

On 8/5/2013 2:48 AM, Jan Beulich wrote:
>>>> On 22.07.13 at 22:50, Sander Eikelenboom <linux@eikelenboom.it> wrote:
>> After commit 2ca9fbd739b8a72b16dd790d0fff7b75f5488fb8 AMD IOMMU: allocate
>> IRTE entries instead of using a static mapping, booting dom0 stalls several
>> times.
>> Sometimes this results in RCU stall warnings from the dom0 kernel, hitting
>> the "any" key, on normal or serial console, makes the boot continue for a
>> while but it stalls several times.
>> (It also stalls on shutdown BTW)
>>
>> I have bisected it to this commit running kernel 3.10.2 and Debian wheezy.
>>
>> xl-dmesg and dmesg from a boot with stall are attached, snippet from dmesg
>> with RCU stall appended below.
>>
>> Note that i do have to run with iommu='no-amd-iommu-perdev-intremap' due to the
>> bios bug that probably will never get fixed.
>> When i run without the 'no-amd-iommu-perdev-intremap' the iommu is disabled and
>> stalls do not occur.
> I can only hope that someone with access to a system exposing this
> problem (you?) can debug it - I certainly have no system where this
> issue could be seen, and I don't recall Suravee having noticed
> anything like this in his testing of those patches.
>
> Jan

I don't have the system which has the exact same BIOS IVRS IOAPIC bug as mentioned
in the origin xl dmesg output you sent. In your case the special IVHD entry for the
first IOAPIC with ID 0x6 is missing from IVRS table.

(XEN) AMD-Vi: IVHD Special: 0000:00:14.0 variety 0x2 handle 0
(XEN) AMD-Vi: IVHD Device Entry: type 0x48 id 0 flags 0
(XEN) AMD-Vi: IVHD Special: 0000:00:00.1 variety 0x1 handle 0x7
(XEN) IVHD Error: no information for IO-APIC 0x6
(XEN) vmap(ffff82c000802000:0x4)
(XEN) AMD-Vi: IOMMU 0 Enabled.
(XEN) AMD-Vi: Using global interrupt remap table is not recommended (see XSA-36)!
(XEN) I/O virtualisation enabled

Here, I was expecting to see a print out showing

(XEN) AMD-Vi: IVHD Special: 0000:00:14.0 variety 0x1 handle 0x6

I could imagine this would cause IOMMU to not correctly setup, and probably should
be disabled.

Suravee

  reply	other threads:[~2013-08-06 15:47 UTC|newest]

Thread overview: 70+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-22 20:50 [xen-unstable] Commit 2ca9fbd739b8a72b16dd790d0fff7b75f5488fb8 AMD IOMMU: allocate IRTE entries instead of using a static mapping, makes dom0 boot process stall several times Sander Eikelenboom
2013-08-05  7:48 ` Jan Beulich
2013-08-06 15:47   ` Suravee Suthikulanit [this message]
2013-08-15 14:43     ` Sander Eikelenboom
2013-08-15 15:15       ` Jan Beulich
2013-08-15 23:22         ` Sander Eikelenboom
2013-08-15 23:41           ` Sander Eikelenboom
2013-08-16  7:21             ` Jan Beulich
2013-08-16  7:42               ` Sander Eikelenboom
2013-08-16  8:03                 ` Jan Beulich
2013-08-16  8:40                   ` Sander Eikelenboom
2013-08-16  9:18                     ` Jan Beulich
2013-08-16 10:44                       ` Sander Eikelenboom
2013-08-16 13:15                         ` Jan Beulich
2013-08-16 13:22                           ` Sander Eikelenboom
2013-08-16 14:43                           ` Sander Eikelenboom
2013-08-16 15:51                           ` Sander Eikelenboom
2013-08-22 22:51                           ` Sander Eikelenboom
2013-08-22 23:42                             ` Sander Eikelenboom
2013-08-23 14:28                             ` Jan Beulich
2013-08-23 14:45                               ` Sander Eikelenboom
2013-08-23 15:05                               ` Sander Eikelenboom
2013-08-23 15:11                                 ` Jan Beulich
2013-08-23 15:21                                   ` Jan Beulich
2013-08-23 15:48                                     ` Sander Eikelenboom
2013-08-23 16:01                                       ` Jan Beulich
2013-08-23 16:06                                         ` Sander Eikelenboom
2013-08-26 15:10                                           ` Suravee Suthikulpanit
2013-08-26 15:33                                             ` Jan Beulich
2013-08-23 17:01                                         ` Sander Eikelenboom
2013-08-23 15:29                                   ` Sander Eikelenboom
2013-08-26  6:59                                     ` Jan Beulich
2013-08-26  9:51                                       ` Sander Eikelenboom
2013-08-26 10:33                                         ` Jan Beulich
2013-08-26 11:07                                           ` Sander Eikelenboom
2013-08-26 11:23                                             ` Jan Beulich
2013-08-26 12:34                                               ` Sander Eikelenboom
2013-08-26 14:15                                                 ` Jan Beulich
2013-08-26 14:35                                                   ` Sander Eikelenboom
2013-08-26 11:21                                           ` Sander Eikelenboom
2013-08-26 11:25                                             ` Jan Beulich
2013-08-26 11:29                                             ` Jan Beulich
2013-08-26 11:36                                               ` Sander Eikelenboom
2013-08-26 15:37                                                 ` Suravee Suthikulpanit
2013-08-26 15:50                                                   ` Suravee Suthikulpanit
2013-08-27  8:23                                                     ` Jan Beulich
2013-08-26 15:50                                                   ` Jan Beulich
2013-08-26 16:19                                                     ` Sander Eikelenboom
2013-08-27  8:00                             ` [PATCH RFC 0/2] AMD IOMMU: allow command line overrides for broken IVRS tables Jan Beulich
2013-08-27  8:05                               ` [PATCH RFC 1/2] PCI: centralize parsing of device coordinates in command line options Jan Beulich
2013-08-27  8:05                               ` [PATCH RFC 2/2] AMD IOMMU: allow command line overrides for broken IVRS tables Jan Beulich
2013-08-27  9:52                                 ` Sander Eikelenboom
2013-08-27 11:09                                   ` Sander Eikelenboom
2013-08-27 12:04                                     ` Jan Beulich
2013-08-27 13:32                                       ` Sander Eikelenboom
2013-08-27 13:55                                         ` Jan Beulich
2013-08-27 14:15                                           ` Andrew Cooper
2013-08-27 16:17                                             ` Jan Beulich
2013-08-27 16:19                                               ` Sander Eikelenboom
2013-08-27 14:11                                       ` Jan Beulich
2013-08-27 14:15                                         ` Sander Eikelenboom
2013-08-27 15:59                                           ` Jan Beulich
2013-08-27 16:14                                             ` Sander Eikelenboom
2013-08-27 16:21                                               ` Jan Beulich
2013-08-27 16:30                                                 ` Sander Eikelenboom
2013-08-27 17:09                                                   ` Sander Eikelenboom
2013-08-28  8:06                                                   ` Jan Beulich
2013-08-28 14:59                                 ` Suravee Suthikulpanit
2013-08-28 15:18                                   ` Jan Beulich
2013-08-27 18:07                               ` [PATCH RFC 0/2] " Keir Fraser

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=52011A78.2040701@amd.com \
    --to=suravee.suthikulpanit@amd.com \
    --cc=JBeulich@suse.com \
    --cc=linux@eikelenboom.it \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.