Re: [Qemu-devel] [PATCH] hw/arm: Silence xlnx-ep108 deprecation warning during tests
On 16 November 2017 at 17:09, Alistair Franciswrote: > On Thu, Nov 16, 2017 at 7:29 AM, Thomas Huth wrote: >> The new deprecation warning for the xlnx-ep108 machine also pops up >> during "make check" which is kind of confusing. Silence it if testing >> mode is enabled. >> >> Signed-off-by: Thomas Huth >> --- >> Note: I slightly shortened the message so that the code still fits >> into the 80-columns limit. >> >> hw/arm/xlnx-zcu102.c | 7 +-- >> 1 file changed, 5 insertions(+), 2 deletions(-) >> >> diff --git a/hw/arm/xlnx-zcu102.c b/hw/arm/xlnx-zcu102.c >> index 9631a53..bbe7d04 100644 >> --- a/hw/arm/xlnx-zcu102.c >> +++ b/hw/arm/xlnx-zcu102.c >> @@ -24,6 +24,7 @@ >> #include "qemu/error-report.h" >> #include "exec/address-spaces.h" >> #include "qemu/log.h" >> +#include "sysemu/qtest.h" >> >> typedef struct XlnxZCU102 { >> MachineState parent_obj; >> @@ -164,8 +165,10 @@ static void xlnx_ep108_init(MachineState *machine) >> { >> XlnxZCU102 *s = EP108_MACHINE(machine); >> >> -info_report("The Xilinx EP108 machine is deprecated, please use the " >> -"ZCU102 machine instead. It has the same features >> supported."); >> +if (!qtest_enabled()) { >> +info_report("The Xilinx EP108 machine is deprecated, please use the >> " >> +"ZCU102 machine (which has the same features) >> instead."); >> +} > > Looks good to me. > > In future maybe it makes sense to never print info_report() when running > qtest. > > Reviewed-by: Alistair Francis Thanks; applied to target-arm.next. -- PMM
Re: [Qemu-devel] [PATCH] hw/arm: Silence xlnx-ep108 deprecation warning during tests
On Thu, Nov 16, 2017 at 7:29 AM, Thomas Huthwrote: > The new deprecation warning for the xlnx-ep108 machine also pops up > during "make check" which is kind of confusing. Silence it if testing > mode is enabled. > > Signed-off-by: Thomas Huth > --- > Note: I slightly shortened the message so that the code still fits > into the 80-columns limit. > > hw/arm/xlnx-zcu102.c | 7 +-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/hw/arm/xlnx-zcu102.c b/hw/arm/xlnx-zcu102.c > index 9631a53..bbe7d04 100644 > --- a/hw/arm/xlnx-zcu102.c > +++ b/hw/arm/xlnx-zcu102.c > @@ -24,6 +24,7 @@ > #include "qemu/error-report.h" > #include "exec/address-spaces.h" > #include "qemu/log.h" > +#include "sysemu/qtest.h" > > typedef struct XlnxZCU102 { > MachineState parent_obj; > @@ -164,8 +165,10 @@ static void xlnx_ep108_init(MachineState *machine) > { > XlnxZCU102 *s = EP108_MACHINE(machine); > > -info_report("The Xilinx EP108 machine is deprecated, please use the " > -"ZCU102 machine instead. It has the same features > supported."); > +if (!qtest_enabled()) { > +info_report("The Xilinx EP108 machine is deprecated, please use the " > +"ZCU102 machine (which has the same features) instead."); > +} Looks good to me. In future maybe it makes sense to never print info_report() when running qtest. Reviewed-by: Alistair Francis Alistair > > xlnx_zynqmp_init(s, machine); > } > -- > 1.8.3.1 > >
Re: [Qemu-devel] [PATCH] hw/arm: Silence xlnx-ep108 deprecation warning during tests
On 11/16/2017 09:29 AM, Thomas Huth wrote: > The new deprecation warning for the xlnx-ep108 machine also pops up > during "make check" which is kind of confusing. Silence it if testing > mode is enabled. > > Signed-off-by: Thomas Huth> --- > Note: I slightly shortened the message so that the code still fits > into the 80-columns limit. > > hw/arm/xlnx-zcu102.c | 7 +-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/hw/arm/xlnx-zcu102.c b/hw/arm/xlnx-zcu102.c > index 9631a53..bbe7d04 100644 > --- a/hw/arm/xlnx-zcu102.c > +++ b/hw/arm/xlnx-zcu102.c > @@ -24,6 +24,7 @@ > #include "qemu/error-report.h" > #include "exec/address-spaces.h" > #include "qemu/log.h" > +#include "sysemu/qtest.h" > > typedef struct XlnxZCU102 { > MachineState parent_obj; > @@ -164,8 +165,10 @@ static void xlnx_ep108_init(MachineState *machine) > { > XlnxZCU102 *s = EP108_MACHINE(machine); > > -info_report("The Xilinx EP108 machine is deprecated, please use the " > -"ZCU102 machine instead. It has the same features > supported."); > +if (!qtest_enabled()) { > +info_report("The Xilinx EP108 machine is deprecated, please use the " > +"ZCU102 machine (which has the same features) instead."); > +} > > xlnx_zynqmp_init(s, machine); > } > I tested it and it did solve the problem. Acked-by: Wei Huang
[Qemu-devel] [PATCH] hw/arm: Silence xlnx-ep108 deprecation warning during tests
The new deprecation warning for the xlnx-ep108 machine also pops up during "make check" which is kind of confusing. Silence it if testing mode is enabled. Signed-off-by: Thomas Huth--- Note: I slightly shortened the message so that the code still fits into the 80-columns limit. hw/arm/xlnx-zcu102.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/hw/arm/xlnx-zcu102.c b/hw/arm/xlnx-zcu102.c index 9631a53..bbe7d04 100644 --- a/hw/arm/xlnx-zcu102.c +++ b/hw/arm/xlnx-zcu102.c @@ -24,6 +24,7 @@ #include "qemu/error-report.h" #include "exec/address-spaces.h" #include "qemu/log.h" +#include "sysemu/qtest.h" typedef struct XlnxZCU102 { MachineState parent_obj; @@ -164,8 +165,10 @@ static void xlnx_ep108_init(MachineState *machine) { XlnxZCU102 *s = EP108_MACHINE(machine); -info_report("The Xilinx EP108 machine is deprecated, please use the " -"ZCU102 machine instead. It has the same features supported."); +if (!qtest_enabled()) { +info_report("The Xilinx EP108 machine is deprecated, please use the " +"ZCU102 machine (which has the same features) instead."); +} xlnx_zynqmp_init(s, machine); } -- 1.8.3.1