Re: [PATCH] release quicklist before free_page

From: Daniel Walker
Date: Mon Jul 23 2007 - 14:38:46 EST


On Mon, 2007-07-23 at 20:23 +0200, Ingo Molnar wrote:
>
> how about: "if you've got some time then please also add a few comments,
> because the code was quite non-obvious to me and I misunderstood it when
> I tried to fix it. Thanks."

Not exactly what I was thinking of .. More like,

"LOCKING NOTE: This code uses the fact that get_cpu_var disables
preemption to maintain cpu affinity during the call to free_page()
otherwise it might free pages on another cpu."

(btw, your comment above comes off very snide, which I don't appreciate.
I haven't done anything that remotely warrants that.)

Daniel

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/