On Thu, Jul 12, 2018 at 8:56 AM, Takashi Iwai <tiwai@xxxxxxx> wrote: > On Thu, 12 Jul 2018 08:54:34 +0200, > Daniel Vetter wrote: >> >> On Thu, Jul 12, 2018 at 09:29:01AM +0800, Feng Tang wrote: >> > On Tue, Jun 26, 2018 at 10:29:16AM +0800, Feng Tang wrote: >> > > On Mon, Jun 25, 2018 at 05:36:32PM +0200, Daniel Vetter wrote: >> > >> > > Hi Daneil/Jani/Takashi, >> > > >> > > When I was testing this patch from Takashi, I further checked the kernel >> > > module code, and found that: we may need NOT to add any new codes to >> > > prepare for i915's async probe feature! >> > > >> > > Say when i915 module is being loader due to HDA's request_module() call, >> > > in the callchain, do_init_module() has such code: >> > > >> > > if (!mod->async_probe_requested && (current->flags & PF_USED_ASYNC)) >> > > async_synchronize_full(); >> > > >> > > This will garantee the asynced probe is done before it returns. >> > > >> > > I have just tested and this seems to be enough. If I am not wrong, then >> > > we can take the i915 async patch directly. What do you think? >> > >> > Ping for comments, thanks! >> >> Ram (who's working on the hdcp2 code) just learned the hard way that if >> i915 registration gets delayed then audio fails to load. So if you want to >> make i915 fully async, then you _must_ fix the audio load stuff. > > Does my component completion patch help for that scenario? Hm, must have missed it. Do you have a patchwork link? Also adding Ram so he can test this out. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
