-
Notifications
You must be signed in to change notification settings - Fork 32
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
probems with you vaapi patch for chromium #2
Comments
Part of the problem might be that I've switched to using gn, and haven't been updating the gyp files. I'll take a look shortly. |
Can you confirm that the following files are being compiled? It looks like they should based on the patch you're using:
|
yes. with my patch these files it has been builded, but with you patch no sorry for the late greetings |
Ah, I see what you mean. I'll need to add back in the part of the patch that modifies the gyp files as well. |
I've updated the patch, but it may or may not work yet (I can't check as I've switched to using GN). Can you test it and see if it works? |
|
Updated again, can you check now? |
|
Fixed again. |
in build process! |
Builded OK. but vaapi not work :(
same as my r13 patch on older version of chromium :( i'm not sure if is a problem of GYP build system need time for update my script to adapt to GN (and i no sure if all of my "hacks" works ok wit GN) using nvidia 367.27 with libglvnd by nvidia package greetings |
There's another user reporting Nvidia drivers not working, and I don't know why that is. I don't think I can figure out why the error is occurring, as the main error appears to be in driver-specific code. |
@saiarcot895 I am using latest version of chromium-dev from @sl1pkn07 with your patch & I can confirm that VAAPI doesn't work, it crashes with the same error that he had. Not only that, but it breaks the entire GPU process. chrome://gpu reports "GPU process was unable to boot: GPU access is disabled in chrome://settings." |
@sl1pkn07 Can you make a build with debugging symbols included (at least |
roger -g1 is ninja option, right? |
GYP option, I think. You should be able to add it into the CFLAGS. |
okere |
:( |
That's...odd. Building with debugging symbols shouldn't cause a linker error. On another note, it looks like you're building the Debug configuration; I'm not sure if that's intentional or not. It is possible to get debugging symbols without doing a Debug build. I think adding |
tnx for the sugestion (I knew I forgot something :( ) in this run also now added '-Gconfig=Debug' (http://zurlinux.com/?p=180) |
nope. same error with -Dlinux_strip_binary=0, -Dremove_webcore_debug_symbols=0 and -Gconfig=Debug . makepkg (archlinux packager) adds '-g -fvar-tracking-assignments' into CFLAGS/CXXFLAGS when set options=('debug') (actived in both attempts) the changes: https://sl1pkn07.wtf/paste/view/f20a3175 |
Can you verify that a Release build still works with the changes made? Also, what is |
I change the cpp flags because if not spaw a tons message like
(DEBUG_ is active when set options=('debug')) |
nope with active debug in makepkg, -Gconfig=Release, -Dremove_webcore_debug_symbols=0, -Dlinux_strip_binary=0, export CPPFLAGS="${CPPFLAGS/2/0}"
all cases, include avobe test, used a bundled clang (builded, not binaries) |
That error means that the clang binary they are using doesn't support the Also, I don't know if the |
finally!: |
@saiarcot895 That's all fine, I'll take a look at merging your changes back into my version. |
with you patch (https://github.com/saiarcot895/chromium-ubuntu-build/blob/master/debian/patches/enable_vaapi_on_linux.diff) (with fix paths for my distro)
https://github.com/saiarcot895/chromium-ubuntu-build/blob/master/debian/patches/specify-max-resolution.patch is only for 1920x1088. my screen is 4K. is fixable? |
@sl1pkn07 Is the output you pasted with the second patch? Both are needed. As for the screen size, yes, you can change the max resolution there. I don't know what impact it'll have on hardware that doesn't support 4K decoding. |
only with first (why is not merged in one?) mm that last patch is only for i965(as i understand in https://cgit.freedesktop.org/vaapi/intel-driver/commit/?id=9a20d6c34cb65e5b85dd16d6c8b3a215c5972b18), exist equivalent for libva-vdpau-driver? how can get the max libva resolution I can handle? |
I have the two parts separated because the second isn't an issue with Chromium itself, and it has been fixed in vaapi. You'll need both patches. I'm not sure if the |
With @sjnewbury patch for 55, applying to 54, I get this error on launching chromium. Even if I use my old version of
|
@sjnewbury is not a ffmpeg glich, because for eaxple fail with all GIF/videos in twitter. without patch all runs fine steps: close all chromium process
without patch, the messages and warnings gone, but the GIF/videos on twitter playback fine (apply both patches ( https://github.com/saiarcot895/chromium-ubuntu-build/blob/master/debian/patches/enable_vaapi_on_linux.diff and https://github.com/saiarcot895/chromium-ubuntu-build/blob/master/debian/patches/specify-max-resolution.patch. both untouched except change the paths of libav drivers on bpf_gpu_policy_linux.cc) |
Ive managed to apply patch https://aur.archlinux.org/packages/chromium-vaapi/ onto chromium 53.0.2785.116 and it runs fine. My issue is playback of 4k h264 files, they stutter a lot. device specification is fine, runs same 4k video smooth in mpv with vaapi hwdec. What could be the issue? I noticed the cpu load dropped by alot after the patch so it runs with the va drivers and decodes with vaapi. |
On Mon, 2016-10-17 at 17:49 -0700, samcv wrote:
I'll see if I can update my 54 patch with the changes saiarcot895 made. |
On Mon, 2016-10-24 at 11:38 -0700, zepto88 wrote:
|
On Wed, 2016-10-19 at 07:16 -0700, Gustavo Alvarez wrote:
|
I found my problem, obviously i thought the problem was inside chromium but instead it was the window manager. When i turned off the compositor i have a steady 30 fps :) |
@saiarcot895 please ping on my email, big thx ! please ? :) |
Hello, Through the thread, I've seen work only being done for Intel iGPU and Nvidia VDPAU. How about AMD GPUs? Is there any working VAAPI patch for ArchLinux on AMD hardware? |
@zepto88 saiarcot895 mentioned earlier that there is some maximum resolution in the patch which specifies what resolution can be decoded. It is set to 1920x1080 I think. Maybe that should be changed in the builds? I have the same issue with 4K videos. |
Everyone subscribed to this thread, Note that there's been a change in the hardware video decoding patch. An Intel contributor is working on committing a patch to compile in hardware video decoding by default. I've taken his patch and am using it instead of the current patch. Currently, the dev PPA has a build (for Ubuntu 16.04 and newer) with the new patch included, and the beta PPA will have a build soon. What this will mean for you is that you will need to manually enable hardware video decoding from the about:flags page. Otherwise, it will not be used. Additionally, there's been a report of this working for AMD as well, so @LHorace, you may want to try this. |
not for me (nvidia with privative drivers). i'm not sure if with nouveau works
one question. with that patch, the "libva-vdpau-driver-chromium" from @sjnewbury is still needed? greetings |
On Thu, 2017-08-17 at 08:38 -0700, Saikrishna Arcot wrote:
Everyone subscribed to this thread,
Note that there's been a change in the hardware video decoding patch.
An Intel contributor is working on committing a patch to compile in
hardware video decoding by default. I've taken his patch and am using
it instead of the current patch. Currently, the dev PPA has a build
(for Ubuntu 16.04 and newer) with the new patch included, and the
beta PPA will have a build soon.
What this will mean for you is that you will need to manually enable
hardware video decoding from the about:flags page. Otherwise, it will
not be used.
Additionally, there's been a report of this working for AMD as well,
so @LHorace, you may want to try this.
This is fantastic news! :-)
|
Great! Although I am not quite sure if it's working? I am on chromium-dev 62.0.3192.0-1 and I did enable the two flags. At first glance, don't appear to be any change in CPU usage, watching a youtube video? No crash and/or video artifacts. |
There is no build of 62.0.3192.0 yet; you probably mean 62.0.3178.0. It seemed to be working for me, at least based on the VAAPI library info being printed out to the console. |
@LHorace @saiarcot895 I just found out it appears video acceleration has been disabled by default. I actually had to use '--enable-accelerated-video' on the command line (or you could enable it in chrome://flags) to actually activate hardware decoding, despite positive va-stuff being printed. CPU usage was high before I enabled it. You can easily check this, because with every hw-accelerated video started the chromium would output an additional (at least is does on mine).
|
@saiarcot895 Copy & paste bud, maybe ArchLinux have the version mixed up, not on Linux atm. Wait, I just remembered, I am using archlinuxcn. Chromium Dev 62.0.3192.0-1. I also see he has, chromium dev-vaapi 61.0.3141.7-1. I'll give that one a shot too. @dagraver Actually, I did see --enable-accelerate-video in my process list after I enabled the two flags. One thing for sure, at least for now, I don't seem to be experiencing any screen tearing. I'll check if I see that information next time I run chromium! |
I've updated my Gentoo chromium ebuild to use the new in-development patch. |
Closing this issue. |
Hi. i'm the maintainer of the chromium-dev package in Archlinux AUR user repository
https://aur.archlinux.org/packages/chromium-dev
i want to use you patch https://github.com/saiarcot895/chromium-ubuntu-build/blob/master/debian/patches/enable_vaapi_on_linux.diff in my package
in older version, i use you patch from launchpad repository, but because I'am not coder, that patch don't work well
for example, this https://aur.archlinux.org/cgit/aur.git/tree/enable_vaapi_on_linux-r13.diff?h=chromium-dev
is more or less sync with your older patch, but now, i want sync with your latest patch (with little changes in some paths for adapt for chromium build with GYP_DEFINES)
for example:
and
all build ok, except the last build rule: (linker)
(complete log of the last rule: https://sl1pkn07.wtf/chromium_log.log)
can you help me?
this is my complete build script: https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=chromium-dev
greetings
The text was updated successfully, but these errors were encountered: