On Fri, Jan 25, 2019 at 11:34 PM Nick Desaulniers ndesaulniers@google.com wrote:
On Fri, Jan 25, 2019 at 11:13 PM Nathan Chancellor natechancellor@gmail.com wrote:
This warning is disabled by default in scripts/Makefile.extrawarn when W= is not provided but this Makefile adds -Wall after this warning is disabled so it shows up in the build when it shouldn't:
In file included from drivers/gpu/drm/i915/intel_breadcrumbs.c:895: drivers/gpu/drm/i915/selftests/intel_breadcrumbs.c:350:34: error: variable 'wq' is uninitialized when used within its own initialization [-Werror,-Wuninitialized] DECLARE_WAIT_QUEUE_HEAD_ONSTACK(wq); ^~ ./include/linux/wait.h:74:63: note: expanded from macro 'DECLARE_WAIT_QUEUE_HEAD_ONSTACK' struct wait_queue_head name = __WAIT_QUEUE_HEAD_INIT_ONSTACK(name) ~~~~ ^~~~ ./include/linux/wait.h:72:33: note: expanded from macro '__WAIT_QUEUE_HEAD_INIT_ONSTACK' ({ init_waitqueue_head(&name); name; }) ^~~~ 1 error generated.
Explicitly disable the warning like commit 46e2068081e9 ("drm/i915: Disable some extra clang warnings").
Link: https://github.com/ClangBuiltLinux/linux/issues/220 Signed-off-by: Nathan Chancellor natechancellor@gmail.com
Reviewed-by: Nick Desaulniers nick.desaulniers@gmail.com
gah! I am on my work device, in that case it should be: Reviewed-by: Nick Desaulniers ndesaulniers@google.com
probably could give Chris Wilson the suggested by tag. https://lore.kernel.org/lkml/154513398652.1108.7150969916024071452@skylake-a...
v1 -> v2:
- Rather than disable the warning for the problematic folder, disable it for the entire folder like Matthias's commit.
Thanks for following up with a v2.
-- Thanks, ~Nick Desaulniers