Re: [PATCH v5 03/13] x86, 64bit: Set extra ident mapping for whole kernel range

2012-12-21 Thread Konrad Rzeszutek Wilk
On Fri, Dec 21, 2012 at 02:35:25PM -0800, Yinghai Lu wrote: > On Fri, Dec 21, 2012 at 2:28 PM, Konrad Rzeszutek Wilk > wrote: > > Perhaps 'spare_directory' ? Or 'spare_table' ? > > we have evolved -v7 and -v8 that is using FANCY/FUNNY patch of #PF > handler set page table from HPA. > Ah, that i

Re: [PATCH v5 03/13] x86, 64bit: Set extra ident mapping for whole kernel range

2012-12-21 Thread H. Peter Anvin
On 12/21/2012 02:51 PM, Yinghai Lu wrote: looks like no one care about kgdb now. hope they could notice until that is broken later. It's late December -- people are off. -hpa -- H. Peter Anvin, Intel Open Source Technology Center I work for Intel. I don't speak on their behalf. --

Re: [PATCH v5 03/13] x86, 64bit: Set extra ident mapping for whole kernel range

2012-12-21 Thread Yinghai Lu
On Fri, Dec 21, 2012 at 2:39 PM, H. Peter Anvin wrote: > On 12/21/2012 02:35 PM, Yinghai Lu wrote: >> >> stop #PF handler after init_mem_mapping >> git://git.kernel.org/pub/scm/linux/kernel/git/yinghai/linux-yinghai.git >> for-x86-boot-v7 >> >> stop #PF handler in x86_64_start_kernel, to keep kgdb

Re: [PATCH v5 03/13] x86, 64bit: Set extra ident mapping for whole kernel range

2012-12-21 Thread H. Peter Anvin
On 12/21/2012 02:35 PM, Yinghai Lu wrote: > > stop #PF handler after init_mem_mapping > git://git.kernel.org/pub/scm/linux/kernel/git/yinghai/linux-yinghai.git > for-x86-boot-v7 > > stop #PF handler in x86_64_start_kernel, to keep kgdb working. > git://git.kernel.org/pub/scm/linux/kernel/git/ying

Re: [PATCH v5 03/13] x86, 64bit: Set extra ident mapping for whole kernel range

2012-12-21 Thread Yinghai Lu
On Fri, Dec 21, 2012 at 2:28 PM, Konrad Rzeszutek Wilk wrote: > Perhaps 'spare_directory' ? Or 'spare_table' ? we have evolved -v7 and -v8 that is using FANCY/FUNNY patch of #PF handler set page table from HPA. Please do check if -v7 and -v8 break xen again. (it should not !, but i only test dom

Re: [PATCH v5 03/13] x86, 64bit: Set extra ident mapping for whole kernel range

2012-12-21 Thread Konrad Rzeszutek Wilk
On Tue, Nov 27, 2012 at 11:50:32PM -0800, Yinghai Lu wrote: > Current when kernel is loaded above 1G, only [_text, _text+2M] is set > up with extra ident page table. > That is not enough, some variables that could be used early are out of > that range, like BRK for early page table. > Need to set m

[PATCH v5 03/13] x86, 64bit: Set extra ident mapping for whole kernel range

2012-11-27 Thread Yinghai Lu
Current when kernel is loaded above 1G, only [_text, _text+2M] is set up with extra ident page table. That is not enough, some variables that could be used early are out of that range, like BRK for early page table. Need to set map for [_text, _end] include text/data/bss/brk... Also current kernel