1. 01 8月, 2018 1 次提交
    • A
      dwarf/line: fix some bugs with the state machine · 3f9875e2
      aarzilli 提交于
      Adds a test that compares the output of our state machine with the
      output of the debug_line reader in the standard library and checks that
      they produce the same output for the debug_line section of grafana as
      compiled on macOS (which is the most interesting case since it uses cgo
      and therefore goes through dsymutil).
      
      A few bugs were uncovered and fixed:
      
      1. is_stmt was reset improperly after a DW_LNS_end_sequence instruction
      2. basic_block, prologue_end and epilogue_begin were not reset after a
         DW_LNS_copy instruction
      3. some opcodes were not decoded properly if the debug_line section
         declares fewer standard opcodes than we know about.
      
      Fixes #1282
      3f9875e2
  2. 25 7月, 2018 1 次提交
    • A
      proc,service,terminal: read defer list · 8f1fc63d
      aarzilli 提交于
      Adds -defer flag to the stack command that decorates the stack traces
      by associating each stack frame with its deferred calls.
      
      Reworks proc.next to use this feature instead of using proc.DeferPC,
      laying the groundwork to implement #1240.
      8f1fc63d
  3. 24 7月, 2018 4 次提交
    • A
      terminal: disable next/step/stepout if current frame isn't 0 · 932aad9e
      aarzilli 提交于
      next/step/stepout should work even if the current frame isn't the
      topmost stack frame, but their behavior should be different in that
      case (they should continue inside the function of the selected frame).
      
      Most of the logic of next/step/stepout would work correctly if we
      simply replaced the call to proc.topframe with something that took a
      frame index. However the breakpoint they set on the first deferred
      function is wrong, and fixing it requires scanning the defer stack and
      matching it to the call stack, something we can't do yet.
      
      Given that enhancing next/step/stepout will take time and the current
      behavior confuses users (see issue #1240) return an error if
      next/step/stepout are called while the currently selected frame isn't
      frame 0.
      
      Updates #1240
      932aad9e
    • R
      Fix casing on delve · c8fcc3c5
      Ramya Achutha Rao 提交于
      c8fcc3c5
    • R
      Add docs on detaching · e6ce7d41
      Ramya Achutha Rao 提交于
      e6ce7d41
    • A
      dwarf/line: remove foundFile "optimization" from LineToPC · 368cbeb0
      aarzilli 提交于
      There is no guarantee that files will end up stored contiguously in the
      debug_line section which makes this optimization wrong in the general
      case.
      In particular with recent versions of go1.11 and a go.mod file present
      the go compiler seems to sometimes produce executables that actually
      violate this assumption.
      368cbeb0
  4. 20 7月, 2018 1 次提交
  5. 14 7月, 2018 1 次提交
  6. 11 7月, 2018 1 次提交
  7. 10 7月, 2018 2 次提交
  8. 03 7月, 2018 4 次提交
    • A
      proc: make proc.(ThreadBlockedError).Error() return a non-empty string · bdcbd8a8
      aarzilli 提交于
      The JSON-RPC layer doesn't like non-nil error that return an empty string
      when the Error method is called and when this happens it shuts down the
      connection to the server.
      Since we can return a ThreadBlockedError to the client it can't have an
      empty string as return value.
      
      Fixes #1251
      bdcbd8a8
    • P
      Handle gdb core files better · 1d8636cd
      Peter Sanford 提交于
      Core files created by gdb can have sections missing that would be
      present in OS created core files.
      
      We work around this by first reading PT_LOAD entries from the exe and
      then reading them from the core.
      
      Fixes #1121
      1d8636cd
    • A
      Documentation: better documentation menu, add guide to writing a client · 78377cad
      aarzilli 提交于
      * Adds some links to the more relevant documentation pages on the main
        README.md
      * Adds a document informally describing how to write a client of delve
      * Adds link to slides describing the architecture of delve
      78377cad
    • A
      *: Fix log level setting in logrus · c53c43d1
      aarzilli 提交于
      Setting the Level field of a logrus logger doesn't actually do anything
      since the Level field simply reports the log level of the last log
      message emitted on the logger.
      The right way to do that is to set logger.Logger.Level.
      Also cleans up newline characters from log messages emitted through
      logrus and fixes the direction of the arrows in the messages emitted by
      rpccommon, which was inconsistent with the arrows of gdbserial.
      c53c43d1
  9. 29 6月, 2018 1 次提交
  10. 27 6月, 2018 1 次提交
    • A
      proc,terminal,service: let headless instances run without connected clients · 9a216211
      aarzilli 提交于
      This pull request makes several changes to delve to allow headless
      instancess that are started with the --accept-multiclient flag to
      keep running even if there is no connected client. Specifically:
      
      1. Makes a headless instance started with --accept-multiclient quit
          after one of the clients sends a Detach request (previously they
          would never ever quit, which was a bug).
      2. Changes proc/gdbserial and proc/native so that they mark the
          Process as exited after they detach, even if they did not kill the
          process during detach. This prevents bugs such as #1231 where we
          attempt to manipulate a target process after we detached from it.
      3. On non --accept-multiclient instances do not kill the target
          process unless we started it or the client specifically requests
          it (previously if the client did not Detach before closing the
          connection we would kill the target process unconditionally)
      4. Add a -c option to the quit command that detaches from the
          headless server after restarting the target.
      5. Change terminal so that, when attached to --accept-multiclient,
          pressing ^C will prompt the user to either disconnect from the
          server or pause the target process. Also extend the exit prompt to
          ask if the user wants to keep the headless server running.
      
      Implements #245, #952, #1159, #1231
      9a216211
  11. 26 6月, 2018 1 次提交
    • A
      rpccommon: restore API port message · 818ed0b2
      aarzilli 提交于
      This message is used by clients to determine the port that a headless
      instance is using, therefore the format can not change or move to a
      different file handle.
      
      Fixes #1245
      818ed0b2
  12. 23 6月, 2018 1 次提交
    • A
      proc: support GNU compressed debug sections (go1.11 support) · 440b4405
      aarzilli 提交于
      Go1.11 switched to the zlib-gnu compression format for debug sections.
      Change proc and and a test in dwarf/line to support this change.
      
      Also deletes some dead code from pkg/proc/bininfo.go that hadn't been
      used in a long time.
      440b4405
  13. 22 6月, 2018 3 次提交
    • D
      *: Update vendor · de7b8c66
      Derek Parker 提交于
      de7b8c66
    • D
      *: Use structured logging · a208c897
      Derek Parker 提交于
      Implements structured logging via Logrus. This gives us a logger per
      boundry that we care about, allowing for easier parsing of logs if users
      have more than one log option enabled. Also, cleans up a lot of
      conditionals in the code by simply silencing the logger at creation as
      opposed to conditionally logging everywhere.
      a208c897
    • S
      proc: add support for separate debug info · 774b5c7c
      Sergio Lopez 提交于
      To save disk space, some distributions strip the debugging information
      from the binaries, putting it in separate files, usually distributed in
      separate packages.
      
      To locate the file containing the debug information for a certain
      binary, an ELF note named ".note.gnu.build-id" is added to the latter,
      which contains a header and a build identification. This identification
      can be used to compose a path with this form:
      
      /usr/lib/debug/.build-id/BUILDID[:2]/BUILDID[2:].debug
      
      With this patch, if Delve can't find the debug information in the main
      binary, it'll try to locate and parse ".note.gnu.build-id", to compose
      and attempt to open a path with the format described above.
      
      Fixes #1206
      774b5c7c
  14. 20 6月, 2018 1 次提交
    • A
      proc,command: fix optimized function warning in 1.9 · e19cbcef
      aarzilli 提交于
      A user complained on the mailing list about having continuous
      "optimized function warnings" on non-optimized functions when using 1.9.
      This commit fixes the problem by disabling optimized function detection
      on 1.9 and earlier (where it's impossible) and adds a test so we don't
      break it again in the future.
      e19cbcef
  15. 15 6月, 2018 2 次提交
    • A
      proc: allow "package/path".varname syntax · 2a2d1040
      aarzilli 提交于
      If the application being debugged imports two packages with the same
      name (but different paths) there was no way to disambiguate the two,
      since the character '/' can not appear inside a go identifier.
      
      By allowing users to use a string literal as the package name a package
      path can be specified.
      2a2d1040
    • A
      service,logflags: log all RPC messages · 454491ce
      aarzilli 提交于
      We occasionally receive bug reports from users of VSCode-go and GoLand.
      GoLand has its own way of capturing the packet exchange between itself
      and delve but VSCode-go (supposedly) doesn't.
      So far this hasn't been a problem since all bug reports were obvious
      bugs on the plugin or easy to reproduce without VSCode-go, but it might
      be helpful in the future to have a way to log the packet exchange
      between dlv and a frontend.
      
      This commit adds a --log-output option to enable logging of all rpc
      messages and changes service/rpccommon accordingly.
      454491ce
  16. 12 6月, 2018 10 次提交
  17. 09 6月, 2018 2 次提交
  18. 24 5月, 2018 3 次提交
    • S
      travis: install dwz package on linux tests · 33780e96
      Sergio Lopez 提交于
      33780e96
    • S
      proc_test: implement test for DWZ compressed DWARF · bafa5120
      Sergio Lopez 提交于
      If dwz binary is available in the system, test delve's ability to find
      deduplicated symbols in the DWARF information.
      
      dwzcompression.go contains a small C function (void fortytwo()) which
      calls glibc's fprintf with stdin as first argument. Normally, stdin
      will be present as a DW_TAG_variable as part of a DW_TAG_compile_unit
      named dwzcompression.cgo2.c.
      
      After running dwz on the binary, stdin is moved to a
      DW_TAG_partial_unit, which is imported from dwzcompression.cgo2.c with
      a DW_TAG_imported_unit.
      
      This test verifies that delve is able to find stdin symbol's type, as a
      way to confirm it understands dwz's compressed/deduplicated DWARF
      information.
      bafa5120
    • S
      proc/test/support: implement EnableDWZCompression · ed71248f
      Sergio Lopez 提交于
      The EnableDWZCompression flag allows tests to request BuildFixture to
      run "dwz" on the Fixture's resulting binary to compress/deduplicate its
      DWARF sections.
      ed71248f