Hi Bertrand,
> 1 ===
> ...
> That's worth additional comments in the code.
There's this comment already about caching the value already, not sure
if you prefer something more?
/* Cache values to reduce contention on lock */
> 2 ===
> ...
> Looks like setting initialized to true is missing once done.
Thanks, will update.
> 3 ===
> ...
> NUM_SYNC_REP_WAIT_MODE is small but as the goal is the keep the lock time as
> short as possible I wonder if it wouldn't be better to use memcpy() here instead
> of this for loop.
>
It results in a "Wdiscarded-qualifiers" which is safe given we take
the lock, but adds noise?
What do you think?
"slot.c:2756:46: warning: passing argument 2 of ‘memcpy’ discards
‘volatile’ qualifier from pointer target type
[-Wdiscarded-qualifiers]"
Thanks,
--
John Hsu - Amazon Web Services
