[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-changelog] [xen stable-4.10] x86/srat: fix end calculation in nodes_cover_memory()



commit 3deb58f832884387e4da4d737fdf9d31e637817a
Author:     Jan Beulich <jbeulich@xxxxxxxx>
AuthorDate: Tue Mar 6 15:52:51 2018 +0100
Commit:     Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Tue Mar 6 15:52:51 2018 +0100

    x86/srat: fix end calculation in nodes_cover_memory()
    
    Along the lines of commit 7226486767 ("x86/srat: fix the end pfn check
    in valid_numa_range()") nodes_cover_memory() also doesn't consistently
    use "end": It's set to an inclusive value initially, but then compared
    to the exclusive "end" field of struct node and also possibly set to
    nodes[j].start, making it exclusive too. Change the initialization to
    make the variable consistently exclusive.
    
    Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
    Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
    master commit: fdbed42649eb064e7c6d1bae2bdd4f46e7b2a160
    master date: 2018-02-15 18:17:32 +0100
---
 xen/arch/x86/srat.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xen/arch/x86/srat.c b/xen/arch/x86/srat.c
index 49d04b887b..166eb44fe2 100644
--- a/xen/arch/x86/srat.c
+++ b/xen/arch/x86/srat.c
@@ -368,7 +368,7 @@ static int __init nodes_cover_memory(void)
                }
 
                start = e820.map[i].addr;
-               end = e820.map[i].addr + e820.map[i].size - 1;
+               end = e820.map[i].addr + e820.map[i].size;
 
                do {
                        found = 0;
--
generated by git-patchbot for /home/xen/git/xen.git#stable-4.10

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/xen-changelog

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.