Eric Lee / smarc-fsl-linux-kernel

24 Sep, 2016

1 commit

  • d522b2cdf   tools: move accounting tool from Documentation ... Browse Code »

    Move accounting tool to tools and remove it from Documentation
    Makefile. Update location information for this tool. Create a
    new Makefile to build accounting. It can be built from top level
    directory or from accounting directory:

    Run make -C tools/accounting or cd tools/accounting; make

    Acked-by: Jonathan Corbet
    Signed-off-by: Shuah Khan

    Shuah Khan
     

28 Apr, 2016

1 commit

  • 570d8e939   taskstats: fix nl parsing in accounting/getdelays.c ... Browse Code »

    The type TASKSTATS_TYPE_NULL should always be ignored.

    When jumping to the next attribute, only the length of the current
    attribute should be added, not the length of all nested attributes.
    This last bug was not visible before commit 80df554275c2, because the
    kernel didn't put more than two nested attributes.

    Fixes: a3baf649ca9c ("[PATCH] per-task-delay-accounting: documentation")
    Fixes: 80df554275c2 ("taskstats: use the libnl API to align nlattr on 64-bit")
    Signed-off-by: Nicolas Dichtel
    Signed-off-by: David S. Miller

    Nicolas Dichtel
     

24 Dec, 2015

1 commit

26 Sep, 2014

1 commit

24 Jun, 2014

1 commit

04 Jul, 2013

1 commit

26 Nov, 2012

1 commit

02 Sep, 2012

1 commit

  • 0de4b9540   doc: getdelays.c: remember to close() socket on error in create_nl_socket() ... Browse Code »

    If the call to setsockopt() fails in
    Documentation/accounting/getdelays.c::create_nl_socket() we return -1
    without closing the socket, thus leaking it when the 'fd' variable
    goes out of scope.

    Easily fixed by just jumping to the 'error' label instead of returning
    since we do the proper cleanup there.

    While I was there I noticed that the error message that is printet if
    setsockopt() fails was broken over two lines - put that on a single
    line so it is easier to grep for.

    Signed-off-by: Jesper Juhl
    Signed-off-by: Jiri Kosina

    Jesper Juhl
     

16 Jun, 2011

