All of lore.kernel.org
 help / color / mirror / Atom feed
From: "H. Peter Anvin" <hpa@zytor.com>
To: KOSAKI Motohiro <kosaki.motohiro@gmail.com>, Tejun Heo <tj@kernel.org>
Cc: Tang Chen <imtangchen@gmail.com>,
	Tang Chen <tangchen@cn.fujitsu.com>,
	robert.moore@intel.com, lv.zheng@intel.com, rjw@sisk.pl,
	lenb@kernel.org, tglx@linutronix.de, mingo@elte.hu,
	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 part5 0/7] Arrange hotpluggable memory as ZONE_MOVABLE.
Date: Wed, 14 Aug 2013 13:30:38 -0700	[thread overview]
Message-ID: <5a1b9edd-8232-498a-b94a-72e028772970@email.android.com> (raw)
In-Reply-To: <520BE891.8090004@gmail.com>

There are systems which can.  They have the ability to remap in hardware.

KOSAKI Motohiro <kosaki.motohiro@gmail.com> wrote:
>(8/14/13 3:55 PM), Tejun Heo wrote:
>> Hello,
>>
>> On Wed, Aug 14, 2013 at 03:40:31PM -0400, KOSAKI Motohiro wrote:
>>> I don't agree it. Please look at other kernel options. A lot of
>these don't
>>> follow you. These behave as direction, not advise.
>>>
>>> I mean the fallback should be implemented at turning on default the
>feature.
>>
>> Yeah, some options are "please try this" and others "do this or
>fail".
>> There's no frigging fundamental rule there.
>
>In this case, we have zero worth for fallback, right?
>
>
>>> I don't read whole discussion and I don't quite understand why no
>kernel
>>> place controlling is relevant. Every unpluggable node is suitable
>for
>>> kernel. If you mean current kernel placement logic don't care
>plugging,
>>> that's a bug.
>>>
>>> If we aim to hot remove, we have to have either kernel relocation or
>>> hotplug awre kernel placement at boot time.
>>
>> What if all nodes are hot pluggable?  Are we moving the kernel
>> dynamically then?
>
>Intel folks already told, we have no such system in practice.
>
>
>>>> Failing to boot is *way* worse reporting mechanism than almost
>>>> everything else.  If the sysadmin is willing to risk machines
>failing
>>>> to come up, she would definitely be willing to check whether which
>>>> memory areas are actually hotpluggable too, right?
>>>
>>> No. see above. Your opinion is not pragmatic useful.
>>
>> No, what you're saying doesn't make any sense.  There are multiple
>> ways to report when something doesn't work.  Failing to boot is *one*
>> of them and not a very good one.  Here, for practical reasons, the
>end
>> result may differ depending on the specifics of the configuration, so
>> more detailed reporting is necessary anyway, so why do you insist on
>> failing the boot?  In what world is it a good thing for the machine
>to
>> fail boot after bios or kernel update?
>
>Because boot failure have no chance to overlook and better way for
>practice.

-- 
Sent from my mobile phone. Please excuse brevity and lack of formatting.

--
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: "H. Peter Anvin" <hpa@zytor.com>
To: KOSAKI Motohiro <kosaki.motohiro@gmail.com>, Tejun Heo <tj@kernel.org>
Cc: Tang Chen <imtangchen@gmail.com>,
	Tang Chen <tangchen@cn.fujitsu.com>,
	robert.moore@intel.com, lv.zheng@intel.com, rjw@sisk.pl,
	lenb@kernel.org, tglx@linutronix.de, mingo@elte.hu,
	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 part5 0/7] Arrange hotpluggable memory as ZONE_MOVABLE.
Date: Wed, 14 Aug 2013 13:30:38 -0700	[thread overview]
Message-ID: <5a1b9edd-8232-498a-b94a-72e028772970@email.android.com> (raw)
In-Reply-To: <520BE891.8090004@gmail.com>

There are systems which can.  They have the ability to remap in hardware.

