From: Nathan Chancellor <nathan@kernel.org> To: Chunhui Li <chunhui.li@mediatek.com> Cc: Petr Mladek <pmladek@suse.com>, Steven Rostedt <rostedt@goodmis.org>, John Ogness <john.ogness@linutronix.de>, Sergey Senozhatsky <senozhatsky@chromium.org>, Matthias Brugger <matthias.bgg@gmail.com>, AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, wsd_upstream@mediatek.com Subject: Re: [PATCH] printk: fix _entry_ptr build warning Date: Wed, 6 Mar 2024 12:40:20 -0700 [thread overview] Message-ID: <20240306194020.GA3711543@dev-arch.thelio-3990X> (raw) In-Reply-To: <20240306092647.16010-1-chunhui.li@mediatek.com> On Wed, Mar 06, 2024 at 05:26:47PM +0800, Chunhui Li wrote: > We build with Werror and suffer build error when > enable CONFIG_PRINTK_INDEX, such as > gfp.h:223:2: error: unused variable '_entry_ptr' > ratelimit.h:31:3: error: unused variable '_entry_ptr' > kallsyms.h:172:2: error: unused variable '_entry_ptr' > [-Werror,-Wunused-variable] > > Fix the warning by appending __attribute__((unused)). > > Signed-off-by: Chunhui Li <chunhui.li@mediatek.com> Are these warnings being emitted from an Android version of clang, specifically 18.0.0 with the build ID of 11209041 (you should be able to tell this from the "bid" field in the BUILD_INFO file in the clang toolchain folder)? This seems remarkably similar to https://github.com/ClangBuiltLinux/linux/issues/1977, which was a report using that version of Android clang. Ultimately, this warning is a bug in a clang change that was quickly found and reverted upstream https://github.com/llvm/llvm-project/commit/f0f395e00e2ec3f1f20ca9021d1554fde73d56c9 https://github.com/llvm/llvm-project/commit/cfa578cde0314935c6eb5d7fa19ec26390d431dd but unfortunately, Android picked a revision for the 11209041 build that was between the landing of the broken commit and its revert. It was quickly fixed with the 11368308 build https://android.googlesource.com/platform/prebuilts/clang/host/linux-x86/+/42d100b025eb5f4a41781348016c148e9e912cf9 so you should just be able to update to that version to resolve this. Cheers, Nathan > --- > include/linux/printk.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/linux/printk.h b/include/linux/printk.h > index 8ef499ab3c1e..749c1c4257f1 100644 > --- a/include/linux/printk.h > +++ b/include/linux/printk.h > @@ -392,7 +392,7 @@ struct pi_entry { > .level = __builtin_constant_p(_level) ? (_level) : NULL, \ > .subsys_fmt_prefix = _subsys_fmt_prefix,\ > }; \ > - static const struct pi_entry *_entry_ptr \ > + static const struct pi_entry *_entry_ptr __attribute__((unused)) \ > __used __section(".printk_index") = &_entry; \ > } \ > } while (0) > -- > 2.18.0 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
WARNING: multiple messages have this Message-ID (diff)
From: Nathan Chancellor <nathan@kernel.org> To: Chunhui Li <chunhui.li@mediatek.com> Cc: Petr Mladek <pmladek@suse.com>, Steven Rostedt <rostedt@goodmis.org>, John Ogness <john.ogness@linutronix.de>, Sergey Senozhatsky <senozhatsky@chromium.org>, Matthias Brugger <matthias.bgg@gmail.com>, AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, wsd_upstream@mediatek.com Subject: Re: [PATCH] printk: fix _entry_ptr build warning Date: Wed, 6 Mar 2024 12:40:20 -0700 [thread overview] Message-ID: <20240306194020.GA3711543@dev-arch.thelio-3990X> (raw) In-Reply-To: <20240306092647.16010-1-chunhui.li@mediatek.com> On Wed, Mar 06, 2024 at 05:26:47PM +0800, Chunhui Li wrote: > We build with Werror and suffer build error when > enable CONFIG_PRINTK_INDEX, such as > gfp.h:223:2: error: unused variable '_entry_ptr' > ratelimit.h:31:3: error: unused variable '_entry_ptr' > kallsyms.h:172:2: error: unused variable '_entry_ptr' > [-Werror,-Wunused-variable] > > Fix the warning by appending __attribute__((unused)). > > Signed-off-by: Chunhui Li <chunhui.li@mediatek.com> Are these warnings being emitted from an Android version of clang, specifically 18.0.0 with the build ID of 11209041 (you should be able to tell this from the "bid" field in the BUILD_INFO file in the clang toolchain folder)? This seems remarkably similar to https://github.com/ClangBuiltLinux/linux/issues/1977, which was a report using that version of Android clang. Ultimately, this warning is a bug in a clang change that was quickly found and reverted upstream https://github.com/llvm/llvm-project/commit/f0f395e00e2ec3f1f20ca9021d1554fde73d56c9 https://github.com/llvm/llvm-project/commit/cfa578cde0314935c6eb5d7fa19ec26390d431dd but unfortunately, Android picked a revision for the 11209041 build that was between the landing of the broken commit and its revert. It was quickly fixed with the 11368308 build https://android.googlesource.com/platform/prebuilts/clang/host/linux-x86/+/42d100b025eb5f4a41781348016c148e9e912cf9 so you should just be able to update to that version to resolve this. Cheers, Nathan > --- > include/linux/printk.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/linux/printk.h b/include/linux/printk.h > index 8ef499ab3c1e..749c1c4257f1 100644 > --- a/include/linux/printk.h > +++ b/include/linux/printk.h > @@ -392,7 +392,7 @@ struct pi_entry { > .level = __builtin_constant_p(_level) ? (_level) : NULL, \ > .subsys_fmt_prefix = _subsys_fmt_prefix,\ > }; \ > - static const struct pi_entry *_entry_ptr \ > + static const struct pi_entry *_entry_ptr __attribute__((unused)) \ > __used __section(".printk_index") = &_entry; \ > } \ > } while (0) > -- > 2.18.0 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2024-03-06 19:40 UTC|newest] Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top 2024-03-06 9:26 [PATCH] printk: fix _entry_ptr build warning Chunhui Li 2024-03-06 9:26 ` Chunhui Li 2024-03-06 19:40 ` Nathan Chancellor [this message] 2024-03-06 19:40 ` Nathan Chancellor 2024-03-07 8:46 ` Chunhui Li (李春辉) 2024-03-07 8:46 ` Chunhui Li (李春辉)
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20240306194020.GA3711543@dev-arch.thelio-3990X \ --to=nathan@kernel.org \ --cc=angelogioacchino.delregno@collabora.com \ --cc=chunhui.li@mediatek.com \ --cc=john.ogness@linutronix.de \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mediatek@lists.infradead.org \ --cc=matthias.bgg@gmail.com \ --cc=pmladek@suse.com \ --cc=rostedt@goodmis.org \ --cc=senozhatsky@chromium.org \ --cc=wsd_upstream@mediatek.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.