All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
To: Chris Wilson <chris@chris-wilson.co.uk>, intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH 0/3] drm/i915: Fix SKL+ 90/270 degree rotated scanout
Date: Wed, 5 Apr 2017 16:49:00 +0300	[thread overview]
Message-ID: <20170405134900.GR30290@intel.com> (raw)
In-Reply-To: <20170331212318.GA9449@nuc-i3427.alporthouse.com>

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

  reply	other threads:[~2017-04-05 13:49 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
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   ` Tvrtko Ursulin
2017-05-19 11:34     ` [Intel-gfx] " 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ä [this message]
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ä

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170405134900.GR30290@intel.com \
    --to=ville.syrjala@linux.intel.com \
    --cc=chris@chris-wilson.co.uk \
    --cc=intel-gfx@lists.freedesktop.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.