On Thu, Nov 18, 2010 at 9:46 AM, Hans Verkuil <hverkuil@xxxxxxxxx> wrote: > >> This driver has quite a few locking issues that would only be made >> worse by your patch. A much better patch for this can be found here: >> >> http://desource.dyndns.org/~atog/gitweb?p=linux-media.git;a=commitdiff;h=9c5d8ebb602e9af46902c5f3d4d4cc80227d3f7c > > Much too big for 2.6.37. I'll just drop this patch from my patch series. > Instead it will rely on the new lock in v4l2_device (BKL replacement) that > serializes all ioctls. For 2.6.38 we can convert it to core assisted > locking which is much easier. > I don't see how this patch is really any bigger than some of the others in your series. Sure there are a lot of deletions, but the number of additions is quite small. There are much worse ways all the locking issues in this driver could have been corrected. This patch manages to do just that while reducing the overall size of the driver at the same time. Regards, David Ellingsworth -- 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
