projects
/
openvswitch
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
xenserver: Fix typo in spec file.
[openvswitch]
/
utilities
/
ovs-ofctl.c
diff --git
a/utilities/ovs-ofctl.c
b/utilities/ovs-ofctl.c
index 6c7e3ac105dccad032c5afcc1fede675a7352d12..fef7fdacc33d00a9318ab6b20ac4ae6cb24b579d 100644
(file)
--- a/
utilities/ovs-ofctl.c
+++ b/
utilities/ovs-ofctl.c
@@
-55,6
+55,10
@@
VLOG_DEFINE_THIS_MODULE(ofctl);
/* --strict: Use strict matching for flow mod commands? */
static bool strict;
/* --strict: Use strict matching for flow mod commands? */
static bool strict;
+/* --readd: If ture, on replace-flows, re-add even flows that have not changed
+ * (to reset flow counters). */
+static bool readd;
+
/* -F, --flow-format: Flow format to use. Either one of NXFF_* to force a
* particular flow format or -1 to let ovs-ofctl choose intelligently. */
static int preferred_flow_format = -1;
/* -F, --flow-format: Flow format to use. Either one of NXFF_* to force a
* particular flow format or -1 to let ovs-ofctl choose intelligently. */
static int preferred_flow_format = -1;
@@
-82,11
+86,13
@@
parse_options(int argc, char *argv[])
{
enum {
OPT_STRICT = UCHAR_MAX + 1,
{
enum {
OPT_STRICT = UCHAR_MAX + 1,
+ OPT_READD,
VLOG_OPTION_ENUMS
};
static struct option long_options[] = {
{"timeout", required_argument, NULL, 't'},
{"strict", no_argument, NULL, OPT_STRICT},
VLOG_OPTION_ENUMS
};
static struct option long_options[] = {
{"timeout", required_argument, NULL, 't'},
{"strict", no_argument, NULL, OPT_STRICT},
+ {"readd", no_argument, NULL, OPT_READD},
{"flow-format", required_argument, NULL, 'F'},
{"more", no_argument, NULL, 'm'},
{"help", no_argument, NULL, 'h'},
{"flow-format", required_argument, NULL, 'F'},
{"more", no_argument, NULL, 'm'},
{"help", no_argument, NULL, 'h'},
@@
-139,6
+145,10
@@
parse_options(int argc, char *argv[])
strict = true;
break;
strict = true;
break;
+ case OPT_READD:
+ readd = true;
+ break;
+
VLOG_OPTION_HANDLERS
STREAM_SSL_OPTION_HANDLERS
VLOG_OPTION_HANDLERS
STREAM_SSL_OPTION_HANDLERS
@@
-184,6
+194,7
@@
usage(void)
vlog_usage();
printf("\nOther options:\n"
" --strict use strict match for flow commands\n"
vlog_usage();
printf("\nOther options:\n"
" --strict use strict match for flow commands\n"
+ " --readd replace flows that haven't changed\n"
" -F, --flow-format=FORMAT force particular flow format\n"
" -m, --more be more verbose printing OpenFlow\n"
" -t, --timeout=SECS give up after SECS seconds\n"
" -F, --flow-format=FORMAT force particular flow format\n"
" -m, --more be more verbose printing OpenFlow\n"
" -t, --timeout=SECS give up after SECS seconds\n"
@@
-233,7
+244,7
@@
open_vconn__(const char *name, const char *default_suffix,
free(datapath_name);
free(datapath_type);
free(datapath_name);
free(datapath_type);
- if (str
str(name, ":"
)) {
+ if (str
chr(name, ':'
)) {
run(vconn_open_block(name, OFP_VERSION, vconnp),
"connecting to %s", name);
} else if (!stat(name, &s) && S_ISSOCK(s.st_mode)) {
run(vconn_open_block(name, OFP_VERSION, vconnp),
"connecting to %s", name);
} else if (!stat(name, &s) && S_ISSOCK(s.st_mode)) {
@@
-247,7
+258,7
@@
open_vconn__(const char *name, const char *default_suffix,
}
open_vconn_socket(socket_name, vconnp);
} else {
}
open_vconn_socket(socket_name, vconnp);
} else {
- ovs_fatal(0, "%s is not a
valid connection method
", name);
+ ovs_fatal(0, "%s is not a
bridge or a socket
", name);
}
free(bridge_path);
}
free(bridge_path);
@@
-261,13
+272,14
@@
open_vconn(const char *name, struct vconn **vconnp)
}
static void *
}
static void *
-alloc_stats_request(size_t
body
_len, uint16_t type, struct ofpbuf **bufferp)
+alloc_stats_request(size_t
rq
_len, uint16_t type, struct ofpbuf **bufferp)
{
struct ofp_stats_msg *rq;
{
struct ofp_stats_msg *rq;
- rq = make_openflow(sizeof *rq + body_len, OFPT_STATS_REQUEST, bufferp);
+
+ rq = make_openflow(rq_len, OFPT_STATS_REQUEST, bufferp);
rq->type = htons(type);
rq->flags = htons(0);
rq->type = htons(type);
rq->flags = htons(0);
- return rq
+ 1
;
+ return rq;
}
static void
}
static void
@@
-333,7
+345,7
@@
static void
dump_trivial_stats_transaction(const char *vconn_name, uint8_t stats_type)
{
struct ofpbuf *request;
dump_trivial_stats_transaction(const char *vconn_name, uint8_t stats_type)
{
struct ofpbuf *request;
- alloc_stats_request(
0
, stats_type, &request);
+ alloc_stats_request(
sizeof(struct ofp_stats_msg)
, stats_type, &request);
dump_stats_transaction(vconn_name, request);
}
dump_stats_transaction(vconn_name, request);
}
@@
-658,7
+670,7
@@
do_flow_mod__(int argc, char *argv[], uint16_t command)
flow_mod_table_id = false;
parse_ofp_flow_mod_str(&requests, &flow_format, &flow_mod_table_id,
flow_mod_table_id = false;
parse_ofp_flow_mod_str(&requests, &flow_format, &flow_mod_table_id,
- argc > 2 ? argv[2] : "", command);
+ argc > 2 ? argv[2] : "", command
, false
);
check_final_format_for_flow_mod(flow_format);
open_vconn(argv[1], &vconn);
check_final_format_for_flow_mod(flow_format);
open_vconn(argv[1], &vconn);
@@
-951,8
+963,7
@@
fte_version_print(const struct fte_version *version)
}
ds_init(&s);
}
ds_init(&s);
- ofp_print_actions(&s, (const struct ofp_action_header *) version->actions,
- version->n_actions * sizeof *version->actions);
+ ofp_print_actions(&s, version->actions, version->n_actions);
printf(" %s\n", ds_cstr(&s));
ds_destroy(&s);
}
printf(" %s\n", ds_cstr(&s));
ds_destroy(&s);
}
@@
-1035,7
+1046,7
@@
read_flows_from_file(const char *filename, struct classifier *cls, int index)
struct flow_mod fm;
ofpbuf_init(&actions, 64);
struct flow_mod fm;
ofpbuf_init(&actions, 64);
- parse_ofp_str(&fm,
&actions, ds_cstr(&s)
);
+ parse_ofp_str(&fm,
OFPFC_ADD, ds_cstr(&s), true
);
version = xmalloc(sizeof *version);
version->cookie = fm.cookie;
version = xmalloc(sizeof *version);
version->cookie = fm.cookie;
@@
-1207,7
+1218,8
@@
do_replace_flows(int argc OVS_UNUSED, char *argv[])
struct fte_version *file_ver = fte->versions[FILE_IDX];
struct fte_version *sw_ver = fte->versions[SWITCH_IDX];
struct fte_version *file_ver = fte->versions[FILE_IDX];
struct fte_version *sw_ver = fte->versions[SWITCH_IDX];
- if (file_ver && (!sw_ver || !fte_version_equals(sw_ver, file_ver))) {
+ if (file_ver
+ && (readd || !sw_ver || !fte_version_equals(sw_ver, file_ver))) {
fte_make_flow_mod(fte, FILE_IDX, OFPFC_ADD, flow_format,
&requests);
}
fte_make_flow_mod(fte, FILE_IDX, OFPFC_ADD, flow_format,
&requests);
}
@@
-1308,7
+1320,7
@@
do_parse_flow(int argc OVS_UNUSED, char *argv[])
list_init(&packets);
parse_ofp_flow_mod_str(&packets, &flow_format, &flow_mod_table_id,
list_init(&packets);
parse_ofp_flow_mod_str(&packets, &flow_format, &flow_mod_table_id,
- argv[1], OFPFC_ADD);
+ argv[1], OFPFC_ADD
, false
);
print_packet_list(&packets);
}
print_packet_list(&packets);
}