[lkml]   [2017]   [Aug]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] MAINTAINERS: Add files under drivers/virt/* to CHAR and MISC DRIVERS section

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(+)
>>>> index 6a349b5c0c2a..db633caa5a1d 100644
>>>> --- a/MAINTAINERS
>>>> +++ b/MAINTAINERS
>>>> @@ -3284,6 +3284,7 @@ T: git 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

> 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

> 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.



 \ /
  Last update: 2017-08-28 16:19    [W:0.037 / U:0.060 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site