[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] xen: Kconfig: Add DEBUG_XEN config option

On 01/02/17 12:19, Andrii Anisov wrote:
> From: Andrii Anisov <andrii_anisov@xxxxxxxx>
> Add a debug option to enable xen drivers debug code.
> Signed-off-by: Andrii Anisov <andrii_anisov@xxxxxxxx>

In future you might want to add the maintainers to the recipient list.

> ---
>  drivers/xen/Kconfig  | 8 ++++++++
>  drivers/xen/Makefile | 2 ++
>  2 files changed, 10 insertions(+)
> diff --git a/drivers/xen/Kconfig b/drivers/xen/Kconfig
> index f15bb3b7..733c433 100644
> --- a/drivers/xen/Kconfig
> +++ b/drivers/xen/Kconfig
> @@ -298,4 +298,12 @@ config XEN_SYMS
>  config XEN_HAVE_VPMU
>         bool
> +config DEBUG_XEN
> +     bool "XEN Drivers debug"
> +     depends on DEBUG_KERNEL
> +     help
> +       Say Y here if you want to enable XEN drivers debug code.
> +
> +       If you are unsure about this, say N here.

So is this really a sensible thing to do? I don't see the value for
such a global config option possibly switching so many different
drivers to debug mode.

In reality you want to do something like "debug" for a single driver
only during time of development. This won't need a global config
option but just a "#define" in the driver which is active while
developing and testing and which should be removed or commented out
when the final submission of the driver is done.


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.