1. 12 6月, 2018 1 次提交
  2. 09 6月, 2018 2 次提交
  3. 24 5月, 2018 4 次提交
    • 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
    • S
      proc: add support for dwz compressed DWARF · 38307f92
      Sergio Lopez 提交于
      'dwz' is a tool that reduces the size of DWARF sections by
      deduplicating symbols. The deduplicated symbols are moved from their
      original 'compile unit' to a 'partial unit', which is then referenced
      from its original location with an 'imported unit' tag.
      
      In the case of Go binaries, all symbols are located in a single
      'compile unit', and the name of each symbol contains a reference to its
      package, so 'dwz' is not able to deduplicate them. But still, some C
      symbols included in the binary are deduplicated, which also alters the
      structure of the DWARF sections, making delve unable to parse them
      (crashing in the attempt).
      
      While it would've been possible to simply ignore the C symbols, or
      blindly loading all then into BinaryInfo members (packageVars,
      Functions...), for correctness sake this change tries to do the right
      thing, staging symbols into temporary partialUnit objects, moving them
      to BinaryInfo when they are actually requested  by a 'imported unit'
      tag.
      38307f92
  4. 23 5月, 2018 1 次提交
  5. 22 5月, 2018 1 次提交
  6. 19 5月, 2018 1 次提交
    • A
      proc/native,proc/gdbserial: let target access terminal · cc86bde5
      aarzilli 提交于
      Change the linux verison of proc/native and proc/gdbserial (with
      debugserver) so that they let the target process use the terminal when
      delve is launched in headless mode.
      
      Windows already worked, proc/gdbserial (with rr) already worked.
      I couldn't find a way to make proc/gdbserial (with lldb-server) work.
      
      No tests are added because I can't think of a way to test for
      foregroundness of a process.
      
      Fixes #65
      cc86bde5
  7. 16 5月, 2018 1 次提交
    • J
      Keep searching for file:line until found · c7cde8b1
      Jay Mundrawala 提交于
      Go seems to be generating multiple compilation units that have
      the same file. I think this happens for functions that get inlined.
      Without this patch, those inlined functions break the ability to set
      a breakpoint at other lines in the file. I was able to load the same
      binary in gdb and set a breakpoints throughout the file without issue.
      
      ```
      ➜ objdump --dwarf=decodedline automate-gateway | grep handler/users.go
      .../handler/users.go:[++]
      s/.../handler/users.go           20            0xb6dd88
      .../handler/users.go:[++]
      s/.../handler/users.go           20            0xb6e50f
      .../handler/users.go:[++]
      s/automate-gateway/handler/users.go           32            0xb66640
      ```
      
      Inlined functions are still a little weird. setting a breakpoint on
      a function that gets inlined picks the first occurence. That being
      said, I think delve should still do something reasonable for the rest
      of the lines in the file.
      c7cde8b1
  8. 10 5月, 2018 1 次提交
  9. 08 5月, 2018 1 次提交
  10. 05 5月, 2018 1 次提交
  11. 27 4月, 2018 1 次提交
  12. 24 4月, 2018 3 次提交
    • A
      proc: Flag shadowed arguments as shadowed · 7fd47749
      aarzilli 提交于
      Fixes #951
      7fd47749
    • A
      proc: cache entire frame in FrameToScope instead of variablesByTag · 21be5946
      aarzilli 提交于
      Caching the frame in variablesByTag is problematic:
      
      1. accounting for variables that are (partially) stored in registers is
      complicated (see issue #1106)
      2. for some types (strings, interfaces...) simply creating the Variable
      object reads memory, which therefore happens before we can do any
      caching.
      
      Instead cache the entire frame when the EvalScope object is created.
      The cached range is between the SP value of the current frame and the
      CFA of the preceeding frame, if available, or the CFA of the current
      frame otherwise.
      
      Fixes #1106
      21be5946
    • A
      proc: change memCache to delay reading · a5574bcd
      aarzilli 提交于
      Change memCache so that the preloaded memory is not read immediately
      but only after the actual read to the preloaded range.
      
      This allows us to request caching the entire stack frame every time we
      create an eval scope and no unnecessary reads will be made even if the
      user is just trying to evaluate a global variable.
      a5574bcd
  13. 20 4月, 2018 2 次提交
  14. 19 4月, 2018 1 次提交
  15. 18 4月, 2018 1 次提交
  16. 14 4月, 2018 4 次提交
    • A
      proc: Short circuit evaluation of && and || like go does · b9c4a1d6
      aarzilli 提交于
      Change evaluation of binary operators so that both && and || only
      evaluate their second argument conditionally, like go does.
      b9c4a1d6
    • A
      proc_test: instrument TestFrameEvaluation · e1f2626b
      aarzilli 提交于
      I've seen TestFrameEvaluation fail in CI in the past. It's been a while
      since the last time and I couldn't reproduce it locally at all. I'd
      like to have some instrumentation in case it happens again.
      e1f2626b
    • A
      terminal: make printcontext use SelectedGoroutine · 4f70ff0a
      aarzilli 提交于
      printcontext should use SelectedGoroutine instead of trusting that the
      goroutine running on current thread matches the SelectedGoroutine.
      
      When the user switches to a parked goroutine CurrentThread and
      SelectedGoroutine will diverge.
      
      Almost all calls to printcontext are safe, they happen after a continue
      command returns when SelectedGoroutine and CurrentThread always agree,
      but the calls in frameCommand and listCommand are wrong.
      
      Additionally we should stop reporting an error when the debugger is
      stopped on an unknown PC address.
      4f70ff0a
    • A
      4e177bb9
  17. 11 4月, 2018 4 次提交
  18. 29 3月, 2018 1 次提交
  19. 28 3月, 2018 1 次提交
  20. 27 3月, 2018 1 次提交
    • A
      proc: support inlining · 290e8e75
      aarzilli 提交于
      Go 1.10 added inlined calls to debug_info, this commit adds support
      for DW_TAG_inlined_call to delve, both for stack traces (where
      inlined calls will appear as normal stack frames) and to correct
      the behavior of next, step and stepout.
      
      The calls to Next and Frame of stackIterator continue to work
      unchanged and only return real stack frames, after reading each line
      appendInlinedCalls is called to unpacked all the inlined calls that
      involve the current PC.
      
      The fake stack frames produced by appendInlinedCalls are
      distinguished from real stack frames by having the Inlined attribute
      set to true. Also their Current and Call locations are treated
      differently. The Call location will be changed to represent the
      position inside the inlined call, while the Current location will
      always reference the real stack frame. This is done because:
      
      * next, step and stepout need to access the debug_info entry of
      the real function they are stepping through
      * we are already manipulating Call in different ways while Current
      is just what we read from the call stack
      
      The strategy remains mostly the same, we disassemble the function
      and we set a breakpoint on each instruction corresponding to a
      different file:line. The function in question will be the one
      corresponding to the first real (i.e. non-inlined) stack frame.
      
      * If the current function contains inlined calls, 'next' will not
      set any breakpoints on instructions that belong to inlined calls. We
      do not do this for 'step'.
      
      * If we are inside an inlined call that makes other inlined
      functions, 'next' will not set any breakpoints that belong to
      inlined calls that are children of the current inlined call.
      
      * If the current function is inlined the breakpoint on the return
      address won't be set, because inlined frames don't have a return
      address.
      
      * The code we use for stepout doesn't work at all if we are inside
      an inlined call, instead we call 'next' but instruct it to remove
      all PCs belonging to the current inlined call.
      290e8e75
  21. 23 3月, 2018 2 次提交
    • Y
      Extend the "frame" command to set the current frame. (#1110) · 82aff3f1
      Yasushi Saito 提交于
      * Extend the "frame" command to set the current frame.
      
      Command
      
        frame 3
      
      sets up so that subsequent "print", "set", "whatis" command
      will operate on frame 3.
      
        frame 3 print foo
      
      continues to work.
      
      Added "up", "down". They move the current frame up or down.
      
      Implementation note:
      
      This changes removes "scopePrefix" mode from the terminal/command.go and instead
      have the command examine the goroutine/frame value to see if it is invoked in a
      scoped context.
      
      * Rename Command.Frame -> Command.frame.
      82aff3f1
    • A
      proc,vendor: show global variables in disassembly · ec8dc3a1
      aarzilli 提交于
      updates vendored version of x86asm, adds a symbol lookup function to
      pass to the disassembler.
      
      This will show global symbol names in the disassembly like go tool
      objdump does.
      ec8dc3a1
  22. 21 3月, 2018 2 次提交
  23. 20 3月, 2018 1 次提交
  24. 16 3月, 2018 1 次提交
  25. 09 3月, 2018 1 次提交