aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTim Newsome <tim@sifive.com>2017-12-26 13:44:26 -0800
committerPaul Fertser <fercerpav@gmail.com>2018-01-13 09:51:42 +0000
commit79cbeafe9f948217d388045882015bdd00df706c (patch)
tree22890d3913fd6edcfac642ecd26a303ef9d2732f
parente9493cc20a0c2160696b24757274ec4ee302ac39 (diff)
downloadriscv-openocd-79cbeafe9f948217d388045882015bdd00df706c.zip
riscv-openocd-79cbeafe9f948217d388045882015bdd00df706c.tar.gz
riscv-openocd-79cbeafe9f948217d388045882015bdd00df706c.tar.bz2
Support listening on port 0.
When listening on port 0, the system will assign a random open port. We use this to run multiple OpenOCD instances against multiple simulators as part of regression testing. This mechanism means the various test instances don't have to coordinate to ensure they don't reuse any ports. The required changes are minimal: 1. Don't increment the port number when it's 0. 2. Print out which port was assigned by the system. Change-Id: I404c801fc405e9d8eb8420562c02e78d4db6242f Signed-off-by: Tim Newsome <tim@sifive.com> Reviewed-on: http://openocd.zylin.com/4316 Tested-by: jenkins Reviewed-by: Andreas Fritiofson <andreas.fritiofson@gmail.com>
-rw-r--r--src/server/gdb_server.c8
-rw-r--r--src/server/server.c6
2 files changed, 13 insertions, 1 deletions
diff --git a/src/server/gdb_server.c b/src/server/gdb_server.c
index fc0868b..b2d13d7 100644
--- a/src/server/gdb_server.c
+++ b/src/server/gdb_server.c
@@ -3090,7 +3090,13 @@ static int gdb_target_add_one(struct target *target)
if (!*end) {
if (parse_long(gdb_port_next, &portnumber) == ERROR_OK) {
free(gdb_port_next);
- gdb_port_next = alloc_printf("%d", portnumber+1);
+ if (portnumber) {
+ gdb_port_next = alloc_printf("%d", portnumber+1);
+ } else {
+ /* Don't increment if gdb_port is 0, since we're just
+ * trying to allocate an unused port. */
+ gdb_port_next = strdup("0");
+ }
}
}
}
diff --git a/src/server/server.c b/src/server/server.c
index 517d62a..1e52e97 100644
--- a/src/server/server.c
+++ b/src/server/server.c
@@ -298,6 +298,12 @@ int add_service(char *name,
free_service(c);
return ERROR_FAIL;
}
+
+ struct sockaddr_in addr_in;
+ socklen_t addr_in_size = sizeof(addr_in);
+ getsockname(c->fd, (struct sockaddr *)&addr_in, &addr_in_size);
+ LOG_INFO("Listening on port %hu for %s connections",
+ ntohs(addr_in.sin_port), name);
} else if (c->type == CONNECTION_STDINOUT) {
c->fd = fileno(stdin);