Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754504AbbG0QFN (ORCPT ); Mon, 27 Jul 2015 12:05:13 -0400 Received: from mail-wi0-f172.google.com ([209.85.212.172]:33075 "EHLO mail-wi0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753279AbbG0QFL (ORCPT ); Mon, 27 Jul 2015 12:05:11 -0400 Date: Mon, 27 Jul 2015 18:05:05 +0200 From: Ingo Molnar To: Denys Vlasenko Cc: Linus Torvalds , "Krzysztof A. Sobiecki" , Steven Rostedt , Borislav Petkov , "H. Peter Anvin" , Andy Lutomirski , Oleg Nesterov , Frederic Weisbecker , Alexei Starovoitov , Will Drewry , Kees Cook , x86@kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/3] x86/asm/entry/32: Remove most of SYSCALL32 code, part 1 Message-ID: <20150727160505.GA25158@gmail.com> References: <1437745668-31802-1-git-send-email-dvlasenk@redhat.com> <1437745668-31802-2-git-send-email-dvlasenk@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1437745668-31802-2-git-send-email-dvlasenk@redhat.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2480 Lines: 66 * Denys Vlasenko wrote: > SYSCALL32 code is nearly identical to SYSCALL32, except for initial > section. Merge them. > > The removal is split into two parts, to make review eaiser. This is part 1. > > auditsys_entry_common and auditsys_exit macros are indented one more tab without > any changes. This prevents diff from becoming unreadable. > They will be removed in part 2. > > Signed-off-by: Denys Vlasenko > CC: Ingo Molnar > CC: Linus Torvalds > CC: Krzysztof A. Sobiecki > CC: Steven Rostedt > CC: Borislav Petkov > CC: "H. Peter Anvin" > CC: Andy Lutomirski > CC: Oleg Nesterov > CC: Frederic Weisbecker > CC: Alexei Starovoitov > CC: Will Drewry > CC: Kees Cook > CC: x86@kernel.org > CC: linux-kernel@vger.kernel.org > --- > arch/x86/entry/entry_64_compat.S | 237 +++++++++++---------------------------- > 1 file changed, 63 insertions(+), 174 deletions(-) Yeah, so I realize that this is already a 'cleaner', split up version of your original change - but the diffstat is still way too large: _please_ split it up into 2-3 further steps to make each step really, utterly obvious. As you must have experienced it with recent regressions in the x86 entry code, the smaller our assembly changes, the easier our job of doing a revert is, when such a change regresses ... I don't care if it ends up being 5-7 patches - that's far more preferable to having to decode difficult looking regressions. For example in hindsight I regret that I did not insist on further splitting up this commit: 53e9accf0f76 ("x86/asm/entry/32: Do not use R9 in SYSCALL32 entry point") and that was a small patch already: arch/x86/entry/ia32entry.S | 19 ++++++++----------- 1 file changed, 8 insertions(+), 11 deletions(-) I'm not going to make that mistake again: if you want to change the x86 assembly code, you need to learn to decompose dangerous changes into maximally split up, atomic, bisectable steps. Agreed? Thanks, Ingo -- 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/