On Mon, Jun 06, 2011 at 07:43:39PM +0200, Hans de Goede wrote: > Unplugging a pwc cam while an app has the /dev/video# node open leads > to an oops in pwc_video_close when the app closes the node, because > the disconnect handler has free-ed the pdev struct pwc_video_close > tries to use. Instead of adding some sort of bandaid for this. > fix it properly using the v4l2 core's new(ish) behavior of keeping the > v4l2_dev structure around until both unregister has been called, and > all file handles referring to it have been closed: > > Embed the v4l2_dev structure in the pdev structure and define a v4l2 dev > release callback releasing the pdev structure (and thus also the embedded > v4l2 dev structure. > > Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx> > --- > drivers/media/video/pwc/pwc-ctrl.c | 2 +- > drivers/media/video/pwc/pwc-if.c | 152 +++++++++++------------------------- > drivers/media/video/pwc/pwc.h | 4 +- > 3 files changed, 50 insertions(+), 108 deletions(-) <formletter> This is not the correct way to submit patches for inclusion in the stable kernel tree. Please read Documentation/stable_kernel_rules.txt for how to do this properly. </formletter> -- 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
