Re: [PATCH v8 10/10] tracing/hyper-v: trace hyperv_mmu_flush_tlb_others()

From: Andy Shevchenko
Date: Fri Jun 09 2017 - 14:23:59 EST


On Fri, Jun 9, 2017 at 9:04 PM, Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:
> On Fri, 9 Jun 2017 15:27:36 +0200
> Vitaly Kuznetsov <vkuznets@xxxxxxxxxx> wrote:


>> +#if IS_ENABLED(CONFIG_HYPERV)
>
> Hmm, this is new to me. I know you can use IS_ENABLED() inside C code,
> but I've never seen it used for preprocessor directives. This usually
> is just:
>
> #ifdef CONFIG_HYPERV
>
> Other than that, this looks fine to me.

That is the magic of IS_ENABLED(), IS_BUILTIN() macros.
So, the code above is fine as is.

--
With Best Regards,
Andy Shevchenko