Subject: Re: [RFCv2 9/9] pseries: Automatically resize HPT for memory hot add/remove

From: Anshuman Khandual <[email-masked]>
Date: Mon, 01 Feb 2016 14:21:46 +0530

On 01/29/2016 10:54 AM, David Gibson wrote:
>  #ifdef CONFIG_MEMORY_HOTPLUG
> +void resize_hpt_for_hotplug(unsigned long new_mem_size)
> +{
> +	unsigned target_hpt_shift;
> +
> +	if (!ppc_md.resize_hpt)
> +		return;
> +
> +	target_hpt_shift = htab_shift_for_mem_size(new_mem_size);
> +
> +	/*
> +	 * To avoid lots of HPT resizes if memory size is fluctuating
> +	 * across a boundary, we deliberately have some hysterisis

What do you mean by 'memory size is fluctuating across a boundary' ?
Through memory hotplug interface ? Why some one will do that ? I
can understand why we dont have this check in the sysfs debug path
as we would like to test any memory HPT re sizing scenario we want
in any sequence of increase or decrease we want.

Overall the RFC V2 looks pretty good. Looking forward to see the
host side of the code for this feature.