From: Valentine Sinitsyn <valentine.sinitsyn@gmail.com>
To: David Kiarie <davidkiarie4@gmail.com>, Jan Kiszka <jan.kiszka@web.de>
Cc: QEMU Developers <qemu-devel@nongnu.org>,
"Michael S. Tsirkin" <mst@redhat.com>,
Marcel Apfelbaum <marcel@redhat.com>,
imammedo@redhat.com
Subject: Re: [Qemu-devel] [V9 2/4] hw/i386: ACPI table for AMD IOMMU
Date: Wed, 4 May 2016 12:50:16 +0500 [thread overview]
Message-ID: <5729A9B8.3060603@gmail.com> (raw)
In-Reply-To: <CABdVeADct0X4gDE_1LE0We=u_Usj3N889bFRdbSMPdwPo_J2AA@mail.gmail.com>
On 29.04.2016 13:28, David Kiarie wrote:
> On Fri, Apr 29, 2016 at 9:09 AM, Jan Kiszka <jan.kiszka@web.de> wrote:
>> On 2016-04-25 00:12, David Kiarie wrote:
>>> Add IVRS table for AMD IOMMU. Generate IVRS or DMAR
>>> depending on emulated IOMMU
>>
>> It seems you lack scope descriptions for the PCI devices in the system.
>> At least, this is what our jailhouse config generator complains about
>> right now (didn't look into details yet). If so, the guest OS will
>> likely not configure the IOMMU appropriately as it thinks that the
>> devices are passed through anyway.
>>
>> On Intel, there is an easy way to state "catch them all" in the ACPI
>> table. Maybe AMD has this as well so that you don't need to add
>> individual devices.
>
> This is what am currently doing. From what I can see, linux correctly
> detects the range but I will have a look at it again before sending
> the next version.
>
> There was another issue raised by Michael about denying VFIO in case a
> user requests for IOMMU support. I did look at this but since VFIO is
> started with '-device' I didn't find a way to deny it in case of IOMMU
> support except by parsing command line arguments which turned out to
> be a bit ugly.
>
> We could exclude VFIO from translation by IOMMU by excluding it from
> the scope descriptions but am not quite sure this will remedy all the
> problems relating to VFIO and IOMMU. This will also require the device
> scope to explicitly list all the devices.
You may also try listing everything "up to" VFIO, then everything
"after" the VFIO, I guess.
Valentine
next prev parent reply other threads:[~2016-05-04 7:50 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-24 22:12 [Qemu-devel] [V9 0/4] AMD IOMMU David Kiarie
2016-04-24 22:12 ` [Qemu-devel] [V9 1/4] hw/i386: Introduce " David Kiarie
2016-04-24 22:12 ` [Qemu-devel] [V9 2/4] hw/i386: ACPI table for " David Kiarie
2016-04-29 6:09 ` Jan Kiszka
2016-04-29 8:28 ` David Kiarie
2016-05-04 7:50 ` Valentine Sinitsyn [this message]
2016-05-04 7:45 ` Valentine Sinitsyn
2016-04-24 22:12 ` [Qemu-devel] [V9 3/4] hw/core: Add AMD IOMMU to machine properties David Kiarie
2016-04-25 3:00 ` Peter Xu
2016-04-24 22:12 ` [Qemu-devel] [V9 4/4] hw/pci-host: Emulate AMD IOMMU David Kiarie
2016-04-25 3:06 ` Peter Xu
2016-04-24 22:16 ` [Qemu-devel] [V9 0/4] " David Kiarie
2016-04-25 7:13 ` Jan Kiszka
2016-04-25 12:53 ` Marcel Apfelbaum
-- strict thread matches above, loose matches on Subject: below --
2016-04-29 22:42 David Kiarie
2016-04-29 22:42 ` [Qemu-devel] [V9 2/4] hw/i386: ACPI table for " David Kiarie
2016-05-01 13:45 ` Michael S. Tsirkin
2016-05-02 8:27 ` David Kiarie
2016-05-02 9:13 ` Igor Mammedov
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=5729A9B8.3060603@gmail.com \
--to=valentine.sinitsyn@gmail.com \
--cc=davidkiarie4@gmail.com \
--cc=imammedo@redhat.com \
--cc=jan.kiszka@web.de \
--cc=marcel@redhat.com \
--cc=mst@redhat.com \
--cc=qemu-devel@nongnu.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).