Mohammad, This fix should go upstream. cc'ing the powertop list. Regards, Amit On Thu, Jun 26, 2014 at 12:27 PM, Mohammad Merajul Islam Molla <meraj.enigma(a)gmail.com&gt; wrote: > powertop creates temporary file in /tmp folder during calibration. > Simply closing does not remove these files. > Instead use unlink to remove the file. > > diff --git a/src/calibrate/calibrate.cpp b/src/calibrate/calibrate.cpp > index db368e0..1336c3f 100644 > --- a/src/calibrate/calibrate.cpp > +++ b/src/calibrate/calibrate.cpp > @@ -242,7 +242,7 @@ static void *burn_disk(void *dummy) > write(fd, buffer, 64*1024); > fdatasync(fd); > } > - close(fd); > + unlink(filename); > return NULL; > } > > > > -- > Thanks, > -Meraj > > _______________________________________________ > linaro-dev mailing list > linaro-dev(a)lists.linaro.org > http://lists.linaro.org/mailman/listinfo/linaro-dev _______________________________________________ PowerTop mailing list PowerTop(a)lists.01.org https://lists.01.org/mailman/listinfo/powertop

Hi Alexandra, Sorry for late reply due to holidays. I have resent the patches. Please check. -- Thanks, -Meraj On Tue, Jul 29, 2014 at 12:13 AM, Alexandra Yates <alexandra.yates(a)linux.intel.com&gt; wrote: > Hi Mohammad and Amit, > >> Mohammad, >> >> This fix should go upstream. cc'ing the powertop list. >> >> Regards, >> Amit >> >> On Thu, Jun 26, 2014 at 12:27 PM, Mohammad Merajul Islam Molla >> <meraj.enigma(a)gmail.com&gt; wrote: >>> powertop creates temporary file in /tmp folder during calibration. >>> Simply closing does not remove these files. >>> Instead use unlink to remove the file. >>> >>> diff --git a/src/calibrate/calibrate.cpp b/src/calibrate/calibrate.cpp >>> index db368e0..1336c3f 100644 >>> --- a/src/calibrate/calibrate.cpp >>> +++ b/src/calibrate/calibrate.cpp >>> @@ -242,7 +242,7 @@ static void *burn_disk(void *dummy) >>> write(fd, buffer, 64*1024); >>> fdatasync(fd); >>> } >>> - close(fd); >>> + unlink(filename); >>> return NULL; >>> } >>> >>> >>> >>> -- >>> Thanks, >>> -Meraj >>> >>> _______________________________________________ >>> linaro-dev mailing list >>> linaro-dev(a)lists.linaro.org >>> http://lists.linaro.org/mailman/listinfo/linaro-dev >> _______________________________________________ >> PowerTop mailing list >> PowerTop(a)lists.01.org >> https://lists.01.org/mailman/listinfo/powertop >> > > Please send your patch against the tip of the tree. I'm having > difficulties applying it. Here is the error message: > > Patch is empty. Was it split wrong? > If you would prefer to skip this patch, instead run "git am --skip". > To restore the original branch and stop patching run "git am --abort". > > Thank you, > Alexandra.