commit | 1ac3b626d3289545354003a546911e948f771fb1 | [log] [tgz] |
---|---|---|
author | Drew Davenport <ddavenport@google.com> | Thu Sep 05 10:59:16 2024 -0600 |
committer | Drew Davenport <ddavenport@google.com> | Fri Sep 06 10:13:45 2024 -0600 |
tree | 5ea81a3a826a77f1bc71dccb1972ec2267ff540c | |
parent | 49965857477b6ba138b7c0796c02971ce6cc1358 [diff] |
drm_hwcomposer: Mitigate race condition in HwcDisplay destructor There is a race between the main thread and vsync thread when tearing down the HwcDisplay. In the HwcDisplay destructor, Deinit is called, which will call StopThread on the vsync worker, and release the VSyncWorker reference. The main thread is holding the main lock, and the VSyncWorker thread may be waiting on the main lock in the out_event callback. After the HwcDisplay destructor is complete, the main thread will eventually release the main lock. At this point, the vsync thread can wake and will have a dangling pointer to 'this'. This can be mitigated by: - Explicitly set the HwcDisplay pipeline to nullptr, which will stop the vsync thread. - Release the main lock after setting the pipeline to nullptr and sleep, to allow the vsync thread to complete A more robust solution would be to provide a mechanism to ensure that the HwcDisplay knows that the vsync thread has completed before the HwcDisplay has been destructed. Change-Id: I79626427a94330b15bc138dad3163d0f2d934466 Signed-off-by: Drew Davenport <ddavenport@google.com>
Patches to drm_hwcomposer are very much welcome, we really want this to be the universal HW composer implementation for Android and similar platforms. So please bring on porting patches, bugfixes, improvements for documentation and new features.
A short list of contribution guidelines:
Submit changes via gitlab merge requests on gitlab.freedesktop.org.
drm_hwcomposer is Apache 2.0 Licensed and we require contributions to follow the developer's certificate of origin: http://developercertificate.org/.
When submitting new code please follow the naming conventions documented in the generated documentation. Also please make full use of all the helpers and convenience macros provided by drm_hwcomposer. The below command can help you with formatting of your patches:
git diff | clang-format-diff-15 -p 1 -style=file
Hardware specific changes should be tested on relevant platforms before committing.
If you need inspiration, please checkout our TODO issues.
Happy hacking!