From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joerg Roedel Subject: Re: [PATCH 3/3] Fix TSC MSR read in nested SVM Date: Sun, 7 Aug 2011 14:04:02 +0200 Message-ID: <20110807120402.GE30353@8bytes.org> References: <1312289591-nyh@il.ibm.com> <201108021255.p72CtNjD002153@rice.haifa.ibm.com> <20110803210052.GB31923@amt.cnet> <20110805153224.GB16953@amt.cnet> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Zachary Amsden , Joerg Roedel , Nadav Har'El , kvm@vger.kernel.org, Bandan Das , avi@redhat.com To: Marcelo Tosatti Return-path: Received: from 8bytes.org ([88.198.83.132]:41843 "EHLO 8bytes.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752076Ab1HGMEF (ORCPT ); Sun, 7 Aug 2011 08:04:05 -0400 Content-Disposition: inline In-Reply-To: <20110805153224.GB16953@amt.cnet> Sender: kvm-owner@vger.kernel.org List-ID: On Fri, Aug 05, 2011 at 12:32:24PM -0300, Marcelo Tosatti wrote: > > So to keep consistency between the remaining MSRs, i agree with Nadav's > > patch. Apparently SVM's original patches to return L1 TSC were aimed at > > fixing the problem VMX is facing now, which is fixed by introduction > > read_l1_tsc helpers. > > Joerg, can you review and ack Nadav's SVM patch? TIA Tested-by: Joerg Roedel Acked-by: Joerg Roedel Reviewed and tested (didn't apply cleanly, but was easy to fix that up). Looks all good so far. Regards, Joerg