Messages in this thread | | | From | Andy Lutomirski <> | Date | Tue, 24 Mar 2015 12:46:42 -0700 | Subject | Re: [tip:x86/asm] x86/asm/entry: Add user_mode_ignore_vm86() |
| |
On Tue, Mar 24, 2015 at 12:44 PM, Ingo Molnar <mingo@kernel.org> wrote: > > * Andy Lutomirski <luto@amacapital.net> wrote: > >> On Mon, Mar 23, 2015 at 5:26 AM, tip-bot for Andy Lutomirski >> <tipbot@zytor.com> wrote: >> > Commit-ID: a67e7277d01ccfd39b0db5a198c2643cc19dd79c >> > Gitweb: http://git.kernel.org/tip/a67e7277d01ccfd39b0db5a198c2643cc19dd79c >> > Author: Andy Lutomirski <luto@kernel.org> >> > AuthorDate: Wed, 18 Mar 2015 18:33:29 -0700 >> > Committer: Ingo Molnar <mingo@kernel.org> >> > CommitDate: Mon, 23 Mar 2015 11:13:36 +0100 >> > >> > x86/asm/entry: Add user_mode_ignore_vm86() >> > >> > user_mode() is dangerous and user_mode_vm() has a confusing name. >> > >> > Add user_mode_ignore_vm86() (equivalent to current user_mode()). >> > We'll change the small number of legitimate users of user_mode() >> > to user_mode_ignore_vm86(). >> > >> > Inspired by grsec, although this works rather differently. >> >> Ingo, does this mean that you changed your mind or do you still want >> a patch to delete user_mode_ignore_vm86 and just use user_mode >> everywhere instead? > > Would be still nice to have it as an add on patch, if you agree with > my arguments.
Given that there are only a very small number of callers left and they're all Obviously Correct (tm), I'm not too worried about it. Maybe if we kill off __copy_to_user, I'll be inspired to kill off user_mode_ignore_vm86 as well :)
--Andy
> > I picked up your series as-is because it's correct and because it > already improves things a lot in this area. > > Thanks, > > Ingo
-- Andy Lutomirski AMA Capital Management, LLC
| |