[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-commits] [qemu/qemu] 415826: Allow UNIX socket option for VNC webs
|
From: |
Richard Henderson |
|
Subject: |
[Qemu-commits] [qemu/qemu] 415826: Allow UNIX socket option for VNC websocket |
|
Date: |
Tue, 14 May 2024 23:58:16 -0700 |
Branch: refs/heads/staging
Home: https://github.com/qemu/qemu
Commit: 41582637b1577c261f19634bdd1bb5e7ed6e258d
https://github.com/qemu/qemu/commit/41582637b1577c261f19634bdd1bb5e7ed6e258d
Author: Sergii Zasenko <sergii@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 <sergii@zasenko.name>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Message-Id: <20230724100353.16628-1-sergii@zasenko.name>
Commit: 77bf310084dad38b3a2badf01766c659056f1cf2
https://github.com/qemu/qemu/commit/77bf310084dad38b3a2badf01766c659056f1cf2
Author: Dongwon Kim <dongwon.kim@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.kasireddy@intel.com>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Cc: Marc-André Lureau <marcandre.lureau@redhat.com>
Cc: Daniel P. Berrangé <berrange@redhat.com>
Signed-off-by: Dongwon Kim <dongwon.kim@intel.com>
Acked-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Message-Id: <20240426225059.3871283-1-dongwon.kim@intel.com>
Commit: e4e62514e3cc2fc9dbae44af8b80f61c730beab4
https://github.com/qemu/qemu/commit/e4e62514e3cc2fc9dbae44af8b80f61c730beab4
Author: Dongwon Kim <dongwon.kim@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.lureau@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Cc: Philippe Mathieu-Daudé <philmd@linaro.org>
Cc: Daniel P. Berrangé <berrange@redhat.com>
Cc: Vivek Kasireddy <vivek.kasireddy@intel.com>
Signed-off-by: Dongwon Kim <dongwon.kim@intel.com>
Message-Id: <20240508175403.3399895-2-dongwon.kim@intel.com>
Commit: 6e6ae491dad9a7ba813c7f1e70ce972c35c2d4ab
https://github.com/qemu/qemu/commit/6e6ae491dad9a7ba813c7f1e70ce972c35c2d4ab
Author: Dongwon Kim <dongwon.kim@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é <berrange@redhat.com>)
-- Let qemu_dmabuf_fee to call qemu_dmabuf_close before freeing
the struct to make sure fd is closed.
(Daniel P. Berrangé <berrange@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.lureau@redhat.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Cc: Philippe Mathieu-Daudé <philmd@linaro.org>
Cc: Daniel P. Berrangé <berrange@redhat.com>
Cc: Vivek Kasireddy <vivek.kasireddy@intel.com>
Signed-off-by: Dongwon Kim <dongwon.kim@intel.com>
Message-Id: <20240508175403.3399895-3-dongwon.kim@intel.com>
Commit: 6779a3076f295fafe52d43049fa954426c1d594a
https://github.com/qemu/qemu/commit/6779a3076f295fafe52d43049fa954426c1d594a
Author: Dongwon Kim <dongwon.kim@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é <berrange@redhat.com>)
-- Use new qemu_dmabuf_dup_fd() instead of dup(qemu_dmabuf_get_fd()).
(Daniel P. Berrangé <berrange@redhat.com>)
Suggested-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Cc: Philippe Mathieu-Daudé <philmd@linaro.org>
Cc: Daniel P. Berrangé <berrange@redhat.com>
Cc: Vivek Kasireddy <vivek.kasireddy@intel.com>
Signed-off-by: Dongwon Kim <dongwon.kim@intel.com>
Message-Id: <20240508175403.3399895-4-dongwon.kim@intel.com>
Commit: fa6426805b124400cfb700b75e8fe4a89dd2ed7a
https://github.com/qemu/qemu/commit/fa6426805b124400cfb700b75e8fe4a89dd2ed7a
Author: Dongwon Kim <dongwon.kim@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.lureau@redhat.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Cc: Philippe Mathieu-Daudé <philmd@linaro.org>
Cc: Daniel P. Berrangé <berrange@redhat.com>
Cc: Vivek Kasireddy <vivek.kasireddy@intel.com>
Signed-off-by: Dongwon Kim <dongwon.kim@intel.com>
Message-Id: <20240508175403.3399895-5-dongwon.kim@intel.com>
Commit: c0fcd6334ff673b571ac068f28b6b779bdade8a2
https://github.com/qemu/qemu/commit/c0fcd6334ff673b571ac068f28b6b779bdade8a2
Author: Dongwon Kim <dongwon.kim@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é <berrange@redhat.com>)
call G_DEFINE_AUTOPTR_CLEANUP_FUNC for qemu_dmabuf_free()
(Daniel P. Berrangé <berrange@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é <berrange@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.lureau@redhat.com>)
--- Use g_autoptr macro to define *dmabuf for auto clean up instead of
calling qemu_dmabuf_free
(Marc-André Lureau <marcandre.lureau@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.lureau@redhat.com>)
Suggested-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Cc: Philippe Mathieu-Daudé <philmd@linaro.org>
Cc: Daniel P. Berrangé <berrange@redhat.com>
Cc: Vivek Kasireddy <vivek.kasireddy@intel.com>
Signed-off-by: Dongwon Kim <dongwon.kim@intel.com>
Message-Id: <20240508175403.3399895-6-dongwon.kim@intel.com>
Commit: db81dd6bdc3425145398c6634c329c2182144d6d
https://github.com/qemu/qemu/commit/db81dd6bdc3425145398c6634c329c2182144d6d
Author: Dongwon Kim <dongwon.kim@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.lureau@redhat.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Cc: Philippe Mathieu-Daudé <philmd@linaro.org>
Cc: Daniel P. Berrangé <berrange@redhat.com>
Cc: Vivek Kasireddy <vivek.kasireddy@intel.com>
Signed-off-by: Dongwon Kim <dongwon.kim@intel.com>
Message-Id: <20240508175403.3399895-7-dongwon.kim@intel.com>
Commit: 36b8e6b4e17cb1e45c902689d1366fa93571bfbd
https://github.com/qemu/qemu/commit/36b8e6b4e17cb1e45c902689d1366fa93571bfbd
Author: hikalium <hikalium@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 <hikalium@hikalium.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Message-Id: <20240512111435.30121-2-hikalium@hikalium.com>
Commit: 37e91415018db3656b46cdea8f9e4d47b3ff130d
https://github.com/qemu/qemu/commit/37e91415018db3656b46cdea8f9e4d47b3ff130d
Author: hikalium <hikalium@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 <hikalium@hikalium.com>
Acked-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Message-Id: <20240512111435.30121-3-hikalium@hikalium.com>
Commit: 2e701e6785cd8cc048c608751c6e4f6253c67ab6
https://github.com/qemu/qemu/commit/2e701e6785cd8cc048c608751c6e4f6253c67ab6
Author: Bernhard Beschow <shentey@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 <shentey@gmail.com>
Acked-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Message-ID: <20240512095945.1879-1-shentey@gmail.com>
Commit: 265aad58e9cab31d0e69c374ec2efcede7fa8881
https://github.com/qemu/qemu/commit/265aad58e9cab31d0e69c374ec2efcede7fa8881
Author: Richard Henderson <richard.henderson@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.lureau@redhat.com"
# gpg: Good signature from "Marc-André Lureau <marcandre.lureau@redhat.com>"
[full]
# gpg: aka "Marc-André Lureau <marcandre.lureau@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.henderson@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
- [Qemu-commits] [qemu/qemu] 415826: Allow UNIX socket option for VNC websocket,
Richard Henderson <=