1 commit

  • f6e07d380   Documentation: update cgroupfs mount point ... Browse Code »

    According to commit 676db4af0430 ("cgroupfs: create /sys/fs/cgroup to
    mount cgroupfs on") the canonical mountpoint for the cgroup filesystem
    is /sys/fs/cgroup. Hence, this should be used in the documentation.

    Signed-off-by: Jörg Sommer
    Acked-by: Paul Menage
    Signed-off-by: Randy Dunlap
    Signed-off-by: Linus Torvalds

    Jörg Sommer
     

27 May, 2011

3 commits

  • 02d54f092   getdelays: show average CPU/IO/SWAP/RECLAIM delays ... Browse Code »

    I find it very handy to show the average delays in milliseconds.

    Example output (on 100 concurrent dd reading sparse files):

    CPU count real total virtual total delay total delay average
    986 3223509952 3207643301 38863410579 39.415ms
    IO count delay total delay average
    0 0 0ms
    SWAP count delay total delay average
    0 0 0ms
    RECLAIM count delay total delay average
    1059 5131834899 4ms
    dd: read=0, write=0, cancelled_write=0

    Signed-off-by: Wu Fengguang
    Cc: Mel Gorman
    Cc: Balbir Singh
    Reviewed-by: Satoru Moriya
    Reviewed-by: KOSAKI Motohiro
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Wu Fengguang
     
  • 4ed960b14   Documentation/accounting/getdelays.c: handle sendto() failures ... Browse Code »

    Fixes

    Documentation/accounting/getdelays.c: In function `get_family_id':
    Documentation/accounting/getdelays.c:172:14: warning: variable `rc' set but not used [-Wunused-but-set-variable]

    Reported-by: "Justin P. Mattock"
    Cc: Balbir Singh
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Andrew Morton
     
  • fbdd91a62   Documentation/accounting/getdelays.c: fix unused var warning ... Browse Code »

    Fixes

    Documentation/accounting/getdelays.c: In function `main':
    Documentation/accounting/getdelays.c:436:7: warning: variable `i' set but not used [-Wunused-but-set-variable]

    Signed-off-by: Justin P. Mattock
    Cc: Balbir Singh
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Justin P. Mattock
     

23 Dec, 2010

1 commit

  • 4be2c95d1   taskstats: pad taskstats netlink response for aligment issues on ia64 ... Browse Code »

    The taskstats structure is internally aligned on 8 byte boundaries but the
    layout of the aggregrate reply, with two NLA headers and the pid (each 4
    bytes), actually force the entire structure to be unaligned. This causes
    the kernel to issue unaligned access warnings on some architectures like
    ia64. Unfortunately, some software out there doesn't properly unroll the
    NLA packet and assumes that the start of the taskstats structure will
    always be 20 bytes from the start of the netlink payload. Aligning the
    start of the taskstats structure breaks this software, which we don't
    want. So, for now the alignment only happens on architectures that
    require it and those users will have to update to fixed versions of those
    packages. Space is reserved in the packet only when needed. This ifdef
    should be removed in several years e.g. 2012 once we can be confident
    that fixed versions are installed on most systems. We add the padding
    before the aggregate since the aggregate is already a defined type.

    Commit 85893120 ("delayacct: align to 8 byte boundary on 64-bit systems")
    previously addressed the alignment issues by padding out the pid field.
    This was supposed to be a compatible change but the circumstances
    described above mean that it wasn't. This patch backs out that change,
    since it was a hack, and introduces a new NULL attribute type to provide
    the padding. Padding the response with 4 bytes avoids allocating an
    aligned taskstats structure and copying it back. Since the structure
    weighs in at 328 bytes, it's too big to do it on the stack.

    Signed-off-by: Jeff Mahoney
    Reported-by: Brian Rogers
    Cc: Jeff Mahoney
    Cc: Guillaume Chazarain
    Cc: Balbir Singh
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Jeff Mahoney
     

28 Oct, 2010

1 commit

  • db9e5679d   delay-accounting: reimplement -c for getdelays.c to report information on a target command ... Browse Code »

    Task delay-accounting was identified as one means of determining how long
    a process spends in congestion_wait() without adding new statistics. For
    example, if the workload should not be doing IO, delay-accounting could
    reveal how long it was spending in unexpected IO or delays.
    Unfortunately, on closer examination it was clear that getdelays does not
    act as documented.

    Commit a3baf649 ("per-task-delay-accounting: documentation") added
    Documentation/accounting/getdelays.c with a -c switch that was documented
    to fork/exec a child and report statistics on it but for reasons that are
    unclear to me, commit 9e06d3f9 deleted support for this switch but did not
    update the documentation. It might be an oversight or it might be because
    the control flow of the program meant that accounting information would be
    printed once early in the lifetime of the program making it of limited
    use.

    This patch reimplements -c for getdelays.c to act as documented. Unlike
    the original version, it waits until the command completes before printing
    any information on it. An example of it being used looks like

    $ ./getdelays -d -c find /home/mel -name mel
    print delayacct stats ON
    /home/mel
    /home/mel/.notes-wine/drive_c/windows/profiles/mel
    /home/mel/.wine/drive_c/windows/profiles/mel
    /home/mel/git-configs/dot.kde/share/apps/konqueror/home/mel
    PID 5923

    CPU count real total virtual total delay total
    42779 5051232096 5164722692 564207988
    IO count delay total
    41727 97804147758
    SWAP count delay total
    0 0
    RECLAIM count delay total
    0 0

    [akpm@linux-foundation.org: coding-style fixes]
    Signed-off-by: Mel Gorman
    Acked-by: Balbir Singh
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Mel Gorman
     

23 Sep, 2009

1 commit

17 Jun, 2009

1 commit

16 Jan, 2009

1 commit

31 Oct, 2008

1 commit

13 Aug, 2008

2 commits

  • 666593137   docsrc: fix getdelays printk formats ... Browse Code »

    Fix printf format type warnings (seen on alpha & ia64):

    Documentation/accounting/getdelays.c:206: warning: format '%15llu' expects type 'long long unsigned int', but argument 6 has type '__u64'
    Documentation/accounting/getdelays.c:206: warning: format '%15llu' expects type 'long long unsigned int', but argument 7 has type '__u64'
    Documentation/accounting/getdelays.c:206: warning: format '%15llu' expects type 'long long unsigned int', but argument 8 has type '__u64'
    Documentation/accounting/getdelays.c:206: warning: format '%15llu' expects type 'long long unsigned int', but argument 9 has type '__u64'
    Documentation/accounting/getdelays.c:206: warning: format '%15llu' expects type 'long long unsigned int', but argument 12 has type '__u64'
    Documentation/accounting/getdelays.c:206: warning: format '%15llu' expects type 'long long unsigned int', but argument 13 has type '__u64'
    Documentation/accounting/getdelays.c:206: warning: format '%15llu' expects type 'long long unsigned int', but argument 16 has type '__u64'
    Documentation/accounting/getdelays.c:206: warning: format '%15llu' expects type 'long long unsigned int', but argument 17 has type '__u64'
    Documentation/accounting/getdelays.c:214: warning: format '%15llu' expects type 'long long unsigned int', but argument 4 has type '__u64'
    Documentation/accounting/getdelays.c:214: warning: format '%15llu' expects type 'long long unsigned int', but argument 5 has type '__u64'
    Documentation/accounting/getdelays.c:221: warning: format '%llu' expects type 'long long unsigned int', but argument 2 has type '__u64'
    Documentation/accounting/getdelays.c:221: warning: format '%llu' expects type 'long long unsigned int', but argument 3 has type '__u64'
    Documentation/accounting/getdelays.c:221: warning: format '%llu' expects type 'long long unsigned int', but argument 4 has type '__u64'
    Documentation/accounting/getdelays.c:221: warning: format '%llu' expects type 'long long unsigned int', but argument 5 has type '__u64'
    Documentation/accounting/getdelays.c:221: warning: format '%llu' expects type 'long long unsigned int', but argument 6 has type '__u64'
    Documentation/accounting/getdelays.c:236: warning: 'cmd_type' may be used uninitialized in this function

    Signed-off-by: Randy Dunlap
    Cc: Balbir Singh
    Cc: Sam Ravnborg
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Randy Dunlap
     
  • 3794f3e81   docsrc: build Documentation/ sources ... Browse Code »

    Currently source files in the Documentation/ sub-dir can easily bit-rot
    since they are not generally buildable, either because they are hidden in
    text files or because there are no Makefile rules for them. This needs to
    be fixed so that the source files remain usable and good examples of code
    instead of bad examples.

    Add the ability to build source files that are in the Documentation/ dir.
    Add to Kconfig as "BUILD_DOCSRC" config symbol.

    Use "CONFIG_BUILD_DOCSRC=1 make ..." to build objects from the
    Documentation/ sources. Or enable BUILD_DOCSRC in the *config system.
    However, this symbol depends on HEADERS_CHECK since the header files need
    to be installed (for userspace builds).

    Built (using cross-tools) for x86-64, i386, alpha, ia64, sparc32,
    sparc64, powerpc, sh, m68k, & mips.

    Signed-off-by: Randy Dunlap
    Reviewed-by: Sam Ravnborg
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Randy Dunlap
     

27 Jul, 2008

1 commit

26 Jul, 2008

2 commits

05 Jul, 2008

1 commit

09 Feb, 2008

1 commit

  • 10e6f32bd   getdelays: fix gcc warnings ... Browse Code »

    Fix gcc warnings in getdelays.c:

    Documentation/accounting/getdelays.c: In function 'task_context_switch_counts':
    Documentation/accounting/getdelays.c:214: warning: format '%15lu' expects type 'long unsigned int', but argument 4 has type '__u64'
    Documentation/accounting/getdelays.c:214: warning: format '%15lu' expects type 'long unsigned int', but argument 5 has type '__u64'
    Documentation/accounting/getdelays.c: In function 'main':
    Documentation/accounting/getdelays.c:402: warning: format '%d' expects type 'int', but argument 2 has type 'long unsigned int'
    Documentation/accounting/getdelays.c: In function 'get_family_id':
    Documentation/accounting/getdelays.c:171: warning: 'id' may be used uninitialized in this function

    One warning is not a problem and can be dismissed:
    Documentation/accounting/getdelays.c: In function 'main':
    Documentation/accounting/getdelays.c:236: warning: 'cmd_type' may be used uninitialized in this function

    Signed-off-by: Randy Dunlap
    Acked-by: Balbir Singh
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Randy Dunlap
     

15 Nov, 2007

1 commit

  • 546040dc4   make getdelays cgroupstats aware ... Browse Code »

    Update the getdelays utility to become cgroupstats aware. A new -C option has
    been added. It takes in a control group path and prints out a summary of the
    states of tasks in the control group

    Signed-off-by: Balbir Singh
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Balbir Singh
     

20 Oct, 2007

1 commit

  • 846c7bb05   Add cgroupstats ... Browse Code »

    This patch is inspired by the discussion at
    http://lkml.org/lkml/2007/4/11/187 and implements per cgroup statistics
    as suggested by Andrew Morton in http://lkml.org/lkml/2007/4/11/263. The
    patch is on top of 2.6.21-mm1 with Paul's cgroups v9 patches (forward
    ported)

    This patch implements per cgroup statistics infrastructure and re-uses
    code from the taskstats interface. A new set of cgroup operations are
    registered with commands and attributes. It should be very easy to
    *extend* per cgroup statistics, by adding members to the cgroupstats
    structure.

    The current model for cgroupstats is a pull, a push model (to post
    statistics on interesting events), should be very easy to add. Currently
    user space requests for statistics by passing the cgroup file
    descriptor. Statistics about the state of all the tasks in the cgroup
    is returned to user space.

    TODO's/NOTE:

    This patch provides an infrastructure for implementing cgroup statistics.
    Based on the needs of each controller, we can incrementally add more statistics,
    event based support for notification of statistics, accumulation of taskstats
    into cgroup statistics in the future.

    Sample output

    # ./cgroupstats -C /cgroup/a
    sleeping 2, blocked 0, running 1, stopped 0, uninterruptible 0

    # ./cgroupstats -C /cgroup/
    sleeping 154, blocked 0, running 0, stopped 0, uninterruptible 0

    If the approach looks good, I'll enhance and post the user space utility for
    the same

    Feedback, comments, test results are always welcome!

    [akpm@linux-foundation.org: build fix]
    Signed-off-by: Balbir Singh
    Cc: Paul Menage
    Cc: Jay Lan
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Balbir Singh
     

17 Oct, 2007

1 commit

23 Aug, 2007

1 commit

  • efbee7f1c   Documentation: fix getdelays.c printf bug ... Browse Code »

    Commit b663a79c191508f27cd885224b592a878c0ba0f6 ("taskstats: add
    context-switch counters") incorrectly removed a comma from a printf
    statement. This causes corruption in the output printing or a seg
    fault.

    Signed-off-by: Michael Neuling
    Acked-by: Balbir Singh
    Signed-off-by: Linus Torvalds

    Michael Neuling
     

17 Jul, 2007

1 commit

  • b663a79c1   taskstats: add context-switch counters ... Browse Code »

    Make available to the user the following task and process performance
    statistics:

    * Involuntary Context Switches (task_struct->nivcsw)
    * Voluntary Context Switches (task_struct->nvcsw)

    Statistics information is available from:
    1. taskstats interface (Documentation/accounting/)
    2. /proc/PID/status (task only).

    This data is useful for detecting hyperactivity patterns between processes.

    [akpm@linux-foundation.org: cleanup]
    Signed-off-by: Maxim Uvarov
    Cc: Shailabh Nagar
    Cc: Balbir Singh
    Cc: Jay Lan
    Cc: Jonathan Lim
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Maxim Uvarov
     

09 May, 2007

2 commits

11 Dec, 2006

2 commits

  • cf709844d   [PATCH] io-accounting: add to getdelays ... Browse Code »

    Wire up the IO accounting into getdelays.c.

    Usage:

    To display I/O stats for each exitting task:

    vmm:/home/akpm> ./getdelays -m0,1,2,3 -i -l
    cpumask 0 maskset 1
    printing IO accounting
    listen forever
    rm: read=8192, write=0, cancelled_write=0
    cvs: read=733184, write=4255744, cancelled_write=4096
    make: read=217088, write=0, cancelled_write=0
    cc1: read=4263936, write=12288, cancelled_write=0
    as: read=811008, write=8192, cancelled_write=0
    gcc: read=323584, write=0, cancelled_write=12288
    cc1: read=0, write=8192, cancelled_write=0
    as: read=4096, write=4096, cancelled_write=0
    gcc: read=16384, write=0, cancelled_write=4096
    as: read=4096, write=4096, cancelled_write=0
    gcc: read=16384, write=0, cancelled_write=8192
    ld: read=1011712, write=16384, cancelled_write=0
    collect2: read=626688, write=0, cancelled_write=0
    gcc: read=204800, write=0, cancelled_write=0
    cc1: read=0, write=8192, cancelled_write=0
    as: read=4096, write=4096, cancelled_write=0
    gcc: read=16384, write=0, cancelled_write=8192
    ld: read=8192, write=16384, cancelled_write=0
    collect2: read=49152, write=0, cancelled_write=0
    gcc: read=0, write=0, cancelled_write=0
    cc1: read=0, write=4096, cancelled_write=0
    ld: read=4096, write=12288, cancelled_write=0
    collect2: read=49152, write=0, cancelled_write=0
    gcc: read=0, write=0, cancelled_write=0

    To display I/O stats for a particular presently-running task:

    vmm:/home/akpm> ./getdelays -i -p $(pidof crond)
    printing IO accounting
    crond: read=61440, write=0, cancelled_write=0

    Cc: Jay Lan
    Cc: Shailabh Nagar
    Cc: Balbir Singh
    Cc: Chris Sturtivant
    Cc: Tony Ernst
    Cc: Guillaume Thouvenin
    Cc: David Wright
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Andrew Morton
     
  • d2f7bf134   [PATCH] getdelays: various fixes ... Browse Code »

    - Various cleanups

    - Report errors to stderr, not stdout

    - A printf was missing a \n and was hiding from me.

    Cc: Jay Lan
    Cc: Shailabh Nagar
    Cc: Balbir Singh
    Cc: Chris Sturtivant
    Cc: Tony Ernst
    Cc: Guillaume Thouvenin
    Cc: David Wright
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Andrew Morton
     

30 Nov, 2006

2 commits

  • 4ae0edc21   Fix typos in /Documentation : 'U-Z' ... Browse Code »

    This patch fixes typos in various Documentation txts. The patch addresses some
    +words starting with the letters 'U-Z'.

    Looks like I made it through the alphabet...just in time to start over again
    +too! Maybe I can fit more profound fixes into the next round...? Time will
    +tell. :)

    Signed-off-by: Matt LaPlante
    Acked-by: Randy Dunlap
    Signed-off-by: Adrian Bunk

    Matt LaPlante
     
  • fa00e7e15   Fix typos in /Documentation : 'T'' ... Browse Code »

    This patch fixes typos in various Documentation txts. The patch addresses some
    +words starting with the letter 'T'.

    Signed-off-by: Matt LaPlante
    Acked-by: Randy Dunlap
    Signed-off-by: Adrian Bunk

    Matt LaPlante
     

04 Nov, 2006

1 commit

01 Oct, 2006

2 commits

  • db5fed26b   [PATCH] csa accounting taskstats update ... Browse Code »

    ChangeLog:
    Feedbacks from Andrew Morton:
    - define TS_COMM_LEN to 32
    - change acct_stimexpd field of task_struct to be of
    cputime_t, which is to be used to save the tsk->stime
    of last timer interrupt update.
    - a new Documentation/accounting/taskstats-struct.txt
    to describe fields of taskstats struct.

    Feedback from Balbir Singh:
    - keep the stime of a task to be zero when both stime
    and utime are zero as recoreded in task_struct.

    Misc:
    - convert accumulated RSS/VM from platform dependent
    pages-ticks to MBytes-usecs in the kernel

    Cc: Shailabh Nagar
    Cc: Balbir Singh
    Cc: Jes Sorensen
    Cc: Chris Sturtivant
    Cc: Tony Ernst
    Cc: Guillaume Thouvenin
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Jay Lan
     
  • 7d1bdca9b   [PATCH] Fix getdelays.c - cpumask length and error reporting ... Browse Code »

    Fix the length passed while (un)registering cpumask. We were passing sizeof
    the array, make it strlen().

    Error value printed in fatal errors should be derived from the message. The
    message contains an nlmsgerr embedded with an error value. We must report
    that value to the user.

    Signed-off-by: Balbir Singh
    Cc: Jamal Hadi
    Cc: Shailabh Nagar
    Cc: Thomas Graf
    Cc: "David S. Miller"
    Cc: Jay Lan
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Balbir Singh
     

01 Aug, 2006

1 commit

  • 163ecdff0   [PATCH] delay accounting: temporarily enable by default ... Browse Code »

    Enable delay accounting by default so that feature gets coverage testing
    without requiring special measures.

    Earlier, it was off by default and had to be enabled via a boot time param.
    This patch reverses the default behaviour to improve coverage testing. It
    can be removed late in the kernel development cycle if its believed users
    shouldn't have to incur any cost if they don't want delay accounting. Or
    it can be retained forever if the utility of the stats is deemed common
    enough to warrant keeping the feature on.

    Signed-off-by: Shailabh Nagar
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Shailabh Nagar