Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753469AbbDALw4 (ORCPT ); Wed, 1 Apr 2015 07:52:56 -0400 Received: from e06smtp17.uk.ibm.com ([195.75.94.113]:56088 "EHLO e06smtp17.uk.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752973AbbDALwx (ORCPT ); Wed, 1 Apr 2015 07:52:53 -0400 Date: Wed, 1 Apr 2015 13:52:44 +0200 From: Cornelia Huck To: "Michael S. Tsirkin" Cc: linux-kernel@vger.kernel.org, Pawel Moll , virtio-dev@lists.oasis-open.org, Rusty Russell , virtualization@lists.linux-foundation.org, linux-api@vger.kernel.org Subject: Re: [PATCH v3 1/6] virtio_balloon: transitional interface Message-ID: <20150401135244.5a8d7ad1.cornelia.huck@de.ibm.com> In-Reply-To: <20150401124539-mutt-send-email-mst@redhat.com> References: <1427884468-23930-1-git-send-email-mst@redhat.com> <1427884468-23930-2-git-send-email-mst@redhat.com> <20150401124539-mutt-send-email-mst@redhat.com> Organization: IBM Deutschland Research & Development GmbH Vorsitzende des Aufsichtsrats: Martina Koederitz =?UTF-8?B?R2VzY2jDpGZ0c2bDvGhydW5nOg==?= Dirk Wittkopp Sitz der Gesellschaft: =?UTF-8?B?QsO2Ymxpbmdlbg==?= Registergericht: Amtsgericht Stuttgart, HRB 243294 X-Mailer: Claws Mail 3.8.0 (GTK+ 2.24.10; i686-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-TM-AS-MML: disable X-Content-Scanned: Fidelis XPS MAILER x-cbid: 15040111-0029-0000-0000-0000040835B5 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1152 Lines: 31 On Wed, 1 Apr 2015 12:49:02 +0200 "Michael S. Tsirkin" wrote: > On Wed, Apr 01, 2015 at 12:35:45PM +0200, Michael S. Tsirkin wrote: > > Virtio 1.0 doesn't include a modern balloon device. > > But it's not a big change to support a transitional > > balloon device: this has the advantage of supporting > > existing drivers, transparently. > > > > Signed-off-by: Michael S. Tsirkin > > --- > > include/uapi/linux/virtio_balloon.h | 11 ++++++++-- > > drivers/virtio/virtio_balloon.c | 43 +++++++++++++++++++++++++++++-------- > > 2 files changed, 43 insertions(+), 11 deletions(-) > > So all in all 32 LOC added, and 9 out of these > deal with endian-ness differences. > > Seems like a small cost for guests to pay for a clean spec, no? > I'm not opposed to this per se, but I'm not totally convinced of the usefulness :) Seeing the qemu side would be helpful. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/