Re: [PATCH part5 0/7] Arrange hotpluggable memory as ZONE_MOVABLE.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 08/13/2013 12:46 AM, Tejun Heo wrote:
Hello, Tang.
......

But, different users have different ways to use memory hotplug.

Hotswaping any particular chunk of memory is the goal we will reach
finally. But it is on specific hardware. In most current machines, we
can use movable node to manage resource in node unit.

And also, without this movablenode boot option, the MOVABLE_NODE
functionality, which is already in the kernel, will not be able to
work. All nodes has kernel memory means no movable node.

So, how about this: Just like MOVABLE_NODE functionality, introduce
a new config option. When we have better solutions for memory hotplug,
we shutoff or remove the config and related code.

For now, at least make movable node work.

Hi tj,
cc hpa,

I explained above because hpa said he thought the whole approach is
wrong. I think node hotplug is meaningful for users. And without this
patch-set, MOVABLE_NODE means nothing. This is all above.

Since you replied his email in previous emails, I just replied to
answer both of you. Sorry for the misunderstanding. :)


We are talking completely past each other.  I'll just try to clarify
what I was saying.  Can you please do the same?  Let's re-sync on the
discussion.

* Adding an option to tell the kernel to try to stay away from
   hotpluggable nodes is fine.  I have no problem with that at all.

Agreed.


* The patchsets upto this point have been somehow trying to reorder
   operations shomehow such that *no* memory allocation happens before
   memblock is populated with hotplug information.

Yes, this is exactly what I want to do.


* However, we already *know* that the memory the kernel image is
   occupying won't be removeable.  It's highly likely that the amount
   of memory allocation before NUMA / hotplug information is fully
   populated is pretty small.  Also, it's highly likely that small
   amount of memory right after the kernel image is contained in the
   same NUMA node, so if we allocate memory close to the kernel image,
   it's likely that we don't contaminate hotpluggable node.  We're
   talking about few megs at most right after the kernel image.  I
   can't see how that would make any noticeable difference.

This point, I don't quite agree. What you said is highly likely, but
not definitely. Users may find they lost hotpluggable memory.

The node the kernel resides in won't be removable. This is agreed.
But I still want SRAT earlier for the following reasons:

1. For a production provided to users, the firmware specified how
   many nodes are hotpluggable. When the system is up, if users
   found they lost movable nodes, I think it could be messy.

2. Reorder SRAT parsing earlier is not that difficult to do. The
   only procedures reordered are acpi tables initialization and
   acpi_initrd_override. The acpi part patches are being reviewed.
   And it is better solution. If possible, I think we should do it.

In summary, I don't want early memory allocation with hotpluggable
memory to be opportunistic.


* Once hotplug information is available, allocation can happen as
   usual and the kernel can report the nodes which are actually
   hotpluggable - marked as hotpluggable by the firmware&&  didn't get
   contaminated during early alloc&&  didn't get overflow allocations
   afterwards.  Note that we need such mechanism no matter what as the
   kernel image can be loaded into hotpluggable nodes and reporting
   that to userland is the only thing the kernel can do for cases like
   that short of denying memory unplug on such nodes.

Agreed.


The whole thing would be a lot simpler and generic.  It doesn't even
have to care about which mechanism is being used to acquire all those
information.  What am I missing here?

Sorry for the misunderstanding.

Thanks. :)

--
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




[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux