Re: [RFC PATCH v4 16/36] i386/tdvf: Introduce function to parse TDVF metadata

[Xiaoyao Li <xiaoyao.li@xxxxxxxxx>]

On 5/24/2022 3:02 PM, Gerd Hoffmann wrote:
>    Hi,
>
> > +static int tdvf_parse_section_entry(const TdvfSectionEntry *src,
> > +                                     TdxFirmwareEntry *entry)
>
> > +    /* sanity check */
>
> That is what the whole function is doing.  So rename it to
> tdvf_check_section_entry to clarify that?

I will rename it to tdvf_parse_and_check_section_entry() since it first 
parses the section entries from TDVF to software defined data structure 
TdxFirmwareEntry

> take care,
>    Gerd