Branch: refs/heads/master Home: https://github.com/qemu/qemu Commit: 41582637b1577c261f19634bdd1bb5e7ed6e258d https://github.com/qemu/qemu/commit/41582637b1577c261f19634bdd1bb5e7ed6e258d Author: Sergii Zasenko <ser...@zasenko.name> Date: 2024-05-14 (Tue, 14 May 2024)
Changed paths: M qemu-options.hx M ui/vnc.c Log Message: ----------- Allow UNIX socket option for VNC websocket - Remove unix socket option limitation for VNC websocket - Reflect websocket option changes in documentation Signed-off-by: Sergii Zasenko <ser...@zasenko.name> Reviewed-by: Marc-André Lureau <marcandre.lur...@redhat.com> Message-Id: <20230724100353.16628-1-ser...@zasenko.name> Commit: 77bf310084dad38b3a2badf01766c659056f1cf2 https://github.com/qemu/qemu/commit/77bf310084dad38b3a2badf01766c659056f1cf2 Author: Dongwon Kim <dongwon....@intel.com> Date: 2024-05-14 (Tue, 14 May 2024) Changed paths: M ui/gtk-egl.c M ui/gtk-gl-area.c Log Message: ----------- ui/gtk: Draw guest frame at refresh cycle Draw routine needs to be manually invoked in the next refresh if there is a scanout blob from the guest. This is to prevent a situation where there is a scheduled draw event but it won't happen bacause the window is currently in inactive state (minimized or tabified). If draw is not done for a long time, gl_block timeout and/or fence timeout (on the guest) will happen eventually. v2: Use gd_gl_area_draw(vc) in gtk-gl-area.c Suggested-by: Vivek Kasireddy <vivek.kasire...@intel.com> Cc: Gerd Hoffmann <kra...@redhat.com> Cc: Marc-André Lureau <marcandre.lur...@redhat.com> Cc: Daniel P. Berrangé <berra...@redhat.com> Signed-off-by: Dongwon Kim <dongwon....@intel.com> Acked-by: Marc-André Lureau <marcandre.lur...@redhat.com> Message-Id: <20240426225059.3871283-1-dongwon....@intel.com> Commit: e4e62514e3cc2fc9dbae44af8b80f61c730beab4 https://github.com/qemu/qemu/commit/e4e62514e3cc2fc9dbae44af8b80f61c730beab4 Author: Dongwon Kim <dongwon....@intel.com> Date: 2024-05-14 (Tue, 14 May 2024) Changed paths: M ui/gtk-egl.c M ui/gtk-gl-area.c M ui/gtk.c Log Message: ----------- ui/gtk: Check if fence_fd is equal to or greater than 0 'fence_fd' needs to be validated always before being referenced And the passing condition should include '== 0' as 0 is a valid value for the file descriptor. Suggested-by: Marc-André Lureau <marcandre.lur...@redhat.com> Reviewed-by: Daniel P. Berrangé <berra...@redhat.com> Cc: Philippe Mathieu-Daudé <phi...@linaro.org> Cc: Daniel P. Berrangé <berra...@redhat.com> Cc: Vivek Kasireddy <vivek.kasire...@intel.com> Signed-off-by: Dongwon Kim <dongwon....@intel.com> Message-Id: <20240508175403.3399895-2-dongwon....@intel.com> Commit: 6e6ae491dad9a7ba813c7f1e70ce972c35c2d4ab https://github.com/qemu/qemu/commit/6e6ae491dad9a7ba813c7f1e70ce972c35c2d4ab Author: Dongwon Kim <dongwon....@intel.com> Date: 2024-05-14 (Tue, 14 May 2024) Changed paths: M include/ui/console.h A include/ui/dmabuf.h A ui/dmabuf.c M ui/meson.build Log Message: ----------- ui/console: new dmabuf.h and dmabuf.c for QemuDmaBuf struct and helpers New header and source files are added for containing QemuDmaBuf struct definition and newly introduced helpers for creating/freeing the struct and accessing its data. v10: Change the license type for both dmabuf.h and dmabuf.c from MIT to GPL to be in line with QEMU's default license v11: -- Added new helpers, qemu_dmabuf_close for closing dmabuf->fd, qemu_dmabuf_dup_fd for duplicating dmabuf->fd (Daniel P. Berrangé <berra...@redhat.com>) -- Let qemu_dmabuf_fee to call qemu_dmabuf_close before freeing the struct to make sure fd is closed. (Daniel P. Berrangé <berra...@redhat.com>) v12: Not closing fd in qemu_dmabuf_free because there are cases fd should still be available even after the struct is destroyed (e.g. virtio-gpu: res->dmabuf_fd). Suggested-by: Marc-André Lureau <marcandre.lur...@redhat.com> Reviewed-by: Marc-André Lureau <marcandre.lur...@redhat.com> Cc: Philippe Mathieu-Daudé <phi...@linaro.org> Cc: Daniel P. Berrangé <berra...@redhat.com> Cc: Vivek Kasireddy <vivek.kasire...@intel.com> Signed-off-by: Dongwon Kim <dongwon....@intel.com> Message-Id: <20240508175403.3399895-3-dongwon....@intel.com> Commit: 6779a3076f295fafe52d43049fa954426c1d594a https://github.com/qemu/qemu/commit/6779a3076f295fafe52d43049fa954426c1d594a Author: Dongwon Kim <dongwon....@intel.com> Date: 2024-05-14 (Tue, 14 May 2024) Changed paths: M hw/display/vhost-user-gpu.c M hw/display/virtio-gpu-udmabuf.c M hw/vfio/display.c M ui/console.c M ui/dbus-console.c M ui/dbus-listener.c M ui/egl-headless.c M ui/egl-helpers.c M ui/gtk-egl.c M ui/gtk-gl-area.c M ui/gtk.c M ui/spice-display.c Log Message: ----------- ui/console: Use qemu_dmabuf_get_..() helpers instead This commit updates all instances where fields within the QemuDmaBuf struct are directly accessed, replacing them with calls to these new helper functions. v6: fix typos in helper names in ui/spice-display.c v7: removed prefix, "dpy_gl_" from all helpers v8: Introduction of helpers was removed as those were already added by the previous commit v11: -- Use new qemu_dmabuf_close() instead of close(qemu_dmabuf_get_fd()). (Daniel P. Berrangé <berra...@redhat.com>) -- Use new qemu_dmabuf_dup_fd() instead of dup(qemu_dmabuf_get_fd()). (Daniel P. Berrangé <berra...@redhat.com>) Suggested-by: Marc-André Lureau <marcandre.lur...@redhat.com> Reviewed-by: Marc-André Lureau <marcandre.lur...@redhat.com> Cc: Philippe Mathieu-Daudé <phi...@linaro.org> Cc: Daniel P. Berrangé <berra...@redhat.com> Cc: Vivek Kasireddy <vivek.kasire...@intel.com> Signed-off-by: Dongwon Kim <dongwon....@intel.com> Message-Id: <20240508175403.3399895-4-dongwon....@intel.com> Commit: fa6426805b124400cfb700b75e8fe4a89dd2ed7a https://github.com/qemu/qemu/commit/fa6426805b124400cfb700b75e8fe4a89dd2ed7a Author: Dongwon Kim <dongwon....@intel.com> Date: 2024-05-14 (Tue, 14 May 2024) Changed paths: M ui/egl-helpers.c M ui/gtk-egl.c M ui/gtk-gl-area.c M ui/gtk.c Log Message: ----------- ui/console: Use qemu_dmabuf_set_..() helpers instead This commit updates all occurrences where these fields were set directly have been updated to utilize helper functions. v7: removed prefix, "dpy_gl_" from all helpers v8: Introduction of helpers was removed as those were already added by the previous commit Suggested-by: Marc-André Lureau <marcandre.lur...@redhat.com> Reviewed-by: Marc-André Lureau <marcandre.lur...@redhat.com> Cc: Philippe Mathieu-Daudé <phi...@linaro.org> Cc: Daniel P. Berrangé <berra...@redhat.com> Cc: Vivek Kasireddy <vivek.kasire...@intel.com> Signed-off-by: Dongwon Kim <dongwon....@intel.com> Message-Id: <20240508175403.3399895-5-dongwon....@intel.com> Commit: c0fcd6334ff673b571ac068f28b6b779bdade8a2 https://github.com/qemu/qemu/commit/c0fcd6334ff673b571ac068f28b6b779bdade8a2 Author: Dongwon Kim <dongwon....@intel.com> Date: 2024-05-14 (Tue, 14 May 2024) Changed paths: M hw/display/vhost-user-gpu.c M hw/display/virtio-gpu-udmabuf.c M hw/vfio/display.c M include/hw/vfio/vfio-common.h M include/hw/virtio/virtio-gpu.h M ui/dbus-listener.c Log Message: ----------- ui/console: Use qemu_dmabuf_new() and free() helpers instead This commit introduces utility functions for the creation and deallocation of QemuDmaBuf instances. Additionally, it updates all relevant sections of the codebase to utilize these new utility functions. v7: remove prefix, "dpy_gl_" from all helpers qemu_dmabuf_free() returns without doing anything if input is null (Daniel P. Berrangé <berra...@redhat.com>) call G_DEFINE_AUTOPTR_CLEANUP_FUNC for qemu_dmabuf_free() (Daniel P. Berrangé <berra...@redhat.com>) v8: Introduction of helpers was removed as those were already added by the previous commit v9: set dmabuf->allow_fences to 'true' when dmabuf is created in virtio_gpu_create_dmabuf()/virtio-gpu-udmabuf.c removed unnecessary spaces were accidently added in the patch, 'ui/console: Use qemu_dmabuf_new() a...' v11: Calling qemu_dmabuf_close was removed as closing dmabuf->fd will be done in qemu_dmabuf_free anyway. (Daniel P. Berrangé <berra...@redhat.com>) v12: --- Calling qemu_dmabuf_close separately as qemu_dmabuf_free doesn't do it. --- 'dmabuf' is now allocated space so it should be freed at the end of dbus_scanout_texture v13: --- Immediately free dmabuf after it is released to prevent possible leaking of the ptr (Marc-André Lureau <marcandre.lur...@redhat.com>) --- Use g_autoptr macro to define *dmabuf for auto clean up instead of calling qemu_dmabuf_free (Marc-André Lureau <marcandre.lur...@redhat.com>) v14: --- (vhost-user-gpu) Change qemu_dmabuf_free back to g_clear_pointer as it was done because of some misunderstanding (v13). --- (vhost-user-gpu) g->dmabuf[m->scanout_id] needs to be set to NULL to prevent freed dmabuf to be accessed again in case if(fd==-1)break; happens (before new dmabuf is allocated). Otherwise, it would cause invalid memory access when the same function is executed. Also NULL check should be done before qemu_dmabuf_close (it asserts dmabuf!=NULL.). (Marc-André Lureau <marcandre.lur...@redhat.com>) Suggested-by: Marc-André Lureau <marcandre.lur...@redhat.com> Cc: Philippe Mathieu-Daudé <phi...@linaro.org> Cc: Daniel P. Berrangé <berra...@redhat.com> Cc: Vivek Kasireddy <vivek.kasire...@intel.com> Signed-off-by: Dongwon Kim <dongwon....@intel.com> Message-Id: <20240508175403.3399895-6-dongwon....@intel.com> Commit: db81dd6bdc3425145398c6634c329c2182144d6d https://github.com/qemu/qemu/commit/db81dd6bdc3425145398c6634c329c2182144d6d Author: Dongwon Kim <dongwon....@intel.com> Date: 2024-05-14 (Tue, 14 May 2024) Changed paths: M include/ui/dmabuf.h M ui/dmabuf.c Log Message: ----------- ui/console: move QemuDmaBuf struct def to dmabuf.c To complete privatizing process of QemuDmaBuf, QemuDmaBuf struct def is moved to dmabuf.c Suggested-by: Marc-André Lureau <marcandre.lur...@redhat.com> Reviewed-by: Marc-André Lureau <marcandre.lur...@redhat.com> Cc: Philippe Mathieu-Daudé <phi...@linaro.org> Cc: Daniel P. Berrangé <berra...@redhat.com> Cc: Vivek Kasireddy <vivek.kasire...@intel.com> Signed-off-by: Dongwon Kim <dongwon....@intel.com> Message-Id: <20240508175403.3399895-7-dongwon....@intel.com> Commit: 36b8e6b4e17cb1e45c902689d1366fa93571bfbd https://github.com/qemu/qemu/commit/36b8e6b4e17cb1e45c902689d1366fa93571bfbd Author: hikalium <hikal...@hikalium.com> Date: 2024-05-14 (Tue, 14 May 2024) Changed paths: M ui/gtk.c M ui/trace-events Log Message: ----------- ui/gtk: Add gd_motion_event trace event Add gd_motion_event trace event for making it easy to debug gd_motion_event related issues. Signed-off-by: hikalium <hikal...@hikalium.com> Reviewed-by: Marc-André Lureau <marcandre.lur...@redhat.com> Message-Id: <20240512111435.30121-2-hikal...@hikalium.com> Commit: 37e91415018db3656b46cdea8f9e4d47b3ff130d https://github.com/qemu/qemu/commit/37e91415018db3656b46cdea8f9e4d47b3ff130d Author: hikalium <hikal...@hikalium.com> Date: 2024-05-14 (Tue, 14 May 2024) Changed paths: M ui/gtk.c Log Message: ----------- ui/gtk: Fix mouse/motion event scaling issue with GTK display backend Remove gtk_widget_get_scale_factor() usage from the calculation of the motion events in the GTK backend to make it work correctly on environments that have `gtk_widget_get_scale_factor() != 1`. This scale factor usage had been introduced in the commit f14aab420c and at that time the window size was used for calculating the things and it was working correctly. However, in the commit 2f31663ed4 the logic switched to use the widget size instead of window size and because of the change the usage of scale factor becomes invalid (since widgets use `vc->gfx.scale_{x, y}` for scaling). Tested on Crostini on ChromeOS (15823.51.0) with an external display. Fixes: 2f31663ed4 ("ui/gtk: use widget size for cursor motion event") Fixes: f14aab420c ("ui: fix incorrect pointer position on highdpi with gtk") Signed-off-by: hikalium <hikal...@hikalium.com> Acked-by: Marc-André Lureau <marcandre.lur...@redhat.com> Message-Id: <20240512111435.30121-3-hikal...@hikalium.com> Commit: 2e701e6785cd8cc048c608751c6e4f6253c67ab6 https://github.com/qemu/qemu/commit/2e701e6785cd8cc048c608751c6e4f6253c67ab6 Author: Bernhard Beschow <shen...@gmail.com> Date: 2024-05-14 (Tue, 14 May 2024) Changed paths: M ui/sdl2.c Log Message: ----------- ui/sdl2: Allow host to power down screen By default, SDL disables the screen saver which prevents the host from powering down the screen even if the screen is locked. This results in draining the battery needlessly when the host isn't connected to a wall charger. Fix that by enabling the screen saver. Signed-off-by: Bernhard Beschow <shen...@gmail.com> Acked-by: Marc-André Lureau <marcandre.lur...@redhat.com> Message-ID: <20240512095945.1879-1-shen...@gmail.com> Commit: 265aad58e9cab31d0e69c374ec2efcede7fa8881 https://github.com/qemu/qemu/commit/265aad58e9cab31d0e69c374ec2efcede7fa8881 Author: Richard Henderson <richard.hender...@linaro.org> Date: 2024-05-15 (Wed, 15 May 2024) Changed paths: M hw/display/vhost-user-gpu.c M hw/display/virtio-gpu-udmabuf.c M hw/vfio/display.c M include/hw/vfio/vfio-common.h M include/hw/virtio/virtio-gpu.h M include/ui/console.h A include/ui/dmabuf.h M qemu-options.hx M ui/console.c M ui/dbus-console.c M ui/dbus-listener.c A ui/dmabuf.c M ui/egl-headless.c M ui/egl-helpers.c M ui/gtk-egl.c M ui/gtk-gl-area.c M ui/gtk.c M ui/meson.build M ui/sdl2.c M ui/spice-display.c M ui/trace-events M ui/vnc.c Log Message: ----------- Merge tag 'ui-pull-request' of https://gitlab.com/marcandre.lureau/qemu into staging UI: small fixes and improvements # -----BEGIN PGP SIGNATURE----- # # iQJQBAABCAA6FiEEh6m9kz+HxgbSdvYt2ujhCXWWnOUFAmZDZEAcHG1hcmNhbmRy # ZS5sdXJlYXVAcmVkaGF0LmNvbQAKCRDa6OEJdZac5UxvD/9HWbB8JdbV8lNCLePT # a6RUWSqLyP/cV0FCw9URYgAjAYROO966dZopCH7+Sz6goC8tk3IFUoqL0LbtZQjK # zMNueGRbwJj0iGMxFG4wuWBpBF6Dzc4sh90TF3XWSE8PMpWsDY+sP3VRu4sP1qu7 # OmCGTuSwNUugxazPLxvbTpLMnco9b+asAGlAU6WqpcURmia7XN7dBLGzfQ9vMxuc # L5od+pPGfcxuj3ETMG+5OQlIZH1lmX3465LajkUDVxffNfznqMVDYyo4sKNW5KOY # u420AoACeVsANWce1Aw2ekj1ETsvqxj23RClNIgdpDbMsGk9eM6eS+6vRctcM6z4 # wMH6GAKKI3AWj7Q6qY4096bcdNmYD/GOs9dgswqYjf+JLzEVcI1dHQ36K124nKH0 # t+9t3UUx1NBMwAp+EEN94W1ClwOZ0zvapS8zNaf76KIi9Eb4vrIyOlzdTM7SU4kC # CQ4Tu9MBB5WIqzhsVtIH36zDBasgAU8DCtpelDY1AJiODGiQbfZi4yo8eEiQMS1s # onixsXa7zyCCpmxwkYmvF54RbZFlPXxmdvu0jYxKddbEuTGX/Y3qvDAWv1kz6iJS # iGmYtokfkv86XBCTGTAb3QEmFfOWcLnPc59Gg0TF3zyzY3q05nU/qjuIlgYedR/o # TnnNYbyqXumojRCd69Dyy3THEg== # =SW9v # -----END PGP SIGNATURE----- # gpg: Signature made Tue 14 May 2024 03:16:48 PM CEST # gpg: using RSA key 87A9BD933F87C606D276F62DDAE8E10975969CE5 # gpg: issuer "marcandre.lur...@redhat.com" # gpg: Good signature from "Marc-André Lureau <marcandre.lur...@redhat.com>" [full] # gpg: aka "Marc-André Lureau <marcandre.lur...@gmail.com>" [full] * tag 'ui-pull-request' of https://gitlab.com/marcandre.lureau/qemu: ui/sdl2: Allow host to power down screen ui/gtk: Fix mouse/motion event scaling issue with GTK display backend ui/gtk: Add gd_motion_event trace event ui/console: move QemuDmaBuf struct def to dmabuf.c ui/console: Use qemu_dmabuf_new() and free() helpers instead ui/console: Use qemu_dmabuf_set_..() helpers instead ui/console: Use qemu_dmabuf_get_..() helpers instead ui/console: new dmabuf.h and dmabuf.c for QemuDmaBuf struct and helpers ui/gtk: Check if fence_fd is equal to or greater than 0 ui/gtk: Draw guest frame at refresh cycle Allow UNIX socket option for VNC websocket Signed-off-by: Richard Henderson <richard.hender...@linaro.org> Compare: https://github.com/qemu/qemu/compare/3d48b6b687c5...265aad58e9ca To unsubscribe from these emails, change your notification settings at https://github.com/qemu/qemu/settings/notifications