Re: [kbuild-all] [tip: timers/core] posix-cpu-timers: Use common permission check in posix_cpu_clock_get()

From: Philip Li
Date: Thu Aug 29 2019 - 21:04:39 EST


On Thu, Aug 29, 2019 at 01:21:59PM +0200, Thomas Gleixner wrote:
> On Thu, 29 Aug 2019, kbuild test robot wrote:
> > Thank you for the patch! Yet something to improve:
> >
> > [auto build test ERROR on linus/master]
> > [cannot apply to v5.3-rc6 next-20190828]
> > [if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
>
> I have no idea what you are testing there.
oops, looks we have missed some info in report. We actually test patches
from mailing list we monitor.
>
> > kernel/time/posix-cpu-timers.c: In function 'posix_cpu_clock_get':
> > >> kernel/time/posix-cpu-timers.c:275:8: error: implicit declaration of function 'get_task_for_clock'; did you mean 'get_task_struct'? [-Werror=implicit-function-declaration]
> > tsk = get_task_for_clock(clock);
> > ^~~~~~~~~~~~~~~~~~
> > get_task_struct
> > >> kernel/time/posix-cpu-timers.c:275:6: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
> > tsk = get_task_for_clock(clock);
> > ^
> > cc1: some warnings being treated as errors
>
> That commit comes _after_ the commit which introduced the function and
> get_task_for_clock() is defined above posix_cpu_clock_get(), so I assume
> you missed to apply the commit on which this depends on.
thanks for info, currently the bot can't figure out the dependency if two
patches are not in one series, or we didn't find the right base repo to
apply, which we will improve continuously.

>
> Thanks,
>
> tglx
> _______________________________________________
> kbuild-all mailing list
> kbuild-all@xxxxxxxxxxxx
> https://lists.01.org/mailman/listinfo/kbuild-all