Re: [PATCH RFC] mm/memory_hotplug: Introduce memory block types

On Mon, 26 Nov 2018 14:33:29 +0100
David Hildenbrand [off-list ref] wrote:

On 26.11.18 13:30, David Hildenbrand wrote:

quoted

On 23.11.18 19:06, Michal Suchánek wrote:  

quoted

quoted

If we are going to fake the driver information we may as well add the
type attribute and be done with it.

I think the problem with the patch was more with the semantic than the
attribute itself.

What is normal, paravirtualized, and standby memory?

I can understand DIMM device, baloon device, or whatever mechanism for
adding memory you might have.

I can understand "memory designated as standby by the cluster
administrator".

However, DIMM vs baloon is orthogonal to standby and should not be
conflated into one property.

paravirtualized means nothing at all in relationship to memory type and
the desired online policy to me.  

Right, so with whatever we come up, it should allow to make a decision
in user space about
- if memory is to be onlined automatically  

And I will think about if we really should model standby memory. Maybe
it is really better to have in user space something like (as Dan noted)

If it is possible to designate the memory as standby or online in the
s390 admin interface and the kernel does have access to this
information it makes sense to forward it to userspace (as separate
s390-specific property). If not then you need to make some kind of
assumption like below and the user can tune the script according to
their usecase.

if (isS390x() && type == "dimm") {
	/* don't online, on s390x system DIMMs are standby memory */
}

Thanks

Michal