From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Date: Tue, 05 May 2020 14:03:52 +0000 Subject: [PATCH] omapfb: don't annotate dss_conv_list as __initdata Message-Id: <20200505140407.373983-1-arnd@arndb.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Bartlomiej Zolnierkiewicz , Marco Elver Cc: linux-fbdev@vger.kernel.org, Arnd Bergmann , "Paul E. McKenney" , clang-built-linux@googlegroups.com, Greg Kroah-Hartman , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Alexios Zavras , Thomas Gleixner , linux-omap@vger.kernel.org, Enrico Weigelt , Allison Randal With the kcsan changes, __read_once_size() is not inlined, but clang can decide to emit a version that hardcodes the address, which in turn triggers a warning for dss_conv_list being __initdata but __read_once_size() not being __init: WARNING: modpost: vmlinux.o(.text+0x6e4d7a): Section mismatch in reference from the function __read_once_size() to the variable .init.data:dss_conv_list The function __read_once_size() references the variable __initdata dss_conv_list. This is often because __read_once_size lacks a __initdata annotation or the annotation of dss_conv_list is wrong. This is clearly a false positive warning, but it's hard to tell who is to blame for it. Work around it by removing the __initdata annotation, wasting the space of two pointers in return for getting rid of the warning. Fixes: dfd402a4c4ba ("kcsan: Add Kernel Concurrency Sanitizer infrastructure") Signed-off-by: Arnd Bergmann --- drivers/video/fbdev/omap2/omapfb/dss/omapdss-boot-init.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/video/fbdev/omap2/omapfb/dss/omapdss-boot-init.c b/drivers/video/fbdev/omap2/omapfb/dss/omapdss-boot-init.c index 0ae0cab252d3..29fdff9c95f6 100644 --- a/drivers/video/fbdev/omap2/omapfb/dss/omapdss-boot-init.c +++ b/drivers/video/fbdev/omap2/omapfb/dss/omapdss-boot-init.c @@ -19,7 +19,7 @@ #include #include -static struct list_head dss_conv_list __initdata; +static struct list_head dss_conv_list; static const char prefix[] __initconst = "omapdss,"; -- 2.26.0