From mboxrd@z Thu Jan 1 00:00:00 1970 From: Toshi Kani Subject: Re: [PATCH 0/8] x86, acpi: Move acpi_initrd_override() earlier. Date: Thu, 22 Aug 2013 14:11:32 -0600 Message-ID: <1377202292.10300.693.camel@misato.fc.hp.com> References: <20130821130647.GB19286@mtj.dyndns.org> <5214D60A.2090309@gmail.com> <20130821153639.GA17432@htj.dyndns.org> <1377113503.10300.492.camel@misato.fc.hp.com> <20130821195410.GA2436@htj.dyndns.org> <1377116968.10300.514.camel@misato.fc.hp.com> <20130821204041.GC2436@htj.dyndns.org> <1377124595.10300.594.camel@misato.fc.hp.com> <20130822033234.GA2413@htj.dyndns.org> <1377186729.10300.643.camel@misato.fc.hp.com> <20130822183130.GA3490@mtj.dyndns.org> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20130822183130.GA3490@mtj.dyndns.org> Sender: owner-linux-mm@kvack.org To: Tejun Heo Cc: Zhang Yanfei , Tang Chen , konrad.wilk@oracle.com, robert.moore@intel.com, lv.zheng@intel.com, rjw@sisk.pl, lenb@kernel.org, tglx@linutronix.de, mingo@elte.hu, hpa@zytor.com, akpm@linux-foundation.org, trenn@suse.de, yinghai@kernel.org, jiang.liu@huawei.com, wency@cn.fujitsu.com, laijs@cn.fujitsu.com, isimatu.yasuaki@jp.fujitsu.com, izumi.taku@jp.fujitsu.com, mgorman@suse.de, minchan@kernel.org, mina86@mina86.com, gong.chen@linux.intel.com, vasilis.liaskovitis@profitbricks.com, lwoodman@redhat.com, riel@redhat.com, jweiner@redhat.com, prarit@redhat.com, zhangyanfei@cn.fujitsu.com, yanghy@cn.fujitsu.com, x86@kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-acpi@vger.kernel.org List-Id: linux-acpi@vger.kernel.org Hello Tejun, On Thu, 2013-08-22 at 14:31 -0400, Tejun Heo wrote: > On Thu, Aug 22, 2013 at 09:52:09AM -0600, Toshi Kani wrote: > > I understand that you are concerned about stability of the ACPI stuff, > > which I think is a valid point, but most of (if not all) of the > > ACPI-related issues come from ACPI namespace/methods, which is a very > > different thing. Please do not mix up those two. The ACPI > > I have no objection to implementing self-conftained earlyprintk > support. If that's all you want to do, please go ahead but do not > pull in initrd override or ACPICA into it. If you are referring ACPICA as the AML interpreter, right, we do not move it up as I explained before. We are trying to move up the ACPI table init code (which is part of ACPICA, but has nothing to do with AML.) Note that ia64 also uses ACPI, and calls acpi_table_init() in setup_arch() before initializing the bootmap in find_memory(). > > namespace/methods stuff remains the same and continues to be initialized > > at very late in the boot sequence. > > > > What's making the patchset complicated is acpi_initrd_override(), which > > is intended for developers and allows overwriting ACPI bits at their own > > risk. This feature won't be used by regular users. > > Yeah, please forget about that in earlyboot. It doesn't make any > sense to fiddle with initrd that early during boot. I think the reason why Tang is working on this stuff again is that his previous change (which was once accepted) had broken initrd. So, he'd have to support it this time... > > If you are referring the issue of kernel image location, it is a > > limitation in the current implementation, not a technical limitation. I > > know other OS that supports movable memory and puts the kernel image > > into a movable memory with SRAT by changing the bootloader. > > I'm not saying that problem shouldn't be solved. I'm saying what you > guys are pushing doesn't help solving it at all. It's too late in the > boot process. It needs to be handled either by bootloader or earlier > kernel kexecing the actual one and super-early SRAT doens't help at > all in either case, so what's the point of pulling ACPI code in when > it doesn't contribute to solving the problem properly? It's too late for the kernel image itself, but it prevents allocating kernel memory from movable ranges after that. I'd say it solves a half of the issue this time. > > Also, how do you support local page tables without pursing SRAT early? > > Does it even matter with huge mappings? It's gonna be contained in a > single page anyway, right? Are the huge mappings always used? We cannot force user programs to use huge pages, can we? > > Initializing page tables on large systems may take a long time, and I do > > think that earlyprink needs to be available before that point. > > Yeah, sure, implement it in *minimal* way which doesn't affect > anything if not explicitly enabled by kernel param like other > earlyprintks. It doens't make any sense to add dependency to acpi > from early boot for that. It makes sense because it needs to obtain the config info from ACPI tables. As for the maintainability, I am far more concerned with your suggestion of having a separate page table init code when SRAT is used. This kind of divergence is a recipe of breakage. Thanks, -Toshi -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753011Ab3HVUND (ORCPT ); Thu, 22 Aug 2013 16:13:03 -0400 Received: from g4t0016.houston.hp.com ([15.201.24.19]:18090 "EHLO g4t0016.houston.hp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752053Ab3HVUNB (ORCPT ); Thu, 22 Aug 2013 16:13:01 -0400 Message-ID: <1377202292.10300.693.camel@misato.fc.hp.com> Subject: Re: [PATCH 0/8] x86, acpi: Move acpi_initrd_override() earlier. From: Toshi Kani To: Tejun Heo Cc: Zhang Yanfei , Tang Chen , konrad.wilk@oracle.com, robert.moore@intel.com, lv.zheng@intel.com, rjw@sisk.pl, lenb@kernel.org, tglx@linutronix.de, mingo@elte.hu, hpa@zytor.com, akpm@linux-foundation.org, trenn@suse.de, yinghai@kernel.org, jiang.liu@huawei.com, wency@cn.fujitsu.com, laijs@cn.fujitsu.com, isimatu.yasuaki@jp.fujitsu.com, izumi.taku@jp.fujitsu.com, mgorman@suse.de, minchan@kernel.org, mina86@mina86.com, gong.chen@linux.intel.com, vasilis.liaskovitis@profitbricks.com, lwoodman@redhat.com, riel@redhat.com, jweiner@redhat.com, prarit@redhat.com, zhangyanfei@cn.fujitsu.com, yanghy@cn.fujitsu.com, x86@kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-acpi@vger.kernel.org Date: Thu, 22 Aug 2013 14:11:32 -0600 In-Reply-To: <20130822183130.GA3490@mtj.dyndns.org> References: <20130821130647.GB19286@mtj.dyndns.org> <5214D60A.2090309@gmail.com> <20130821153639.GA17432@htj.dyndns.org> <1377113503.10300.492.camel@misato.fc.hp.com> <20130821195410.GA2436@htj.dyndns.org> <1377116968.10300.514.camel@misato.fc.hp.com> <20130821204041.GC2436@htj.dyndns.org> <1377124595.10300.594.camel@misato.fc.hp.com> <20130822033234.GA2413@htj.dyndns.org> <1377186729.10300.643.camel@misato.fc.hp.com> <20130822183130.GA3490@mtj.dyndns.org> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.6.4 (3.6.4-3.fc18) Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello Tejun, On Thu, 2013-08-22 at 14:31 -0400, Tejun Heo wrote: > On Thu, Aug 22, 2013 at 09:52:09AM -0600, Toshi Kani wrote: > > I understand that you are concerned about stability of the ACPI stuff, > > which I think is a valid point, but most of (if not all) of the > > ACPI-related issues come from ACPI namespace/methods, which is a very > > different thing. Please do not mix up those two. The ACPI > > I have no objection to implementing self-conftained earlyprintk > support. If that's all you want to do, please go ahead but do not > pull in initrd override or ACPICA into it. If you are referring ACPICA as the AML interpreter, right, we do not move it up as I explained before. We are trying to move up the ACPI table init code (which is part of ACPICA, but has nothing to do with AML.) Note that ia64 also uses ACPI, and calls acpi_table_init() in setup_arch() before initializing the bootmap in find_memory(). > > namespace/methods stuff remains the same and continues to be initialized > > at very late in the boot sequence. > > > > What's making the patchset complicated is acpi_initrd_override(), which > > is intended for developers and allows overwriting ACPI bits at their own > > risk. This feature won't be used by regular users. > > Yeah, please forget about that in earlyboot. It doesn't make any > sense to fiddle with initrd that early during boot. I think the reason why Tang is working on this stuff again is that his previous change (which was once accepted) had broken initrd. So, he'd have to support it this time... > > If you are referring the issue of kernel image location, it is a > > limitation in the current implementation, not a technical limitation. I > > know other OS that supports movable memory and puts the kernel image > > into a movable memory with SRAT by changing the bootloader. > > I'm not saying that problem shouldn't be solved. I'm saying what you > guys are pushing doesn't help solving it at all. It's too late in the > boot process. It needs to be handled either by bootloader or earlier > kernel kexecing the actual one and super-early SRAT doens't help at > all in either case, so what's the point of pulling ACPI code in when > it doesn't contribute to solving the problem properly? It's too late for the kernel image itself, but it prevents allocating kernel memory from movable ranges after that. I'd say it solves a half of the issue this time. > > Also, how do you support local page tables without pursing SRAT early? > > Does it even matter with huge mappings? It's gonna be contained in a > single page anyway, right? Are the huge mappings always used? We cannot force user programs to use huge pages, can we? > > Initializing page tables on large systems may take a long time, and I do > > think that earlyprink needs to be available before that point. > > Yeah, sure, implement it in *minimal* way which doesn't affect > anything if not explicitly enabled by kernel param like other > earlyprintks. It doens't make any sense to add dependency to acpi > from early boot for that. It makes sense because it needs to obtain the config info from ACPI tables. As for the maintainability, I am far more concerned with your suggestion of having a separate page table init code when SRAT is used. This kind of divergence is a recipe of breakage. Thanks, -Toshi