From: Yinghai Lu <yinghai@kernel.org> To: Matt Fleming <matt.fleming@intel.com>, Thomas Gleixner <tglx@linutronix.de>, "H. Peter Anvin" <hpa@zytor.com>, Ingo Molnar <mingo@redhat.com>, Jiri Kosina <jkosina@suse.cz>, Borislav Petkov <bp@suse.de>, Bjorn Helgaas <bhelgaas@google.com> Cc: linux-kernel@vger.kernel.org, linux-efi@vger.kernel.org, linux-pci@vger.kernel.org, Yinghai Lu <yinghai@kernel.org> Subject: [PATCH 0/8] x86, boot: clean up setup_data handling Date: Sat, 28 Feb 2015 18:17:31 -0800 [thread overview] Message-ID: <1425176259-30087-1-git-send-email-yinghai@kernel.org> (raw) Now we setup_data is reserved via memblock and e820 and different handlers have different ways, and it is confusing. 1. SETUP_E820_EXT: is consumed early and will not copy or access again. have memory wasted. 2. SETUP_EFI: is accessed via ioremap every time at early stage. have memory wasted. 3. SETUP_DTB: is copied locally. have memory wasted. 4. SETUP_PCI: is accessed via ioremap for every pci devices, even run-time. 5. SETUP_KASLR: is accessed early, will not copy or access again. have memory wasted. Also setup_data is exported to debugfs for debug purpose. Here will convert to let every handler to decide how to handle it. and will not reserve the setup_data generally, so will not waste memory and also make memblock/e820 keep page aligned. 1. not touch E820 anymore. 2. copy SETUP_EFI to __initdata variable and access it without ioremap. 3. SETUP_DTB: reserver and copy to local and free. 4. SETUP_PCI: reverve localy and convert to list, to avoid keeping ioremap. 5. SETUP_KASLR: fix accessing kaslr_enabled accessing... 6. export SETUP_PCI via sysfs. Those patches could be applied on top of tip/x86/urgent with SETUP_KASLR support. Yinghai Lu (8): x86, kaslr: get kaslr_enabled back correctly x86: Kill E820_RESERVED_KERN x86, efi: copy SETUP_EFI data and access directly x86, of: let add_dtb reserve by itself x86, boot: Add add_pci handler for SETUP_PCI x86: kill not used setup_data handling code x86, pci: convert SETUP_PCI data to list x86, pci: export SETUP_PCI data via sysfs arch/x86/include/asm/efi.h | 2 +- arch/x86/include/asm/pci.h | 2 + arch/x86/include/asm/prom.h | 9 +- arch/x86/include/uapi/asm/e820.h | 9 -- arch/x86/kernel/devicetree.c | 39 +++--- arch/x86/kernel/e820.c | 6 +- arch/x86/kernel/kdebugfs.c | 142 -------------------- arch/x86/kernel/setup.c | 57 ++------ arch/x86/kernel/tboot.c | 3 +- arch/x86/mm/init_64.c | 11 +- arch/x86/pci/common.c | 281 ++++++++++++++++++++++++++++++++++++--- arch/x86/platform/efi/efi.c | 13 +- arch/x86/platform/efi/efi_64.c | 13 +- arch/x86/platform/efi/quirks.c | 23 +--- 14 files changed, 336 insertions(+), 274 deletions(-) -- 1.8.4.5
WARNING: multiple messages have this Message-ID (diff)
From: Yinghai Lu <yinghai-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> To: Matt Fleming <matt.fleming-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>, Thomas Gleixner <tglx-hfZtesqFncYOwBW4kG4KsQ@public.gmane.org>, "H. Peter Anvin" <hpa-YMNOUZJC4hwAvxtiuMwx3w@public.gmane.org>, Ingo Molnar <mingo-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>, Jiri Kosina <jkosina-AlSwsSmVLrQ@public.gmane.org>, Borislav Petkov <bp-l3A5Bk7waGM@public.gmane.org>, Bjorn Helgaas <bhelgaas-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org> Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Yinghai Lu <yinghai-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> Subject: [PATCH 0/8] x86, boot: clean up setup_data handling Date: Sat, 28 Feb 2015 18:17:31 -0800 [thread overview] Message-ID: <1425176259-30087-1-git-send-email-yinghai@kernel.org> (raw) Now we setup_data is reserved via memblock and e820 and different handlers have different ways, and it is confusing. 1. SETUP_E820_EXT: is consumed early and will not copy or access again. have memory wasted. 2. SETUP_EFI: is accessed via ioremap every time at early stage. have memory wasted. 3. SETUP_DTB: is copied locally. have memory wasted. 4. SETUP_PCI: is accessed via ioremap for every pci devices, even run-time. 5. SETUP_KASLR: is accessed early, will not copy or access again. have memory wasted. Also setup_data is exported to debugfs for debug purpose. Here will convert to let every handler to decide how to handle it. and will not reserve the setup_data generally, so will not waste memory and also make memblock/e820 keep page aligned. 1. not touch E820 anymore. 2. copy SETUP_EFI to __initdata variable and access it without ioremap. 3. SETUP_DTB: reserver and copy to local and free. 4. SETUP_PCI: reverve localy and convert to list, to avoid keeping ioremap. 5. SETUP_KASLR: fix accessing kaslr_enabled accessing... 6. export SETUP_PCI via sysfs. Those patches could be applied on top of tip/x86/urgent with SETUP_KASLR support. Yinghai Lu (8): x86, kaslr: get kaslr_enabled back correctly x86: Kill E820_RESERVED_KERN x86, efi: copy SETUP_EFI data and access directly x86, of: let add_dtb reserve by itself x86, boot: Add add_pci handler for SETUP_PCI x86: kill not used setup_data handling code x86, pci: convert SETUP_PCI data to list x86, pci: export SETUP_PCI data via sysfs arch/x86/include/asm/efi.h | 2 +- arch/x86/include/asm/pci.h | 2 + arch/x86/include/asm/prom.h | 9 +- arch/x86/include/uapi/asm/e820.h | 9 -- arch/x86/kernel/devicetree.c | 39 +++--- arch/x86/kernel/e820.c | 6 +- arch/x86/kernel/kdebugfs.c | 142 -------------------- arch/x86/kernel/setup.c | 57 ++------ arch/x86/kernel/tboot.c | 3 +- arch/x86/mm/init_64.c | 11 +- arch/x86/pci/common.c | 281 ++++++++++++++++++++++++++++++++++++--- arch/x86/platform/efi/efi.c | 13 +- arch/x86/platform/efi/efi_64.c | 13 +- arch/x86/platform/efi/quirks.c | 23 +--- 14 files changed, 336 insertions(+), 274 deletions(-) -- 1.8.4.5
next reply other threads:[~2015-03-01 2:18 UTC|newest] Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-03-01 2:17 Yinghai Lu [this message] 2015-03-01 2:17 ` [PATCH 0/8] x86, boot: clean up setup_data handling Yinghai Lu 2015-03-01 2:17 ` [PATCH 1/8] x86, kaslr: get kaslr_enabled back correctly Yinghai Lu 2015-03-01 2:40 ` Yinghai Lu 2015-03-01 2:40 ` Yinghai Lu 2015-03-01 9:45 ` Borislav Petkov 2015-03-01 15:23 ` Ingo Molnar 2015-03-01 15:23 ` Ingo Molnar 2015-03-01 19:27 ` Yinghai Lu 2015-03-01 19:49 ` Borislav Petkov 2015-03-01 20:24 ` Yinghai Lu 2015-03-01 20:24 ` Yinghai Lu 2015-03-01 20:29 ` Borislav Petkov 2015-03-01 20:29 ` Borislav Petkov 2015-03-01 20:41 ` Yinghai Lu 2015-03-02 8:56 ` Borislav Petkov 2015-03-02 8:56 ` Borislav Petkov 2015-03-02 11:04 ` Yinghai Lu 2015-03-02 11:04 ` Yinghai Lu 2015-03-02 14:53 ` Borislav Petkov 2015-03-02 18:58 ` Yinghai Lu 2015-03-02 20:25 ` Borislav Petkov 2015-03-02 22:10 ` Yinghai Lu 2015-03-02 22:10 ` Yinghai Lu 2015-03-01 19:39 ` Borislav Petkov 2015-03-02 9:39 ` Jiri Kosina 2015-03-02 13:13 ` Matt Fleming 2015-03-02 13:13 ` Matt Fleming 2015-03-01 2:17 ` [PATCH 2/8] x86: Kill E820_RESERVED_KERN Yinghai Lu 2015-03-01 2:17 ` [PATCH 3/8] x86, efi: copy SETUP_EFI data and access directly Yinghai Lu 2015-03-01 2:17 ` [PATCH 4/8] x86, of: let add_dtb reserve by itself Yinghai Lu 2015-03-01 2:17 ` [PATCH 5/8] x86, boot: Add add_pci handler for SETUP_PCI Yinghai Lu 2015-03-01 2:17 ` [PATCH 6/8] x86: kill not used setup_data handling code Yinghai Lu 2015-03-01 2:17 ` Yinghai Lu 2015-03-01 2:17 ` [PATCH 7/8] x86, pci: convert SETUP_PCI data to list Yinghai Lu 2015-03-01 2:17 ` [PATCH 8/8] x86, pci: export SETUP_PCI data via sysfs Yinghai Lu
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=1425176259-30087-1-git-send-email-yinghai@kernel.org \ --to=yinghai@kernel.org \ --cc=bhelgaas@google.com \ --cc=bp@suse.de \ --cc=hpa@zytor.com \ --cc=jkosina@suse.cz \ --cc=linux-efi@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-pci@vger.kernel.org \ --cc=matt.fleming@intel.com \ --cc=mingo@redhat.com \ --cc=tglx@linutronix.de \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.