Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-22 Thread H. Peter Anvin
Huang, Ying wrote: > > My intention is that we have 3 possible schemes for kernel to use boot > information. > That's not an intention, it's an observation. > 1. Use "linked list" only. Then if booted with old bootloader which uses > "zero page" protocol, the "zero page" information provided

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-22 Thread Huang, Ying
On Thu, 2007-08-23 at 00:28 +0800, H. Peter Anvin wrote: > huang ying wrote: > > > > My proposal: Use Peter proposed "linked list of struct setup_data" > > style boot protocol as long term goal. > > > > To smooth the transforming process, the following back compatible > > scheme can be taken: > >

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-22 Thread H. Peter Anvin
Eric W. Biederman wrote: > I need to review HPA changes a little more from his rewrite in C. > I think he messed up with the guarantee of initialization in that > rewrite, but I'm not quite certain. I beg to differ, sir! In my rewritten code, the "zeropage" is 4K of zero-initialized .bss, into

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-22 Thread H. Peter Anvin
huang ying wrote: > > My proposal: Use Peter proposed "linked list of struct setup_data" > style boot protocol as long term goal. > > To smooth the transforming process, the following back compatible > scheme can be taken: > > 1. Keep zero page as an informal external boot protocol, and marked

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-22 Thread Eric W. Biederman
Andi Kleen <[EMAIL PROTECTED]> writes: > On Tue, Aug 21, 2007 at 11:43:38PM -0700, Yinghai Lu wrote: >> On 8/21/07, Andi Kleen <[EMAIL PROTECTED]> wrote: >> > > current LinuxBIOS's path: the elfboot in LinuxBIOS will prepare the >> > > e820 table, and jump to startup_32 in kernel. is that not

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-22 Thread huang ying
On 8/22/07, Andi Kleen <[EMAIL PROTECTED]> wrote: > The short term fix is probably to just add a version number to > the zero page and make sure new changes only add stuff to the > end and the kernel has reasonable compat code for old versions. > > Then LinuxBIOS would need to be changed to supply

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-22 Thread Andi Kleen
On Tue, Aug 21, 2007 at 11:43:38PM -0700, Yinghai Lu wrote: > On 8/21/07, Andi Kleen <[EMAIL PROTECTED]> wrote: > > > current LinuxBIOS's path: the elfboot in LinuxBIOS will prepare the > > > e820 table, and jump to startup_32 in kernel. is that not good and > > > simple? > > > > The problem is

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-22 Thread Yinghai Lu
On 8/21/07, Andi Kleen <[EMAIL PROTECTED]> wrote: > > current LinuxBIOS's path: the elfboot in LinuxBIOS will prepare the > > e820 table, and jump to startup_32 in kernel. is that not good and > > simple? > > The problem is that the zero page cannot be changed at all in this > setup. Or rather it

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-22 Thread Yinghai Lu
On 8/21/07, Andi Kleen <[EMAIL PROTECTED]> wrote: > > current LinuxBIOS's path: the elfboot in LinuxBIOS will prepare the > > e820 table, and jump to startup_32 in kernel. is that not good and > > simple? > > The problem is that the zero page cannot be changed at all in this > setup. Or rather it

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-22 Thread Yinghai Lu
On 8/21/07, Andi Kleen [EMAIL PROTECTED] wrote: current LinuxBIOS's path: the elfboot in LinuxBIOS will prepare the e820 table, and jump to startup_32 in kernel. is that not good and simple? The problem is that the zero page cannot be changed at all in this setup. Or rather it can be only

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-22 Thread Yinghai Lu
On 8/21/07, Andi Kleen [EMAIL PROTECTED] wrote: current LinuxBIOS's path: the elfboot in LinuxBIOS will prepare the e820 table, and jump to startup_32 in kernel. is that not good and simple? The problem is that the zero page cannot be changed at all in this setup. Or rather it can be only

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-22 Thread Andi Kleen
On Tue, Aug 21, 2007 at 11:43:38PM -0700, Yinghai Lu wrote: On 8/21/07, Andi Kleen [EMAIL PROTECTED] wrote: current LinuxBIOS's path: the elfboot in LinuxBIOS will prepare the e820 table, and jump to startup_32 in kernel. is that not good and simple? The problem is that the zero page

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-22 Thread huang ying
On 8/22/07, Andi Kleen [EMAIL PROTECTED] wrote: The short term fix is probably to just add a version number to the zero page and make sure new changes only add stuff to the end and the kernel has reasonable compat code for old versions. Then LinuxBIOS would need to be changed to supply that

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-22 Thread Eric W. Biederman
Andi Kleen [EMAIL PROTECTED] writes: On Tue, Aug 21, 2007 at 11:43:38PM -0700, Yinghai Lu wrote: On 8/21/07, Andi Kleen [EMAIL PROTECTED] wrote: current LinuxBIOS's path: the elfboot in LinuxBIOS will prepare the e820 table, and jump to startup_32 in kernel. is that not good and

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-22 Thread H. Peter Anvin
huang ying wrote: My proposal: Use Peter proposed linked list of struct setup_data style boot protocol as long term goal. To smooth the transforming process, the following back compatible scheme can be taken: 1. Keep zero page as an informal external boot protocol, and marked it as

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-22 Thread H. Peter Anvin
Eric W. Biederman wrote: I need to review HPA changes a little more from his rewrite in C. I think he messed up with the guarantee of initialization in that rewrite, but I'm not quite certain. I beg to differ, sir! In my rewritten code, the zeropage is 4K of zero-initialized .bss, into which

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-22 Thread Huang, Ying
On Thu, 2007-08-23 at 00:28 +0800, H. Peter Anvin wrote: huang ying wrote: My proposal: Use Peter proposed linked list of struct setup_data style boot protocol as long term goal. To smooth the transforming process, the following back compatible scheme can be taken: 1. Keep zero

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-22 Thread H. Peter Anvin
Huang, Ying wrote: My intention is that we have 3 possible schemes for kernel to use boot information. That's not an intention, it's an observation. 1. Use linked list only. Then if booted with old bootloader which uses zero page protocol, the zero page information provided by bootloader

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-21 Thread Andi Kleen
> current LinuxBIOS's path: the elfboot in LinuxBIOS will prepare the > e820 table, and jump to startup_32 in kernel. is that not good and > simple? The problem is that the zero page cannot be changed at all in this setup. Or rather it can be only changed by breaking LinuxBios. -Andi - To

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-21 Thread Yinghai Lu
On 8/21/07, Andi Kleen <[EMAIL PROTECTED]> wrote: > On Tue, Aug 21, 2007 at 03:41:44AM -0700, H. Peter Anvin wrote: > > Andi Kleen wrote: > > >> - "struct boot_params" (the zeropage) is kept as a legacy interface. > > > > > > Legacy interface for what? Just for kexec utils which never should > >

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-21 Thread Andi Kleen
On Tue, Aug 21, 2007 at 03:41:44AM -0700, H. Peter Anvin wrote: > Andi Kleen wrote: > >> - "struct boot_params" (the zeropage) is kept as a legacy interface. > > > > Legacy interface for what? Just for kexec utils which never should > > have been using it anyways keeping backwards cruft around

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-21 Thread H. Peter Anvin
Andi Kleen wrote: >> - "struct boot_params" (the zeropage) is kept as a legacy interface. > > Legacy interface for what? Just for kexec utils which never should > have been using it anyways keeping backwards cruft around seems > misplac.ed Worse. LinuxBIOS. :( -hpa - To unsubscribe

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-21 Thread Andi Kleen
> - "struct boot_params" (the zeropage) is kept as a legacy interface. Legacy interface for what? Just for kexec utils which never should have been using it anyways keeping backwards cruft around seems misplac.ed -Andi - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-21 Thread Andi Kleen
- struct boot_params (the zeropage) is kept as a legacy interface. Legacy interface for what? Just for kexec utils which never should have been using it anyways keeping backwards cruft around seems misplac.ed -Andi - To unsubscribe from this list: send the line unsubscribe linux-kernel in the

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-21 Thread H. Peter Anvin
Andi Kleen wrote: - struct boot_params (the zeropage) is kept as a legacy interface. Legacy interface for what? Just for kexec utils which never should have been using it anyways keeping backwards cruft around seems misplac.ed Worse. LinuxBIOS. :( -hpa - To unsubscribe from this

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-21 Thread Andi Kleen
On Tue, Aug 21, 2007 at 03:41:44AM -0700, H. Peter Anvin wrote: Andi Kleen wrote: - struct boot_params (the zeropage) is kept as a legacy interface. Legacy interface for what? Just for kexec utils which never should have been using it anyways keeping backwards cruft around seems

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-21 Thread Yinghai Lu
On 8/21/07, Andi Kleen [EMAIL PROTECTED] wrote: On Tue, Aug 21, 2007 at 03:41:44AM -0700, H. Peter Anvin wrote: Andi Kleen wrote: - struct boot_params (the zeropage) is kept as a legacy interface. Legacy interface for what? Just for kexec utils which never should have been using it

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-21 Thread Andi Kleen
current LinuxBIOS's path: the elfboot in LinuxBIOS will prepare the e820 table, and jump to startup_32 in kernel. is that not good and simple? The problem is that the zero page cannot be changed at all in this setup. Or rather it can be only changed by breaking LinuxBios. -Andi - To

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-20 Thread Huang, Ying
On Mon, 2007-08-20 at 20:54 -0700, H. Peter Anvin wrote: > Huang, Ying wrote: > > > > I think the "next" field can be u32 instead of u64. Because the linked > > list of struct setup_data is prepared by bootloader, which can control > > the memory location. > > > > That's making some pretty

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-20 Thread H. Peter Anvin
Huang, Ying wrote: > > I think the "next" field can be u32 instead of u64. Because the linked > list of struct setup_data is prepared by bootloader, which can control > the memory location. > That's making some pretty serious assumptions on future boot loaders and environments. > Previously, I

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-20 Thread Huang, Ying
On Mon, 2007-08-20 at 10:12 -0700, H. Peter Anvin wrote: > Huang, Ying wrote: > >> > >> I propose that, in addition to the aforementioned version number and > >> magic fields, we add a pointer, which should be the last pointer added > >> that doesn't point into I/O space or reserved memory (i.e.

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-20 Thread Matthew Garrett
On Mon, Aug 20, 2007 at 10:05:11AM -0700, H. Peter Anvin wrote: > Yinghai Lu wrote: > > > > someone told me that EFI PEI will be 32 bit ( for mem etc > > initialization), and after that will be 64 bit, so the Run time > > service will be 64 bit only, and it will only support 64 bit OS with > >

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-20 Thread San Mehat
On 8/20/07, H. Peter Anvin <[EMAIL PROTECTED]> wrote: > Yinghai Lu wrote: > > > > someone told me that EFI PEI will be 32 bit ( for mem etc > > initialization), and after that will be 64 bit, so the Run time > > service will be 64 bit only, and it will only support 64 bit OS with > > EFI. and they

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-20 Thread H. Peter Anvin
Huang, Ying wrote: >> >> I propose that, in addition to the aforementioned version number and >> magic fields, we add a pointer, which should be the last pointer added >> that doesn't point into I/O space or reserved memory (i.e. memory that >> is off limit anyway for the operating system.) >> >>

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-20 Thread H. Peter Anvin
Yinghai Lu wrote: > > someone told me that EFI PEI will be 32 bit ( for mem etc > initialization), and after that will be 64 bit, so the Run time > service will be 64 bit only, and it will only support 64 bit OS with > EFI. and they have another mode to emulate the legacy BIOS to boot > 32bit OS.

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-20 Thread H. Peter Anvin
Yinghai Lu wrote: someone told me that EFI PEI will be 32 bit ( for mem etc initialization), and after that will be 64 bit, so the Run time service will be 64 bit only, and it will only support 64 bit OS with EFI. and they have another mode to emulate the legacy BIOS to boot 32bit OS.

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-20 Thread H. Peter Anvin
Huang, Ying wrote: I propose that, in addition to the aforementioned version number and magic fields, we add a pointer, which should be the last pointer added that doesn't point into I/O space or reserved memory (i.e. memory that is off limit anyway for the operating system.) This pointer

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-20 Thread San Mehat
On 8/20/07, H. Peter Anvin [EMAIL PROTECTED] wrote: Yinghai Lu wrote: someone told me that EFI PEI will be 32 bit ( for mem etc initialization), and after that will be 64 bit, so the Run time service will be 64 bit only, and it will only support 64 bit OS with EFI. and they have another

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-20 Thread Matthew Garrett
On Mon, Aug 20, 2007 at 10:05:11AM -0700, H. Peter Anvin wrote: Yinghai Lu wrote: someone told me that EFI PEI will be 32 bit ( for mem etc initialization), and after that will be 64 bit, so the Run time service will be 64 bit only, and it will only support 64 bit OS with EFI. and they

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-20 Thread Huang, Ying
On Mon, 2007-08-20 at 10:12 -0700, H. Peter Anvin wrote: Huang, Ying wrote: I propose that, in addition to the aforementioned version number and magic fields, we add a pointer, which should be the last pointer added that doesn't point into I/O space or reserved memory (i.e. memory that

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-20 Thread H. Peter Anvin
Huang, Ying wrote: I think the next field can be u32 instead of u64. Because the linked list of struct setup_data is prepared by bootloader, which can control the memory location. That's making some pretty serious assumptions on future boot loaders and environments. Previously, I think

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-20 Thread Huang, Ying
On Mon, 2007-08-20 at 20:54 -0700, H. Peter Anvin wrote: Huang, Ying wrote: I think the next field can be u32 instead of u64. Because the linked list of struct setup_data is prepared by bootloader, which can control the memory location. That's making some pretty serious assumptions

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-19 Thread Huang, Ying
On Sun, 2007-08-19 at 16:25 -0600, Eric W. Biederman wrote: > "Huang, Ying" <[EMAIL PROTECTED]> writes: > >> > +#define EFI_LOADER_SIG ((unsigned char *)(PARAM+0x1c0)) > >> > +#define EFI_MEMDESC_SIZE (*((unsigned int *) (PARAM+0x1c4))) > >> > +#define EFI_MEMDESC_VERSION (*((unsigned int *)

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-19 Thread Huang, Ying
On Fri, 2007-08-17 at 09:11 -0700, H. Peter Anvin wrote: > Huang, Ying wrote: > >> > >> Has the zero page documentation and version numbering project > >> made any progress? I think we cannot merge this without at least > >> the version number > > > > More than that. You need to be able to boot

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-19 Thread Yinghai Lu
On 8/19/07, Eric W. Biederman <[EMAIL PROTECTED]> wrote: > "Huang, Ying" <[EMAIL PROTECTED]> writes: > > > On Thu, 2007-08-16 at 07:22 +0800, H. Peter Anvin wrote: > >> Andrew Morton wrote: > >> > On Mon, 13 Aug 2007 15:30:19 +0800 > >> > "Huang, Ying" <[EMAIL PROTECTED]> wrote: > >> > > >> >>

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-19 Thread Eric W. Biederman
"H. Peter Anvin" <[EMAIL PROTECTED]> writes: > Huang, Ying wrote: >> >> One question: >> >> The boot_params.efi_info.efi_systab is defined as u32. But it should be >> u64 on x86_64, because it comes from firmware and is not controlled by >> bootloader. But, changing it from u32 to u64 will

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-19 Thread Eric W. Biederman
"Huang, Ying" <[EMAIL PROTECTED]> writes: > On Thu, 2007-08-16 at 07:22 +0800, H. Peter Anvin wrote: >> Andrew Morton wrote: >> > On Mon, 13 Aug 2007 15:30:19 +0800 >> > "Huang, Ying" <[EMAIL PROTECTED]> wrote: >> > >> >> Following sets of patches add EFI/UEFI (Unified Extensible Firmware >> >>

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-19 Thread Eric W. Biederman
Huang, Ying [EMAIL PROTECTED] writes: On Thu, 2007-08-16 at 07:22 +0800, H. Peter Anvin wrote: Andrew Morton wrote: On Mon, 13 Aug 2007 15:30:19 +0800 Huang, Ying [EMAIL PROTECTED] wrote: Following sets of patches add EFI/UEFI (Unified Extensible Firmware Interface) runtime services

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-19 Thread Eric W. Biederman
H. Peter Anvin [EMAIL PROTECTED] writes: Huang, Ying wrote: One question: The boot_params.efi_info.efi_systab is defined as u32. But it should be u64 on x86_64, because it comes from firmware and is not controlled by bootloader. But, changing it from u32 to u64 will break current i386

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-19 Thread Yinghai Lu
On 8/19/07, Eric W. Biederman [EMAIL PROTECTED] wrote: Huang, Ying [EMAIL PROTECTED] writes: On Thu, 2007-08-16 at 07:22 +0800, H. Peter Anvin wrote: Andrew Morton wrote: On Mon, 13 Aug 2007 15:30:19 +0800 Huang, Ying [EMAIL PROTECTED] wrote: Following sets of patches add

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-19 Thread Huang, Ying
On Fri, 2007-08-17 at 09:11 -0700, H. Peter Anvin wrote: Huang, Ying wrote: Has the zero page documentation and version numbering project made any progress? I think we cannot merge this without at least the version number More than that. You need to be able to boot a 32-bit kernel

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-19 Thread Huang, Ying
On Sun, 2007-08-19 at 16:25 -0600, Eric W. Biederman wrote: Huang, Ying [EMAIL PROTECTED] writes: +#define EFI_LOADER_SIG ((unsigned char *)(PARAM+0x1c0)) +#define EFI_MEMDESC_SIZE (*((unsigned int *) (PARAM+0x1c4))) +#define EFI_MEMDESC_VERSION (*((unsigned int *) (PARAM+0x1c8)))

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-17 Thread H. Peter Anvin
Huang, Ying wrote: >> >> Has the zero page documentation and version numbering project >> made any progress? I think we cannot merge this without at least >> the version number > More than that. You need to be able to boot a 32-bit kernel on a 64-bit system, so anything that breaks that is a

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-17 Thread H. Peter Anvin
Huang, Ying wrote: Has the zero page documentation and version numbering project made any progress? I think we cannot merge this without at least the version number More than that. You need to be able to boot a 32-bit kernel on a 64-bit system, so anything that breaks that is a nonstarter.

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-16 Thread Huang, Ying
On Thu, 2007-08-16 at 16:11 +0200, Andi Kleen wrote: > Huang, Ying wrote: > > On Thu, 2007-08-16 at 06:42 +0800, Andrew Morton wrote: > >> On Mon, 13 Aug 2007 15:30:19 +0800 > >> "Huang, Ying" <[EMAIL PROTECTED]> wrote: > >> > >>> Following sets of patches add EFI/UEFI (Unified Extensible Firmware

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-16 Thread H. Peter Anvin
Huang, Ying wrote: > > One question: > > The boot_params.efi_info.efi_systab is defined as u32. But it should be > u64 on x86_64, because it comes from firmware and is not controlled by > bootloader. But, changing it from u32 to u64 will break current i386 EFI > support, should we change it and

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-16 Thread Andi Kleen
Huang, Ying wrote: > On Thu, 2007-08-16 at 06:42 +0800, Andrew Morton wrote: >> On Mon, 13 Aug 2007 15:30:19 +0800 >> "Huang, Ying" <[EMAIL PROTECTED]> wrote: >> >>> Following sets of patches add EFI/UEFI (Unified Extensible Firmware >>> Interface) runtime services support to x86_64 architecture.

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-16 Thread Huang, Ying
On Thu, 2007-08-16 at 07:22 +0800, H. Peter Anvin wrote: > Andrew Morton wrote: > > On Mon, 13 Aug 2007 15:30:19 +0800 > > "Huang, Ying" <[EMAIL PROTECTED]> wrote: > > > >> Following sets of patches add EFI/UEFI (Unified Extensible Firmware > >> Interface) runtime services support to x86_64

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-16 Thread Huang, Ying
On Thu, 2007-08-16 at 07:16 +0800, Andrew Morton wrote: > On Mon, 13 Aug 2007 15:30:19 +0800 > "Huang, Ying" <[EMAIL PROTECTED]> wrote: > > > Following sets of patches add EFI/UEFI (Unified Extensible Firmware > > Interface) runtime services support to x86_64 architecture. > > OK, we have a

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-16 Thread Huang, Ying
On Thu, 2007-08-16 at 06:42 +0800, Andrew Morton wrote: > On Mon, 13 Aug 2007 15:30:19 +0800 > "Huang, Ying" <[EMAIL PROTECTED]> wrote: > > > Following sets of patches add EFI/UEFI (Unified Extensible Firmware > > Interface) runtime services support to x86_64 architecture. > > I had to rework

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-16 Thread Huang, Ying
On Thu, 2007-08-16 at 06:42 +0800, Andrew Morton wrote: On Mon, 13 Aug 2007 15:30:19 +0800 Huang, Ying [EMAIL PROTECTED] wrote: Following sets of patches add EFI/UEFI (Unified Extensible Firmware Interface) runtime services support to x86_64 architecture. I had to rework these a bit due

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-16 Thread Huang, Ying
On Thu, 2007-08-16 at 07:16 +0800, Andrew Morton wrote: On Mon, 13 Aug 2007 15:30:19 +0800 Huang, Ying [EMAIL PROTECTED] wrote: Following sets of patches add EFI/UEFI (Unified Extensible Firmware Interface) runtime services support to x86_64 architecture. OK, we have a major trainwreck

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-16 Thread Huang, Ying
On Thu, 2007-08-16 at 07:22 +0800, H. Peter Anvin wrote: Andrew Morton wrote: On Mon, 13 Aug 2007 15:30:19 +0800 Huang, Ying [EMAIL PROTECTED] wrote: Following sets of patches add EFI/UEFI (Unified Extensible Firmware Interface) runtime services support to x86_64 architecture. OK,

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-16 Thread Andi Kleen
Huang, Ying wrote: On Thu, 2007-08-16 at 06:42 +0800, Andrew Morton wrote: On Mon, 13 Aug 2007 15:30:19 +0800 Huang, Ying [EMAIL PROTECTED] wrote: Following sets of patches add EFI/UEFI (Unified Extensible Firmware Interface) runtime services support to x86_64 architecture. I had to rework

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-16 Thread H. Peter Anvin
Huang, Ying wrote: One question: The boot_params.efi_info.efi_systab is defined as u32. But it should be u64 on x86_64, because it comes from firmware and is not controlled by bootloader. But, changing it from u32 to u64 will break current i386 EFI support, should we change it and fix the

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-16 Thread Huang, Ying
On Thu, 2007-08-16 at 16:11 +0200, Andi Kleen wrote: Huang, Ying wrote: On Thu, 2007-08-16 at 06:42 +0800, Andrew Morton wrote: On Mon, 13 Aug 2007 15:30:19 +0800 Huang, Ying [EMAIL PROTECTED] wrote: Following sets of patches add EFI/UEFI (Unified Extensible Firmware Interface)

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-15 Thread H. Peter Anvin
Andrew Morton wrote: > On Mon, 13 Aug 2007 15:30:19 +0800 > "Huang, Ying" <[EMAIL PROTECTED]> wrote: > >> Following sets of patches add EFI/UEFI (Unified Extensible Firmware >> Interface) runtime services support to x86_64 architecture. > > OK, we have a major trainwreck when these patches meet

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-15 Thread Andrew Morton
On Mon, 13 Aug 2007 15:30:19 +0800 "Huang, Ying" <[EMAIL PROTECTED]> wrote: > Following sets of patches add EFI/UEFI (Unified Extensible Firmware > Interface) runtime services support to x86_64 architecture. OK, we have a major trainwreck when these patches meet Peter's get-newsetup.patch. I'm

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-15 Thread Andrew Morton
On Mon, 13 Aug 2007 15:30:19 +0800 "Huang, Ying" <[EMAIL PROTECTED]> wrote: > Following sets of patches add EFI/UEFI (Unified Extensible Firmware > Interface) runtime services support to x86_64 architecture. I had to rework these a bit due to clashes with x86_64-add-acpi-reboot-option.patch.

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-15 Thread Andrew Morton
On Mon, 13 Aug 2007 15:30:19 +0800 Huang, Ying [EMAIL PROTECTED] wrote: Following sets of patches add EFI/UEFI (Unified Extensible Firmware Interface) runtime services support to x86_64 architecture. I had to rework these a bit due to clashes with x86_64-add-acpi-reboot-option.patch. Please

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-15 Thread Andrew Morton
On Mon, 13 Aug 2007 15:30:19 +0800 Huang, Ying [EMAIL PROTECTED] wrote: Following sets of patches add EFI/UEFI (Unified Extensible Firmware Interface) runtime services support to x86_64 architecture. OK, we have a major trainwreck when these patches meet Peter's get-newsetup.patch. I'm

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-15 Thread H. Peter Anvin
Andrew Morton wrote: On Mon, 13 Aug 2007 15:30:19 +0800 Huang, Ying [EMAIL PROTECTED] wrote: Following sets of patches add EFI/UEFI (Unified Extensible Firmware Interface) runtime services support to x86_64 architecture. OK, we have a major trainwreck when these patches meet Peter's

[PATCH 0/3] x86_64 EFI runtime service support

2007-08-13 Thread Huang, Ying
Following sets of patches add EFI/UEFI (Unified Extensible Firmware Interface) runtime services support to x86_64 architecture. The patches have been tested against 2.6.23-rc2 kernel on Intel platforms with EFI1.10 and UEFI2.0 firmware. This patch set is based on previous x86_64 EFI boot support

[PATCH 0/3] x86_64 EFI runtime service support

2007-08-13 Thread Huang, Ying
Following sets of patches add EFI/UEFI (Unified Extensible Firmware Interface) runtime services support to x86_64 architecture. The patches have been tested against 2.6.23-rc2 kernel on Intel platforms with EFI1.10 and UEFI2.0 firmware. This patch set is based on previous x86_64 EFI boot support