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 <thuth@redhat.com>
---
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
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 <thuth@redhat.com> > --- > 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 <wei@redhat.com>
On Thu, Nov 16, 2017 at 7:29 AM, Thomas Huth <thuth@redhat.com> 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 <thuth@redhat.com> > --- > 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.francis@xilinx.com> Alistair > > xlnx_zynqmp_init(s, machine); > } > -- > 1.8.3.1 > >
On 16 November 2017 at 17:09, Alistair Francis <alistair.francis@xilinx.com> wrote: > On Thu, Nov 16, 2017 at 7:29 AM, Thomas Huth <thuth@redhat.com> 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 <thuth@redhat.com> >> --- >> 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.francis@xilinx.com> Thanks; applied to target-arm.next. -- PMM
© 2016 - 2024 Red Hat, Inc.