Alan, Patch is applied and tested, works perfectly.
Thanks again, -Ben -----Original Message----- >From: "D. Ben Noble" <[email protected]> >Sent: Jan 24, 2009 11:56 PM >To: Alan Jenkins <[email protected]> >Cc: [email protected], [email protected] >Subject: Re: [PATCH] fix dump mode to not read from stdin > >Alan, > >Thanks for the additional investigation and subsequent patch. Is this patch >going to be implemented in a forthcoming version of PowerTOP? > >Regards, > >-Ben > >On Sat, Jan 24, 2009 at 10:07 AM, Alan Jenkins <[email protected]> >wrote: >> This allows powertop to be run non-interactively, e.g. from cron, >> as requested by D. Ben Noble. Without this fix, it exits immediately. >> >> Index: powertop.c >> =================================================================== >> --- powertop.c (revision 317) >> +++ powertop.c (working copy) >> @@ -858,7 +858,8 @@ >> >> >> FD_ZERO(&rfds); >> - FD_SET(0, &rfds); >> + if (!dump) >> + FD_SET(0, &rfds); >> tv.tv_sec = ticktime; >> tv.tv_usec = (ticktime - tv.tv_sec) * 1000000;; >> do_proc_irq(); >> >> >> _______________________________________________ Power mailing list [email protected] http://www.bughost.org/mailman/listinfo/power
