From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753499AbdHUMEy (ORCPT ); Mon, 21 Aug 2017 08:04:54 -0400 Received: from mx1.redhat.com ([209.132.183.28]:49510 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751959AbdHUMEw (ORCPT ); Mon, 21 Aug 2017 08:04:52 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com B3AEC5F7BA Authentication-Results: ext-mx10.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com Authentication-Results: ext-mx10.extmail.prod.ext.phx2.redhat.com; spf=fail smtp.mailfrom=hdegoede@redhat.com Subject: Re: [RFC 1/2] misc: Add vboxguest driver for Virtual Box Guest integration From: Hans de Goede To: Arnd Bergmann Cc: Michael Thayer , "Knut St . Osmundsen" , Greg Kroah-Hartman , Alexander Viro , Larry Finger , Linux Kernel Mailing List , Linux FS-devel Mailing List References: <20170811132324.6216-1-hdegoede@redhat.com> <20170811132324.6216-2-hdegoede@redhat.com> <82e9619b-0e6d-ae10-28c8-87295ae85389@redhat.com> Message-ID: Date: Mon, 21 Aug 2017 14:04:49 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.39]); Mon, 21 Aug 2017 12:04:52 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, On 21-08-17 13:43, Hans de Goede wrote: > I'm also thinking that under Linux since we never use userspace to > alloc memory for the balloon, we don't need to involve userspace at > all, the drivers interrupt handler could detect the event that the > host wants to change the balloonsize itself and start a workqueue > item which then asks for what the new size should be and do the > adjustment all inside the kernel. I think that will even lead to > simpler code as we no longer need to track the balloon userspace > owner and do reclaim when it dies. > > Typing this out I really see no reason to not do this inside the > kernel, so I think I'm going to implement that right away / for > the [RFC v2] I plan to post later today or tomorrow. Done: https://github.com/jwrdegoede/vboxguest/commit/2920882cb0ce9af11a843affc58d4a08ef47e16b Regards, Hans