[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] ppc/pnv: Silence missing BMC warning with qtest
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH] ppc/pnv: Silence missing BMC warning with qtest |
Date: |
Mon, 22 Jun 2020 10:24:17 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.5.0 |
On 6/22/20 9:53 AM, Greg Kurz wrote:
> On Mon, 22 Jun 2020 09:13:46 +0200
> Cédric Le Goater <clg@kaod.org> wrote:
>
>> On 6/22/20 8:57 AM, Greg Kurz wrote:
>>> The device introspect test in qtest emits some warnings with the
>>> the pnv machine types during the "nodefaults" phase:
>>>
>>> TEST check-qtest-ppc64: tests/qtest/device-introspect-test
>>> qemu-system-ppc64: warning: machine has no BMC device. Use '-device
>>> ipmi-bmc-sim,id=bmc0 -device isa-ipmi-bt,bmc=bmc0,irq=10' to define
>>> one
>>> qemu-system-ppc64: warning: machine has no BMC device. Use '-device
>>> ipmi-bmc-sim,id=bmc0 -device isa-ipmi-bt,bmc=bmc0,irq=10' to define
>>> one
>>> qemu-system-ppc64: warning: machine has no BMC device. Use '-device
>>> ipmi-bmc-sim,id=bmc0 -device isa-ipmi-bt,bmc=bmc0,irq=10' to define
>>> one
>>>
>>> This is expected since the pnv machine doesn't create the internal
>>> BMC simulator fallback when "-nodefaults" is passed on the command
>>> line, but these warnings appear in ci logs and confuse people.
>>>
>>> Not having a BMC isn't recommended but it is still a supported
>>> configuration, so a straightforward fix is to just silent this
>>> warning when qtest is enabled.
>>>
>>> Fixes: 25f3170b0654 ("ppc/pnv: Create BMC devices only when defaults are
>>> enabled")
>>> Reported-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>>> Signed-off-by: Greg Kurz <groug@kaod.org>
>>
>> Reviewed-by: Cédric Le Goater <clg@kaod.org>
>>
>> It looks good but could you reproduce ?
>>
>
> Yup, this test is only run in "slow" mode, eg:
>
> make check-qtest-ppc64 SPEED=slow
Indeed:
https://gitlab.com/qemu-project/qemu/-/jobs/603546723#L3337
See in .gitlab-ci.yml:
build-disabled:
...
- make -j"$JOBS"
- make -j"$JOBS" check-qtest SPEED=slow
Thomas, FYI this job is now timeouting most of the time.
>
>
>> C.
>>
>>> ---
>>> hw/ppc/pnv.c | 9 ++++++---
>>> 1 file changed, 6 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/hw/ppc/pnv.c b/hw/ppc/pnv.c
>>> index 806a5d9a8d34..1622d29b4ba7 100644
>>> --- a/hw/ppc/pnv.c
>>> +++ b/hw/ppc/pnv.c
>>> @@ -21,6 +21,7 @@
>>> #include "qemu-common.h"
>>> #include "qemu/units.h"
>>> #include "qapi/error.h"
>>> +#include "sysemu/qtest.h"
>>> #include "sysemu/sysemu.h"
>>> #include "sysemu/numa.h"
>>> #include "sysemu/reset.h"
>>> @@ -587,9 +588,11 @@ static void pnv_reset(MachineState *machine)
>>> bmc = pnv_bmc_find(&error_fatal);
>>> if (!pnv->bmc) {
>>> if (!bmc) {
>>> - warn_report("machine has no BMC device. Use '-device "
>>> - "ipmi-bmc-sim,id=bmc0 -device
>>> isa-ipmi-bt,bmc=bmc0,irq=10' "
>>> - "to define one");
>>> + if (!qtest_enabled()) {
>>> + warn_report("machine has no BMC device. Use '-device "
>>> + "ipmi-bmc-sim,id=bmc0 -device
>>> isa-ipmi-bt,bmc=bmc0,irq=10' "
>>> + "to define one");
>>> + }
>>> } else {
>>> pnv_bmc_set_pnor(bmc, pnv->pnor);
>>> pnv->bmc = bmc;
>>>
>>>
>>
>