From: Mark Brown <broonie@kernel.org>
To: Li Zhijian <zhijianx.li@intel.com>
Cc: kbuild-all@lists.01.org, kernel test robot <lkp@intel.com>,
Catalin Marinas <catalin.marinas@arm.com>,
Andrei Vagin <avagin@gmail.com>,
Vincenzo Frascino <vincenzo.frascino@arm.com>,
Will Deacon <will@kernel.org>,
linux-arm-kernel@lists.infradead.org
Subject: Re: [kbuild-all] Re: [PATCH 4/5] arm64: vdso: Add getcpu() implementation
Date: Mon, 8 Jun 2020 12:09:01 +0100 [thread overview]
Message-ID: <20200608110901.GA4593@sirena.org.uk> (raw)
In-Reply-To: <e1b2f86e-8eef-3266-8c87-93c6660576e8@intel.com>
[-- Attachment #1.1: Type: text/plain, Size: 1036 bytes --]
On Mon, Jun 08, 2020 at 03:46:57PM +0800, Li Zhijian wrote:
> On 6/6/20 12:35 AM, Mark Brown wrote:
> > > arch/arm64/kernel/vdso/vgettimeofday.c:9:5: warning: no previous prototype for '__kernel_clock_gettime' [-Wmissing-prototypes]
> > > 9 | int __kernel_clock_gettime(clockid_t clock,
> > > | ^~~~~~~~~~~~~~~~~~~~~~
> > I'm not seeing this here and what we're doing is in line with the
> > existing idiom as can be seen from the __kernel_clock_gettime() code
> > flagging the same thing. Possibly an old/outdated toolchain?
> Actually, the origin mail contained 4 warnings, and only one is *new*, as
> the mail mentioned that
> > All warnings (new ones prefixed by >>, old ones prefixed by <<):
> 0Day/LKP had marked the *new* ones prefixed by '>>'
My point here is that all the warnings, both old and new, look spurious
and the new warning comes from following the pattern that generates the
existing warnings. I'm not seeing any of those warnings locally or in
other build services like KernelCI.
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
[-- Attachment #2: Type: text/plain, Size: 176 bytes --]
_______________________________________________
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:[~2020-06-08 11:09 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-06-05 13:11 [PATCH 0/5] arm64: vdso: getcpu() support Mark Brown
2020-06-05 13:11 ` [PATCH 1/5] arm64: vdso: Provide a define when building the vDSO Mark Brown
2020-06-05 13:11 ` [PATCH 2/5] arm64: vdso: Add per-CPU data Mark Brown
2020-06-05 13:11 ` [PATCH 3/5] arm64: vdso: Initialise the per-CPU vDSO data Mark Brown
2020-06-05 13:11 ` [PATCH 4/5] arm64: vdso: Add getcpu() implementation Mark Brown
2020-06-05 16:11 ` kernel test robot
2020-06-05 16:35 ` Mark Brown
2020-06-08 7:46 ` [kbuild-all] " Li Zhijian
2020-06-08 11:09 ` Mark Brown [this message]
2020-06-08 14:20 ` Philip Li
2020-06-07 2:04 ` kernel test robot
2020-06-05 13:11 ` [PATCH 5/5] selftests: vdso: Support arm64 in getcpu() test Mark Brown
2020-06-17 18:25 ` [PATCH 0/5] arm64: vdso: getcpu() support Mark Brown
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=20200608110901.GA4593@sirena.org.uk \
--to=broonie@kernel.org \
--cc=avagin@gmail.com \
--cc=catalin.marinas@arm.com \
--cc=kbuild-all@lists.01.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=lkp@intel.com \
--cc=vincenzo.frascino@arm.com \
--cc=will@kernel.org \
--cc=zhijianx.li@intel.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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).