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

Raghavendra K T
2024-12-06T08:28:36Z

On 12/6/2024 1:20 PM, Dan Williams wrote:
> Raghavendra K T wrote:
>>
[...]
>> B. /proc/iomem
>>
>> $ vimdiff success fail
>>
>> 4050000000-604fffffff : Soft Reserved
>> | 164 4050000000-604fffffff : Soft Reserved
>> 165 4050000000-604fffffff : CXL Window 0
>> | 165 4050000000-604fffffff : CXL Window 0
>> 166 4080000000-5fffffffff : dax1.0
>> |
>> ------------------------------------------------------------------------
>> 167 4080000000-5fffffffff : System RAM (kmem)
>> |
>> --------------------------------------------------------------------
>
> My eyes only know how to read unified diff (diff -u) format. Is this
> saying that in the failure case the System RAM range for dax1.0 is
> missing?

Sorry for that formatting. I realized later.
Yes. dax1.0 missing in the failure case.

Looks like problem is that,

4050000000-604fffffff : Soft Reserved
4050000000-604fffffff : CXL Window 0
4080000000-5fffffffff : dax1.0
4080000000-5fffffffff : System RAM (kmem)

this use case where,

|A<------------->D| CXL window
|..|B<-->C|.......| kmem

B->C range that falls within A->D as in above somehow
not covered after the patch.

Thanks and Regards
- Raghu