On Wed, Jul 31, 2024 at 07:01:38AM +0000, Bertrand Drouvot wrote: > I share the same understanding and I think those can be removed. > > The patch LGTM.
That sounds about right. All the volatile references we have here have been kept under the assumption that a memory barrier is required. As we hold spin locks in these areas, that should not be necessary anyway. So LGTM as well. A quick lookup at the rest of contrib/ is showing me that the remaining volatile references point at uses with TRY/CATCH blocks, where we require them. -- Michael
signature.asc
Description: PGP signature
