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

Release 1.8.4 with Wayland fixes #132

Open
yol opened this issue Nov 2, 2017 · 10 comments
Open

Release 1.8.4 with Wayland fixes #132

yol opened this issue Nov 2, 2017 · 10 comments

Comments

@yol
Copy link
Contributor

yol commented Nov 2, 2017

Hi,

in the next few months we would like to release Kodi with native Wayland support. libva-1 is sadly broken on Wayland in all currently available public releases as the only release that includes fixes is 2.0.0. Realistically though, we cannot expect all current distributions to immediately jump to libva-2 as it would lead to recompilation of lots of packages because of the API/ABI changes - so we cannot depend on libva-2 in Kodi.

Instead, it would be nice to have the following patches applied in a public release of the libva-1 series (they should all apply cleanly to 1.8.3 I think) so we could depend on that:

The first two are absolutely necessary, the other two are nice-to-have. It really should not be a lot of work so I'm hoping for a positive reply :-) Feel free to ask if you need more information on the issues we're facing.

cc @wsnipex

@yol
Copy link
Contributor Author

yol commented Nov 2, 2017

Note: I guess the API version would have to be bumped to 0.41 so we could detect the presence of the fixes.

@fritsch
Copy link

fritsch commented Nov 11, 2017

@xhaihao what's your opinion here. With e.g. Ubuntu running wayland by default, but still using vaapi 1.x I consider this a bugfix.

@chivakker what's your take on that?

@chivakker
Copy link
Contributor

There's no plan to cut a 1.8.x release. IIUC, Ubuntu can handle these patches when they package vaapi which I guess would be 1.8.3.x

@yol
Copy link
Contributor Author

yol commented Nov 11, 2017

There's no plan to cut a 1.8.x release. IIUC, Ubuntu can handle these patches when they package vaapi which I guess would be 1.8.3.x

The problem remains that we somehow have to detect if the libva version is recent enough, because older versions will cause problems/crashes so playback via libva needs to be disabled for them. If Ubuntu and/or other distributions (it's not only about Ubuntu after all) decide to patch 1.8.3, we still will not have any programmatic way to decide whether the libva available on the system is safe for usage with Wayland.

@wsnipex
Copy link

wsnipex commented Nov 12, 2017

Not releasing a fixed 1.8.x version means most distros will have broken vaapi on wayland, which is quickly becoming the default windowing system.
This is not a really big problem for Kodi on ubuntu, since we can easily provide a patched packaged on our PPAs, but all other distros would have to do the same.

@xhaihao
Copy link
Contributor

xhaihao commented Nov 17, 2017

I merged all wayland fixes you mentioned into v1.8-branch and I will add a new tag for these fixes. Thx

@chivakker
Copy link
Contributor

Before tagging I would like to have an analysis of other possibilities. Because the request is not just a tag but an actual bump on va version.

Can Kodi have an experimental flag where it plumbs everything to work on libva-2.0 and potentially carry a copy of libva-2.0 if distros are not ready?

@wsnipex
Copy link

wsnipex commented Nov 20, 2017

Kodi already compiles and runs with libva 2.0. We could also add a copy of it, but it would be rather pointless in regard to distro packaging.
Most distros have a very strict policy to not allow bundled libs, so they will either
1.) patch it out
2.) not package or upgrade kodi

@yol
Copy link
Contributor Author

yol commented Nov 24, 2017

@chivakker Having a copy of libva-2.0 would also mean that you have to bundle all the drivers (at the very least intel and mesa), which is really not how this is intended to work.

@xhaihao As @chivakker correctly pointed out, we would still need the va version (ideally both package and API) bumped.

@kwizart
Copy link

kwizart commented Dec 6, 2017

Having libva-intel-driver-1.8.4 was also mentioned at :
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4724#c3

There would be an additional commit needed for backport into 1.8x branch
intel/intel-vaapi-driver@9d66570

Thx

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

No branches or pull requests

6 participants