From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:37849) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yx4sq-00025i-7P for qemu-devel@nongnu.org; Mon, 25 May 2015 22:46:25 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Yx4sn-0003bf-6q for qemu-devel@nongnu.org; Mon, 25 May 2015 22:46:24 -0400 Date: Tue, 26 May 2015 12:46:28 +1000 From: David Gibson Message-ID: <20150526024628.GA30620@voom.redhat.com> References: <1429964684-23872-1-git-send-email-aik@ozlabs.ru> <1429964684-23872-7-git-send-email-aik@ozlabs.ru> <55633A54.8080807@ozlabs.ru> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="/9DWx/yDrRhgMJTb" Content-Disposition: inline In-Reply-To: <55633A54.8080807@ozlabs.ru> Subject: Re: [Qemu-devel] [PATCH qemu v7 06/14] spapr_iommu: Introduce "enabled" state for TCE table List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alexey Kardashevskiy Cc: Alexander Graf , Michael Roth , qemu-devel@nongnu.org, Alex Williamson , qemu-ppc@nongnu.org, Paolo Bonzini --/9DWx/yDrRhgMJTb Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, May 26, 2015 at 01:05:56AM +1000, Alexey Kardashevskiy wrote: > Hi Paolo, >=20 > I have had a conversation with Mike and it turns out I am not allowed to > create/remove memory regions dynamically (docs/memory.txt:101); otherwise > "destroying regions during reset causes assertion in RCU thread during > PHB/IOMMU unplug/unparent". Is it because patch just missing some > unref()/unparent() call or it is totally wrong and I have to implement > subregions (on a PCI bus address space) myself if I want dynamic DMA > windows? Thanks! So, the sentences after that one note an exception for alias and container regions. I think iommu regions should behave similarly - in a sense they're just a procedurally generated collection of alias regions. If it's not true now that they can be unparented at any time like alias regions, we should probably try to make it true. --=20 David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson --/9DWx/yDrRhgMJTb Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJVY96EAAoJEGw4ysog2bOSCGsP/0yLSx+3Z86V0L3rhA3zaZxm Chu7R7ge7Fx4oW2ydnQO+QEzbGEixNEwxYFkM1LBRjDRCkL6kGmacBopsEqeIDlw yMr17Hky/CqjB41BMCOLM3DaK2LL5lbegM37f2xI5cHEkpR0irypLWYzOT7JjZmu NBI4iYwl50Uisvb28vj7i0xc9l95/8+SKzmGDNsS/XidN4vCiY9YVijtbmL3kZ61 oU3wuo427071FqqF4jJsUEZXnZBIbI/dqHxGBuSpdlcaW5m1P5c0tvWG0hGGQhph YzG8+1sHbXacYjF/oArb47TZemKd4xtAftEQ5ChJyYNeUU1u118bxdgaGI2HdeEV TiergzFz5evNcKC+3jD2kLHIuSqFPuILJSyV3T13psF/hCpq9rj68C+w99hOl97q wCmzVV1tXk4V5rjID4FArk3LB6BKtVD+QHDqmCxDyAFkFjpivgoSIT8y07xNKOkJ CzXjI6gcbX/VvmonvxnUJD3H9ipKcPyQgwa5vARniTzluklIHeB0TrY1Bpz53yC5 Pot6ltMekrYCYH9JI655gOD+VYTslf9XPdLaXN0xqF56jqYWMigSJTY2+sMJWOGh 25fTMgFz2JdWpuhbqCBNKaXwEwFPH0fGi+ooS+1xkYnKUwDxK0mCf9MjfwKe5YMa 6iwsLrFKJoA3Ebg+GaSC =we+W -----END PGP SIGNATURE----- --/9DWx/yDrRhgMJTb--