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

[Raghavendra K T <raghavendra.kt@amd.com>]

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