From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754053AbcBWQNM (ORCPT ); Tue, 23 Feb 2016 11:13:12 -0500 Received: from mx2.parallels.com ([199.115.105.18]:53642 "EHLO mx2.parallels.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751809AbcBWQNK (ORCPT ); Tue, 23 Feb 2016 11:13:10 -0500 Subject: Re: [PATCH 2/2] virtio_balloon: export 'available' memory to balloon statistics To: "Michael S. Tsirkin" References: <1455637852-7323-1-git-send-email-den@openvz.org> <1455637852-7323-3-git-send-email-den@openvz.org> <20160223170828-mutt-send-email-mst@redhat.com> <56CC7A37.2030807@openvz.org> <20160223174952-mutt-send-email-mst@redhat.com> CC: , , Igor Redko , Andrew Morton From: "Denis V. Lunev" Message-ID: <56CC850A.1060704@openvz.org> Date: Tue, 23 Feb 2016 19:12:58 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: <20160223174952-mutt-send-email-mst@redhat.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit X-ClientProxiedBy: US-EXCH.sw.swsoft.com (10.255.249.47) To US-EXCH2.sw.swsoft.com (10.255.249.46) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/23/2016 06:53 PM, Michael S. Tsirkin wrote: > On Tue, Feb 23, 2016 at 06:26:47PM +0300, Denis V. Lunev wrote: >> On 02/23/2016 06:10 PM, Michael S. Tsirkin wrote: >>> On Tue, Feb 16, 2016 at 06:50:52PM +0300, Denis V. Lunev wrote: >>>> From: Igor Redko >>>> >>>> Add a new field, VIRTIO_BALLOON_S_AVAIL, to virtio_balloon memory >>>> statistics protocol, corresponding to 'Available' in /proc/meminfo. >>>> >>>> It indicates to the hypervisor how big the balloon can be inflated >>>> without pushing the guest system to swap. >>>> >>>> Signed-off-by: Igor Redko >>>> Reviewed-by: Roman Kagan >>>> Signed-off-by: Denis V. Lunev >>>> CC: Michael S. Tsirkin >>>> CC: Andrew Morton >>> Oops - I missed the fact that this affects host/guest ABI. >>> >>> Can you please submit ABI update proposal to virtio tc? >>> Spec patch would be even better. >>> >>> This is important to ensure there are no conflicts >>> with other features being developed in parallel. >> hmmm >> >> From my point of view ABI remains untouched. > Anything exposed by guest to host is ABI. > Once we add stuff there, we never can remove it > as some host might rely on it. > >> The guest can send any amount of ; >> pairs and unknown tags are properly ignored >> by the host. >> >> That is why I think that this change is safe. > What happens if someone uses the tag you > used for VIRTIO_BALLOON_S_AVAIL, for some > other purpose? > Any tools using VIRTIO_BALLOON_S_AVAIL will be confused. actually this constant resides in QEMU only, values are reported above using JSON and string tags. > Really, it's not hard to get a tag number from virtio TC, > so please just do this. > ok. So do you propose to negotiate maximum allowed tag to send at the driver start time? we will have to guard this exchange with proper flag in feature space then. This could be done but from my point of view this looks like serious over-complication. Do we have somebody who can judge? Den