lkml.org 
[lkml]   [2017]   [Oct]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 01/16] hyper-v: trace vmbus_on_msg_dpc()
On Mon, 2 Oct 2017 08:18:50 +0800
kbuild test robot <lkp@intel.com> wrote:

> Hi Vitaly,
>
> [auto build test WARNING on linus/master]
> [also build test WARNING on v4.14-rc3 next-20170929]
> [if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
>
> url: https://github.com/0day-ci/linux/commits/Vitaly-Kuznetsov/hyper-v-trace-vmbus_on_msg_dpc/20171002-062040
> config: i386-randconfig-x017-201740 (attached as .config)
> compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
> reproduce:
> # save the attached .config to linux build tree
> make ARCH=i386
>
> All warnings (new ones prefixed by >>):
>
> In file included from include/trace/define_trace.h:95:0,
> from drivers/hv/hv_trace.h:29,
> from drivers/hv/hv_trace.c:4:
> include/trace/trace_events.h:759:13: warning: 'print_fmt_vmbus_hdr_msg' defined but not used [-Wunused-variable]
> static char print_fmt_##call[] = print; \
> ^
> >> drivers/hv/./hv_trace.h:9:1: note: in expansion of macro 'DECLARE_EVENT_CLASS'

Interesting. I'm wondering if this happens because the class "print" is
not used.

Actually, I think the default print should be used by one event, and
if other events need to be made that change the print, it should use
the DEFINE_EVENT_PRINT()

But this may be moot because I think Stephen Hemminger made it back to
DEFINE_EVENT()s.

-- Steve


> DECLARE_EVENT_CLASS(vmbus_hdr_msg,
> ^~~~~~~~~~~~~~~~~~~
> In file included from include/trace/define_trace.h:95:0,
> from drivers/hv/hv_trace.h:29,
> from drivers/hv/hv_trace.c:4:
> include/trace/trace_events.h:363:37: warning: 'trace_event_type_funcs_vmbus_hdr_msg' defined but not used [-Wunused-variable]
> static struct trace_event_functions trace_event_type_funcs_##call = { \
> ^
> >> drivers/hv/./hv_trace.h:9:1: note: in expansion of macro 'DECLARE_EVENT_CLASS'
> DECLARE_EVENT_CLASS(vmbus_hdr_msg,
> ^~~~~~~~~~~~~~~~~~~
>
> vim +/DECLARE_EVENT_CLASS +9 drivers/hv/./hv_trace.h
>
> 8
> > 9 DECLARE_EVENT_CLASS(vmbus_hdr_msg,
> 10 TP_PROTO(const struct vmbus_channel_message_header *hdr),
> 11 TP_ARGS(hdr),
> 12 TP_STRUCT__entry(__field(unsigned int, msgtype)),
> 13 TP_fast_assign(__entry->msgtype = hdr->msgtype;),
> 14 TP_printk("msgtype=%d", __entry->msgtype)
> 15 );
> 16
>
> ---
> 0-DAY kernel test infrastructure Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all Intel Corporation

\
 
 \ /
  Last update: 2017-10-04 02:08    [W:0.050 / U:0.664 seconds]
©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site