On Wed, Dec 6, 2023 at 11:47 AM Nhat Pham nphamcs@gmail.com wrote:
[...]
Hmm so how should we proceed from here? How about this:
a) I can send a fixlet to move the enablement check above the stats flushing + use mem_cgroup_flush_stats b) Then maybe, you can send a fixlet to update this new callsite?
Does that sound reasonable?
I just sent out the fixlet. Yosry and Chengming, let me know if that looks good. Thank you both for detecting this issue and proposing the fix!
The fixlet looks good, and Andrew already took care of (b) before I could send a followup fixlet out :)