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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 20 Oct 2009 14:47:38 -0600 From: Alex Chiang <achiang@...com> To: David Rientjes <rientjes@...gle.com> Cc: akpm@...ux-foundation.org, Randy Dunlap <randy.dunlap@...cle.com>, linux-mm@...ck.org, linux-kernel@...r.kernel.org, Greg KH <greg@...ah.com> Subject: Re: [PATCH 5/5] Documentation: ABI: document /sys/devices/system/cpu/ * David Rientjes <rientjes@...gle.com>: > On Mon, 19 Oct 2009, Alex Chiang wrote: > > > diff --git a/Documentation/ABI/testing/sysfs-devices-cpu b/Documentation/ABI/testing/sysfs-devices-cpu > > new file mode 100644 > > index 0000000..9070889 > > --- /dev/null > > +++ b/Documentation/ABI/testing/sysfs-devices-cpu > > Shouldn't this be called sysfs-devices-system-cpu? > > I see what you're doing: /sys/devices/system/node/* files are contained in > sysfs-devices-memory, but I think it would be helpful to have a more > strict naming scheme so that the contents of a sysfs directory are > described by a file of the same name. Yeah, I was just trying to follow an earlier example. But you're right, since I'm creating a brand new file, I can do it the Right Way (tm). > > @@ -0,0 +1,42 @@ > > +What: /sys/devices/system/cpu/ > > +Date: October 2009 > > +Contact: Linux kernel mailing list <linux-kernel@...r.kernel.org> > > +Description: > > + A collection of CPU attributes, including cache information, > > + topology, and frequency. It also contains a mechanism to > > + logically hotplug CPUs. > > + > > + The actual attributes present are architecture and > > + configuration dependent. > > + > > + > > +What: /sys/devices/system/cpu/$cpu/online > > cpu# ? Sure, will change (depending on response to my earlier email). > > +Date: January 2006 > > +Contact: Linux kernel mailing list <linux-kernel@...r.kernel.org> > > +Description: > > + When CONFIG_HOTPLUG_CPU is enabled, allows the user to > > + discover and change the online state of a CPU. To discover > > + the state: > > This is present even without CONFIG_HOTPLUG_CPU. That's what I get for not checking. Thank you for correcting me. > > + > > + cat /sys/devices/system/cpu/$cpu/online > > + > > + A value of 0 indicates the CPU is offline. A value of 1 > > + indicates it is online. To change the state, echo the > > + desired new state into the file: > > + > > + echo [0|1] > /sys/devices/system/cpu/$cpu/online > > + > > + For more information, please read Documentation/cpu-hotplug.txt > > + > > + > > +What: /sys/devices/system/cpu/$cpu/node > > +Date: October 2009 > > +Contact: Linux memory management mailing list <linux-mm@...ck.org> > > +Description: > > + When CONFIG_NUMA is enabled, a symbolic link that points > > + to the corresponding NUMA node directory. > > + > > + For example, the following symlink is created for cpu42 > > + in NUMA node 2: > > + > > + /sys/devices/system/cpu/cpu42/node2 -> ../../node/node2 > > > > > Would it be possible for you to document all entities in > /sys/devices/system/cpu/* in this new file (requiring a folding of > Documentation/ABI/testing/sysfs-devices-cache_disable into it)? I'll give it a go. There are quite a few things in that directory though, like topology information, frequency, etc. that I wasn't so excited about documenting. But if that's the tax to create my new symlinks, I'll pay it. ;) Thanks for the review, /ac -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists