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

RE: [Xen-devel] [PATCH] [retry 1] 1/3 Refactor Xen support for Intel Tur

To: "Langsdorf, Mark" <mark.langsdorf@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: RE: [Xen-devel] [PATCH] [retry 1] 1/3 Refactor Xen support for Intel Turbo boost
From: "Yu, Ke" <ke.yu@xxxxxxxxx>
Date: Thu, 1 Apr 2010 12:44:41 +0800
Accept-language: en-US
Acceptlanguage: en-US
Cc:
Delivery-date: Wed, 31 Mar 2010 21:47:54 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <DDE95BDD51FB774B804A9C029E735EE103E4119913@xxxxxxxxxxxxxxxxxxx>
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: <201003301925.01066.mark.langsdorf@xxxxxxx> <33AB447FBD802F4E932063B962385B351D6D6148@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <DDE95BDD51FB774B804A9C029E735EE103E4119913@xxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcrQaMOsR3gjR7/qQ4eHykA6uWJkSAAOjjdgABFlAWAAFgGxQA==
Thread-topic: [Xen-devel] [PATCH] [retry 1] 1/3 Refactor Xen support for Intel Turbo boost
> > I would like to change this as following, to keep hypercall
> > interface backward compatible.  And adding padding allows
> > future extension.
> >
> > diff -r 91232efdcfdc xen/include/public/sysctl.h
> > --- a/xen/include/public/sysctl.h       Tue Mar 30 08:36:41 2010 +0100
> > +++ b/xen/include/public/sysctl.h       Wed Mar 31 15:58:48 2010 +0800
> > @@ -333,7 +333,10 @@ struct xen_get_cpufreq_para {
> >      union {
> >          struct  xen_userspace userspace;
> >          struct  xen_ondemand ondemand;
> > +        uint8_t pad[32];
> >      } u;
> > +
> > +    int32_t turbo_enabled;
> >  };
> >
> > And you may want to change "struct xc_get_cpufreq_para" as well.
> 
> I don't think you need to pad it that way.  struct xen_sysctl
> already has a 128 byte pad defined for it.
> 
> Also, doing so triggers a build error macro in
> arch/x86/setup.c line 915.

Oh, the build error shows the length is reaching the limit of 128 bytes, in 
this case, padding is not necessary since there is few room for future 
extension. 

Best Regards
Ke


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

<Prev in Thread] Current Thread [Next in Thread>