Re: [Xen-devel] [PATCH v2 3/7] build: convert verbose to Kconfig

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 03.05.16 at 16:29, <cardoe@xxxxxxxxxx> wrote:
> --- a/xen/Kconfig.debug
> +++ b/xen/Kconfig.debug
> @@ -15,4 +15,11 @@ config CRASH_DEBUG
>         If you want to be able to attach gdb to Xen to be able to debug
>         Xen if it crashes then say Y.
>  
> +config VERBOSE_DEBUG
> +     bool "Verbose debug messages"
> +     default y

With the enclosing "if" adjusted, this would need to become
"default DEBUG", and I think that's a good idea anyway (i.e. even
if that other change was rejected by someone else, this would still
make the code prepared to _some_ future adjustment to that "if").

> @@ -17,10 +16,7 @@ include $(XEN_ROOT)/Config.mk
>  # Hardcoded configuration implications and dependencies.
>  # Do this is a neater way if it becomes unwieldy.
>  ifeq ($(debug),y)
> -verbose       := y
>  frame_pointer := y
> -else
> -CFLAGS += -DNDEBUG

Ah, here's the change that belongs into patch 1.

Again, with those adjustments
Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel