Re: [Xen-devel] [MINOR FIXES 1/2] mm: fix memory cleanup

2017-02-17 Thread Andrew Cooper
On 17/02/17 10:47, Norbert Manthey wrote: > During destroying the m2p mapping, the loop variable was always incremented > by one, as the current version used a compare operator on the left hand side, > which always evaluated to true, i.e. > > i += 1UL < (L2_PAGETABLE_SHIFT - 2) > > The fix

[Xen-devel] [MINOR FIXES 1/2] mm: fix memory cleanup

2017-02-17 Thread Norbert Manthey
During destroying the m2p mapping, the loop variable was always incremented by one, as the current version used a compare operator on the left hand side, which always evaluated to true, i.e. i += 1UL < (L2_PAGETABLE_SHIFT - 2) The fix increments the value of the variable by the actual page size