WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

[Xen-devel] RE: [PATCH] CPUID level 0x00000007:0 (ebx) is word 9, instea

To: Keir Fraser <keir.xen@xxxxxxxxx>, Jan Beulich <JBeulich@xxxxxxxxxx>
Subject: [Xen-devel] RE: [PATCH] CPUID level 0x00000007:0 (ebx) is word 9, instead of word 7
From: "Li, Xin" <xin.li@xxxxxxxxx>
Date: Wed, 1 Jun 2011 22:41:07 +0800
Accept-language: zh-CN, en-US
Acceptlanguage: zh-CN, en-US
Cc: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Wed, 01 Jun 2011 07:42:52 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <CA0B9F0C.1B5B6%keir.xen@xxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <4DE5F9ED0200007800044BCD@xxxxxxxxxxxxxxxxxx> <CA0B9F0C.1B5B6%keir.xen@xxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcwgJ8NXuIq+Qkwdekihnyl8xMNeHgAN3W6A
Thread-topic: [PATCH] CPUID level 0x00000007:0 (ebx) is word 9, instead of word 7
> >>>> On 31.05.11 at 16:59, "Li, Xin" <xin.li@xxxxxxxxx> wrote:
> >>> can't see why we would need to stay in sync with Linux's capability
> >>> array indices.
> >>
> >> why?  Typically we reuse Linux code unless Xen has its special logic.
> >
> > So you would suggest leaving indices 7 and 8 unused instead?
> > Looking at current Linux, we certainly could convert Xen to use
> > index 8 for virtualization features, but since these are being
> > tracked differently already anyway I don't see a value in this.
> >
> > As to index 7, just look at ARAT - we're already diverging from
> > Linux here (having it allocated in index 3).
> >
> > Bottom line is that I think keeping the names (and in various cases
> > the grouping together, namely when the bits are grouped together
> > in some CPUID leaf's output) in sync is desirable, but following
> > Linux to the bit doesn't always make sense. After all, some thinking
> > will always be necessary when porting over patches.
> 
> Agreed.

Okay, I raised this because I saw 2 bugs coming from the differences
between Xen and Linux.  Anyway let's keep in mind that we should 
refer to Linux design and code to save some effort and reduce bugs.
Thanks!
-Xin

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

<Prev in Thread] Current Thread [Next in Thread>
  • [Xen-devel] RE: [PATCH] CPUID level 0x00000007:0 (ebx) is word 9, instead of word 7, Li, Xin <=