On Thu, Mar 22, 2018 at 04:30:29PM -0700, Luck, Tony wrote: > On Thu, Mar 22, 2018 at 01:49:22PM -0700, Alison Schofield wrote: > > + if (!topology_same_node(c, o) && > > + (c->x86_vendor == X86_VENDOR_INTEL && > > + c->x86_model == INTEL_FAM6_SKYLAKE_X)) { > > Maybe make life easier in the future to add more models > to the list by using x86_match_cpu() here? > > -Tony
Tony - Am I on the right track below? Define like this: static const __initconst struct x86_cpu_id snc_cpu[] = { { X86_VENDOR_INTEL, 6, INTEL_FAM6_SKYLAKE_X }, {} }; Use like this: if (!topology_same_node(c, o) && x86_match_cpu(snc_cpu)) { Thanks, alisons