Re: [Xen-devel] [PATCH] Fix domheap structure allocation when N

>>> "Yang, Xiaowei" <xiaowei.yang@xxxxxxxxx> 20.03.09 09:34 >>>
>Jan Beulich wrote:
>>>>> "Yang, Xiaowei" <xiaowei.yang@xxxxxxxxx> 20.03.09 06:05 >>>
>>> DIRECTMAP_VIRT_END can't be passed to virt_to_mfn(), as it's just beyond 
>>> direct map boundary and triggers ASSERT very early at boot time.
>> 
>> While I agree to the analysis, I would think that this
>> 
>> +              mfn + needed <= virt_to_mfn(DIRECTMAP_VIRT_END - PAGE_SIZE) )
>> 
>> should rather be
>> 
>> +              mfn + needed <= virt_to_mfn(DIRECTMAP_VIRT_END - 1) + 1 )
>> 
>virt_to_mfn(DIRECTMAP_VIRT_END - 1) is equal to 

Depending on whether DIRECTMAP_VIRT_END is the last byte or the first
following byte. Using "- 1" avoids such a dependency.

>virt_to_mfn(DIRECTMAP_VIRT_END - PAGE_SIZE). Why +1? We use '<=' here.

Because on the left side of the comparison we also calculate the first
following mfn, not the last included one.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

<Prev in Thread]

Current Thread

[Next in Thread>

Previous by Date:

Re: [Xen-devel] [RFO] #2: removing a concurrency bottleneck, Keir Fraser

Next by Date:

[Xen-devel] Re: [Patch 1/3]RAS(Part II)--Intel MCA enalbing in XEN, Keir Fraser

Previous by Thread:

Re: [Xen-devel] [PATCH] Fix domheap structure allocation when NUMA=on, Yang, Xiaowei

Next by Thread:

Re: [Xen-devel] [PATCH] Fix domheap structure allocation when NUMA=on, Keir Fraser

Indexes:

[Date] [Thread] [Top] [All Lists]

WARNING - OLD ARCHIVES

xen-devel

Re: [Xen-devel] [PATCH] Fix domheap structure allocation when NUMA=on