From: Joerg Roedel <joro@8bytes.org>
To: Alex Williamson <alex.williamson@redhat.com>
Cc: Avi Kivity <avi@redhat.com>,
Benjamin Herrenschmidt <benh@kernel.crashing.org>,
kvm@vger.kernel.org, Anthony Liguori <anthony@codemonkey.ws>,
David Gibson <dwg@au1.ibm.com>, Paul Mackerras <pmac@au1.ibm.com>,
Alexey Kardashevskiy <aik@au1.ibm.com>,
"linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
linuxppc-dev <linuxppc-dev@lists.ozlabs.org>
Subject: Re: kvm PCI assignment & VFIO ramblings
Date: Thu, 4 Aug 2011 12:41:05 +0200 [thread overview]
Message-ID: <20110804104105.GC22329@8bytes.org> (raw)
In-Reply-To: <1312230476.2653.395.camel@bling.home>
On Mon, Aug 01, 2011 at 02:27:36PM -0600, Alex Williamson wrote:
> It's not clear to me how we could skip it. With VT-d, we'd have to
> implement an emulated interrupt remapper and hope that the guest picks
> unused indexes in the host interrupt remapping table before it could do
> anything useful with direct access to the MSI-X table. Maybe AMD IOMMU
> makes this easier?
AMD IOMMU provides remapping tables per-device, and not a global one.
But that does not make direct guest-access to the MSI-X table safe. The
table contains the table contains the interrupt-type and the vector
which is used as an index into the remapping table by the IOMMU. So when
the guest writes into its MSI-X table the remapping-table in the host
needs to be updated too.
Joerg
WARNING: multiple messages have this Message-ID (diff)
From: Joerg Roedel <joro@8bytes.org>
To: Alex Williamson <alex.williamson@redhat.com>
Cc: Alexey Kardashevskiy <aik@au1.ibm.com>,
kvm@vger.kernel.org, Paul Mackerras <pmac@au1.ibm.com>,
David Gibson <dwg@au1.ibm.com>, Avi Kivity <avi@redhat.com>,
Anthony Liguori <anthony@codemonkey.ws>,
"linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
linuxppc-dev <linuxppc-dev@lists.ozlabs.org>
Subject: Re: kvm PCI assignment & VFIO ramblings
Date: Thu, 4 Aug 2011 12:41:05 +0200 [thread overview]
Message-ID: <20110804104105.GC22329@8bytes.org> (raw)
In-Reply-To: <1312230476.2653.395.camel@bling.home>
On Mon, Aug 01, 2011 at 02:27:36PM -0600, Alex Williamson wrote:
> It's not clear to me how we could skip it. With VT-d, we'd have to
> implement an emulated interrupt remapper and hope that the guest picks
> unused indexes in the host interrupt remapping table before it could do
> anything useful with direct access to the MSI-X table. Maybe AMD IOMMU
> makes this easier?
AMD IOMMU provides remapping tables per-device, and not a global one.
But that does not make direct guest-access to the MSI-X table safe. The
table contains the table contains the interrupt-type and the vector
which is used as an index into the remapping table by the IOMMU. So when
the guest writes into its MSI-X table the remapping-table in the host
needs to be updated too.
Joerg
next prev parent reply other threads:[~2011-08-04 10:41 UTC|newest]
Thread overview: 322+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-29 23:58 kvm PCI assignment & VFIO ramblings Benjamin Herrenschmidt
2011-07-29 23:58 ` Benjamin Herrenschmidt
2011-07-30 18:20 ` Alex Williamson
2011-07-30 18:20 ` [Qemu-devel] " Alex Williamson
2011-07-30 18:20 ` Alex Williamson
2011-07-30 23:54 ` Benjamin Herrenschmidt
2011-07-30 23:54 ` [Qemu-devel] " Benjamin Herrenschmidt
2011-07-30 23:54 ` Benjamin Herrenschmidt
2011-08-01 18:59 ` Alex Williamson
2011-08-01 18:59 ` [Qemu-devel] " Alex Williamson
2011-08-01 18:59 ` Alex Williamson
2011-08-02 2:00 ` Benjamin Herrenschmidt
2011-08-02 2:00 ` [Qemu-devel] " Benjamin Herrenschmidt
2011-08-02 2:00 ` Benjamin Herrenschmidt
2011-07-30 23:55 ` Benjamin Herrenschmidt
2011-07-30 23:55 ` [Qemu-devel] " Benjamin Herrenschmidt
2011-07-30 23:55 ` Benjamin Herrenschmidt
2011-08-02 8:28 ` David Gibson
2011-08-02 8:28 ` [Qemu-devel] " David Gibson
2011-08-02 8:28 ` David Gibson
2011-08-02 18:14 ` Alex Williamson
2011-08-02 18:14 ` [Qemu-devel] " Alex Williamson
2011-08-02 18:14 ` Alex Williamson
2011-08-02 18:35 ` Alex Williamson
2011-08-02 18:35 ` [Qemu-devel] " Alex Williamson
2011-08-02 18:35 ` Alex Williamson
2011-08-03 2:04 ` David Gibson
2011-08-03 2:04 ` [Qemu-devel] " David Gibson
2011-08-03 2:04 ` David Gibson
2011-08-03 3:44 ` Alex Williamson
2011-08-03 3:44 ` [Qemu-devel] " Alex Williamson
2011-08-03 3:44 ` Alex Williamson
2011-08-04 0:39 ` David Gibson
2011-08-04 0:39 ` [Qemu-devel] " David Gibson
2011-08-08 8:28 ` Avi Kivity
2011-08-08 8:28 ` [Qemu-devel] " Avi Kivity
2011-08-08 8:28 ` Avi Kivity
2011-08-09 23:24 ` Alex Williamson
2011-08-09 23:24 ` [Qemu-devel] " Alex Williamson
2011-08-09 23:24 ` Alex Williamson
2011-08-10 2:48 ` Benjamin Herrenschmidt
2011-08-10 2:48 ` [Qemu-devel] " Benjamin Herrenschmidt
2011-08-10 2:48 ` Benjamin Herrenschmidt
2011-08-20 16:51 ` Alex Williamson
2011-08-20 16:51 ` [Qemu-devel] " Alex Williamson
2011-08-20 16:51 ` Alex Williamson
2011-08-22 5:55 ` David Gibson
2011-08-22 5:55 ` [Qemu-devel] " David Gibson
2011-08-22 5:55 ` David Gibson
2011-08-22 15:45 ` Alex Williamson
2011-08-22 15:45 ` [Qemu-devel] " Alex Williamson
2011-08-22 21:01 ` Benjamin Herrenschmidt
2011-08-22 21:01 ` [Qemu-devel] " Benjamin Herrenschmidt
2011-08-22 21:01 ` Benjamin Herrenschmidt
2011-08-23 19:30 ` Alex Williamson
2011-08-23 19:30 ` [Qemu-devel] " Alex Williamson
2011-08-23 19:30 ` Alex Williamson
2011-08-23 23:51 ` Benjamin Herrenschmidt
2011-08-23 23:51 ` [Qemu-devel] " Benjamin Herrenschmidt
2011-08-23 23:51 ` Benjamin Herrenschmidt
2011-08-24 3:40 ` Alexander Graf
2011-08-24 3:40 ` [Qemu-devel] " Alexander Graf
2011-08-24 3:40 ` Alexander Graf
2011-08-24 14:47 ` Alex Williamson
2011-08-24 14:47 ` [Qemu-devel] " Alex Williamson
2011-08-24 14:47 ` Alex Williamson
2011-08-24 8:43 ` Joerg Roedel
2011-08-24 8:43 ` [Qemu-devel] " Joerg Roedel
2011-08-24 8:43 ` Joerg Roedel
2011-08-24 14:56 ` Alex Williamson
2011-08-24 14:56 ` [Qemu-devel] " Alex Williamson
2011-08-24 14:56 ` Alex Williamson
2011-08-25 11:01 ` Roedel, Joerg
2011-08-25 11:01 ` [Qemu-devel] " Roedel, Joerg
2011-08-25 11:01 ` Roedel, Joerg
2011-08-23 2:38 ` David Gibson
2011-08-23 2:38 ` [Qemu-devel] " David Gibson
2011-08-23 2:38 ` David Gibson
2011-08-23 16:23 ` Alex Williamson
2011-08-23 16:23 ` [Qemu-devel] " Alex Williamson
2011-08-23 16:23 ` Alex Williamson
2011-08-23 23:41 ` Benjamin Herrenschmidt
2011-08-23 23:41 ` [Qemu-devel] " Benjamin Herrenschmidt
2011-08-23 23:41 ` Benjamin Herrenschmidt
2011-08-24 3:36 ` Alexander Graf
2011-08-24 3:36 ` [Qemu-devel] " Alexander Graf
2011-08-24 3:36 ` Alexander Graf
2011-08-22 6:30 ` Avi Kivity
2011-08-22 6:30 ` [Qemu-devel] " Avi Kivity
2011-08-22 6:30 ` Avi Kivity
2011-08-22 10:46 ` Joerg Roedel
2011-08-22 10:46 ` [Qemu-devel] " Joerg Roedel
2011-08-22 10:46 ` Joerg Roedel
2011-08-22 10:51 ` Avi Kivity
2011-08-22 10:51 ` [Qemu-devel] " Avi Kivity
2011-08-22 10:51 ` Avi Kivity
2011-08-22 12:36 ` Roedel, Joerg
2011-08-22 12:36 ` [Qemu-devel] " Roedel, Joerg
2011-08-22 12:36 ` Roedel, Joerg
2011-08-22 12:42 ` Avi Kivity
2011-08-22 12:42 ` [Qemu-devel] " Avi Kivity
2011-08-22 12:42 ` Avi Kivity
2011-08-22 12:55 ` Roedel, Joerg
2011-08-22 12:55 ` [Qemu-devel] " Roedel, Joerg
2011-08-22 12:55 ` Roedel, Joerg
2011-08-22 13:06 ` Avi Kivity
2011-08-22 13:06 ` [Qemu-devel] " Avi Kivity
2011-08-22 13:06 ` Avi Kivity
2011-08-22 13:15 ` Roedel, Joerg
2011-08-22 13:15 ` [Qemu-devel] " Roedel, Joerg
2011-08-22 13:15 ` Roedel, Joerg
2011-08-22 13:17 ` Avi Kivity
2011-08-22 13:17 ` [Qemu-devel] " Avi Kivity
2011-08-22 13:17 ` Avi Kivity
2011-08-22 14:37 ` Roedel, Joerg
2011-08-22 14:37 ` [Qemu-devel] " Roedel, Joerg
2011-08-22 14:37 ` Roedel, Joerg
2011-08-22 20:53 ` Benjamin Herrenschmidt
2011-08-22 20:53 ` [Qemu-devel] " Benjamin Herrenschmidt
2011-08-22 20:53 ` Benjamin Herrenschmidt
2011-08-22 17:25 ` Joerg Roedel
2011-08-22 17:25 ` [Qemu-devel] " Joerg Roedel
2011-08-22 17:25 ` Joerg Roedel
2011-08-22 19:17 ` Alex Williamson
2011-08-22 19:17 ` [Qemu-devel] " Alex Williamson
2011-08-22 19:17 ` Alex Williamson
2011-08-23 13:14 ` Roedel, Joerg
2011-08-23 13:14 ` [Qemu-devel] " Roedel, Joerg
2011-08-23 13:14 ` Roedel, Joerg
2011-08-23 17:08 ` Alex Williamson
2011-08-23 17:08 ` [Qemu-devel] " Alex Williamson
2011-08-23 17:08 ` Alex Williamson
2011-08-24 8:52 ` Roedel, Joerg
2011-08-24 8:52 ` [Qemu-devel] " Roedel, Joerg
2011-08-24 8:52 ` Roedel, Joerg
2011-08-24 15:07 ` Alex Williamson
2011-08-24 15:07 ` [Qemu-devel] " Alex Williamson
2011-08-24 15:07 ` Alex Williamson
2011-08-25 12:31 ` Roedel, Joerg
2011-08-25 12:31 ` [Qemu-devel] " Roedel, Joerg
2011-08-25 12:31 ` Roedel, Joerg
2011-08-25 13:25 ` Alexander Graf
2011-08-25 13:25 ` [Qemu-devel] " Alexander Graf
2011-08-25 13:25 ` Alexander Graf
2011-08-26 4:24 ` David Gibson
2011-08-26 4:24 ` [Qemu-devel] " David Gibson
2011-08-26 4:24 ` David Gibson
2011-08-26 9:24 ` Roedel, Joerg
2011-08-26 9:24 ` [Qemu-devel] " Roedel, Joerg
2011-08-26 9:24 ` Roedel, Joerg
2011-08-28 13:14 ` Avi Kivity
2011-08-28 13:14 ` [Qemu-devel] " Avi Kivity
2011-08-28 13:14 ` Avi Kivity
2011-08-28 13:56 ` Joerg Roedel
2011-08-28 13:56 ` [Qemu-devel] " Joerg Roedel
2011-08-28 13:56 ` Joerg Roedel
2011-08-28 14:04 ` Avi Kivity
2011-08-28 14:04 ` [Qemu-devel] " Avi Kivity
2011-08-28 14:04 ` Avi Kivity
2011-08-30 16:14 ` Joerg Roedel
2011-08-30 16:14 ` [Qemu-devel] " Joerg Roedel
2011-08-30 16:14 ` Joerg Roedel
2011-08-22 21:03 ` Benjamin Herrenschmidt
2011-08-22 21:03 ` [Qemu-devel] " Benjamin Herrenschmidt
2011-08-22 21:03 ` Benjamin Herrenschmidt
2011-08-23 13:18 ` Roedel, Joerg
2011-08-23 13:18 ` [Qemu-devel] " Roedel, Joerg
2011-08-23 13:18 ` Roedel, Joerg
2011-08-23 23:35 ` Benjamin Herrenschmidt
2011-08-23 23:35 ` [Qemu-devel] " Benjamin Herrenschmidt
2011-08-23 23:35 ` Benjamin Herrenschmidt
2011-08-24 8:53 ` Roedel, Joerg
2011-08-24 8:53 ` [Qemu-devel] " Roedel, Joerg
2011-08-24 8:53 ` Roedel, Joerg
2011-08-22 20:29 ` aafabbri
2011-08-22 20:29 ` [Qemu-devel] " aafabbri
2011-08-22 20:29 ` aafabbri
2011-08-22 20:49 ` Benjamin Herrenschmidt
2011-08-22 20:49 ` [Qemu-devel] " Benjamin Herrenschmidt
2011-08-22 21:38 ` aafabbri
2011-08-22 21:38 ` [Qemu-devel] " aafabbri
2011-08-22 21:38 ` aafabbri
2011-08-22 21:49 ` Benjamin Herrenschmidt
2011-08-22 21:49 ` [Qemu-devel] " Benjamin Herrenschmidt
2011-08-22 21:49 ` Benjamin Herrenschmidt
2011-08-23 0:52 ` aafabbri
2011-08-23 0:52 ` [Qemu-devel] " aafabbri
2011-08-23 0:52 ` aafabbri
2011-08-23 6:54 ` Benjamin Herrenschmidt
2011-08-23 6:54 ` [Qemu-devel] " Benjamin Herrenschmidt
2011-08-23 6:54 ` Benjamin Herrenschmidt
2011-08-23 11:09 ` Joerg Roedel
2011-08-23 11:09 ` [Qemu-devel] " Joerg Roedel
2011-08-23 11:09 ` Joerg Roedel
2011-08-23 17:01 ` Alex Williamson
2011-08-23 17:01 ` [Qemu-devel] " Alex Williamson
2011-08-23 17:01 ` Alex Williamson
2011-08-23 17:33 ` Aaron Fabbri
2011-08-23 17:33 ` [Qemu-devel] " Aaron Fabbri
2011-08-23 17:33 ` Aaron Fabbri
2011-08-23 18:01 ` Alex Williamson
2011-08-23 18:01 ` [Qemu-devel] " Alex Williamson
2011-08-23 18:01 ` Alex Williamson
2011-08-24 9:10 ` Joerg Roedel
2011-08-24 9:10 ` [Qemu-devel] " Joerg Roedel
2011-08-24 9:10 ` Joerg Roedel
2011-08-24 21:13 ` Alex Williamson
2011-08-24 21:13 ` [Qemu-devel] " Alex Williamson
2011-08-24 21:13 ` Alex Williamson
2011-08-25 10:54 ` Roedel, Joerg
2011-08-25 10:54 ` [Qemu-devel] " Roedel, Joerg
2011-08-25 10:54 ` Roedel, Joerg
2011-08-25 15:38 ` Don Dutile
2011-08-25 15:38 ` [Qemu-devel] " Don Dutile
2011-08-25 15:38 ` Don Dutile
2011-08-25 16:46 ` Roedel, Joerg
2011-08-25 16:46 ` [Qemu-devel] " Roedel, Joerg
2011-08-25 16:46 ` Roedel, Joerg
2011-08-25 17:20 ` Alex Williamson
2011-08-25 17:20 ` [Qemu-devel] " Alex Williamson
2011-08-25 17:20 ` Alex Williamson
2011-08-25 18:05 ` Joerg Roedel
2011-08-25 18:05 ` [Qemu-devel] " Joerg Roedel
2011-08-25 18:05 ` Joerg Roedel
2011-08-26 18:04 ` Alex Williamson
2011-08-26 18:04 ` [Qemu-devel] " Alex Williamson
2011-08-26 18:04 ` Alex Williamson
2011-08-30 16:13 ` Joerg Roedel
2011-08-30 16:13 ` [Qemu-devel] " Joerg Roedel
2011-08-30 16:13 ` Joerg Roedel
2011-08-23 11:04 ` Joerg Roedel
2011-08-23 11:04 ` [Qemu-devel] " Joerg Roedel
2011-08-23 11:04 ` Joerg Roedel
2011-08-23 16:54 ` aafabbri
2011-08-23 16:54 ` [Qemu-devel] " aafabbri
2011-08-23 16:54 ` aafabbri
2011-08-24 9:14 ` Roedel, Joerg
2011-08-24 9:14 ` [Qemu-devel] " Roedel, Joerg
2011-08-24 9:14 ` Roedel, Joerg
2011-08-24 9:33 ` David Gibson
2011-08-24 9:33 ` [Qemu-devel] " David Gibson
2011-08-24 9:33 ` David Gibson
2011-08-24 11:03 ` Roedel, Joerg
2011-08-24 11:03 ` [Qemu-devel] " Roedel, Joerg
2011-08-24 11:03 ` Roedel, Joerg
2011-08-26 4:20 ` David Gibson
2011-08-26 4:20 ` [Qemu-devel] " David Gibson
2011-08-26 4:20 ` David Gibson
2011-08-26 9:33 ` Roedel, Joerg
2011-08-26 9:33 ` [Qemu-devel] " Roedel, Joerg
2011-08-26 9:33 ` Roedel, Joerg
2011-08-26 14:07 ` Alexander Graf
2011-08-26 14:07 ` [Qemu-devel] " Alexander Graf
2011-08-26 14:07 ` Alexander Graf
2011-08-26 15:24 ` Joerg Roedel
2011-08-26 15:24 ` [Qemu-devel] " Joerg Roedel
2011-08-26 15:24 ` Joerg Roedel
2011-08-26 15:29 ` Alexander Graf
2011-08-26 15:29 ` [Qemu-devel] " Alexander Graf
2011-08-26 15:29 ` Alexander Graf
2011-08-26 17:52 ` Aaron Fabbri
2011-08-26 17:52 ` [Qemu-devel] " Aaron Fabbri
2011-08-26 19:35 ` Chris Wright
2011-08-26 19:35 ` [Qemu-devel] " Chris Wright
2011-08-26 19:35 ` Chris Wright
2011-08-26 20:17 ` Aaron Fabbri
2011-08-26 20:17 ` [Qemu-devel] " Aaron Fabbri
2011-08-26 20:17 ` Aaron Fabbri
2011-08-26 21:06 ` Chris Wright
2011-08-26 21:06 ` [Qemu-devel] " Chris Wright
2011-08-26 21:06 ` Chris Wright
2011-08-30 1:29 ` David Gibson
2011-08-30 1:29 ` [Qemu-devel] " David Gibson
2011-08-30 1:29 ` David Gibson
2011-08-04 10:35 ` Joerg Roedel
2011-08-04 10:35 ` [Qemu-devel] " Joerg Roedel
2011-08-04 10:35 ` Joerg Roedel
2011-07-30 22:21 ` Benjamin Herrenschmidt
2011-07-30 22:21 ` Benjamin Herrenschmidt
2011-08-01 16:40 ` Alex Williamson
2011-08-01 16:40 ` Alex Williamson
2011-08-02 1:29 ` Benjamin Herrenschmidt
2011-07-31 14:09 ` Avi Kivity
2011-07-31 14:09 ` Avi Kivity
2011-08-01 20:27 ` Alex Williamson
2011-08-01 20:27 ` Alex Williamson
2011-08-02 8:32 ` Avi Kivity
2011-08-02 8:32 ` Avi Kivity
2011-08-04 10:41 ` Joerg Roedel [this message]
2011-08-04 10:41 ` Joerg Roedel
2011-08-05 10:26 ` Benjamin Herrenschmidt
2011-08-05 10:26 ` Benjamin Herrenschmidt
2011-08-05 12:57 ` Joerg Roedel
2011-08-05 12:57 ` Joerg Roedel
2011-08-02 1:27 ` Benjamin Herrenschmidt
2011-08-02 1:27 ` Benjamin Herrenschmidt
2011-08-02 9:12 ` Avi Kivity
2011-08-02 9:12 ` Avi Kivity
2011-08-02 12:58 ` Benjamin Herrenschmidt
2011-08-02 12:58 ` Benjamin Herrenschmidt
2011-08-02 13:39 ` Avi Kivity
2011-08-02 13:39 ` Avi Kivity
2011-08-02 15:34 ` Alex Williamson
2011-08-02 15:34 ` Alex Williamson
2011-08-02 21:29 ` Konrad Rzeszutek Wilk
2011-08-02 21:29 ` Konrad Rzeszutek Wilk
2011-08-03 1:02 ` Alex Williamson
2011-08-03 1:02 ` Alex Williamson
2011-08-02 14:39 ` Alex Williamson
2011-08-02 14:39 ` Alex Williamson
2011-08-01 2:48 ` David Gibson
2011-08-04 10:27 ` Joerg Roedel
2011-08-04 10:27 ` Joerg Roedel
2011-08-05 10:42 ` Benjamin Herrenschmidt
2011-08-05 10:42 ` Benjamin Herrenschmidt
2011-08-05 13:44 ` Joerg Roedel
2011-08-05 13:44 ` Joerg Roedel
2011-08-05 22:49 ` Benjamin Herrenschmidt
2011-08-05 22:49 ` Benjamin Herrenschmidt
2011-08-05 15:10 ` Alex Williamson
2011-08-05 15:10 ` Alex Williamson
2011-08-08 6:07 ` David Gibson
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=20110804104105.GC22329@8bytes.org \
--to=joro@8bytes.org \
--cc=aik@au1.ibm.com \
--cc=alex.williamson@redhat.com \
--cc=anthony@codemonkey.ws \
--cc=avi@redhat.com \
--cc=benh@kernel.crashing.org \
--cc=dwg@au1.ibm.com \
--cc=kvm@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=pmac@au1.ibm.com \
/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.