Re: [PATCH net-next] netdevsim: Fix build error without CONFIG_INET
From: Jakub Kicinski
Date: Mon Aug 19 2019 - 17:59:10 EST
On Mon, 19 Aug 2019 20:08:25 +0800, YueHaibing wrote:
> If CONFIG_INET is not set, building fails:
> drivers/net/netdevsim/dev.o: In function `nsim_dev_trap_report_work':
> dev.c:(.text+0x67b): undefined reference to `ip_send_check'
> Add CONFIG_INET Kconfig dependency to fix this.
> Reported-by: Hulk Robot <hulkci@xxxxxxxxxx>
> Fixes: da58f90f11f5 ("netdevsim: Add devlink-trap support")
> Signed-off-by: YueHaibing <yuehaibing@xxxxxxxxxx>
Hmm.. I'd rather the test module did not have hard dependencies on
marginally important config options. We have done a pretty good job
so far limiting the requirements though separating the code out at
compilation object level. The more tests depend on netdevsim and the
more bots we have running tests against randconfig - the more important
This missing reference here is for calculating a checksum over a
constant header.. could we perhaps just hard code the checksum?