On Tue, 11 Feb 2014 14:19:03 +0530 akash.goel@xxxxxxxxx wrote: > @@ -810,6 +815,7 @@ static void gen6_ppgtt_insert_entries(struct i915_address_space *vm, > pt_vaddr[act_pte] = > vm->pte_encode(sg_page_iter_dma_address(&sg_iter), > cache_level, true); > + > if (++act_pte == I915_PPGTT_PT_ENTRIES) { > kunmap_atomic(pt_vaddr); > pt_vaddr = NULL; Some extra whitespace here. Otherwise: Reviewed-by: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> Might be good to expose this as a param too, so userspace could use it for stuff that shouldn't be written... -- Jesse Barnes, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx