lkml.org 
[lkml]   [2007]   [Nov]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [patch 1/3] bdi patches
    From
    Date
    > > Provide a place in sysfs for the backing_dev_info object.
    > > This allows us to see and set the various BDI specific variables.
    >
    > You don't say what the place is, and I'm not quite familiar enough
    > with sysfs internals to figure it out my self. Help?

    /sys/class/bdi (which sometimes links into /sys/devices/virtual/bdi)

    > And while I was looking I noticed that bdi_register (and bdi_init_fmt)
    > takes a second argument 'parent', which is always NULL, and which is
    > undocumented as to purpose.
    > If no-one would ever add another call to bdi_register, why have the
    > second arg, and if they might, how would they know what to put there?
    >
    > Finally, the omission of NFS bothers me - and makes me wonder if the
    > choice of name in sysfs is appropriate.
    >
    > Would a program ever want to generate the name (in sysfs) for a
    > particular bdi? If so, how would it do it.

    I'll let Peter answer these, when he gets back.

    > It seems to me after a fairly quick look that a bdi is always
    > associated with a device number. For block devices the device number
    > is obvious. For NFS and FUSE, the device number is an anon device
    > number allocated at mount time.
    > Maybe the name of the bdi should be based on that number. Then it
    > would be possible to map directly from e.g. a file to the bdi that the
    > file would be written to.

    Sounds like a good idea to me.

    Miklos
    -
    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: 2007-11-30 09:09    [W:0.035 / U:0.384 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site