On Mon, 17 Jun 2002, Abhishek Nayani wrote:
> While documenting the do_mmap_pgoff() function, i found this
> snippet of code very suspicious:
>
> /* Private writable mapping? Check memory availability.. */
> if ((vm_flags & (VM_SHARED | VM_WRITE)) == VM_WRITE &&
> !(flags & MAP_NORESERVE) &&
> !vm_enough_memory(len >> PAGE_SHIFT))
> return -ENOMEM;
>
> Here we need to quit if *any* one of the condition is true. So I
> think it should be "||" instead of "&&". As according to the present
> code, it quits only if all the 3 conditions is true, which is wrong.
No, the code is correct.
The only case where we end up allocating new pages for
this mapping is when the mapping is both writable and
private.
Read-only mappings and shared mappings always have backing
store, the file from which the mapping comes and empty_zero_page.
regards,
Rik
-- Bravely reimplemented by the knights who say "NIH".http://www.surriel.com/ http://distro.conectiva.com/
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Sun Jun 23 2002 - 22:00:19 EST