[PATCH] input: Make keep alive configurable (generic)

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/OpenBSC@lists.osmocom.org/.

Jacob Erlbeck jerlbeck at sysmocom.de
Thu Jan 16 17:10:37 UTC 2014


This patch adds a generic keep alive configuration layer that mainly
consists of additional fields in e1_input structs and VTY commands
and extensions.

Ticket: OW#1060
Sponsored-by: On-Waves ehf
---
 TODO-RELEASE                    |    2 ++
 include/osmocom/abis/e1_input.h |    7 ++++
 src/e1_input_vty.c              |   69 +++++++++++++++++++++++++++++++++++++++
 3 files changed, 78 insertions(+)
 create mode 100644 TODO-RELEASE

diff --git a/TODO-RELEASE b/TODO-RELEASE
new file mode 100644
index 0000000..98b9a64
--- /dev/null
+++ b/TODO-RELEASE
@@ -0,0 +1,2 @@
+#library	what		description / commit summary line
+libosmoabis	abi-change	input: Make keep alive configurable (generic)
diff --git a/include/osmocom/abis/e1_input.h b/include/osmocom/abis/e1_input.h
index 0abf0b8..9b77893 100644
--- a/include/osmocom/abis/e1_input.h
+++ b/include/osmocom/abis/e1_input.h
@@ -12,6 +12,7 @@
 #include <osmocom/abis/lapd.h>
 
 #define NUM_E1_TS   32
+#define E1INP_USE_DEFAULT (-1)
 
 enum e1inp_sign_type {
 	E1INP_SIGN_NONE,
@@ -134,6 +135,7 @@ struct e1inp_driver {
 	void (*close)(struct e1inp_sign_link *link);
 	void (*vty_show)(struct vty *vty, struct e1inp_line *line);
 	int default_delay;
+	int has_keepalive;
 };
 
 struct e1inp_line_ops {
@@ -163,6 +165,11 @@ struct e1inp_line {
 	unsigned int port_nr;
 	struct rate_ctr_group *rate_ctr;
 
+	/* keepalive configuration */
+	int keepalive_num_probes; /* 0: disable, num, or E1INP_USE_DEFAULT */
+	int keepalive_idle_timeout; /* secs, or E1INP_USE_DEFAULT */
+	int keepalive_probe_interval; /* secs or E1INP_USE_DEFAULT */
+
 	/* array of timestlots */
 	struct e1inp_ts ts[NUM_E1_TS];
 	unsigned int num_ts;
diff --git a/src/e1_input_vty.c b/src/e1_input_vty.c
index d99c853..0b4adb2 100644
--- a/src/e1_input_vty.c
+++ b/src/e1_input_vty.c
@@ -88,6 +88,56 @@ DEFUN(cfg_e1line_port, cfg_e1_line_port_cmd,
 	return CMD_SUCCESS;
 }
 
+#define KEEPALIVE_HELP "Enable keep-alive probing\n"
+static int set_keepalive_params(struct vty *vty, int e1_nr,
+				int idle, int num_probes, int probe_interval)
+{
+	struct e1inp_line *line = e1inp_line_find(e1_nr);
+
+	if (!line) {
+		vty_out(vty, "%% Line %d doesn't exist%s", e1_nr, VTY_NEWLINE);
+		return CMD_WARNING;
+	}
+	if (!line->driver->has_keepalive && num_probes != 0) {
+		vty_out(vty, "%% Driver '%s' does not support keep alive%s",
+			line->driver->name, VTY_NEWLINE);
+		return CMD_WARNING;
+	}
+
+	line->keepalive_idle_timeout = idle;
+	line->keepalive_num_probes = num_probes;
+	line->keepalive_probe_interval = probe_interval;
+
+	return CMD_SUCCESS;
+}
+
+DEFUN(cfg_e1line_keepalive, cfg_e1_line_keepalive_cmd,
+	"e1_line <0-255> keepalive",
+	E1_LINE_HELP KEEPALIVE_HELP)
+{
+	return set_keepalive_params(vty, atoi(argv[0]),
+				    E1INP_USE_DEFAULT, E1INP_USE_DEFAULT,
+				    E1INP_USE_DEFAULT);
+}
+
+DEFUN(cfg_e1line_keepalive_params, cfg_e1_line_keepalive_params_cmd,
+	"e1_line <0-255> keepalive <1-300> <1-20> <1-300>",
+	E1_LINE_HELP KEEPALIVE_HELP
+	"Idle interval in seconds before probes are sent\n"
+	"Number of probes to sent\n"
+	"Delay between probe packets in seconds\n")
+{
+	return set_keepalive_params(vty, atoi(argv[0]),
+				    atoi(argv[1]), atoi(argv[2]), atoi(argv[3]));
+}
+
+DEFUN(cfg_e1line_no_keepalive, cfg_e1_line_no_keepalive_cmd,
+	"no e1_line <0-255> keepalive",
+	NO_STR E1_LINE_HELP KEEPALIVE_HELP)
+{
+	return set_keepalive_params(vty, atoi(argv[0]), 0, 0, 0);
+}
+
 DEFUN(cfg_e1line_name, cfg_e1_line_name_cmd,
 	"e1_line <0-255> name .LINE",
 	E1_LINE_HELP "Set name for this line\n" "Human readable name\n")
@@ -135,6 +185,22 @@ static int e1inp_config_write(struct vty *vty)
 		if (line->name)
 			vty_out(vty, " e1_line %u name %s%s", line->num,
 				line->name, VTY_NEWLINE);
+		if (!line->keepalive_num_probes)
+			vty_out(vty, " no e1_line %u keepalive%s", line->num,
+				VTY_NEWLINE);
+		else if (line->keepalive_idle_timeout == E1INP_USE_DEFAULT &&
+			 line->keepalive_num_probes == E1INP_USE_DEFAULT &&
+			 line->keepalive_probe_interval == E1INP_USE_DEFAULT)
+			vty_out(vty, " e1_line %u keepalive%s", line->num,
+				VTY_NEWLINE);
+		else
+			vty_out(vty, " e1_line %u keepalive %d %d %d%s",
+				line->num,
+				line->keepalive_idle_timeout,
+				line->keepalive_num_probes,
+				line->keepalive_probe_interval,
+				VTY_NEWLINE);
+
 	}
 	return CMD_SUCCESS;
 }
@@ -281,6 +347,9 @@ int e1inp_vty_init(void)
 	install_element(L_E1INP_NODE, &cfg_e1_line_driver_cmd);
 	install_element(L_E1INP_NODE, &cfg_e1_line_port_cmd);
 	install_element(L_E1INP_NODE, &cfg_e1_line_name_cmd);
+	install_element(L_E1INP_NODE, &cfg_e1_line_keepalive_cmd);
+	install_element(L_E1INP_NODE, &cfg_e1_line_keepalive_params_cmd);
+	install_element(L_E1INP_NODE, &cfg_e1_line_no_keepalive_cmd);
 
 	install_element_ve(&show_e1drv_cmd);
 	install_element_ve(&show_e1line_cmd);
-- 
1.7.9.5





More information about the OpenBSC mailing list