Re: [patch] paravirt: VDSO page is essential

Ingo Molnar
2007-03-05T13:47:08Z

* Rusty Russell <rusty@rustcorp.com.au> wrote:

> -#ifdef CONFIG_PARAVIRT
> +#if defined(CONFIG_COMPAT_VDSO) && defined(CONFIG_RESERVE_TOP)

NACK - my patch is quite a bit simpler and yours only increases the
#ifdef jungle. If there's any complication of the VDSO coming from some
other hypervisor support patch then I will judge that in full context,
when it's submitted. Meanwhile, my patch is a must-have for v2.6.21.

Ingo