[lkml]   [2010]   [Dec]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: delayacct: alignment changes break iotop
    Hash: SHA1

    On 12/13/2010 07:57 AM, Balbir Singh wrote:
    > * Dan Carpenter <> [2010-12-13 14:37:45]:
    >> Iotop uses hardcoded offsets to find the taskstats struct members.
    >> This got changed in 2.6.37 so it now iotop doesn't work on amd64. The
    >> offending commit is:
    >> commit 85893120699f8bae8caa12a8ee18ab5fceac978e
    >> Author: Jeff Mahoney <>
    >> Date: Wed Oct 27 15:34:43 2010 -0700
    >> delayacct: align to 8 byte boundary on 64-bit systems
    >> Brian Rogers gets the reported-by tag. The bugzilla entry is:
    > Thanks for the report, looks like the change did not even bump the
    > version field. Sorry, its my fault, I should have caught it earlier.
    > iotop hard coding member offsets is not bad as long as we don't break
    > ABI (expected from us). Any chance you could dump the offsets before
    > and after the change?

    In your February response and again in September, you did suggest a
    version bump. I'm not sure why that didn't get integrated but I still
    don't see how it's necessary for code that actually follows the interface.

    iotop doesn't. It's broken. It doesn't even honor that version field and
    worse yet, it doesn't even honor the packet format which specifically
    doesn't define hard offsets. Rather it defines a protocol that tags
    fields and supplies the offsets in the packet.

    The getdelays.c code that ships with the kernel even demonstrates this,
    so there's no excuse for half-assing it like this.

    - -Jeff

    - --
    Jeff Mahoney
    SUSE Labs
    Version: GnuPG v2.0.16 (GNU/Linux)
    Comment: Using GnuPG with SUSE -

    -----END PGP SIGNATURE-----

     \ /
      Last update: 2010-12-13 16:39    [W:0.020 / U:7.952 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site