Re: [PATCH v2] powerpc/pseries: Fix maximum memory value

2019-07-04 Thread Aravinda Prasad
On Thursday 04 July 2019 04:43 PM, Michael Ellerman wrote: > "Naveen N. Rao" writes: >> Nathan Lynch wrote: >>> Aravinda Prasad writes: Calculating the maximum memory based on the number of lmbs and lmb size does not account for the RMA region. Hence use memory_hotplug_max(), wh

Re: [PATCH v2] powerpc/pseries: Fix maximum memory value

2019-07-04 Thread Michael Ellerman
"Naveen N. Rao" writes: > Nathan Lynch wrote: >> Aravinda Prasad writes: >>> Calculating the maximum memory based on the number of lmbs >>> and lmb size does not account for the RMA region. Hence >>> use memory_hotplug_max(), which already accounts for the >>> RMA region, to fetch the maximum mem

Re: [PATCH v2] powerpc/pseries: Fix maximum memory value

2019-06-30 Thread Aravinda Prasad
On Friday 28 June 2019 10:57 PM, Nathan Lynch wrote: > Aravinda Prasad writes: >> Calculating the maximum memory based on the number of lmbs >> and lmb size does not account for the RMA region. Hence >> use memory_hotplug_max(), which already accounts for the >> RMA region, to fetch the maximum

Re: [PATCH v2] powerpc/pseries: Fix maximum memory value

2019-06-28 Thread Nathan Lynch
"Naveen N. Rao" writes: > Nathan Lynch wrote: >> It would likely help with review and future maintenance if the semantics >> and intended use of the MaxMem field are made a little more >> explicit. For example, is it supposed to include persistent memory? >> Perhaps a follow-up patch could address

Re: [PATCH v2] powerpc/pseries: Fix maximum memory value

2019-06-28 Thread Naveen N. Rao
Nathan Lynch wrote: Aravinda Prasad writes: Calculating the maximum memory based on the number of lmbs and lmb size does not account for the RMA region. Hence use memory_hotplug_max(), which already accounts for the RMA region, to fetch the maximum memory value. Thanks to Nathan Lynch for sugge

Re: [PATCH v2] powerpc/pseries: Fix maximum memory value

2019-06-28 Thread Nathan Lynch
Aravinda Prasad writes: > Calculating the maximum memory based on the number of lmbs > and lmb size does not account for the RMA region. Hence > use memory_hotplug_max(), which already accounts for the > RMA region, to fetch the maximum memory value. Thanks to > Nathan Lynch for suggesting the mem

[PATCH v2] powerpc/pseries: Fix maximum memory value

2019-06-27 Thread Aravinda Prasad
Calculating the maximum memory based on the number of lmbs and lmb size does not account for the RMA region. Hence use memory_hotplug_max(), which already accounts for the RMA region, to fetch the maximum memory value. Thanks to Nathan Lynch for suggesting the memory_hotplug_max() function. Fixes: