Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

narrow down video issues in nightlies #31

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

psaavedra
Copy link
Member

@psaavedra psaavedra commented Feb 20, 2025

I’m investigating a performance regression in WPEWebKit main compared to 2.46 on RPi5 (Mesa).

The summary is the following:

Debugging So Far:

  • No hardware decoding support for RPi5, but playback was working fine up to 2.46 with GST DMABuf video sink enabled.
  • Checked: USE(GBM) is ON.
  • Verified: s_isDMABufDisabled in initializeDMABufAvailability() is set correctly (to 0).
  • It is not clear to me whether the video is producing DMA-BUF or GL textures. The fact that the DMABUF sink is enabled does not necessarily mean that DMABUF is being used; it depends on the video.

@psaavedra psaavedra self-assigned this Feb 20, 2025
Copy link

💿️ Image for wpe-nightly-raspberrypi5 (ref: 52f181c).

@psaavedra psaavedra force-pushed the psaavedra/narrow-down-video-issue branch 2 times, most recently from c308b0e to 97ff797 Compare February 21, 2025 11:34
Copy link

💿️ Image for wpe-nightly-raspberrypi5 (ref: 97ff797).

Copy link

💿️ Image for wpe-nightly-raspberrypi5 (ref: 2e7e969).

@psaavedra psaavedra force-pushed the psaavedra/narrow-down-video-issue branch from 2e7e969 to 1582447 Compare February 24, 2025 13:45
Copy link

💿️ Image for wpe-nightly-raspberrypi5 (ref: 1582447).

@psaavedra
Copy link
Member Author

http://people.igalia.com/psaavedra/gst-wpe-nightly_debug_logs.tgz and https://people.igalia.com/psaavedra/gst-wpe-nightly_debug_logs_v2.tgz -> this is the full GST debug info running this scenario:

# Lower rank of AV1 decoder to disable it
export GST_PLUGIN_FEATURE_RANK="avdec_av1:0"

# Set directory for GStreamer debug output
export GST_DEBUG_DUMP_DOT_DIR=/tmp/gst-out/  
export GST_DEBUG='*:DEBUG'  
unset WEBKIT_GST_DISABLE_GL_SINK   # == export WEBKIT_GST_DISABLE_GL_SINK=1

context: running versions:

  • mesa 24.0.7 (24.0 was originally released on Jan 2024)
  • gst: 1.24.12 (1.24.0 was originally released on 4 March 2024)
  • webkit: nightly version (a todays's commit)

btw, it the non working scenario: there is a slightly improvement in gst 1.24 (9FPS) comparing to 1.22 (FPS)

@psaavedra
Copy link
Member Author

@psaavedra psaavedra force-pushed the psaavedra/narrow-down-video-issue branch 2 times, most recently from 76fa1ed to e8fb83e Compare February 25, 2025 17:46
Copy link

💿️ Image for wpe-nightly-raspberrypi5 (ref: e8fb83e).

Copy link

github-actions bot commented Mar 5, 2025

💿️ Image for wpe-nightly-raspberrypi5 (ref: 43d9852).

Copy link

github-actions bot commented Mar 5, 2025

💿️ Image for wpe-nightly-raspberrypi3-mesa (ref: 43d9852).

@psaavedra psaavedra force-pushed the psaavedra/narrow-down-video-issue branch from 43d9852 to 8d42f99 Compare March 5, 2025 13:23
Copy link

github-actions bot commented Mar 5, 2025

💿️ Image for wpe-nightly-raspberrypi5 (ref: 8d42f99).

Copy link

github-actions bot commented Mar 5, 2025

💿️ Image for wpe-nightly-raspberrypi3-mesa (ref: 8d42f99).

@psaavedra psaavedra force-pushed the psaavedra/narrow-down-video-issue branch from 8d42f99 to 827cd07 Compare March 5, 2025 18:31
Copy link

github-actions bot commented Mar 5, 2025

💿️ Image for wpe-nightly-raspberrypi3-mesa (ref: 827cd07).

Copy link

github-actions bot commented Mar 5, 2025

💿️ Image for wpe-nightly-raspberrypi5 (ref: 827cd07).

@psaavedra psaavedra force-pushed the psaavedra/narrow-down-video-issue branch from 827cd07 to 35d9065 Compare March 6, 2025 06:53
Copy link

github-actions bot commented Mar 6, 2025

💿️ Image for wpe-nightly-raspberrypi5 (ref: 35d9065).

Copy link

github-actions bot commented Mar 6, 2025

💿️ Image for wpe-nightly-raspberrypi3-mesa (ref: 35d9065).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant