Rafael Vanoni wrote:

> Here's a patch fixing return values for the dtrace side of PowerTOP
> plus an all around header cleanup.
> 
> thanks
> Rafael
========================================================
diff -r 9b1c31702fc2 usr/src/cmd/powertop/cpufreq.c
--- a/usr/src/cmd/powertop/cpufreq.c    Tue Jul 15 10:09:25 2008 +0100
+++ b/usr/src/cmd/powertop/cpufreq.c    Tue Jul 15 12:13:57 2008 +0100
@@ -148,35 +147,35 @@
         * Setup DTrace to look for CPU frequency changes
         */
        if ((g_dtp = dtrace_open(DTRACE_VERSION, 0, &err)) == NULL) {
-               pt_error("cannot open dtrace library: %s\n",
+               pt_error("cpufreq.c : cannot open dtrace library: %s\n",
                    dtrace_errmsg(NULL, err));
-               return (-1);
+               return (-2);
        }
========================================================
why use (-2) to return here?

Thanks,
-Aubrey

Reply via email to