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@xxxxxxxxxxxxxxx>
> > Cc: <stable@xxxxxxxxxxxxxxx> [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
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxx. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>
