Re: [PATCH 02/18] mm: make vm_mmap killable

From: Vlastimil Babka
Date: Fri Mar 11 2016 - 04:59:47 EST


On 02/29/2016 02:26 PM, Michal Hocko wrote:
From: Michal Hocko <mhocko@xxxxxxxx>

All the callers of vm_mmap seem to check for the failure already
and bail out in one way or another on the error which means that

Hmm I'm not that sure about this one:
17 1071 fs/binfmt_elf.c <<load_elf_binary>>

Assigns result of vm_mmap() to "error" variable which is never checked. Shouldn't __must_check trigger here?