From mboxrd@z Thu Jan  1 00:00:00 1970
Content-Type: multipart/mixed; boundary="===============8171880709903856719=="
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:43:02 -0500
Message-ID: <20201124204302.GA1941016@google.com>
In-Reply-To: <202011250343.EPLZDqfG-lkp@intel.com>
List-Id: <oe-kbuild-all.lists.linux.dev>

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

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.git =
sched/core-sched
> head:   7b55b5ef19fbadf111b8d60d320437bfc0467a79
> commit: 636f9841873bff3684ee81155804b796d58408a4 [4/12] sched: Core-wide =
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/sbi=
n/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         # https://git.kernel.org/pub/scm/linux/kernel/git/peterz/queue.gi=
t/commit/?id=3D636f9841873bff3684ee81155804b796d58408a4
>         git remote add peterz-queue https://git.kernel.org/pub/scm/linux/=
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.cros=
s 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_c=
ore_get' [-Wmissing-prototypes]
>      123 | void sched_core_get(void)
>          |      ^~~~~~~~~~~~~~
> >> kernel/sched/core.c:131:6: warning: no previous prototype for 'sched_c=
ore_put' [-Wmissing-prototypes]
>      131 | void sched_core_put(void)
>          |      ^~~~~~~~~~~~~~
>    kernel/sched/core.c:2900:6: warning: no previous prototype for 'sched_=
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 warning.

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 compiler.

thanks,

 - Joel

--===============8171880709903856719==--