KOSAKI Motohiro <kosaki.motohiro@gmail.com> wrote:
>(8/14/13 3:55 PM), Tejun Heo wrote:
>> Hello,
>>
>> On Wed, Aug 14, 2013 at 03:40:31PM -0400, KOSAKI Motohiro wrote:
>>> I don't agree it. Please look at other kernel options. A lot of
>these don't
>>> follow you. These behave as direction, not advise.
>>>
>>> I mean the fallback should be implemented at turning on default the
>feature.
>>
>> Yeah, some options are "please try this" and others "do this or
>fail".
>> There's no frigging fundamental rule there.
>
>In this case, we have zero worth for fallback, right?
>
>
>>> I don't read whole discussion and I don't quite understand why no
>kernel
>>> place controlling is relevant. Every unpluggable node is suitable
>for
>>> kernel. If you mean current kernel placement logic don't care
>plugging,
>>> that's a bug.
>>>
>>> If we aim to hot remove, we have to have either kernel relocation or
>>> hotplug awre kernel placement at boot time.
>>
>> What if all nodes are hot pluggable?  Are we moving the kernel
>> dynamically then?
>
>Intel folks already told, we have no such system in practice.
>
>
>>>> Failing to boot is *way* worse reporting mechanism than almost
>>>> everything else.  If the sysadmin is willing to risk machines
>failing
>>>> to come up, she would definitely be willing to check whether which
>>>> memory areas are actually hotpluggable too, right?
>>>
>>> No. see above. Your opinion is not pragmatic useful.
>>
>> No, what you're saying doesn't make any sense.  There are multiple
>> ways to report when something doesn't work.  Failing to boot is *one*
>> of them and not a very good one.  Here, for practical reasons, the
>end
>> result may differ depending on the specifics of the configuration, so
>> more detailed reporting is necessary anyway, so why do you insist on
>> failing the boot?  In what world is it a good thing for the machine
>to
>> fail boot after bios or kernel update?
>
>Because boot failure have no chance to overlook and better way for
>practice.

-- 
Sent from my mobile phone. Please excuse brevity and lack of formatting.

  reply	other threads:[~2013-08-14 20:30 UTC|newest]

