Re: [Xen-devel] [PATCH 4/8] x86/AMD: distinguish compute units from hyper-threads

2018-07-12 Thread Jan Beulich
>>> On 12.07.18 at 15:02, wrote: > On 11/07/18 13:07, Jan Beulich wrote: >> Fam17 replaces CUs by HTs, which we should reflect accordingly, even if >> the difference is not very big. The most relevant change (requiring some >> code restructuring) is that the topoext feature no longer means there

Re: [Xen-devel] [PATCH 4/8] x86/AMD: distinguish compute units from hyper-threads

2018-07-12 Thread Andrew Cooper
On 11/07/18 13:07, Jan Beulich wrote: > Fam17 replaces CUs by HTs, which we should reflect accordingly, even if > the difference is not very big. The most relevant change (requiring some > code restructuring) is that the topoext feature no longer means there is > a valid CU ID. > > Take the

Re: [Xen-devel] [PATCH 4/8] x86/AMD: distinguish compute units from hyper-threads

2018-07-11 Thread Brian Woods
On Wed, Jul 11, 2018 at 06:07:42AM -0600, Jan Beulich wrote: > Fam17 replaces CUs by HTs, which we should reflect accordingly, even if > the difference is not very big. The most relevant change (requiring some > code restructuring) is that the topoext feature no longer means there is > a valid CU

[Xen-devel] [PATCH 4/8] x86/AMD: distinguish compute units from hyper-threads

2018-07-11 Thread Jan Beulich
Fam17 replaces CUs by HTs, which we should reflect accordingly, even if the difference is not very big. The most relevant change (requiring some code restructuring) is that the topoext feature no longer means there is a valid CU ID. Take the opportunity and convert wrongly plain int variables in