From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Liu, Jinsong" Subject: Re: Xen 4.2 TODO / Release Plan Date: Mon, 6 Aug 2012 17:06:01 +0000 Message-ID: References: <1343988573.21372.45.camel@zakaz.uk.xensource.com> <501BC799020000780009278B@nat28.tlf.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <501BC799020000780009278B@nat28.tlf.novell.com> Content-Language: en-US List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Jan Beulich , Ian Campbell , Keir Fraser Cc: xen-devel List-Id: xen-devel@lists.xenproject.org Jan Beulich wrote: >>>> On 03.08.12 at 12:28, Keir Fraser wrote: >> On 03/08/2012 11:09, "Ian Campbell" wrote: >> >>> On Mon, 2012-07-30 at 09:30 +0100, Ian Campbell wrote: >>>> * vMCE save/restore changes, to simplify migration 4.2->4.3 >>>> with new vMCE in 4.3. (Jinsong Liu, Jan Beulich) >>> >>> Where are we with this? >>> >>> Is it still a viable candidate for 4.2, now that we have reached rc1 >>> (almost 2)? >> >> Didn't we already take the trivial patch that will ease the >> transition to >> 4.3? > > We took one necessary patch, but I think at least the second > one of the recently posted series would also be needed. And > the really important patch for migration forward compatibility > was patch 5 in that series, yet I wouldn't want to take patches > 3 and 4 for 4.2. > > In any case, the series is in need of resubmission anyway. > Perhaps (if that's possible, I didn't check in too much detail) > reordering patch 5 could be done at once. > Patch 2 has been updated according to Jan's comments. As for patch 5, it cannot be reordered w/o patch 3 checked in (patch 5 is for save/restore MCi_CTL2, a newly added MSR at patch 3). In fact we could remove patch 5 totally, and don't add MCi_CTL2 (this MSR is nothing to do with vmce logic itself, the only reason why we add it in new vmce is to get perfromance benefit (but very trivial), so it's OK not to add it and remove patch 5). Another benefit of not add MCi_CTL2 is, to avoid difference between Intel and AMD code. Hence I think it's an acceptable approach to keep current vmce (not implement MCi_CTL2). Your opinion? Thanks, Jinsong