From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751366AbdH1O0J (ORCPT ); Mon, 28 Aug 2017 10:26:09 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:43048 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751170AbdH1O0I (ORCPT ); Mon, 28 Aug 2017 10:26:08 -0400 Date: Mon, 28 Aug 2017 16:26:13 +0200 From: Greg Kroah-Hartman To: Hans de Goede Cc: Arnd Bergmann , linux-kernel@vger.kernel.org Subject: Re: [PATCH] MAINTAINERS: Add files under drivers/virt/* to CHAR and MISC DRIVERS section Message-ID: <20170828142613.GA21720@kroah.com> References: <20170825115623.22785-1-hdegoede@redhat.com> <20170825131006.GA14726@kroah.com> <0d43b1d1-1976-dad5-fe45-de854417ef4f@redhat.com> <20170828140852.GA30541@kroah.com> <43c3b97f-20c0-9232-432e-96af6bf369f6@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <43c3b97f-20c0-9232-432e-96af6bf369f6@redhat.com> User-Agent: Mutt/1.8.3 (2017-05-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Aug 28, 2017 at 04:13:38PM +0200, Hans de Goede wrote: > Hi, > > On 28-08-17 16:08, Greg Kroah-Hartman wrote: > > On Fri, Aug 25, 2017 at 03:11:25PM +0200, Hans de Goede wrote: > > > Hi, > > > > > > On 25-08-17 15:10, Greg Kroah-Hartman wrote: > > > > On Fri, Aug 25, 2017 at 01:56:23PM +0200, Hans de Goede wrote: > > > > > Cc: Greg Kroah-Hartman > > > > > Cc: Arnd Bergmann > > > > > Signed-off-by: Hans de Goede > > > > > --- > > > > > MAINTAINERS | 1 + > > > > > 1 file changed, 1 insertion(+) > > > > > > > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > > > > index 6a349b5c0c2a..db633caa5a1d 100644 > > > > > --- a/MAINTAINERS > > > > > +++ b/MAINTAINERS > > > > > @@ -3284,6 +3284,7 @@ T: git git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git > > > > > S: Supported > > > > > F: drivers/char/* > > > > > F: drivers/misc/* > > > > > +F: drivers/virt/* > > > > > > > > Really? I'm now in charge of this directory too? That's news to me... > > > > > > This was suggested by Arnd, as he preferred putting the vboxguest driver > > > under drivers/virt rather then drivers/misc. Note that currently there is > > > only one .c file under drivers/virt: fsl_hypervisor.c > > > > Ok, but how about we wait until there is more than one file in here? > > That is fine by me. > > > And what happened to all of the other "virt" drivers, why are they not here? > > Some vmware drivers are currently under drivers/misc, Arnd's proposal was > to move those to drivers/virt. virtio and hyperv have their own subsys > dirs. > > > Perhaps using this directory isn't really a good idea? > > I started out with the same idea which is why v1 of my RFC posting of the > vboxguest driver used drivers/misc, alternatively we could move the single > driver currently under drivers/virt to drivers/misc and stop using > drivers/virt. > > > I don't know... > > Arnd indicated he has a preference for using drivers/virt for vboxguest + > the vmware drivers, I'm fine either way. This really is a bit of a bikeshed- > ish discussion IMHO. So if you and/or Arnd pick one I will happily follow > whatever you choose. Ah, if we move the vmware ones there, I have no problem with it, I missed that part of the discussion. thanks, greg k-h