Re: [PATCH] s390: Hypervisor File System

From: Greg KH
Date: Thu May 04 2006 - 11:35:27 EST


On Thu, May 04, 2006 at 05:01:07PM +0200, Michael Holzheu wrote:
> Hi Greg,
>
> Greg KH <greg@xxxxxxxxx> wrote on 05/04/2006 04:42:59 PM:
> > > I would suggest do do it like /sys/kernel and put the code
> > > into kernel/ksysfs.c and include/linux/kobject.h
> >
> > No, if you do that then every kernel gets that mount point, when almost
> > no one really wants it :)
> >
> > If you leave it as a separate file, then the build system can just
> > include the file as needed.
> >
>
> So you want a new config option CONFIG_HYPERVISOR?

Sure. But don't make it a user selectable config option, but rather,
one your S390 option sets.

That way the Xen and other groups can also set it when they need it.

> When no one except for us wants it, wouldn't it be best
> then to create /sys/hypervisor first in the hypfs code?
>
> If someone else needs it in the future, we still can move
> it common code.

The Xen people need it too. Now who knows when their code will ever hit
mainline...

thanks,

greg k-h
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/