Re: [Qemu-devel] [PATCH v3 1/4] balloon: Allow multiple inhibit users

2018-08-07 Thread Michael S. Tsirkin
On Tue, Aug 07, 2018 at 02:08:26PM -0600, Alex Williamson wrote: > On Tue, 7 Aug 2018 22:44:11 +0300 > "Michael S. Tsirkin" wrote: > > > On Tue, Aug 07, 2018 at 01:31:22PM -0600, Alex Williamson wrote: > > > A simple true/false internal state does not allow multiple users. Fix > > > this within

Re: [Qemu-devel] [PATCH v3 1/4] balloon: Allow multiple inhibit users

2018-08-07 Thread Alex Williamson
On Tue, 7 Aug 2018 22:44:11 +0300 "Michael S. Tsirkin" wrote: > On Tue, Aug 07, 2018 at 01:31:22PM -0600, Alex Williamson wrote: > > A simple true/false internal state does not allow multiple users. Fix > > this within the existing interface by converting to a counter, so long > > as the counter

Re: [Qemu-devel] [PATCH v3 1/4] balloon: Allow multiple inhibit users

2018-08-07 Thread Michael S. Tsirkin
On Tue, Aug 07, 2018 at 01:31:22PM -0600, Alex Williamson wrote: > A simple true/false internal state does not allow multiple users. Fix > this within the existing interface by converting to a counter, so long > as the counter is elevated, ballooning is inhibited. > > Reviewed-by: David Hildenbra

[Qemu-devel] [PATCH v3 1/4] balloon: Allow multiple inhibit users

2018-08-07 Thread Alex Williamson
A simple true/false internal state does not allow multiple users. Fix this within the existing interface by converting to a counter, so long as the counter is elevated, ballooning is inhibited. Reviewed-by: David Hildenbrand Reviewed-by: Peter Xu Reviewed-by: Cornelia Huck Signed-off-by: Alex