All of lore.kernel.org
 help / color / mirror / Atom feed
From: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
To: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>,
	intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH v3 0/9] drm/i915/perf: add OA interrupt support
Date: Fri, 26 Apr 2019 16:20:32 +0800	[thread overview]
Message-ID: <63d8b5e1-9ac2-b25d-0b8e-eb1a29ecc40c@intel.com> (raw)
In-Reply-To: <d7232526-8f7f-b506-3b52-3a33286b3596@intel.com>

FYI,

MDAPI got tired of waiting for this to land upstream : 
https://github.com/intel/metrics-discovery/commit/7b6399d5d5e5ef5fcc018a48853b46d0803da441

Apart from squashing the last commit, any other change needed?

Thanks,

-Lionel

On 03/04/2019 00:36, Lionel Landwerlin wrote:
> On 02/04/2019 14:49, Joonas Lahtinen wrote:
>> Quoting Lionel Landwerlin (2019-03-25 12:34:44)
>>> Ping?
>> The last patch should be squashed, I think we want to minimize the
>> amount of versions. Or do you intend to backport only portion of
>> the series somewhere?
>
>
> No backport intended :)
>
>
>>
>> Can you link to the userspace side changes?
>
>
> https://github.com/rib/gputop/pull/198
>
>
>>
>> Regards, Joonas
>>
>>> On 26/02/2019 14:29, Lionel Landwerlin wrote:
>>>> Hi all,
>>>>
>>>> This third iteration adds an i915 perf revision number through
>>>> getparam so that application can more easily find out what feature of
>>>> i915-perf are available.
>>>>
>>>> The patches containing uAPI updates have been updated to indicate what
>>>> version is required to those changes to be available.
>>>>
>>>> Cheers,
>>>>
>>>> Lionel Landwerlin (9):
>>>>     drm/i915/perf: rework aging tail workaround
>>>>     drm/i915/perf: move pollin setup to non hw specific code
>>>>     drm/i915/perf: only append status when data is available
>>>>     drm/i915/perf: introduce a versioning of the i915-perf uapi
>>>>     drm/i915/perf: add new open param to configure polling of OA 
>>>> buffer
>>>>     drm/i915: handle interrupts from the OA unit
>>>>     drm/i915/perf: add interrupt enabling parameter
>>>>     drm/i915/perf: add flushing ioctl
>>>>     drm/i915/perf: bump i915-perf revision
>>>>
>>>>    drivers/gpu/drm/i915/i915_drv.c         |   3 +
>>>>    drivers/gpu/drm/i915/i915_drv.h         |  59 +++-
>>>>    drivers/gpu/drm/i915/i915_irq.c         |  39 ++-
>>>>    drivers/gpu/drm/i915/i915_perf.c        | 403 
>>>> ++++++++++++++----------
>>>>    drivers/gpu/drm/i915/i915_reg.h         |   7 +
>>>>    drivers/gpu/drm/i915/intel_ringbuffer.c |   2 +
>>>>    include/uapi/drm/i915_drm.h             |  61 ++++
>>>>    7 files changed, 390 insertions(+), 184 deletions(-)
>>>>
>>>> -- 
>>>> 2.20.1
>>>>
>>> _______________________________________________
>>> Intel-gfx mailing list
>>> Intel-gfx@lists.freedesktop.org
>>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx


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

      reply	other threads:[~2019-04-26  8:20 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-02-26 14:29 [PATCH v3 0/9] drm/i915/perf: add OA interrupt support Lionel Landwerlin
2019-02-26 14:29 ` [PATCH v3 1/9] drm/i915/perf: rework aging tail workaround Lionel Landwerlin
2019-02-26 14:29 ` [PATCH v3 2/9] drm/i915/perf: move pollin setup to non hw specific code Lionel Landwerlin
2019-02-26 14:29 ` [PATCH v3 3/9] drm/i915/perf: only append status when data is available Lionel Landwerlin
2019-02-26 14:29 ` [PATCH v3 4/9] drm/i915/perf: introduce a versioning of the i915-perf uapi Lionel Landwerlin
2019-02-26 14:29 ` [PATCH v3 5/9] drm/i915/perf: add new open param to configure polling of OA buffer Lionel Landwerlin
2019-02-26 14:29 ` [PATCH v3 6/9] drm/i915: handle interrupts from the OA unit Lionel Landwerlin
2019-02-26 14:29 ` [PATCH v3 7/9] drm/i915/perf: add interrupt enabling parameter Lionel Landwerlin
2019-02-26 14:29 ` [PATCH v3 8/9] drm/i915/perf: add flushing ioctl Lionel Landwerlin
2019-02-26 14:29 ` [PATCH v3 9/9] drm/i915/perf: bump i915-perf revision Lionel Landwerlin
2019-02-26 14:55 ` ✗ Fi.CI.CHECKPATCH: warning for drm/i915/perf: add OA interrupt support (rev4) Patchwork
2019-02-26 14:59 ` ✗ Fi.CI.SPARSE: " Patchwork
2019-02-26 15:20 ` ✓ Fi.CI.BAT: success " Patchwork
2019-02-26 18:04 ` ✓ Fi.CI.IGT: " Patchwork
2019-03-25 10:34 ` [PATCH v3 0/9] drm/i915/perf: add OA interrupt support Lionel Landwerlin
2019-04-02 11:49   ` Joonas Lahtinen
2019-04-02 16:36     ` Lionel Landwerlin
2019-04-26  8:20       ` 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=63d8b5e1-9ac2-b25d-0b8e-eb1a29ecc40c@intel.com \
    --to=lionel.g.landwerlin@intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=joonas.lahtinen@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 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.