[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping
From: |
Wen Congyang |
Subject: |
Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping |
Date: |
Fri, 09 Mar 2012 10:26:56 +0800 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.9) Gecko/20100413 Fedora/3.0.4-2.fc13 Thunderbird/3.0.4 |
At 03/09/2012 10:05 AM, HATAYAMA Daisuke Wrote:
> From: Wen Congyang <address@hidden>
> Subject: Re: [RFC][PATCH 05/16 v8] Add API to get memory mapping
> Date: Fri, 09 Mar 2012 09:46:31 +0800
>
>> At 03/09/2012 08:40 AM, HATAYAMA Daisuke Wrote:
>>> From: Wen Congyang <address@hidden>
>>> Subject: Re: [RFC][PATCH 05/16 v8] Add API to get memory mapping
>>> Date: Thu, 08 Mar 2012 16:52:29 +0800
>>>
>>>> At 03/07/2012 11:27 PM, HATAYAMA Daisuke Wrote:
>>>>> From: Wen Congyang <address@hidden>
>>>>> Subject: [RFC][PATCH 05/16 v8] Add API to get memory mapping
>>>>> Date: Fri, 02 Mar 2012 18:18:23 +0800
>>>>>
>>>>>
>
>>>>> How does the memory portion referenced by PT_LOAD program headers with
>>>>> p_vaddr == 0 looks through gdb? If we cannot access such portions,
>>>>> part not referenced by the page table CR3 has is unnecessary, isn't
>>>>> it?
>>>>
>>>> The part is unnecessary if you use gdb. But it is necessary if you use
>>>> crash.
>>>>
>>>
>>> crash users would not use paging option because even if without using
>>> it, we can see all memory well, so the paging option is only for gdb
>>> users.
>>
>> Yes, the paging option is only for gdb users. The default value if off.
>>
>>>
>>> It looks to me that the latter part only complicates the logic. If
>>> instead, collecting virtual addresses only, way of handling PT_LOAD
>>> entries become simpler, for example, they no longer need to be
>>> physically contiguous in a single entry, and reviewing and maintaince
>>> becomes easy.
>>
>> Sorry, I donot understand what do you want to say.
>>
>
> The processing that adds part not referenced by page table to vmcore
> is meaningless for gdb. crash doesn't require it. So, it only
> complicates the current logic.
If the paging mode is on, we can also use crash to analyze the vmcore.
As the comment methioned, the memory used by the 1st kernel may be not
referenced by the page table, so we neet this logic.
Thanks
Wen Congyang
>
> Thanks.
> HATAYAMA, Daisuke
>
>
- [Qemu-devel] [RFC][PATCH 01/16 v8] Add API to create memory mapping list, (continued)
- [Qemu-devel] [RFC][PATCH 01/16 v8] Add API to create memory mapping list, Wen Congyang, 2012/03/02
- [Qemu-devel] [RFC][PATCH 02/16 v8] Add API to check whether a physical address is I/O address, Wen Congyang, 2012/03/02
- [Qemu-devel] [RFC][PATCH 04/16 v8] Add API to check whether paging mode is enabled, Wen Congyang, 2012/03/02
- [Qemu-devel] [RFC][PATCH 03/16 v8] implement cpu_get_memory_mapping(), Wen Congyang, 2012/03/02
- [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping, Wen Congyang, 2012/03/02
- Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping, HATAYAMA Daisuke, 2012/03/07
- Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping, Wen Congyang, 2012/03/08
- Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping, HATAYAMA Daisuke, 2012/03/08
- Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping, Wen Congyang, 2012/03/08
- Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping, HATAYAMA Daisuke, 2012/03/08
- Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping,
Wen Congyang <=
- Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping, HATAYAMA Daisuke, 2012/03/08
- Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping, Jan Kiszka, 2012/03/09
- Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping, Wen Congyang, 2012/03/09
- Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping, Jan Kiszka, 2012/03/09
- Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping, Jan Kiszka, 2012/03/09
- Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping, HATAYAMA Daisuke, 2012/03/09
- Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping, Jan Kiszka, 2012/03/09
- Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping, HATAYAMA Daisuke, 2012/03/12
- Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping, HATAYAMA Daisuke, 2012/03/12
- Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping, Wen Congyang, 2012/03/11