[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-changelog] [xen master] xen/arm: Fix the issue in cmp_mmio_handler used in find_mmio_handler
*** New tip has no discernable common ancestor with previous history *** Previously reported histories end at: e01fa4e223fc077e18936d890dbd4813c3e7c9b1 3a28f760508fb35c430edac17a9efde5aff6d1d5 123c7793797502b222300eb710cd3873dcca41ee c40317f11b3f05e7c06a2213560c8471081f2662 a6f2cdb633bf519244a16674031b8034b581ba7f 8e4b2676685f50bc26f03b5f62d8b7aea8e69dbf 2f5af2c962c05b789bdd65b46c74711e903f86d0 695bb5f504ab48c1d546446f104c1b6c0ead126d 9053a74c08fd6abf43bb45ff932b4386de7e8510 16b1414de91b5a82a0996c67f6db3af7d7e32873 commit b7ed331353a14f43f53eaf6a3a543ec8385193a3 Author: Bhupinder Thakur <bhupinder.thakur@xxxxxxxxxx> AuthorDate: Fri Sep 29 11:29:46 2017 +0530 Commit: Stefano Stabellini <sstabellini@xxxxxxxxxx> CommitDate: Fri Sep 29 11:50:20 2017 -0700 xen/arm: Fix the issue in cmp_mmio_handler used in find_mmio_handler This patch fixes the wrong range check done in cmp_mmio_handler(). This function returns -1 , 0 or 1 based on whether the key value is below the range, in the range or above the range where the range is (start, start+size). However, it should check against (start, start+size-1) because start+size falls outside the range. This resulted in returning a wrong mmio_handler for a given mmio address which happened to be start+size. This bug was introduced when the mmio region search switched from linear search to binary search in the following commit: 8047e09 "xen/arm: io: Use binary search for mmio handler lookup". Signed-off-by: Bhupinder Thakur <bhupinder.thakur@xxxxxxxxxx> Signed-off-by: Stefano Stabellini <sstabellini@xxxxxxxxxx> Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx> -- generated by git-patchbot for /home/xen/git/xen.git#master _______________________________________________ Xen-changelog mailing list Xen-changelog@xxxxxxxxxxxxx https://lists.xenproject.org/xen-changelog
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |