On Tue, 2022-04-12 at 16:11 +0000, Dario Faggioli wrote:
> + else if ( is_hardware_domain(d) )
> + {
> + /*
> + * In absence of dom0_vcpus_pin, the hard and soft affinity
> of
> + * domain-0 is controlled by the dom0_nodes parameter. At
> this point
> + * it has been parsed and decoded, and we have the result of
> that
> + * in the dom0_cpus mask.
> + */
> + if ( !dom0_affinity_relaxed )
> + sched_set_affinity(unit, &dom0_cpus, &cpumask_all);
> + else
> + sched_set_affinity(unit, &cpumask_all, &dom0_cpus);
> + }
>
Oh, one thing: AFAICT, it's fine to use, e.g., dom0_affinity_relaxed
(that iis __initdata) here.But I'm never too sure whenever I touch stuff like that... Is it actually fine? If no, I think we can drop the annotation, or add non initdata- variables. Regards -- Dario Faggioli, Ph.D http://about.me/dario.faggioli Virtualization Software Engineer SUSE Labs, SUSE https://www.suse.com/ ------------------------------------------------------------------- <<This happens because _I_ choose it to happen!>> (Raistlin Majere)
signature.asc
Description: This is a digitally signed message part
