From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:36454) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gVFPF-0000bY-5a for qemu-devel@nongnu.org; Fri, 07 Dec 2018 07:38:57 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gVFPB-0002lp-6B for qemu-devel@nongnu.org; Fri, 07 Dec 2018 07:38:57 -0500 Date: Fri, 7 Dec 2018 13:38:41 +0100 From: Cornelia Huck Message-ID: <20181207133841.102a923e.cohuck@redhat.com> In-Reply-To: <7e03f504-807d-12d4-368d-ed3b069b351e@de.ibm.com> References: <20181205145131.28467-1-cohuck@redhat.com> <5d274f2c-23a2-0c4f-9f2d-07cbe529c5ac@de.ibm.com> <20181207132946.00df1f5a.cohuck@redhat.com> <7e03f504-807d-12d4-368d-ed3b069b351e@de.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH RFC] vfio-ap: flag as compatible with balloon List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Christian Borntraeger Cc: Tony Krowiak , Halil Pasic , Pierre Morel , Alex Williamson , David Hildenbrand , qemu-s390x@nongnu.org, qemu-devel@nongnu.org On Fri, 7 Dec 2018 13:32:14 +0100 Christian Borntraeger wrote: > On 07.12.2018 13:29, Cornelia Huck wrote: > > On Fri, 7 Dec 2018 13:17:02 +0100 > > Christian Borntraeger wrote: > > > >> On 05.12.2018 15:51, Cornelia Huck wrote: > >>> vfio-ap devices do not pin any pages in the host. Therefore, they > >>> are belived to be compatible with memory ballooning. > >>> > >>> Flag them as compatible, so both vfio-ap and a balloon can be > >>> used simultaneously. > >>> > >>> Signed-off-by: Cornelia Huck > >> > >> Does it make sense to add cc stable for 3.1? > > > > Can do that, given that s390x systems really rely on the ballooner in > > general. > > Well, not relying is the wrong word. It is just strange that things like > > 2 > 1 > > do work > but suddenly no longer work when an AP device is added. > Let me rephrase that: Current setups rely on the functionality, and we don't want to break them by adding ap :)