include/hw/virtio/virtio-gpu.h | 5 +++-- include/ui/console.h | 4 ++++ hw/display/virtio-gpu-udmabuf-stubs.c | 3 ++- hw/display/virtio-gpu-udmabuf.c | 22 ++++++++++++++-------- hw/display/virtio-gpu.c | 4 ++-- ui/egl-helpers.c | 25 +++++++++++++++++++++---- ui/gtk-egl.c | 10 ++++++++++ ui/gtk.c | 23 +++++++++++++++++++++++ 8 files changed, 79 insertions(+), 17 deletions(-)
The following changes since commit b1fd92137e4d485adeec8e9f292f928ff335b76c:
Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into staging (2021-11-03 13:07:30 -0400)
are available in the Git repository at:
git://git.kraxel.org/qemu tags/egl-20211105-pull-request
for you to fetch changes up to 1350ff156b25be65c599ecca9957ce6726c6d383:
ui/gtk-egl: blitting partial guest fb to the proper scanout surface (2021-11-05 12:29:44 +0100)
----------------------------------------------------------------
gtk: a collection of egl fixes.
----------------------------------------------------------------
Dongwon Kim (6):
virtio-gpu: splitting one extended mode guest fb into n-scanouts
ui/gtk-egl: un-tab and re-tab should destroy egl surface and context
ui/gtk-egl: make sure the right context is set as the current
ui/gtk-egl: guest fb texture needs to be regenerated when
reinitializing egl
ui/gtk: gd_draw_event returns FALSE when no cairo surface is bound
ui/gtk-egl: blitting partial guest fb to the proper scanout surface
include/hw/virtio/virtio-gpu.h | 5 +++--
include/ui/console.h | 4 ++++
hw/display/virtio-gpu-udmabuf-stubs.c | 3 ++-
hw/display/virtio-gpu-udmabuf.c | 22 ++++++++++++++--------
hw/display/virtio-gpu.c | 4 ++--
ui/egl-helpers.c | 25 +++++++++++++++++++++----
ui/gtk-egl.c | 10 ++++++++++
ui/gtk.c | 23 +++++++++++++++++++++++
8 files changed, 79 insertions(+), 17 deletions(-)
--
2.31.1
On 11/5/21 7:30 AM, Gerd Hoffmann wrote: > The following changes since commit b1fd92137e4d485adeec8e9f292f928ff335b76c: > > Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into staging (2021-11-03 13:07:30 -0400) > > are available in the Git repository at: > > git://git.kraxel.org/qemu tags/egl-20211105-pull-request > > for you to fetch changes up to 1350ff156b25be65c599ecca9957ce6726c6d383: > > ui/gtk-egl: blitting partial guest fb to the proper scanout surface (2021-11-05 12:29:44 +0100) > > ---------------------------------------------------------------- > gtk: a collection of egl fixes. > > ---------------------------------------------------------------- > > Dongwon Kim (6): > virtio-gpu: splitting one extended mode guest fb into n-scanouts > ui/gtk-egl: un-tab and re-tab should destroy egl surface and context > ui/gtk-egl: make sure the right context is set as the current > ui/gtk-egl: guest fb texture needs to be regenerated when > reinitializing egl > ui/gtk: gd_draw_event returns FALSE when no cairo surface is bound > ui/gtk-egl: blitting partial guest fb to the proper scanout surface > > include/hw/virtio/virtio-gpu.h | 5 +++-- > include/ui/console.h | 4 ++++ > hw/display/virtio-gpu-udmabuf-stubs.c | 3 ++- > hw/display/virtio-gpu-udmabuf.c | 22 ++++++++++++++-------- > hw/display/virtio-gpu.c | 4 ++-- > ui/egl-helpers.c | 25 +++++++++++++++++++++---- > ui/gtk-egl.c | 10 ++++++++++ > ui/gtk.c | 23 +++++++++++++++++++++++ > 8 files changed, 79 insertions(+), 17 deletions(-) Applied, thanks. r~
On 11/5/21 18:13, Richard Henderson wrote:
> On 11/5/21 7:30 AM, Gerd Hoffmann wrote:
>> The following changes since commit
>> b1fd92137e4d485adeec8e9f292f928ff335b76c:
>>
>> Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream'
>> into staging (2021-11-03 13:07:30 -0400)
>>
>> are available in the Git repository at:
>>
>> git://git.kraxel.org/qemu tags/egl-20211105-pull-request
>>
>> for you to fetch changes up to 1350ff156b25be65c599ecca9957ce6726c6d383:
>>
>> ui/gtk-egl: blitting partial guest fb to the proper scanout surface
>> (2021-11-05 12:29:44 +0100)
>>
>> ----------------------------------------------------------------
>> gtk: a collection of egl fixes.
>>
>> ----------------------------------------------------------------
>>
>> Dongwon Kim (6):
>> virtio-gpu: splitting one extended mode guest fb into n-scanouts
>> ui/gtk-egl: un-tab and re-tab should destroy egl surface and context
>> ui/gtk-egl: make sure the right context is set as the current
>> ui/gtk-egl: guest fb texture needs to be regenerated when
>> reinitializing egl
>> ui/gtk: gd_draw_event returns FALSE when no cairo surface is bound
>> ui/gtk-egl: blitting partial guest fb to the proper scanout surface
>>
>> include/hw/virtio/virtio-gpu.h | 5 +++--
>> include/ui/console.h | 4 ++++
>> hw/display/virtio-gpu-udmabuf-stubs.c | 3 ++-
>> hw/display/virtio-gpu-udmabuf.c | 22 ++++++++++++++--------
>> hw/display/virtio-gpu.c | 4 ++--
>> ui/egl-helpers.c | 25 +++++++++++++++++++++----
>> ui/gtk-egl.c | 10 ++++++++++
>> ui/gtk.c | 23 +++++++++++++++++++++++
>> 8 files changed, 79 insertions(+), 17 deletions(-)
>
> Applied, thanks.
Ubuntu 18.04.4 LTS:
ui/gtk-egl.c:159:13: error: implicit declaration of function
'egl_dmabuf_release_texture' is invalid in C99
[-Werror,-Wimplicit-function-declaration]
egl_dmabuf_release_texture(vc->gfx.guest_fb.dmabuf);
^
ui/gtk-egl.c:159:13: error: this function declaration is not a prototype
[-Werror,-Wstrict-prototypes]
2 errors generated.
https://app.travis-ci.com/gitlab/qemu-project/qemu/builds/241272737
On 11/5/21 19:26, Philippe Mathieu-Daudé wrote:
> On 11/5/21 18:13, Richard Henderson wrote:
>> On 11/5/21 7:30 AM, Gerd Hoffmann wrote:
>>> The following changes since commit
>>> b1fd92137e4d485adeec8e9f292f928ff335b76c:
>>>
>>> Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream'
>>> into staging (2021-11-03 13:07:30 -0400)
>>>
>>> are available in the Git repository at:
>>>
>>> git://git.kraxel.org/qemu tags/egl-20211105-pull-request
>>>
>>> for you to fetch changes up to 1350ff156b25be65c599ecca9957ce6726c6d383:
>>>
>>> ui/gtk-egl: blitting partial guest fb to the proper scanout surface
>>> (2021-11-05 12:29:44 +0100)
>>>
>>> ----------------------------------------------------------------
>>> gtk: a collection of egl fixes.
>>>
>>> ----------------------------------------------------------------
>>>
>>> Dongwon Kim (6):
>>> virtio-gpu: splitting one extended mode guest fb into n-scanouts
>>> ui/gtk-egl: un-tab and re-tab should destroy egl surface and context
>>> ui/gtk-egl: make sure the right context is set as the current
>>> ui/gtk-egl: guest fb texture needs to be regenerated when
>>> reinitializing egl
>>> ui/gtk: gd_draw_event returns FALSE when no cairo surface is bound
>>> ui/gtk-egl: blitting partial guest fb to the proper scanout surface
>>>
>>> include/hw/virtio/virtio-gpu.h | 5 +++--
>>> include/ui/console.h | 4 ++++
>>> hw/display/virtio-gpu-udmabuf-stubs.c | 3 ++-
>>> hw/display/virtio-gpu-udmabuf.c | 22 ++++++++++++++--------
>>> hw/display/virtio-gpu.c | 4 ++--
>>> ui/egl-helpers.c | 25 +++++++++++++++++++++----
>>> ui/gtk-egl.c | 10 ++++++++++
>>> ui/gtk.c | 23 +++++++++++++++++++++++
>>> 8 files changed, 79 insertions(+), 17 deletions(-)
>>
>> Applied, thanks.
>
> Ubuntu 18.04.4 LTS:
>
> ui/gtk-egl.c:159:13: error: implicit declaration of function
> 'egl_dmabuf_release_texture' is invalid in C99
> [-Werror,-Wimplicit-function-declaration]
> egl_dmabuf_release_texture(vc->gfx.guest_fb.dmabuf);
> ^
> ui/gtk-egl.c:159:13: error: this function declaration is not a prototype
> [-Werror,-Wstrict-prototypes]
> 2 errors generated.
>
> https://app.travis-ci.com/gitlab/qemu-project/qemu/builds/241272737
>
This seems to fix but I have no clue whether it is correct:
-- >8 --
diff --git a/ui/gtk-egl.c b/ui/gtk-egl.c
index f2026e4b5c9..45cb67712df 100644
--- a/ui/gtk-egl.c
+++ b/ui/gtk-egl.c
@@ -156,8 +156,10 @@ void gd_egl_refresh(DisplayChangeListener *dcl)
surface_gl_create_texture(vc->gfx.gls, vc->gfx.ds);
}
+#ifdef CONFIG_GBM
if (vc->gfx.guest_fb.dmabuf) {
egl_dmabuf_release_texture(vc->gfx.guest_fb.dmabuf);
gd_egl_scanout_dmabuf(dcl, vc->gfx.guest_fb.dmabuf);
}
+#endif
}
---
On 05/11/2021 18:49, Philippe Mathieu-Daudé wrote:
> On 11/5/21 19:26, Philippe Mathieu-Daudé wrote:
>> On 11/5/21 18:13, Richard Henderson wrote:
>>> On 11/5/21 7:30 AM, Gerd Hoffmann wrote:
>>>> The following changes since commit
>>>> b1fd92137e4d485adeec8e9f292f928ff335b76c:
>>>>
>>>> Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream'
>>>> into staging (2021-11-03 13:07:30 -0400)
>>>>
>>>> are available in the Git repository at:
>>>>
>>>> git://git.kraxel.org/qemu tags/egl-20211105-pull-request
>>>>
>>>> for you to fetch changes up to 1350ff156b25be65c599ecca9957ce6726c6d383:
>>>>
>>>> ui/gtk-egl: blitting partial guest fb to the proper scanout surface
>>>> (2021-11-05 12:29:44 +0100)
>>>>
>>>> ----------------------------------------------------------------
>>>> gtk: a collection of egl fixes.
>>>>
>>>> ----------------------------------------------------------------
>>>>
>>>> Dongwon Kim (6):
>>>> virtio-gpu: splitting one extended mode guest fb into n-scanouts
>>>> ui/gtk-egl: un-tab and re-tab should destroy egl surface and context
>>>> ui/gtk-egl: make sure the right context is set as the current
>>>> ui/gtk-egl: guest fb texture needs to be regenerated when
>>>> reinitializing egl
>>>> ui/gtk: gd_draw_event returns FALSE when no cairo surface is bound
>>>> ui/gtk-egl: blitting partial guest fb to the proper scanout surface
>>>>
>>>> include/hw/virtio/virtio-gpu.h | 5 +++--
>>>> include/ui/console.h | 4 ++++
>>>> hw/display/virtio-gpu-udmabuf-stubs.c | 3 ++-
>>>> hw/display/virtio-gpu-udmabuf.c | 22 ++++++++++++++--------
>>>> hw/display/virtio-gpu.c | 4 ++--
>>>> ui/egl-helpers.c | 25 +++++++++++++++++++++----
>>>> ui/gtk-egl.c | 10 ++++++++++
>>>> ui/gtk.c | 23 +++++++++++++++++++++++
>>>> 8 files changed, 79 insertions(+), 17 deletions(-)
>>>
>>> Applied, thanks.
>>
>> Ubuntu 18.04.4 LTS:
>>
>> ui/gtk-egl.c:159:13: error: implicit declaration of function
>> 'egl_dmabuf_release_texture' is invalid in C99
>> [-Werror,-Wimplicit-function-declaration]
>> egl_dmabuf_release_texture(vc->gfx.guest_fb.dmabuf);
>> ^
>> ui/gtk-egl.c:159:13: error: this function declaration is not a prototype
>> [-Werror,-Wstrict-prototypes]
>> 2 errors generated.
>>
>> https://app.travis-ci.com/gitlab/qemu-project/qemu/builds/241272737
>>
>
> This seems to fix but I have no clue whether it is correct:
>
> -- >8 --
> diff --git a/ui/gtk-egl.c b/ui/gtk-egl.c
> index f2026e4b5c9..45cb67712df 100644
> --- a/ui/gtk-egl.c
> +++ b/ui/gtk-egl.c
> @@ -156,8 +156,10 @@ void gd_egl_refresh(DisplayChangeListener *dcl)
> surface_gl_create_texture(vc->gfx.gls, vc->gfx.ds);
> }
> +#ifdef CONFIG_GBM
> if (vc->gfx.guest_fb.dmabuf) {
> egl_dmabuf_release_texture(vc->gfx.guest_fb.dmabuf);
> gd_egl_scanout_dmabuf(dcl, vc->gfx.guest_fb.dmabuf);
> }
> +#endif
> }
>
> ---
I see the same error here trying to build QEMU git master on Debian Buster
(oldstable). The fix looks reasonable to me in that it matches the CONFIG_GBM guards
around other similar functions and the resulting binary appears to work, so:
Reviewed-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
ATB,
Mark.
+Thomas & Daniel for Travis-CI
On 11/8/21 09:12, Mark Cave-Ayland wrote:
> On 05/11/2021 18:49, Philippe Mathieu-Daudé wrote:
>> On 11/5/21 19:26, Philippe Mathieu-Daudé wrote:
>>> On 11/5/21 18:13, Richard Henderson wrote:
>>>> On 11/5/21 7:30 AM, Gerd Hoffmann wrote:
>>>>> The following changes since commit
>>>>> b1fd92137e4d485adeec8e9f292f928ff335b76c:
>>>>>
>>>>> Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream'
>>>>> into staging (2021-11-03 13:07:30 -0400)
>>>>>
>>>>> are available in the Git repository at:
>>>>>
>>>>> git://git.kraxel.org/qemu tags/egl-20211105-pull-request
>>>>>
>>>>> for you to fetch changes up to
>>>>> 1350ff156b25be65c599ecca9957ce6726c6d383:
>>>>>
>>>>> ui/gtk-egl: blitting partial guest fb to the proper scanout
>>>>> surface
>>>>> (2021-11-05 12:29:44 +0100)
>>>>>
>>>>> ----------------------------------------------------------------
>>>>> gtk: a collection of egl fixes.
>>>>>
>>>>> ----------------------------------------------------------------
>>>>>
>>>>> Dongwon Kim (6):
>>>>> virtio-gpu: splitting one extended mode guest fb into n-scanouts
>>>>> ui/gtk-egl: un-tab and re-tab should destroy egl surface and
>>>>> context
>>>>> ui/gtk-egl: make sure the right context is set as the current
>>>>> ui/gtk-egl: guest fb texture needs to be regenerated when
>>>>> reinitializing egl
>>>>> ui/gtk: gd_draw_event returns FALSE when no cairo surface is bound
>>>>> ui/gtk-egl: blitting partial guest fb to the proper scanout
>>>>> surface
>>>>>
>>>>> include/hw/virtio/virtio-gpu.h | 5 +++--
>>>>> include/ui/console.h | 4 ++++
>>>>> hw/display/virtio-gpu-udmabuf-stubs.c | 3 ++-
>>>>> hw/display/virtio-gpu-udmabuf.c | 22 ++++++++++++++--------
>>>>> hw/display/virtio-gpu.c | 4 ++--
>>>>> ui/egl-helpers.c | 25
>>>>> +++++++++++++++++++++----
>>>>> ui/gtk-egl.c | 10 ++++++++++
>>>>> ui/gtk.c | 23 +++++++++++++++++++++++
>>>>> 8 files changed, 79 insertions(+), 17 deletions(-)
>>>>
>>>> Applied, thanks.
>>>
>>> Ubuntu 18.04.4 LTS:
>>>
>>> ui/gtk-egl.c:159:13: error: implicit declaration of function
>>> 'egl_dmabuf_release_texture' is invalid in C99
>>> [-Werror,-Wimplicit-function-declaration]
>>> egl_dmabuf_release_texture(vc->gfx.guest_fb.dmabuf);
>>> ^
>>> ui/gtk-egl.c:159:13: error: this function declaration is not a prototype
>>> [-Werror,-Wstrict-prototypes]
>>> 2 errors generated.
>>>
>>> https://app.travis-ci.com/gitlab/qemu-project/qemu/builds/241272737
>>>
>>
>> This seems to fix but I have no clue whether it is correct:
>>
>> -- >8 --
>> diff --git a/ui/gtk-egl.c b/ui/gtk-egl.c
>> index f2026e4b5c9..45cb67712df 100644
>> --- a/ui/gtk-egl.c
>> +++ b/ui/gtk-egl.c
>> @@ -156,8 +156,10 @@ void gd_egl_refresh(DisplayChangeListener *dcl)
>> surface_gl_create_texture(vc->gfx.gls, vc->gfx.ds);
>> }
>> +#ifdef CONFIG_GBM
>> if (vc->gfx.guest_fb.dmabuf) {
>> egl_dmabuf_release_texture(vc->gfx.guest_fb.dmabuf);
>> gd_egl_scanout_dmabuf(dcl, vc->gfx.guest_fb.dmabuf);
>> }
>> +#endif
>> }
>>
>> ---
>
> I see the same error here trying to build QEMU git master on Debian
> Buster (oldstable). The fix looks reasonable to me in that it matches
> the CONFIG_GBM guards around other similar functions and the resulting
> binary appears to work, so:
>
> Reviewed-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Thank you, I'll post a formal patch then.
I wonder why this got merged while this configuration is covered in
Travis-CI. Is it that we have a too high failure rate than we don't
use it anymore?
On Mon, Nov 08, 2021 at 09:17:19AM +0100, Philippe Mathieu-Daudé wrote:
> +Thomas & Daniel for Travis-CI
>
> On 11/8/21 09:12, Mark Cave-Ayland wrote:
> > On 05/11/2021 18:49, Philippe Mathieu-Daudé wrote:
> >> On 11/5/21 19:26, Philippe Mathieu-Daudé wrote:
> >>> On 11/5/21 18:13, Richard Henderson wrote:
> >>>> On 11/5/21 7:30 AM, Gerd Hoffmann wrote:
> >>>>> The following changes since commit
> >>>>> b1fd92137e4d485adeec8e9f292f928ff335b76c:
> >>>>>
> >>>>> Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream'
> >>>>> into staging (2021-11-03 13:07:30 -0400)
> >>>>>
> >>>>> are available in the Git repository at:
> >>>>>
> >>>>> git://git.kraxel.org/qemu tags/egl-20211105-pull-request
> >>>>>
> >>>>> for you to fetch changes up to
> >>>>> 1350ff156b25be65c599ecca9957ce6726c6d383:
> >>>>>
> >>>>> ui/gtk-egl: blitting partial guest fb to the proper scanout
> >>>>> surface
> >>>>> (2021-11-05 12:29:44 +0100)
> >>>>>
> >>>>> ----------------------------------------------------------------
> >>>>> gtk: a collection of egl fixes.
> >>>>>
> >>>>> ----------------------------------------------------------------
> >>>>>
> >>>>> Dongwon Kim (6):
> >>>>> virtio-gpu: splitting one extended mode guest fb into n-scanouts
> >>>>> ui/gtk-egl: un-tab and re-tab should destroy egl surface and
> >>>>> context
> >>>>> ui/gtk-egl: make sure the right context is set as the current
> >>>>> ui/gtk-egl: guest fb texture needs to be regenerated when
> >>>>> reinitializing egl
> >>>>> ui/gtk: gd_draw_event returns FALSE when no cairo surface is bound
> >>>>> ui/gtk-egl: blitting partial guest fb to the proper scanout
> >>>>> surface
> >>>>>
> >>>>> include/hw/virtio/virtio-gpu.h | 5 +++--
> >>>>> include/ui/console.h | 4 ++++
> >>>>> hw/display/virtio-gpu-udmabuf-stubs.c | 3 ++-
> >>>>> hw/display/virtio-gpu-udmabuf.c | 22 ++++++++++++++--------
> >>>>> hw/display/virtio-gpu.c | 4 ++--
> >>>>> ui/egl-helpers.c | 25
> >>>>> +++++++++++++++++++++----
> >>>>> ui/gtk-egl.c | 10 ++++++++++
> >>>>> ui/gtk.c | 23 +++++++++++++++++++++++
> >>>>> 8 files changed, 79 insertions(+), 17 deletions(-)
> >>>>
> >>>> Applied, thanks.
> >>>
> >>> Ubuntu 18.04.4 LTS:
> >>>
> >>> ui/gtk-egl.c:159:13: error: implicit declaration of function
> >>> 'egl_dmabuf_release_texture' is invalid in C99
> >>> [-Werror,-Wimplicit-function-declaration]
> >>> egl_dmabuf_release_texture(vc->gfx.guest_fb.dmabuf);
> >>> ^
> >>> ui/gtk-egl.c:159:13: error: this function declaration is not a prototype
> >>> [-Werror,-Wstrict-prototypes]
> >>> 2 errors generated.
> >>>
> >>> https://app.travis-ci.com/gitlab/qemu-project/qemu/builds/241272737
> >>>
> >>
> >> This seems to fix but I have no clue whether it is correct:
> >>
> >> -- >8 --
> >> diff --git a/ui/gtk-egl.c b/ui/gtk-egl.c
> >> index f2026e4b5c9..45cb67712df 100644
> >> --- a/ui/gtk-egl.c
> >> +++ b/ui/gtk-egl.c
> >> @@ -156,8 +156,10 @@ void gd_egl_refresh(DisplayChangeListener *dcl)
> >> surface_gl_create_texture(vc->gfx.gls, vc->gfx.ds);
> >> }
> >> +#ifdef CONFIG_GBM
> >> if (vc->gfx.guest_fb.dmabuf) {
> >> egl_dmabuf_release_texture(vc->gfx.guest_fb.dmabuf);
> >> gd_egl_scanout_dmabuf(dcl, vc->gfx.guest_fb.dmabuf);
> >> }
> >> +#endif
> >> }
> >>
> >> ---
> >
> > I see the same error here trying to build QEMU git master on Debian
> > Buster (oldstable). The fix looks reasonable to me in that it matches
> > the CONFIG_GBM guards around other similar functions and the resulting
> > binary appears to work, so:
> >
> > Reviewed-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
>
> Thank you, I'll post a formal patch then.
>
> I wonder why this got merged while this configuration is covered in
> Travis-CI. Is it that we have a too high failure rate than we don't
> use it anymore?
I've not looked at travis in ages what matters is GitLab CI, and there
the problem is that we're building ubuntu 20.04 not 18.04 so didn't
catch the older problem.
I don't know what Mark sees a problem on Debian Buster though, as we
successfully built there AFAICT
https://gitlab.com/qemu-project/qemu/-/jobs/1756202449
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 :|
> > >> +#ifdef CONFIG_GBM
> > >> if (vc->gfx.guest_fb.dmabuf) {
> > >> egl_dmabuf_release_texture(vc->gfx.guest_fb.dmabuf);
> > >> gd_egl_scanout_dmabuf(dcl, vc->gfx.guest_fb.dmabuf);
> > >> }
> > >> +#endif
> I don't know what Mark sees a problem on Debian Buster though, as we
> successfully built there AFAICT
>
> https://gitlab.com/qemu-project/qemu/-/jobs/1756202449
Probably depends on whenever libgbm-dev (or whatever the package
is named on debian) is installed or not.
take care,
Gerd
On 11/8/21 10:22, Daniel P. Berrangé wrote:
> On Mon, Nov 08, 2021 at 09:17:19AM +0100, Philippe Mathieu-Daudé wrote:
>> +Thomas & Daniel for Travis-CI
>>
>> On 11/8/21 09:12, Mark Cave-Ayland wrote:
>>> On 05/11/2021 18:49, Philippe Mathieu-Daudé wrote:
>>>> On 11/5/21 19:26, Philippe Mathieu-Daudé wrote:
>>>>> On 11/5/21 18:13, Richard Henderson wrote:
>>>>>> On 11/5/21 7:30 AM, Gerd Hoffmann wrote:
>>>>>>> The following changes since commit
>>>>>>> b1fd92137e4d485adeec8e9f292f928ff335b76c:
>>>>>>>
>>>>>>> Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream'
>>>>>>> into staging (2021-11-03 13:07:30 -0400)
>>>>>>>
>>>>>>> are available in the Git repository at:
>>>>>>>
>>>>>>> git://git.kraxel.org/qemu tags/egl-20211105-pull-request
>>>>>>>
>>>>>>> for you to fetch changes up to
>>>>>>> 1350ff156b25be65c599ecca9957ce6726c6d383:
>>>>>>>
>>>>>>> ui/gtk-egl: blitting partial guest fb to the proper scanout
>>>>>>> surface
>>>>>>> (2021-11-05 12:29:44 +0100)
>>>>>>>
>>>>>>> ----------------------------------------------------------------
>>>>>>> gtk: a collection of egl fixes.
>>>>>>>
>>>>>>> ----------------------------------------------------------------
>>>>>>>
>>>>>>> Dongwon Kim (6):
>>>>>>> virtio-gpu: splitting one extended mode guest fb into n-scanouts
>>>>>>> ui/gtk-egl: un-tab and re-tab should destroy egl surface and
>>>>>>> context
>>>>>>> ui/gtk-egl: make sure the right context is set as the current
>>>>>>> ui/gtk-egl: guest fb texture needs to be regenerated when
>>>>>>> reinitializing egl
>>>>>>> ui/gtk: gd_draw_event returns FALSE when no cairo surface is bound
>>>>>>> ui/gtk-egl: blitting partial guest fb to the proper scanout
>>>>>>> surface
>>>>>>>
>>>>>>> include/hw/virtio/virtio-gpu.h | 5 +++--
>>>>>>> include/ui/console.h | 4 ++++
>>>>>>> hw/display/virtio-gpu-udmabuf-stubs.c | 3 ++-
>>>>>>> hw/display/virtio-gpu-udmabuf.c | 22 ++++++++++++++--------
>>>>>>> hw/display/virtio-gpu.c | 4 ++--
>>>>>>> ui/egl-helpers.c | 25
>>>>>>> +++++++++++++++++++++----
>>>>>>> ui/gtk-egl.c | 10 ++++++++++
>>>>>>> ui/gtk.c | 23 +++++++++++++++++++++++
>>>>>>> 8 files changed, 79 insertions(+), 17 deletions(-)
>>>>>>
>>>>>> Applied, thanks.
>>>>>
>>>>> Ubuntu 18.04.4 LTS:
>>>>>
>>>>> ui/gtk-egl.c:159:13: error: implicit declaration of function
>>>>> 'egl_dmabuf_release_texture' is invalid in C99
>>>>> [-Werror,-Wimplicit-function-declaration]
>>>>> egl_dmabuf_release_texture(vc->gfx.guest_fb.dmabuf);
>>>>> ^
>>>>> ui/gtk-egl.c:159:13: error: this function declaration is not a prototype
>>>>> [-Werror,-Wstrict-prototypes]
>>>>> 2 errors generated.
>>>>>
>>>>> https://app.travis-ci.com/gitlab/qemu-project/qemu/builds/241272737
>>>>>
>>>>
>>>> This seems to fix but I have no clue whether it is correct:
>>>>
>>>> -- >8 --
>>>> diff --git a/ui/gtk-egl.c b/ui/gtk-egl.c
>>>> index f2026e4b5c9..45cb67712df 100644
>>>> --- a/ui/gtk-egl.c
>>>> +++ b/ui/gtk-egl.c
>>>> @@ -156,8 +156,10 @@ void gd_egl_refresh(DisplayChangeListener *dcl)
>>>> surface_gl_create_texture(vc->gfx.gls, vc->gfx.ds);
>>>> }
>>>> +#ifdef CONFIG_GBM
>>>> if (vc->gfx.guest_fb.dmabuf) {
>>>> egl_dmabuf_release_texture(vc->gfx.guest_fb.dmabuf);
>>>> gd_egl_scanout_dmabuf(dcl, vc->gfx.guest_fb.dmabuf);
>>>> }
>>>> +#endif
>>>> }
>>>>
>>>> ---
>>>
>>> I see the same error here trying to build QEMU git master on Debian
>>> Buster (oldstable). The fix looks reasonable to me in that it matches
>>> the CONFIG_GBM guards around other similar functions and the resulting
>>> binary appears to work, so:
>>>
>>> Reviewed-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
>>
>> Thank you, I'll post a formal patch then.
>>
>> I wonder why this got merged while this configuration is covered in
>> Travis-CI. Is it that we have a too high failure rate than we don't
>> use it anymore?
>
> I've not looked at travis in ages what matters is GitLab CI, and there
I Cc'ed you because I thought it was you who suggested to add the
Travis-CI - GitLab bridge (external job). Maybe it was Thomas.
> the problem is that we're building ubuntu 20.04 not 18.04 so didn't
> catch the older problem.
>
> I don't know what Mark sees a problem on Debian Buster though, as we
> successfully built there AFAICT
>
> https://gitlab.com/qemu-project/qemu/-/jobs/1756202449
But here libgbm is installed:
Run-time dependency gbm found: YES 18.3.6
The issue is on hosts without it.
On 08/11/2021 09:22, Daniel P. Berrangé wrote:
> On Mon, Nov 08, 2021 at 09:17:19AM +0100, Philippe Mathieu-Daudé wrote:
>> +Thomas & Daniel for Travis-CI
>>
>> On 11/8/21 09:12, Mark Cave-Ayland wrote:
>>> On 05/11/2021 18:49, Philippe Mathieu-Daudé wrote:
>>>> On 11/5/21 19:26, Philippe Mathieu-Daudé wrote:
>>>>> On 11/5/21 18:13, Richard Henderson wrote:
>>>>>> On 11/5/21 7:30 AM, Gerd Hoffmann wrote:
>>>>>>> The following changes since commit
>>>>>>> b1fd92137e4d485adeec8e9f292f928ff335b76c:
>>>>>>>
>>>>>>> Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream'
>>>>>>> into staging (2021-11-03 13:07:30 -0400)
>>>>>>>
>>>>>>> are available in the Git repository at:
>>>>>>>
>>>>>>> git://git.kraxel.org/qemu tags/egl-20211105-pull-request
>>>>>>>
>>>>>>> for you to fetch changes up to
>>>>>>> 1350ff156b25be65c599ecca9957ce6726c6d383:
>>>>>>>
>>>>>>> ui/gtk-egl: blitting partial guest fb to the proper scanout
>>>>>>> surface
>>>>>>> (2021-11-05 12:29:44 +0100)
>>>>>>>
>>>>>>> ----------------------------------------------------------------
>>>>>>> gtk: a collection of egl fixes.
>>>>>>>
>>>>>>> ----------------------------------------------------------------
>>>>>>>
>>>>>>> Dongwon Kim (6):
>>>>>>> virtio-gpu: splitting one extended mode guest fb into n-scanouts
>>>>>>> ui/gtk-egl: un-tab and re-tab should destroy egl surface and
>>>>>>> context
>>>>>>> ui/gtk-egl: make sure the right context is set as the current
>>>>>>> ui/gtk-egl: guest fb texture needs to be regenerated when
>>>>>>> reinitializing egl
>>>>>>> ui/gtk: gd_draw_event returns FALSE when no cairo surface is bound
>>>>>>> ui/gtk-egl: blitting partial guest fb to the proper scanout
>>>>>>> surface
>>>>>>>
>>>>>>> include/hw/virtio/virtio-gpu.h | 5 +++--
>>>>>>> include/ui/console.h | 4 ++++
>>>>>>> hw/display/virtio-gpu-udmabuf-stubs.c | 3 ++-
>>>>>>> hw/display/virtio-gpu-udmabuf.c | 22 ++++++++++++++--------
>>>>>>> hw/display/virtio-gpu.c | 4 ++--
>>>>>>> ui/egl-helpers.c | 25
>>>>>>> +++++++++++++++++++++----
>>>>>>> ui/gtk-egl.c | 10 ++++++++++
>>>>>>> ui/gtk.c | 23 +++++++++++++++++++++++
>>>>>>> 8 files changed, 79 insertions(+), 17 deletions(-)
>>>>>>
>>>>>> Applied, thanks.
>>>>>
>>>>> Ubuntu 18.04.4 LTS:
>>>>>
>>>>> ui/gtk-egl.c:159:13: error: implicit declaration of function
>>>>> 'egl_dmabuf_release_texture' is invalid in C99
>>>>> [-Werror,-Wimplicit-function-declaration]
>>>>> egl_dmabuf_release_texture(vc->gfx.guest_fb.dmabuf);
>>>>> ^
>>>>> ui/gtk-egl.c:159:13: error: this function declaration is not a prototype
>>>>> [-Werror,-Wstrict-prototypes]
>>>>> 2 errors generated.
>>>>>
>>>>> https://app.travis-ci.com/gitlab/qemu-project/qemu/builds/241272737
>>>>>
>>>>
>>>> This seems to fix but I have no clue whether it is correct:
>>>>
>>>> -- >8 --
>>>> diff --git a/ui/gtk-egl.c b/ui/gtk-egl.c
>>>> index f2026e4b5c9..45cb67712df 100644
>>>> --- a/ui/gtk-egl.c
>>>> +++ b/ui/gtk-egl.c
>>>> @@ -156,8 +156,10 @@ void gd_egl_refresh(DisplayChangeListener *dcl)
>>>> surface_gl_create_texture(vc->gfx.gls, vc->gfx.ds);
>>>> }
>>>> +#ifdef CONFIG_GBM
>>>> if (vc->gfx.guest_fb.dmabuf) {
>>>> egl_dmabuf_release_texture(vc->gfx.guest_fb.dmabuf);
>>>> gd_egl_scanout_dmabuf(dcl, vc->gfx.guest_fb.dmabuf);
>>>> }
>>>> +#endif
>>>> }
>>>>
>>>> ---
>>>
>>> I see the same error here trying to build QEMU git master on Debian
>>> Buster (oldstable). The fix looks reasonable to me in that it matches
>>> the CONFIG_GBM guards around other similar functions and the resulting
>>> binary appears to work, so:
>>>
>>> Reviewed-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
>>
>> Thank you, I'll post a formal patch then.
>>
>> I wonder why this got merged while this configuration is covered in
>> Travis-CI. Is it that we have a too high failure rate than we don't
>> use it anymore?
>
> I've not looked at travis in ages what matters is GitLab CI, and there
> the problem is that we're building ubuntu 20.04 not 18.04 so didn't
> catch the older problem.
>
> I don't know what Mark sees a problem on Debian Buster though, as we
> successfully built there AFAICT
>
> https://gitlab.com/qemu-project/qemu/-/jobs/1756202449
That's interesting. Below is the configure output from my laptop which was generated
with "./configure --target-list=m68k-softmmu --prefix=/home/build/rel-qemu-git":
The Meson build system
Version: 0.59.3
Source dir: /home/build/src/qemu/git/qemu
Build dir: /home/build/src/qemu/git/qemu/build
Build type: native build
Project name: qemu
Project version: 6.1.50
C compiler for the host machine: cc -m64 -mcx16 (gcc 8.3.0 "cc (Debian 8.3.0-6) 8.3.0")
C linker for the host machine: cc -m64 -mcx16 ld.bfd 2.31.1
Host machine cpu family: x86_64
Host machine cpu: x86_64
Program sh found: YES (/bin/sh)
Program python3 found: YES (/usr/bin/python3)
WARNING: Broken python installation detected. Python files installed by Meson might
not be found by python interpreter.
C++ compiler for the host machine: c++ -m64 -mcx16 (gcc 8.3.0 "c++ (Debian 8.3.0-6)
8.3.0")
C++ linker for the host machine: c++ -m64 -mcx16 ld.bfd 2.31.1
Program cgcc found: NO
Library m found: YES
Run-time dependency threads found: YES
Library util found: YES
Run-time dependency appleframeworks found: NO (tried framework)
Found pkg-config: /usr/bin/pkg-config (0.29)
Run-time dependency pixman-1 found: YES 0.36.0
Run-time dependency zlib found: YES 1.2.11
Has header "libaio.h" : YES
Library aio found: YES
Run-time dependency liburing found: NO (tried pkgconfig)
Run-time dependency libxml-2.0 found: NO (tried pkgconfig)
Run-time dependency libnfs found: YES 3.0.0
Run-time dependency appleframeworks found: NO (tried framework)
Run-time dependency libseccomp found: YES 2.3.3
Has header "cap-ng.h" : YES
Library cap-ng found: YES
Run-time dependency xkbcommon found: YES 0.8.2
Has header "libvdeplug.h" : YES
Library vdeplug found: YES
Run-time dependency libpulse found: YES 12.2
Run-time dependency alsa found: YES 1.1.8
Run-time dependency jack found: NO (tried pkgconfig)
Run-time dependency spice-protocol found: YES 0.12.14
Run-time dependency spice-server found: YES 0.14.0
Library rt found: YES
Library dl found: YES
Checking for function "dlopen" with dependency -ldl: YES
Run-time dependency libiscsi found: YES 1.18.0
Run-time dependency libzstd found: NO (tried pkgconfig)
Run-time dependency virglrenderer found: NO (tried pkgconfig)
Run-time dependency libcurl found: YES 7.64.0
Run-time dependency libudev found: YES 241
Library mpathpersist found: NO
Run-time dependency ncursesw found: YES 6.1.20181013
Has header "brlapi.h" : YES
Library brlapi found: YES
Run-time dependency sdl2 found: YES 2.0.9
Run-time dependency sdl2_image found: NO (tried pkgconfig)
Library rados found: YES
Has header "rbd/librbd.h" : YES
Library rbd found: YES
Run-time dependency glusterfs-api found: YES 7.5.5
Has header "bzlib.h" : NO
Has header "lzfse.h" : NO
Has header "sys/soundcard.h" : YES
Run-time dependency gbm found: NO (tried pkgconfig)
Dependency gnutls found: NO found 3.6.7 but need: '>=3.6.14'
Run-time dependency gnutls found: NO (tried pkgconfig)
Run-time dependency gnutls found: YES 3.6.7
libgcrypt-config found: YES (/usr/bin/libgcrypt-config) 1.8.4
Run-time dependency libgcrypt found: YES 1.8.4
Run-time dependency gtk+-3.0 found: YES 3.24.5
Run-time dependency gtk+-x11-3.0 found: YES 3.24.5
Run-time dependency vte-2.91 found: YES 0.54.2
Run-time dependency x11 found: YES 1.6.7
Run-time dependency libpng found: YES 1.6.36
Run-time dependency libjpeg found: YES 1.5.2
Has header "sasl/sasl.h" : YES
Library sasl2 found: YES
Has header "security/pam_appl.h" : NO
Has header "snappy-c.h" : NO
Has header "lzo/lzo1x.h" : NO
Run-time dependency libcacard found: YES 2.6.1
Run-time dependency u2f-emu found: NO (tried pkgconfig)
Run-time dependency libusbredirparser-0.5 found: YES 0.8.0
Run-time dependency libusb-1.0 found: YES 1.0.22
Run-time dependency libpmem found: NO (tried pkgconfig)
Run-time dependency libdaxctl found: NO (tried pkgconfig)
Run-time dependency libtasn1 found: YES 4.13
Run-time dependency libkeyutils found: NO (tried pkgconfig)
Checking for function "gettid" : NO
Run-time dependency fuse3 found: NO (tried pkgconfig)
Run-time dependency libbpf found: NO (tried pkgconfig)
Has header "sys/epoll.h" : YES
Has header "linux/magic.h" : YES
Has header "valgrind/valgrind.h" : YES
Has header "linux/btrfs.h" : YES
Has header "libdrm/drm.h" : YES
Has header "pty.h" : YES
Has header "sys/disk.h" : NO
Has header "sys/ioccom.h" : NO
Has header "sys/kcov.h" : NO
Checking for function "accept4" : YES
Checking for function "clock_adjtime" : YES
Checking for function "dup3" : YES
Checking for function "fallocate" : YES
Checking for function "posix_fallocate" : YES
Checking for function "posix_memalign" : YES
Checking for function "ppoll" : YES
Checking for function "preadv" : YES
Checking for function "sem_timedwait" with dependency threads: YES
Checking for function "sendfile" : YES
Checking for function "setns" : YES
Checking for function "unshare" : YES
Checking for function "syncfs" : YES
Checking for function "sync_file_range" : YES
Checking for function "timerfd_create" : YES
Checking for function "copy_file_range" : YES
Checking for function "openpty" with dependency -lutil: YES
Checking for function "strchrnul" : YES
Checking for function "system" : YES
Header <byteswap.h> has symbol "bswap_32" : YES
Header <sys/epoll.h> has symbol "epoll_create1" : YES
Header <unistd.h> has symbol "environ" : YES
Header <linux/falloc.h> has symbol "FALLOC_FL_PUNCH_HOLE" : YES
Header <linux/falloc.h> has symbol "FALLOC_FL_KEEP_SIZE" : YES
Header <linux/falloc.h> has symbol "FALLOC_FL_ZERO_RANGE" : YES
Has header "linux/fiemap.h" : YES
Header <linux/fs.h> has symbol "FS_IOC_FIEMAP" : YES
Checking for function "getrandom" : YES
Header <sys/random.h> has symbol "GRND_NONBLOCK" : YES
Header <sys/inotify.h> has symbol "inotify_init" : YES
Header <sys/inotify.h> has symbol "inotify_init1" : YES
Header <sys/uio.h> has symbol "struct iovec" : YES
Header <machine/bswap.h> has symbol "bswap32" : NO
Header <sys/prctl.h> has symbol "PR_SET_TIMERSLACK" : YES
Header <linux/rtnetlink.h> has symbol "IFLA_PROTO_DOWN" : YES
Header <sys/sysmacros.h> has symbol "makedev" : YES
Header <getopt.h> has symbol "optreset" : NO
Header <utmpx.h> has symbol "struct utmpx" : YES
Header <netinet/in.h> has symbol "IPPROTO_MPTCP" : NO
Checking whether type "struct sigevent" has member "sigev_notify_thread_id" : NO
Checking whether type "struct stat" has member "st_atim" : YES
Header <sys/socket.h> has symbol "struct mmsghdr" : YES
Has header "linux/ip.h" : YES
Program scripts/minikconf.py found: YES (/usr/bin/python3
/home/build/src/qemu/git/qemu/scripts/minikconf.py)
Configuring m68k-softmmu-config-target.h using configuration
Configuring m68k-softmmu-config-devices.mak with command
Reading depfile:
/home/build/src/qemu/git/qemu/build/meson-private/m68k-softmmu-config-devices.mak.d
Configuring m68k-softmmu-config-devices.h using configuration
Run-time dependency capstone found: NO (tried pkgconfig)
Configuring capstone-defs.h using configuration
Run-time dependency slirp found: NO (tried pkgconfig)
Configuring libslirp-version.h using configuration
Library fdt found: YES
Configuring config-host.h using configuration
Program scripts/hxtool found: YES (/home/build/src/qemu/git/qemu/scripts/hxtool)
Program scripts/shaderinclude.pl found: YES (/usr/bin/env perl
/home/build/src/qemu/git/qemu/scripts/shaderinclude.pl)
Program scripts/qapi-gen.py found: YES (/usr/bin/python3
/home/build/src/qemu/git/qemu/scripts/qapi-gen.py)
Program scripts/qemu-version.sh found: YES
(/home/build/src/qemu/git/qemu/scripts/qemu-version.sh)
Executing subproject libvhost-user
libvhost-user| Project name: libvhost-user
libvhost-user| Project version: undefined
libvhost-user| C compiler for the host machine: cc -m64 -mcx16 (gcc 8.3.0 "cc (Debian
8.3.0-6) 8.3.0")
libvhost-user| C linker for the host machine: cc -m64 -mcx16 ld.bfd 2.31.1
libvhost-user| Dependency threads found: YES unknown (cached)
libvhost-user| Dependency glib-2.0 found: YES 6.1.50 (overridden)
libvhost-user| Build targets in project: 11
libvhost-user| Subproject libvhost-user finished.
Program cat found: YES (/bin/cat)
Program scripts/decodetree.py found: YES (/usr/bin/python3
/home/build/src/qemu/git/qemu/scripts/decodetree.py)
Program ../scripts/modules/module_block.py found: YES (/usr/bin/python3
/home/build/src/qemu/git/qemu/block/../scripts/modules/module_block.py)
Program ../scripts/block-coroutine-wrapper.py found: YES (/usr/bin/python3
/home/build/src/qemu/git/qemu/block/../scripts/block-coroutine-wrapper.py)
Program scripts/modinfo-collect.py found: YES
(/home/build/src/qemu/git/qemu/scripts/modinfo-collect.py)
Program scripts/modinfo-generate.py found: YES
(/home/build/src/qemu/git/qemu/scripts/modinfo-generate.py)
Program nm found: YES
Program scripts/undefsym.py found: YES (/usr/bin/python3
/home/build/src/qemu/git/qemu/scripts/undefsym.py)
Program scripts/feature_to_c.sh found: YES (/bin/sh
/home/build/src/qemu/git/qemu/scripts/feature_to_c.sh)
Configuring 50-qemu-virtiofsd.json using configuration
Program qemu-keymap found: NO
Program cp found: YES (/bin/cp)
Program sphinx-build-3 sphinx-build found: NO
Program python3 found: YES (/usr/bin/python3)
Program diff found: YES (/usr/bin/diff)
Program dbus-daemon found: YES (/usr/bin/dbus-daemon)
Program /usr/bin/gdbus-codegen found: YES (/usr/bin/gdbus-codegen)
Program initrd-stress.sh found: YES
(/home/build/src/qemu/git/qemu/tests/migration/initrd-stress.sh)
Program xgettext found: YES (/usr/bin/xgettext)
Build targets in project: 423
qemu 6.1.50
Directories
Install prefix : /home/build/rel-qemu-git
BIOS directory : share/qemu
firmware path : /home/build/rel-qemu-git/share/qemu-firmware
binary directory : bin
library directory : lib
module directory : lib/qemu
libexec directory : libexec
include directory : include
config directory : /home/build/rel-qemu-git/etc
local state directory : /home/build/rel-qemu-git/var
Manual directory : share/man
Doc directory : /home/build/rel-qemu-git/share/doc
Build directory : /home/build/src/qemu/git/qemu/build
Source path : /home/build/src/qemu/git/qemu
GIT submodules : ui/keycodemapdb meson
tests/fp/berkeley-testfloat-3 tests/fp/berkeley-softfloat-3 dtc capstone slirp
Host binaries
git : git
make : make
python : /usr/bin/python3 (version: 3.7)
sphinx-build : NO
genisoimage : /usr/bin/genisoimage
smbd : "/usr/sbin/smbd"
Configurable features
Documentation : NO
system-mode emulation : YES
user-mode emulation : NO
block layer : YES
Install blobs : YES
module support : NO
fuzzing support : NO
Audio drivers : pa oss
Trace backends : log
QOM debugging : YES
vhost-kernel support : YES
vhost-net support : YES
vhost-crypto support : YES
vhost-scsi support : YES
vhost-vsock support : YES
vhost-user support : YES
vhost-user-blk server support: YES
vhost-user-fs support : YES
vhost-vdpa support : YES
build guest agent : YES
Compilation
host CPU : x86_64
host endianness : little
C compiler : cc -m64 -mcx16
Host C compiler : cc -m64 -mcx16
C++ compiler : c++ -m64 -mcx16
CFLAGS : -O2 -g
CXXFLAGS : -O2 -g
QEMU_CFLAGS : -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2
-D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -Wstrict-prototypes
-Wredundant-decls -Wundef -Wwrite-strings -Wmissing-prototypes -fno-strict-aliasing
-fno-common -fwrapv -Wold-style-declaration -Wold-style-definition -Wtype-limits
-Wformat-security -Wformat-y2k -Winit-self -Wignored-qualifiers -Wempty-body
-Wnested-externs -Wendif-labels -Wexpansion-to-defined -Wimplicit-fallthrough=2
-Wno-missing-include-dirs -Wno-shift-negative-value -Wno-psabi -fstack-protector-strong
QEMU_LDFLAGS : -Wl,--warn-common -Wl,-z,relro -Wl,-z,now
-fstack-protector-strong
profiler : NO
link-time optimization (LTO) : NO
PIE : YES
static build : NO
malloc trim support : YES
membarrier : NO
debug stack usage : NO
mutex debugging : NO
memory allocator : system
avx2 optimization : YES
avx512f optimization : NO
gprof enabled : NO
gcov : NO
thread sanitizer : NO
CFI support : NO
strip binaries : YES
sparse : NO
mingw32 support : NO
m68k tests : m68k-linux-gnu-gcc via debian-m68k-cross
Targets and accelerators
KVM support : NO
HAX support : NO
HVF support : NO
WHPX support : NO
NVMM support : NO
Xen support : YES
xen ctrl version : 41100
TCG support : YES
TCG backend : native (x86_64)
TCG plugins : YES
TCG debug enabled : NO
target list : m68k-softmmu
default devices : YES
out of process emulation : YES
Block layer support
coroutine backend : ucontext
coroutine pool : YES
Block whitelist (rw) :
Block whitelist (ro) :
Use block whitelist in tools : NO
VirtFS support : YES
build virtiofs daemon : YES
Live block migration : YES
replication support : YES
bochs support : YES
cloop support : YES
dmg support : YES
qcow v1 support : YES
vdi support : YES
vvfat support : YES
qed support : YES
parallels support : YES
FUSE exports : NO
Crypto
TLS priority : "NORMAL"
GNUTLS support : YES 3.6.7
GNUTLS crypto : NO
libgcrypt : YES 1.8.4
nettle : NO
crypto afalg : NO
rng-none : NO
Linux keyring : YES
Dependencies
SDL support : YES
SDL image support : NO
GTK support : YES
pixman : YES 0.36.0
VTE support : YES 0.54.2
slirp support : internal
libtasn1 : YES 4.13
PAM : NO
iconv support : YES
curses support : YES
virgl support : NO
curl support : YES 7.64.0
Multipath support : NO
VNC support : YES
VNC SASL support : YES
VNC JPEG support : YES 1.5.2
VNC PNG support : YES 1.6.36
OSS support : YES
ALSA support : YES 1.1.8
PulseAudio support : YES 12.2
JACK support : NO
brlapi support : YES
vde support : YES
netmap support : NO
l2tpv3 support : YES
Linux AIO support : YES
Linux io_uring support : NO
ATTR/XATTR support : YES
RDMA support : NO
PVRDMA support : NO
fdt support : internal
libcap-ng support : YES
bpf support : NO
spice protocol support : YES 0.12.14
spice server support : YES 0.14.0
rbd support : YES
xfsctl support : YES
smartcard support : YES 2.6.1
U2F support : NO
libusb : YES 1.0.22
usb net redir : YES 0.8.0
OpenGL support : YES
GBM : NO
libiscsi support : YES 1.18.0
libnfs support : YES 3.0.0
seccomp support : YES 2.3.3
GlusterFS support : YES 7.5.5
TPM support : YES
libssh support : NO
lzo support : NO
snappy support : NO
bzip2 support : NO
lzfse support : NO
zstd support : NO
NUMA host support : YES
libxml2 : NO
capstone : internal
libpmem support : NO
libdaxctl support : NO
libudev : YES 241
FUSE lseek : NO
Subprojects
libvhost-user : YES
Found ninja-1.8.2 at /usr/bin/ninja
I tend to use GTK most of the time, so maybe related to SDL/virgl?
ATB,
Mark.
On 11/8/21 10:41, Mark Cave-Ayland wrote:
> On 08/11/2021 09:22, Daniel P. Berrangé wrote:
>
>> On Mon, Nov 08, 2021 at 09:17:19AM +0100, Philippe Mathieu-Daudé wrote:
>>> +Thomas & Daniel for Travis-CI
>>>
>>> On 11/8/21 09:12, Mark Cave-Ayland wrote:
>>>> On 05/11/2021 18:49, Philippe Mathieu-Daudé wrote:
>>>>> On 11/5/21 19:26, Philippe Mathieu-Daudé wrote:
>>>>>> On 11/5/21 18:13, Richard Henderson wrote:
>>>>>>> On 11/5/21 7:30 AM, Gerd Hoffmann wrote:
>>>>>>>> The following changes since commit
>>>>>>>> b1fd92137e4d485adeec8e9f292f928ff335b76c:
>>>>>>>>
>>>>>>>> Merge remote-tracking branch
>>>>>>>> 'remotes/bonzini/tags/for-upstream'
>>>>>>>> into staging (2021-11-03 13:07:30 -0400)
>>>>>>>>
>>>>>>>> are available in the Git repository at:
>>>>>>>>
>>>>>>>> git://git.kraxel.org/qemu tags/egl-20211105-pull-request
>>>>>>>>
>>>>>>>> for you to fetch changes up to
>>>>>>>> 1350ff156b25be65c599ecca9957ce6726c6d383:
>>>>>>>>
>>>>>>>> ui/gtk-egl: blitting partial guest fb to the proper scanout
>>>>>>>> surface
>>>>>>>> (2021-11-05 12:29:44 +0100)
>>>>>>>>
>>>>>>>> ----------------------------------------------------------------
>>>>>>>> gtk: a collection of egl fixes.
>>>>>>>>
>>>>>>>> ----------------------------------------------------------------
>>>>>>>>
>>>>>>>> Dongwon Kim (6):
>>>>>>>> virtio-gpu: splitting one extended mode guest fb into
>>>>>>>> n-scanouts
>>>>>>>> ui/gtk-egl: un-tab and re-tab should destroy egl surface and
>>>>>>>> context
>>>>>>>> ui/gtk-egl: make sure the right context is set as the current
>>>>>>>> ui/gtk-egl: guest fb texture needs to be regenerated when
>>>>>>>> reinitializing egl
>>>>>>>> ui/gtk: gd_draw_event returns FALSE when no cairo surface
>>>>>>>> is bound
>>>>>>>> ui/gtk-egl: blitting partial guest fb to the proper scanout
>>>>>>>> surface
>>>>>>>>
>>>>>>>> include/hw/virtio/virtio-gpu.h | 5 +++--
>>>>>>>> include/ui/console.h | 4 ++++
>>>>>>>> hw/display/virtio-gpu-udmabuf-stubs.c | 3 ++-
>>>>>>>> hw/display/virtio-gpu-udmabuf.c | 22
>>>>>>>> ++++++++++++++--------
>>>>>>>> hw/display/virtio-gpu.c | 4 ++--
>>>>>>>> ui/egl-helpers.c | 25
>>>>>>>> +++++++++++++++++++++----
>>>>>>>> ui/gtk-egl.c | 10 ++++++++++
>>>>>>>> ui/gtk.c | 23
>>>>>>>> +++++++++++++++++++++++
>>>>>>>> 8 files changed, 79 insertions(+), 17 deletions(-)
>>>>>>>
>>>>>>> Applied, thanks.
>>>>>>
>>>>>> Ubuntu 18.04.4 LTS:
>>>>>>
>>>>>> ui/gtk-egl.c:159:13: error: implicit declaration of function
>>>>>> 'egl_dmabuf_release_texture' is invalid in C99
>>>>>> [-Werror,-Wimplicit-function-declaration]
>>>>>> egl_dmabuf_release_texture(vc->gfx.guest_fb.dmabuf);
>>>>>> ^
>>>>>> ui/gtk-egl.c:159:13: error: this function declaration is not a
>>>>>> prototype
>>>>>> [-Werror,-Wstrict-prototypes]
>>>>>> 2 errors generated.
>>>>>>
>>>>>> https://app.travis-ci.com/gitlab/qemu-project/qemu/builds/241272737
>>>>>>
>>>>>
>>>>> This seems to fix but I have no clue whether it is correct:
>>>>>
>>>>> -- >8 --
>>>>> diff --git a/ui/gtk-egl.c b/ui/gtk-egl.c
>>>>> index f2026e4b5c9..45cb67712df 100644
>>>>> --- a/ui/gtk-egl.c
>>>>> +++ b/ui/gtk-egl.c
>>>>> @@ -156,8 +156,10 @@ void gd_egl_refresh(DisplayChangeListener *dcl)
>>>>> surface_gl_create_texture(vc->gfx.gls, vc->gfx.ds);
>>>>> }
>>>>> +#ifdef CONFIG_GBM
>>>>> if (vc->gfx.guest_fb.dmabuf) {
>>>>> egl_dmabuf_release_texture(vc->gfx.guest_fb.dmabuf);
>>>>> gd_egl_scanout_dmabuf(dcl, vc->gfx.guest_fb.dmabuf);
>>>>> }
>>>>> +#endif
>>>>> }
>>>>>
>>>>> ---
>>>>
>>>> I see the same error here trying to build QEMU git master on Debian
>>>> Buster (oldstable). The fix looks reasonable to me in that it matches
>>>> the CONFIG_GBM guards around other similar functions and the resulting
>>>> binary appears to work, so:
>>>>
>>>> Reviewed-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
>>>
>>> Thank you, I'll post a formal patch then.
>>>
>>> I wonder why this got merged while this configuration is covered in
>>> Travis-CI. Is it that we have a too high failure rate than we don't
>>> use it anymore?
>>
>> I've not looked at travis in ages what matters is GitLab CI, and there
>> the problem is that we're building ubuntu 20.04 not 18.04 so didn't
>> catch the older problem.
>>
>> I don't know what Mark sees a problem on Debian Buster though, as we
>> successfully built there AFAICT
>>
>> https://gitlab.com/qemu-project/qemu/-/jobs/1756202449
>
> That's interesting. Below is the configure output from my laptop which
> was generated with "./configure --target-list=m68k-softmmu
> --prefix=/home/build/rel-qemu-git":
...
> Run-time dependency gbm found: NO (tried pkgconfig)
> Dependencies
> SDL support : YES
> SDL image support : NO
> GTK support : YES
> pixman : YES 0.36.0
> iconv support : YES
> curses support : YES
> virgl support : NO
> OpenGL support : YES
> GBM : NO
>
> I tend to use GTK most of the time, so maybe related to SDL/virgl?
© 2016 - 2024 Red Hat, Inc.