On Wed, Aug 28, 2013 at 07:48:23PM +0000, Christoph Lameter wrote: > 3. Retrieve the content of the current processors instance of a per cpu variable. > > DEFINE_PER_CPU(int, u); Shouldn't this be 'y' ? > int x = __get_cpu_var(y) > > Converts to > > int x = __this_cpu_read(y); > > > 4. Retrieve the content of a percpu struct > > DEFINE_PER_CPU(struct mystruct, y); > struct mystruct x = __get_cpu_var(y); > > Converts to > > memcpy(this_cpu_ptr(&x), y, sizeof(x)); Are you sure this one's correct? Isn't 'y' the per-cpu variable? Even though I don't see any in this patch, it's probably a good thing to get the patch description correct. I think you need Will Deacon's ack for this, but I think he's away for a while. -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
