From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754060AbaJAStK (ORCPT ); Wed, 1 Oct 2014 14:49:10 -0400 Received: from mail-pd0-f174.google.com ([209.85.192.174]:35326 "EHLO mail-pd0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753878AbaJAStJ (ORCPT ); Wed, 1 Oct 2014 14:49:09 -0400 From: Andy Lutomirski To: Thomas Gleixner , X86 ML , Ingo Molnar , "H. Peter Anvin" Cc: Sebastian Lackner , Anish Bhatt , "linux-kernel@vger.kernel.org" , Chuck Ebbert , Andy Lutomirski Subject: [PATCH v4 0/2] x86_64,entry: Clear NT on entry and speed up switch_to Date: Wed, 1 Oct 2014 11:49:03 -0700 Message-Id: X-Mailer: git-send-email 1.9.3 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Anish Bhatt noticed that user programs can set RFLAGS.NT before syscall or sysenter, and the kernel entry code doesn't filter out NT. This causes kernel C code and, depending on thread flags, the exit slow path to run with NT set. The former is a little bit scary (imagine calling into EFI with NT set), and the latter will fail with #GP and send a spurious SIGSEGV. One answer would be "don't do that". But the kernel can do better here. These patches filter NT on all kernel entries. For syscall (both bitnesses), this is free. For sysenter, it seems to cost very little (less than my ability to measure, although I didn't try that hard). Patch 2, which isn't tagged for -stable, speeds up context switches by avoiding saving and restoring flags, so this series should be a decent overall performance win. See: https://bugs.winehq.org/show_bug.cgi?id=33275 Note to bikeshedders: I have no desire to go crazy micro-optimizing the sysenter path. :) This version seems to be good enough (and should be a performance *increase* for most workloads). Changes from v3: - Added a better description of the impact in patch 1 Changes from v2: - Move the flag fixup out of line - Fix a CFI buglet Changes from v1: - Spell stable@vger.kernel.org correctly - Tidy up changelog text - Actually commit an asm constraint fix in patch 2 (egads!) - Replace the unconditional popfq with a branch Andy Lutomirski (2): x86_64,entry: Filter RFLAGS.NT on entry from userspace x86_64: Don't save flags on context switch arch/x86/ia32/ia32entry.S | 18 +++++++++++++++++- arch/x86/include/asm/switch_to.h | 12 ++++++++---- arch/x86/kernel/cpu/common.c | 2 +- 3 files changed, 26 insertions(+), 6 deletions(-) -- 1.9.3