>>> On 03.05.16 at 16:29, <car...@cardoe.com> 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 <jbeul...@suse.com> _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
