Hi, all Update my status: I have finished most of thecode related changes including the krealloc fixes (both for original code and my new code). And I'm slowly moving the kexec related stuff to efi_kexec.c, this involves some other cleanups, such as: efi_systab_init: -> move 64bit code to efi_64.c, move 32bit code to efi_32.c efi_setup global variable is changing to physical address, ioremapping it when necessary instead of directly using the virt addr because previously the virt addr will be freed until entering virtual mode, it's possible a leak in case something is wrong in the middle also it occupies the ioremap slot a little bit long. There's also other problems, so I still need a few days probably a week to carefully restructure the code. Thanks Dave Sending from web interface so sorry for the email format. ----- Original Message ----- From: "Borislav Petkov" <bp@xxxxxxxxx> To: "Dave Young" <dyoung at redhat.com> Cc: mjg59 at srcf.ucam.org, linux-efi at vger.kernel.org, "toshi kani" <toshi.kani at hp.com>, matt at console-pimps.org, greg at kroah.com, x86 at kernel.org, kexec at lists.infradead.org, linux-kernel at vger.kernel.org, "James Bottomley" <James.Bottomley at HansenPartnership.com>, horms at verge.net.au, ebiederm at xmission.com, hpa at zytor.com, vgoyal at redhat.com Sent: Monday, December 2, 2013 5:40:20 PM Subject: Re: [PATCH v4 06/12] efi: export efi runtime memory mapping to sysfs On Mon, Dec 02, 2013 at 10:59:42AM +0800, Dave Young wrote: > They are only called if there's setup_data SETUP_EFI passed in. Yes, I > think only kexec need that part of code. But I hesitated to mess the > code with a lot of #if. Will think about it. Well, the accepted strategy with the efi code is to have function calls in efi.c and then have different versions of those functions defined for 32 and 64-bit, i.e. efi_32.c and efi_64.c. I can imagine adding an efi_kexec.c could make the whole deal very clearly arranged without ifdeffery. -- Regards/Gruss, Boris. Sent from a fat crate under my desk. Formatting is fine. -- _______________________________________________ kexec mailing list kexec at lists.infradead.org http://lists.infradead.org/mailman/listinfo/kexec