Hi Zi Yan, Lorenzo,
Thank you for the detailed discussion. I have been following the thread closely and it has been very insightful.
Zi Yan's fix is excellent and I appreciate the rigorous analysis. Lorenzo's feedback has also deepened my own understanding of the subtleties around the FORCE_READ macro.
Out of curiosity, I also checked the `(void)` prefixing on Godbolt. As Zi Yan concluded, the resulting assembly appears identical.
I will be happy to join any future discussions regarding the exact behavior of volatile in this context.
For this patch, it's definitely LGTM from my side as well, so. Reviewed-by:wang lian lianux.mm@gmail.com
Thanks, wang lian