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 kirill.shutemov@linux.intel.com Cc: stable@vger.kernel.org [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.