[igt-dev] [PATCH i-g-t] lib/i915: Add a helper lib to read mmio registers via ioctl

Teres Alexis, Alan Previn alan.previn.teres.alexis at intel.com
Fri Aug 25 17:41:34 UTC 2023


On Fri, 2023-08-25 at 10:25 -0700, Teres Alexis, Alan Previn wrote:
> > > -	do_ioctl(drm_fd, DRM_IOCTL_I915_REG_READ, &rr);
> > > +	i915_reg_read_ioctl(drm_fd, 0x2358, &val);
> > ----------------------------------- ^^^^^^
> > Maybe define it at i915_mmio_reg.h ?
> > 
> > #define MMIO_REG_RENDER_RING_TIMESTAMP 0x2358
> > 
> alan: yes - my bad - i should have done that up front.
Just realized this thing is used/defined in a few places (below is the grep)
but i guess the right place would be lib/intel_reg.h with all the others?
I also feel we dont need the "MMIO_REG" in front (like others in intel_reg.h).
I'll re-rev and we can take a 2nd look. Thanks in advance.

   tests/i915/i915_pm_rpm.c:1862:	i915_reg_read_ioctl(drm_fd, 0x2358, &val);
   tests/i915/gem_reg_read.c:57:#define RENDER_RING_TIMESTAMP 0x2358
   tests/i915/gem_ctx_isolation.c:230:	{ "RCS timestamp", GEN6, ~0u, 0x2358 },
   
   tests/xe/xe_mmio.c:23:#define RCS_TIMESTAMP 0x2358
   
   lib/intel_reg.h:462:#define TIMESTAMP_QW           0x2358
   tools/intel_forcewaked.c:62:	return (intel_register_read(mmio_data, 0x2358) != 0);
   tools/i915-perf/i915_perf_recorder.c:626:#define RENDER_RING_TIMESTAMP 0x2358
   


More information about the igt-dev mailing list