Re: [PATCH -mm -v3 1/2] i386/x86_64 boot: setup data

2007-09-28 Thread H. Peter Anvin
Huang, Ying wrote: > Hi, Peter, > > On Wed, 2007-09-19 at 09:04 -0700, H. Peter Anvin wrote: >> Huang, Ying wrote: >>> Known Issues: >>> >>> 1. Where is safe to place the linked list of setup_data? >>> Because the length of the linked list of setup_data is variable, it >>> can not be copied into

Re: [PATCH -mm -v3 1/2] i386/x86_64 boot: setup data

2007-09-28 Thread H. Peter Anvin
Huang, Ying wrote: Hi, Peter, On Wed, 2007-09-19 at 09:04 -0700, H. Peter Anvin wrote: Huang, Ying wrote: Known Issues: 1. Where is safe to place the linked list of setup_data? Because the length of the linked list of setup_data is variable, it can not be copied into BSS segment of

Re: [PATCH -mm -v3 1/2] i386/x86_64 boot: setup data

2007-09-27 Thread Huang, Ying
Hi, Peter, On Wed, 2007-09-19 at 09:04 -0700, H. Peter Anvin wrote: > Huang, Ying wrote: > > > > Known Issues: > > > > 1. Where is safe to place the linked list of setup_data? > > Because the length of the linked list of setup_data is variable, it > > can not be copied into BSS segment of

Re: [PATCH -mm -v3 1/2] i386/x86_64 boot: setup data

2007-09-27 Thread Huang, Ying
Hi, Peter, On Wed, 2007-09-19 at 09:04 -0700, H. Peter Anvin wrote: Huang, Ying wrote: Known Issues: 1. Where is safe to place the linked list of setup_data? Because the length of the linked list of setup_data is variable, it can not be copied into BSS segment of kernel as that of

Re: [PATCH -mm -v3 1/2] i386/x86_64 boot: setup data

2007-09-22 Thread H. Peter Anvin
Jeremy Fitzhardinge wrote: > > Do you actually need a linked list of data? This is similar to the > changes to bzImage to support booting bzImage a paravirt environment, > but I just proposed a pointer to a single info structure, along with a > field to identify the boot environment (ie,

Re: [PATCH -mm -v3 1/2] i386/x86_64 boot: setup data

2007-09-22 Thread Jeremy Fitzhardinge
Huang, Ying wrote: > This patch add a field of 64-bit physical pointer to NULL terminated > single linked list of struct setup_data to real-mode kernel > header. This is used as a more extensible boot parameters passing > mechanism. > > This patch has been tested against 2.6.23-rc6-mm1 kernel on

Re: [PATCH -mm -v3 1/2] i386/x86_64 boot: setup data

2007-09-22 Thread Jeremy Fitzhardinge
Huang, Ying wrote: This patch add a field of 64-bit physical pointer to NULL terminated single linked list of struct setup_data to real-mode kernel header. This is used as a more extensible boot parameters passing mechanism. This patch has been tested against 2.6.23-rc6-mm1 kernel on x86_64.

Re: [PATCH -mm -v3 1/2] i386/x86_64 boot: setup data

2007-09-22 Thread H. Peter Anvin
Jeremy Fitzhardinge wrote: Do you actually need a linked list of data? This is similar to the changes to bzImage to support booting bzImage a paravirt environment, but I just proposed a pointer to a single info structure, along with a field to identify the boot environment (ie,

Re: [PATCH -mm -v3 1/2] i386/x86_64 boot: setup data

2007-09-19 Thread H. Peter Anvin
Huang, Ying wrote: > > Known Issues: > > 1. Where is safe to place the linked list of setup_data? > Because the length of the linked list of setup_data is variable, it > can not be copied into BSS segment of kernel as that of "zero > page". We must find a safe place for it, where it will not be

[PATCH -mm -v3 1/2] i386/x86_64 boot: setup data

2007-09-19 Thread Huang, Ying
This patch add a field of 64-bit physical pointer to NULL terminated single linked list of struct setup_data to real-mode kernel header. This is used as a more extensible boot parameters passing mechanism. This patch has been tested against 2.6.23-rc6-mm1 kernel on x86_64. It is based on the

[PATCH -mm -v3 1/2] i386/x86_64 boot: setup data

2007-09-19 Thread Huang, Ying
This patch add a field of 64-bit physical pointer to NULL terminated single linked list of struct setup_data to real-mode kernel header. This is used as a more extensible boot parameters passing mechanism. This patch has been tested against 2.6.23-rc6-mm1 kernel on x86_64. It is based on the

Re: [PATCH -mm -v3 1/2] i386/x86_64 boot: setup data

2007-09-19 Thread H. Peter Anvin
Huang, Ying wrote: Known Issues: 1. Where is safe to place the linked list of setup_data? Because the length of the linked list of setup_data is variable, it can not be copied into BSS segment of kernel as that of zero page. We must find a safe place for it, where it will not be