On 9/5/24 07:24, Jarkko Sakkinen wrote:
>> + for_each_online_node(nid) {
>> + if (!node_isset(nid, sgx_numa_mask) &&
>> + node_state(nid, N_MEMORY) && node_state(nid, N_CPU))
>> + pr_info("node%d has both CPUs and memory but doesn't have an EPC section\n",
>> + nid);
> Is this enough, or is there anything that would need to be done
> automatically if this happens? With a tracepoint you could react to such
> even but I'm totally fine with this.
There's always the theoretical chance that there are nodes being
hotplugged or that this is all running in a VM that has some whacky
topology.
But this simple pr_info() provides good coverage for the common cases
and shouldn't be too much of a burden for the weridos.
