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

Crash on Sway master and hardware acceleration #13255

Closed
stacyharper opened this issue Jan 9, 2024 · 3 comments
Closed

Crash on Sway master and hardware acceleration #13255

stacyharper opened this issue Jan 9, 2024 · 3 comments

Comments

@stacyharper
Copy link

Important Information

  • mpv version
mpv 0.37.0-818ce7c51a6b9179307950e919983e0909942098 Copyright © 2000-2023 mpv/MPlayer/mplayer2 projects
libplacebo version: v6.338.1
FFmpeg version: 6.1
FFmpeg library versions:
   libavutil       58.29.100
   libavcodec      60.31.102
   libavformat     60.16.100
   libswscale      7.5.100
   libavfilter     9.12.100
   libswresample   4.12.100

-- sway version

sway version 1.9-dev-c5fd8c05 (Jan  9 2024, branch 'master')

Reproduction steps

Build and install Sway master, play a video with hardware acceleration.

Probably related to swaywm/sway#7897, and the drop of wl_drm?

Here my full command is mpv -v --no-config --hwdec=auto-safe seedbox/animes/One\ Piece\ KAI\ Saga\ 02\ Baroque\ Works\ VOSTFR\ DVDrip\ FanKai-Knuckalex/One\ Piece\ Kai\ 10\ -\ Arc\ Royaume\ de\ Drum\ -\ part\ 2.mp4 > out.log 2>&1

Expected behavior

It works

Actual behavior

Segmentation fault

Log file

out.log

@llyyr
Copy link
Contributor

llyyr commented Jan 9, 2024

Not our bug, as seen from the sway issue. Should be fixed by intel/libva#790

You can also just revert swaywm/sway@7e69a70 temporarily

@sfan5
Copy link
Member

sfan5 commented Jan 9, 2024

It still should not be segfaulting however (unless that's literally libva doing it).
Do you have a backtrace?

@llyyr
Copy link
Contributor

llyyr commented Jan 9, 2024

Yes, it is libva doing it intel/libva#788

I guess we can add a check here, but shouldn't need to
https://github.com/mpv-player/mpv/blob/master/video/out/hwdec/hwdec_vaapi.c#L52-L57

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

No branches or pull requests

3 participants