Hello, 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. > 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. > 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? > 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? > 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. Thanks. -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
