<div dir="ltr">Thanks for the reviews!<div><br></div><div>I'll get started on a V2 :) </div><div><br></div><div>--Nico </div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Oct 5, 2020 at 5:32 AM Peter Krempa <<a href="mailto:pkrempa@redhat.com">pkrempa@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, Oct 05, 2020 at 11:11:54 +0200, David Hildenbrand wrote:<br>
> On 05.10.20 10:06, Peter Krempa wrote:<br>
> > On Fri, Oct 02, 2020 at 11:59:43 -0400, Nico Pache wrote:<br>
> >> xml:<br>
> >> <devices><br>
> >>   <memballoon model='virtio' free-page-reporting='on'/><br>
> > <br>
> > according to what you state in the cover-letter this is a<br>
> > 'virtio-balloon-pci' feature. We usually put stuff which depends on a<br>
> > specific model of the device into the <driver> subelement of the device<br>
> > element.<br>
> <br>
> IIRC, this should work for virtio-balloon-*, including virtio-balloon-ccw.<br>
<br>
In that case the qemu capability detection should be added also for the<br>
ccw version and capability name needs to be modified to not include pci.<br>
<br>
</blockquote></div>