[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20250907014216.2691844-3-jay.vosburgh@canonical.com>
Date: Sat, 6 Sep 2025 18:42:14 -0700
From: Jay Vosburgh <jay.vosburgh@...onical.com>
To: netdev@...r.kernel.org
Cc: Stephen Hemminger <stephen@...workplumber.org>,
David Ahern <dsahern@...il.com>
Subject: [PATCH 2/4 iproute2-next] tc: Add get_size64 and get_size64_and_cell
In preparation for accepting 64 bit burst sizes, create 64-bit
versions of get_size and get_size_and_cell. The 32-bit versions become
wrappers around the 64-bit versions.
Signed-off-by: Jay Vosburgh <jay.vosburgh@...onical.com>
---
include/utils.h | 1 +
lib/utils_math.c | 19 ++++++++++++++++++-
tc/tc_util.c | 21 +++++++++++++++++++--
tc/tc_util.h | 1 +
4 files changed, 39 insertions(+), 3 deletions(-)
diff --git a/include/utils.h b/include/utils.h
index 9a81494dd3e3..128cbb59cb90 100644
--- a/include/utils.h
+++ b/include/utils.h
@@ -163,6 +163,7 @@ int get_addr64(__u64 *ap, const char *cp);
int get_rate(unsigned int *rate, const char *str);
int get_rate64(__u64 *rate, const char *str);
int get_size(unsigned int *size, const char *str);
+int get_size64(__u64 *size, const char *str);
int hex2mem(const char *buf, uint8_t *mem, int count);
char *hexstring_n2a(const __u8 *str, int len, char *buf, int blen);
diff --git a/lib/utils_math.c b/lib/utils_math.c
index 9ef3dd6ed93b..a7e747440039 100644
--- a/lib/utils_math.c
+++ b/lib/utils_math.c
@@ -87,7 +87,7 @@ int get_rate64(__u64 *rate, const char *str)
return 0;
}
-int get_size(unsigned int *size, const char *str)
+int get_size64(__u64 *size, const char *str)
{
double sz;
char *p;
@@ -121,3 +121,20 @@ int get_size(unsigned int *size, const char *str)
return 0;
}
+
+int get_size(unsigned int *size, const char *str)
+{
+ __u64 sz64;
+ int rv;
+
+ rv = get_size64(&sz64, str);
+ *size = sz64;
+
+ if (rv)
+ return rv;
+
+ if (sz64 > UINT_MAX)
+ return -1;
+
+ return 0;
+}
diff --git a/tc/tc_util.c b/tc/tc_util.c
index ff0ac170730b..45d76e7578d4 100644
--- a/tc/tc_util.c
+++ b/tc/tc_util.c
@@ -257,14 +257,14 @@ tc_print_rate(enum output_type t, const char *key, const char *fmt,
print_rate(use_iec, t, key, fmt, rate);
}
-int get_size_and_cell(unsigned int *size, int *cell_log, char *str)
+int get_size64_and_cell(__u64 *size, int *cell_log, char *str)
{
char *slash = strchr(str, '/');
if (slash)
*slash = 0;
- if (get_size(size, str))
+ if (get_size64(size, str))
return -1;
if (slash) {
@@ -286,6 +286,23 @@ int get_size_and_cell(unsigned int *size, int *cell_log, char *str)
return 0;
}
+int get_size_and_cell(unsigned int *size, int *cell_log, char *str)
+{
+ __u64 size64;
+ int rv;
+
+ rv = get_size64_and_cell(&size64, cell_log, str);
+ if (rv)
+ return rv;
+
+ if (size64 > UINT32_MAX)
+ return -1;
+
+ *size = size64;
+
+ return 0;
+}
+
void print_devname(enum output_type type, int ifindex)
{
const char *ifname = ll_index_to_name(ifindex);
diff --git a/tc/tc_util.h b/tc/tc_util.h
index ec2063729b07..8ebca3963d94 100644
--- a/tc/tc_util.h
+++ b/tc/tc_util.h
@@ -80,6 +80,7 @@ int get_qdisc_handle(__u32 *h, const char *str);
int get_percent_rate(unsigned int *rate, const char *str, const char *dev);
int get_percent_rate64(__u64 *rate, const char *str, const char *dev);
int get_size_and_cell(unsigned int *size, int *cell_log, char *str);
+int get_size64_and_cell(__u64 *size, int *cell_log, char *str);
int get_linklayer(unsigned int *val, const char *arg);
void tc_print_rate(enum output_type t, const char *key, const char *fmt,
--
2.25.1
Powered by blists - more mailing lists