Re: [PATCH 3/7] vmw_balloon: fix inflation of 64-bit GFNs

From: Nadav Amit
Date: Wed Jun 13 2018 - 14:27:30 EST


at 11:21 AM, Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:

> On Wed, Jun 13, 2018 at 04:03:55AM -0700, Nadav Amit wrote:
>> When balloon batching is not supported by the hypervisor, the guest
>> frame number (GFN) must fit in 32-bit. However, due to a bug, this check
>> was mistakenly ignored. In practice, when total RAM is greater than
>> 16TB, the balloon does not work currently, making this bug unlikely to
>> happen.
>>
>> Fixes: ef0f8f112984 ("VMware balloon: partially inline
>> vmballoon_reserve_page.")
>
> Why have a bugfix as the 3rd patch of the series? Please put it first,
> so that it can get into Linus's tree sooner.
>
> Also, no need to line-wrap this, and drop the blank line.
>
> And finally, does this need to go to the stable kernels? If so, please
> add the correct line for that.

I will fix the message. Patches 3-7 deal with minor bugs, some of which have
been there since day one, so I didnât put a âFixesâ tag.

Iâll follow your direction and put patches 1-2 at the end.