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

Open Source and information security mailing list archives

Message-ID: <20190830010856.GE857@intel.com>
Date:   Fri, 30 Aug 2019 09:08:56 +0800
From:   Philip Li <philip.li@...el.com>
To:     Thomas Gleixner <tglx@...utronix.de>
Cc:     kbuild test robot <lkp@...el.com>,
        linux-tip-commits@...r.kernel.org,
        Frederic Weisbecker <frederic@...nel.org>,
        linux-kernel@...r.kernel.org,
        tip-bot2 for Thomas Gleixner <tip-bot2@...utronix.de>,
        Borislav Petkov <bp@...en8.de>, kbuild-all@...org,
        Ingo Molnar <mingo@...nel.org>
Subject: Re: [kbuild-all] [tip: timers/core] posix-cpu-timers: Use common
 permission check in posix_cpu_clock_get()

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@...ts.01.org
> https://lists.01.org/mailman/listinfo/kbuild-all

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives