From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-lj1-f200.google.com (mail-lj1-f200.google.com [209.85.208.200]) by kanga.kvack.org (Postfix) with ESMTP id 1B9696B0006 for ; Mon, 30 Jul 2018 09:35:25 -0400 (EDT) Received: by mail-lj1-f200.google.com with SMTP id t10-v6so2643554lji.12 for ; Mon, 30 Jul 2018 06:35:25 -0700 (PDT) Received: from mail-sor-f65.google.com (mail-sor-f65.google.com. [209.85.220.65]) by mx.google.com with SMTPS id p15-v6sor2279146ljh.91.2018.07.30.06.35.23 for (Google Transport Security); Mon, 30 Jul 2018 06:35:23 -0700 (PDT) MIME-Version: 1.0 References: <20180730130134.yvn5tcmoavuxtwt5@kshutemo-mobl1> In-Reply-To: <20180730130134.yvn5tcmoavuxtwt5@kshutemo-mobl1> From: Amit Pundir Date: Mon, 30 Jul 2018 19:04:46 +0530 Message-ID: Subject: Re: Linux 4.18-rc7 Content-Type: text/plain; charset="UTF-8" Sender: owner-linux-mm@kvack.org List-ID: To: kirill@shutemov.name Cc: "Kirill A. Shutemov" , Andrew Morton , Dmitry Vyukov , Oleg Nesterov , aarcange@redhat.com, Linus Torvalds , Greg Kroah-Hartman , John Stultz , linux-mm@kvack.org, lkml , youling 257 On Mon, 30 Jul 2018 at 18:31, Kirill A. Shutemov wrote: > > On Mon, Jul 30, 2018 at 12:17:46PM +0530, Amit Pundir wrote: > > On Mon, 30 Jul 2018 at 03:39, Linus Torvalds > > wrote: > > > > > > So unless something odd happens, this should be the last rc for 4.18. > > > > > > Nothing particularly odd happened this last week - we got the usual > > > random set of various minor fixes all over. About two thirds of it is > > > drivers - networking, staging and usb stands out, but there's a little > > > bit of stuff all over (clk, block, gpu, nvme..). > > > > > > Outside of drivers, the bulk is some core networking stuff, with > > > random changes elsewhere (minor arch updates, filesystems, core > > > kernel, test scripts). > > > > > > The appended shortlog gives a flavor of the details. > > > > > > Linus > > > > > > --- > > > Kirill A. Shutemov (3): > > > mm: introduce vma_init() > > > mm: use vma_init() to initialize VMAs on stack and data segments > > > mm: fix vma_is_anonymous() false-positives > > > > Hi, I have run into AOSP userspace crash with v4.18-rc7, leading to > > above mm patches. bfd40eaff5ab ("mm: fix vma_is_anonymous() > > false-positives") to be specific. The same userspace is working fine > > with v4.18-rc6. > > > > I didn't yet look into what is going wrong from userspace point of > > view, but I just wanted to give you a heads up on this. I'll be happy > > to assist in further debugging/diagnosis if required. > > Youling reported basically the same bug with zygote crashing, but on > x86-64. > > I think I missed vma_set_anonymous() somewhere, but I fail to see where. > > Could you check if removing 'vma->vm_ops = &dummy_vm_ops;" from vma_init > makes the problem go away? Yes removing 'vma->vm_ops = &dummy_vm_ops;" from vma_init() works. Crash is gone with that change. > > Any chance the code that crashes can be run under strace? Running strace on zygote is going to be a pain. I can check logcat again and see if any other relatively less complex process is crashing with similar backtrace and try to run that with strace if that is still required. Regards, Amit Pundir > > > Here is the crash log from logcat, if it helps: > > F DEBUG : *** *** *** *** *** *** *** *** *** *** *** *** *** *** *** *** > > F DEBUG : Build fingerprint: > > 'Android/db410c32_only/db410c32_only:Q/OC-MR1/102:userdebug/test-key > > F DEBUG : Revision: '0' > > F DEBUG : ABI: 'arm' > > F DEBUG : pid: 2261, tid: 2261, name: zygote >>> zygote <<< > > F DEBUG : signal 7 (SIGBUS), code 2 (BUS_ADRERR), fault addr 0xec00008 > > .. .. > > F DEBUG : backtrace: > > F DEBUG : #00 pc 00001c04 /system/lib/libc.so (memset+48) > > F DEBUG : #01 pc 0010c513 /system/lib/libart.so > > (create_mspace_with_base+82) > > F DEBUG : #02 pc 0015c601 /system/lib/libart.so > > (art::gc::space::DlMallocSpace::CreateMspace(void*, unsigned int, > > unsigned int)+40) > > F DEBUG : #03 pc 0015c3ed /system/lib/libart.so > > (art::gc::space::DlMallocSpace::CreateFromMemMap(art::MemMap*, > > std::__1::basic_string > 1::char_traits, std::__1::allocator> const&, unsigned int, > > unsigned int, unsigned int, unsigned int, bool)+36) > > F DEBUG : #04 pc 0013c9ab /system/lib/libart.so > > (art::gc::Heap::Heap(unsigned int, unsigned int, unsigned int, > > unsigned int, double, double, unsigned int, unsigned int, > > std::__1::basic_string, > > std::__1::allocator> const&, art::InstructionSet, > > art::gc::CollectorType, art::gc::CollectorType, > > art::gc::space::LargeObjectSpaceType, unsigned int, unsigned int, > > unsigned int, bool, unsigned int, unsigned int, bool, bool, bool, > > bool, bool, bool, bool, bool, bool, bool, bool, unsigned long > > long)+1674) > > DEBUG : #05 pc 00318201 /system/lib/libart.so > > (art::Runtime::Init(art::RuntimeArgumentMap&&)+7036) > > DEBUG : #06 pc 0031af19 /system/lib/libart.so > > (art::Runtime::Create(std::__1::vector > std::__1::char_traits, std::__1::allocator>, void const*>, > > std::__1::allocator > std::__1::char_traits, std::__1::allocator>, void > > const*>>> const&, bool)+68) > > F DEBUG : #07 pc 0023c353 /system/lib/libart.so (JNI_CreateJavaVM+658) > > F DEBUG : #08 pc 0000205f /system/lib/libandroid_runtime.so > > (android::AndroidRuntime::startVm(_JavaVM**, _JNIEnv**, bool)+5038) > > F DEBUG : #09 pc 00002381 /system/lib/libandroid_runtime.so > > (android::AndroidRuntime::start(char const*, > > android::Vector const&, bool)+196) > > F DEBUG : #10 pc 0000046b /system/bin/app_process32 (main+702) > > > > Regards, > > Amit Pundir > > > > -- > Kirill A. Shutemov