On Mon, Jan 18, 2016 at 10:01:24AM +0000, Tvrtko Ursulin wrote: > > On 13/01/16 19:11, Dave Gordon wrote: > >On 13/01/16 19:01, yu.dai@xxxxxxxxx wrote: > >>From: Alex Dai <yu.dai@xxxxxxxxx> > >> > >>During driver unloading, the guc_client created for command submission > >>needs to be released to avoid memory leak. > >> > >>The struct_mutex needs to be held before tearing down GuC. > >> > >>v1: Move i915_guc_submission_disable out of i915_guc_submission_fini and > >> take struct_mutex lock before release GuC client. (Dave Gordon) > >>v2: Add the locking for failure case in guc_fw_fetch. (Dave Gordon) > >> Add i915_guc_submission_fini for failure case in > >>intel_guc_ucode_load. > >> > >>Signed-off-by: Alex Dai <yu.dai@xxxxxxxxx> > > > >LGTM. > > > >Reviewed-by: Dave Gordon <david.s.gordon@xxxxxxxxx> > > Patch merged, thanks! CI resulted in warnings for this patch. Per our latest discussion in Jesse's meeting please reply to the CI mail with your analysis of why these are all preexisting failures and with links to bugzilla. Otherwise this patch can't go in. I'll paste you the link to the internally wiki with the BKM. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx