All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v7 00/11] Enable i915 perf stream for Haswell OA unit
@ 2016-10-24 23:19 Robert Bragg
  2016-10-24 23:19 ` [PATCH v7 01/11] drm/i915: Add i915 perf infrastructure Robert Bragg
                   ` (11 more replies)
  0 siblings, 12 replies; 30+ messages in thread
From: Robert Bragg @ 2016-10-24 23:19 UTC (permalink / raw)
  To: intel-gfx
  Cc: dri-devel, Matthew Auld, Sourab Gupta, Daniel Vetter, Robert Bragg

Rebased on nightly, including recent review updates (CI wasn't happy picking up
the replies updating individual patches).

This also reverts back to pinning the context upfront when opening a stream for
a single context, instead of hooking into pinning and updating OACONTROL on the
fly.

Chris has repeatedly suggested he'd prefer to have the driver work with an
upfront pin, as it used to, instead of with the hook. It was changed last time
based on feedback considering some concern with the shrinker. At least from
inspection it does /seem/ safe to assume a pinned vma will reliably block the
shrinker from freeing ctx pages and the shrinker itself doesn't unpin things.
I'm not fully certain of the interaction with the _gem.c _context_lost() code
path which aims to unpin last_context. At least the code is a little simpler
this way, so maybe if Daniel is happy that his original concern was overly
cautious (or no longer an issue with the latest code), then this change is ok.

- Robert

Robert Bragg (11):
  drm/i915: Add i915 perf infrastructure
  drm/i915: rename OACONTROL GEN7_OACONTROL
  drm/i915: return EACCES for check_cmd() failures
  drm/i915: don't whitelist oacontrol in cmd parser
  drm/i915: Add 'render basic' Haswell OA unit config
  drm/i915: Enable i915 perf stream for Haswell OA unit
  drm/i915: advertise available metrics via sysfs
  drm/i915: Add dev.i915.perf_stream_paranoid sysctl option
  drm/i915: add oa_event_min_timer_exponent sysctl
  drm/i915: Add more Haswell OA metric sets
  drm/i915: Add a kerneldoc summary for i915_perf.c

 drivers/gpu/drm/i915/Makefile          |    4 +
 drivers/gpu/drm/i915/gvt/handlers.c    |    2 +-
 drivers/gpu/drm/i915/i915_cmd_parser.c |   45 +-
 drivers/gpu/drm/i915/i915_drv.c        |    9 +
 drivers/gpu/drm/i915/i915_drv.h        |  155 +++
 drivers/gpu/drm/i915/i915_oa_hsw.c     |  752 ++++++++++++++
 drivers/gpu/drm/i915/i915_oa_hsw.h     |   38 +
 drivers/gpu/drm/i915/i915_perf.c       | 1689 ++++++++++++++++++++++++++++++++
 drivers/gpu/drm/i915/i915_reg.h        |  340 ++++++-
 include/uapi/drm/i915_drm.h            |  133 +++
 10 files changed, 3127 insertions(+), 40 deletions(-)
 create mode 100644 drivers/gpu/drm/i915/i915_oa_hsw.c
 create mode 100644 drivers/gpu/drm/i915/i915_oa_hsw.h
 create mode 100644 drivers/gpu/drm/i915/i915_perf.c

-- 
2.10.1

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 30+ messages in thread

end of thread, other threads:[~2016-10-27 10:16 UTC | newest]

Thread overview: 30+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-10-24 23:19 [PATCH v7 00/11] Enable i915 perf stream for Haswell OA unit Robert Bragg
2016-10-24 23:19 ` [PATCH v7 01/11] drm/i915: Add i915 perf infrastructure Robert Bragg
2016-10-25 15:58   ` Matthew Auld
2016-10-24 23:19 ` [PATCH v7 02/11] drm/i915: rename OACONTROL GEN7_OACONTROL Robert Bragg
2016-10-24 23:19 ` [PATCH v7 03/11] drm/i915: return EACCES for check_cmd() failures Robert Bragg
2016-10-25 21:53   ` Matthew Auld
2016-10-24 23:19 ` [PATCH v7 04/11] drm/i915: don't whitelist oacontrol in cmd parser Robert Bragg
2016-10-25 22:02   ` Matthew Auld
2016-10-24 23:19 ` [PATCH v7 05/11] drm/i915: Add 'render basic' Haswell OA unit config Robert Bragg
2016-10-24 23:19 ` [PATCH v7 06/11] drm/i915: Enable i915 perf stream for Haswell OA unit Robert Bragg
2016-10-25 21:35   ` Matthew Auld
2016-10-25 23:51     ` Robert Bragg
2016-10-26  8:54       ` Chris Wilson
2016-10-26 15:17         ` Robert Bragg
2016-10-26 15:37           ` [Intel-gfx] " Ville Syrjälä
2016-10-26 16:42             ` Robert Bragg
2016-10-26 16:52               ` Daniel Vetter
2016-10-26 16:54               ` [Intel-gfx] " Ville Syrjälä
     [not found]                 ` <CAMou1-2j1FTGSCGh=0nq3BokVNe2U308M4STK-G6iz5-n5Gtfg@mail.gmail.com>
2016-10-26 18:53                   ` Robert Bragg
2016-10-26 10:08       ` Matthew Auld
2016-10-26 15:03         ` Robert Bragg
2016-10-26 21:53           ` Robert Bragg
2016-10-25 23:05   ` Chris Wilson
2016-10-25 23:12     ` Chris Wilson
2016-10-24 23:19 ` [PATCH v7 07/11] drm/i915: advertise available metrics via sysfs Robert Bragg
2016-10-24 23:19 ` [PATCH v7 08/11] drm/i915: Add dev.i915.perf_stream_paranoid sysctl option Robert Bragg
2016-10-24 23:19 ` [PATCH v7 09/11] drm/i915: add oa_event_min_timer_exponent sysctl Robert Bragg
2016-10-24 23:19 ` [PATCH v7 10/11] drm/i915: Add more Haswell OA metric sets Robert Bragg
2016-10-24 23:19 ` [PATCH v7 11/11] drm/i915: Add a kerneldoc summary for i915_perf.c Robert Bragg
2016-10-27 10:16 ` ✗ Fi.CI.BAT: failure for Enable i915 perf stream for Haswell OA unit Patchwork

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.