On Mon, Apr 29, 2019 at 9:29 AM Doug Ledford <dledford@xxxxxxxxxx> wrote: > > > drivers/infiniband/core/uverbs_main.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) This trivial one-liner is actually incorrect. It should use 'vmf->address', because the point of the ZERO_PAGE argument is to pick the page with the right virtual address alias for broken architectures that need those kinds. I'm actually surprised s390 wants it, usually it's just MIPS that has the horribly broken virtual address translation stuff. But it looks like for s390 it's at least only a performance issue (ie it causes some aliases in L1 that cause cacheline ping-pong rather than anything else). Linus