On 17-03-10 01:56:04, Jan Beulich wrote:
> >>> On 10.03.17 at 02:32, wrote:
> > On 17-03-08 07:56:54, Jan Beulich wrote:
> >> >>> On 15.02.17 at 09:49, wrote:
[...]
> >> > +if ( !cpu_has(¤t_cpu_data, X86_FEATURE_PQE) )
> >>
> >> Do you really mean to not universally check the global (boot C
>>> On 10.03.17 at 02:32, wrote:
> On 17-03-08 07:56:54, Jan Beulich wrote:
>> >>> On 15.02.17 at 09:49, wrote:
>> > -static int psr_cpu_prepare(unsigned int cpu)
>> > +static void cpu_init_work(void)
>> > +{
>> > +struct psr_socket_info *info;
>> > +unsigned int socket;
>> > +unsigne
On 17-03-08 07:56:54, Jan Beulich wrote:
> >>> On 15.02.17 at 09:49, wrote:
[...]
> > -static int psr_cpu_prepare(unsigned int cpu)
> > +static void cpu_init_work(void)
> > +{
> > +struct psr_socket_info *info;
> > +unsigned int socket;
> > +unsigned int cpu = smp_processor_id();
> >
>>> On 15.02.17 at 09:49, wrote:
> This patch implements the CPU init and free flow including L3 CAT
> initialization and feature list free.
I think from the final-effect-of-the-patch point of view the L3 CAT
aspect is more relevant, so that's what should appear in the title.
> @@ -136,11 +140,8
On Mon, Feb 27, 2017 at 02:42:31PM +0800, Yi Sun wrote:
> On 17-02-26 17:41:08, Wei Liu wrote:
> > On Wed, Feb 15, 2017 at 04:49:19PM +0800, Yi Sun wrote:
> > > This patch implements the CPU init and free flow including L3 CAT
> > > initialization and feature list free.
> > >
> > > Signed-off-by:
>>> Wei Liu 02/26/17 6:41 PM >>>
>On Wed, Feb 15, 2017 at 04:49:19PM +0800, Yi Sun wrote:
>> +static void free_feature(struct psr_socket_info *info)
>> +{
>> +struct feat_node *feat, *next;
>> +
>> +if ( !info )
>> +return;
>> +
>> +/*
>> + * Free resources of features. But
On 17-02-26 17:41:08, Wei Liu wrote:
> On Wed, Feb 15, 2017 at 04:49:19PM +0800, Yi Sun wrote:
> > This patch implements the CPU init and free flow including L3 CAT
> > initialization and feature list free.
> >
> > Signed-off-by: Yi Sun
>
> Either you need to use a separate patch to move cpuid_c
On Wed, Feb 15, 2017 at 04:49:19PM +0800, Yi Sun wrote:
> This patch implements the CPU init and free flow including L3 CAT
> initialization and feature list free.
>
> Signed-off-by: Yi Sun
Either you need to use a separate patch to move cpuid_count_leaf or you
should state it is moved in the co
This patch implements the CPU init and free flow including L3 CAT
initialization and feature list free.
Signed-off-by: Yi Sun
---
v8:
- remove unused parameter of psr_cpu_prepare.
- add comments in cpu_fini_work.
- coding style fix.
---
xen/arch/x86/cpuid.c| 6 --
xen/a
