all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#41150] [PATCH] gnu: Add powerstat.
@ 2020-05-09 15:15 Tobias Geerinckx-Rice via Guix-patches via
  2020-05-09 17:01 ` Mathieu Othacehe
  0 siblings, 1 reply; 8+ messages in thread
From: Tobias Geerinckx-Rice via Guix-patches via @ 2020-05-09 15:15 UTC (permalink / raw)
  To: 41150

* gnu/packages/linux.scm (powerstat): New public variable.
---
 gnu/packages/linux.scm | 44 ++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 44 insertions(+)

diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm
index d2bffe8d8c..0d558f0277 100644
--- a/gnu/packages/linux.scm
+++ b/gnu/packages/linux.scm
@@ -1195,7 +1195,50 @@ at login.  Local and dynamic reconfiguration are its key features.")
 ;;; Miscellaneous.
 ;;;
 
+(define-public powerstat
+  (package
+    (name "powerstat")
+    (version "0.02.22")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append "https://kernel.ubuntu.com/~cking/tarballs/"
+                           "powerstat/powerstat-" version ".tar.gz"))
+       (sha256
+        (base32 "0r355b9syqa2nhfy8ksvxyy5d58v0isf983842js091s6liy0x7g"))))
+    (build-system gnu-build-system)
+    (arguments
+     `(#:make-flags
+       (list "CC=gcc"
+             (string-append "prefix=" (assoc-ref %outputs "out")))
+       #:tests? #f                      ; no test suite
+       #:phases
+       (modify-phases %standard-phases
+         (add-after 'unpack 'respect-$prefix
+           ;; https://bugs.launchpad.net/ubuntu/+source/powerstat/+bug/1877744
+           (lambda _
+             (substitute* "Makefile"
+               (("DIR=/usr/") "DIR=$(prefix)/"))
+             #t))
+         (delete 'configure))))         ; no configure script
+    (home-page "https://kernel.ubuntu.com/~cking/powerstat/")
+    (synopsis "Measure system power consumption")
+    (description
+     "Powerstat measures and reports your computer's power consumption in real
+time.  On mobile PCs, it uses ACPI battery information to measure the power
+drain of the entire system.
+
+Powerstat can also report @acronym{RAPL, Running Average Power Limit} power
+domain measurements.  These are available only on some hardware such as Intel
+Sandybridge and newer, and cover only part of the machine's components such as
+CPU, DRAM, and graphics.  However, they provide accurate and immediate readings
+and don't require a battery at all.
+
+The output is like @command{vmstat} but also shows power consumption statistics:
+at the end of a run, @command{powerstat} will calculate the average, standard
+deviation, and minimum and maximum values.  It can show a nice histogram too.")
+    (license license:gpl2)))
+
 (define-public psmisc
   (package
     (name "psmisc")
-- 
2.26.2





^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [bug#41150] [PATCH] gnu: Add powerstat.
  2020-05-09 15:15 [bug#41150] [PATCH] gnu: Add powerstat Tobias Geerinckx-Rice via Guix-patches via
@ 2020-05-09 17:01 ` Mathieu Othacehe
  2020-05-09 17:32   ` Tobias Geerinckx-Rice via Guix-patches via
  2020-05-09 20:42   ` bug#41150: " Tobias Geerinckx-Rice via Guix-patches via
  0 siblings, 2 replies; 8+ messages in thread
From: Mathieu Othacehe @ 2020-05-09 17:01 UTC (permalink / raw)
  To: 41150; +Cc: me


Hey Tobias,

> +    (arguments
> +     `(#:make-flags
> +       (list "CC=gcc"

Hard-coding CC this way will prevent cross-compilation. You can have a
look to "crda" or "libaio" for examples.

As this is a recurring pattern, we should find a way to add it somewhere
in the build-system. But for now, the ugly snippet appending target
should work :).

Otherwise this looks fine!

Thanks,

Mathieu




^ permalink raw reply	[flat|nested] 8+ messages in thread

* [bug#41150] [PATCH] gnu: Add powerstat.
  2020-05-09 17:01 ` Mathieu Othacehe
@ 2020-05-09 17:32   ` Tobias Geerinckx-Rice via Guix-patches via
  2020-05-09 20:03     ` Jean-Baptiste Note
  2020-05-09 20:42   ` bug#41150: " Tobias Geerinckx-Rice via Guix-patches via
  1 sibling, 1 reply; 8+ messages in thread
From: Tobias Geerinckx-Rice via Guix-patches via @ 2020-05-09 17:32 UTC (permalink / raw)
  To: Mathieu Othacehe; +Cc: 41150

[-- Attachment #1: Type: text/plain, Size: 1057 bytes --]

Mathieu,

Mathieu Othacehe 写道:
> Hard-coding CC this way will prevent cross-compilation. You can 
> have a
> look to "crda" or "libaio" for examples.

Eh?  This is the first time anyone's ever mentioned that.  Thanks! 
It's obvious how much cross-compilation means to me.

(You realise there are *hundreds* of CC=gcc lines in current Guix, 
right?)

Don't get me wrong: I love that Guix still cares so much about 
cross-compilation, and the tremendous effort I see in the logs 
every month to make it work better, when (at least a few years 
ago) it was all the rage to declare it dead since VMs exist or 
something.

> As this is a recurring pattern, we should find a way to add it 
> somewhere
> in the build-system. But for now, the ugly snippet appending 
> target
> should work :).

I agree that handling (only) standard GNU variables[0] in the GNU 
build system makes sense.

I'll add the snippet of power.

Kind regards,

T G-R

[0]: 
https://www.gnu.org/prep/standards/standards.html#Utilities-in-Makefiles

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [bug#41150] [PATCH] gnu: Add powerstat.
  2020-05-09 17:32   ` Tobias Geerinckx-Rice via Guix-patches via
@ 2020-05-09 20:03     ` Jean-Baptiste Note
  2020-05-09 21:04       ` Tobias Geerinckx-Rice via Guix-patches via
  0 siblings, 1 reply; 8+ messages in thread
From: Jean-Baptiste Note @ 2020-05-09 20:03 UTC (permalink / raw)
  To: 41150; +Cc: othacehe, me

Hi there,

There are actually more than 300 such instances, and counting...

Couldn't we get a "magic" variable %target-cc like we have
%output, %outputs -- then we could just stubstitute gcc for this
variable...

I would do it, if I only knew where these are defined, but my scheme
skills are definitely lacking :)

Kind regards,
Jean-Baptiste




^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#41150: [PATCH] gnu: Add powerstat.
  2020-05-09 17:01 ` Mathieu Othacehe
  2020-05-09 17:32   ` Tobias Geerinckx-Rice via Guix-patches via
@ 2020-05-09 20:42   ` Tobias Geerinckx-Rice via Guix-patches via
  2020-05-10  7:24     ` [bug#41150] " Mathieu Othacehe
  1 sibling, 1 reply; 8+ messages in thread
From: Tobias Geerinckx-Rice via Guix-patches via @ 2020-05-09 20:42 UTC (permalink / raw)
  To: Mathieu Othacehe; +Cc: 41150-done

[-- Attachment #1: Type: text/plain, Size: 597 bytes --]

Mathieu again,

Mathieu Othacehe 写道:
> Hard-coding CC this way will prevent cross-compilation. You can 
> have a
> look to "crda" or "libaio" for examples.

Do you happen to know why libaio specifies the full path to gcc 
(only) when cross-compiling?  I didn't bother modifying it to 
test.

I cross-built powerstat for aarch64 on my laptop (then ran it by 
accident -- modern GNU/Linux is mad).  I'm not restricting 
platforms (yet) since my knowledge of ACPI support outside of x86 
does not exist.

Pushed as 510a8eb1b8b4f2686606e427b8965c126b81ed8a.

Thanks!

T G-R

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [bug#41150] [PATCH] gnu: Add powerstat.
  2020-05-09 20:03     ` Jean-Baptiste Note
@ 2020-05-09 21:04       ` Tobias Geerinckx-Rice via Guix-patches via
  2020-05-09 21:10         ` Tobias Geerinckx-Rice via Guix-patches via
  0 siblings, 1 reply; 8+ messages in thread
From: Tobias Geerinckx-Rice via Guix-patches via @ 2020-05-09 21:04 UTC (permalink / raw)
  To: Jean-Baptiste Note; +Cc: othacehe, 41150

[-- Attachment #1: Type: text/plain, Size: 1528 bytes --]

Jean-Baptiste!

Jean-Baptiste Note 写道:
> There are actually more than 300 such instances

*Hundreds*! :-p

On a positive note there are 3 fewer occurences on c-u (308) than 
master (311).

I used a simple ‘grep CC=gcc | wc -l’; I suspect you did something 
similar.

> and counting...

It shouldn't increase if people posts their patches for review 
(...and they actually get reviewed...).  CC=gcc is an old habit 
but not difficult to break.

> Couldn't we get a "magic" variable %target-cc like we have
> %output, %outputs -- then we could just stubstitute gcc for this
> variable...

I'm (not yet?) (no longer?) convinced that's a good idea once 
gnu-build-system takes care of those 308 packages and we're left 
with the exceptions.

When I replied to Mathieu's last mail I hadn't looked at the code 
yet:

  (let ((target ,(%current-target-system)))
    (list (string-append "CC=" (if target
                                   (string-append target "-gcc")
                                   "gcc"))))

To me, abstracting that is beyond overkill.

However, I don't know much about cross-compiling.  TBH I'd be 
sowewhat surprised if none of the CROSS-* procedures I regularly 
scroll past do something like this already.

> I would do it, if I only knew where these are defined, but my 
> scheme
> skills are definitely lacking :)

Mainly (gnu packages cross-base).

Kind regards,

T G-R, currently building kernels to properly answer your 
hibernation mails...

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [bug#41150] [PATCH] gnu: Add powerstat.
  2020-05-09 21:04       ` Tobias Geerinckx-Rice via Guix-patches via
@ 2020-05-09 21:10         ` Tobias Geerinckx-Rice via Guix-patches via
  0 siblings, 0 replies; 8+ messages in thread
From: Tobias Geerinckx-Rice via Guix-patches via @ 2020-05-09 21:10 UTC (permalink / raw)
  To: Jean-Baptiste Note; +Cc: 41150

[-- Attachment #1: Type: text/plain, Size: 447 bytes --]

Tobias Geerinckx-Rice 写道:
> Jean-Baptiste Note 写道:
>> I would do it, if I only knew where these are defined, but my 
>> scheme
>> skills are definitely lacking :)
>
> Mainly (gnu packages cross-base).

...or if you mean the % friends: (guix utils).  %target-gcc could 
be computed at evaluation (as opposed to build) time so is 
relatively simple [see previous mail for ‘too simple’ argument].

Kind regards,

T G-R

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [bug#41150] [PATCH] gnu: Add powerstat.
  2020-05-09 20:42   ` bug#41150: " Tobias Geerinckx-Rice via Guix-patches via
@ 2020-05-10  7:24     ` Mathieu Othacehe
  0 siblings, 0 replies; 8+ messages in thread
From: Mathieu Othacehe @ 2020-05-10  7:24 UTC (permalink / raw)
  To: 41150; +Cc: me


Hey,

> Do you happen to know why libaio specifies the full path to gcc (only) when
> cross-compiling?  I didn't bother modifying it to test.

Just tested, this does not seem useful.

> I cross-built powerstat for aarch64 on my laptop (then ran it by accident --
> modern GNU/Linux is mad).  I'm not restricting platforms (yet) since my
> knowledge of ACPI support outside of x86 does not exist.

It's for sure less common than on x86, but it seems that it exists, see:
https://www.kernel.org/doc/Documentation/arm64/arm-acpi.txt.

> Pushed as 510a8eb1b8b4f2686606e427b8965c126b81ed8a.

Thanks for fixing it :)

Mathieu




^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2020-05-10  7:25 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-05-09 15:15 [bug#41150] [PATCH] gnu: Add powerstat Tobias Geerinckx-Rice via Guix-patches via
2020-05-09 17:01 ` Mathieu Othacehe
2020-05-09 17:32   ` Tobias Geerinckx-Rice via Guix-patches via
2020-05-09 20:03     ` Jean-Baptiste Note
2020-05-09 21:04       ` Tobias Geerinckx-Rice via Guix-patches via
2020-05-09 21:10         ` Tobias Geerinckx-Rice via Guix-patches via
2020-05-09 20:42   ` bug#41150: " Tobias Geerinckx-Rice via Guix-patches via
2020-05-10  7:24     ` [bug#41150] " Mathieu Othacehe

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.