Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1702213 > unrolled thread

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

Started byGreg KH <gregkh@linuxfoundation.org>
First post2017-08-02 18:10 +0200
Last post2017-08-02 18:50 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  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

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

FromGreg KH <gregkh@linuxfoundation.org>
Date2017-08-02 18:10 +0200
SubjectRe: [PATCH 1/2] Add sysfs based cpuinfo structure
Message-ID<ua4nE-3Eo-1@gated-at.bofh.it>
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?

> +
> +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?

thanks,

greg k-h

[toc] | [next] | [standalone]


#1702264

FromFelix Schnizlein <fschnizlein@suse.com>
Date2017-08-02 18:50 +0200
Message-ID<ua50m-3TJ-7@gated-at.bofh.it>
In reply to#1702213

[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
> 

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web