lkml.org 
[lkml]   [2014]   [Sep]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] x86/vdso: Add prctl to set per-process VDSO load
From
On Tue, Sep 16, 2014 at 5:13 PM, Andi Kleen <andi@firstfloor.org> wrote:
> Richard Larocque <rlarocque@google.com> 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@linux.intel.com -- Speaking for myself only
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> 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.


\
 
 \ /
  Last update: 2014-09-17 03:01    [W:0.064 / U:1.132 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site