Hi Mauro, On Tue, Jun 07, 2016 at 10:11:33AM -0300, Mauro Carvalho Chehab wrote: > Em Mon, 6 Jun 2016 17:13:12 -0600 > Shuah Khan <shuahkh@xxxxxxxxxxxxxxx> escreveu: > > > > A few general comments on the patch --- I agree we've had the problem from > > > the day one, but it's really started showing up recently. I agree with the > > > taken approach of separating the lifetimes of both media device and the > > > devnode. However, I don't think the patch as such is enough. > > Do you or Laurent has an alternative patchset to fix those issues? From > where I sit, we have a serious bug that it is already known for a while, > but nobody really tried to fix so far, except for Shuah and myself. If I had, I would have posted it. :-I > So, if you don't have any alternative patch ready to be merged, I'll > apply what we have later today, together with the patch that fixes cdev > livetime management: > https://patchwork.linuxtv.org/patch/34201/ > > This will allow it to be tested to a broader audience and check if > the known issues will be fixed. I'll add a C/C stable, but my plan is > to not send it as a fix for 4.7. Instead, to keep the fixes on our tree > up to the next merge window, giving us ~5 weeks for testing. > > As this is a Kernel only issue, it can be changed later if someone pops > up with a better approach. Ok. I haven't had time to review Shuah's patch but the direction taken sound good. -- Regards, Sakari Ailus e-mail: sakari.ailus@xxxxxx XMPP: sailus@xxxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
