From: Malcolm Crossley <malcolm.crossley@citrix.com>
To: Jan Beulich <JBeulich@suse.com>
Cc: Andrew Cooper <andrew.cooper3@citrix.com>,
Joerg Roedel <joro@8bytes.org>,
nhorman@tuxdriver.com, xen-devel <xen-devel@lists.xen.org>
Subject: Re: VT-d interrup remapping errata workaround
Date: Mon, 8 Jul 2013 13:45:50 +0100 [thread overview]
Message-ID: <51DAB47E.3070507@citrix.com> (raw)
In-Reply-To: <51DACCD702000078000E342C@nat28.tlf.novell.com>
On 08/07/13 13:29, Jan Beulich wrote:
> All,
>
> just having spotted the backport of Linux commit 03bbcb2e I notice
> a certain discrepancy with the Xen commit having the same purpose
> as well as with the actual specification updates:
>
> The Linux solution keys off of device IDs 3403 and 3406, as listed in
> the specification update, but this way fails to cover the X58 chipset,
> which has - under different numbers (62 and 69) - the same errata
> (the Linux commit message also only mentions erratum 53 for the
> 55x0 chipsets, albeit I believe 47 is as much of an issue there as it is
> for Xen).
>
> The Xen solution keys off of ID 342e, with no explanation in the
> commit description on where this association comes from.
>
> I therefore wonder whether
> - the Xen solution may have false positives
> - the Linux solution would still have false negatives even after
> adding ID 3405
>
> Jan
>
Hi Jan,
I have used the VT-d control PCI device which is common across X58/5520/5500
See page 233
http://www.intel.co.uk/content/dam/doc/datasheet/x58-express-chipset-datasheet.pdf
and
page 255
http://www.intel.co.uk/content/dam/doc/datasheet/5520-5500-chipset-ioh-datasheet.pdf
It seems the Linux workaround only applies to the 5520 and 5500 chipsets.
The pci ID database suggests 0x342e will detect 7500 chipset (which does
not have the interrupt remapping errata) but that chipset never reports
revision 0x13 so the errata workaround will never be applied.
Malcolm
next prev parent reply other threads:[~2013-07-08 12:45 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-08 12:29 VT-d interrup remapping errata workaround Jan Beulich
2013-07-08 12:45 ` Malcolm Crossley [this message]
2013-07-08 12:56 ` Malcolm Crossley
2013-07-08 13:39 ` Neil Horman
2013-07-08 13:33 ` Neil Horman
2013-07-08 13:42 ` Jan Beulich
2013-07-08 16:22 ` Neil Horman
2013-07-08 17:11 ` Neil Horman
2013-07-09 6:40 ` Jan Beulich
2013-07-09 9:34 ` Malcolm Crossley
2013-07-09 9:48 ` Jan Beulich
2013-07-08 19:13 ` [RFC PATCH] iommu/vt-d: Expand interrupt remapping quirk to cover x58 chipset Neil Horman
2013-07-09 6:44 ` 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=51DAB47E.3070507@citrix.com \
--to=malcolm.crossley@citrix.com \
--cc=JBeulich@suse.com \
--cc=andrew.cooper3@citrix.com \
--cc=joro@8bytes.org \
--cc=nhorman@tuxdriver.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.