From patchwork Tue Apr 7 15:21:18 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Chris Wilson X-Patchwork-Id: 6172341 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 85E7A9F725 for ; Tue, 7 Apr 2015 15:22:18 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 66994203A5 for ; Tue, 7 Apr 2015 15:22:17 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) by mail.kernel.org (Postfix) with ESMTP id 5817F203AD for ; Tue, 7 Apr 2015 15:22:16 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 66F966E5AE; Tue, 7 Apr 2015 08:22:15 -0700 (PDT) X-Original-To: intel-gfx@lists.freedesktop.org Delivered-To: intel-gfx@lists.freedesktop.org Received: from relay.fireflyinternet.com (hostedrelay.fireflyinternet.com [109.228.30.76]) by gabe.freedesktop.org (Postfix) with ESMTP id 805726E5AB for ; Tue, 7 Apr 2015 08:22:13 -0700 (PDT) X-Default-Received-SPF: pass (skip=forwardok (res=PASS)) x-ip-name=78.156.65.138; Received: from haswell.alporthouse.com (unverified [78.156.65.138]) by relay.fireflyinternet.com (FireflyRelay1) with ESMTP id 439648-1305619 for multiple; Tue, 07 Apr 2015 16:22:12 +0100 From: Chris Wilson To: intel-gfx@lists.freedesktop.org Date: Tue, 7 Apr 2015 16:21:18 +0100 Message-Id: <1428420094-18352-55-git-send-email-chris@chris-wilson.co.uk> X-Mailer: git-send-email 2.1.4 In-Reply-To: <1428420094-18352-1-git-send-email-chris@chris-wilson.co.uk> References: <1428420094-18352-1-git-send-email-chris@chris-wilson.co.uk> X-Authenticated-User: chris.alporthouse@surfanytime.net Subject: [Intel-gfx] [PATCH 54/70] drm/i915: Cache last cmd descriptor when parsing 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, T_RP_MATCHES_RCVD,UNPARSEABLE_RELAY autolearn=ham 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 The cmd parser has the biggest impact on the BLT ring, because it is relatively verbose composed to the other engines as the vertex data is inline. It also typically has runs of repeating commands (again since the vertex data is inline, it typically has sequences of XY_SETUP_BLT, XY_SCANLINE_BLT..) We can easily reduce the impact of cmdparsing on benchmarks by then caching the last descriptor and comparing it against the next command header. To get maximum benefit, we also want to take advantage of skipping a few validations and length determinations if the header is unchanged between commands. ivb i7-3720QM: x11perf -dot: before 52.3M, after 124M (max 203M) glxgears: before 7310 fps, after 7550 fps (max 7860 fps) v2: Fix initial cached cmd descriptor to match MI_NOOP. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/i915_cmd_parser.c | 129 +++++++++++++++------------------ drivers/gpu/drm/i915/i915_drv.h | 10 +-- 2 files changed, 62 insertions(+), 77 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_cmd_parser.c b/drivers/gpu/drm/i915/i915_cmd_parser.c index 4a80ab953715..f4d3e7dc3835 100644 --- a/drivers/gpu/drm/i915/i915_cmd_parser.c +++ b/drivers/gpu/drm/i915/i915_cmd_parser.c @@ -754,6 +754,9 @@ void i915_cmd_parser_fini_ring(struct intel_engine_cs *ring) fini_hash_table(ring); } +/* + * Returns a pointer to a descriptor for the command specified by cmd_header. + */ static const struct drm_i915_cmd_descriptor* find_cmd_in_table(struct intel_engine_cs *ring, u32 cmd_header) @@ -773,37 +776,6 @@ find_cmd_in_table(struct intel_engine_cs *ring, return NULL; } -/* - * Returns a pointer to a descriptor for the command specified by cmd_header. - * - * The caller must supply space for a default descriptor via the default_desc - * parameter. If no descriptor for the specified command exists in the ring's - * command parser tables, this function fills in default_desc based on the - * ring's default length encoding and returns default_desc. - */ -static const struct drm_i915_cmd_descriptor* -find_cmd(struct intel_engine_cs *ring, - u32 cmd_header, - struct drm_i915_cmd_descriptor *default_desc) -{ - const struct drm_i915_cmd_descriptor *desc; - u32 mask; - - desc = find_cmd_in_table(ring, cmd_header); - if (desc) - return desc; - - mask = ring->get_cmd_length_mask(cmd_header); - if (!mask) - return NULL; - - BUG_ON(!default_desc); - default_desc->flags = CMD_DESC_SKIP; - default_desc->length.mask = mask; - - return default_desc; -} - static bool valid_reg(const u32 *table, int count, u32 addr) { if (table && count != 0) { @@ -844,17 +816,6 @@ static bool check_cmd(const struct intel_engine_cs *ring, const bool is_master, bool *oacontrol_set) { - if (desc->flags & CMD_DESC_REJECT) { - DRM_DEBUG_DRIVER("CMD: Rejected command: 0x%08X\n", *cmd); - return false; - } - - if ((desc->flags & CMD_DESC_MASTER) && !is_master) { - DRM_DEBUG_DRIVER("CMD: Rejected master-only command: 0x%08X\n", - *cmd); - return false; - } - if (desc->flags & CMD_DESC_REGISTER) { u32 reg_addr = cmd[desc->reg.offset] & desc->reg.mask; @@ -954,12 +915,13 @@ int i915_parse_cmds(struct intel_engine_cs *ring, { u32 tmp[128]; struct sg_page_iter src_iter, dst_iter; - const struct drm_i915_cmd_descriptor *desc; + struct drm_i915_cmd_descriptor default_desc = { CMD_DESC_SKIP }; + const struct drm_i915_cmd_descriptor *desc = &default_desc; + u32 last_cmd_header = 0; int needs_clflush = 0; void *src, *dst; unsigned in, out; - u32 *buf, partial = 0, length; - struct drm_i915_cmd_descriptor default_desc = { 0 }; + u32 *buf, partial = 0, length = 1; bool oacontrol_set = false; /* OACONTROL tracking. See check_cmd() */ int ret = 0; @@ -1053,37 +1015,60 @@ int i915_parse_cmds(struct intel_engine_cs *ring, } do { - if (*cmd == MI_BATCH_BUFFER_END) { - if (oacontrol_set) { - DRM_DEBUG_DRIVER("CMD: batch set OACONTROL but did not clear it\n"); - ret = -EINVAL; - } else - ret = 0; - goto unmap; - } + if (*cmd != last_cmd_header) { + if (*cmd == MI_BATCH_BUFFER_END) { + if (unlikely(oacontrol_set)) { + DRM_DEBUG_DRIVER("CMD: batch set OACONTROL but did not clear it\n"); + ret = -EINVAL; + } else + ret = 0; + goto unmap; + } - desc = find_cmd(ring, *cmd, &default_desc); - if (!desc) { - DRM_DEBUG_DRIVER("CMD: Unrecognized command: 0x%08X\n", - *cmd); - goto unmap; - } + desc = find_cmd_in_table(ring, *cmd); + if (desc) { + if (unlikely(desc->flags & CMD_DESC_REJECT)) { + DRM_DEBUG_DRIVER("CMD: Rejected command: 0x%08X\n", *cmd); + goto unmap; + } + + if (unlikely((desc->flags & CMD_DESC_MASTER) && !is_master)) { + DRM_DEBUG_DRIVER("CMD: Rejected master-only command: 0x%08X\n", *cmd); + goto unmap; + } + + /* + * If the batch buffer contains a + * chained batch, return an error that + * tells the caller to abort and + * dispatch the workload as a + * non-secure batch. + */ + if (unlikely(desc->cmd.value == MI_BATCH_BUFFER_START)) { + ret = -EACCES; + goto unmap; + } + + if (desc->flags & CMD_DESC_FIXED) + length = desc->length.fixed; + else + length = (*cmd & desc->length.mask) + LENGTH_BIAS; + } else { + u32 mask = ring->get_cmd_length_mask(*cmd); + if (unlikely(!mask)) { + DRM_DEBUG_DRIVER("CMD: Unrecognized command: 0x%08X\n", *cmd); + goto unmap; + } + + default_desc.length.mask = mask; + desc = &default_desc; + + length = (*cmd & mask) + LENGTH_BIAS; + } - /* - * If the batch buffer contains a chained batch, return an - * error that tells the caller to abort and dispatch the - * workload as a non-secure batch. - */ - if (desc->cmd.value == MI_BATCH_BUFFER_START) { - ret = -EACCES; - goto unmap; + last_cmd_header = *cmd; } - if (desc->flags & CMD_DESC_FIXED) - length = desc->length.fixed; - else - length = ((*cmd & desc->length.mask) + LENGTH_BIAS); - if (cmd + length > page_end) { if (length + cmd > batch_end) { DRM_DEBUG_DRIVER("CMD: Command length exceeds batch length: 0x%08X length=%u batchlen=%td\n", diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h index 4b46c5b5eb44..c48909f6baa2 100644 --- a/drivers/gpu/drm/i915/i915_drv.h +++ b/drivers/gpu/drm/i915/i915_drv.h @@ -2205,12 +2205,12 @@ struct drm_i915_cmd_descriptor { * is the DRM master */ u32 flags; +#define CMD_DESC_SKIP (0) #define CMD_DESC_FIXED (1<<0) -#define CMD_DESC_SKIP (1<<1) -#define CMD_DESC_REJECT (1<<2) -#define CMD_DESC_REGISTER (1<<3) -#define CMD_DESC_BITMASK (1<<4) -#define CMD_DESC_MASTER (1<<5) +#define CMD_DESC_REJECT (1<<1) +#define CMD_DESC_REGISTER (1<<2) +#define CMD_DESC_BITMASK (1<<3) +#define CMD_DESC_MASTER (1<<4) /* * The command's unique identification bits and the bitmask to get them.