Re: [PATCH 2.6.16-rc6-xen] export Xen Hypervisor attributes to sysfs

From: Mike D. Day
Date: Tue Mar 21 2006 - 06:30:07 EST


Arjan van de Ven wrote:

I appreciate the counter argument as well, but think this should be a configurable option.

"a config option" is a cop-out. By proposing a config option you even
admit it's not essential for userspace. The last thing this should be is
a config option; it either is needed and important, and should be there
always, or it's redundant and should be done in userspace. "A config
option" is just admitting that it is the later, but that you need to
justify the time you spent on the kernel patch somewhere ;-)

No, not at all. There are two types of Xen kernels: privileged and non-privileged. A privileged kernel has access to real hardware devices. This patch is intended for privileged xen kernel. That's why it needs to be a configurable option.

Mike
-
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/