From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34846) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Voqtb-0001Ts-H1 for qemu-devel@nongnu.org; Fri, 06 Dec 2013 03:36:29 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VoqtV-0005hD-Ii for qemu-devel@nongnu.org; Fri, 06 Dec 2013 03:36:23 -0500 Received: from mx1.redhat.com ([209.132.183.28]:64985) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VoqtV-0005h5-AJ for qemu-devel@nongnu.org; Fri, 06 Dec 2013 03:36:17 -0500 Message-ID: <52A18C77.9040700@redhat.com> Date: Fri, 06 Dec 2013 09:36:07 +0100 From: Paolo Bonzini MIME-Version: 1.0 References: <1386054500.25757.10.camel@nexus> <529D90A6.2080801@lab.ntt.co.jp> <52A0186A.2050207@lab.ntt.co.jp> <1386224104.3091.3.camel@nexus> <52A04732.4040105@redhat.com> <52A07C5A.9090105@lab.ntt.co.jp> <52A08541.6090702@redhat.com> <52A09EF4.5080800@lab.ntt.co.jp> <20131205161707.GB17277@amt.cnet> <52A0AC09.4090202@redhat.com> <52A189B2.4060305@lab.ntt.co.jp> In-Reply-To: <52A189B2.4060305@lab.ntt.co.jp> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH] target-i386: clear guest TSC on reset List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?ISO-8859-1?Q?Fernando_Luis_V=E1zquez_Cao?= Cc: Gleb Natapov , Will Auld , Marcelo Tosatti , qemu-devel@nongnu.org, kvm@vger.kernel.org Il 06/12/2013 09:24, Fernando Luis V=E1zquez Cao ha scritto: >=20 > Could we start with the patch that I already sent? It's been > tested, it is conservative in the sense that it does the minimum > necessary to fix an existing bug, and should be easy to > backport. I will be replying to this email with an updated > version that has a more appropriate and less scary patch > description. >=20 > I will also be sending a patch that makes the TSC writeback > unconditional, but this one should probably be kept on hold > until it is properly tested. If you test it, I can drop the "if" myself from your patch. Paolo