qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Jan Kiszka <jan.kiszka@web.de>
To: David Kiarie <davidkiarie4@gmail.com>, qemu-devel@nongnu.org
Cc: mst@redhat.com, peterx@redhat.com, valentine.sinitsyn@gmail.com,
	jan.kiszka@web.de, marcel@redhat.com, imammedo@redhat.com
Subject: Re: [Qemu-devel] [V9 0/4] AMD IOMMU
Date: Wed, 4 May 2016 08:12:33 +0200	[thread overview]
Message-ID: <572992D1.7090408@web.de> (raw)
In-Reply-To: <1461969763-5193-1-git-send-email-davidkiarie4@gmail.com>

On 2016-04-30 00:42, David Kiarie wrote:
> These series adds AMD IOMMU support to Qemu. It's currently in the 9th version.
> 
> In this series I have (hopefully) addressed all the comments made in the previous version.
> I have also tested and successfully passed-through PCI device 'ac97' with more devices to be tested.
> 

I've done some basic testing with a Jailhouse setup and found it
working. The ACPI table is now properly parsed and the DMA remapping was
not disturbing the system after Jailhouse was activated.

However, it was also still not intervening after I started to corrupt
the configuration, removed DMA target properties from most of the RAM or
dropped PCI devices.

You are not dropping invalid remapping requests, are you? According to
the logs, you are detecting them at least:

(amd-iommu)amd_iommu_get_dte: Device Table at 0x3b0d4000
(amd-iommu)amd_iommu_get_dte: Pte entry at 0x0 is invalid
(amd-iommu)amd_iommu_translate: devid: 00:02.0 gpa 0x32f39480 hpa 0x32f39000

It's a bit hard to test right now if remapping is actually properly
working in all important cases if you do not reject invalid ones.

Jan

  parent reply	other threads:[~2016-05-04  6:13 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-29 22:42 [Qemu-devel] [V9 0/4] AMD IOMMU David Kiarie
2016-04-29 22:42 ` [Qemu-devel] [V9 1/4] hw/i386: Introduce " David Kiarie
2016-05-01 14:14   ` Michael S. Tsirkin
2016-05-03 16:11     ` David Kiarie
2016-05-03 16:18       ` Jan Kiszka
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
2016-04-29 22:42 ` [Qemu-devel] [V9 3/4] hw/core: Add AMD IOMMU to machine properties David Kiarie
2016-05-01  9:15   ` Marcel Apfelbaum
     [not found] ` <1461969763-5193-5-git-send-email-davidkiarie4@gmail.com>
2016-05-01  9:15   ` [Qemu-devel] [V9 4/4] hw/pci-host: Emulate AMD IOMMU Marcel Apfelbaum
2016-05-01 13:47 ` [Qemu-devel] [V9 0/4] " Michael S. Tsirkin
2016-05-01 14:00   ` Michael S. Tsirkin
2016-05-02  8:33     ` David Kiarie
2016-05-04  6:12 ` Jan Kiszka [this message]
2016-05-04  7:05   ` David Kiarie
2016-05-04  7:26     ` Jan Kiszka
2016-05-04  7:39     ` Valentine Sinitsyn
2016-05-04 10:51       ` David Kiarie
2016-05-04 10:58         ` Valentine Sinitsyn
2016-05-04 11:02           ` David Kiarie
2016-05-04 11:05             ` Valentine Sinitsyn
2016-05-05 14:20               ` David Kiarie
  -- strict thread matches above, loose matches on Subject: below --
2016-04-24 22:12 David Kiarie
2016-04-24 22:16 ` David Kiarie
2016-04-25  7:13   ` Jan Kiszka
2016-04-25 12:53 ` Marcel Apfelbaum

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=572992D1.7090408@web.de \
    --to=jan.kiszka@web.de \
    --cc=davidkiarie4@gmail.com \
    --cc=imammedo@redhat.com \
    --cc=marcel@redhat.com \
    --cc=mst@redhat.com \
    --cc=peterx@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=valentine.sinitsyn@gmail.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 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).