Re: [RFC PATCH] resource: Fix CXL node not populated issue

Huang, Ying
2024-12-03T06:27:24Z

Raghavendra K T <raghavendra.kt@amd.com> writes:

> Before:
> ~]$ numastat -m
> ...
> Node 0 Node 1 Total
> --------------- --------------- ---------------
> MemTotal 128096.18 128838.48 256934.65
>
> After:
> $ numastat -m
> .....
> Node 0 Node 1 Node 2 Total
> --------------- --------------- --------------- ---------------
> MemTotal 128054.16 128880.51 129024.00 385958.67
>
> Current patch reverts the effect of first commit where the issue is seen.

This doesn't root cause the issue. Please trace the code path
(e.g. return value of region_intersects(), in which code path) at least.
Without these information, we cannot come up with a proper fix to your issue.

> git bisect had led to below commit
> Fixes: b4afe4183ec7 ("resource: fix region_intersects() vs add_memory_driver_managed()")

This breaks you case, sorry about that. But this also fixed a real bug
too. So, it's not appropriate just to revert it blindly.

[snip]

---
Best Regards,
Huang, Ying