public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: "H. Peter Anvin" <hpa@linux.intel.com>
To: Linus Torvalds <torvalds@linux-foundation.org>,
	Andy Lutomirski <luto@amacapital.net>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Stefani Seibold <stefani@seibold.net>,
	the arch/x86 maintainers <x86@kernel.org>,
	Dave Jones <davej@redhat.com>,
	Martin Runge <Martin.Runge@rohde-schwarz.com>,
	Andreas Brief <Andreas.Brief@rohde-schwarz.com>
Subject: Re: [PATCH v2] x86: Remove compat vdso support
Date: Tue, 11 Mar 2014 09:14:19 -0700	[thread overview]
Message-ID: <531F365B.4060000@linux.intel.com> (raw)
In-Reply-To: <CA+55aFzB9jjOjxT=m=5t04L3JONe5HhEhuHUXtqDYREmEjQrgQ@mail.gmail.com>

On 03/11/2014 08:30 AM, Linus Torvalds wrote:
> On Tue, Mar 11, 2014 at 7:53 AM, Andy Lutomirski <luto@amacapital.net> wrote:
>>
>> I wonder if we can actually detect buggy glibc versions at runtime.
> 
> No, don't do that. That way lies madness.
> 
> What might be acceptable then is to just keep the old config name, and
> if the COMPAT_VDSO config is enabled, you just disable the non-compat
> vdso. At least that way, presumably any opensuse people would have
> their kernel config continue working.
> 
> Then if people have that enabled but didn't  need it, you can enable
> it at runtime with
> 
>     echo 1 > /proc/sys/abi/vsyscall32
> 
> which presumably would need to be exposed on 32-bit kernels too (it
> looks like a x86-64-only thing right now)
> 
> The important thing is that we do *not* break user space. Not ever.
> Not knowingly.
> 

As much as I wouldn't mind getting rid of the compat vdso, I really
don't understand why the trivial solution is being ruled out -- the
trivial solution being to just reserve a little more space in the fixmap
area.

I know Andy wants to move the vdso into a normal vma, which I certainly
support, but it is definitely the non-compat case.

	-hpa



  reply	other threads:[~2014-03-11 16:22 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-11  1:03 [PATCH v2] x86: Remove compat vdso support Andy Lutomirski
2014-03-11  1:39 ` Linus Torvalds
2014-03-11  2:37   ` Andy Lutomirski
2014-03-11  3:09     ` Linus Torvalds
2014-03-11  4:10       ` Andy Lutomirski
2014-03-11  8:37         ` Ingo Molnar
2014-03-11  9:36         ` Linus Torvalds
2014-03-11 14:53           ` Andy Lutomirski
2014-03-11 15:30             ` Linus Torvalds
2014-03-11 16:14               ` H. Peter Anvin [this message]
2014-03-11 16:30                 ` Linus Torvalds
2014-03-11 16:42                   ` Andy Lutomirski
2014-03-11 16:42                   ` H. Peter Anvin
2014-03-11 16:45                     ` Andy Lutomirski
2014-03-11 16:50                       ` Andy Lutomirski
2014-03-11 16:52                         ` H. Peter Anvin
2014-03-11 17:09                         ` Linus Torvalds
2014-03-11 17:14                           ` H. Peter Anvin
2014-03-11 17:16                           ` Andy Lutomirski
2014-03-12  8:30                           ` Stefani Seibold
2014-03-12 14:41                             ` Linus Torvalds
2014-03-12 15:46                               ` Linus Torvalds
2014-03-12 16:04                                 ` Linus Torvalds
2014-03-12 16:18                                   ` Brian Gerst
2014-03-12 16:18                                 ` Andy Lutomirski
2014-03-12 19:41                                 ` Linus Torvalds
2014-03-12 20:52                                   ` Andy Lutomirski
2014-03-12 21:37                                   ` H. Peter Anvin
2014-03-12 21:45                                     ` Andy Lutomirski
2014-03-12 21:46                                     ` Linus Torvalds
2014-03-12 21:49                                       ` Andy Lutomirski
2014-03-12 23:06                                         ` H. Peter Anvin
2014-03-12 23:43                                           ` Andy Lutomirski
2014-03-12 23:46                                             ` H. Peter Anvin
2014-03-13 16:23                                   ` Thomas Gleixner
2014-03-12 13:55                           ` One Thousand Gnomes
2014-03-13  7:08                           ` George Spelvin
2014-03-11 17:03                       ` H. Peter Anvin
2014-03-11 17:07                       ` Linus Torvalds

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=531F365B.4060000@linux.intel.com \
    --to=hpa@linux.intel.com \
    --cc=Andreas.Brief@rohde-schwarz.com \
    --cc=Martin.Runge@rohde-schwarz.com \
    --cc=davej@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=luto@amacapital.net \
    --cc=stefani@seibold.net \
    --cc=torvalds@linux-foundation.org \
    --cc=x86@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox