>>> On 02.09.16 at 12:00, <andrew.coop...@citrix.com> wrote:
> On 02/09/16 08:57, Jan Beulich wrote:
>>>>> On 01.09.16 at 21:21, <andrew.coop...@citrix.com> wrote:
>>> --- a/xen/include/xen/lib.h
>>> +++ b/xen/include/xen/lib.h
>>> @@ -68,8 +68,10 @@ extern void debugtrace_dump(void);
>>> extern void debugtrace_printk(const char *fmt, ...)
>>> __attribute__ ((format (printf, 1, 2)));
>>> #else
>>> -#define debugtrace_dump() ((void)0)
>>> -#define debugtrace_printk(_f, ...) ((void)0)
>>> +static inline void debugtrace_dump(void) {};
>>> +static inline void
>>> + __attribute__ ((format (printf, 1, 2)))
>>> +debugtrace_printk(const char *fmt, ...) {};
>>> #endif
>> With the stray semicolons dropped
>
> Oh yes of course.
>
>> and provided this (at least a little surprisingly) doesn't cause any other
>> fallout with
>> DEBUG_TRACE_DUMP defined,
>
> I don't know what you mean by this. We strictly only change the
> non-defined case,
Oh, sorry.
> and that compiles fine on all architectures.
Good.
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
