lkml.org 
[lkml]   [2008]   [Jan]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[PATCH] getdelays: fix gcc warnings
    From: Randy Dunlap <randy.dunlap@oracle.com>

    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 <randy.dunlap@oracle.com>
    ---
    Documentation/accounting/getdelays.c | 6 +++---
    1 file changed, 3 insertions(+), 3 deletions(-)

    --- linux-2.6.24-rc7.orig/Documentation/accounting/getdelays.c
    +++ linux-2.6.24-rc7/Documentation/accounting/getdelays.c
    @@ -168,7 +168,7 @@ int get_family_id(int sd)
    char buf[256];
    } ans;

    - int id, rc;
    + int id = 0, rc;
    struct nlattr *na;
    int rep_len;

    @@ -209,7 +209,7 @@ void print_delayacct(struct taskstats *t
    void task_context_switch_counts(struct taskstats *t)
    {
    printf("\n\nTask %15s%15s\n"
    - " %15lu%15lu\n",
    + " %15llu%15llu\n",
    "voluntary", "nonvoluntary",
    t->nvcsw, t->nivcsw);
    }
    @@ -399,7 +399,7 @@ int main(int argc, char *argv[])
    goto done;
    }

    - PRINTF("nlmsghdr size=%d, nlmsg_len=%d, rep_len=%d\n",
    + PRINTF("nlmsghdr size=%zu, nlmsg_len=%d, rep_len=%d\n",
    sizeof(struct nlmsghdr), msg.n.nlmsg_len, rep_len);



    \
     
     \ /
      Last update: 2008-01-16 01:11    [W:0.021 / U:29.676 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site