From mboxrd@z Thu Jan 1 00:00:00 1970 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752496AbeAGJCf (ORCPT + 1 other); Sun, 7 Jan 2018 04:02:35 -0500 Received: from mx2.suse.de ([195.135.220.15]:35479 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752069AbeAGJCc (ORCPT ); Sun, 7 Jan 2018 04:02:32 -0500 Date: Sun, 7 Jan 2018 10:02:29 +0100 From: Michal Hocko To: Anshuman Khandual Cc: akpm@linux-foundation.org, mm-commits@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-fsdevel@vger.kernel.org, linux-next@vger.kernel.org, sfr@canb.auug.org.au, broonie@kernel.org Subject: ppc elf_map breakage with MAP_FIXED_NOREPLACE (was: Re: mmotm 2018-01-04-16-19 uploaded) Message-ID: <20180107090229.GB24862@dhcp22.suse.cz> References: <5a4ec4bc.u5I/HzCSE6TLVn02%akpm@linux-foundation.org> <7e35e16a-d71c-2ec8-03ed-b07c2af562f8@linux.vnet.ibm.com> <20180105084631.GG2801@dhcp22.suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.9.2 (2017-12-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: On Sun 07-01-18 12:19:32, Anshuman Khandual wrote: > On 01/05/2018 02:16 PM, Michal Hocko wrote: [...] > > Could you give us more information about the failure please. Debugging > > patch from http://lkml.kernel.org/r/20171218091302.GL16951@dhcp22.suse.cz > > should help to see what is the clashing VMA. > > Seems like its re-requesting the same mapping again. It always seems to be the same mapping which is a bit strange as we have multiple binaries here. Are these binaries any special? Does this happen to all bianries (except for init which has obviously started successfully)? Could you add an additional debugging (at the do_mmap layer) to see who is requesting the mapping for the first time? > [ 23.423642] 9148 (sed): Uhuuh, elf segment at 0000000010030000 requested but the memory is mapped already > [ 23.423706] requested [10030000, 10040000] mapped [10030000, 10040000] 100073 anon I also find it a bit unexpected that this is an anonymous mapping because the elf loader should always map a file backed one. > [ 23.426089] 9151 (sed): Uhuuh, elf segment at 0000000010030000 requested but the memory is mapped already > [ 23.426232] requested [10030000, 10040000] mapped [10030000, 10040000] 100073 anon > [ 23.429048] 9154 (sed): Uhuuh, elf segment at 0000000010030000 requested but the memory is mapped already > [ 23.429196] requested [10030000, 10040000] mapped [10030000, 10040000] 100073 anon > [ 23.482766] 9164 (sed): Uhuuh, elf segment at 0000000010030000 requested but the memory is mapped already > [ 23.482904] requested [10030000, 10040000] mapped [10030000, 10040000] 100073 anon > [ 23.485849] 9167 (sed): Uhuuh, elf segment at 0000000010030000 requested but the memory is mapped already > [ 23.485945] requested [10030000, 10040000] mapped [10030000, 10040000] 100073 anon > [ 76.041836] 9262 (hostname): Uhuuh, elf segment at 0000000010020000 requested but the memory is mapped already > [ 76.041965] requested [10020000, 10030000] mapped [10020000, 10030000] 100073 anon > [ 76.207197] 9285 (pkg-config): Uhuuh, elf segment at 0000000010020000 requested but the memory is mapped already > [ 76.207326] requested [10020000, 10030000] mapped [10020000, 10030000] 100073 anon > [ 76.371073] 9299 (sed): Uhuuh, elf segment at 0000000010030000 requested but the memory is mapped already > [ 76.371165] requested [10030000, 10040000] mapped [10030000, 10040000] 100073 anon > > > I have fixed/changed the debug patch a bit > > > diff --git a/fs/binfmt_elf.c b/fs/binfmt_elf.c > index d8c5657..a43eccaa 100644 > --- a/fs/binfmt_elf.c > +++ b/fs/binfmt_elf.c > @@ -372,11 +372,35 @@ static unsigned long elf_map(struct file *filep, unsigned long addr, > } else > map_addr = vm_mmap(filep, addr, size, prot, type, off); > > - if ((type & MAP_FIXED_NOREPLACE) && BAD_ADDR(map_addr)) > + if ((type & MAP_FIXED_NOREPLACE) && BAD_ADDR(map_addr)) { > + struct vm_area_struct *vma; > + unsigned long end; > + > + if (total_size) > + end = addr + total_size; > + else > + end = addr + size; > + > pr_info("%d (%s): Uhuuh, elf segment at %p requested but the memory is mapped already\n", > task_pid_nr(current), current->comm, > (void *)addr); > > + vma = find_vma(current->mm, addr); > + if (vma && vma->vm_start <= addr) { > + pr_info("requested [%lx, %lx] mapped [%lx, %lx] %lx ", addr, end, > + vma->vm_start, vma->vm_end, vma->vm_flags); > + if (!vma->vm_file) { > + pr_cont("anon\n"); > + } else { > + char path[512]; > + char *p = file_path(vma->vm_file, path, sizeof(path)); > + if (IS_ERR(p)) > + p = "?"; > + pr_cont("\"%s\"\n", kbasename(p)); > + } > + dump_stack(); > + } > + } > return(map_addr); > } > > > -- Michal Hocko SUSE Labs From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Sun, 7 Jan 2018 10:02:29 +0100 From: Michal Hocko To: Anshuman Khandual Cc: akpm@linux-foundation.org, mm-commits@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-fsdevel@vger.kernel.org, linux-next@vger.kernel.org, sfr@canb.auug.org.au, broonie@kernel.org Subject: ppc elf_map breakage with MAP_FIXED_NOREPLACE (was: Re: mmotm 2018-01-04-16-19 uploaded) Message-ID: <20180107090229.GB24862@dhcp22.suse.cz> References: <5a4ec4bc.u5I/HzCSE6TLVn02%akpm@linux-foundation.org> <7e35e16a-d71c-2ec8-03ed-b07c2af562f8@linux.vnet.ibm.com> <20180105084631.GG2801@dhcp22.suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: owner-linux-mm@kvack.org List-ID: On Sun 07-01-18 12:19:32, Anshuman Khandual wrote: > On 01/05/2018 02:16 PM, Michal Hocko wrote: [...] > > Could you give us more information about the failure please. Debugging > > patch from http://lkml.kernel.org/r/20171218091302.GL16951@dhcp22.suse.cz > > should help to see what is the clashing VMA. > > Seems like its re-requesting the same mapping again. It always seems to be the same mapping which is a bit strange as we have multiple binaries here. Are these binaries any special? Does this happen to all bianries (except for init which has obviously started successfully)? Could you add an additional debugging (at the do_mmap layer) to see who is requesting the mapping for the first time? > [ 23.423642] 9148 (sed): Uhuuh, elf segment at 0000000010030000 requested but the memory is mapped already > [ 23.423706] requested [10030000, 10040000] mapped [10030000, 10040000] 100073 anon I also find it a bit unexpected that this is an anonymous mapping because the elf loader should always map a file backed one. > [ 23.426089] 9151 (sed): Uhuuh, elf segment at 0000000010030000 requested but the memory is mapped already > [ 23.426232] requested [10030000, 10040000] mapped [10030000, 10040000] 100073 anon > [ 23.429048] 9154 (sed): Uhuuh, elf segment at 0000000010030000 requested but the memory is mapped already > [ 23.429196] requested [10030000, 10040000] mapped [10030000, 10040000] 100073 anon > [ 23.482766] 9164 (sed): Uhuuh, elf segment at 0000000010030000 requested but the memory is mapped already > [ 23.482904] requested [10030000, 10040000] mapped [10030000, 10040000] 100073 anon > [ 23.485849] 9167 (sed): Uhuuh, elf segment at 0000000010030000 requested but the memory is mapped already > [ 23.485945] requested [10030000, 10040000] mapped [10030000, 10040000] 100073 anon > [ 76.041836] 9262 (hostname): Uhuuh, elf segment at 0000000010020000 requested but the memory is mapped already > [ 76.041965] requested [10020000, 10030000] mapped [10020000, 10030000] 100073 anon > [ 76.207197] 9285 (pkg-config): Uhuuh, elf segment at 0000000010020000 requested but the memory is mapped already > [ 76.207326] requested [10020000, 10030000] mapped [10020000, 10030000] 100073 anon > [ 76.371073] 9299 (sed): Uhuuh, elf segment at 0000000010030000 requested but the memory is mapped already > [ 76.371165] requested [10030000, 10040000] mapped [10030000, 10040000] 100073 anon > > > I have fixed/changed the debug patch a bit > > > diff --git a/fs/binfmt_elf.c b/fs/binfmt_elf.c > index d8c5657..a43eccaa 100644 > --- a/fs/binfmt_elf.c > +++ b/fs/binfmt_elf.c > @@ -372,11 +372,35 @@ static unsigned long elf_map(struct file *filep, unsigned long addr, > } else > map_addr = vm_mmap(filep, addr, size, prot, type, off); > > - if ((type & MAP_FIXED_NOREPLACE) && BAD_ADDR(map_addr)) > + if ((type & MAP_FIXED_NOREPLACE) && BAD_ADDR(map_addr)) { > + struct vm_area_struct *vma; > + unsigned long end; > + > + if (total_size) > + end = addr + total_size; > + else > + end = addr + size; > + > pr_info("%d (%s): Uhuuh, elf segment at %p requested but the memory is mapped already\n", > task_pid_nr(current), current->comm, > (void *)addr); > > + vma = find_vma(current->mm, addr); > + if (vma && vma->vm_start <= addr) { > + pr_info("requested [%lx, %lx] mapped [%lx, %lx] %lx ", addr, end, > + vma->vm_start, vma->vm_end, vma->vm_flags); > + if (!vma->vm_file) { > + pr_cont("anon\n"); > + } else { > + char path[512]; > + char *p = file_path(vma->vm_file, path, sizeof(path)); > + if (IS_ERR(p)) > + p = "?"; > + pr_cont("\"%s\"\n", kbasename(p)); > + } > + dump_stack(); > + } > + } > return(map_addr); > } > > > -- Michal Hocko SUSE Labs -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org