Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Thu, 6 Jun 2019 13:51:08 +0200
From:   Daniel Kiper <daniel.kiper@oracle.com>
To:     Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Cc:     linux-kernel@vger.kernel.org, x86@kernel.org,
        dpsmith@apertussolutions.com, eric.snowberg@oracle.com,
        hpa@zytor.com, kanth.ghatraju@oracle.com, ross.philipson@oracle.com
Subject: Re: [PATCH RFC 0/2] x86/boot: Introduce the setup_header2
Message-ID: <20190606115108.sfp2bnu3qzdby4h7@tomti.i.net-space.pl>
References: <20190524095504.12894-1-daniel.kiper@oracle.com>
 <20190605135031.62grhhxn2pfbkcdg@tomti.i.net-space.pl>
 <20190605140117.GA32106@char.us.oracle.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20190605140117.GA32106@char.us.oracle.com>
User-Agent: NeoMutt/20170113 (1.7.2)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Wed, Jun 05, 2019 at 10:01:17AM -0400, Konrad Rzeszutek Wilk wrote:
> On Wed, Jun 05, 2019 at 03:50:31PM +0200, Daniel Kiper wrote:
> > On Fri, May 24, 2019 at 11:55:02AM +0200, Daniel Kiper wrote:
> > > Hi,
> > >
> > > This change is needed to properly start the Linux kernel in Intel TXT mode and
> > > is a part of the TrenchBoot project (https://github.com/TrenchBoot).
>
> Can you please expand more on this?
>
> Nice explanation of why, other alternative solutions that didn't work, and so on.

OK.

> > > Daniel
> > >
> > >  Documentation/x86/boot.txt               | 55 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > >  arch/x86/Kconfig                         |  7 +++++++
> > >  arch/x86/boot/Makefile                   |  2 +-
> > >  arch/x86/boot/compressed/Makefile        |  5 +++--
> > >  arch/x86/boot/compressed/setup_header2.S | 18 ++++++++++++++++++
> > >  arch/x86/boot/compressed/sl_stub.S       | 28 ++++++++++++++++++++++++++++
> > >  arch/x86/boot/header.S                   |  3 ++-
> > >  arch/x86/boot/tools/build.c              |  8 ++++++++
> > >  arch/x86/include/uapi/asm/bootparam.h    |  1 +
> > >  9 files changed, 123 insertions(+), 4 deletions(-)
> > >
> > > Daniel Kiper (2):
> > >       x86/boot: Introduce the setup_header2
> > >       x86/boot: Introduce dummy MLE header
> >
> > Ping?
>
> Can you add Ingo and Thomas to the To: next time please?

OK.

> Also please drop the second patch.

Why? This is an example how to use the setup_header2.

Daniel