Re: [RFT][PATCH v2 0/2] PM / QoS: Device resume latency framework fix

From: Tero Kristo
Date: Wed Nov 08 2017 - 04:11:10 EST


On 07/11/17 03:08, Rafael J. Wysocki wrote:
On Monday, November 6, 2017 2:46:48 PM CET Geert Uytterhoeven wrote:
Hi Rafael,

On Fri, Nov 3, 2017 at 12:42 PM, Rafael J. Wysocki <rjw@xxxxxxxxxxxxx> wrote:
On Thursday, November 2, 2017 12:00:27 AM CET Rafael J. Wysocki wrote:
This series is a replacement for commit 0cc2b4e5a020 (PM / QoS: Fix device
resume latency PM QoS) that had to be reverted due to problems introduced by it.

This time the genpd PM QoS governor is first updated to be more consistent
and the PM QoS changes are made on top of that which simplifies the second
patch quite a bit.

This is based on the linux-next branch from linux-pm.git as of now (should
also apply to the current mainline just fine).

Please test if you can or let me know if you have any comments.

The v2 removes a couple of redundant checks from the first patch (and add
comments to explain why the checks are not needed) and fixes up the
"no constraint" value collision with a valid constraint multiplied by
NSEC_PER_USEC in the second patch.

Please test if possible and let me know about any issues.

With this series, the 3 issues I reported before do not happen.

Tested-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>

Thank you.

Thanks!

Rafael


The latest patches (#1 v3 and #2 v4) seem to work fine in my sanity tests also.

Tested-by: Tero Kristo <t-kristo@xxxxxx>
--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki