Le 26/09/2017 à 03:44, Vaibhav Jain a écrit :
Hi Christophe,

A minor nitpick

Christophe Lombard <clomb...@linux.vnet.ibm.com> writes:

+       for (dar = (addr & ~(page_size - 1)); dar < (addr + size); dar += 
page_size) {
+               if (dar < vma->vm_start || dar > vma->vm_end) {
Code comment in mm_types.h for vm_end says it "The first byte after our
end address within vm_mm". So this condition should be
(dar < vma->vm_start || dar >= vma->vm_end)

~ Vaibhav


You are right. Thanks for the review

Reply via email to