Message ID | 20170404052047.11096-1-oohall@gmail.com |
---|---|
State | Accepted |
Headers | show |
Oliver O'Halloran <oohall@gmail.com> writes: > The hostboot reserved ranges are [start, end] pairs rather than > [start, end) so we need to stick a +1 in there to calculate the > size properly. > > Signed-off-by: Oliver O'Halloran <oohall@gmail.com> thanks, merged to master as of c2bfa357ad3b8f8d56bbd6ef895795e1d2f1dc6b
diff --git a/hdata/memory.c b/hdata/memory.c index 9b0c7666a510..2df1136c3627 100644 --- a/hdata/memory.c +++ b/hdata/memory.c @@ -475,7 +475,8 @@ static void get_hb_reserved_mem(struct HDIF_common_hdr *ms_vpd) prlog(PR_DEBUG, "MEM: Reserve '%s' %#" PRIx64 "-%#" PRIx64 " (type/inst=0x%08x)\n", label, start_addr, end_addr, be32_to_cpu(hb_resv_mem->type_instance)); - mem_reserve_hw(label, start_addr, end_addr - start_addr); + + mem_reserve_hw(label, start_addr, end_addr - start_addr + 1); } }
The hostboot reserved ranges are [start, end] pairs rather than [start, end) so we need to stick a +1 in there to calculate the size properly. Signed-off-by: Oliver O'Halloran <oohall@gmail.com> --- hdata/memory.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)