All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Jan Beulich" <JBeulich@suse.com>
To: Sander Eikelenboom <linux@eikelenboom.it>
Cc: Andrew Cooper <andrew.cooper3@citrix.com>,
	Suravee Suthikulanit <suravee.suthikulpanit@amd.com>,
	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: Mon, 26 Aug 2013 15:15:10 +0100	[thread overview]
Message-ID: <521B7F0E02000078000EE6F7@nat28.tlf.novell.com> (raw)
In-Reply-To: <1895434160.20130826143413@eikelenboom.it>

>>> On 26.08.13 at 14:34, Sander Eikelenboom <linux@eikelenboom.it> wrote:

> Monday, August 26, 2013, 1:23:01 PM, you wrote:
> 
>>>>> On 26.08.13 at 13:07, Sander Eikelenboom <linux@eikelenboom.it> wrote:
>>> Don't know that for sure, will check if there are any related messages in 
>>> dmesg
>>> I only see it enabling HPET, but i don't see anything about MSI ..
> 
>> The primary indicator, if I'm not mistaken, would be 3 IRQs in
>> /proc/interrupts that show type HPET_MSI (and I suspect you
>> don't see any).
> 
> Ah found out why, another quirk and silently applying:
> 
> /*
>  * HPET MSI on some boards (ATI SB700/SB800) has side effect on
>  * floppy DMA. Disable HPET MSI on such platforms.
>  * See erratum #27 (Misinterpreted MSI Requests May Result in
>  * Corrupted LPC DMA Data) in AMD Publication #46837,
>  * "SB700 Family Product Errata", Rev. 1.0, March 2010.
>  */
> static void force_disable_hpet_msi(struct pci_dev *unused)
> {
>         hpet_msi_disable = 1;
> }
> 
> how many quirks can a machine have ...

Indeed.

In any case - I take this as a sign that I'll submit the earlier
patch as is, without your Tested-by (because you couldn't
really test other than that your box doesn't stall anymore).

Thanks for your patience,
Jan

  reply	other threads:[~2013-08-26 14:15 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
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 [this message]
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=521B7F0E02000078000EE6F7@nat28.tlf.novell.com \
    --to=jbeulich@suse.com \
    --cc=andrew.cooper3@citrix.com \
    --cc=linux@eikelenboom.it \
    --cc=suravee.suthikulpanit@amd.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 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.