Thread overview: 165+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-08 10:16 [PATCH part5 0/7] Arrange hotpluggable memory as ZONE_MOVABLE Tang Chen
2013-08-08 10:16 ` Tang Chen
2013-08-08 10:16 ` [PATCH part5 1/7] x86: get pg_data_t's memory from other node Tang Chen
2013-08-08 10:16   ` Tang Chen
2013-08-12 14:39   ` Tejun Heo
2013-08-12 14:39     ` Tejun Heo
2013-08-12 15:12     ` Tang Chen
2013-08-12 15:12       ` Tang Chen
2013-08-08 10:16 ` [PATCH part5 2/7] x86, numa, mem_hotplug: Skip all the regions the kernel resides in Tang Chen
2013-08-08 10:16   ` Tang Chen
2013-08-08 10:16 ` [PATCH part5 3/7] memblock, numa: Introduce flag into memblock Tang Chen
2013-08-08 10:16   ` Tang Chen
2013-08-08 10:16 ` [PATCH part5 4/7] memblock, mem_hotplug: Introduce MEMBLOCK_HOTPLUG flag to mark hotpluggable regions Tang Chen
2013-08-08 10:16   ` Tang Chen
2013-08-08 10:16 ` [PATCH part5 5/7] memblock, mem_hotplug: Make memblock skip hotpluggable regions by default Tang Chen
2013-08-08 10:16   ` Tang Chen
2013-08-14 21:54   ` Naoya Horiguchi
2013-08-14 21:54     ` Naoya Horiguchi
2013-08-15  5:15     ` Tang Chen
2013-08-15  5:15       ` Tang Chen
2013-08-08 10:16 ` [PATCH part5 6/7] mem-hotplug: Introduce movablenode boot option to {en|dis}able using SRAT Tang Chen
2013-08-08 10:16   ` Tang Chen
2013-08-08 10:16 ` [PATCH part5 7/7] x86, numa, acpi, memory-hotplug: Make movablenode have higher priority Tang Chen
2013-08-08 10:16   ` Tang Chen
2013-08-09 16:32 ` [PATCH part5 0/7] Arrange hotpluggable memory as ZONE_MOVABLE Tejun Heo
2013-08-09 16:32   ` Tejun Heo
2013-08-12  6:33   ` Tang Chen
2013-08-12  8:54   ` Tang Chen
2013-08-12  8:54     ` Tang Chen
2013-08-12 14:50 ` Tejun Heo
2013-08-12 14:50   ` Tejun Heo
2013-08-12 15:14   ` H. Peter Anvin
2013-08-12 15:14     ` H. Peter Anvin
2013-08-12 15:23     ` Tejun Heo
2013-08-12 15:23       ` Tejun Heo
2013-08-12 16:29       ` Tang Chen
2013-08-12 16:29         ` Tang Chen
2013-08-12 16:46         ` Tejun Heo
2013-08-12 16:46           ` Tejun Heo
2013-08-12 18:23           ` Tang Chen
2013-08-12 18:23             ` Tang Chen
2013-08-12 20:20             ` Tejun Heo
2013-08-12 20:20               ` Tejun Heo
2013-08-12 20:49               ` Luck, Tony
2013-08-12 20:49                 ` Luck, Tony
2013-08-12 20:54                 ` Tejun Heo
2013-08-12 20:54                   ` Tejun Heo
2013-08-12 20:57                   ` H. Peter Anvin
2013-08-12 20:57                     ` H. Peter Anvin
2013-08-12 21:06                     ` Yinghai Lu
2013-08-12 21:06                       ` Yinghai Lu
2013-08-12 21:08                       ` Tejun Heo
2013-08-12 21:08                         ` Tejun Heo
2013-08-12 21:12                         ` H. Peter Anvin
2013-08-12 21:12                           ` H. Peter Anvin
2013-08-12 21:14                           ` Tejun Heo
2013-08-12 21:14                             ` Tejun Heo
2013-08-12 21:11                       ` H. Peter Anvin
2013-08-12 21:11                         ` H. Peter Anvin
2013-08-12 21:11                   ` Luck, Tony
2013-08-12 21:11                     ` Luck, Tony
2013-08-12 21:25                     ` Yinghai Lu
2013-08-12 21:25                       ` Yinghai Lu
2013-08-12 21:28                       ` H. Peter Anvin
2013-08-12 21:28                         ` H. Peter Anvin
2013-08-13  5:14                     ` H. Peter Anvin
2013-08-13  5:14                       ` H. Peter Anvin
2013-08-13  6:14           ` Tang Chen
2013-08-13  6:14             ` Tang Chen
2013-08-13  9:56             ` Tang Chen
2013-08-13  9:56               ` Tang Chen
2013-08-13 14:38               ` Tejun Heo
2013-08-13 14:38                 ` Tejun Heo
2013-08-13 22:33               ` Yinghai Lu
2013-08-13 22:33                 ` Yinghai Lu
2013-08-14  1:22                 ` Tang Chen
2013-08-14  1:22                   ` Tang Chen
2013-08-15 19:06                   ` Toshi Kani
2013-08-15 19:06                     ` Toshi Kani
2013-08-15 20:28                     ` Yinghai Lu
2013-08-15 20:28                       ` Yinghai Lu
2013-08-16  2:08                       ` Tang Chen
2013-08-16  2:08                         ` Tang Chen
2013-08-16  4:21                         ` Yinghai Lu
2013-08-16  4:21                           ` Yinghai Lu
2013-08-19  3:07                           ` Tang Chen
2013-08-19  3:07                             ` Tang Chen
2013-08-19  3:28                             ` Yinghai Lu
2013-08-19  3:28                               ` Yinghai Lu
2013-08-15  8:42                 ` Tang Chen
2013-08-15  8:42                   ` Tang Chen
2013-08-15 12:19                   ` Tejun Heo
2013-08-15 12:19                     ` Tejun Heo
2013-08-15 12:44                     ` Tang Chen
2013-08-15 12:44                       ` Tang Chen
2013-08-15 12:49                       ` Tejun Heo
2013-08-15 12:49                         ` Tejun Heo
2013-08-15 12:52                         ` Tang Chen
2013-08-15 12:52                           ` Tang Chen
2013-08-15 14:37                       ` Yinghai Lu
2013-08-15 14:37                         ` Yinghai Lu
2013-08-15 14:45                         ` Tejun Heo
2013-08-15 14:45                           ` Tejun Heo
2013-08-15 15:05                           ` Yinghai Lu
2013-08-15 15:05                             ` Yinghai Lu
2013-08-15 15:10                             ` Tejun Heo
2013-08-15 15:10                               ` Tejun Heo
2013-08-15 19:49                               ` Toshi Kani
2013-08-15 19:49                                 ` Toshi Kani
2013-08-15 19:08                             ` Luck, Tony
2013-08-15 19:08                               ` Luck, Tony
2013-08-15 19:34                               ` Yinghai Lu
2013-08-15 19:34                                 ` Yinghai Lu
2013-08-15 14:35                   ` Yinghai Lu
2013-08-15 14:35                     ` Yinghai Lu
2013-08-16  1:16                     ` Tang Chen
2013-08-16  1:16                       ` Tang Chen
2013-08-12 15:41   ` Tang Chen
2013-08-12 15:41     ` Tang Chen
2013-08-12 15:46     ` Tejun Heo
2013-08-12 15:46       ` Tejun Heo
2013-08-12 16:19       ` Tang Chen
2013-08-12 16:19         ` Tang Chen
2013-08-12 16:22         ` Tejun Heo
2013-08-12 16:22           ` Tejun Heo
2013-08-12 17:01           ` Tang Chen
2013-08-12 17:01             ` Tang Chen
2013-08-12 17:23             ` H. Peter Anvin
2013-08-12 17:23               ` H. Peter Anvin
2013-08-14 18:22               ` KOSAKI Motohiro
2013-08-14 18:22                 ` KOSAKI Motohiro
2013-08-12 18:07             ` Tejun Heo
2013-08-12 18:07               ` Tejun Heo
2013-08-14 18:15               ` KOSAKI Motohiro
2013-08-14 18:15                 ` KOSAKI Motohiro
2013-08-14 18:23                 ` Tejun Heo
2013-08-14 18:23                   ` Tejun Heo
2013-08-14 19:40                   ` KOSAKI Motohiro
2013-08-14 19:40                     ` KOSAKI Motohiro
2013-08-14 19:55                     ` Tejun Heo
2013-08-14 19:55                       ` Tejun Heo
2013-08-14 20:29                       ` KOSAKI Motohiro
2013-08-14 20:29                         ` KOSAKI Motohiro
2013-08-14 20:30                         ` H. Peter Anvin [this message]
2013-08-14 20:30                           ` H. Peter Anvin
2013-08-14 20:35                         ` Tejun Heo
2013-08-14 20:35                           ` Tejun Heo
2013-08-14 21:17                           ` KOSAKI Motohiro
2013-08-14 21:17                             ` KOSAKI Motohiro
2013-08-14 21:36                             ` Tejun Heo
2013-08-14 21:36                               ` Tejun Heo
2013-08-15  1:08                               ` KOSAKI Motohiro
2013-08-15  1:08                                 ` KOSAKI Motohiro
2013-08-15  1:21                                 ` Tejun Heo
2013-08-15  1:21                                   ` Tejun Heo
2013-08-15  1:33                                   ` Tejun Heo
2013-08-15  1:33                                     ` Tejun Heo
2013-08-15  1:44                                     ` KOSAKI Motohiro
2013-08-15  1:44                                       ` KOSAKI Motohiro
2013-08-15  2:22                                       ` Tejun Heo
2013-08-15  2:22                                         ` Tejun Heo
2013-08-15  1:38                                   ` KOSAKI Motohiro
2013-08-15  1:38                                     ` KOSAKI Motohiro
2013-08-15  1:51                                     ` Tejun Heo
2013-08-15  1:51                                       ` Tejun Heo

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=5a1b9edd-8232-498a-b94a-72e028772970@email.android.com \
    --to=hpa@zytor.com \
    --cc=akpm@linux-foundation.org \
    --cc=gong.chen@linux.intel.com \
    --cc=imtangchen@gmail.com \
    --cc=isimatu.yasuaki@jp.fujitsu.com \
    --cc=izumi.taku@jp.fujitsu.com \
    --cc=jiang.liu@huawei.com \
    --cc=jweiner@redhat.com \
    --cc=kosaki.motohiro@gmail.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=tj@kernel.org \
    --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@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: link
Be 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.