On 16 March 2016 at 10:46, Paolo Bonzini <pbonz...@redhat.com> wrote: > Avoid need for qemu/log.h inclusion. > > Signed-off-by: Paolo Bonzini <pbonz...@redhat.com> > --- > target-arm/helper.c | 14 ++++++++++++++ > target-arm/internals.h | 15 +-------------- > 2 files changed, 15 insertions(+), 14 deletions(-) > > diff --git a/target-arm/helper.c b/target-arm/helper.c > index eaded41..f71a78f 100644 > --- a/target-arm/helper.c > +++ b/target-arm/helper.c > @@ -5810,6 +5810,20 @@ static void do_v7m_exception_exit(CPUARMState *env) > pointer. */ > } > > +void arm_log_exception(int idx) > +{ > + if (qemu_loglevel_mask(CPU_LOG_INT)) { > + const char *exc = NULL; > + > + if (idx >= 0 && idx < ARRAY_SIZE(excnames)) { > + exc = excnames[idx]; > + } > + if (!exc) { > + exc = "unknown"; > + } > + qemu_log_mask(CPU_LOG_INT, "Taking exception %d [%s]\n", idx, exc); > + } > +} > void arm_v7m_cpu_do_interrupt(CPUState *cs) > {
Missing blank line after '}'. Also, arm_log_exception() is now only used in helper.c so you could just make it static in this file. thanks -- PMM