lkml.org 
[lkml]   [2006]   [Mar]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRe: [PATCH] EDAC: core EDAC support code
    Date
    On Monday 06 March 2006 10:22, Randy.Dunlap wrote:
    > On Mon, 6 Mar 2006 10:14:22 -0800 Dave Peterson wrote:
    > > On Sunday 05 March 2006 02:30, Arjan van de Ven wrote:
    > > > On Sun, 2006-03-05 at 11:18 +0100, Arjan van de Ven wrote:
    > > > > > +/* Main MC kobject release() function */
    > > > > > +static void edac_memctrl_master_release(struct kobject *kobj)
    > > > > > +{
    > > > > > + debugf1("EDAC MC: " __FILE__ ": %s()\n", __func__);
    > > > > > +}
    > > > > > +
    > > > >
    > > > > ehhh how on earth can this be right?
    > > >
    > > > oh and this stuff also violates the "one value per file" rule; can we
    > > > fix that urgently before it becomes part of the ABI in 2.6.16??
    > >
    > > Ok, I'll admit to being a bit clueless about this. I'm not familiar
    > > with the "one value per file" rule; can someone please explain?
    >
    > it's in Documentation/filesystems/sysfs.txt
    > Strongly preferred.

    Ok, I assume the comment refers to the following:

    Attributes should be ASCII text files, preferably with only one value
    per file. It is noted that it may not be efficient to contain only
    value per file, so it is socially acceptable to express an array of
    values of the same type.

    I was initially a bit confused because I thought the comment
    specifically pertained to the piece of code shown above. I need to
    take a closer look at the EDAC sysfs code - I'm not as familiar with
    some of its details as I should be. Thanks for pointing out the
    issue.

    Dave
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2006-03-07 18:06    [W:0.024 / U:60.812 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site