Change the behavior of bhyve such that the gdb listening port is opt-in

rather than opt-out.

Prior to this change if the "-g" option was not specified then a listening
socket for tunneling gdb packets would be opened at port 6466. If a second
virtual machine is fired up, also without the "-g" option, then that would
fail because there is already a listener on port 6466.

After this change if a gdb tunnel port needs to be created it needs to be
explicitly specified with a "-g <portnum>" command line option.

Reviewed by:	grehan@
Approved by:	re@ (blanket)
This commit is contained in:
Neel Natu 2013-10-08 16:36:17 +00:00
parent 7aa27dbac5
commit 4a06a0fe79
Notes: svn2git 2020-12-20 02:59:44 +00:00
svn path=/head/; revision=256156
2 changed files with 3 additions and 5 deletions

View File

@ -123,7 +123,7 @@ usage(int code)
" <vmname>\n"
" -a: local apic is in XAPIC mode (default is X2APIC)\n"
" -A: create an ACPI table\n"
" -g: gdb port (default is %d and 0 means don't open)\n"
" -g: gdb port\n"
" -c: # cpus (default 1)\n"
" -p: pin vcpu 'n' to host cpu 'pincpu + n'\n"
" -H: vmexit from the guest on hlt\n"
@ -134,7 +134,7 @@ usage(int code)
" -s: <slot,driver,configinfo> PCI slot config\n"
" -S: <slot,driver,configinfo> legacy PCI slot config\n"
" -m: memory size in MB\n",
progname, DEFAULT_GDB_PORT);
progname);
exit(code);
}
@ -504,7 +504,7 @@ main(int argc, char *argv[])
bvmcons = 0;
progname = basename(argv[0]);
gdb_port = DEFAULT_GDB_PORT;
gdb_port = 0;
guest_ncpus = 1;
ioapic = 0;
memsize = 256 * MB;

View File

@ -29,8 +29,6 @@
#ifndef _DBGPORT_H_
#define _DBGPORT_H_
#define DEFAULT_GDB_PORT 6466
void init_dbgport(int port);
#endif