Re: [PATCH 01 of 11] mmu-notifier-core

From: Linus Torvalds
Date: Wed May 07 2008 - 16:32:09 EST




On Wed, 7 May 2008, Andrew Morton wrote:
>
> The patch looks OK to me.

As far as I can tell, authorship has been destroyed by at least two of the
patches (ie Christoph seems to be the author, but Andrea seems to have
dropped that fact).

> The proposal is that we sneak this into 2.6.26. Are there any
> sufficiently-serious objections to this?

Yeah, too late and no upside.

That "locking" code is also too ugly to live, at least without some
serious arguments for why it has to be done that way. Sorting the locks?
In a vmalloc'ed area? And calling this something innocuous like
"mm_lock()"? Hell no.

That code needs some serious re-thinking.

Linus
--
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/