From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Lezcano Subject: Re: [PATCH V10] powercap/drivers/idle_inject: Add an idle injection framework Date: Tue, 26 Jun 2018 13:02:10 +0200 Message-ID: <7d9403db-e26e-619e-c0d4-e601b6d51a78@linaro.org> References: <1530010416-30692-1-git-send-email-daniel.lezcano@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: In-Reply-To: <1530010416-30692-1-git-send-email-daniel.lezcano@linaro.org> Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: rjw@rjwysocki.net Cc: linux-kernel@vger.kernel.org, Viresh Kumar , Eduardo Valentin , Javi Merino , Leo Yan , Kevin Wangtao , Vincent Guittot , Rui Zhang , Daniel Thompson , Peter Zijlstra , Andrea Parri , "open list:POWER MANAGEMENT CORE" List-Id: linux-pm@vger.kernel.org On 26/06/2018 12:53, Daniel Lezcano wrote: > Initially, the cpu_cooling device for ARM was changed by adding a new > policy inserting idle cycles. The intel_powerclamp driver does a > similar action. > > Instead of implementing idle injections privately in the cpu_cooling > device, move the idle injection code in a dedicated framework and give > the opportunity to other frameworks to make use of it. > > The framework relies on the smpboot kthreads which handles via its > main loop the common code for hotplugging and [un]parking. > > This code was previously tested with the cpu cooling device and went > through several iterations. It results now in split code and API > exported in the header file. It was tested with the cpu cooling device > with success. > > Signed-off-by: Daniel Lezcano > Cc: Viresh Kumar > Cc: Eduardo Valentin > Cc: Javi Merino > Cc: Leo Yan > Cc: Kevin Wangtao > Cc: Vincent Guittot > Cc: Rui Zhang > Cc: Daniel Thompson > Cc: Peter Zijlstra > Cc: Andrea Parri > --- > > V10: > - Remove set_freezable (Rafael J. Wysocki) > - Reduce the prefixed names 'injection' by 'inject" (Rafael J. Wysocki) > - Rename 'idle_injection_wakeup_fn()' to 'idle_inject_timer_fn()' (Rafael J. Wysocki) > - Shorten label 'out_rollback_per_cpu' to 'out_rollback' (Rafael J. Wysocki) According to these changes I updated : https://git.linaro.org/people/daniel.lezcano/linux.git thermal/idle-inject-v4 -- Daniel -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog