From mboxrd@z Thu Jan 1 00:00:00 1970 From: panand@redhat.com (Pratyush Anand) Date: Thu, 23 Jun 2016 21:16:24 +0530 Subject: [PATCH v19 10/13] arm64: kdump: add VMCOREINFO's for user-space coredump tools In-Reply-To: <20160623081545.GA20774@linaro.org> References: <6e96110e85e7b1167043d789e85f9c916fdf281a.1466120418.git.geoff@infradead.org> <20160620053222.GB14309@dhcppc9> <20160622055940.GY20774@linaro.org> <20160623071412.GA16398@dhcppc9> <20160623081545.GA20774@linaro.org> Message-ID: <20160623154624.GC16398@dhcppc9> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 23/06/2016:05:42:28 PM, AKASHI Takahiro wrote: > On Thu, Jun 23, 2016 at 12:44:12PM +0530, Pratyush Anand wrote: > > Hi Takahiro, > > > > Thanks for your reply. > > > > On 22/06/2016:02:59:41 PM, AKASHI Takahiro wrote: > > > On Mon, Jun 20, 2016 at 11:02:22AM +0530, Pratyush Anand wrote: > > > > +Atsushi > > > > > > > > Hi Takahiro, > > > > > > > > On 16/06/2016:11:48:28 PM, Geoff Levand wrote: > > > > > From: AKASHI Takahiro > > > > > > > > > > For the current crash utility, we need to know, at least, > > > > > - kimage_voffset > > > > > - PHYS_OFFSET > > > > > to handle the contents of core dump file (/proc/vmcore) correctly due to > > > > > the introduction of KASLR (CONFIG_RANDOMIZE_BASE) in v4.6. > > > > > This patch puts them as VMCOREINFO's into the file. > > > > > > > > > > - VA_BITS > > > > > is also added for makedumpfile command. > > > > > > > > Thanks for adding them. They are quite helpful for makedumpfile as well. > > > > > > > > > More VMCOREINFO's may be added later. > > > > > > > > Yes, we will need to pass VMCOREINFO_SYMBOL(_text) and VMCOREINFO_SYMBOL(_end) > > > > in order to work with makedumpfile. > > > > > > I know that adding those symbols is the easiest way, but > > > theoretically, if we know the physical address of "swapper_pg_dir", > > > > But, we know only it's virtual address. > > What I meant here is that, if we know the physical address of "swapper_pg_dir", > we don't have to know neither of "_text", "_end", "kimage_voffset" nor > "PHYS_OFFSET". > I just wanted to ask you whether you thought of this possibility. OK, Let me clarify the needs from makedumpfile perspective. Atsushi can correct me if I am wrong: - As a minimal we need some way of reading page table entries. Currently vmcore has only virtual address of swapper_pg_dir, but if you can provide __pa(swapper_pg_dir) in vmcore, then we do not need either "_text", "_end" or "kimage_voffset". - However, "PHYS_OFFSET" might still be needed. makedumpfile core code needs to know phys_base. Currently we find it from the PT_LOAD segments. We treat the lowest start of segment's LMAs as the physical base. I am not sure, if that is still true with latest KASAN changes. - Further, if you do not provide "_text" and "_end", then we will have to translate each address with complex page table read process, which would slow makedumpfile execution significantly. Now, please let us know that what will be the kernel strategy, so that I can re-organise makedumpfile accordingly. > > > > > instead of its virtual address, we can access all the memory pointed to > > > by any kernel virtual address. > > > How do you rationalize that we need to know "_text" and "_end"? > > > > Well, we need some mechanism so that we can decide if an address can be > > translated using linear mapping of virt_to_phys(). > > All the entries in MMU tables are based on "physical" addresses, > and so we don't care whether a given address is in linear mapping or not > in walking through MMU. > See what I mean? Yes, yes, I thought, we have only virtual swapper_pg_dir, so I was looking for a way to translate *atleast* that into physical. Thanks for your input. Those are helpful. ~Pratyush > > Thanks, > -Takahiro AKASHI > > > Alternatively, probably we can do like this: > > -- Translate all address between "SYMBOL(swapper_pg_dir)" and "SYMBOL(swapper_pg_dir) > > + SWAPPER_DIR_SIZE" using virt_to_phys() and now we can read values from > > dumpfile using that physical address. This way we can get PGD/PMD/PUD values. > > -- PTE values may lie out side this range, however that address should still be > > linearly translatable. We can use virt_to_phys() macro from them as well. > > > > In summary, we can translate address of PGD/PMD/PUD/PTE using virt_to_phys() > > and rest all can be translated using page table entries. > > > > ~Pratyush From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-qk0-f169.google.com ([209.85.220.169]) by bombadil.infradead.org with esmtps (Exim 4.85_2 #1 (Red Hat Linux)) id 1bG6qA-0007lj-6W for kexec@lists.infradead.org; Thu, 23 Jun 2016 15:46:51 +0000 Received: by mail-qk0-f169.google.com with SMTP id t127so111972063qkf.1 for ; Thu, 23 Jun 2016 08:46:29 -0700 (PDT) Date: Thu, 23 Jun 2016 21:16:24 +0530 From: Pratyush Anand Subject: Re: [PATCH v19 10/13] arm64: kdump: add VMCOREINFO's for user-space coredump tools Message-ID: <20160623154624.GC16398@dhcppc9> References: <6e96110e85e7b1167043d789e85f9c916fdf281a.1466120418.git.geoff@infradead.org> <20160620053222.GB14309@dhcppc9> <20160622055940.GY20774@linaro.org> <20160623071412.GA16398@dhcppc9> <20160623081545.GA20774@linaro.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20160623081545.GA20774@linaro.org> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "kexec" Errors-To: kexec-bounces+dwmw2=infradead.org@lists.infradead.org To: AKASHI Takahiro , Atsushi Kumagai Cc: Mark Rutland , Geoff Levand , Catalin Marinas , Will Deacon , marc.zyngier@arm.com, James Morse , kexec@lists.infradead.org, linux-arm-kernel@lists.infradead.org On 23/06/2016:05:42:28 PM, AKASHI Takahiro wrote: > On Thu, Jun 23, 2016 at 12:44:12PM +0530, Pratyush Anand wrote: > > Hi Takahiro, > > > > Thanks for your reply. > > > > On 22/06/2016:02:59:41 PM, AKASHI Takahiro wrote: > > > On Mon, Jun 20, 2016 at 11:02:22AM +0530, Pratyush Anand wrote: > > > > +Atsushi > > > > > > > > Hi Takahiro, > > > > > > > > On 16/06/2016:11:48:28 PM, Geoff Levand wrote: > > > > > From: AKASHI Takahiro > > > > > > > > > > For the current crash utility, we need to know, at least, > > > > > - kimage_voffset > > > > > - PHYS_OFFSET > > > > > to handle the contents of core dump file (/proc/vmcore) correctly due to > > > > > the introduction of KASLR (CONFIG_RANDOMIZE_BASE) in v4.6. > > > > > This patch puts them as VMCOREINFO's into the file. > > > > > > > > > > - VA_BITS > > > > > is also added for makedumpfile command. > > > > > > > > Thanks for adding them. They are quite helpful for makedumpfile as well. > > > > > > > > > More VMCOREINFO's may be added later. > > > > > > > > Yes, we will need to pass VMCOREINFO_SYMBOL(_text) and VMCOREINFO_SYMBOL(_end) > > > > in order to work with makedumpfile. > > > > > > I know that adding those symbols is the easiest way, but > > > theoretically, if we know the physical address of "swapper_pg_dir", > > > > But, we know only it's virtual address. > > What I meant here is that, if we know the physical address of "swapper_pg_dir", > we don't have to know neither of "_text", "_end", "kimage_voffset" nor > "PHYS_OFFSET". > I just wanted to ask you whether you thought of this possibility. OK, Let me clarify the needs from makedumpfile perspective. Atsushi can correct me if I am wrong: - As a minimal we need some way of reading page table entries. Currently vmcore has only virtual address of swapper_pg_dir, but if you can provide __pa(swapper_pg_dir) in vmcore, then we do not need either "_text", "_end" or "kimage_voffset". - However, "PHYS_OFFSET" might still be needed. makedumpfile core code needs to know phys_base. Currently we find it from the PT_LOAD segments. We treat the lowest start of segment's LMAs as the physical base. I am not sure, if that is still true with latest KASAN changes. - Further, if you do not provide "_text" and "_end", then we will have to translate each address with complex page table read process, which would slow makedumpfile execution significantly. Now, please let us know that what will be the kernel strategy, so that I can re-organise makedumpfile accordingly. > > > > > instead of its virtual address, we can access all the memory pointed to > > > by any kernel virtual address. > > > How do you rationalize that we need to know "_text" and "_end"? > > > > Well, we need some mechanism so that we can decide if an address can be > > translated using linear mapping of virt_to_phys(). > > All the entries in MMU tables are based on "physical" addresses, > and so we don't care whether a given address is in linear mapping or not > in walking through MMU. > See what I mean? Yes, yes, I thought, we have only virtual swapper_pg_dir, so I was looking for a way to translate *atleast* that into physical. Thanks for your input. Those are helpful. ~Pratyush > > Thanks, > -Takahiro AKASHI > > > Alternatively, probably we can do like this: > > -- Translate all address between "SYMBOL(swapper_pg_dir)" and "SYMBOL(swapper_pg_dir) > > + SWAPPER_DIR_SIZE" using virt_to_phys() and now we can read values from > > dumpfile using that physical address. This way we can get PGD/PMD/PUD values. > > -- PTE values may lie out side this range, however that address should still be > > linearly translatable. We can use virt_to_phys() macro from them as well. > > > > In summary, we can translate address of PGD/PMD/PUD/PTE using virt_to_phys() > > and rest all can be translated using page table entries. > > > > ~Pratyush _______________________________________________ kexec mailing list kexec@lists.infradead.org http://lists.infradead.org/mailman/listinfo/kexec