From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 39501 invoked by alias); 12 Mar 2015 14:36:37 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 39491 invoked by uid 89); 12 Mar 2015 14:36:36 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-4.0 required=5.0 tests=AWL,BAYES_00,SPF_HELO_PASS,T_RP_MATCHES_RCVD autolearn=ham version=3.3.2 X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Thu, 12 Mar 2015 14:36:30 +0000 Received: from int-mx14.intmail.prod.int.phx2.redhat.com (int-mx14.intmail.prod.int.phx2.redhat.com [10.5.11.27]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id t2CEaRJL023934 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL); Thu, 12 Mar 2015 10:36:27 -0400 Received: from tranklukator.brq.redhat.com (dhcp-1-208.brq.redhat.com [10.34.1.208]) by int-mx14.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with SMTP id t2CEaPCd004202; Thu, 12 Mar 2015 10:36:25 -0400 Received: by tranklukator.brq.redhat.com (nbSMTP-1.00) for uid 500 oleg@redhat.com; Thu, 12 Mar 2015 15:34:41 +0100 (CET) Date: Thu, 12 Mar 2015 14:36:00 -0000 From: Oleg Nesterov To: Jan Kratochvil , Andy Lutomirski Cc: Sergio Durigan Junior , GDB Patches , Pedro Alves , linux-kernel@vger.kernel.org Subject: vvar, gup && coredump Message-ID: <20150312143438.GA4338@redhat.com> References: <878ufc9kau.fsf@redhat.com> <20150305154827.GA9441@host1.jankratochvil.net> <87zj7r5fpz.fsf@redhat.com> <20150305205744.GA13165@host1.jankratochvil.net> <20150311200052.GA22654@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150311200052.GA22654@redhat.com> User-Agent: Mutt/1.5.18 (2008-05-17) X-SW-Source: 2015-03/txt/msg00338.txt.bz2 Add cc's, change subject. On 03/11, Oleg Nesterov wrote: > > On 03/05, Jan Kratochvil wrote: > > > > On Thu, 05 Mar 2015 21:52:56 +0100, Sergio Durigan Junior wrote: > > > On Thursday, March 05 2015, Jan Kratochvil wrote: > > > > On Thu, 05 Mar 2015 04:48:09 +0100, Sergio Durigan Junior wrote: > > > > Currently it also tries to dump [vvar] (by default rules) but that is > > > > unreadable for some reason, causing: > > > > warning: Memory read failed for corefile section, 8192 bytes at 0x7ffff6ceb000. > > > > ^^^^^^^^^^^^^^ > > > It would be good to get a reply from a kernel aware person what does it mean > > before such patch gets accepted. It can be also just a Linux kernel bug. > > _So far_ this doesn't look like a kernel bug to me. > > But! I need to recheck. In fact, it seems to me that I should discuss > this on lkml. I have some concerns, but most probably this is only my > misunderstanding, I need to read this (new to me) code more carefully. Hi Andy, we need your help ;) So, the problem is that gdb can't access the "vvar" mapping which looks like the "normal" vma from user-space pov. Technically this is clear. vvar_mapping->pages is the "dummy" no_pages[] array, get_user_pages() can't succeed. In fact even follow_page() can't work because of VM_PFNMAP/_PAGE_SPECIAL set by remap_pfn_range(). What is not clear: do we really want gup() to fail? Or it is not trivial to turn __vvar_page into the "normal" page? (to simplify the discussion, lets ignore hpet mapping for now). Because this doesn't look consistent. gdb tries to "coredump" the live process like the kernel does, but fails to dump the "r--p ... [vvar]" region. OK, gdb can look at VM_DONTDUMP bit in "VmFlags:" field in /proc/pid/smaps and skip this vma. But, why (afaics) the kernel dumps this vma then? Lets look at vma_dump_size(), /* always dump the vdso and vsyscall sections */ if (always_dump_vma(vma)) goto whole; if (vma->vm_flags & VM_DONTDUMP) return 0; so the kernel ignores VM_DONTDUMP in this case, always_dump_vma() returns true because of special_mapping_name(). Perhaps we should check VM_DONTDUMP before always_dump_vma() ? Or. We can teach gdb to read and dump its own "vvar" mapping to mimic the kernel behaviour, this is the same read-only memory. But this hack doesn't look nice, gdb should not know "too much" about the kernel internals. Oleg.