> +static struct sched_domain_topology_level power9_topology[] = { > +#ifdef CONFIG_SCHED_SMT > + { cpu_smt_mask, powerpc_smt_flags, SD_INIT_NAME(SMT) }, > +#endif > + { shared_cache_mask, powerpc_shared_cache_flags, SD_INIT_NAME(CACHE) }, > + { cpu_cpu_mask, SD_INIT_NAME(DIE) }, > + { NULL, }, > +}; > +
Should we name this domain as MC instead of CACHE. Then it would be compatible with other archs like x86/arm/s390. MC stands for Multi-Core so it gels with what we have. Also we could enable CONFIG_SCHED_MC and reuse cpu_core_flags() instead of adding powerpc_shared_cache_flags.