[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: adding state checker for gamma lut values (rev7)

2019-05-04 Thread Patchwork
== Series Details == Series: drm/i915: adding state checker for gamma lut values (rev7) URL : https://patchwork.freedesktop.org/series/58039/ State : failure == Summary == CI Bug Log - changes from CI_DRM_6045 -> Patchwork_12967 Summary ---

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915: adding state checker for gamma lut values (rev7)

2019-05-04 Thread Patchwork
== Series Details == Series: drm/i915: adding state checker for gamma lut values (rev7) URL : https://patchwork.freedesktop.org/series/58039/ State : warning == Summary == $ dim sparse origin/drm-tip Sparse version: v0.5.2 Commit: drm/i915: Introduce vfunc read_luts() to create hw lut Okay! C

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: adding state checker for gamma lut values (rev7)

2019-05-04 Thread Patchwork
== Series Details == Series: drm/i915: adding state checker for gamma lut values (rev7) URL : https://patchwork.freedesktop.org/series/58039/ State : warning == Summary == $ dim checkpatch origin/drm-tip 1e8ed6586232 drm/i915: Introduce vfunc read_luts() to create hw lut b2195f303ff1 drm/i915:

[Intel-gfx] [v5][PATCH 11/11] drm/i915: Add intel_color_lut_equal() to compare hw and sw gamma/degamma lut values

2019-05-04 Thread Swati Sharma
v3: Rebase v4: -Renamed intel_compare_color_lut() to intel_color_lut_equal() [Jani] -Added the default label above the correct label [Jani] -Corrected smatch warn "variable dereferenced before check" [Dan Carpenter] v5: -Added condition (!blob1 && !blob2) return true [Jani] -Called PIPE

[Intel-gfx] [v5][PATCH 07/11] drm/i915: Extract glk_read_luts()

2019-05-04 Thread Swati Sharma
In this patch, gamma and degamma hw blobs are created for GLK. v4: -No need to initialize *blob [Jani] -Removed right shifts [Jani] -Dropped dev local var [Jani] v5: -Returned blob instead of assigning it internally within the function [Ville] -Renamed glk_get_color_config() to gl

[Intel-gfx] [v5][PATCH 10/11] drm/i915: Extract ilk_read_luts()

2019-05-04 Thread Swati Sharma
In this patch, hw gamma blob is created for ILK. v4: -No need to initialize *blob [Jani] -Removed right shifts [Jani] -Dropped dev local var [Jani] v5: -Returned blob instead of assigning it internally within the function [Ville] -Renamed ilk_get_color_config() to ilk_read_luts()

[Intel-gfx] [v5][PATCH 09/11] drm/i915: Extract ivb_read_luts()

2019-05-04 Thread Swati Sharma
In this patch, gamma and degamma hw blobs are created for IVB. v4: -No need to initialize *blob [Jani] -Removed right shifts [Jani] -Dropped dev local var [Jani] v5: -Returned blob instead of assigning it internally within the function [Ville] -Renamed ivb_get_color_config() to iv

[Intel-gfx] [v5][PATCH 08/11] drm/i915: Extract bdw_read_luts()

2019-05-04 Thread Swati Sharma
In this patch, gamma and degamma hw blobs are created for BDW. v4: -No need to initialize *blob [Jani] -Removed right shifts [Jani] -Dropped dev local var [Jani] v5: -Returned blob instead of assigning it internally within the function [Ville] -Renamed bdw_get_color_config() to bd

[Intel-gfx] [v5][PATCH 06/11] drm/i915: Extract icl_read_luts()

2019-05-04 Thread Swati Sharma
In this patch, gamma and degamma hw blobs are created for ICL. v4: -No need to initialize *blob [Jani] -Removed right shifts [Jani] -Dropped dev local var [Jani] v5: -Returned blob instead of assigning it internally within the function [Ville] -Renamed icl_get_color_config() to ic

[Intel-gfx] [v5][PATCH 03/11] drm/i915: Extract i9xx_read_luts()

2019-05-04 Thread Swati Sharma
In this patch, hw gamma blob is created for the legacy gamma. Also, function intel_color_lut_pack is added to convert hw value with given bit_precision to lut property val. v4: -No need to initialize *blob [Jani] -Removed right shifts [Jani] -Dropped dev local var [Jani] v5: -Returned blob

[Intel-gfx] [v5][PATCH 04/11] drm/i915: Extract chv_read_luts()

2019-05-04 Thread Swati Sharma
In this patch, hw gamma and degamma blob is created for cherryview. v4: -No need to initialize *blob [Jani] -Removed right shifts [Jani] -Dropped dev local var [Jani] v5: -Returned blob instead of assigning it internally within the function [Ville] -Renamed function cherryvi

[Intel-gfx] [v5][PATCH 05/11] drm/i915: Extract i965_read_luts()

2019-05-04 Thread Swati Sharma
In this patch, hw gamma blob is created for i965. v4: -No need to initialize *blob [Jani] -Removed right shifts [Jani] -Dropped dev local var [Jani] v5: -Returned blob instead of assigning it internally within the function [Ville] -Renamed i965_get_color_config() to i965_read_lut(

[Intel-gfx] [v5][PATCH 02/11] drm/i915: Enable intel_color_read_luts()

2019-05-04 Thread Swati Sharma
In this patch, intel_color_read_luts() is enabled and support for read_luts() will be added platform by platform incrementally in the follow-up patches. v4: -Renamed intel_get_color_config to intel_color_get_config [Jani] -Added the user early on such that support for get_color_config() c

[Intel-gfx] [v5][PATCH 01/11] drm/i915: Introduce vfunc read_luts() to create hw lut

2019-05-04 Thread Swati Sharma
In this patch, a vfunc read_luts() is introduced to create a hw lut i.e. lut having values read from gamma/degamma registers which will later be used to compare with sw lut to validate gamma/degamma lut values. v3: -Rebase v4: -Renamed intel_get_color_config to intel_color_get_config [Jani] -W

[Intel-gfx] [PATCH 00/11] drm/i915: adding state checker for gamma lut values

2019-05-04 Thread Swati Sharma
Thanks to Jani N, Matt and Ville for the review comments. In this patch series, added state checker to validate gamma and degamma lut values. This reads hardware state, and compares the originally requested state to the state read from hardware. v1: -Implementation done for legacy platforms (re

Re: [Intel-gfx] [PATCH v4 1/1] drm/fb-helper: Avoid race with DRM userspace

2019-05-04 Thread Noralf Trønnes
Den 25.04.2019 10.31, skrev Noralf Trønnes: > drm_fb_helper_is_bound() is used to check if DRM userspace is in control. > This is done by looking at the fb on the primary plane. By the time > fb-helper gets around to committing, it's possible that the facts have > changed. > > Avoid this race by

Re: [Intel-gfx] [v8 01/10] drm: Add HDR source metadata property

2019-05-04 Thread Jonas Karlman
On 2019-04-09 18:44, Uma Shankar wrote: > This patch adds a blob property to get HDR metadata > information from userspace. This will be send as part > of AVI Infoframe to panel. > > It also implements get() and set() functions for HDR output > metadata property.The blob data is received from users

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Disable semaphore busywaits on saturated systems

2019-05-04 Thread Patchwork
== Series Details == Series: drm/i915: Disable semaphore busywaits on saturated systems URL : https://patchwork.freedesktop.org/series/60278/ State : success == Summary == CI Bug Log - changes from CI_DRM_6043_full -> Patchwork_12966_full S

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Disable semaphore busywaits on saturated systems

2019-05-04 Thread Patchwork
== Series Details == Series: drm/i915: Disable semaphore busywaits on saturated systems URL : https://patchwork.freedesktop.org/series/60278/ State : success == Summary == CI Bug Log - changes from CI_DRM_6043 -> Patchwork_12966 Summary ---

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915: Disable semaphore busywaits on saturated systems

2019-05-04 Thread Patchwork
== Series Details == Series: drm/i915: Disable semaphore busywaits on saturated systems URL : https://patchwork.freedesktop.org/series/60278/ State : warning == Summary == $ dim sparse origin/drm-tip Sparse version: v0.5.2 Commit: drm/i915: Disable semaphore busywaits on saturated systems +./i

[Intel-gfx] [PATCH i-g-t] i915/gem_exec_schedule: Semaphore priority fixups

2019-05-04 Thread Chris Wilson
A stray git add from my test boxen -- we were being careful enough to preserve priority and ordering to match the implicit policies. Signed-off-by: Chris Wilson --- tests/i915/gem_exec_schedule.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/i915/gem_exec_schedule.c b/tests/i915/ge

[Intel-gfx] [CI] drm/i915: Disable semaphore busywaits on saturated systems

2019-05-04 Thread Chris Wilson
Asking the GPU to busywait on a memory address, perhaps not unexpectedly in hindsight for a shared system, leads to bus contention that affects CPU programs trying to concurrently access memory. This can manifest as a drop in transcode throughput on highly over-saturated workloads. The only clue o