Re: [PATCH] x86/vdso: Add prctl to set per-process VDSO load

From: Richard Larocque
Date: Tue Sep 16 2014 - 20:21:33 EST


On Tue, Sep 16, 2014 at 5:13 PM, Andi Kleen <andi@xxxxxxxxxxxxxx> wrote:
> Richard Larocque <rlarocque@xxxxxxxxxx> writes:
>
> Perhaps I'm missing something, but how do you modify the AUX vector
> for the children?
>
>> +config VDSO_DISABLE_PRCTL
>> + depends on X86
>> + bool "prctl to disable VDSO loading"
>> + ---help---
>> + Enabling this option adds support for prctl calls that
>> + set and retrieve a per-process flag to disable VDSO loading on
>> + exec() for this process and all of its children.
>
> I don't think it makes any sense to have a config for a simple
> feature like this. Just do it unconditionally.
>
> -Andi
>
> --
> ak@xxxxxxxxxxxxxxx -- Speaking for myself only
>
> --
> 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/

The AUX vector issue is a mistake on my part.

Speaking of mistakes, the IDs in the commit message do not match those
in the patch itself. That needs to be fixed, too.

Point taken about the config. I'll remove it in the next version of the patch.

Thanks for the feedback.
--
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/