On 7/24/19 3:31 PM, Mark Balançian wrote:
> On Jul 24, 2019, at 6:08 AM, Hans Verkuil <hverk...@xs4all.nl
> <mailto:hverk...@xs4all.nl>> 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
