Re: [Xen-devel] [PATCH v12 13/23] x86: refactor psr: CDP: implement CPU init flow.

2017-07-04 Thread Yi Sun
On 17-07-04 01:28:32, Jan Beulich wrote: > >>> On 04.07.17 at 03:40, wrote: > > On 17-06-30 03:18:53, Jan Beulich wrote: > >> >>> Yi Sun 06/30/17 10:05 AM >>> > >> >On 17-06-30 01:33:02, Jan Beulich wrote: > >> >> >>> Yi Sun 06/30/17 9:01 AM >>> > > There will be problem if we substract 1 before

Re: [Xen-devel] [PATCH v12 13/23] x86: refactor psr: CDP: implement CPU init flow.

2017-07-04 Thread Jan Beulich
>>> On 04.07.17 at 03:40, wrote: > On 17-06-30 03:18:53, Jan Beulich wrote: >> >>> Yi Sun 06/30/17 10:05 AM >>> >> >On 17-06-30 01:33:02, Jan Beulich wrote: >> >> >>> Yi Sun 06/30/17 9:01 AM >>> >> >> >This accords to spec: >> >> >"For CDP operations, COS_MAX_CDP is equal to (CPUID.(EAX=10H, >>

Re: [Xen-devel] [PATCH v12 13/23] x86: refactor psr: CDP: implement CPU init flow.

2017-07-03 Thread Yi Sun
On 17-06-30 03:18:53, Jan Beulich wrote: > >>> Yi Sun 06/30/17 10:05 AM >>> > >On 17-06-30 01:33:02, Jan Beulich wrote: > >> >>> Yi Sun 06/30/17 9:01 AM >>> > >> >This accords to spec: > >> >"For CDP operations, COS_MAX_CDP is equal to (CPUID.(EAX=10H, > >> >ECX=1):EDX.COS_MAX_CAT >>1)." > >> >

Re: [Xen-devel] [PATCH v12 13/23] x86: refactor psr: CDP: implement CPU init flow.

2017-06-30 Thread Jan Beulich
>>> Yi Sun 06/30/17 10:05 AM >>> >On 17-06-30 01:33:02, Jan Beulich wrote: >> >>> Yi Sun 06/30/17 9:01 AM >>> >> >This accords to spec: >> >"For CDP operations, COS_MAX_CDP is equal to (CPUID.(EAX=10H, >> >ECX=1):EDX.COS_MAX_CAT >>1)." >> > >> >HW should make sure it is even number. >> >> And h

Re: [Xen-devel] [PATCH v12 13/23] x86: refactor psr: CDP: implement CPU init flow.

2017-06-30 Thread Yi Sun
On 17-06-30 01:33:02, Jan Beulich wrote: > >>> Yi Sun 06/30/17 9:01 AM >>> > >On 17-06-30 00:40:35, Jan Beulich wrote: > >> >>> Yi Sun 06/14/17 3:26 AM >>> > >> > @@ -253,6 +271,26 @@ static void cat_init_feature(const struct > >> > cpuid_leaf *regs, > >> > > >> > break; > >> > > >>

Re: [Xen-devel] [PATCH v12 13/23] x86: refactor psr: CDP: implement CPU init flow.

2017-06-30 Thread Jan Beulich
>>> Yi Sun 06/30/17 9:01 AM >>> >On 17-06-30 00:40:35, Jan Beulich wrote: >> >>> Yi Sun 06/14/17 3:26 AM >>> >> > @@ -253,6 +271,26 @@ static void cat_init_feature(const struct cpuid_leaf >> > *regs, >> > >> > break; >> > >> > +case PSR_SOCKET_L3_CDP: >> > +{ >> > +u

Re: [Xen-devel] [PATCH v12 13/23] x86: refactor psr: CDP: implement CPU init flow.

2017-06-30 Thread Yi Sun
On 17-06-30 00:40:35, Jan Beulich wrote: > >>> Yi Sun 06/14/17 3:26 AM >>> > > @@ -253,6 +271,26 @@ static void cat_init_feature(const struct cpuid_leaf > > *regs, > > > > break; > > > > +case PSR_SOCKET_L3_CDP: > > +{ > > +uint64_t val; > > + > > +/* Cut half

Re: [Xen-devel] [PATCH v12 13/23] x86: refactor psr: CDP: implement CPU init flow.

2017-06-29 Thread Jan Beulich
>>> Yi Sun 06/14/17 3:26 AM >>> > @@ -253,6 +271,26 @@ static void cat_init_feature(const struct cpuid_leaf > *regs, > > break; > > +case PSR_SOCKET_L3_CDP: > +{ > +uint64_t val; > + > +/* Cut half of cos_max when CDP is enabled. */ > +feat->cos_max >>

[Xen-devel] [PATCH v12 13/23] x86: refactor psr: CDP: implement CPU init flow.

2017-06-13 Thread Yi Sun
This patch implements the CPU init flow for CDP. The flow is almost same as L3 CAT. Signed-off-by: Yi Sun --- v12: - move 'type[]' assignment into l3_cdp_props declaration to make it be 'const'. (suggested by Jan Beulich) - remove "L2 CAT" indication in printk. (suggeste