Author: neel
Date: Tue Oct  8 16:36:17 2013
New Revision: 256156
URL: http://svnweb.freebsd.org/changeset/base/256156

Log:
  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)

Modified:
  head/usr.sbin/bhyve/bhyverun.c
  head/usr.sbin/bhyve/dbgport.h

Modified: head/usr.sbin/bhyve/bhyverun.c
==============================================================================
--- head/usr.sbin/bhyve/bhyverun.c      Tue Oct  8 16:01:43 2013        
(r256155)
+++ head/usr.sbin/bhyve/bhyverun.c      Tue Oct  8 16:36:17 2013        
(r256156)
@@ -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;

Modified: head/usr.sbin/bhyve/dbgport.h
==============================================================================
--- head/usr.sbin/bhyve/dbgport.h       Tue Oct  8 16:01:43 2013        
(r256155)
+++ head/usr.sbin/bhyve/dbgport.h       Tue Oct  8 16:36:17 2013        
(r256156)
@@ -29,8 +29,6 @@
 #ifndef _DBGPORT_H_
 #define        _DBGPORT_H_
 
-#define        DEFAULT_GDB_PORT        6466
-
 void   init_dbgport(int port);
 
 #endif
_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "[email protected]"

Reply via email to