On Thu, 26 Jun 2014 18:23:56 +0100 John.C.Harrison@xxxxxxxxx wrote: > + * Premption-related registers > + */ > +#define RING_UHPTR(base) ((base)+0x134) > +#define UHPTR_GFX_ADDR_ALIGN (0x7) > +#define UHPTR_VALID (0x1) > +#define RING_PREEMPT_ADDR 0x0214c > +#define PREEMPT_BATCH_LEVEL_MASK (0x3) > +#define BB_PREEMPT_ADDR 0x02148 > +#define SBB_PREEMPT_ADDR 0x0213c > +#define RS_PREEMPT_STATUS 0x0215c I couldn't find these easily, and the GFX_ADDR_ALIGN is just page alignment right? So you might not need that one. But overall looks fine. Reviewed-by: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> -- Jesse Barnes, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx