Re: [PATCH cgroup/for-4.15] sched/cputime: Add dummy cputime_adjust() implementation for CONFIG_VIRT_CPU_ACCOUNTING_NATIVE
From: Peter Zijlstra
Date: Tue Sep 26 2017 - 05:10:24 EST
On Mon, Sep 25, 2017 at 02:34:21PM -0700, Tejun Heo wrote:
> Hello,
>
> Another follow-up fix. I missed a dummy implementation leading to
> bulid failure on s390. Because the bug and fix are trivial, I
> directly applied the patch to cgroup/for-4.15. Please let me know if
> it should be routed differently.
>
> Thanks.
> ------ 8< ------
> From 8157a7faf94135386bf04b1cf94e6efd3fb94702 Mon Sep 17 00:00:00 2001
> From: Tejun Heo <tj@xxxxxxxxxx>
> Date: Mon, 25 Sep 2017 14:27:54 -0700
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit
>
> cfb766da54d9 ("sched/cputime: Expose cputime_adjust()") made
> cputime_adjust() public for cgroup basic cpu stat support; however,
> the commit forgot to add a dummy implementaiton for
> CONFIG_VIRT_CPU_ACCOUNTING_NATIVE leading to compiler errors on some
> s390 configurations.
>
> Fix it by adding the missing dummy implementation.
>
> Reported-by: âkbuild-all@xxxxxxâ <kbuild-all@xxxxxx>
> Fixes: cfb766da54d9 ("sched/cputime: Expose cputime_adjust()")
> Signed-off-by: Tejun Heo <tj@xxxxxxxxxx>
Seems fine,
Acked-by: Peter Zijlstra (Intel) <peterz@xxxxxxxxxxxxx>