[dpdk-dev] [PATCH v2 2/7] drivers/net/e1000: Fix missing brackets
Panu Matilainen
pmatilai at redhat.com
Thu Mar 24 07:54:30 CET 2016
On 03/24/2016 02:35 AM, Lu, Wenzhuo wrote:
> Hi Thomas, Aaron,
>>> Wenzhuo, do you agree to fix the base driver here?
> Honestly I find the logic has some problem and maybe more changes needed. I'm talking with our kernel driver contactors to make sure they have no concern about my idea.
> And Aaron, do we really need to fix this code? For I find this function is not called. So it has no real impact to us. Could we just wait until kernel driver fixes it and leverage the new version? Thanks.
It breaks the build with gcc >= 6.0 due to -Werror so yes we need to fix
it, one way or the other.
As spelled out in an earlier mail
(http://dpdk.org/ml/archives/dev/2016-March/034290.html), for somebody
building with gcc >= 6.0 the options basically are:
1) disable the driver entirely
2) build with -Wno-error
3) paper over it with local warning disablers
4) patch the thing locally
If the offending function really is not used at all in DPDK context then
3) is an entirely valid option for an upstream solution, ie something
like (untested)
--- a/drivers/net/e1000/Makefile
+++ b/drivers/net/e1000/Makefile
@@ -54,6 +54,9 @@ else
#
CFLAGS_BASE_DRIVER = -Wno-uninitialized -Wno-unused-parameter
CFLAGS_BASE_DRIVER += -Wno-unused-variable
+ifeq ($(shell test $(GCC_VERSION) -ge 60 && echo 1), 1)
+CFLAGS_BASE_DRIVER += -Wno-misleading-indentation
+endif
endif
#
- Panu -
More information about the dev
mailing list