* [PATCH iproute2-next] tc: add 'kind' property to 'csum' action
@ 2019-01-31 17:58 Davide Caratti
2019-02-03 17:12 ` David Ahern
0 siblings, 1 reply; 2+ messages in thread
From: Davide Caratti @ 2019-01-31 17:58 UTC (permalink / raw)
To: David Ahern, Stephen Hemminger; +Cc: netdev
unlike other TC actions already supporting JSON printout, 'csum' does not
print the value of TCA_KIND in the 'kind' property: remove 'csum' word
from 'csum' property, and add a separate 'kind' property containing the
action name. The human-readable printout is preserved.
Tested with:
# ./tdc.py -c csum
Cc: Andrea Claudi <aclaudi@redhat.com>
Signed-off-by: Davide Caratti <dcaratti@redhat.com>
---
tc/m_csum.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tc/m_csum.c b/tc/m_csum.c
index 752269d1d020..84396d6a482d 100644
--- a/tc/m_csum.c
+++ b/tc/m_csum.c
@@ -199,10 +199,11 @@ print_csum(struct action_util *au, FILE *f, struct rtattr *arg)
uflag_1 = "?empty";
}
+ print_string(PRINT_ANY, "kind", "%s ", "csum");
snprintf(buf, sizeof(buf), "%s%s%s%s%s%s%s",
uflag_1, uflag_2, uflag_3,
uflag_4, uflag_5, uflag_6, uflag_7);
- print_string(PRINT_ANY, "csum", "csum (%s) ", buf);
+ print_string(PRINT_ANY, "csum", "(%s) ", buf);
print_action_control(f, "action ", sel->action, "\n");
print_uint(PRINT_ANY, "index", "\tindex %u", sel->index);
--
2.20.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH iproute2-next] tc: add 'kind' property to 'csum' action
2019-01-31 17:58 [PATCH iproute2-next] tc: add 'kind' property to 'csum' action Davide Caratti
@ 2019-02-03 17:12 ` David Ahern
0 siblings, 0 replies; 2+ messages in thread
From: David Ahern @ 2019-02-03 17:12 UTC (permalink / raw)
To: Davide Caratti, Stephen Hemminger; +Cc: netdev
On 1/31/19 10:58 AM, Davide Caratti wrote:
> unlike other TC actions already supporting JSON printout, 'csum' does not
> print the value of TCA_KIND in the 'kind' property: remove 'csum' word
> from 'csum' property, and add a separate 'kind' property containing the
> action name. The human-readable printout is preserved.
>
> Tested with:
> # ./tdc.py -c csum
>
> Cc: Andrea Claudi <aclaudi@redhat.com>
> Signed-off-by: Davide Caratti <dcaratti@redhat.com>
> ---
> tc/m_csum.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
applied to iproute2-next. Thanks
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-02-03 17:12 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-01-31 17:58 [PATCH iproute2-next] tc: add 'kind' property to 'csum' action Davide Caratti
2019-02-03 17:12 ` David Ahern
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox