[PATCH] Drivers: hv: vmbus: Don't leak memory when a channel is rescinde

[PATCH] Drivers: hv: vmbus: Don't leak memory when a channel is rescinde

[Dexuan Cui]

Hi, Can you please help to backport the patch to the stable tree?

Tn Linus's tree, the patch's commit ID is:
5e030d5ce9d99a899b648413139ff65bab12b038

As described in the its changelog, it fixes a memory leak issue, which
affects VMBus users like the hv_sock driver. Rolf (in Cc) reported that the
memory can be eaten up quickly with hv_sock press test.

The patch only needs to be applied to the 4.10 and 4.9 branches, whose
latest versions are v4.10.6 and v4.9.18, respectively.

Thanks,
-- Dexuan

Re: [PATCH] Drivers: hv: vmbus: Don't leak memory when a channel is rescinde

[Rolf Neugebauer]

On Mon, Mar 27, 2017 at 10:33 AM, Dexuan Cui <decui@microsoft.com> wrote:
> Hi, Can you please help to backport the patch to the stable tree?
>
> Tn Linus's tree, the patch's commit ID is:
> 5e030d5ce9d99a899b648413139ff65bab12b038
>
> As described in the its changelog, it fixes a memory leak issue, which
> affects VMBus users like the hv_sock driver. Rolf (in Cc) reported that the
> memory can be eaten up quickly with hv_sock press test.
>
> The patch only needs to be applied to the 4.10 and 4.9 branches, whose
> latest versions are v4.10.6 and v4.9.18, respectively.

Thanks Dexuan.

I can confirm that 5e030d5ce9d99a899b64 ("Drivers: hv: vmbus: Don't
leak memory when a channel is rescinded") fixes a memory leak with the
hv_sock driver which was introduced with 4.9.14 with
ccb61f8a99e6c29df4fb96a65dad4fad740d5be9 ("Drivers: hv:
vmbus: Fix a rescind handling bug") and in v4.10.2 onwards with
ec6f27bd19e0f9142c71d293d4c3b381348550b  ("Drivers: hv:
vmbus: Fix a rescind handling bug").

I've tested both a 4.9.18 and 4.10.6 kernel with  5e030d5ce9d99a899b64
("Drivers: hv: vmbus: Don't leak memory when a channel is rescinded")
applied and can confirm that the memory leak is fixed by this commit.

It would be great to get it included in the next round of stable releases.

Tested-by: Rolf Neugebauer <rolf.neugebauer@docker.com>

Thanks
Rolf


>
> Thanks,
> -- Dexuan