On 7/24/19 3:31 PM, Mark Balançian wrote: > On Jul 24, 2019, at 6:08 AM, Hans Verkuil <hverkuil@xxxxxxxxx <mailto:hverkuil@xxxxxxxxx>> wrote: >> >> What tw686x_video_free() does really should be done in the release function >> of the video_device: vdev->release is currently set to video_device_release, >> but that should be a custom function that calls dev->dma_ops->free. > > Hello all, > > There just appears some possible race condition as detected by the tool my supervisor has given me access to. I have to write a patch for > the linux kernel as an assignment. By the above, does this mean I may please write the custom function that calls dev->dma_ops->free to > replace the vdev->release = video_device_release line? Certainly. That's definitely a sensible change. Regards, Hans
