Hi, Gavin, On Wed, Feb 22, 2023 at 10:44:07AM +1100, Gavin Shan wrote: > Peter, could you please give some hints for me to understand the atomic > and non-atomic update here? Ok, I will drop this part of changes in next > revision with the assumption that we have atomic update supported for > ARM64.
See commit f39b7d2b96. Please don't remove the change in this patch. The comment was just something I thought about when reading, not something I suggested to change. If to remove it we'll need to remove the whole chunk not your changes alone here. Still, please take it with a grain of salt before anyone can help to confirm because I can miss something else here. In short: before we know anything solidly, your current code is exactly correct, AFAICT. Thanks, -- Peter Xu