On 07/24/2013 02:48 AM, Tejun Heo wrote:
On Fri, Jul 19, 2013 at 03:59:14PM +0800, Tang Chen wrote:
The Hot-Pluggable field in SRAT suggests if the memory could be
hotplugged while the system is running. Print it as well when
parsing SRAT will help users to know which memory is hotpluggable.

Signed-off-by: Tang Chen<tangc...@cn.fujitsu.com>
Reviewed-by: Wanpeng Li<liw...@linux.vnet.ibm.com>

Acked-by: Tejun Heo<t...@kernel.org>

But a nit below

+       pr_info("SRAT: Node %u PXM %u [mem %#010Lx-%#010Lx] %s\n",
+               node, pxm,
+               (unsigned long long) start, (unsigned long long) end - 1,
+               hotpluggable ? "Hot Pluggable" : "");

The following would be more conventional.

   "...10Lx]%s\n", ..., hotpluggable ? " Hot Pluggable" : ""

Also, isn't "Hot Pluggable" a bit too verbose?  "hotplug" should be
fine, I think.


Hi tj, Joe,

OK,will change it as you guys said.
Thank you very much.

Thanks.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to