On 30/06/2021 17:35, Philippe Mathieu-Daudé wrote:
> On 6/30/21 3:38 PM, Tom Lendacky wrote:
>> On 6/30/21 12:46 AM, Dov Murik wrote:
>>> Add assertion in pc_system_ovmf_table_find that verifies that the flash
>>> was indeed previously parsed (looking for the OVMF table) by
>>> pc_system_parse_ovmf_flash.
>>>
>>> Now pc_system_ovmf_table_find distinguishes between "no one called
>>> pc_system_parse_ovmf_flash" (which will abort due to assertion failure)
>>> and "the flash was parsed but no OVMF table was found, or it is invalid"
>>> (which will return false).
>>>
>>> Suggested-by: Philippe Mathieu-Daudé <phi...@redhat.com>
>>> Signed-off-by: Dov Murik <dovmu...@linux.ibm.com>
>>
>> Does the qemu coding style prefer not initializing the bool to false since
>> it will default to that?
>
> Indeed, you are right, and checkpatch will block this patch:
>
> ERROR: do not initialise statics to 0 or NULL
> #33: FILE: hw/i386/pc_sysfw.c:129:
> +static bool ovmf_flash_parsed = false;
>
> total: 1 errors, 0 warnings, 28 lines checked
oops, missed that in my flow.
Sent a v3 series with this fix.
>
>> Otherwise,
>>
>> Reviewed-by: Tom Lendacky <thomas.lenda...@amd.com>
Thanks, Tom!
-Dov
