01:32karenw: Are there any rules on what `stride` should be for a shm_pool wl_buffer?
01:32karenw: I was assuming width * 4 was enough (XRGB8888), but I'm just getting invalid width, height or stride (1310x740, 5240)
01:39soreau: https://gitlab.freedesktop.org/wayland/wayland/-/blob/main/src/wayland-shm.c?ref_type=heads#L225-L227
01:45karenw: soreau: Thanks. I think all of those things were true. The actual error seems to be destroying the wl_buffer too early instead of waiting for the release() event.
01:45karenw: Which feel like the wrong error tbh, maybe there's no better one.
01:55karenw: Yeah I have no idea how kwin ends up sending me that error looking at the source. wut.
05:02DemiMarie: Is “libwayland busy-loops if a message exceeds the buffer size limit” something that should get a CVE?
05:04DemiMarie: karenw: Check with Sway or another compositor that uses wlroots. wlroots gets wl_shm stride checking correct, which most compositors do not.
05:34wlb: weston/main: Benjamin Herrenschmidt * Add support for FreeRDP 3.x https://gitlab.freedesktop.org/wayland/weston/commit/89f3a8a71e0c .gitlab-ci.yml .gitlab-ci/debian-install.sh libweston/backend-rdp/meson.build libweston/backend-rdp/rdp.c libweston/backend-rdp/rdp.h libweston/backend-rdp/rdpclip.c
05:34wlb: weston Merge request !1589 merged \o/ (Add support for FreeRDP 3.x https://gitlab.freedesktop.org/wayland/weston/-/merge_requests/1589)
05:37daniels: benh: oops, maybe - feel free to fix if so
07:55wlb: weston/main: Lukasz Czechowski * vnc: Allow to disable Transport Layer Security https://gitlab.freedesktop.org/wayland/weston/commit/58a0abcb5872 frontend/main.c include/libweston/backend-vnc.h libweston/backend-vnc/vnc.c
07:55wlb: weston/main: Lukasz Czechowski * vnc: Enable password authentication in non-TLS mode https://gitlab.freedesktop.org/wayland/weston/commit/3ecdd0486c3f libweston/backend-vnc/vnc.c
07:55wlb: weston/main: Lukasz Czechowski * man: Update documentation of weston-vnc https://gitlab.freedesktop.org/wayland/weston/commit/2fe23ad38938 man/weston-vnc.man
07:55wlb: weston Merge request !1555 merged \o/ (vnc: Disable TLS encryption if server cert and key are not set https://gitlab.freedesktop.org/wayland/weston/-/merge_requests/1555)
08:15jadahl: any need for a w-p release now with the screen copy thing in there? there is also the toplevel icons but seems noone has asked for a release for it yet (would also have been nice to have the undefined wl_buffer issue to be fixed)
08:17whot: jadahl: it'd be nice to get the tablet rel dials into a release, right now w-p is blocking that
08:18jadahl: oh, it also sits there waiting? had missed that
08:26wlb: weston Merge request !1596 opened by Marius Vlad (mvlad) build: bump to version 13.0.91 for the alpha release https://gitlab.freedesktop.org/wayland/weston/-/merge_requests/1596
08:36daniels: jadahl: yeah, don't see any reason to not do one
08:37jadahl: +1, sorry for the delays. I blame the summer.
08:39wlb: weston/main: Marius Vlad * build: bump to version 13.0.91 for the alpha release https://gitlab.freedesktop.org/wayland/weston/commit/49278973c025 meson.build
08:39wlb: weston Merge request !1596 merged \o/ (build: bump to version 13.0.91 for the alpha release https://gitlab.freedesktop.org/wayland/weston/-/merge_requests/1596)
08:42daniels: no prob :)
08:43wlb: weston New tag: 13.0.91 https://gitlab.freedesktop.org/wayland/weston/tags/13.0.91
09:28any1: jadahl: No need for a release yet. I'm actually trying to figure out if I can maybe improve it a little bit to make frame pacing more predictable.
09:29any1: But if I can't figure it out soon, we'll just have to bump the protocol version when I do
09:57jadahl: any1: sure, can wait a bit
09:57jadahl: I just re-read what whot said and the rel dials isn't yet merged so that is not a reason to get anything out. toplevel icons could be, but as I said, if the undefined buffer semantics can be fixed, that'd be a nice bonus
12:40wlb: wayland.freedesktop.org Merge request !82 opened by Marius Vlad (mvlad) releases: add weston 13.0.91 release https://gitlab.freedesktop.org/wayland/wayland.freedesktop.org/-/merge_requests/82
12:42wlb: wayland.freedesktop.org/main: Marius Vlad * releases: add weston 13.0.91 release https://gitlab.freedesktop.org/wayland/wayland.freedesktop.org/commit/2cceff4889e8 releases.html
12:42wlb: wayland.freedesktop.org Merge request !82 merged \o/ (releases: add weston 13.0.91 release https://gitlab.freedesktop.org/wayland/wayland.freedesktop.org/-/merge_requests/82)
13:37wlb: wayland-protocols Merge request !328 closed (ext-image-copy-capture-v1: Allow capturing without buffer)
13:44any1: jadahl: I turns out that I just had a bit of a fixation that's been resolved now. image-copy-capture is good to go. There's no need to rush it out though. The wlroots implementation hasn't been reviewed yet.
14:54wlb: weston Merge request !1597 opened by Kirill Primak (vyivel) clients: don't redraw on xdg_toplevel.configure https://gitlab.freedesktop.org/wayland/weston/-/merge_requests/1597
15:07jadahl: any1: alright! day mostly ran out of time, I'll wait for tomorrow anyway
20:07Company: 1100fps in kwin @200% with HDR
20:07Company: 900fps without
20:07Company: 1850fps without color-management support
20:08Company: I think the 1100fps thing is because I'm running out of memory bandwidth, and HDR uses 16bit
20:09Company: the 900fps is because of the extra offscreen and the gamma 2.2 conversion
20:10Company: (note: that is pretty much exclusively GTK times, compositors just throw away the buffers, the benchmark just spams them with that many attach(); commit(); cycles)
20:11Company: mutter also does 1850
20:12Company: and forcing 16bit via GDK_DEBUG=high-depth also goes down to 1100
20:12Company: and I'm typing in the wrong channel
20:13Company: zamundaaa[m], mclasen: ^^^ (people I'm aiming for are here though)