On Tue, Aug 01 2023 at 12:35, Gautham R. Shenoy wrote:
> On Fri, Jul 28, 2023 at 02:13:08PM +0200, Thomas Gleixner wrote:
>> +static void topo_set_ids(struct topo_scan *tscan)
>> +{
>> + struct cpuinfo_x86 *c = tscan->c;
>> + u32 apicid = c->topo.apicid;
>> +
>> + c->topo.pkg_id = topo_shift_apicid(apicid, TOPO_ROOT_DOMAIN);
>
> Shouldn't this use TOPO_PKG_DOMAIN instead of TOPO_ROOT_DOMAIN ?
Yup. It does not make a difference in that case. That's why I didn't
notice, but let me fix this for conistency sake.
