When we set perf policy to high, we say "high".
So when we set to low, say "low", not "manual".
(Both manual high and manual low are manual.)
And it's not "client reply", it's a reply to the client, right?
Jan
Index: apmd.c
===================================================================
RCS file: /cvs/src/usr.sbin/apmd/apmd.c,v
retrieving revision 1.77
diff -u -p -r1.77 apmd.c
--- apmd.c 11 Oct 2015 20:23:49 -0000 1.77
+++ apmd.c 16 Nov 2015 17:22:16 -0000
@@ -277,7 +277,7 @@ handle_client(int sock_fd, int ctl_fd)
case SETPERF_LOW:
doperf = PERF_MANUAL;
reply.newstate = NORMAL;
- syslog(LOG_NOTICE, "setting hw.perfpolicy to manual");
+ syslog(LOG_NOTICE, "setting hw.perfpolicy to low");
setperfpolicy("low");
break;
case SETPERF_HIGH:
@@ -305,7 +305,7 @@ handle_client(int sock_fd, int ctl_fd)
reply.perfmode = doperf;
reply.vno = APMD_VNO;
if (send(cli_fd, &reply, sizeof(reply), 0) != sizeof(reply))
- syslog(LOG_INFO, "client reply botch");
+ syslog(LOG_INFO, "reply to client botched");
close(cli_fd);
return reply.newstate;
