xf86-video-intel: Upgrade to recent git
v2 cover xf86-video-intel: Upgrade to recent git
Changes since v1:
* Reverted the earlier workaround for gst-player
Original cover letter:
Currently xvideo is broken on skylake because our xf86-video-intel is way too old (it's the last snapshot but that was 21 months ago). The problem can be reproduced with gtk-play (select goom visualization from context menu) or testxvideo from xvideo-tests.
Most distros have moved to a recent git commit already (many have also started using the modesetting driver instead of xf86-video-intel for modern intel hardware). Unfortunately we did not do the same earlier in this release.
I'm not happy to send a change like this at this point of the release but backporting the xvideo fix alone does not seem feasible. I've done my usual smoke testing for graphics changes but this of course affects all Intel graphics hardware, most of which I can't test with.
Thanks,
Jussi
The following changes since commit 3a73fe0efcb7aeefcca7011bba887caad03d4d03:
beaglebone.conf: produce wic images for Beaglebone (2016-10-01 21:45:57 +0100)
are available in the git repository at:
git://git.yoctoproject.org/poky-contrib jku/xf86-video-intel
http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=jku/xf86-video-intel
Jussi Kukkonen (2):
xf86-video-intel: Upgrade to recent git
Revert "gst-player: Disable visualizations"
.../0001-gen8-Fix-the-YUV-RGB-shader.patch | 65 --------------------
...-call-to-PixmapSyncDirtyHelper-broken-by-.patch | 30 ----------
.../xf86-video-intel/disable-x11-dri3.patch | 6 +-
...-against-ABI-breakage-in-recent-versions-.patch | 70 ----------------------
.../xorg-driver/xf86-video-intel/udev-fstat.patch | 59 ------------------
...o-intel_2.99.917.bb => xf86-video-intel_git.bb} | 13 ++--
.../0001-Disable-visualizations-by-default.patch | 36 -----------
.../recipes-multimedia/gstreamer/gst-player_git.bb | 1 -
8 files changed, 10 insertions(+), 270 deletions(-)
delete mode 100644 meta/recipes-graphics/xorg-driver/xf86-video-intel/0001-gen8-Fix-the-YUV-RGB-shader.patch
delete mode 100644 meta/recipes-graphics/xorg-driver/xf86-video-intel/0001-uxa-fix-the-call-to-PixmapSyncDirtyHelper-broken-by-.patch
delete mode 100644 meta/recipes-graphics/xorg-driver/xf86-video-intel/sna-Protect-against-ABI-breakage-in-recent-versions-.patch
delete mode 100644 meta/recipes-graphics/xorg-driver/xf86-video-intel/udev-fstat.patch
rename meta/recipes-graphics/xorg-driver/{xf86-video-intel_2.99.917.bb => xf86-video-intel_git.bb} (81%)
delete mode 100644 meta/recipes-multimedia/gstreamer/gst-player/0001-Disable-visualizations-by-default.patchSignature · 1 line
2.1.4
PATCH v2 1/2 xf86-video-intel: Upgrade to recent git
Upgrade from the latest snapshot to a recent git revision. Without this xvideo does not work on skylake: Backporting the specific fixes turned out to be too complex.
Remove patches that are in upstream already, rebase disable-x11-dri3.patch.
Fixes [YOCTO #10041]
---
.../0001-gen8-Fix-the-YUV-RGB-shader.patch | 65 --------------------
...-call-to-PixmapSyncDirtyHelper-broken-by-.patch | 30 ----------
.../xf86-video-intel/disable-x11-dri3.patch | 6 +-
...-against-ABI-breakage-in-recent-versions-.patch | 70 ----------------------
.../xorg-driver/xf86-video-intel/udev-fstat.patch | 59 ------------------
...o-intel_2.99.917.bb => xf86-video-intel_git.bb} | 13 ++--
6 files changed, 10 insertions(+), 233 deletions(-)
delete mode 100644 meta/recipes-graphics/xorg-driver/xf86-video-intel/0001-gen8-Fix-the-YUV-RGB-shader.patch
delete mode 100644 meta/recipes-graphics/xorg-driver/xf86-video-intel/0001-uxa-fix-the-call-to-PixmapSyncDirtyHelper-broken-by-.patch
delete mode 100644 meta/recipes-graphics/xorg-driver/xf86-video-intel/sna-Protect-against-ABI-breakage-in-recent-versions-.patch
delete mode 100644 meta/recipes-graphics/xorg-driver/xf86-video-intel/udev-fstat.patch
rename meta/recipes-graphics/xorg-driver/{xf86-video-intel_2.99.917.bb => xf86-video-intel_git.bb} (81%)meta/recipes-graphics/xorg-driver/xf86-video-intel/0001-gen8-Fix-the-YUV-RGB-shader.patch deleted +0 -65
@@ -1,65 +0,0 @@ -From c43617b739e358064396912c7a7a8028ca91d201 Mon Sep 17 00:00:00 2001 -From: =?UTF-8?q?Ville=20Syrj=C3=A4l=C3=A4?= <[email protected]> -Date: Thu, 16 Apr 2015 20:40:39 +0300 -Subject: [PATCH] gen8: Fix the YUV->RGB shader -MIME-Version: 1.0 -Content-Type: text/plain; charset=UTF-8 -Content-Transfer-Encoding: 8bit - -Use the correct register (Yn_01) with first half of the -Y samples instead of using the register (Yn_23) with the -second half twice when computing the green channel. - -Also use the Yn_01 register name instead of Yn for the red -channel as well, just for a bit of extra consistency. - -Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89807 -Signed-off-by: Ville Syrjälä <[email protected]> -Tested-by: Chris Wilson <[email protected]> - -Upstream-Status: Backport -Signed-off-by: Saul Wold <[email protected]> - ---- - src/render_program/exa_wm_yuv_rgb.g8a | 4 ++-- - src/render_program/exa_wm_yuv_rgb.g8b | 2 +- - 2 files changed, 3 insertions(+), 3 deletions(-) - -diff --git a/src/render_program/exa_wm_yuv_rgb.g8a b/src/render_program/exa_wm_yuv_rgb.g8a -index 7def093..34973ba 100644 ---- a/src/render_program/exa_wm_yuv_rgb.g8a -+++ b/src/render_program/exa_wm_yuv_rgb.g8a -@@ -76,7 +76,7 @@ add (16) Cbn<1>F Cb<8,8,1>F -0.501961F { compr align1 }; - /* - * R = Y + Cr * 1.596 - */ --mov (8) acc0<1>F Yn<8,8,1>F { compr align1 }; -+mov (8) acc0<1>F Yn_01<8,8,1>F { compr align1 }; - mac.sat(8) src_sample_r_01<1>F Crn_01<8,8,1>F 1.596F { compr align1 }; - - mov (8) acc0<1>F Yn_23<8,8,1>F { compr align1 }; -@@ -84,7 +84,7 @@ mac.sat(8) src_sample_r_23<1>F Crn_23<8,8,1>F 1.596F { compr align1 }; - /* - * G = Crn * -0.813 + Cbn * -0.392 + Y - */ --mov (8) acc0<1>F Yn_23<8,8,1>F { compr align1 }; -+mov (8) acc0<1>F Yn_01<8,8,1>F { compr align1 }; - mac (8) acc0<1>F Crn_01<8,8,1>F -0.813F { compr align1 }; - mac.sat(8) src_sample_g_01<1>F Cbn_01<8,8,1>F -0.392F { compr align1 }; - -diff --git a/src/render_program/exa_wm_yuv_rgb.g8b b/src/render_program/exa_wm_yuv_rgb.g8b -index 4494953..2cd6fc4 100644 ---- a/src/render_program/exa_wm_yuv_rgb.g8b -+++ b/src/render_program/exa_wm_yuv_rgb.g8b -@@ -6,7 +6,7 @@ - { 0x80600048, 0x21c03ae8, 0x3e8d02c0, 0x3fcc49ba }, - { 0x00600001, 0x24003ae0, 0x008d0320, 0x00000000 }, - { 0x80600048, 0x21e03ae8, 0x3e8d02e0, 0x3fcc49ba }, -- { 0x00600001, 0x24003ae0, 0x008d0320, 0x00000000 }, -+ { 0x00600001, 0x24003ae0, 0x008d0300, 0x00000000 }, - { 0x00600048, 0x24003ae0, 0x3e8d02c0, 0xbf5020c5 }, - { 0x80600048, 0x22003ae8, 0x3e8d0340, 0xbec8b439 }, - { 0x00600001, 0x24003ae0, 0x008d0320, 0x00000000 }, --- -2.5.0 -
meta/recipes-graphics/xorg-driver/xf86-video-intel/0001-uxa-fix-the-call-to-PixmapSyncDirtyHelper-broken-by-.patch deleted +0 -30
@@ -1,30 +0,0 @@ -From 2c5063938cc809f624e56efd4673041fa8141e81 Mon Sep 17 00:00:00 2001 -From: Martin Peres <[email protected]> -Date: Thu, 9 Jul 2015 11:26:38 +0300 -Subject: [PATCH] uxa: fix the call to PixmapSyncDirtyHelper, broken by - xserver's 90db5ed - -[ickle: switch to HAS_DIRTYTRACKING_ROTATION as suggested by Dave Airlie] -Signed-off-by: Martin Peres <[email protected]> - -Upstream-Status: Backport -Signed-off-by: Saul Wold <[email protected]> ---- - src/compat-api.h | 4 ++++ - 1 file changed, 4 insertions(+) - -diff --git a/src/compat-api.h b/src/compat-api.h -index aa93bee..293e9d7 100644 ---- a/src/compat-api.h -+++ b/src/compat-api.h -@@ -247,3 +247,7 @@ static inline void FreePixmap(PixmapPtr pixmap) - #endif - - #endif -+ -+#if HAS_DIRTYTRACKING_ROTATION -+#define PixmapSyncDirtyHelper(d, dd) PixmapSyncDirtyHelper(d) -+#endif --- -2.5.0 -
meta/recipes-graphics/xorg-driver/xf86-video-intel/disable-x11-dri3.patch +3 -3
@@ -12,6 +12,6 @@ diff --git a/configure.ac b/configure.ac index bd654f3..78a0e0d 100644 --- a/configure.ac +++ b/configure.ac -@@ -251 +251 @@ fi --PKG_CHECK_MODULES(X11_DRI3, [xcb-dri3 xcb-sync xcb-present x11-xcb xshmfence x11 xrender xext libdrm], [x11_dri3="yes"], [x11_dri3="no"]) -+#PKG_CHECK_MODULES(X11_DRI3, [xcb-dri3 xcb-sync xcb-present x11-xcb xshmfence x11 xrender xext libdrm], [x11_dri3="yes"], [x11_dri3="no"]) +@@ -279 +279 @@ fi +-PKG_CHECK_MODULES(X11_DRI3, [xcb-dri3 xcb-sync xcb-xfixes xcb-present x11-xcb xshmfence x11 xcomposite xdamage xrender xrandr xxf86vm xext libdrm], [x11_dri3="yes"], [x11_dri3="no"]) ++#PKG_CHECK_MODULES(X11_DRI3, [xcb-dri3 xcb-sync xcb-xfixes xcb-present x11-xcb xshmfence x11 xcomposite xdamage xrender xrandr xxf86vm xext libdrm], [x11_dri3="yes"], [x11_dri3="no"])
meta/recipes-graphics/xorg-driver/xf86-video-intel/sna-Protect-against-ABI-breakage-in-recent-versions-.patch deleted +0 -70
70 lines not on this page · raw message
meta/recipes-graphics/xorg-driver/xf86-video-intel/udev-fstat.patch deleted +0 -59
59 lines not on this page · raw message
meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb renamed from meta/recipes-graphics/xorg-driver/xf86-video-intel_2.99.917.bb +7 -6
@@ -9,12 +9,13 @@ Infrastructure (DRI)." LIC_FILES_CHKSUM = "file://COPYING;md5=8730ad58d11c7bbad9a7066d69f7808e" -SRC_URI += "file://disable-x11-dri3.patch \ - file://always_include_xorg_server.h.patch \ - file://sna-Protect-against-ABI-breakage-in-recent-versions-.patch \ - file://udev-fstat.patch \ - file://0001-uxa-fix-the-call-to-PixmapSyncDirtyHelper-broken-by-.patch \ - file://0001-gen8-Fix-the-YUV-RGB-shader.patch \ +SRCREV = "8f33f80100096f7790c7b819ce37a3ed8ce8b5fa" +PV = "2.99.917+git${SRCPV}" +S = "${WORKDIR}/git" + +SRC_URI = "git://anongit.freedesktop.org/xorg/driver/xf86-video-intel \ + file://disable-x11-dri3.patch \ + file://always_include_xorg_server.h.patch \ " SRC_URI[md5sum] = "fa196a66e52c0c624fe5d350af7a5e7b"
Signature · 1 line
2.1.4
PATCH v2 2/2 Revert "gst-player: Disable visualizations"
This reverts oe-core commit b79d1bf49b56a97216fb719ac19e4dd9022f15b4.
Now that xf86-video-intel is upgraded, visualizations can be enabled by default.
--- .../0001-Disable-visualizations-by-default.patch | 36 ---------------------- .../recipes-multimedia/gstreamer/gst-player_git.bb | 1 - 2 files changed, 37 deletions(-) delete mode 100644 meta/recipes-multimedia/gstreamer/gst-player/0001-Disable-visualizations-by-default.patch
meta/recipes-multimedia/gstreamer/gst-player/0001-Disable-visualizations-by-default.patch deleted +0 -36
@@ -1,36 +0,0 @@ -From 93e1838f8b8704e4d7d71965f61bee931377b080 Mon Sep 17 00:00:00 2001 -From: Jussi Kukkonen <[email protected]> -Date: Thu, 22 Sep 2016 19:26:40 +0300 -Subject: [PATCH] Disable visualizations by default - -As documented in [YOCTO #10041] visualizations trigger a bug in -xvimagesink on some hardware, which prevents playback in the -mediaplayer. Disable visualizations until the xv issue is solved -to enable playback QA. - -Upstream-Status: Inappropriate [workaround] -Signed-off-by: Jussi Kukkonen <[email protected]> -Signed-off-by: Maxin B. John <[email protected]> ---- - gtk/gtk-play.c | 5 ++--- - 1 file changed, 2 insertions(+), 3 deletions(-) - -diff --git a/gtk/gtk-play.c b/gtk/gtk-play.c -index 8ae0fea..322e599 100644 ---- a/gtk/gtk-play.c -+++ b/gtk/gtk-play.c -@@ -1751,9 +1751,8 @@ gtk_play_constructor (GType type, guint n_construct_params, - g_signal_connect (self->player, "volume-changed", - G_CALLBACK (player_volume_changed_cb), self); - -- /* enable visualization (by default playbin uses goom) */ -- /* if visualization is enabled then use the first element */ -- gst_player_set_visualization_enabled (self->player, TRUE); -+ /* Disable visualization by default */ -+ gst_player_set_visualization_enabled (self->player, FALSE); - - g_signal_connect (G_OBJECT (self), "show", G_CALLBACK (show_cb), NULL); - --- -2.4.0 -
meta/recipes-multimedia/gstreamer/gst-player_git.bb +0 -1
@@ -9,7 +9,6 @@ SRC_URI = "git://github.com/sdroege/gst-player.git \ file://Fix-pause-play.patch;apply=0 \ file://Add-error-signal-emission-for-missing-plugins.patch;apply=0 \ file://0001-gtk-play-provide-similar-behaviour-for-quit-and-clos.patch \ - file://0001-Disable-visualizations-by-default.patch \ file://gst-player.desktop" SRCREV = "ea90e63c1064503f9ba5d59aa4ca604f13ca5def"
Signature · 1 line
2.1.4