On Mon, Nov 02, 2015 at 10:10:23AM -0600, Christoph Lameter wrote: > On Sun, 1 Nov 2015, Jungseok Lee wrote: > > > There is no room to adjust 'atom_size' now when a generic percpu area > > is used. It would be redundant to write down an architecture-specific > > setup_per_cpu_areas() in order to only change the 'atom_size'. Thus, > > this patch adds a new definition, PERCPU_ATOM_SIZE, which is PAGE_SIZE > > by default. The value could be updated if needed by architecture. > > What is atom_size? Why would you want a difference allocation size here? > The percpu area is virtually mapped regardless. So you will have > contiguous addresses even without atom_size.
I haven't looked at the patch 3/3 in detail but I'm pretty sure I'll NAK the approach (and the definition of PERCPU_ATOM_SIZE), therefore rendering this patch unnecessary. IIUC, this is used to enforce some alignment of the per-CPU IRQ stack to be able to check whether the current stack is process or IRQ on exception entry. But there are other, less intrusive ways to achieve the same (e.g. x86). -- Catalin -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
