diff options
| author | Ben Pfaff <blp@ovn.org> | 2017-06-18 09:46:54 +0800 |
|---|---|---|
| committer | Ben Pfaff <blp@ovn.org> | 2017-06-23 16:28:26 +0800 |
| commit | d1fd1ea91242c4fbf68c1292559fe0dd6d96d0e3 (patch) | |
| tree | 6ae73795a52bc25cf19eb0e892af2d9a04330f9c /lib/odp-util.c | |
| parent | 9d71ade0cf5bb6a43f387f5a4765a79f82b0f09d (diff) | |
| download | openvswitch-d1fd1ea91242c4fbf68c1292559fe0dd6d96d0e3.tar.gz | |
ovs-dpctl: New --names option to use port names in flow dumps.
Until now, printing names in "ovs-dpctl dump-flows" was tied to the overall
output verbosity, which in practice meant that to see port names a user had
to see a distracting amount of verbosity. This decouples names from
verbosity.
I'd like to make showing names the default for interactive usage, but so
far names aren't accepted in input so that would frustrate cut-and-paste,
which is an important use of "ovs-dpctl dump-flows" output.
Signed-off-by: Ben Pfaff <blp@ovn.org>
Acked-by: Jan Scheurich <jan.scheurich@ericsson.com>
Tested-by: Jan Scheurich <jan.scheurich@ericsson.com>
Diffstat (limited to 'lib/odp-util.c')
| -rw-r--r-- | lib/odp-util.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/odp-util.c b/lib/odp-util.c index 8e4df797a..d15095558 100644 --- a/lib/odp-util.c +++ b/lib/odp-util.c @@ -2944,7 +2944,7 @@ format_odp_key_attr(const struct nlattr *a, const struct nlattr *ma, break; case OVS_KEY_ATTR_IN_PORT: - if (portno_names && verbose && is_exact) { + if (portno_names && is_exact) { char *name = odp_portno_names_get(portno_names, nl_attr_get_odp_port(a)); if (name) { @@ -3251,7 +3251,7 @@ odp_format_ufid(const ovs_u128 *ufid, struct ds *ds) /* Appends to 'ds' a string representation of the 'key_len' bytes of * OVS_KEY_ATTR_* attributes in 'key'. If non-null, additionally formats the * 'mask_len' bytes of 'mask' which apply to 'key'. If 'portno_names' is - * non-null and 'verbose' is true, translates odp port number to its name. */ + * non-null, translates odp port number to its name. */ void odp_flow_format(const struct nlattr *key, size_t key_len, const struct nlattr *mask, size_t mask_len, |