linux-kernel - Re: [PATCH 1/1] Drivers: hv: vmbus: Add additional distro specific Kconfig options

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20130410154957.GA13849@kroah.com>
Date:	Wed, 10 Apr 2013 08:49:57 -0700
From:	Greg KH <gregkh@...uxfoundation.org>
To:	"K. Y. Srinivasan" <kys@...rosoft.com>
Cc:	linux-kernel@...r.kernel.org, devel@...uxdriverproject.org,
	olaf@...fle.de, apw@...onical.com, jasowang@...hat.com
Subject: Re: [PATCH 1/1] Drivers: hv: vmbus: Add additional distro specific
 Kconfig options

On Wed, Apr 10, 2013 at 09:08:17AM -0700, K. Y. Srinivasan wrote:
> The guest ID captures information about the guest and has room for
> distributions to add distro specific information. Add Kconfig options
> to support distro specific information to be managed easily.
> 
> Signed-off-by: K. Y. Srinivasan <kys@...rosoft.com>
> ---
>  drivers/hv/Kconfig        |   14 ++++++++++++++
>  drivers/hv/hv.c           |    4 +++-
>  drivers/hv/hyperv_vmbus.h |    2 +-
>  3 files changed, 18 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/hv/Kconfig b/drivers/hv/Kconfig
> index 0403b51..d2ca9c7 100644
> --- a/drivers/hv/Kconfig
> +++ b/drivers/hv/Kconfig
> @@ -19,4 +19,18 @@ config HYPERV_BALLOON
>  	help
>  	  Select this option to enable Hyper-V Balloon driver.
>  
> +config HYPERV_GUEST_D1
> +	hex "Distro specific information"
> +	range 0x00 0xff
> +	default 0
> +	help
> +	  This specifies the Distro vendor

And just what is a distro supposed to set the value here to?  For
example, what would Arch Linux pick?  Fedora?  RHEL?  SLES?  openSUSE?
You do know that there are more than 0xff different Linux distro
"vendors"?  Do they all just get to pick a number that they like?

> +config HYPERV_GUEST_D2
> +	hex "Additional Distro specific information"
> +	range 0x0000 0xffff
> +	default 0
> +	help
> +	  Additional Distro specific kernel version information

Again, what is a distro supposed to do here?  What's wrong with the
kernel version information that the kernel already has?  Can't you just
use that string?

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