Quoting Mika Kuoppala (2019-07-10 14:49:05) > Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> writes: > > +static inline unsigned int gen8_pt_count(u64 addr, u64 end) > > +{ > > + GEM_BUG_ON(addr >= end); > > + if ((addr ^ end) & ~I915_PDE_MASK) > > + return I915_PDES - (addr & I915_PDE_MASK); > > Ok, I yield on 512. I915_PDES is fine as it atleast > couples it to mask :O I had already removed them all! :-p Give or take a few GEN8_PDES since that ends up being the single constant we require. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx