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

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

[Tobias Geerinckx-Rice]

* 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

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

[Tobias Geerinckx-Rice]

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

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

[Tobias Geerinckx-Rice]

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

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

[Ludovic Courtès]

Hi Tobias!

Tobias Geerinckx-Rice <me@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’.

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

[Tobias Geerinckx-Rice]

Guix, Ludo',

Pushed as 6e8c75b88435550906e251dd4a9c2eb732f803d4.

Thanks for the review,

T G-R