From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54619) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1axq3n-0001Rc-5y for qemu-devel@nongnu.org; Wed, 04 May 2016 02:13:29 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1axq3b-0005fk-KW for qemu-devel@nongnu.org; Wed, 04 May 2016 02:13:17 -0400 Received: from mout.web.de ([212.227.15.3]:53092) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1axq3b-0005ch-BQ for qemu-devel@nongnu.org; Wed, 04 May 2016 02:13:11 -0400 References: <1461969763-5193-1-git-send-email-davidkiarie4@gmail.com> From: Jan Kiszka Message-ID: <572992D1.7090408@web.de> Date: Wed, 4 May 2016 08:12:33 +0200 MIME-Version: 1.0 In-Reply-To: <1461969763-5193-1-git-send-email-davidkiarie4@gmail.com> Content-Type: text/plain; charset=iso-8859-15 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [V9 0/4] AMD IOMMU List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: David Kiarie , 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 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