Thanks and sorry for missing this in the original commit. Acked-by: Dongwon Kim <dongwon....@intel.com>
> -----Original Message----- > From: Philippe Mathieu-Daudé <phi...@linaro.org> > Sent: Friday, May 17, 2024 8:02 AM > To: qemu-devel@nongnu.org > Cc: Cédric Le Goater <c...@redhat.com>; Kim, Dongwon > <dongwon....@intel.com>; Marc-André Lureau > <marcandre.lur...@redhat.com>; Philippe Mathieu-Daudé > <phi...@linaro.org> > Subject: [PULL 1/5] ui/console: Only declare variable fence_fd when > CONFIG_GBM is defined > > From: Cédric Le Goater <c...@redhat.com> > > This to avoid a build breakage : > > ../ui/gtk-egl.c: In function ‘gd_egl_draw’: > ../ui/gtk-egl.c:73:9: error: unused variable ‘fence_fd’ > [-Werror=unused-variable] > 73 | int fence_fd; > | ^~~~~~~~ > > Fixes: fa6426805b12 ("ui/console: Use qemu_dmabuf_set_..() helpers instead") > Cc: Dongwon Kim <dongwon....@intel.com> > Cc: Marc-André Lureau <marcandre.lur...@redhat.com> > Signed-off-by: Cédric Le Goater <c...@redhat.com> > Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org> > Message-ID: <20240515100520.574383-1-...@redhat.com> > Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org> > --- > ui/gtk-egl.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/ui/gtk-egl.c b/ui/gtk-egl.c index 0473f689c9..9831c10e1b 100644 > --- a/ui/gtk-egl.c > +++ b/ui/gtk-egl.c > @@ -68,9 +68,9 @@ void gd_egl_draw(VirtualConsole *vc) > GdkWindow *window; > #ifdef CONFIG_GBM > QemuDmaBuf *dmabuf = vc->gfx.guest_fb.dmabuf; > + int fence_fd; > #endif > int ww, wh, ws; > - int fence_fd; > > if (!vc->gfx.gls) { > return; > -- > 2.41.0