Re: [PATCH] ThreadSanitizer warnings for cThread

Sat, Dec 10, 2022 at 07:30:50PM +0200, Marko Mäkelä wrote:
>Finally, I figured out what is causing the first report: 
>cThread::description is not protected by cThread::mutex.

Sorry, I failed to notice that even after applying both patches, both 
TSAN reports are still there. The race condition apparently is about the 
Thread->Action() member function call, which is not protected by 
anything. Perhaps cThread::StartThread() should acquire the 
Thread->mutex very early on? Or TSAN simply would insist that 
cThread::Cancel() be always called before the destructor?

	Marko

_______________________________________________
vdr mailing list
[email protected]
https://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr