On Wed, 7 Jan 2009, Linus Torvalds wrote:
> On Wed, 7 Jan 2009, Dave Kleikamp wrote:
> > 
> > Do you need to even do that if CONFIG_DEBUG_PAGEALLOC is unset?
> 
> No.
> 
> > Something like:
> > 
> > #ifdef CONFIG_DEBUG_PAGEALLOC
> >                 /*
> >                  * Need to access the cpu field knowing that
> >                  * DEBUG_PAGEALLOC could have unmapped it if
> >                  * the mutex owner just released it and exited.
> >                  */
> >                 if (probe_kernel_address(&thread->cpu, cpu))
> >                         break;
> > #else
> >             cpu = thread->cpu;
> > #endif
> 
> yes. That would work fine.

What about memory hotplug as Ingo mentioned?

Should that be:

#if defined(CONFIG_DEBUG_PAGEALLOC) || defined(CONFIG_MEMORY_HOTPLUG)

??

-- Steve

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to