GNU bug report logs - #32488
[PATCH] gnu: Add msr-tools.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 32488 in the body.
You can then email your comments to 32488 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Tobias Geerinckx-Rice <me <at> tobias.gr>
To: guix-patches <at> gnu.org
Subject: [PATCH] gnu: Add msr-tools.
Date: Tue, 21 Aug 2018 01:16:33 +0200

* gnu/packages/admin.scm (msr-tools): New public variable.
---
 gnu/packages/admin.scm | 46 ++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 46 insertions(+)

diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
index c9230c329..ac259fc00 100644
--- a/gnu/packages/admin.scm
+++ b/gnu/packages/admin.scm
@@ -468,6 +468,52 @@ asks for a login name and then transfers over to 'login'.  It is extended to
 allow automatic login and starting any app.")
     (license license:gpl2+)))
 
+(define-public msr-tools
+  (package
+    (name "msr-tools")
+    (version "1.3")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append "https://01.org/sites/default/files/downloads/"
+                           name "/" name "-" version ".zip"))
+       (sha256
+        (base32 "07hxmddg0l31kjfmaq84ni142lbbvgq6391r8bd79wpm819pnigr"))))
+    (build-system gnu-build-system)
+    (arguments
+     `(#:make-flags
+       (list (string-append "sbindir=" (assoc-ref %outputs "out") "/sbin"))
+       #:phases
+       (modify-phases %standard-phases
+         (delete 'configure)            ; no configure script
+         (add-before 'install 'create-output-directory
+           (lambda* (#:key outputs #:allow-other-keys)
+             (let* ((out  (assoc-ref outputs "out"))
+                    (sbin (string-append out "/sbin")))
+               (mkdir-p sbin)
+               #t))))
+       #:tests? #f))                    ; no test suite
+    (native-inputs
+     `(("unzip" ,unzip)))
+    (home-page "https://01.org/msr-tools/")
+    (synopsis "Model-Specific Register (@dfn{MSR})")
+    (description
+     "The MSR Tools project provides console utilities to directly access the
+Model-Specific Registers (@dfn{MSR}s) and CPU ID of Intel-compatible processors:
+
+@itemize
+@item @command{cpuid}: show identification and feature information of any CPU
+@item @command{rdmsr}: read MSRs from any CPU or all CPUs
+@item @command{wrmsr}: write to MSRs on any CPU or all CPUs
+@end itemize
+
+These tools can be used to query and modify certain low-level CPU parameters,
+such as the Turbo Boost ratio and Thermal Design Power (@dfn{TDP}) limits.
+
+MSR addresses differ (greatly) between processors, and any such modification can
+be dangerous and may void your CPU or system board's warranty.")
+    (license license:gpl2)))     ; cpuid.c is gpl2, {rd,wr}msr.c are gpl2+
+
 (define-public net-base
   (package
     (name "net-base")
-- 
2.18.0

Message #8 received at 32488 <at> debbugs.gnu.org (full text, mbox):

From: Tobias Geerinckx-Rice <me <at> tobias.gr>
To: 32488 <at> debbugs.gnu.org
Subject: Re: [bug#32488] [PATCH] gnu: Add msr-tools.
Date: Tue, 21 Aug 2018 01:34:25 +0200

Tobias Geerinckx-Rice wrote:
> * gnu/packages/admin.scm (msr-tools): New public variable.

I suspect this is Linux-specific, so it would end up in linux.scm.

Alternatively, I've been considering something like hardware.scm 
where things like ddcutil[0] could live. These things being 
Linux-specific is an (unfortunate) implementation detail, not 
really a meaningful category.

I'm fine with whatever.

Kind regards,

T G-R

[0]: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=32011

Message #11 received at 32488 <at> debbugs.gnu.org (full text, mbox):

From: Tobias Geerinckx-Rice <me <at> tobias.gr>
To: 32488 <at> debbugs.gnu.org
Subject: Re: [bug#32488] [PATCH] gnu: Add msr-tools.
Date: Tue, 21 Aug 2018 01:46:23 +0200

Tobias Geerinckx-Rice wrote:
> * gnu/packages/admin.scm (msr-tools): New public variable.
> ---
>  gnu/packages/admin.scm | 46 
>  ++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 46 insertions(+)
>
> diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
> index c9230c329..ac259fc00 100644
> --- a/gnu/packages/admin.scm
> +++ b/gnu/packages/admin.scm
> @@ -468,6 +468,52 @@ asks for a login name and then transfers 
> over to 'login'.  It is extended to
>  allow automatic login and starting any app.")
>      (license license:gpl2+)))
>  
> +(define-public msr-tools
> +  (package
> +    (name "msr-tools")
> +    (version "1.3")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (string-append 
> "https://01.org/sites/default/files/downloads/"
> +                           name "/" name "-" version ".zip"))
> +       (sha256
> +        (base32 
> "07hxmddg0l31kjfmaq84ni142lbbvgq6391r8bd79wpm819pnigr"))))
> +    (build-system gnu-build-system)
> +    (arguments
> +     `(#:make-flags
> +       (list (string-append "sbindir=" (assoc-ref %outputs 
> "out") "/sbin"))
> +       #:phases
> +       (modify-phases %standard-phases
> +         (delete 'configure)            ; no configure script
> +         (add-before 'install 'create-output-directory
> +           (lambda* (#:key outputs #:allow-other-keys)
> +             (let* ((out  (assoc-ref outputs "out"))
> +                    (sbin (string-append out "/sbin")))
> +               (mkdir-p sbin)
> +               #t))))
> +       #:tests? #f))                    ; no test suite

Also, as Marius just noted, this is all hella x86-specific so just 
imagine a

+    (supported-systems '("i686-linux" "x86_64-linux"))

right around here.

Kind regards,

T G-R

Message #14 received at 32488 <at> debbugs.gnu.org (full text, mbox):

From: ludo <at> gnu.org (Ludovic Courtès)
To: Tobias Geerinckx-Rice <me <at> tobias.gr>
Cc: 32488 <at> debbugs.gnu.org
Subject: Re: [bug#32488] [PATCH] gnu: Add msr-tools.
Date: Thu, 23 Aug 2018 00:44:23 +0200

Hi Tobias!

Tobias Geerinckx-Rice <me <at> tobias.gr> skribis:

> * gnu/packages/admin.scm (msr-tools): New public variable.

LGTM with ‘supported-systems’ as you note.  No strong opinion as to the
module where it should go.  :-)

Thanks,
Ludo’.

Message #19 received at 32488-done <at> debbugs.gnu.org (full text, mbox):

From: Tobias Geerinckx-Rice <me <at> tobias.gr>
To: 32488-done <at> debbugs.gnu.org
Subject: [PATCH] gnu: Add msr-tools.
Date: Thu, 30 Aug 2018 19:53:50 +0200

Guix, Ludo',

Pushed as 6e8c75b88435550906e251dd4a9c2eb732f803d4.

Thanks for the review,

T G-R

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.