* [PATCH 0/3] drm/i915: Fix SKL+ 90/270 degree rotated scanout
@ 2017-03-31 18:00 ville.syrjala
2017-03-31 18:00 ` ville.syrjala
` (4 more replies)
0 siblings, 5 replies; 15+ messages in thread
From: ville.syrjala @ 2017-03-31 18:00 UTC (permalink / raw
To: intel-gfx
From: Ville Syrjälä <ville.syrjala@linux.intel.com>
I figured it's about time I fix what I broke with my fb offset stuff.
I've posted the scaler thing before, but the watermark and fbc stuff
is new.
Based on some quick tests the WM fixes seem effective. Or at least
underruns seemed to disappear when I was running xonotic with 90/270
degree rotation.
Entire series available here:
git://github.com/vsyrjala/linux.git skl_rotation_fixes
Ville Syrjälä (3):
drm/i915: Fix scaling check for 90/270 degree plane rotation
drm/i915: Fix SKL+ watermarks for 90/270 rotation
drm/i915: Fix 90/270 rotated coordinates for FBC
drivers/gpu/drm/i915/intel_display.c | 14 ++++++++------
drivers/gpu/drm/i915/intel_fbc.c | 19 +++++++------------
drivers/gpu/drm/i915/intel_pm.c | 36 ++++++++++++++++++++++++------------
3 files changed, 39 insertions(+), 30 deletions(-)
--
2.10.2
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH 1/3] drm/i915: Fix scaling check for 90/270 degree plane rotation
2017-03-31 18:00 [PATCH 0/3] drm/i915: Fix SKL+ 90/270 degree rotated scanout ville.syrjala
@ 2017-03-31 18:00 ` ville.syrjala
2017-03-31 18:00 ` [PATCH 2/3] drm/i915: Fix SKL+ watermarks for 90/270 rotation ville.syrjala
` (3 subsequent siblings)
4 siblings, 0 replies; 15+ messages in thread
From: ville.syrjala @ 2017-03-31 18:00 UTC (permalink / raw
To: intel-gfx; +Cc: stable, Tvrtko Ursulin
From: Ville Syrjälä <ville.syrjala@linux.intel.com>
Starting from commit b63a16f6cd89 ("drm/i915: Compute display surface
offset in the plane check hook for SKL+") we've already rotated the src
coordinates by 270 degrees by the time we check if a scaler is needed
or not, so we must not account for the rotation a second time.
Previously we did these steps in the opposite order and hence the
scaler check had to deal with rotation itself. The double rotation
handling causes us to enable a scaler pretty much every time 90/270
degree plane rotation is requested, leading to fuzzier fonts and whatnot.
v2: s/unsigned/unsigned int/ to appease checkpatch
Cc: stable@vger.kernel.org
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Reported-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Tested-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Fixes: b63a16f6cd89 ("drm/i915: Compute display surface offset in the plane check hook for SKL+")
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
drivers/gpu/drm/i915/intel_display.c | 14 ++++++++------
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
index 81baa5a9780c..09c9995cafe6 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++ b/drivers/gpu/drm/i915/intel_display.c
@@ -4609,7 +4609,7 @@ static void cpt_verify_modeset(struct drm_device *dev, int pipe)
static int
skl_update_scaler(struct intel_crtc_state *crtc_state, bool force_detach,
- unsigned scaler_user, int *scaler_id, unsigned int rotation,
+ unsigned int scaler_user, int *scaler_id,
int src_w, int src_h, int dst_w, int dst_h)
{
struct intel_crtc_scaler_state *scaler_state =
@@ -4618,9 +4618,12 @@ skl_update_scaler(struct intel_crtc_state *crtc_state, bool force_detach,
to_intel_crtc(crtc_state->base.crtc);
int need_scaling;
- need_scaling = drm_rotation_90_or_270(rotation) ?
- (src_h != dst_w || src_w != dst_h):
- (src_w != dst_w || src_h != dst_h);
+ /*
+ * Src coordinates are already rotated by 270 degrees for
+ * the 90/270 degree plane rotation cases (to match the
+ * GTT mapping), hence no need to account for rotation here.
+ */
+ need_scaling = src_w != dst_w || src_h != dst_h;
/*
* if plane is being disabled or scaler is no more required or force detach
@@ -4682,7 +4685,7 @@ int skl_update_scaler_crtc(struct intel_crtc_state *state)
const struct drm_display_mode *adjusted_mode = &state->base.adjusted_mode;
return skl_update_scaler(state, !state->base.active, SKL_CRTC_INDEX,
- &state->scaler_state.scaler_id, DRM_ROTATE_0,
+ &state->scaler_state.scaler_id,
state->pipe_src_w, state->pipe_src_h,
adjusted_mode->crtc_hdisplay, adjusted_mode->crtc_vdisplay);
}
@@ -4711,7 +4714,6 @@ static int skl_update_scaler_plane(struct intel_crtc_state *crtc_state,
ret = skl_update_scaler(crtc_state, force_detach,
drm_plane_index(&intel_plane->base),
&plane_state->scaler_id,
- plane_state->base.rotation,
drm_rect_width(&plane_state->base.src) >> 16,
drm_rect_height(&plane_state->base.src) >> 16,
drm_rect_width(&plane_state->base.dst),
--
2.10.2
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 1/3] drm/i915: Fix scaling check for 90/270 degree plane rotation
@ 2017-03-31 18:00 ` ville.syrjala
0 siblings, 0 replies; 15+ messages in thread
From: ville.syrjala @ 2017-03-31 18:00 UTC (permalink / raw
To: intel-gfx; +Cc: stable
From: Ville Syrjälä <ville.syrjala@linux.intel.com>
Starting from commit b63a16f6cd89 ("drm/i915: Compute display surface
offset in the plane check hook for SKL+") we've already rotated the src
coordinates by 270 degrees by the time we check if a scaler is needed
or not, so we must not account for the rotation a second time.
Previously we did these steps in the opposite order and hence the
scaler check had to deal with rotation itself. The double rotation
handling causes us to enable a scaler pretty much every time 90/270
degree plane rotation is requested, leading to fuzzier fonts and whatnot.
v2: s/unsigned/unsigned int/ to appease checkpatch
Cc: stable@vger.kernel.org
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Reported-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Tested-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Fixes: b63a16f6cd89 ("drm/i915: Compute display surface offset in the plane check hook for SKL+")
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
drivers/gpu/drm/i915/intel_display.c | 14 ++++++++------
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
index 81baa5a9780c..09c9995cafe6 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++ b/drivers/gpu/drm/i915/intel_display.c
@@ -4609,7 +4609,7 @@ static void cpt_verify_modeset(struct drm_device *dev, int pipe)
static int
skl_update_scaler(struct intel_crtc_state *crtc_state, bool force_detach,
- unsigned scaler_user, int *scaler_id, unsigned int rotation,
+ unsigned int scaler_user, int *scaler_id,
int src_w, int src_h, int dst_w, int dst_h)
{
struct intel_crtc_scaler_state *scaler_state =
@@ -4618,9 +4618,12 @@ skl_update_scaler(struct intel_crtc_state *crtc_state, bool force_detach,
to_intel_crtc(crtc_state->base.crtc);
int need_scaling;
- need_scaling = drm_rotation_90_or_270(rotation) ?
- (src_h != dst_w || src_w != dst_h):
- (src_w != dst_w || src_h != dst_h);
+ /*
+ * Src coordinates are already rotated by 270 degrees for
+ * the 90/270 degree plane rotation cases (to match the
+ * GTT mapping), hence no need to account for rotation here.
+ */
+ need_scaling = src_w != dst_w || src_h != dst_h;
/*
* if plane is being disabled or scaler is no more required or force detach
@@ -4682,7 +4685,7 @@ int skl_update_scaler_crtc(struct intel_crtc_state *state)
const struct drm_display_mode *adjusted_mode = &state->base.adjusted_mode;
return skl_update_scaler(state, !state->base.active, SKL_CRTC_INDEX,
- &state->scaler_state.scaler_id, DRM_ROTATE_0,
+ &state->scaler_state.scaler_id,
state->pipe_src_w, state->pipe_src_h,
adjusted_mode->crtc_hdisplay, adjusted_mode->crtc_vdisplay);
}
@@ -4711,7 +4714,6 @@ static int skl_update_scaler_plane(struct intel_crtc_state *crtc_state,
ret = skl_update_scaler(crtc_state, force_detach,
drm_plane_index(&intel_plane->base),
&plane_state->scaler_id,
- plane_state->base.rotation,
drm_rect_width(&plane_state->base.src) >> 16,
drm_rect_height(&plane_state->base.src) >> 16,
drm_rect_width(&plane_state->base.dst),
--
2.10.2
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 2/3] drm/i915: Fix SKL+ watermarks for 90/270 rotation
2017-03-31 18:00 [PATCH 0/3] drm/i915: Fix SKL+ 90/270 degree rotated scanout ville.syrjala
2017-03-31 18:00 ` ville.syrjala
@ 2017-03-31 18:00 ` ville.syrjala
2017-03-31 18:00 ` [PATCH 3/3] drm/i915: Fix 90/270 rotated coordinates for FBC ville.syrjala
` (2 subsequent siblings)
4 siblings, 0 replies; 15+ messages in thread
From: ville.syrjala @ 2017-03-31 18:00 UTC (permalink / raw
To: intel-gfx; +Cc: stable, Tvrtko Ursulin
From: Ville Syrjälä <ville.syrjala@linux.intel.com>
skl_check_plane_surface() already rotates the clipped plane source
coordinates to match the scanout direction because that's the way
the GTT mapping is set up. Thus we no longer need to rotate the
coordinates in the watermark code.
For cursors we use the non-clipped coordinates which are not rotated
appropriately, but that doesn't actually matter since cursors don't
even support 90/270 degree rotation.
Cc: stable@vger.kernel.org
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Fixes: b63a16f6cd89 ("drm/i915: Compute display surface offset in the plane check hook for SKL+")
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
drivers/gpu/drm/i915/intel_pm.c | 36 ++++++++++++++++++++++++------------
1 file changed, 24 insertions(+), 12 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
index 570bd603f401..6b1caf9ed3ca 100644
--- a/drivers/gpu/drm/i915/intel_pm.c
+++ b/drivers/gpu/drm/i915/intel_pm.c
@@ -3373,20 +3373,26 @@ skl_plane_downscale_amount(const struct intel_crtc_state *cstate,
/* n.b., src is 16.16 fixed point, dst is whole integer */
if (plane->id == PLANE_CURSOR) {
+ /*
+ * Cursors only support 0/180 degree rotation,
+ * hence no need to account for rotation here.
+ */
src_w = pstate->base.src_w;
src_h = pstate->base.src_h;
dst_w = pstate->base.crtc_w;
dst_h = pstate->base.crtc_h;
} else {
+ /*
+ * Src coordinates are already rotated by 270 degrees for
+ * the 90/270 degree plane rotation cases (to match the
+ * GTT mapping), hence no need to account for rotation here.
+ */
src_w = drm_rect_width(&pstate->base.src);
src_h = drm_rect_height(&pstate->base.src);
dst_w = drm_rect_width(&pstate->base.dst);
dst_h = drm_rect_height(&pstate->base.dst);
}
- if (drm_rotation_90_or_270(pstate->base.rotation))
- swap(dst_w, dst_h);
-
downscale_h = max(src_h / dst_h, (uint32_t)DRM_PLANE_HELPER_NO_SCALING);
downscale_w = max(src_w / dst_w, (uint32_t)DRM_PLANE_HELPER_NO_SCALING);
@@ -3417,12 +3423,14 @@ skl_plane_relative_data_rate(const struct intel_crtc_state *cstate,
if (y && format != DRM_FORMAT_NV12)
return 0;
+ /*
+ * Src coordinates are already rotated by 270 degrees for
+ * the 90/270 degree plane rotation cases (to match the
+ * GTT mapping), hence no need to account for rotation here.
+ */
width = drm_rect_width(&intel_pstate->base.src) >> 16;
height = drm_rect_height(&intel_pstate->base.src) >> 16;
- if (drm_rotation_90_or_270(pstate->rotation))
- swap(width, height);
-
/* for planar format */
if (format == DRM_FORMAT_NV12) {
if (y) /* y-plane data rate */
@@ -3505,12 +3513,14 @@ skl_ddb_min_alloc(const struct drm_plane_state *pstate,
fb->modifier != I915_FORMAT_MOD_Yf_TILED)
return 8;
+ /*
+ * Src coordinates are already rotated by 270 degrees for
+ * the 90/270 degree plane rotation cases (to match the
+ * GTT mapping), hence no need to account for rotation here.
+ */
src_w = drm_rect_width(&intel_pstate->base.src) >> 16;
src_h = drm_rect_height(&intel_pstate->base.src) >> 16;
- if (drm_rotation_90_or_270(pstate->rotation))
- swap(src_w, src_h);
-
/* Halve UV plane width and height for NV12 */
if (fb->format->format == DRM_FORMAT_NV12 && !y) {
src_w /= 2;
@@ -3794,13 +3804,15 @@ static int skl_compute_plane_wm(const struct drm_i915_private *dev_priv,
width = intel_pstate->base.crtc_w;
height = intel_pstate->base.crtc_h;
} else {
+ /*
+ * Src coordinates are already rotated by 270 degrees for
+ * the 90/270 degree plane rotation cases (to match the
+ * GTT mapping), hence no need to account for rotation here.
+ */
width = drm_rect_width(&intel_pstate->base.src) >> 16;
height = drm_rect_height(&intel_pstate->base.src) >> 16;
}
- if (drm_rotation_90_or_270(pstate->rotation))
- swap(width, height);
-
cpp = fb->format->cpp[0];
plane_pixel_rate = skl_adjusted_plane_pixel_rate(cstate, intel_pstate);
--
2.10.2
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 3/3] drm/i915: Fix 90/270 rotated coordinates for FBC
2017-03-31 18:00 [PATCH 0/3] drm/i915: Fix SKL+ 90/270 degree rotated scanout ville.syrjala
2017-03-31 18:00 ` ville.syrjala
2017-03-31 18:00 ` [PATCH 2/3] drm/i915: Fix SKL+ watermarks for 90/270 rotation ville.syrjala
@ 2017-03-31 18:00 ` ville.syrjala
2017-04-03 17:57 ` Paulo Zanoni
2017-05-19 11:34 ` Tvrtko Ursulin
2017-03-31 18:21 ` ✓ Fi.CI.BAT: success for drm/i915: Fix SKL+ 90/270 degree rotated scanout Patchwork
2017-03-31 21:23 ` [PATCH 0/3] " Chris Wilson
4 siblings, 2 replies; 15+ messages in thread
From: ville.syrjala @ 2017-03-31 18:00 UTC (permalink / raw
To: intel-gfx; +Cc: stable, Tvrtko Ursulin, Paulo Zanoni
From: Ville Syrjälä <ville.syrjala@linux.intel.com>
The clipped src coordinates have already been rotated by 270 degrees for
when the plane rotation is 90/270 degrees, hence the FBC code should no
longer swap the width and height.
Cc: stable@vger.kernel.org
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
Fixes: b63a16f6cd89 ("drm/i915: Compute display surface offset in the plane check hook for SKL+")
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
drivers/gpu/drm/i915/intel_fbc.c | 19 +++++++------------
1 file changed, 7 insertions(+), 12 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_fbc.c b/drivers/gpu/drm/i915/intel_fbc.c
index ded2add18b26..d93c58410bff 100644
--- a/drivers/gpu/drm/i915/intel_fbc.c
+++ b/drivers/gpu/drm/i915/intel_fbc.c
@@ -82,20 +82,10 @@ static unsigned int get_crtc_fence_y_offset(struct intel_crtc *crtc)
static void intel_fbc_get_plane_source_size(struct intel_fbc_state_cache *cache,
int *width, int *height)
{
- int w, h;
-
- if (drm_rotation_90_or_270(cache->plane.rotation)) {
- w = cache->plane.src_h;
- h = cache->plane.src_w;
- } else {
- w = cache->plane.src_w;
- h = cache->plane.src_h;
- }
-
if (width)
- *width = w;
+ *width = cache->plane.src_w;
if (height)
- *height = h;
+ *height = cache->plane.src_h;
}
static int intel_fbc_calculate_cfb_size(struct drm_i915_private *dev_priv,
@@ -746,6 +736,11 @@ static void intel_fbc_update_state_cache(struct intel_crtc *crtc,
cache->crtc.hsw_bdw_pixel_rate = crtc_state->pixel_rate;
cache->plane.rotation = plane_state->base.rotation;
+ /*
+ * Src coordinates are already rotated by 270 degrees for
+ * the 90/270 degree plane rotation cases (to match the
+ * GTT mapping), hence no need to account for rotation here.
+ */
cache->plane.src_w = drm_rect_width(&plane_state->base.src) >> 16;
cache->plane.src_h = drm_rect_height(&plane_state->base.src) >> 16;
cache->plane.visible = plane_state->base.visible;
--
2.10.2
^ permalink raw reply related [flat|nested] 15+ messages in thread
* ✓ Fi.CI.BAT: success for drm/i915: Fix SKL+ 90/270 degree rotated scanout
2017-03-31 18:00 [PATCH 0/3] drm/i915: Fix SKL+ 90/270 degree rotated scanout ville.syrjala
` (2 preceding siblings ...)
2017-03-31 18:00 ` [PATCH 3/3] drm/i915: Fix 90/270 rotated coordinates for FBC ville.syrjala
@ 2017-03-31 18:21 ` Patchwork
2017-03-31 21:23 ` [PATCH 0/3] " Chris Wilson
4 siblings, 0 replies; 15+ messages in thread
From: Patchwork @ 2017-03-31 18:21 UTC (permalink / raw
To: ville.syrjala; +Cc: intel-gfx
== Series Details ==
Series: drm/i915: Fix SKL+ 90/270 degree rotated scanout
URL : https://patchwork.freedesktop.org/series/22295/
State : success
== Summary ==
Series 22295v1 drm/i915: Fix SKL+ 90/270 degree rotated scanout
https://patchwork.freedesktop.org/api/1.0/series/22295/revisions/1/mbox/
Test kms_cursor_legacy:
Subgroup basic-busy-flip-before-cursor-legacy:
pass -> DMESG-WARN (fi-byt-n2820) fdo#100415
fdo#100415 https://bugs.freedesktop.org/show_bug.cgi?id=100415
fi-bdw-5557u total:278 pass:267 dwarn:0 dfail:0 fail:0 skip:11 time: 436s
fi-bdw-gvtdvm total:278 pass:256 dwarn:8 dfail:0 fail:0 skip:14 time: 425s
fi-bsw-n3050 total:278 pass:239 dwarn:0 dfail:0 fail:0 skip:39 time: 572s
fi-bxt-j4205 total:278 pass:259 dwarn:0 dfail:0 fail:0 skip:19 time: 507s
fi-bxt-t5700 total:278 pass:257 dwarn:1 dfail:0 fail:0 skip:20 time: 552s
fi-byt-j1900 total:278 pass:251 dwarn:0 dfail:0 fail:0 skip:27 time: 487s
fi-byt-n2820 total:278 pass:246 dwarn:1 dfail:0 fail:0 skip:31 time: 482s
fi-hsw-4770 total:278 pass:262 dwarn:0 dfail:0 fail:0 skip:16 time: 411s
fi-hsw-4770r total:278 pass:262 dwarn:0 dfail:0 fail:0 skip:16 time: 411s
fi-ilk-650 total:278 pass:228 dwarn:0 dfail:0 fail:0 skip:50 time: 422s
fi-ivb-3520m total:278 pass:260 dwarn:0 dfail:0 fail:0 skip:18 time: 482s
fi-ivb-3770 total:278 pass:260 dwarn:0 dfail:0 fail:0 skip:18 time: 482s
fi-kbl-7500u total:278 pass:260 dwarn:0 dfail:0 fail:0 skip:18 time: 453s
fi-kbl-7560u total:278 pass:268 dwarn:0 dfail:0 fail:0 skip:10 time: 571s
fi-skl-6260u total:278 pass:268 dwarn:0 dfail:0 fail:0 skip:10 time: 452s
fi-skl-6700hq total:278 pass:261 dwarn:0 dfail:0 fail:0 skip:17 time: 570s
fi-skl-6700k total:278 pass:256 dwarn:4 dfail:0 fail:0 skip:18 time: 462s
fi-skl-6770hq total:278 pass:268 dwarn:0 dfail:0 fail:0 skip:10 time: 494s
fi-skl-gvtdvm total:278 pass:265 dwarn:0 dfail:0 fail:0 skip:13 time: 432s
fi-snb-2520m total:278 pass:250 dwarn:0 dfail:0 fail:0 skip:28 time: 527s
fi-snb-2600 total:278 pass:249 dwarn:0 dfail:0 fail:0 skip:29 time: 401s
5a229613e4ef7d50ae0ae42ed50a57f4463d163f drm-tip: 2017y-03m-31d-16h-10m-12s UTC integration manifest
96f25eb drm/i915: Fix 90/270 rotated coordinates for FBC
8098628 drm/i915: Fix SKL+ watermarks for 90/270 rotation
18d0b32 drm/i915: Fix scaling check for 90/270 degree plane rotation
== Logs ==
For more details see: https://intel-gfx-ci.01.org/CI/Patchwork_4376/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 0/3] drm/i915: Fix SKL+ 90/270 degree rotated scanout
2017-03-31 18:00 [PATCH 0/3] drm/i915: Fix SKL+ 90/270 degree rotated scanout ville.syrjala
` (3 preceding siblings ...)
2017-03-31 18:21 ` ✓ Fi.CI.BAT: success for drm/i915: Fix SKL+ 90/270 degree rotated scanout Patchwork
@ 2017-03-31 21:23 ` Chris Wilson
2017-04-05 13:49 ` Ville Syrjälä
4 siblings, 1 reply; 15+ messages in thread
From: Chris Wilson @ 2017-03-31 21:23 UTC (permalink / raw
To: ville.syrjala; +Cc: intel-gfx
On Fri, Mar 31, 2017 at 09:00:53PM +0300, ville.syrjala@linux.intel.com wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> I figured it's about time I fix what I broke with my fb offset stuff.
> I've posted the scaler thing before, but the watermark and fbc stuff
> is new.
>
> Based on some quick tests the WM fixes seem effective. Or at least
> underruns seemed to disappear when I was running xonotic with 90/270
> degree rotation.
The key question for me is would we be able to detect any of the errors
in igt? How can we improve our testing?
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 3/3] drm/i915: Fix 90/270 rotated coordinates for FBC
2017-03-31 18:00 ` [PATCH 3/3] drm/i915: Fix 90/270 rotated coordinates for FBC ville.syrjala
@ 2017-04-03 17:57 ` Paulo Zanoni
2017-05-19 11:34 ` Tvrtko Ursulin
1 sibling, 0 replies; 15+ messages in thread
From: Paulo Zanoni @ 2017-04-03 17:57 UTC (permalink / raw
To: ville.syrjala, intel-gfx; +Cc: stable, Tvrtko Ursulin
Em Sex, 2017-03-31 às 21:00 +0300, ville.syrjala@linux.intel.com
escreveu:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> The clipped src coordinates have already been rotated by 270 degrees
> for
> when the plane rotation is 90/270 degrees, hence the FBC code should
> no
> longer swap the width and height.
I've never payed too much attention to rotation, but based on the
mentioned commits, what's said on the messages and my understanding of
the code, this looks sane, so:
Reviewed-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
And in case someone suggests to just kill
intel_fbc_get_plane_source_size(), I'd like to point that "plane source
size" is wording used by our spec and there's a nice comment explaining
what exactly it's supposed to be, so I'd be in favor of keeping it.
Super bonus point if you end up writing some sort of rotation test for
kms_frontbuffer_tracking or kms_fbc_crc. The problem is that I'm not
entirely too sure about how much the current code structure for those
tests is ready to easily support such a test with minimal efforts.
Needs to be studied.
>
> Cc: stable@vger.kernel.org
> Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
> Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
> Fixes: b63a16f6cd89 ("drm/i915: Compute display surface offset in the
> plane check hook for SKL+")
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> ---
> drivers/gpu/drm/i915/intel_fbc.c | 19 +++++++------------
> 1 file changed, 7 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_fbc.c
> b/drivers/gpu/drm/i915/intel_fbc.c
> index ded2add18b26..d93c58410bff 100644
> --- a/drivers/gpu/drm/i915/intel_fbc.c
> +++ b/drivers/gpu/drm/i915/intel_fbc.c
> @@ -82,20 +82,10 @@ static unsigned int
> get_crtc_fence_y_offset(struct intel_crtc *crtc)
> static void intel_fbc_get_plane_source_size(struct
> intel_fbc_state_cache *cache,
> int *width, int *height)
> {
> - int w, h;
> -
> - if (drm_rotation_90_or_270(cache->plane.rotation)) {
> - w = cache->plane.src_h;
> - h = cache->plane.src_w;
> - } else {
> - w = cache->plane.src_w;
> - h = cache->plane.src_h;
> - }
> -
> if (width)
> - *width = w;
> + *width = cache->plane.src_w;
> if (height)
> - *height = h;
> + *height = cache->plane.src_h;
> }
>
> static int intel_fbc_calculate_cfb_size(struct drm_i915_private
> *dev_priv,
> @@ -746,6 +736,11 @@ static void intel_fbc_update_state_cache(struct
> intel_crtc *crtc,
> cache->crtc.hsw_bdw_pixel_rate = crtc_state-
> >pixel_rate;
>
> cache->plane.rotation = plane_state->base.rotation;
> + /*
> + * Src coordinates are already rotated by 270 degrees for
> + * the 90/270 degree plane rotation cases (to match the
> + * GTT mapping), hence no need to account for rotation here.
> + */
> cache->plane.src_w = drm_rect_width(&plane_state->base.src)
> >> 16;
> cache->plane.src_h = drm_rect_height(&plane_state->base.src)
> >> 16;
> cache->plane.visible = plane_state->base.visible;
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 0/3] drm/i915: Fix SKL+ 90/270 degree rotated scanout
2017-03-31 21:23 ` [PATCH 0/3] " Chris Wilson
@ 2017-04-05 13:49 ` Ville Syrjälä
2017-06-06 8:06 ` Maarten Lankhorst
0 siblings, 1 reply; 15+ messages in thread
From: Ville Syrjälä @ 2017-04-05 13:49 UTC (permalink / raw
To: Chris Wilson, intel-gfx
On Fri, Mar 31, 2017 at 10:23:18PM +0100, Chris Wilson wrote:
> On Fri, Mar 31, 2017 at 09:00:53PM +0300, ville.syrjala@linux.intel.com wrote:
> > From: Ville Syrjälä <ville.syrjala@linux.intel.com>
> >
> > I figured it's about time I fix what I broke with my fb offset stuff.
> > I've posted the scaler thing before, but the watermark and fbc stuff
> > is new.
> >
> > Based on some quick tests the WM fixes seem effective. Or at least
> > underruns seemed to disappear when I was running xonotic with 90/270
> > degree rotation.
>
> The key question for me is would we be able to detect any of the errors
> in igt? How can we improve our testing?
The rotation test definitely would need some love. It fails to detect
these problems because it scans out a square image. Making it non-square
would at least catch the use of the scaler when it shouldn't be used.
Detecting the watermark breakage is less clear. I suppose making the
plane have a very wide or very tall aspect ratio might help induce
underruns with the broken wm code.
Another thing that may or may not be missing from the test is panning.
I'd also like to test scaling, but sadly our hardware makes that
rather hard by not allowing us to force nearest and/or linear filtering,
and bspec doesn't actually document what kind of algorithm the hardware
uses for the different filter modes.
--
Ville Syrjälä
Intel OTC
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [PATCH 3/3] drm/i915: Fix 90/270 rotated coordinates for FBC
2017-03-31 18:00 ` [PATCH 3/3] drm/i915: Fix 90/270 rotated coordinates for FBC ville.syrjala
@ 2017-05-19 11:34 ` Tvrtko Ursulin
2017-05-19 11:34 ` Tvrtko Ursulin
1 sibling, 0 replies; 15+ messages in thread
From: Tvrtko Ursulin @ 2017-05-19 11:34 UTC (permalink / raw
To: ville.syrjala, intel-gfx; +Cc: Paulo Zanoni, stable
On 31/03/2017 19:00, ville.syrjala@linux.intel.com wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> The clipped src coordinates have already been rotated by 270 degrees for
> when the plane rotation is 90/270 degrees, hence the FBC code should no
> longer swap the width and height.
>
> Cc: stable@vger.kernel.org
> Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
> Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
> Fixes: b63a16f6cd89 ("drm/i915: Compute display surface offset in the plane check hook for SKL+")
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> ---
> drivers/gpu/drm/i915/intel_fbc.c | 19 +++++++------------
> 1 file changed, 7 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_fbc.c b/drivers/gpu/drm/i915/intel_fbc.c
> index ded2add18b26..d93c58410bff 100644
> --- a/drivers/gpu/drm/i915/intel_fbc.c
> +++ b/drivers/gpu/drm/i915/intel_fbc.c
> @@ -82,20 +82,10 @@ static unsigned int get_crtc_fence_y_offset(struct intel_crtc *crtc)
> static void intel_fbc_get_plane_source_size(struct intel_fbc_state_cache *cache,
> int *width, int *height)
> {
> - int w, h;
> -
> - if (drm_rotation_90_or_270(cache->plane.rotation)) {
> - w = cache->plane.src_h;
> - h = cache->plane.src_w;
> - } else {
> - w = cache->plane.src_w;
> - h = cache->plane.src_h;
> - }
> -
> if (width)
> - *width = w;
> + *width = cache->plane.src_w;
> if (height)
> - *height = h;
> + *height = cache->plane.src_h;
> }
>
> static int intel_fbc_calculate_cfb_size(struct drm_i915_private *dev_priv,
> @@ -746,6 +736,11 @@ static void intel_fbc_update_state_cache(struct intel_crtc *crtc,
> cache->crtc.hsw_bdw_pixel_rate = crtc_state->pixel_rate;
>
> cache->plane.rotation = plane_state->base.rotation;
> + /*
> + * Src coordinates are already rotated by 270 degrees for
> + * the 90/270 degree plane rotation cases (to match the
> + * GTT mapping), hence no need to account for rotation here.
> + */
> cache->plane.src_w = drm_rect_width(&plane_state->base.src) >> 16;
> cache->plane.src_h = drm_rect_height(&plane_state->base.src) >> 16;
> cache->plane.visible = plane_state->base.visible;
>
For the series:
Tested-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Regards,
Tvrtko
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 3/3] drm/i915: Fix 90/270 rotated coordinates for FBC
@ 2017-05-19 11:34 ` Tvrtko Ursulin
0 siblings, 0 replies; 15+ messages in thread
From: Tvrtko Ursulin @ 2017-05-19 11:34 UTC (permalink / raw
To: ville.syrjala, intel-gfx; +Cc: Paulo Zanoni, stable
On 31/03/2017 19:00, ville.syrjala@linux.intel.com wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> The clipped src coordinates have already been rotated by 270 degrees for
> when the plane rotation is 90/270 degrees, hence the FBC code should no
> longer swap the width and height.
>
> Cc: stable@vger.kernel.org
> Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
> Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
> Fixes: b63a16f6cd89 ("drm/i915: Compute display surface offset in the plane check hook for SKL+")
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> ---
> drivers/gpu/drm/i915/intel_fbc.c | 19 +++++++------------
> 1 file changed, 7 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_fbc.c b/drivers/gpu/drm/i915/intel_fbc.c
> index ded2add18b26..d93c58410bff 100644
> --- a/drivers/gpu/drm/i915/intel_fbc.c
> +++ b/drivers/gpu/drm/i915/intel_fbc.c
> @@ -82,20 +82,10 @@ static unsigned int get_crtc_fence_y_offset(struct intel_crtc *crtc)
> static void intel_fbc_get_plane_source_size(struct intel_fbc_state_cache *cache,
> int *width, int *height)
> {
> - int w, h;
> -
> - if (drm_rotation_90_or_270(cache->plane.rotation)) {
> - w = cache->plane.src_h;
> - h = cache->plane.src_w;
> - } else {
> - w = cache->plane.src_w;
> - h = cache->plane.src_h;
> - }
> -
> if (width)
> - *width = w;
> + *width = cache->plane.src_w;
> if (height)
> - *height = h;
> + *height = cache->plane.src_h;
> }
>
> static int intel_fbc_calculate_cfb_size(struct drm_i915_private *dev_priv,
> @@ -746,6 +736,11 @@ static void intel_fbc_update_state_cache(struct intel_crtc *crtc,
> cache->crtc.hsw_bdw_pixel_rate = crtc_state->pixel_rate;
>
> cache->plane.rotation = plane_state->base.rotation;
> + /*
> + * Src coordinates are already rotated by 270 degrees for
> + * the 90/270 degree plane rotation cases (to match the
> + * GTT mapping), hence no need to account for rotation here.
> + */
> cache->plane.src_w = drm_rect_width(&plane_state->base.src) >> 16;
> cache->plane.src_h = drm_rect_height(&plane_state->base.src) >> 16;
> cache->plane.visible = plane_state->base.visible;
>
For the series:
Tested-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Regards,
Tvrtko
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 0/3] drm/i915: Fix SKL+ 90/270 degree rotated scanout
2017-04-05 13:49 ` Ville Syrjälä
@ 2017-06-06 8:06 ` Maarten Lankhorst
2017-06-06 8:29 ` Tvrtko Ursulin
0 siblings, 1 reply; 15+ messages in thread
From: Maarten Lankhorst @ 2017-06-06 8:06 UTC (permalink / raw
To: Ville Syrjälä, Chris Wilson, intel-gfx
Op 05-04-17 om 15:49 schreef Ville Syrjälä:
> On Fri, Mar 31, 2017 at 10:23:18PM +0100, Chris Wilson wrote:
>> On Fri, Mar 31, 2017 at 09:00:53PM +0300, ville.syrjala@linux.intel.com wrote:
>>> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>>>
>>> I figured it's about time I fix what I broke with my fb offset stuff.
>>> I've posted the scaler thing before, but the watermark and fbc stuff
>>> is new.
>>>
>>> Based on some quick tests the WM fixes seem effective. Or at least
>>> underruns seemed to disappear when I was running xonotic with 90/270
>>> degree rotation.
>> The key question for me is would we be able to detect any of the errors
>> in igt? How can we improve our testing?
> The rotation test definitely would need some love. It fails to detect
> these problems because it scans out a square image. Making it non-square
> would at least catch the use of the scaler when it shouldn't be used.
>
> Detecting the watermark breakage is less clear. I suppose making the
> plane have a very wide or very tall aspect ratio might help induce
> underruns with the broken wm code.
>
> Another thing that may or may not be missing from the test is panning.
> I'd also like to test scaling, but sadly our hardware makes that
> rather hard by not allowing us to force nearest and/or linear filtering,
> and bspec doesn't actually document what kind of algorithm the hardware
> uses for the different filter modes.
>
Agreed, the whole series is useful but until we have some tests we may as well not commit it. Nothing prevents it from being broken again in the next commit. :(
I'll take a look and see if I can make kms_rotation_crc break without this test.
~Maarten
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 0/3] drm/i915: Fix SKL+ 90/270 degree rotated scanout
2017-06-06 8:06 ` Maarten Lankhorst
@ 2017-06-06 8:29 ` Tvrtko Ursulin
2017-06-06 15:15 ` Maarten Lankhorst
0 siblings, 1 reply; 15+ messages in thread
From: Tvrtko Ursulin @ 2017-06-06 8:29 UTC (permalink / raw
To: Maarten Lankhorst, Ville Syrjälä, Chris Wilson,
intel-gfx
On 06/06/2017 09:06, Maarten Lankhorst wrote:
> Op 05-04-17 om 15:49 schreef Ville Syrjälä:
>> On Fri, Mar 31, 2017 at 10:23:18PM +0100, Chris Wilson wrote:
>>> On Fri, Mar 31, 2017 at 09:00:53PM +0300, ville.syrjala@linux.intel.com wrote:
>>>> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>>>>
>>>> I figured it's about time I fix what I broke with my fb offset stuff.
>>>> I've posted the scaler thing before, but the watermark and fbc stuff
>>>> is new.
>>>>
>>>> Based on some quick tests the WM fixes seem effective. Or at least
>>>> underruns seemed to disappear when I was running xonotic with 90/270
>>>> degree rotation.
>>> The key question for me is would we be able to detect any of the errors
>>> in igt? How can we improve our testing?
>> The rotation test definitely would need some love. It fails to detect
>> these problems because it scans out a square image. Making it non-square
>> would at least catch the use of the scaler when it shouldn't be used.
>>
>> Detecting the watermark breakage is less clear. I suppose making the
>> plane have a very wide or very tall aspect ratio might help induce
>> underruns with the broken wm code.
>>
>> Another thing that may or may not be missing from the test is panning.
>> I'd also like to test scaling, but sadly our hardware makes that
>> rather hard by not allowing us to force nearest and/or linear filtering,
>> and bspec doesn't actually document what kind of algorithm the hardware
>> uses for the different filter modes.
>>
> Agreed, the whole series is useful but until we have some tests we may as well not commit it. Nothing prevents it from being broken again in the next commit. :(
In case tests hit a stumbling blocks/delays, I would appreciate if this
got reviewed and merged soonish. As it stands I've been applying (and
occasionally forgetting to apply) patches locally since September.
And FWIW I would report if it got re-broken, since I'm using monitors in
portrait, and like to run recent drm-tip to help catch issues missed
elsewhere.
> I'll take a look and see if I can make kms_rotation_crc break without this test.
Would also need to upgrade the test to basic, or count on extended runs
getting attention soon?
Regards,
Tvrtko
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 0/3] drm/i915: Fix SKL+ 90/270 degree rotated scanout
2017-06-06 8:29 ` Tvrtko Ursulin
@ 2017-06-06 15:15 ` Maarten Lankhorst
2017-06-06 16:38 ` Ville Syrjälä
0 siblings, 1 reply; 15+ messages in thread
From: Maarten Lankhorst @ 2017-06-06 15:15 UTC (permalink / raw
To: Tvrtko Ursulin, Ville Syrjälä, Chris Wilson, intel-gfx
Hey,
Op 06-06-17 om 10:29 schreef Tvrtko Ursulin:
>
> On 06/06/2017 09:06, Maarten Lankhorst wrote:
>> Op 05-04-17 om 15:49 schreef Ville Syrjälä:
>>> On Fri, Mar 31, 2017 at 10:23:18PM +0100, Chris Wilson wrote:
>>>> On Fri, Mar 31, 2017 at 09:00:53PM +0300, ville.syrjala@linux.intel.com wrote:
>>>>> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>>>>>
>>>>> I figured it's about time I fix what I broke with my fb offset stuff.
>>>>> I've posted the scaler thing before, but the watermark and fbc stuff
>>>>> is new.
>>>>>
>>>>> Based on some quick tests the WM fixes seem effective. Or at least
>>>>> underruns seemed to disappear when I was running xonotic with 90/270
>>>>> degree rotation.
>>>> The key question for me is would we be able to detect any of the errors
>>>> in igt? How can we improve our testing?
>>> The rotation test definitely would need some love. It fails to detect
>>> these problems because it scans out a square image. Making it non-square
>>> would at least catch the use of the scaler when it shouldn't be used.
>>>
>>> Detecting the watermark breakage is less clear. I suppose making the
>>> plane have a very wide or very tall aspect ratio might help induce
>>> underruns with the broken wm code.
>>>
>>> Another thing that may or may not be missing from the test is panning.
>>> I'd also like to test scaling, but sadly our hardware makes that
>>> rather hard by not allowing us to force nearest and/or linear filtering,
>>> and bspec doesn't actually document what kind of algorithm the hardware
>>> uses for the different filter modes.
>>>
>> Agreed, the whole series is useful but until we have some tests we may as well not commit it. Nothing prevents it from being broken again in the next commit. :(
>
> In case tests hit a stumbling blocks/delays, I would appreciate if this got reviewed and merged soonish. As it stands I've been applying (and occasionally forgetting to apply) patches locally since September.
>
> And FWIW I would report if it got re-broken, since I'm using monitors in portrait, and like to run recent drm-tip to help catch issues missed elsewhere.
>
>> I'll take a look and see if I can make kms_rotation_crc break without this test.
>
> Would also need to upgrade the test to basic, or count on extended runs getting attention soon?
Maybe?
I've pushed the fixed test. Managed to test that the scaler is enabled incorrectly and the WM underruns.
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 0/3] drm/i915: Fix SKL+ 90/270 degree rotated scanout
2017-06-06 15:15 ` Maarten Lankhorst
@ 2017-06-06 16:38 ` Ville Syrjälä
0 siblings, 0 replies; 15+ messages in thread
From: Ville Syrjälä @ 2017-06-06 16:38 UTC (permalink / raw
To: Maarten Lankhorst; +Cc: intel-gfx
On Tue, Jun 06, 2017 at 05:15:30PM +0200, Maarten Lankhorst wrote:
> Hey,
>
> Op 06-06-17 om 10:29 schreef Tvrtko Ursulin:
> >
> > On 06/06/2017 09:06, Maarten Lankhorst wrote:
> >> Op 05-04-17 om 15:49 schreef Ville Syrjälä:
> >>> On Fri, Mar 31, 2017 at 10:23:18PM +0100, Chris Wilson wrote:
> >>>> On Fri, Mar 31, 2017 at 09:00:53PM +0300, ville.syrjala@linux.intel.com wrote:
> >>>>> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
> >>>>>
> >>>>> I figured it's about time I fix what I broke with my fb offset stuff.
> >>>>> I've posted the scaler thing before, but the watermark and fbc stuff
> >>>>> is new.
> >>>>>
> >>>>> Based on some quick tests the WM fixes seem effective. Or at least
> >>>>> underruns seemed to disappear when I was running xonotic with 90/270
> >>>>> degree rotation.
> >>>> The key question for me is would we be able to detect any of the errors
> >>>> in igt? How can we improve our testing?
> >>> The rotation test definitely would need some love. It fails to detect
> >>> these problems because it scans out a square image. Making it non-square
> >>> would at least catch the use of the scaler when it shouldn't be used.
> >>>
> >>> Detecting the watermark breakage is less clear. I suppose making the
> >>> plane have a very wide or very tall aspect ratio might help induce
> >>> underruns with the broken wm code.
> >>>
> >>> Another thing that may or may not be missing from the test is panning.
> >>> I'd also like to test scaling, but sadly our hardware makes that
> >>> rather hard by not allowing us to force nearest and/or linear filtering,
> >>> and bspec doesn't actually document what kind of algorithm the hardware
> >>> uses for the different filter modes.
> >>>
> >> Agreed, the whole series is useful but until we have some tests we may as well not commit it. Nothing prevents it from being broken again in the next commit. :(
> >
> > In case tests hit a stumbling blocks/delays, I would appreciate if this got reviewed and merged soonish. As it stands I've been applying (and occasionally forgetting to apply) patches locally since September.
> >
> > And FWIW I would report if it got re-broken, since I'm using monitors in portrait, and like to run recent drm-tip to help catch issues missed elsewhere.
> >
> >> I'll take a look and see if I can make kms_rotation_crc break without this test.
> >
> > Would also need to upgrade the test to basic, or count on extended runs getting attention soon?
> Maybe?
>
> I've pushed the fixed test. Managed to test that the scaler is enabled incorrectly and the WM underruns.
>
> Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Thanks for the review, and updating the tests.
Series pushed to dinq.
--
Ville Syrjälä
Intel OTC
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2017-06-06 16:38 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-03-31 18:00 [PATCH 0/3] drm/i915: Fix SKL+ 90/270 degree rotated scanout ville.syrjala
2017-03-31 18:00 ` [PATCH 1/3] drm/i915: Fix scaling check for 90/270 degree plane rotation ville.syrjala
2017-03-31 18:00 ` ville.syrjala
2017-03-31 18:00 ` [PATCH 2/3] drm/i915: Fix SKL+ watermarks for 90/270 rotation ville.syrjala
2017-03-31 18:00 ` [PATCH 3/3] drm/i915: Fix 90/270 rotated coordinates for FBC ville.syrjala
2017-04-03 17:57 ` Paulo Zanoni
2017-05-19 11:34 ` [Intel-gfx] " Tvrtko Ursulin
2017-05-19 11:34 ` Tvrtko Ursulin
2017-03-31 18:21 ` ✓ Fi.CI.BAT: success for drm/i915: Fix SKL+ 90/270 degree rotated scanout Patchwork
2017-03-31 21:23 ` [PATCH 0/3] " Chris Wilson
2017-04-05 13:49 ` Ville Syrjälä
2017-06-06 8:06 ` Maarten Lankhorst
2017-06-06 8:29 ` Tvrtko Ursulin
2017-06-06 15:15 ` Maarten Lankhorst
2017-06-06 16:38 ` Ville Syrjälä
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.