Re: [PATCH v5 10/14] efi: only print saved efi runtime maps instead of all memmap ranges for kexec

2013-12-17 Thread Dave Young
On 12/17/13 at 04:58pm, Borislav Petkov wrote: > On Tue, Dec 17, 2013 at 02:34:36PM +0800, Dave Young wrote: > > They are moved to efi.c efi_setup_init(), I'm not sure if I expained > > clear enough, in current code parse_efi_setup only accept one argument > > phys_addr so I will mapping it with

Re: [PATCH v5 10/14] efi: only print saved efi runtime maps instead of all memmap ranges for kexec

2013-12-17 Thread Borislav Petkov
On Tue, Dec 17, 2013 at 02:34:36PM +0800, Dave Young wrote: > They are moved to efi.c efi_setup_init(), I'm not sure if I expained > clear enough, in current code parse_efi_setup only accept one argument > phys_addr so I will mapping it with sizeof(struct setup_data) to > get the payload size then

Re: [PATCH v5 10/14] efi: only print saved efi runtime maps instead of all memmap ranges for kexec

2013-12-17 Thread Borislav Petkov
On Tue, Dec 17, 2013 at 02:34:36PM +0800, Dave Young wrote: They are moved to efi.c efi_setup_init(), I'm not sure if I expained clear enough, in current code parse_efi_setup only accept one argument phys_addr so I will mapping it with sizeof(struct setup_data) to get the payload size then get

Re: [PATCH v5 10/14] efi: only print saved efi runtime maps instead of all memmap ranges for kexec

2013-12-17 Thread Dave Young
On 12/17/13 at 04:58pm, Borislav Petkov wrote: On Tue, Dec 17, 2013 at 02:34:36PM +0800, Dave Young wrote: They are moved to efi.c efi_setup_init(), I'm not sure if I expained clear enough, in current code parse_efi_setup only accept one argument phys_addr so I will mapping it with

Re: [PATCH v5 10/14] efi: only print saved efi runtime maps instead of all memmap ranges for kexec

2013-12-16 Thread Dave Young
On 12/16/13 at 12:33pm, Borislav Petkov wrote: > On Mon, Dec 16, 2013 at 10:00:10AM +0800, Dave Young wrote: > > > > - print_efi_memmap(); > > > > + if (efi_setup) { > > > > + int s; > > > > + struct efi_setup_data *data; > > > > + > > > > + s

Re: [PATCH v5 10/14] efi: only print saved efi runtime maps instead of all memmap ranges for kexec

2013-12-16 Thread Borislav Petkov
On Mon, Dec 16, 2013 at 10:00:10AM +0800, Dave Young wrote: > > > - print_efi_memmap(); > > > + if (efi_setup) { > > > + int s; > > > + struct efi_setup_data *data; > > > + > > > + s = sizeof(*data) + nr_efi_runtime_map * sizeof(data->map[0]); > > > + data =

Re: [PATCH v5 10/14] efi: only print saved efi runtime maps instead of all memmap ranges for kexec

2013-12-16 Thread Borislav Petkov
On Mon, Dec 16, 2013 at 10:00:10AM +0800, Dave Young wrote: - print_efi_memmap(); + if (efi_setup) { + int s; + struct efi_setup_data *data; + + s = sizeof(*data) + nr_efi_runtime_map * sizeof(data-map[0]); + data = early_memremap(efi_setup, s);

Re: [PATCH v5 10/14] efi: only print saved efi runtime maps instead of all memmap ranges for kexec

2013-12-16 Thread Dave Young
On 12/16/13 at 12:33pm, Borislav Petkov wrote: On Mon, Dec 16, 2013 at 10:00:10AM +0800, Dave Young wrote: - print_efi_memmap(); + if (efi_setup) { + int s; + struct efi_setup_data *data; + + s = sizeof(*data) +

Re: [PATCH v5 10/14] efi: only print saved efi runtime maps instead of all memmap ranges for kexec

2013-12-15 Thread Dave Young
> > - print_efi_memmap(); > > + if (efi_setup) { > > + int s; > > + struct efi_setup_data *data; > > + > > + s = sizeof(*data) + nr_efi_runtime_map * sizeof(data->map[0]); > > + data = early_memremap(efi_setup, s); > > + if (!data) > > +

Re: [PATCH v5 10/14] efi: only print saved efi runtime maps instead of all memmap ranges for kexec

2013-12-15 Thread Dave Young
- print_efi_memmap(); + if (efi_setup) { + int s; + struct efi_setup_data *data; + + s = sizeof(*data) + nr_efi_runtime_map * sizeof(data-map[0]); + data = early_memremap(efi_setup, s); + if (!data) + return;

Re: [PATCH v5 10/14] efi: only print saved efi runtime maps instead of all memmap ranges for kexec

2013-12-13 Thread Borislav Petkov
On Mon, Dec 09, 2013 at 05:42:23PM +0800, Dave Young wrote: > For kexec/kdump kernel efi runtime mappings are saved, printing original whole > memmap ranges does not make sense anymore. So introduce a new function to only > print runtime maps in case kexec/kdump kernel is used. > > changelog: >

Re: [PATCH v5 10/14] efi: only print saved efi runtime maps instead of all memmap ranges for kexec

2013-12-13 Thread Borislav Petkov
On Mon, Dec 09, 2013 at 05:42:23PM +0800, Dave Young wrote: For kexec/kdump kernel efi runtime mappings are saved, printing original whole memmap ranges does not make sense anymore. So introduce a new function to only print runtime maps in case kexec/kdump kernel is used. changelog: Matt:

[PATCH v5 10/14] efi: only print saved efi runtime maps instead of all memmap ranges for kexec

2013-12-09 Thread Dave Young
For kexec/kdump kernel efi runtime mappings are saved, printing original whole memmap ranges does not make sense anymore. So introduce a new function to only print runtime maps in case kexec/kdump kernel is used. changelog: Matt: use efi_setup instead of esdata share function

[PATCH v5 10/14] efi: only print saved efi runtime maps instead of all memmap ranges for kexec

2013-12-09 Thread Dave Young
For kexec/kdump kernel efi runtime mappings are saved, printing original whole memmap ranges does not make sense anymore. So introduce a new function to only print runtime maps in case kexec/kdump kernel is used. changelog: Matt: use efi_setup instead of esdata share function