[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH] ppc/pnv: check for firmware file presence
From: |
Cédric Le Goater |
Subject: |
Re: [Qemu-ppc] [PATCH] ppc/pnv: check for firmware file presence |
Date: |
Mon, 25 Sep 2017 11:00:08 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 |
On 09/25/2017 10:52 AM, Greg Kurz wrote:
> On Mon, 25 Sep 2017 10:46:13 +0200
> Thomas Huth <address@hidden> wrote:
>
>> On 25.09.2017 10:33, Cédric Le Goater wrote:
>>> and exit before uslessly trying to load it if the file does
>>
>> s/uslessly/uselessly/
>>
>>> not exists.
>>>
>>> Issue dicovered by Coverity Scan.
>>
>> /dicovered/discovered/
>>
>
> Mentioning the Coverity Id (ie, CID 1381410) is also a usual practice.
ah. too late :/
C.
>>> Signed-off-by: Cédric Le Goater <address@hidden>
>>> ---
>>> hw/ppc/pnv.c | 4 ++++
>>> 1 file changed, 4 insertions(+)
>>>
>>> diff --git a/hw/ppc/pnv.c b/hw/ppc/pnv.c
>>> index 47221158d415..7728eece2eec 100644
>>> --- a/hw/ppc/pnv.c
>>> +++ b/hw/ppc/pnv.c
>>> @@ -570,6 +570,10 @@ static void ppc_powernv_init(MachineState *machine)
>>> }
>>>
>>> fw_filename = qemu_find_file(QEMU_FILE_TYPE_BIOS, bios_name);
>>> + if (!fw_filename) {
>>> + error_report("Could not find OPAL '%s'", bios_name);
>>
>> I'd suggest to say "OPAL firmware" instead of just "OPAL". Not every
>> user knows what OPAL is.
>>
>
> With all the suggested changes:
>
> Reviewed-by: Greg Kurz <address@hidden>
>
>>> + exit(1);
>>> + }
>>>
>>> fw_size = load_image_targphys(fw_filename, FW_LOAD_ADDR, FW_MAX_SIZE);
>>> if (fw_size < 0) {
>>>
>>
>> Thomas
>
>
>