Messages in this thread | | | From | "Chen, Yu C" <> | Subject | RE: [PATCH 2/2][RFC] tools/power turbostat: Introduce reliable RAPL display | Date | Fri, 14 Aug 2020 14:34:01 +0000 |
| |
Hi Len, > From: Len Brown <lenb@kernel.org> > Sent: Friday, August 14, 2020 5:51 AM > To: Chen, Yu C <yu.c.chen@intel.com> > Cc: Linux PM list <linux-pm@vger.kernel.org>; Linux Kernel Mailing List <linux- > kernel@vger.kernel.org>; Zhang, Rui <rui.zhang@intel.com> > Subject: Re: [PATCH 2/2][RFC] tools/power turbostat: Introduce reliable RAPL > display > > why not simply use nanosleep(2) > > Do you mean, use nanosleep rather than the timer to accumulate the RAPL data? After thinking for a while, it looks like if we use nanosleep we might need to create a new thread within the turbostat and sleep every few seconds (according to the RAPL register timeout) to accumulate the running RAPL. And might need to deal with some race conditions between new thread and the main turbostat thread. But yes, it can be switched to nanosleep() to check if the code would look simpler.
BTW, we have a v3 of the patch at https://lore.kernel.org/patchwork/project/lkml/list/?series=439330
Thanks, Chenyu
> -- > Len Brown, Intel Open Source Technology Center
| |