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

From	Felix Schnizlein <fschnizlein@suse.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH 1/2] Add sysfs based cpuinfo structure
Date	2017-08-02 18:50 +0200
Message-ID	<ua50m-3TJ-7@gated-at.bofh.it> (permalink)
References	<ua4nE-3Eo-3@gated-at.bofh.it> <ua4nE-3Eo-5@gated-at.bofh.it> <ua4nE-3Eo-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

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@suse.com>
> > Signed-off-by: Thomas Renninger <trenn@suse.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@vger.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@vger.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
>

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH 1/2] Add sysfs based cpuinfo structure Greg KH <gregkh@linuxfoundation.org> - 2017-08-02 18:10 +0200
  Re: [PATCH 1/2] Add sysfs based cpuinfo structure Felix Schnizlein <fschnizlein@suse.com> - 2017-08-02 18:50 +0200

csiph-web