[lkml]   [2009]   [Oct]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 3/7] Documentation: ABI: /sys/devices/system/cpu/ topology files
    On Wed, 28 Oct 2009, Alex Chiang wrote:

    > > I noticed that this and patches 4-6 were added to gregkh-2.6. Alex, did
    > > you want to seperate these large multi-file sections into individual file
    > > sections so that it is consistent with the remainder of Documentation/ABI?
    > That's one option. Here's another:
    > > > +What: /sys/devices/system/cpu/kernel_max
    > > > +What: /sys/devices/system/cpu/offline
    > > > +What: /sys/devices/system/cpu/online
    > > > +What: /sys/devices/system/cpu/possible
    > > > +What: /sys/devices/system/cpu/present
    > That way, your grep that you displayed in another mail still
    > works.
    > I understand the point about consistency, but... I kinda like the
    > condensed format better, since imho, it's easier to read.
    > Dunno, I don't have very strong feelings really... Greg already
    > took them, so David, if you do have a strong feeling, let me know
    > and I'll ask Greg to drop and I'll do one last respin.

    When I originally saw the format that the Documentation/ABI files were
    written in, I had this premonition that someone would eventually write a
    gui application that allowed users to tune their sysfs files and that it
    would inevitably include commentary on what the file represented by using
    the documentation. It was easily parsable and had a strict file naming
    scheme that this was possible. It seems we've departed from that at a
    relatively late stage in its maturity with no real benefit, but I'll defer
    to the maintainer's preference.

    Regardless, thanks a lot for documenting these files and going the extra
    mile! I'm sure it will come in very useful.

     \ /
      Last update: 2009-10-29 09:47    [W:0.027 / U:6.000 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site