Re: [PATCH] hw/xen/xen_pt: fix missing include

["Daniel P. Berrangé" <berrange@redhat.com>]

On Fri, Jul 25, 2025 at 10:49:58AM -0700, Pierrick Bouvier wrote:
> On 7/25/25 10:22 AM, Daniel P. Berrangé wrote:
> > On Fri, Jul 25, 2025 at 09:57:49AM -0700, Pierrick Bouvier wrote:
> > > FAILED: libqemu-i386-softmmu.a.p/hw_xen_xen_pt.c.o
> > > ...
> > > /home/user/.work/qemu/hw/xen/xen_pt.c: In function ‘xen_pt_need_gsi’:
> > > /home/user/.work/qemu/hw/xen/xen_pt.c:783:9: error: implicit declaration of function ‘error_report’; did you mean ‘error_report_err’? [-Wimplicit-function-declaration]
> > >    783 |         error_report("Cannot open %s: %s", guest_type, strerror(errno));
> > >        |         ^~~~~~~~~~~~
> > >        |         error_report_err
> > > /home/user/.work/qemu/hw/xen/xen_pt.c:783:9: error: nested extern declaration of ‘error_report’ [-Werror=nested-externs]
> > > cc1: all warnings being treated as errors
> > > 
> > > Fixes: 012842c ("log: make '-msg timestamp=on' apply to all qemu_log usage")
> > 
> > The problem pre-dated this, so this should be
> > 
> >   Fixes: cfcacbab38e (xen/passthrough: use gsi to map pirq when dom0 is PVH)
> > 
> > See the prior posting of this fix last week:
> > 
> >    https://lists.nongnu.org/archive/html/qemu-devel/2025-07/msg04831.html
> > 
> 
> While true, this was not a problem as error-report.h was still implicitely
> included (as Markus pointed). We can debate where the regression comes from,
> but from git bisect point of view, this is the trace one, at least with the
> simple "./configure --enable-debug".

It was /only/ implicitly included if you had the 'log' trace backend
enabled - other configurations missed it, hence the build failures
we got in Fedora from commit cfcacbab38e.


With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|