Re: [PATCH 0/3] cpuidle: small improvements & fixes for menu governor

From: Rafael J. Wysocki
Date: Wed Nov 04 2015 - 09:03:19 EST

On 11/4/2015 7:56 AM, Joe Perches wrote:
On Wed, 2015-11-04 at 00:03 +0100, Rafael J. Wysocki wrote:
On 11/3/2015 11:35 PM, Rik van Riel wrote:
On 11/03/2015 05:05 PM, Rafael J. Wysocki wrote:
On 10/28/2015 11:46 PM, riel@xxxxxxxxxx wrote:
While working on a paravirt cpuidle driver for KVM guests, I
noticed a number of small logic errors in the menu governor

These patches should get rid of some artifacts that can break
the logic in the menu governor under certain corner cases, and
make idle state selection work better on CPUs with long C1 exit

I have not seen any adverse effects with them in my (quick)
tests. As expected, they do not seem to do much on systems with
many power states and very low C1 exit latencies and target

Sorry for the trouble, but can you please resend the series with
CCs to
linux-pm@xxxxxxxxxxxxxxx? That will make it way easier to handle
for me.
Not a problem. Done.

What change do I need to send in to ensure that the
linux-pm mailing list shows up in

$ ./scripts/ -f drivers/cpuidle/governors/menu.c
"Rafael J. Wysocki" <rafael.j.wysocki@xxxxxxxxx>
Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>
Rik van Riel <riel@xxxxxxxxxx>
Len Brown <len.brown@xxxxxxxxx>
"Peter Zijlstra (Intel)" <peterz@xxxxxxxxxxxxx>
Javi Merino <javi.merino@xxxxxxx>
linux-kernel@xxxxxxxxxxxxxxx (open list)

I'm not sure why it doesn't show up in there.
If you look at MAINTAINERS under CPUIDLE DRIVERS, linux-pm is
actually listed there.
Because the pattern is just the files in the top level directory
of drivers/cpuidle and not any files in any directory below

From the MAINTAINERS pattern descriptions:

F: Files and directories with wildcard patterns.
A trailing slash includes all files and subdirectory files.
F: drivers/net/ all files in and below
F: drivers/net/* all files in drivers/net, but
not below

Maybe the answer is that needs to be fixed ...
Nope, the MAINTAINERS - CPUIDLE DRIVERS entry does though.

OK, I see.

Care to send this patch with a proper changelog/sign-off?

1 file changed, 1 insertion(+), 1 deletion(-)

index 77ed3a0..8bd5c7e 100644
@@ -3004,7 +3004,7 @@ M: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>
L: linux-pm@xxxxxxxxxxxxxxx
S: Maintained
T: git git://
-F: drivers/cpuidle/*
+F: drivers/cpuidle/
F: include/linux/cpuidle.h

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at