On Thu, Apr 18, 2019 at 02:46:02PM +0800, Tao Xu wrote: > The aim of this patch is to move existing numa global numa_info > into NumaState. > > Suggested-by: Igor Mammedov <[email protected]> > Suggested-by: Eduardo Habkost <[email protected]> > Signed-off-by: Tao Xu <[email protected]>
`ms->numa_state->numa_info[i]` is not a very descriptive expression. Also, we don't need the "numa" prefix anymore, if the identifier is not in global scope. I suggest renaming it to NumaState::nodes. So this: > ms->numa_state->numa_info[i].distance[j] would become: ms->numa_state->nodes[i].distance[j] This wouldn't block me from merging this patch, but it's a suggestion in case you send v3 to address the comments on patch 1/3. -- Eduardo
