Re: [PATCH v2 2/2] mm/debug: sync up latest migrate_reason to migrate_reason_names

From: Huang, Ying
Date: Tue Sep 21 2021 - 22:07:17 EST


John Hubbard <jhubbard@xxxxxxxxxx> writes:

> On 9/21/21 07:06, Huang, Ying wrote:
> ...
>>>> Can we add BUILD_BUG_ON() somewhere to capture at least some
>>>> synchronization issue?
>>>
>>> Hi Huang, we discussed this in the v1 thread with you and John, seems you
>>> missed it. Now we just add a comment to do the synchronization, and we can
>>> figure out a more general way to use strings which in trace_events straight.
>> Got it! And I think we can add the BUILD_BUG_ON() now and delete it
>> when we have a better solution to deal with that. But if you can work
>> out a better solution quickly, that's fine to ignore this.
>>
>
> I have a solution now, it's basically what I sent earlier. There appears to be
> some confusion about it. I'll send it out as a patch that builds on top of
> these two, hopefully in a few hours, if no problems pop up during testing.

Sorry, I didn't read your latest email. The solution looks good!
Thanks!

Best Regards,
Huang, Ying