Intel-GFX Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
To: intel-gfx@lists.freedesktop.org,
	Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
Subject: Re: [PATCH 0/6] drm/i915: expose RCS topology to userspace
Date: Mon, 8 Jan 2018 16:30:34 +0000	[thread overview]
Message-ID: <df362725-6f07-5b7d-a048-4ac3c15af45c@intel.com> (raw)
In-Reply-To: <20171218153520.14181-1-lionel.g.landwerlin@intel.com>

Hey Tvrtko,

I've had only one comment from Chris on this series (addressed locally).
Since atm you might be the only other person in need of this API, do you 
have any feedback on it?

Thanks,

-
Lionel

On 18/12/17 15:35, Lionel Landwerlin wrote:
> Hi all,
>
> This series a respin of a few attempts ([1], [2], [3]) to expose RCS
> topology to userspace.
>
> The motivation for this is to be able to monitor part of the GPU using
> i915 perf and have detailled knowledge about what execution units are
> fused off. Let's say you monitor slice 1, knowing how many EUs are
> actually usuable on that slice is critical to provide normalized
> numbers (like average busyness of the EUs on slice1) on that
> particular part of the GPU. Currently numbers returned like
> I915_PARAM_EU_TOTAL don't give us that information.
>
> This series introduces a new uAPI to query information from i915. It's
> partially inspired by the amdgpu_cs_ioctl, although we don't allow
> setting values onto the device, only queries. There is a good chance
> this new uAPI will be reused for exposing available engines (Tvrtko
> was looking into this).
>
> Thanks,
>
> [1]: https://patchwork.freedesktop.org/patch/185959/
> [2]: https://patchwork.freedesktop.org/series/33436/
> [3]: https://patchwork.freedesktop.org/series/33950/
>
> Lionel Landwerlin (6):
>    drm/i915: store all subslice masks
>    drm/i915/debugfs: reuse max slice/subslices already stored in sseu
>    drm/i915/debugfs: add rcs topology entry
>    drm/i915: add rcs topology to error state
>    drm/i915: add query uAPI
>    drm/i915: expose rcs topology through query uAPI
>
>   drivers/gpu/drm/i915/Makefile            |   1 +
>   drivers/gpu/drm/i915/i915_debugfs.c      |  87 ++++++++++-----
>   drivers/gpu/drm/i915/i915_drv.c          |   3 +-
>   drivers/gpu/drm/i915/i915_drv.h          |  26 ++++-
>   drivers/gpu/drm/i915/i915_gpu_error.c    |  35 ++++++
>   drivers/gpu/drm/i915/i915_query.c        | 186 +++++++++++++++++++++++++++++++
>   drivers/gpu/drm/i915/intel_device_info.c | 169 ++++++++++++++++++++--------
>   drivers/gpu/drm/i915/intel_lrc.c         |   2 +-
>   drivers/gpu/drm/i915/intel_query_info.c  |  88 +++++++++++++++
>   drivers/gpu/drm/i915/intel_ringbuffer.h  |   2 +-
>   include/uapi/drm/i915_drm.h              |  76 +++++++++++++
>   11 files changed, 598 insertions(+), 77 deletions(-)
>   create mode 100644 drivers/gpu/drm/i915/i915_query.c
>   create mode 100644 drivers/gpu/drm/i915/intel_query_info.c
>
> --
> 2.15.1
>

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

      parent reply	other threads:[~2018-01-08 16:30 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-18 15:35 [PATCH 0/6] drm/i915: expose RCS topology to userspace Lionel Landwerlin
2017-12-18 15:35 ` [PATCH 1/6] drm/i915: store all subslice masks Lionel Landwerlin
2018-01-11 11:12   ` Tvrtko Ursulin
2018-01-11 15:48     ` Lionel Landwerlin
2018-01-11 15:57       ` Tvrtko Ursulin
2018-01-11 16:00     ` Lionel Landwerlin
2017-12-18 15:35 ` [PATCH 2/6] drm/i915/debugfs: reuse max slice/subslices already stored in sseu Lionel Landwerlin
2018-01-11 11:21   ` Tvrtko Ursulin
2018-01-11 15:50     ` Lionel Landwerlin
2017-12-18 15:35 ` [PATCH 3/6] drm/i915/debugfs: add rcs topology entry Lionel Landwerlin
2018-01-11 11:31   ` Tvrtko Ursulin
2018-01-11 17:04     ` Lionel Landwerlin
2017-12-18 15:35 ` [PATCH 4/6] drm/i915: add rcs topology to error state Lionel Landwerlin
2017-12-18 15:43   ` Chris Wilson
2017-12-18 15:35 ` [PATCH 5/6] drm/i915: add query uAPI Lionel Landwerlin
2018-01-11 12:19   ` Tvrtko Ursulin
2018-01-11 17:08     ` Lionel Landwerlin
2017-12-18 15:35 ` [PATCH 6/6] drm/i915: expose rcs topology through " Lionel Landwerlin
2018-01-11 12:45   ` Tvrtko Ursulin
2018-01-11 18:38     ` Lionel Landwerlin
2018-01-12  8:55       ` Tvrtko Ursulin
2017-12-18 16:07 ` ✗ Fi.CI.BAT: warning for drm/i915: expose RCS topology to userspace Patchwork
2018-01-08 16:30 ` Lionel Landwerlin [this message]

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=df362725-6f07-5b7d-a048-4ac3c15af45c@intel.com \
    --to=lionel.g.landwerlin@intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=tvrtko.ursulin@linux.intel.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox