From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752136AbdLDVPW (ORCPT ); Mon, 4 Dec 2017 16:15:22 -0500 Received: from mail-wm0-f67.google.com ([74.125.82.67]:44129 "EHLO mail-wm0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750995AbdLDVPU (ORCPT ); Mon, 4 Dec 2017 16:15:20 -0500 X-Google-Smtp-Source: AGs4zMaSpPSKzDPwByx43mWScdIGkTyUbExummk5AyvLetn4y5EcglVit3w6/ngoAWJ4x5nQuZ+moA== Date: Tue, 5 Dec 2017 00:15:17 +0300 From: "Kirill A. Shutemov" To: Thomas Gleixner Cc: "Kirill A. Shutemov" , Ingo Molnar , x86@kernel.org, "H. Peter Anvin" , Linus Torvalds , Andy Lutomirski , Cyrill Gorcunov , Borislav Petkov , Andi Kleen , linux-mm@kvack.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org Subject: Re: [PATCHv3 1/5] x86/boot/compressed/64: Detect and handle 5-level paging at boot-time Message-ID: <20171204211517.nss6cn5llroehyts@node.shutemov.name> References: <20171204124059.63515-1-kirill.shutemov@linux.intel.com> <20171204124059.63515-2-kirill.shutemov@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20170609 (1.8.3) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Dec 04, 2017 at 09:29:45PM +0100, Thomas Gleixner wrote: > On Mon, 4 Dec 2017, Kirill A. Shutemov wrote: > > > This patch prepare decompression code to boot-time switching between 4- > > and 5-level paging. > > This is the very wrong reason for tagging this commit stable. > > > > > Signed-off-by: Kirill A. Shutemov > > Cc: [4.14+] > > Adding cc stable requires a Fixes tag as well. > > > +int l5_paging_required(void) > > +{ > > + /* Check i leaf 7 is supported. */ > > So you introduce the typo here and then you fix it in the next patch which > is the actual bug fix as an completely unrelated hunk. > > -- a/arch/x86/boot/compressed/pgtable_64.c > +++ b/arch/x86/boot/compressed/pgtable_64.c > @@ -2,7 +2,7 @@ > > int l5_paging_required(void) > { > - /* Check i leaf 7 is supported. */ > + /* Check if leaf 7 is supported. */ > > That's just careless and sloppy. > > I fixed it up once more along with the lousy changelogs because this crap, > which you not even thought about addressing it when shoving your 5-level > support into 4.14 needs to be fixed. > > I'm really tired of your sloppiness. You waste everyones time just by > ignoring feedback and continuing to do what you think is enough. Works for > me is _NOT_ enough for kernel development. Sorry. I screwed it up. I'll do my best to not waste your time again. > I'm not even looking at the rest of the series unless someone else has the > stomach to do so and sends a Reviewed-by. > > Alternatively you can sit down and look at the changelogs and the code and > figure out whether it matches what I told you over and over. Once you think > it does, then please feel free to resend it, but be sure that I'm going to > apply the most restrictive crap filter on anything which comes from you > from now on. Fair enough. I'll recheck everything in the morning and send them again. Thanks, and sorry again for wasting your time. -- Kirill A. Shutemov