From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:55723) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1azlBB-0000K8-Rk for qemu-devel@nongnu.org; Mon, 09 May 2016 09:24:58 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1azlB6-0003rV-Li for qemu-devel@nongnu.org; Mon, 09 May 2016 09:24:56 -0400 Received: from mail-wm0-x231.google.com ([2a00:1450:400c:c09::231]:38276) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1azlB6-0003rN-Ei for qemu-devel@nongnu.org; Mon, 09 May 2016 09:24:52 -0400 Received: by mail-wm0-x231.google.com with SMTP id g17so186833496wme.1 for ; Mon, 09 May 2016 06:24:52 -0700 (PDT) Date: Mon, 9 May 2016 14:24:49 +0100 From: Stefan Hajnoczi Message-ID: <20160509132449.GF3372@stefanha-x1.localdomain> References: <1461600281-23048-1-git-send-email-rjones@redhat.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="qftxBdZWiueWNAVY" Content-Disposition: inline In-Reply-To: <1461600281-23048-1-git-send-email-rjones@redhat.com> Subject: Re: [Qemu-devel] [PATCH v6] Add optionrom compatible with fw_cfg DMA version List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Richard W.M. Jones" Cc: qemu-devel@nongnu.org, marc.mari.barcelo@gmail.com, ehabkost@redhat.com, mst@redhat.com, kraxel@redhat.com, stefanha@redhat.com, pbonzini@redhat.com, lersek@redhat.com, rth@twiddle.net --qftxBdZWiueWNAVY Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Apr 25, 2016 at 05:04:40PM +0100, Richard W.M. Jones wrote: > v5 -> v6: >=20 > - Changed the xen_load_linux assertion as suggested by Stefan. >=20 > - I renamed the variables in get_e801_addr function, since the > registers were really (16 bit 8086-style) AX, not EAX etc. Also I > changed the GCC asm to make it a little bit more efficient. I > verified by disassembling the function that GCC is still generating > the right code after this change. >=20 > - Re-tested with small (342K) libguestfs initramfs and with large > (20M) Fedora initramfs, and works fine in both cases. No one has picked this up, so I have (for QEMU 2.7). Thanks, applied to my block tree: https://github.com/stefanha/qemu/commits/block Stefan --qftxBdZWiueWNAVY Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQEcBAEBAgAGBQJXMI+hAAoJEJykq7OBq3PIdlYIAK+cg20o8jltHJMvBNAQjGEI 5eBlvLpVDcKmML5l5M3S+j5AYosnHZN8wQerhrFKyKwy7qVleih8dLqXJQ6DLx1/ Mk/E8UjGfZdrwA2wFi0EQywslzEg+P3MG1JaLuhxoDsX9qTNvF0+bJbsw51Su2u2 ZZWvarmRjSUFr7Y0TJQ8aD2DQXAFDT4AczUUZMdI5kuajzWCibi+VLcLAKWNUmRf feG5H3bsvngHA3oXszdgcPTY1uvUOscpfcN23dBdbIWaD3mupVi6FgceMY8o5LHf YLi6vmf2YdkZFs0V32np2Bd95tiTrRYM3Z8O6tXkjMX1jGXpKmZdj+Sq42Cixtk= =4QcG -----END PGP SIGNATURE----- --qftxBdZWiueWNAVY--