From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:60858) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Ufzmt-0001oD-ER for qemu-devel@nongnu.org; Fri, 24 May 2013 17:44:37 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Ufzmq-00075Q-4o for qemu-devel@nongnu.org; Fri, 24 May 2013 17:44:35 -0400 Received: from mail-ea0-x233.google.com ([2a00:1450:4013:c01::233]:45631) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Ufzmp-00075K-UF for qemu-devel@nongnu.org; Fri, 24 May 2013 17:44:32 -0400 Received: by mail-ea0-f179.google.com with SMTP id z16so2823961ead.10 for ; Fri, 24 May 2013 14:44:30 -0700 (PDT) Sender: Paolo Bonzini Message-ID: <519FDF35.207@redhat.com> Date: Fri, 24 May 2013 23:44:21 +0200 From: Paolo Bonzini MIME-Version: 1.0 References: In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] FPU x86 instructions error List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Clemens Kolbitsch Cc: "qemu-devel@nongnu.org" Il 24/05/2013 23:39, Clemens Kolbitsch ha scritto: > Hi guys, > > we recently had an issue with running a program using FPU instructions > to obtain the current EIP (basically a weird way of "call 0; pop eax") > that was not working on QEMU (with TCG). > > Looking at the problem, we found this patch to be useful/fixing the issue: > > https://launchpadlibrarian.net/140457932/patch-qemu-1.5.0-fpip.diff > > Looking through the DEVEL archives, I found this patch > > http://lists.nongnu.org/archive/html/qemu-devel/2011-06/msg01206.html > > that adds the FPU flags to the environment, but is only using them for KVM. > > I was wondering - since the above patch is rather old, you have > probably come across it before - if there was a reason for not > including it in QEMU (I checked in git:master and it's not applied). > If there isn't, maybe it'd be worth re-considering :) For the TCG patch, there is no Signed-off-by and using a helper is not necessary. For the KVM patch, it simply fell through the cracks, I believe. Paolo