Re: [Xen-devel] [PATCH] xen/balloon: cancel ballooning if adding new memory failed

[David Vrabel <david.vrabel@xxxxxxxxxx>]

On 29/01/15 13:36, Ian Campbell wrote:
> On Mon, 2014-09-01 at 18:52 +0100, David Vrabel wrote:
>> If the balloon driver is adding additional memory regions to the
>> balloon and add_memory() fails it will likely continuously fail so
>> cancel the balloon operation.
>>
>> Signed-off-by: David Vrabel <david.vrabel@xxxxxxxxxx>
> 
> https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=776448 and
> https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1304001 seem to
> suggest this should be a candidate for stable backports?

It's up to the distro kernel maintainer to request it if they think it
is important.  It's a harmless message with no side effects so I don't
think it qualifies.

David

>>  drivers/xen/balloon.c |    4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/xen/balloon.c b/drivers/xen/balloon.c
>> index 5c660c7..1e0a317 100644
>> --- a/drivers/xen/balloon.c
>> +++ b/drivers/xen/balloon.c
>> @@ -230,8 +230,8 @@ static enum bp_state reserve_additional_memory(long 
>> credit)
>>      rc = add_memory(nid, hotplug_start_paddr, balloon_hotplug << 
>> PAGE_SHIFT);
>>  
>>      if (rc) {
>> -            pr_info("%s: add_memory() failed: %i\n", __func__, rc);
>> -            return BP_EAGAIN;
>> +            pr_warn("Cannot add additional memory (%i)\n", rc);
>> +            return BP_ECANCELED;
>>      }
>>  
>>      balloon_hotplug -= credit;
> 
> 


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel