[lkml]   [2002]   [Sep]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [RFC][PATCH] sard changes for 2.5.34
    Rick Lindsley wrote:
    > kstat should be a lighter-weight per-cpu thing. But the current
    > disk accounting in there would make it 12 kilobytes per CPU.
    > My vote: remove the disk accounting from kernel_stat and use this.
    > I have a patch from another contributor that takes disk stats out of
    > kstat and puts them into their own global structure. I'll give that
    > some attention.

    OK, that's a start. I think there was some work done on making
    kernel_stat percpu as well.

    Cleaning up and speeding up kernel_stat is a spearate exercise
    of course, but as we need to change userspace we may as well roll
    it all up together.

    > > What follows works, but needs refinements. Comments welcome.
    > What are those refinements?
    > A couple I mentioned in my message: double collection of stats, and an
    > ugly hd_struct added to gendisk. In addition, we should remove the
    > restriction on which and how many disks are reported on.
    > Lastly, a bit of a philosophical question. /proc/stat and (with this
    > patch) /proc/diskstats provide some of the same information. Should
    > a) all of it appear in /proc/stat?
    > b) all of it appear in /proc/diskstats?
    > c) keep the current (limited) info in /proc/stat (for backward
    > compatibility) and introduce the expanded info in
    > /proc/diskstats?
    > My preference is b, but I'm open to other opinions.

    b). Let's get the kernel right and change userspace to follow. We have
    another accounting patch which breaks top(1), so Rik has fixed it (and
    is feeding the fixes upstream).

    > What userspace tools are available for interpreting this
    > information?
    > None that I'm aware of, although /proc/diskstats is formatted in a
    > program-friendly way. Sample output (warning: wide):

    Does it work with the utilities at What is the
    relationship with the 2.4 sard work? (I've never used sard, so words
    of one syllable please ;))

    If we can get this work playing nicely with those existing sard tools,
    get the kernel_stat stuff cleaned up and get the relevant userspace
    tools working and merged upstream then we have a neat bundle to submit.
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 13:28    [W:0.033 / U:0.072 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site