linux-kernel - Re: [PATCH v2 0/2] x86_64: Sigcontext improvements

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 10 Mar 2015 07:05:21 -0700
From:	Andy Lutomirski <luto@...capital.net>
To:	Ingo Molnar <mingo@...nel.org>, Andi Kleen <andi@...stfloor.org>,
	"H. Peter Anvin" <hpa@...or.com>
Cc:	Al Viro <viro@...iv.linux.org.uk>, X86 ML <x86@...nel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	Andy Lutomirski <luto@...capital.net>,
	Oleg Nesterov <oleg@...hat.com>, Borislav Petkov <bp@...en8.de>
Subject: Re: [PATCH v2 0/2] x86_64: Sigcontext improvements

Sorry, forgot to CC Oleg and Borislav.  Let me know if you want me to
forward the rest of the thread to you.

On Tue, Mar 10, 2015 at 7:03 AM, Andy Lutomirski <luto@...capital.net> wrote:
> Patch 1 is IMO a bug fix.
>
> Patch 2 is a cleanup and avoids some confusion.  It's also sort of an
> API break (no ABI change) because it removes a struct field.  But that
> struct field has never done anything at all as far as I can tell.
>
> AFAICT this series got bogged down in discussion about patch 2 last
> time.  I'm only including patch 2 for completeness here -- patch 1
> is IMO far more useful, and I'm fine with patch 1 being merged and
> tabling patch 2 again.
>
> Changes from v1:
>  - Add the __USER_DS fix, caught by my sigreturn test.
>  - Re-tested locally.
>
> Andy Lutomirski (2):
>   x86_64,signal: Fix SS handling for signals delivered to 64-bit
>     programs
>   x86_64,signal: Remove 'fs' and 'gs' from sigcontext
>
>  arch/x86/include/asm/sigcontext.h      |  6 +++---
>  arch/x86/include/uapi/asm/sigcontext.h |  6 +++---
>  arch/x86/kernel/signal.c               | 24 +++++++++++++-----------
>  3 files changed, 19 insertions(+), 17 deletions(-)
>
> --
> 2.3.0
>



-- 
Andy Lutomirski
AMA Capital Management, LLC
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/