From: Tejun Heo <tj@kernel.org> To: Toshi Kani <toshi.kani@hp.com> Cc: Zhang Yanfei <zhangyanfei.yes@gmail.com>, Tang Chen <tangchen@cn.fujitsu.com>, 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 Subject: Re: [PATCH 0/8] x86, acpi: Move acpi_initrd_override() earlier. Date: Thu, 22 Aug 2013 17:21:11 -0400 [thread overview] Message-ID: <20130822212111.GF3490@mtj.dyndns.org> (raw) In-Reply-To: <1377205598.10300.715.camel@misato.fc.hp.com> Hello, Toshi. On Thu, Aug 22, 2013 at 03:06:38PM -0600, Toshi Kani wrote: > Since some node(s) won't be ejectable, this solution is reasonable as > the first step. I do not think it is a distraction. I view your But does this contribute to reaching the next step? If so, how? I can't see how and that's why I said this was a distraction. > suggestion as a distraction of supporting local page tables, though. Hmmm... > Local page table and memory hotplug are two separate things. That is, > local page tables can be supported on all NUMA platforms without hotplug > support. Are you sure huge mapping will solve everything for all types > of applications, and therefore local page tables won't be needed at all? When you throw around terms like "all" and "at all", you can't reach rational discussion about engineering trade-offs. I was asking you whether it was reasonable to do per-node page table when most machines support huge page mappings which makes the whole thing rather pointless. Of course there will be some niche cases where this might not be optimal but do you think that would be enough to justify the added complexity and churn? If you think so, can you please elaborate? > When someone changes the page table init code, who will test it with the > special allocation code? What are you worrying about? Are you saying that allocating page table towards top or bottom of memory would be more disruptive and difficult to debug than pulling in ACPI init and SRAT information into the process? Am I missing something here? Thanks. -- tejun -- 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: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
WARNING: multiple messages have this Message-ID (diff)
From: Tejun Heo <tj@kernel.org> To: Toshi Kani <toshi.kani@hp.com> Cc: Zhang Yanfei <zhangyanfei.yes@gmail.com>, Tang Chen <tangchen@cn.fujitsu.com>, 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 Subject: Re: [PATCH 0/8] x86, acpi: Move acpi_initrd_override() earlier. Date: Thu, 22 Aug 2013 17:21:11 -0400 [thread overview] Message-ID: <20130822212111.GF3490@mtj.dyndns.org> (raw) In-Reply-To: <1377205598.10300.715.camel@misato.fc.hp.com> Hello, Toshi. On Thu, Aug 22, 2013 at 03:06:38PM -0600, Toshi Kani wrote: > Since some node(s) won't be ejectable, this solution is reasonable as > the first step. I do not think it is a distraction. I view your But does this contribute to reaching the next step? If so, how? I can't see how and that's why I said this was a distraction. > suggestion as a distraction of supporting local page tables, though. Hmmm... > Local page table and memory hotplug are two separate things. That is, > local page tables can be supported on all NUMA platforms without hotplug > support. Are you sure huge mapping will solve everything for all types > of applications, and therefore local page tables won't be needed at all? When you throw around terms like "all" and "at all", you can't reach rational discussion about engineering trade-offs. I was asking you whether it was reasonable to do per-node page table when most machines support huge page mappings which makes the whole thing rather pointless. Of course there will be some niche cases where this might not be optimal but do you think that would be enough to justify the added complexity and churn? If you think so, can you please elaborate? > When someone changes the page table init code, who will test it with the > special allocation code? What are you worrying about? Are you saying that allocating page table towards top or bottom of memory would be more disruptive and difficult to debug than pulling in ACPI init and SRAT information into the process? Am I missing something here? Thanks. -- tejun
next prev parent reply other threads:[~2013-08-22 21:21 UTC|newest] Thread overview: 116+ messages / expand[flat|nested] mbox.gz Atom feed top 2013-08-21 10:15 [PATCH 0/8] x86, acpi: Move acpi_initrd_override() earlier Tang Chen 2013-08-21 10:15 ` Tang Chen 2013-08-21 10:15 ` [PATCH 1/8] x86: Make get_ramdisk_{image|size}() global Tang Chen 2013-08-21 10:15 ` Tang Chen 2013-08-21 10:15 ` [PATCH 2/8] x86, microcode: Use get_ramdisk_{image|size}() in microcode handling Tang Chen 2013-08-21 10:15 ` Tang Chen 2013-08-21 10:15 ` [PATCH 3/8] x86, acpi: Move table_sigs[] to stack Tang Chen 2013-08-21 10:15 ` Tang Chen 2013-08-21 10:15 ` [PATCH 4/8] x86, acpi, brk: Extend BRK 256KB to store acpi override tables Tang Chen 2013-08-21 10:15 ` Tang Chen 2013-08-21 10:15 ` [PATCH 5/8] x86, brk: Make extend_brk() available with va/pa Tang Chen 2013-08-21 10:15 ` Tang Chen 2013-08-21 12:26 ` Konrad Rzeszutek Wilk 2013-08-21 12:26 ` Konrad Rzeszutek Wilk 2013-08-21 12:35 ` H. Peter Anvin 2013-08-21 12:35 ` H. Peter Anvin 2013-08-21 14:42 ` Konrad Rzeszutek Wilk 2013-08-21 14:42 ` Konrad Rzeszutek Wilk 2013-08-21 15:04 ` H. Peter Anvin 2013-08-21 15:04 ` H. Peter Anvin 2013-08-21 10:15 ` [PATCH 6/8] x86, acpi: Make acpi_initrd_override() available with va or pa Tang Chen 2013-08-21 10:15 ` Tang Chen 2013-08-21 10:15 ` [PATCH 7/8] x86, acpi, brk: Make early_alloc_acpi_override_tables_buf() available with va/pa Tang Chen 2013-08-21 10:15 ` Tang Chen 2013-08-21 10:15 ` [PATCH 8/8] x86, acpi: Do acpi_initrd_override() earlier in head_32.S/head64.c Tang Chen 2013-08-21 10:15 ` Tang Chen 2013-08-21 10:42 ` [PATCH 0/8] x86, acpi: Move acpi_initrd_override() earlier Tang Chen 2013-08-21 10:42 ` Tang Chen 2013-08-21 13:06 ` Tejun Heo 2013-08-21 13:06 ` Tejun Heo 2013-08-21 15:00 ` Zhang Yanfei 2013-08-21 15:00 ` Zhang Yanfei 2013-08-21 15:36 ` Tejun Heo 2013-08-21 15:36 ` Tejun Heo 2013-08-21 19:31 ` Toshi Kani 2013-08-21 19:31 ` Toshi Kani 2013-08-21 19:54 ` Tejun Heo 2013-08-21 19:54 ` Tejun Heo 2013-08-21 20:29 ` Toshi Kani 2013-08-21 20:29 ` Toshi Kani 2013-08-21 20:40 ` Tejun Heo 2013-08-21 20:40 ` Tejun Heo 2013-08-21 22:36 ` Toshi Kani 2013-08-21 22:36 ` Toshi Kani 2013-08-22 3:32 ` Tejun Heo 2013-08-22 3:32 ` Tejun Heo 2013-08-22 15:52 ` Toshi Kani 2013-08-22 15:52 ` Toshi Kani 2013-08-22 18:31 ` Tejun Heo 2013-08-22 18:31 ` Tejun Heo 2013-08-22 19:39 ` Zhang Yanfei 2013-08-22 19:39 ` Zhang Yanfei 2013-08-22 19:45 ` Tejun Heo 2013-08-22 19:45 ` Tejun Heo 2013-08-22 20:11 ` Toshi Kani 2013-08-22 20:11 ` Toshi Kani 2013-08-22 20:21 ` Tejun Heo 2013-08-22 20:21 ` Tejun Heo 2013-08-22 20:35 ` Tejun Heo 2013-08-22 20:35 ` Tejun Heo 2013-08-22 21:06 ` Toshi Kani 2013-08-22 21:06 ` Toshi Kani 2013-08-22 21:21 ` Tejun Heo [this message] 2013-08-22 21:21 ` Tejun Heo 2013-08-22 22:17 ` Toshi Kani 2013-08-22 22:17 ` Toshi Kani 2013-08-23 13:04 ` Tejun Heo 2013-08-23 13:04 ` Tejun Heo 2013-08-23 13:08 ` H. Peter Anvin 2013-08-23 13:08 ` H. Peter Anvin 2013-08-23 14:19 ` Tejun Heo 2013-08-23 14:19 ` Tejun Heo 2013-08-23 14:24 ` H. Peter Anvin 2013-08-23 14:24 ` H. Peter Anvin 2013-08-23 14:24 ` H. Peter Anvin 2013-08-23 14:35 ` Tejun Heo 2013-08-23 14:35 ` Tejun Heo 2013-08-23 14:57 ` Tejun Heo 2013-08-23 14:57 ` Tejun Heo 2013-08-23 16:14 ` Toshi Kani 2013-08-23 16:14 ` Toshi Kani 2013-08-23 16:24 ` Tejun Heo 2013-08-23 16:24 ` Tejun Heo 2013-08-23 17:13 ` Toshi Kani 2013-08-23 17:13 ` Toshi Kani 2013-08-23 17:29 ` Zhang Yanfei 2013-08-23 17:29 ` Zhang Yanfei 2013-08-23 16:54 ` Zhang Yanfei 2013-08-23 16:54 ` Zhang Yanfei 2013-08-23 18:18 ` Yinghai Lu 2013-08-23 18:18 ` Yinghai Lu 2013-08-23 18:25 ` H. Peter Anvin 2013-08-23 20:08 ` Yinghai Lu 2013-08-23 20:30 ` Russ Anderson 2013-08-23 20:48 ` Yinghai Lu 2013-08-23 21:50 ` chen tang 2013-08-23 21:52 ` Moore, Robert 2013-08-23 22:05 ` Yinghai Lu 2013-08-23 22:08 ` Yinghai Lu 2013-08-23 22:40 ` chen tang 2013-08-23 23:04 ` Yinghai Lu 2013-08-24 2:41 ` Russ Anderson 2013-08-23 20:33 ` chen tang 2013-08-23 20:33 ` chen tang 2013-08-23 21:08 ` Yinghai Lu 2013-08-23 21:08 ` Yinghai Lu 2013-08-23 22:27 ` chen tang 2013-08-23 22:27 ` chen tang 2013-08-23 18:29 ` Toshi Kani 2013-08-23 18:29 ` Toshi Kani 2013-08-23 21:37 ` chen tang 2013-08-23 21:37 ` chen tang 2013-08-23 21:52 ` Tejun Heo 2013-08-23 21:52 ` Tejun Heo 2013-08-23 23:56 ` chen tang 2013-08-23 23:56 ` chen tang
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=20130822212111.GF3490@mtj.dyndns.org \ --to=tj@kernel.org \ --cc=akpm@linux-foundation.org \ --cc=gong.chen@linux.intel.com \ --cc=hpa@zytor.com \ --cc=isimatu.yasuaki@jp.fujitsu.com \ --cc=izumi.taku@jp.fujitsu.com \ --cc=jiang.liu@huawei.com \ --cc=jweiner@redhat.com \ --cc=konrad.wilk@oracle.com \ --cc=laijs@cn.fujitsu.com \ --cc=lenb@kernel.org \ --cc=linux-acpi@vger.kernel.org \ --cc=linux-doc@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mm@kvack.org \ --cc=lv.zheng@intel.com \ --cc=lwoodman@redhat.com \ --cc=mgorman@suse.de \ --cc=mina86@mina86.com \ --cc=minchan@kernel.org \ --cc=mingo@elte.hu \ --cc=prarit@redhat.com \ --cc=riel@redhat.com \ --cc=rjw@sisk.pl \ --cc=robert.moore@intel.com \ --cc=tangchen@cn.fujitsu.com \ --cc=tglx@linutronix.de \ --cc=toshi.kani@hp.com \ --cc=trenn@suse.de \ --cc=vasilis.liaskovitis@profitbricks.com \ --cc=wency@cn.fujitsu.com \ --cc=x86@kernel.org \ --cc=yanghy@cn.fujitsu.com \ --cc=yinghai@kernel.org \ --cc=zhangyanfei.yes@gmail.com \ --cc=zhangyanfei@cn.fujitsu.com \ /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.