Re: [libvirt] [PATCH 1/3] network: consolidate connection count updates for device pool

2016-02-12 Thread John Ferlan
On 02/11/2016 04:37 PM, Laine Stump wrote: > networkReleaseActualDevice() and networkNotifyActualDevice() both were > updating the individual devices' connections count in two separate > places (unlike networkAllocateActualDevice() which does it in a single > unified place after success:). The

[libvirt] [PATCH 1/3] network: consolidate connection count updates for device pool

2016-02-11 Thread Laine Stump
networkReleaseActualDevice() and networkNotifyActualDevice() both were updating the individual devices' connections count in two separate places (unlike networkAllocateActualDevice() which does it in a single unified place after success:). The code is correct, but prone to confusion / later