Re: [PATCH iproute2] iptuntap: fix multi-queue flag display
From: Maxime Coquelin <hidden>
Date: 2021-09-14 08:22:33
On 9/1/21 5:48 PM, David Marchand wrote:
quoted hunk ↗ jump to hunk
When creating a tap with multi_queue flag, this flag is not displayed when dumping: $ ip tuntap add tap23 mode tap multi_queue $ ip tuntap tap23: tap persist0x100 While at it, add a space between known flags and hexdump of unknown ones. Fixes: c41e038f48a3 ("iptuntap: allow creation of multi-queue tun/tap device") Signed-off-by: David Marchand <redacted> --- ip/iptuntap.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-)diff --git a/ip/iptuntap.c b/ip/iptuntap.c index 9cdb4a80..96ca1ae7 100644 --- a/ip/iptuntap.c +++ b/ip/iptuntap.c@@ -243,6 +243,9 @@ static void print_flags(long flags) if (flags & IFF_ONE_QUEUE) print_string(PRINT_ANY, NULL, " %s", "one_queue"); + if (flags & IFF_MULTI_QUEUE) + print_string(PRINT_ANY, NULL, " %s", "multi_queue"); + if (flags & IFF_VNET_HDR) print_string(PRINT_ANY, NULL, " %s", "vnet_hdr");@@ -253,9 +256,10 @@ static void print_flags(long flags) print_string(PRINT_ANY, NULL, " %s", "filter"); flags &= ~(IFF_TUN | IFF_TAP | IFF_NO_PI | IFF_ONE_QUEUE | - IFF_VNET_HDR | IFF_PERSIST | IFF_NOFILTER); + IFF_MULTI_QUEUE | IFF_VNET_HDR | IFF_PERSIST | + IFF_NOFILTER); if (flags) - print_0xhex(PRINT_ANY, NULL, "%#llx", flags); + print_0xhex(PRINT_ANY, NULL, " %#llx", flags); close_json_array(PRINT_JSON, NULL); }
Reviewed-by: Maxime Coquelin <redacted> Thanks, Maxime