On Fri, Oct 16, 2015 at 07:39:20PM +0530, Goel, Akash wrote: > > Discussed sometime back about this patch with Chris. He mainly has 2 > concerns with it. > 1. The linear walk used by the patch to detect the overlapping > objects would be expensive. > 2. Restriction to disallow !RCS submissions for non-default > contexts, which could lead to lot of conflicts for the placements, > once multiple engines like media/blit/vebox are used > > Both of them can be addressed by the subsequent patches. > Chris already has the patch ready to reduce the validation overhead > with the use of rbtree and there should be no implications of > allowing the non-default contexts for !RCS submissions in > execbuffer. > > In the standalone form, the patch looks good, so > Reviewed-by: "Akash Goel <akash.goel@xxxxxxxxx>" I am sorry, but this patch does not reflect the final version that I wrote, in particular does not provide the support I actually use inside the kernel. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
