lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <5214998F.200@cn.fujitsu.com>
Date:	Wed, 21 Aug 2013 18:42:23 +0800
From:	Tang Chen <tangchen@...fujitsu.com>
To:	konrad.wilk@...cle.com, robert.moore@...el.com, lv.zheng@...el.com,
	rjw@...k.pl, lenb@...nel.org, tglx@...utronix.de, mingo@...e.hu,
	hpa@...or.com, akpm@...ux-foundation.org, tj@...nel.org,
	trenn@...e.de, yinghai@...nel.org, jiang.liu@...wei.com,
	wency@...fujitsu.com, laijs@...fujitsu.com,
	isimatu.yasuaki@...fujitsu.com, izumi.taku@...fujitsu.com,
	mgorman@...e.de, minchan@...nel.org, mina86@...a86.com,
	gong.chen@...ux.intel.com, vasilis.liaskovitis@...fitbricks.com,
	lwoodman@...hat.com, riel@...hat.com, jweiner@...hat.com,
	prarit@...hat.com, zhangyanfei@...fujitsu.com,
	yanghy@...fujitsu.com, Toshi Kani <toshi.kani@...com>,
	"Luck, Tony" <tony.luck@...el.com>
CC:	x86@...nel.org, linux-doc@...r.kernel.org,
	linux-kernel@...r.kernel.org, linux-mm@...ck.org,
	linux-acpi@...r.kernel.org
Subject: Re: [PATCH 0/8] x86, acpi: Move acpi_initrd_override() earlier.

Hi all,

This patch-set has not been fully tested. I sent them first for you
to review. Please comment if we can agree on this solution.

Thanks.:)

On 08/21/2013 06:15 PM, Tang Chen wrote:
> This patch-set aims to move acpi_initrd_override() earlier on x86.
> Some of the patches are from Yinghai's patch-set:
> https://lkml.org/lkml/2013/6/14/561
>
> The difference between this patch-set and Yinghai's original patch-set are:
> 1. This patch-set doesn't split acpi_initrd_override(), but call it as a
>     whole operation at early time.
> 2. Allocate memory from BRK to store override tables.
>     (This idea is also from Yinghai.)
>
>
> [Current state]
>
> The current Linux kernel will initialize acpi tables like the following:
>
> 1. Find all acpi override table provided by users in initrd.
>     (Linux allows users to override acpi tables in firmware, by specifying
>     their own tables in initrd.)
>
> 2. Use acpica code to initialize acpi global root table list and install all
>     tables into it. If any override tables exists, use it to override the one
>     provided by firmware.
>
> Then others can parse these tables and get useful info.
>
> Both of the two steps happen after direct mapping page tables are setup.
>
> [Issues]
>
> In the current Linux kernel, the initialization of acpi tables is too late for
> new functionalities.
>
> We have some issues about this:
>
> * For memory hotplug, we need ACPI SRAT at early time to be aware of which memory
>    ranges are hotpluggable, and prevent bootmem allocator from allocating memory
>    for the kernel. (Kernel pages cannot be hotplugged because )
>
> * As suggested by Yinghai Lu<yinghai@...nel.org>, we should allocate page tables
>    in local node. This also needs SRAT before direct mapping page tables are setup.
>
> * As mentioned by Toshi Kani<toshi.kani@...com>, ACPI SCPR/DBGP/DBG2 tables
>    allow the OS to initialize serial console/debug ports at early boot time. The
>    earlier it can be initialized, the better this feature will be.  These tables
>    are not currently used by Linux due to a licensing issue, but it could be
>    addressed some time soon.
>
>
> [What are we doing]
>
> We are trying to initialize acip tables as early as possible. But Linux kernel
> allows users to override acpi tables by specifying their own tables in initrd.
> So we have to do acpi_initrd_override() earlier first.
>
>
> [About this patch-set]
>
> This patch-set aims to move acpi_initrd_override() as early as possible on x86.
> As suggested by Yinghai, we are trying to do it like this:
>
> On 32bit: do it in head_32.S, before paging is enabled. In this case, we can
>            access initrd with physical address without page tables.
>
> On 64bit: do it in head_64.c, after paging is enabled but before direct mapping
>            is setup.
>
> And also, acpi_initrd_override() needs to allocate memory for override tables.
> But at such an early time, there is no memory allocator works. So the basic idea
> from Yinghai is to use BRK. We will extend BRK 256KB in this patch-set.
>
>
> Tang Chen (6):
>    x86, acpi: Move table_sigs[] to stack.
>    x86, acpi, brk: Extend BRK 256KB to store acpi override tables.
>    x86, brk: Make extend_brk() available with va/pa.
>    x86, acpi: Make acpi_initrd_override() available with va or pa.
>    x86, acpi, brk: Make early_alloc_acpi_override_tables_buf() available
>      with va/pa.
>    x86, acpi: Do acpi_initrd_override() earlier in head_32.S/head64.c.
>
> Yinghai Lu (2):
>    x86: Make get_ramdisk_{image|size}() global.
>    x86, microcode: Use get_ramdisk_{image|size}() in microcode handling.
>
>   arch/x86/include/asm/dmi.h              |    2 +-
>   arch/x86/include/asm/setup.h            |   11 +++-
>   arch/x86/kernel/head64.c                |    4 +
>   arch/x86/kernel/head_32.S               |    4 +
>   arch/x86/kernel/microcode_intel_early.c |    8 +-
>   arch/x86/kernel/setup.c                 |   93 ++++++++++++++++------
>   arch/x86/mm/init.c                      |    2 +-
>   arch/x86/xen/enlighten.c                |    2 +-
>   arch/x86/xen/mmu.c                      |    6 +-
>   arch/x86/xen/p2m.c                      |   27 ++++---
>   drivers/acpi/osl.c                      |  130 ++++++++++++++++++++-----------
>   include/linux/acpi.h                    |    5 +-
>   12 files changed, 196 insertions(+), 98 deletions(-)
>
> --
> To unsubscribe, send a message with 'unsubscribe linux-mm' in
> the body to majordomo@...ck.org.  For more info on Linux MM,
> see: http://www.linux-mm.org/ .
> Don't email:<a href=mailto:"dont@...ck.org">  email@...ck.org</a>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