Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755764Ab3HWRPJ (ORCPT ); Fri, 23 Aug 2013 13:15:09 -0400 Received: from g4t0014.houston.hp.com ([15.201.24.17]:39643 "EHLO g4t0014.houston.hp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755572Ab3HWRPH (ORCPT ); Fri, 23 Aug 2013 13:15:07 -0400 Message-ID: <1377278016.10300.814.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: Fri, 23 Aug 2013 11:13:36 -0600 In-Reply-To: <20130823162444.GL3277@htj.dyndns.org> References: <20130822033234.GA2413@htj.dyndns.org> <1377186729.10300.643.camel@misato.fc.hp.com> <20130822183130.GA3490@mtj.dyndns.org> <1377202292.10300.693.camel@misato.fc.hp.com> <20130822202158.GD3490@mtj.dyndns.org> <1377205598.10300.715.camel@misato.fc.hp.com> <20130822212111.GF3490@mtj.dyndns.org> <1377209861.10300.756.camel@misato.fc.hp.com> <20130823130440.GC10322@mtj.dyndns.org> <1377274448.10300.777.camel@misato.fc.hp.com> <20130823162444.GL3277@htj.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 Content-Length: 2100 Lines: 41 Hello, On Fri, 2013-08-23 at 12:24 -0400, Tejun Heo wrote: > On Fri, Aug 23, 2013 at 10:14:08AM -0600, Toshi Kani wrote: > > I still think acpi table info should be available earlier, but I do not > > think I can convince you on this. This can be religious debate. > > I'm curious. If there aren't substantial enough benefits, why would > you still want to pull it earlier when it brings in things like initrd > override and crafting the code carefully so that it's safe to execute > it from different address modes and so on? Please note that x86 is > not ia64. The early environment is completely different not only > technically but also in its diversity and suckiness. It wasn't too > long ago that vendors were screwing up ACPI left and right. It has > been getting better but there's a reason why, for example, we still > consider e820 to be the authoritative information over ACPI. Firmware generates tables, and provides them via some interface. Memory map table can be provided via e820 or EFI memory map. Memory topology table is provided via ACPI. I agree to prioritize one table over the other when there is overlap. But in the end, it is the firmware that generates the tables. Because it is provided via ACPI does not make it suddenly unreliable. I think table info from e820/EFI/ACPI should be available at the same time. To me, it makes more sense to use the hotplug info to initialize memblock than try to find a way to workaround without it. I think we will continue to be in that way to find a workaround in this direction. I came from ia64 background, and am not very familiar with x86. So, you may be very right about that x86 is different. I also agree that initrd is making it unnecessarily complicated. We may see some initial issues, but my hope is that the code gets matured over the time. Thanks, -Toshi -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/