Re: [PATCH-next v5 0/7] x86/boot: Remove run-time relocations from compressed kernel

From: Kees Cook
Date: Wed Jul 29 2020 - 22:38:41 EST


On Wed, Jul 29, 2020 at 06:23:41PM -0400, Arvind Sankar wrote:
> On Wed, Jul 29, 2020 at 03:04:43PM -0700, Kees Cook wrote:
> > On Fri, Jul 17, 2020 at 04:17:54PM -0400, Arvind Sankar wrote:
> > > Same as v5 previously posted, but rebased onto next-20200717.
> > >
> > > v5: https://lore.kernel.org/lkml/20200715004133.1430068-1-nivedita@xxxxxxxxxxxx/
> >
> > BTW, some bits of feedback on process:
> >
> > - please don't re-use version numbers, this is confusing :)
>
> This was a special case as there were no actual changes in this version.

It ended up missing some review tags, etc.

> > - please fix whatever is happening with the "In-Reply-To:" headers, the
> > threading is extremely hard to understand (each patch isn't a reply to
> > the cover letter, and everything is a reply to a single earlier email)
>
> I think this is because I'm passing --in-reply-to to format-patch.
> Seems like I should specify that in send-email instead, hopefully that
> will fix the threading.

Ah yes, I think would do it -- it overrides the normal threading that
send-email does. Usually --in-reply-to is for a single email.

Thanks for checking!

--
Kees Cook