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>
Cc: igt-dev@lists.freedesktop.org
Subject: Re: [igt-dev] [PATCH i-g-t v2 7/7] tests/gem_render_copy: Add a subtest for AUX_CCS_E
Date: Fri, 6 Jul 2018 15:02:00 +0300	[thread overview]
Message-ID: <20180706120200.GH5565@intel.com> (raw)
In-Reply-To: <153081950867.7594.5867843616964241359@cwilso3-mobl.ger.corp.intel.com>

On Thu, Jul 05, 2018 at 08:38:29PM +0100, Chris Wilson wrote:
> Quoting Ville Syrjala (2018-07-05 19:28:09)
> > From: Ville Syrjälä <ville.syrjala@linux.intel.com>
> > 
> > Add a new subtest that does renders the test pattern into a
> > compressed buffer. And we'll follow it up with another copy
> > back to an uncompressed buffer so that we also test the
> > capability to sampled from compressed buffers, and also so
> > that we can actually compare the results against the reference
> > image.
> 
> Ok, so CCS is being carried along in an adjacent image allocated just
> after the principle buf. (From the SURFACE_STATE, I expect the HW can
> use any address as compressed storage, so this layout is for
> convenience? I ask because I was thinking that maybe it would be easier
> to have a tiled bo and a separate aux bo.)

Hmm. I guess the render engine doesn't have that annoying "aux must
be above the main surface" limitation of the display engine. So yeah,
we could add a separate aux bo here. But any test involving the
display would need to stick to the one bo apporach.

-- 
Ville Syrjälä
Intel
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

  reply	other threads:[~2018-07-06 12:02 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-07-04 16:16 [igt-dev] [PATCH i-g-t 1/7] lib/rendercopy: Simplify reloc offsets Ville Syrjala
2018-07-04 16:16 ` [igt-dev] [PATCH i-g-t 2/7] lib: Constify igt_buf Ville Syrjala
2018-07-04 16:47   ` Chris Wilson
2018-07-04 16:16 ` [igt-dev] [PATCH i-g-t 3/7] lib/rendercopy: Set the upper 32bits of surface base address on gen8+ Ville Syrjala
2018-07-04 16:37   ` Chris Wilson
2018-07-04 16:41   ` Chris Wilson
2018-07-04 18:47     ` Ville Syrjälä
2018-07-04 18:51       ` Chris Wilson
2018-07-05 18:44   ` Chris Wilson
2018-07-04 16:16 ` [igt-dev] [PATCH i-g-t 4/7] lib: Add aux surface state to igt_buf Ville Syrjala
2018-07-05 18:45   ` Chris Wilson
2018-07-04 16:16 ` [igt-dev] [PATCH i-g-t 5/7] lib/rendercopy: Add enough surface state for AUX_CCS_E Ville Syrjala
2018-07-05 18:50   ` Chris Wilson
2018-07-04 16:16 ` [igt-dev] [PATCH i-g-t 6/7] tests/gem_render_copy: Add subtests for different tiling modes Ville Syrjala
2018-07-04 16:36   ` Chris Wilson
2018-07-04 16:45   ` Chris Wilson
2018-07-05 18:27   ` [igt-dev] [PATCH i-g-t v2 " Ville Syrjala
2018-07-05 18:43     ` Chris Wilson
2018-07-04 16:16 ` [igt-dev] [PATCH i-g-t 7/7] tests/gem_render_copy: Add a subtest for AUX_CCS_E Ville Syrjala
2018-07-05 18:28   ` [igt-dev] [PATCH i-g-t v2 " Ville Syrjala
2018-07-05 19:38     ` Chris Wilson
2018-07-06 12:02       ` Ville Syrjälä [this message]
2018-07-04 16:46 ` [igt-dev] [PATCH i-g-t 1/7] lib/rendercopy: Simplify reloc offsets Chris Wilson
2018-07-04 17:10 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/7] " Patchwork
2018-07-04 22:31 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
2018-07-06  0:03 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/7] lib/rendercopy: Simplify reloc offsets (rev3) Patchwork
2018-07-06 15:26 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
2018-07-06 16:42   ` 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=20180706120200.GH5565@intel.com \
    --to=ville.syrjala@linux.intel.com \
    --cc=chris@chris-wilson.co.uk \
    --cc=igt-dev@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.