From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Widawsky Subject: =?utf-8?q?=5BPATCH_07/10=5D_intel-gpu-tools=3A_regist?= =?utf-8?q?er_range_handling_for_forcewake_hooks?= Date: Sun, 17 Jul 2011 16:25:45 -0700 Message-ID: <1310945148-6777-8-git-send-email-ben@bwidawsk.net> References: <1310945148-6777-1-git-send-email-ben@bwidawsk.net> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1689707502==" Return-path: Received: from cloud01.chad-versace.us (184-106-247-128.static.cloud-ips.com [184.106.247.128]) by gabe.freedesktop.org (Postfix) with ESMTP id D1FFA9EB5E for ; Sun, 17 Jul 2011 16:25:56 -0700 (PDT) In-Reply-To: <1310945148-6777-1-git-send-email-ben@bwidawsk.net> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: intel-gfx@lists.freedesktop.org Cc: Ben Widawsky List-Id: intel-gfx@lists.freedesktop.org --===============1689707502== Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable We can deprecate the old code by using the non-safe flag in the new API. The safe flag should allow the previous behavior to continue. The code also adds some range checking on register access. This code is gives hooks to prevent tools from doing bad things. v2: fixes from Chris try /debug after /sys/kernel/debug fails stat the debugfs file before returning success whitespace fix warn for gen2/gen3/gen7 Cc: Chris Wilson Signed-off-by: Ben Widawsky --- lib/Makefile.am | 1 + lib/intel_chipset.h | 8 ++ lib/intel_gpu_tools.h | 27 ++++++++ lib/intel_mmio.c | 162 +++++++++++++++++++++++++++++++++++++++++++= +- lib/intel_reg_map.c | 178 +++++++++++++++++++++++++++++++++++++++++++= ++++++ 5 files changed, 375 insertions(+), 1 deletions(-) diff --git a/lib/Makefile.am b/lib/Makefile.am index 0c9380d..4612cd5 100644 --- a/lib/Makefile.am +++ b/lib/Makefile.am @@ -8,6 +8,7 @@ libintel_tools_la_SOURCES =3D \ intel_mmio.c \ intel_pci.c \ intel_reg.h \ + intel_reg_map.c \ instdone.c \ instdone.h \ drmtest.h diff --git a/lib/intel_chipset.h b/lib/intel_chipset.h index c3db3ab..a38f661 100755 --- a/lib/intel_chipset.h +++ b/lib/intel_chipset.h @@ -168,3 +168,11 @@ =20 #define HAS_BLT_RING(devid) (IS_GEN6(devid) || \ IS_GEN7(devid)) + +#define IS_BROADWATER(devid) (devid =3D=3D PCI_CHIP_I946_GZ || \ + devid =3D=3D PCI_CHIP_I965_G_1 || \ + devid =3D=3D PCI_CHIP_I965_Q || \ + devid =3D=3D PCI_CHIP_I965_G) + +#define IS_CRESTLINE(devid) (devid =3D=3D PCI_CHIP_I965_GM || \ + devid =3D=3D PCI_CHIP_I965_GME) diff --git a/lib/intel_gpu_tools.h b/lib/intel_gpu_tools.h index acee657..a145fb9 100644 --- a/lib/intel_gpu_tools.h +++ b/lib/intel_gpu_tools.h @@ -37,6 +37,33 @@ extern void *mmio; void intel_get_mmio(struct pci_device *pci_dev); =20 +/* New style register access API */ +int intel_register_access_init(struct pci_device *pci_dev, int safe); +void intel_register_access_fini(void); +uint32_t intel_register_read(uint32_t reg); +void intel_register_write(uint32_t reg, uint32_t val); + +#define INTEL_RANGE_RSVD (0<<0) /* Shouldn't be read or written */ +#define INTEL_RANGE_READ (1<<0) +#define INTEL_RANGE_WRITE (1<<1) +#define INTEL_RANGE_RW (INTEL_RANGE_READ | INTEL_RANGE_WRITE) +#define INTEL_RANGE_END (1<<31) + +struct intel_register_range { + uint32_t base; + uint32_t size; + uint32_t flags; +}; + +struct intel_register_map { + struct intel_register_range *map; + uint32_t top; + uint32_t alignment_mask; +}; +struct intel_register_map intel_get_register_map(uint32_t devid); +struct intel_register_range *intel_get_register_range(struct intel_regis= ter_map map, uint32_t offset, int mode); + + static inline uint32_t INREG(uint32_t reg) { diff --git a/lib/intel_mmio.c b/lib/intel_mmio.c index 0228a87..31967d9 100644 --- a/lib/intel_mmio.c +++ b/lib/intel_mmio.c @@ -22,12 +22,16 @@ * * Authors: * Eric Anholt + * Ben Widawsky * */ =20 #include -#include #include +#include +#include +#include +#include #include #include #include @@ -41,6 +45,16 @@ =20 void *mmio; =20 +static struct _mmio_data { + int inited; + bool safe; + char debugfs_path[FILENAME_MAX]; + char debugfs_forcewake_path[FILENAME_MAX]; + uint32_t i915_devid; + struct intel_register_map map; + int key; +} mmio_data; + void intel_map_file(char *file) { @@ -89,3 +103,149 @@ intel_get_mmio(struct pci_device *pci_dev) } } =20 +/* + * If successful, i915_debugfs_path and i915_debugfs_forcewake_path are = both + * updated with the correct path. + */ +static int +find_debugfs_path(char *dri_base) +{ + char buf[FILENAME_MAX]; + struct stat sb; + int i, ret; + + for (i =3D 0; i < 16; i++) { + snprintf(buf, FILENAME_MAX, "%s/%i/name", dri_base, i); + + snprintf(mmio_data.debugfs_path, FILENAME_MAX, + "%s/%i/", dri_base, i); + snprintf(mmio_data.debugfs_forcewake_path, FILENAME_MAX, + "%s/%i/i915_forcewake_user", dri_base, i); + + ret =3D stat(mmio_data.debugfs_forcewake_path, &sb); + if (ret) { + mmio_data.debugfs_path[0] =3D 0; + mmio_data.debugfs_forcewake_path[0] =3D 0; + } else + return 0; + } + + return -1; +} + +static int +get_forcewake_lock(void) +{ + return open(mmio_data.debugfs_forcewake_path, 0); +} + +static void +release_forcewake_lock(int fd) +{ + close(fd); +} + +/* + * Initialize register access library. + * + * @pci_dev: pci device we're mucking with + * @safe: use safe register access tables + */ +int +intel_register_access_init(struct pci_device *pci_dev, int safe) +{ + int ret; + + /* after old API is deprecated, remove this */ + if (mmio =3D=3D NULL) + intel_get_mmio(pci_dev); + + assert(mmio !=3D NULL); + + if (mmio_data.inited) + return -1; + + mmio_data.safe =3D safe !=3D 0 ? true : false; + + /* Find where the forcewake lock is */ + ret =3D find_debugfs_path("/sys/kernel/debug/dri"); + if (ret) { + ret =3D find_debugfs_path("/debug/dri"); + if (ret) { + fprintf(stderr, "Couldn't find path to dri/debugfs entry\n"); + return ret; + } + } + + mmio_data.i915_devid =3D pci_dev->device_id; + if (mmio_data.safe) + mmio_data.map =3D intel_get_register_map(mmio_data.i915_devid); + + mmio_data.key =3D get_forcewake_lock(); + mmio_data.inited++; + return 0; +} + +void +intel_register_access_fini(void) +{ + release_forcewake_lock(mmio_data.key); + mmio_data.inited--; +} + +uint32_t +intel_register_read(uint32_t reg) +{ + struct intel_register_range *range; + uint32_t ret; + + assert(mmio_data.inited); + + if (IS_GEN6(mmio_data.i915_devid)) + assert(mmio_data.key !=3D -1); + + if (!mmio_data.safe) + goto read_out; + + range =3D intel_get_register_range(mmio_data.map, + reg, + INTEL_RANGE_READ); + + if(!range) { + fprintf(stderr, "Register read blocked for safety " + "(*0x%08x)\n", reg); + ret =3D 0xffffffff; + goto out; + } + +read_out: + ret =3D *(volatile uint32_t *)((volatile char *)mmio + reg); +out: + return ret; +} + +void +intel_register_write(uint32_t reg, uint32_t val) +{ + struct intel_register_range *range; + + assert(mmio_data.inited); + + if (IS_GEN6(mmio_data.i915_devid)) + assert(mmio_data.key !=3D -1); + + if (!mmio_data.safe) + goto write_out; + + range =3D intel_get_register_range(mmio_data.map, + reg, + INTEL_RANGE_WRITE); + + if (!range) { + fprintf(stderr, "Register write blocked for safety " + "(*0x%08x =3D 0x%x)\n", reg, val); + } + +write_out: + *(volatile uint32_t *)((volatile char *)mmio + reg) =3D val; +} diff --git a/lib/intel_reg_map.c b/lib/intel_reg_map.c new file mode 100644 index 0000000..86cf9a6 --- /dev/null +++ b/lib/intel_reg_map.c @@ -0,0 +1,178 @@ +/* + * Copyright =A9 2011 Intel Corporation + * + * Permission is hereby granted, free of charge, to any person obtaining= a + * copy of this software and associated documentation files (the "Softwa= re"), + * to deal in the Software without restriction, including without limita= tion + * the rights to use, copy, modify, merge, publish, distribute, sublicen= se, + * and/or sell copies of the Software, and to permit persons to whom the + * Software is furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice (including the = next + * paragraph) shall be included in all copies or substantial portions of= the + * Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRE= SS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILI= TY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SH= ALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR = OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISI= NG + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER D= EALINGS + * IN THE SOFTWARE. + * + * Authors: + * Ben Widawsky + * + */ + +#include +#include +#include +#include +#include "intel_gpu_tools.h" + +static struct intel_register_range gen_bwcl_register_map[] =3D { + {0x00000000, 0x00000fff, INTEL_RANGE_RW}, + {0x00001000, 0x00000fff, INTEL_RANGE_RSVD}, + {0x00002000, 0x00000fff, INTEL_RANGE_RW}, + {0x00003000, 0x000001ff, INTEL_RANGE_RW}, + {0x00003200, 0x00000dff, INTEL_RANGE_RW}, + {0x00004000, 0x000003ff, INTEL_RANGE_RSVD}, + {0x00004400, 0x00000bff, INTEL_RANGE_RSVD}, + {0x00005000, 0x00000fff, INTEL_RANGE_RW}, + {0x00006000, 0x00000fff, INTEL_RANGE_RW}, + {0x00007000, 0x000003ff, INTEL_RANGE_RW}, + {0x00007400, 0x000014ff, INTEL_RANGE_RW}, + {0x00008900, 0x000006ff, INTEL_RANGE_RSVD}, + {0x00009000, 0x00000fff, INTEL_RANGE_RSVD}, + {0x0000a000, 0x00000fff, INTEL_RANGE_RW}, + {0x0000b000, 0x00004fff, INTEL_RANGE_RSVD}, + {0x00010000, 0x00003fff, INTEL_RANGE_RW}, + {0x00014000, 0x0001bfff, INTEL_RANGE_RSVD}, + {0x00030000, 0x0000ffff, INTEL_RANGE_RW}, + {0x00040000, 0x0001ffff, INTEL_RANGE_RSVD}, + {0x00060000, 0x0000ffff, INTEL_RANGE_RW}, + {0x00070000, 0x00002fff, INTEL_RANGE_RW}, + {0x00073000, 0x00000fff, INTEL_RANGE_RW}, + {0x00074000, 0x0000bfff, INTEL_RANGE_RSVD}, + {0x00000000, 0x00000000, INTEL_RANGE_END} +}; + +static struct intel_register_range gen4_register_map[] =3D { + {0x00000000, 0x00000fff, INTEL_RANGE_RW}, + {0x00001000, 0x00000fff, INTEL_RANGE_RSVD}, + {0x00002000, 0x00000fff, INTEL_RANGE_RW}, + {0x00003000, 0x000001ff, INTEL_RANGE_RW}, + {0x00003200, 0x00000dff, INTEL_RANGE_RW}, + {0x00004000, 0x000003ff, INTEL_RANGE_RW}, + {0x00004400, 0x00000bff, INTEL_RANGE_RW}, + {0x00005000, 0x00000fff, INTEL_RANGE_RW}, + {0x00006000, 0x00000fff, INTEL_RANGE_RW}, + {0x00007000, 0x000003ff, INTEL_RANGE_RW}, + {0x00007400, 0x000014ff, INTEL_RANGE_RW}, + {0x00008900, 0x000006ff, INTEL_RANGE_RSVD}, + {0x00009000, 0x00000fff, INTEL_RANGE_RSVD}, + {0x0000a000, 0x00000fff, INTEL_RANGE_RW}, + {0x0000b000, 0x00004fff, INTEL_RANGE_RSVD}, + {0x00010000, 0x00003fff, INTEL_RANGE_RW}, + {0x00014000, 0x0001bfff, INTEL_RANGE_RSVD}, + {0x00030000, 0x0000ffff, INTEL_RANGE_RW}, + {0x00040000, 0x0001ffff, INTEL_RANGE_RSVD}, + {0x00060000, 0x0000ffff, INTEL_RANGE_RW}, + {0x00070000, 0x00002fff, INTEL_RANGE_RW}, + {0x00073000, 0x00000fff, INTEL_RANGE_RW}, + {0x00074000, 0x0000bfff, INTEL_RANGE_RSVD}, + {0x00000000, 0x00000000, INTEL_RANGE_END} +}; + +/* The documentation is a little sketchy on these register ranges. */ +static struct intel_register_range gen6_gt_register_map[] =3D { + {0x00000000, 0x00000fff, INTEL_RANGE_RW}, + {0x00001000, 0x00000fff, INTEL_RANGE_RSVD}, + {0x00002000, 0x00000fff, INTEL_RANGE_RW}, + {0x00003000, 0x000001ff, INTEL_RANGE_RW}, + {0x00003200, 0x00000dff, INTEL_RANGE_RW}, + {0x00004000, 0x00000fff, INTEL_RANGE_RW}, + {0x00005000, 0x0000017f, INTEL_RANGE_RW}, + {0x00005180, 0x00000e7f, INTEL_RANGE_RW}, + {0x00006000, 0x00001fff, INTEL_RANGE_RW}, + {0x00008000, 0x000007ff, INTEL_RANGE_RW}, + {0x00008800, 0x000000ff, INTEL_RANGE_RSVD}, + {0x00008900, 0x000006ff, INTEL_RANGE_RW}, + {0x00009000, 0x00000fff, INTEL_RANGE_RSVD}, + {0x0000a000, 0x00000fff, INTEL_RANGE_RW}, + {0x0000b000, 0x00004fff, INTEL_RANGE_RSVD}, + {0x00010000, 0x00001fff, INTEL_RANGE_RW}, + {0x00012000, 0x000003ff, INTEL_RANGE_RW}, + {0x00012400, 0x00000bff, INTEL_RANGE_RW}, + {0x00013000, 0x00000fff, INTEL_RANGE_RW}, + {0x00014000, 0x00000fff, INTEL_RANGE_RW}, + {0x00015000, 0x0000cfff, INTEL_RANGE_RW}, + {0x00022000, 0x00000fff, INTEL_RANGE_RW}, + {0x00023000, 0x00000fff, INTEL_RANGE_RSVD}, + {0x00024000, 0x00000fff, INTEL_RANGE_RW}, + {0x00025000, 0x0000afff, INTEL_RANGE_RSVD}, + {0x00030000, 0x0000ffff, INTEL_RANGE_RW}, + {0x00040000, 0x0000ffff, INTEL_RANGE_RW}, + {0x00050000, 0x0000ffff, INTEL_RANGE_RW}, + {0x00060000, 0x0000ffff, INTEL_RANGE_RW}, + {0x00070000, 0x00003fff, INTEL_RANGE_RW}, + {0x00074000, 0x0008bfff, INTEL_RANGE_RSVD}, + {0x00100000, 0x00007fff, INTEL_RANGE_RW}, + {0x00108000, 0x00037fff, INTEL_RANGE_RSVD}, + {0x00140000, 0x0003ffff, INTEL_RANGE_RW}, + {0x00000000, 0x00000000, INTEL_RANGE_END} +}; + +struct intel_register_map +intel_get_register_map(uint32_t devid) +{ + struct intel_register_map map; + + if (IS_GEN6(devid)) { + map.map =3D gen6_gt_register_map; + map.top =3D 0x180000; + } else if (IS_BROADWATER(devid) || IS_CRESTLINE(devid)) { + map.map =3D gen_bwcl_register_map; + map.top =3D 0x80000; + } else if (IS_GEN4(devid) || IS_GEN5(devid)) { + map.map =3D gen4_register_map; + map.top =3D 0x80000; + } else { + fprintf(stderr, "Gen2/3 Ranges are not supported. Please use " + "unsafe access."); + abort(); + } + + map.alignment_mask =3D 0x3; + + return map; +} + +struct intel_register_range * +intel_get_register_range(struct intel_register_map map, uint32_t offset,= int mode) +{ + struct intel_register_range *range =3D map.map; + uint32_t align =3D map.alignment_mask; + + if (offset & map.alignment_mask) + return NULL; + + if (offset >=3D map.top) + return NULL; + + while (!(range->flags & INTEL_RANGE_END)) { + /* list is assumed to be in order */ + if (offset < range->base) + break; + + if ( (offset >=3D range->base) && + (offset + align) <=3D (range->base + range->size)) { + if ((mode & range->flags) =3D=3D mode) + return range; + } + range++; + } + + return NULL; +} --=20 1.7.6 --===============1689707502== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx --===============1689707502==--