[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <63635101-df2d-c698-6112-8db2e747a95c@intel.com>
Date: Thu, 10 Nov 2022 01:40:46 -0800
From: Sohil Mehta <sohil.mehta@...el.com>
To: "Joseph, Jithu" <jithu.joseph@...el.com>, <hdegoede@...hat.com>,
<markgross@...nel.org>
CC: <tglx@...utronix.de>, <mingo@...hat.com>, <bp@...en8.de>,
<dave.hansen@...ux.intel.com>, <x86@...nel.org>, <hpa@...or.com>,
<gregkh@...uxfoundation.org>, <ashok.raj@...el.com>,
<tony.luck@...el.com>, <linux-kernel@...r.kernel.org>,
<platform-driver-x86@...r.kernel.org>, <patches@...ts.linux.dev>,
<ravi.v.shankar@...el.com>, <thiago.macieira@...el.com>,
<athenas.jimenez.gonzalez@...el.com>
Subject: Re: [PATCH v2 10/14] platform/x86/intel/ifs: Add metadata validation
On 11/9/2022 5:22 PM, Joseph, Jithu wrote:
> Here is the layout of the metadata section in an IFS test image.
> Test Data (chunks) follow the struct meta_data defined in load.c
>
Thanks, this helps clarify things around the test data placement in this
patch. I think it would be useful to include the below information in
the commit message as well.
Given that,
Reviewed-by: Sohil Mehta <sohil.mehta@...el.com>
> IFS Metadata layout
> +----------------------+ 0
> |META_TYPE_IFS (=1) |
> +----------------------+
> |meta_size |
> +----------------------+
> |test type |
> +----------------------+
> |fusa info |
> +----------------------+
> |total images |
> +----------------------+
> |current image# |
> +----------------------+
> |total chunks |
> +----------------------+
> |starting chunk |
> +----------------------+
> |size per chunk |
> +----------------------+
> |chunks per stride |
> +----------------------+
> |Reserved[54] |
> +----------------------+ 256
> | |
> | |
> | |
> | |
> |Test Data/Chunks |
> | |
> | |
> | |
> | |
> +----------------------+ meta_size
> | META_TYPE_END (=0) |
> +----------------------+ meta_size + 4
> | size of end (=8) |
> | |
> +----------------------+ meta_size + 8
>
>
> Jithu
Powered by blists - more mailing lists