From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34801) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a0xAQ-0005nS-0z for qemu-devel@nongnu.org; Mon, 23 Nov 2015 14:52:51 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1a0xAM-0000re-S1 for qemu-devel@nongnu.org; Mon, 23 Nov 2015 14:52:49 -0500 Received: from mx1.redhat.com ([209.132.183.28]:39416) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a0xAM-0000rX-K5 for qemu-devel@nongnu.org; Mon, 23 Nov 2015 14:52:46 -0500 References: <87si401wpf.fsf@blackfin.pond.sub.org> <874mggo3yc.fsf@blackfin.pond.sub.org> <243512039.12566588.1448050691647.JavaMail.zimbra@redhat.com> <87oael587p.fsf@blackfin.pond.sub.org> <802808837.15238786.1448280956337.JavaMail.zimbra@redhat.com> <87r3jg26fw.fsf@blackfin.pond.sub.org> <1032710289.15307566.1448286512454.JavaMail.zimbra@redhat.com> <87610sx0zd.fsf@blackfin.pond.sub.org> <560475200.15358637.1448288207922.JavaMail.zimbra@redhat.com> <87mvu4srid.fsf@blackfin.pond.sub.org> From: Eric Blake Message-ID: <56536E89.4090408@redhat.com> Date: Mon, 23 Nov 2015 12:52:41 -0700 MIME-Version: 1.0 In-Reply-To: <87mvu4srid.fsf@blackfin.pond.sub.org> Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="6SMV0MII7fhbnk2jmRdHHim5c3wJEdx4j" Subject: Re: [Qemu-devel] ivshmem property size should be a size, not a string List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Markus Armbruster , =?UTF-8?Q?Marc-Andr=c3=a9_Lureau?= Cc: marcandre lureau , Claudio Fontana , qemu-devel@nongnu.org, Luiz Capitulino This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --6SMV0MII7fhbnk2jmRdHHim5c3wJEdx4j Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 11/23/2015 07:46 AM, Markus Armbruster wrote: >>> If it's not broken, please explain to me how the guest should find ou= t >>> whether its ivshmem device sports a doorbell. >> >> If you have received ID, you should be good to use the doorbell. >=20 > That's not a complete answer, so let me try a different tack. >=20 > What value will a read of register IVPosition yield >=20 > * if the device has no doorbell: >=20 > I think the answer is -1. >=20 > * if the device has a doorbell, but it isn't ready, yet: >=20 > I think the answer is -1. >=20 > * if the device has a doorbell, and it is ready. >=20 > This is the part you answered already: >=3D 0. >=20 > Please correct mistakes. For what it's worth, I'm agreeing with Markus here - we have a race in the guest learning whether doorbell is supported, and it's better to do a clean break in API for 2.5 along with ALL the other fixes into using a sane union of types (splitting between ivshmem-plain and ivshmem-doorbell). If you absolutely want it, you can still maintain 'ivshmem' as a front-end that maps to either ivshmem-plain, ivshmem-doorbell, or an error (nonsensical combination of requests), but having a sane interface as your starting point, rather than an amalgamation of cruft that exists only due to the early implementation, seems like the way to go. I'm still waiting for any evidence that we even care about backwards compatibility - what apps are out there that are actively using ivshmem with its horrible pre-2.5 interface, and why can they not be fixed to use the sane 2.5 interface? Libvirt is NOT exposing ivshmem yet, in part because the 2.4 interface was not very robust. We already have a clean break now due to all your work for 2.5; let's take advantage of it, and make 2.5 robust, rather than breaking things again in 2.6. --=20 Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org --6SMV0MII7fhbnk2jmRdHHim5c3wJEdx4j Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 Comment: Public key at http://people.redhat.com/eblake/eblake.gpg Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQEcBAEBCAAGBQJWU26JAAoJEKeha0olJ0Nq+xAH+wYta54/9lU5ZhSae52uJc2D /HcN5ja6g/kCra/UuK76AsUKvBZXY9wZG1PJ+/fkk5gofsMCqB2gZmQzYCkxsrAe 7uIZX9W7YTDc+PjwUE7nZzFbRaada3dwsWDhj2oYzSDNbiK7+Z134LHXWFIMT0Ra 8/DkwjUSWqKRwdS1Q9vNxAR40gjNndjtcq4owv+CN2/g44A2THAIzXFJW5Oci8av nthhN2qwVNKw2G4U9usXNspXGM2c25uFEzkzDS2me5p2jiwVodysGf6/Mr0fZbYv 9DA2t+pU8wYa7yYwcW6RvS3163KaKrtRli9ckKXki0AIg+Fc6g1NXbac8OfNo+U= =wlvn -----END PGP SIGNATURE----- --6SMV0MII7fhbnk2jmRdHHim5c3wJEdx4j--