linux-next: build failure after merge of the tip tree

From: Stephen Rothwell
Date: Wed Nov 01 2017 - 06:51:59 EST


Hi all,

After merging the tip tree, today's linux-next build (x86_64 allmodconfig)
failed like this:

In file included from include/linux/workqueue.h:8:0,
from include/linux/srcu.h:34,
from include/linux/notifier.h:15,
from include/linux/memory_hotplug.h:6,
from include/linux/mmzone.h:779,
from include/linux/gfp.h:5,
from include/linux/umh.h:4,
from include/linux/kmod.h:22,
from include/linux/module.h:13,
from drivers/net/ethernet/chelsio/cxgb/common.h:43,
from drivers/net/ethernet/chelsio/cxgb/sge.c:39:
drivers/net/ethernet/chelsio/cxgb/sge.c: In function 't1_sge_create':
include/linux/timer.h:176:24: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
__setup_timer(timer, (TIMER_FUNC_TYPE)callback, \
^
include/linux/timer.h:122:25: note: in definition of macro '__setup_timer'
(_timer)->function = (_fn); \
^
drivers/net/ethernet/chelsio/cxgb/sge.c:2081:3: note: in expansion of macro 'timer_setup'
timer_setup(&sge->espibug_timer,
^
drivers/net/ethernet/chelsio/cxgb/sge.c:2082:31: warning: comparison between pointer and integer
adapter->params.nports > 1 ? espibug_workaround_t204 : espibug_workaround,
^
include/linux/timer.h:122:25: note: in definition of macro '__setup_timer'
(_timer)->function = (_fn); \
^
drivers/net/ethernet/chelsio/cxgb/sge.c:2081:3: note: in expansion of macro 'timer_setup'
timer_setup(&sge->espibug_timer,
^
include/linux/timer.h:122:22: error: assignment from incompatible pointer type [-Werror=incompatible-pointer-types]
(_timer)->function = (_fn); \
^
include/linux/timer.h:176:3: note: in expansion of macro '__setup_timer'
__setup_timer(timer, (TIMER_FUNC_TYPE)callback, \
^
drivers/net/ethernet/chelsio/cxgb/sge.c:2081:3: note: in expansion of macro 'timer_setup'
timer_setup(&sge->espibug_timer,
^

and another similar.

Caused by commit

cacd2b3fb981 ("chelsio: Convert timers to use timer_setup()")

from te net-next tree interacting with commit

52f737c2da40 ("timer: Provide wrappers safe for use with LOCKDEP")

from the tip tree.

Looks like the LOCKDEP sage version of timer_setup() needs to put
parentheses around the usages of its parameters?

--
Cheers,
Stephen Rothwell