From mboxrd@z Thu Jan  1 00:00:00 1970
Content-Type: multipart/mixed; boundary="===============8591872113379131027=="
MIME-Version: 1.0
From: Joel Fernandes <joel@joelfernandes.org>
To: kbuild-all@lists.01.org
Subject: Re: [peterz-queue:sched/core-sched 4/12] kernel/sched/core.c:123:6: warning: no previous prototype for 'sched_core_get'
Date: Tue, 24 Nov 2020 15:52:57 -0500
Message-ID: <20201124205257.GC812262@google.com>
In-Reply-To: <20201124204302.GA1941016@google.com>
List-Id: <oe-kbuild-all.lists.linux.dev>

--===============8591872113379131027==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable

On Tue, Nov 24, 2020 at 03:43:02PM -0500, Joel Fernandes wrote:
> On Wed, Nov 25, 2020 at 03:22:48AM +0800, kernel test robot wrote:
> > tree:   https://git.kernel.org/pub/scm/linux/kernel/git/peterz/queue.gi=
t sched/core-sched
> > head:   7b55b5ef19fbadf111b8d60d320437bfc0467a79
> > commit: 636f9841873bff3684ee81155804b796d58408a4 [4/12] sched: Core-wid=
e rq->lock
> > config: powerpc-allyesconfig (attached as .config)
> > compiler: powerpc64-linux-gcc (GCC) 9.3.0
> > reproduce (this is a W=3D1 build):
> >         wget https://raw.githubusercontent.com/intel/lkp-tests/master/s=
bin/make.cross -O ~/bin/make.cross
> >         chmod +x ~/bin/make.cross
> >         # https://git.kernel.org/pub/scm/linux/kernel/git/peterz/queue.=
git/commit/?id=3D636f9841873bff3684ee81155804b796d58408a4
> >         git remote add peterz-queue https://git.kernel.org/pub/scm/linu=
x/kernel/git/peterz/queue.git
> >         git fetch --no-tags peterz-queue sched/core-sched
> >         git checkout 636f9841873bff3684ee81155804b796d58408a4
> >         # save the attached .config to linux build tree
> >         COMPILER_INSTALL_PATH=3D$HOME/0day COMPILER=3Dgcc-9.3.0 make.cr=
oss ARCH=3Dpowerpc =

> > =

> > If you fix the issue, kindly add following tag as appropriate
> > Reported-by: kernel test robot <lkp@intel.com>
> > =

> > All warnings (new ones prefixed by >>):
> > =

> > >> kernel/sched/core.c:123:6: warning: no previous prototype for 'sched=
_core_get' [-Wmissing-prototypes]
> >      123 | void sched_core_get(void)
> >          |      ^~~~~~~~~~~~~~
> > >> kernel/sched/core.c:131:6: warning: no previous prototype for 'sched=
_core_put' [-Wmissing-prototypes]
> >      131 | void sched_core_put(void)
> >          |      ^~~~~~~~~~~~~~
> >    kernel/sched/core.c:2900:6: warning: no previous prototype for 'sche=
d_set_stop_task' [-Wmissing-prototypes]
> >     2900 | void sched_set_stop_task(int cpu, struct task_struct *stop)
> >          |      ^~~~~~~~~~~~~~~~~~~
> =

> At least this sched_set_stop_task one is not a recently introduced warnin=
g.
> =

> This prototype warning does seem bogus though. Do we fix it? Obvious quick
> fix is the declar sched_core_{get,put}() in sched.h to pacify the compile=
r.

BTW, I did add it to my tree already in:
93de21e503a3 ("sched: Move core-scheduler interfacing code to a new file")
https://git.kernel.org/pub/scm/linux/kernel/git/jfern/linux.git/log/?h=3Dco=
resched

Until the interface patches are in, we could define sched_core_get/put with
"__maybe_unused static" to silence the warnings.

thanks,

 - Joel

--===============8591872113379131027==--