pktgen: Fix inconsistent of format with argument type in pktgen.c
authorYe Bin <yebin10@huawei.com>
Wed, 30 Sep 2020 01:08:37 +0000 (09:08 +0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 2 Oct 2020 01:45:23 +0000 (18:45 -0700)
Fix follow warnings:
[net/core/pktgen.c:925]: (warning) %u in format string (no. 1)
requires 'unsigned int' but the argument type is 'signed int'.
[net/core/pktgen.c:942]: (warning) %u in format string (no. 1)
requires 'unsigned int' but the argument type is 'signed int'.
[net/core/pktgen.c:962]: (warning) %u in format string (no. 1)
requires 'unsigned int' but the argument type is 'signed int'.
[net/core/pktgen.c:984]: (warning) %u in format string (no. 1)
requires 'unsigned int' but the argument type is 'signed int'.
[net/core/pktgen.c:1149]: (warning) %d in format string (no. 1)
requires 'int' but the argument type is 'unsigned int'.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Ye Bin <yebin10@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/pktgen.c

index 44fdbb9..1059786 100644 (file)
@@ -922,7 +922,7 @@ static ssize_t pktgen_if_write(struct file *file,
                        pkt_dev->min_pkt_size = value;
                        pkt_dev->cur_pkt_size = value;
                }
-               sprintf(pg_result, "OK: min_pkt_size=%u",
+               sprintf(pg_result, "OK: min_pkt_size=%d",
                        pkt_dev->min_pkt_size);
                return count;
        }
@@ -939,7 +939,7 @@ static ssize_t pktgen_if_write(struct file *file,
                        pkt_dev->max_pkt_size = value;
                        pkt_dev->cur_pkt_size = value;
                }
-               sprintf(pg_result, "OK: max_pkt_size=%u",
+               sprintf(pg_result, "OK: max_pkt_size=%d",
                        pkt_dev->max_pkt_size);
                return count;
        }
@@ -959,7 +959,7 @@ static ssize_t pktgen_if_write(struct file *file,
                        pkt_dev->max_pkt_size = value;
                        pkt_dev->cur_pkt_size = value;
                }
-               sprintf(pg_result, "OK: pkt_size=%u", pkt_dev->min_pkt_size);
+               sprintf(pg_result, "OK: pkt_size=%d", pkt_dev->min_pkt_size);
                return count;
        }
 
@@ -981,7 +981,7 @@ static ssize_t pktgen_if_write(struct file *file,
 
                i += len;
                pkt_dev->nfrags = value;
-               sprintf(pg_result, "OK: frags=%u", pkt_dev->nfrags);
+               sprintf(pg_result, "OK: frags=%d", pkt_dev->nfrags);
                return count;
        }
        if (!strcmp(name, "delay")) {
@@ -1146,7 +1146,7 @@ static ssize_t pktgen_if_write(struct file *file,
                     (!(pkt_dev->odev->priv_flags & IFF_TX_SKB_SHARING)))))
                        return -ENOTSUPP;
                pkt_dev->burst = value < 1 ? 1 : value;
-               sprintf(pg_result, "OK: burst=%d", pkt_dev->burst);
+               sprintf(pg_result, "OK: burst=%u", pkt_dev->burst);
                return count;
        }
        if (!strcmp(name, "node")) {