Ok, Please ignore this patch. It is really part of the pre-emption series. Was included as is being used in Android. Peter. -----Original Message----- From: Chris Wilson [mailto:chris@xxxxxxxxxxxxxxxxxx] Sent: Friday, January 8, 2016 3:20 PM To: Antoine, Peter Cc: intel-gfx@xxxxxxxxxxxxxxxxxxxxx; daniel.vetter@xxxxxxxx; Gordon, Dave Subject: Re: [PATCH 3/3] drm/i915: Wait after context init with GuC Submission On Fri, Jan 08, 2016 at 03:03:54PM +0000, Peter Antoine wrote: > Per-context initialisation GPU instructions (which are injected > directly into the ringbuffer rather than being submitted as a batch) > should not be allowed to mix with user-generated batches in the same > submission; it will cause confusion for the GuC (which might merge a > subsequent preemptive request with the non-preemptive initialisation > code), and for the scheduler, which wouldn't know how to re-inject a > non-batch request if it were the victim of preemption. Move the context init into the first request. Problem solved. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
