linux-kernel - Re: [PATCH 1/2] Add sysfs based cpuinfo structure

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20170802164621.43ckgv2vieudvifc@angmar>
Date:   Wed, 2 Aug 2017 18:46:21 +0200
From:   Felix Schnizlein <fschnizlein@...e.com>
To:     Greg KH <gregkh@...uxfoundation.org>
Cc:     linux-kernel@...r.kernel.org, x86@...nel.org,
        yanmin.zhang@...el.com, mingo@...hat.com, trenn@...e.com
Subject: Re: [PATCH 1/2] Add sysfs based cpuinfo structure

On 02.08.17, Greg KH wrote:
> On Wed, Aug 02, 2017 at 05:14:11PM +0200, Felix Schnizlein wrote:
> > Create a new sysfs attribute group called 'info' for each
> > online cpu. The cleaned up cpuinfo shows up in a sysfs
> > subdirectory here: /sys/devices/system/cpu/cpu#/info.
> > 
> > Define preprocessor macros (CPUINFO_DEFINE_* and CPUINFO_ATTR) to make
> > defining new sysfs attributes for cpuinfo more easy.
> > 
> > Add basic documentation for the info sysfs tree.
> > 
> > Signed-off-by: Felix Schnizlein <fschnizlein@...e.com>
> > Signed-off-by: Thomas Renninger <trenn@...e.com>
> > ---
> >  Documentation/ABI/testing/sysfs-devices-system-cpu | 17 ++++++++
> >  arch/Kconfig                                       |  3 ++
> >  drivers/base/Makefile                              |  1 +
> >  drivers/base/cpuinfo.c                             | 48 ++++++++++++++++++++++
> >  include/linux/cpuhotplug.h                         |  1 +
> >  include/linux/cpuinfo.h                            | 43 +++++++++++++++++++
> >  6 files changed, 113 insertions(+)
> >  create mode 100644 drivers/base/cpuinfo.c
> >  create mode 100644 include/linux/cpuinfo.h
> > 
> > diff --git a/Documentation/ABI/testing/sysfs-devices-system-cpu b/Documentation/ABI/testing/sysfs-devices-system-cpu
> > index f3d5817..fbaf165 100644
> > --- a/Documentation/ABI/testing/sysfs-devices-system-cpu
> > +++ b/Documentation/ABI/testing/sysfs-devices-system-cpu
> > @@ -373,3 +373,20 @@ Contact:	Linux kernel mailing list <linux-kernel@...r.kernel.org>
> >  Description:	information about CPUs heterogeneity.
> >  
> >  		cpu_capacity: capacity of cpu#.
> > +
> > +What:		/sys/devices/system/cpu/cpu#/info/
> > +Date:		June 2017
> > +Contact:	Linux kernel mailing list <linux-kernel@...r.kernel.org>
> > +Description:	Various information about the online cpu. Which attributes are
> > +		available depends on the architecture, kernel configuration and
> > +                cpu model.
> 
> tabs?
Oh.. it seems I missed this line. I will fix this!
> 
> > +
> > +What:		/sys/devices/system/cpu/cpu#/info/
> > +		/sys/devices/system/cpu/cpu#/info/vendor_id
> > +		/sys/devices/system/cpu/cpu#/info/cpu_family
> > +		/sys/devices/system/cpu/cpu#/info/model
> > +		/sys/devices/system/cpu/cpu#/info/model_name
> > +		/sys/devices/system/cpu/cpu#/info/stepping
> > +		/sys/devices/system/cpu/cpu#/info/flags
> > +		/sys/devices/system/cpu/cpu#/info/bugs
> > +		/sys/devices/system/cpu/cpu#/info/bogomips
> 
> Can you guarantee that all arches will be able to provide these?
Of course, I can not. I move it too the x86 section in the second patch.
It seems messed up the last rebase.

Sorry for the noise and stupid mistakes...

And thank you for the feedback and your patience!


Felix
> 
> thanks,
> 
> greg k-h
> 

Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)