On 11/22/2016 06:19 AM, Anshuman Khandual wrote: > @@ -393,6 +393,9 @@ enum node_states { > N_MEMORY = N_HIGH_MEMORY, > #endif > N_CPU, /* The node has one or more cpus */ > +#ifdef CONFIG_COHERENT_DEVICE > + N_COHERENT_DEVICE, > +#endif > NR_NODE_STATES > };
Don't we really want this to be N_MEMORY_ISOLATED? Or, better yet, N_MEMORY_UNISOLATED so that we can just drop the bitmap in for N_MEMORY and not have to do any bit manipulation operations at runtime.