diff options
author | Jani Nikula <[email protected]> | 2023-07-11 14:02:14 +0300 |
---|---|---|
committer | Tvrtko Ursulin <[email protected]> | 2023-07-17 13:39:04 +0100 |
commit | 2c27770a7bc88ef7f6614d11d96d8e62017d0b78 (patch) | |
tree | 10b4b5d05ecda173ee10c3bb25bf07b4d7e381ff /lib/test_fortify/write_overflow-strlcpy-src.c | |
parent | 785b3f667b4bf98804cad135005e964df0c750de (diff) |
Revert "drm/i915: use localized __diag_ignore_all() instead of per file"
This reverts commit 88e9664434c994e97a9f6f8cdd1535495c660cea.
__diag_ignore_all() only works for GCC 8 or later.
-Woverride-init (from -Wextra, enabled in i915 Makefile) combined with
CONFIG_WERROR=y or W=e breaks the build for older GCC.
With i386_defconfig and x86_64_defconfig enabling CONFIG_WERROR=y by
default, we really need to roll back the change.
An alternative would be to disable -Woverride-init in the Makefile for
GCC <8, but the revert seems like the safest bet now.
Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/8768
Reported-by: John Garry <[email protected]>
References: https://lore.kernel.org/r/[email protected]
References: https://lore.kernel.org/r/[email protected]
Fixes: 88e9664434c9 ("drm/i915: use localized __diag_ignore_all() instead of per file")
Cc: Gustavo Sousa <[email protected]>
Cc: Joonas Lahtinen <[email protected]>
Cc: Rodrigo Vivi <[email protected]>
Cc: Tvrtko Ursulin <[email protected]>
Signed-off-by: Jani Nikula <[email protected]>
Reviewed-by: Gustavo Sousa <[email protected]>
Tested-by: John Garry <[email protected]>
Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
(cherry picked from commit 290d161045753240f2100b8f44660426ecc97be5)
Signed-off-by: Tvrtko Ursulin <[email protected]>
Diffstat (limited to 'lib/test_fortify/write_overflow-strlcpy-src.c')
0 files changed, 0 insertions, 0 deletions