From patchwork Tue Feb 16 05:27:15 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: sourab.gupta@intel.com X-Patchwork-Id: 8321111 Return-Path: X-Original-To: patchwork-intel-gfx@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 2DD169F38B for ; Tue, 16 Feb 2016 05:25:45 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 34A20202BE for ; Tue, 16 Feb 2016 05:25:44 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) by mail.kernel.org (Postfix) with ESMTP id 221DA2025A for ; Tue, 16 Feb 2016 05:25:43 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id B4F936E72D; Tue, 16 Feb 2016 05:25:42 +0000 (UTC) X-Original-To: intel-gfx@lists.freedesktop.org Delivered-To: intel-gfx@lists.freedesktop.org Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by gabe.freedesktop.org (Postfix) with ESMTP id 826756E72D for ; Tue, 16 Feb 2016 05:25:41 +0000 (UTC) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga102.jf.intel.com with ESMTP; 15 Feb 2016 21:25:40 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.22,454,1449561600"; d="scan'208";a="903355337" Received: from sourab-desktop.iind.intel.com ([10.223.82.63]) by fmsmga001.fm.intel.com with ESMTP; 15 Feb 2016 21:25:38 -0800 From: sourab.gupta@intel.com To: intel-gfx@lists.freedesktop.org Date: Tue, 16 Feb 2016 10:57:15 +0530 Message-Id: <1455600439-18480-8-git-send-email-sourab.gupta@intel.com> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1455600439-18480-1-git-send-email-sourab.gupta@intel.com> References: <1455600439-18480-1-git-send-email-sourab.gupta@intel.com> Cc: Jabin Wu , Sourab Gupta Subject: [Intel-gfx] [PATCH 07/11] drm/i915: Add support for having pid output with OA report X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_MED, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP From: Sourab Gupta This patch introduces flags and adds support for having pid output with the OA reports generated through the RCS commands. When the stream is opened with pid sample type, the pid information is also captured through the command stream samples and forwarded along with the OA reports. Signed-off-by: Sourab Gupta --- drivers/gpu/drm/i915/i915_drv.h | 1 + drivers/gpu/drm/i915/i915_perf.c | 39 ++++++++++++++++++++++++++++++++++++++- include/uapi/drm/i915_drm.h | 7 +++++++ 3 files changed, 46 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h index 44fcbf4..a8b374f 100644 --- a/drivers/gpu/drm/i915/i915_drv.h +++ b/drivers/gpu/drm/i915/i915_drv.h @@ -1808,6 +1808,7 @@ struct i915_perf_cs_data_node { struct drm_i915_gem_request *request; u32 offset; u32 ctx_id; + u32 pid; }; struct drm_i915_private { diff --git a/drivers/gpu/drm/i915/i915_perf.c b/drivers/gpu/drm/i915/i915_perf.c index 2904745..ea331eb 100644 --- a/drivers/gpu/drm/i915/i915_perf.c +++ b/drivers/gpu/drm/i915/i915_perf.c @@ -55,6 +55,7 @@ static u32 i915_perf_stream_paranoid = true; struct oa_sample_data { u32 source; u32 ctx_id; + u32 pid; const u8 *report; }; @@ -96,6 +97,7 @@ static struct i915_oa_format gen8_plus_oa_formats[I915_OA_FORMAT_MAX] = { #define SAMPLE_OA_REPORT (1<<0) #define SAMPLE_OA_SOURCE_INFO (1<<1) #define SAMPLE_CTX_ID (1<<2) +#define SAMPLE_PID (1<<3) struct perf_open_properties { @@ -255,6 +257,7 @@ static void i915_perf_command_stream_hook_oa(struct drm_i915_gem_request *req) } entry->ctx_id = ctx->global_id; + entry->pid = current->pid; i915_gem_request_assign(&entry->request, req); insert_perf_entry(dev_priv, entry); @@ -405,6 +408,12 @@ static bool append_oa_sample(struct i915_perf_stream *stream, read_state->buf += 4; } + if (sample_flags & SAMPLE_PID) { + if (copy_to_user(read_state->buf, &data->pid, 4)) + return false; + read_state->buf += 4; + } + if (sample_flags & SAMPLE_OA_REPORT) { if (copy_to_user(read_state->buf, data->report, report_size)) return false; @@ -446,6 +455,10 @@ static bool append_oa_buffer_sample(struct i915_perf_stream *stream, if (sample_flags & SAMPLE_CTX_ID) data.ctx_id = 0; +#warning "FIXME: append_oa_buffer_sample: deduce pid for periodic samples based on most recent RCS pid for ctx" + if (sample_flags & SAMPLE_PID) + data.pid = 0; + if (sample_flags & SAMPLE_OA_REPORT) data.report = report; @@ -685,6 +698,9 @@ static bool append_oa_rcs_sample(struct i915_perf_stream *stream, if (sample_flags & SAMPLE_CTX_ID) data.ctx_id = node->ctx_id; + if (sample_flags & SAMPLE_PID) + data.pid = node->pid; + if (sample_flags & SAMPLE_OA_REPORT) data.report = report; @@ -1307,6 +1323,7 @@ static int i915_oa_stream_init(struct i915_perf_stream *stream, struct drm_i915_private *dev_priv = stream->dev_priv; bool require_oa_unit = props->sample_flags & (SAMPLE_OA_REPORT | SAMPLE_OA_SOURCE_INFO); + bool require_cs_mode = props->sample_flags & SAMPLE_PID; int format_size; int ret; @@ -1416,8 +1433,20 @@ static int i915_oa_stream_init(struct i915_perf_stream *stream, require_cs_mode = true; } + if (require_cs_mode && !props->cs_mode) { + DRM_ERROR("PID sampling requires a ring to be specified"); + ret = -EINVAL; + goto cs_error; + } + if (props->cs_mode) { - if (!(props->sample_flags & SAMPLE_CTX_ID)) { + /* + * The only time we should allow enabling CS mode if it's not + * strictly required, is if SAMPLE_CTX_ID has been requested + * as it's usable with periodic OA or CS sampling. + */ + if (!require_cs_mode && + !(props->sample_flags & SAMPLE_CTX_ID)) { DRM_ERROR( "Ring given without requesting any CS data to sample"); ret = -EINVAL; @@ -1426,6 +1455,11 @@ static int i915_oa_stream_init(struct i915_perf_stream *stream, stream->cs_mode = true; + if (props->sample_flags & SAMPLE_PID) { + stream->sample_flags |= SAMPLE_PID; + stream->sample_size += 4; + } + ret = alloc_command_stream_buf(dev_priv); if (ret) goto cs_error; @@ -2045,6 +2079,9 @@ static int read_properties_unlocked(struct drm_i915_private *dev_priv, case DRM_I915_PERF_SAMPLE_CTX_ID_PROP: props->sample_flags |= SAMPLE_CTX_ID; break; + case DRM_I915_PERF_SAMPLE_PID_PROP: + props->sample_flags |= SAMPLE_PID; + break; case DRM_I915_PERF_PROP_MAX: BUG(); } diff --git a/include/uapi/drm/i915_drm.h b/include/uapi/drm/i915_drm.h index 55b553b..3353926 100644 --- a/include/uapi/drm/i915_drm.h +++ b/include/uapi/drm/i915_drm.h @@ -1219,6 +1219,12 @@ enum drm_i915_perf_property_id { */ DRM_I915_PERF_SAMPLE_CTX_ID_PROP, + /** + * The value of this property set to 1 requests inclusion of pid in the + * perf sample data. + */ + DRM_I915_PERF_SAMPLE_PID_PROP, + DRM_I915_PERF_PROP_MAX /* non-ABI */ }; @@ -1268,6 +1274,7 @@ enum drm_i915_perf_record_type { * * { u32 source_info; } && DRM_I915_PERF_SAMPLE_OA_SOURCE_PROP * { u32 ctx_id; } && DRM_I915_PERF_SAMPLE_CTX_ID_PROP + * { u32 pid; } && DRM_I915_PERF_SAMPLE_PID_PROP * { u32 oa_report[]; } && DRM_I915_PERF_SAMPLE_OA_PROP * }; */