profile
viewpoint
chainhelen chainhelen China Open source, can I help you.

chainhelen/bptree 4

Implementing bplustree base on disk(kv store).

chainhelen/godbg 3

the debugger on linux platform for go (Toys)

chainhelen/ACdream 0

programming community of ACM (node.js + express4 + mongodb)

chainhelen/c4 0

C in four functions

chainhelen/chlang 0

to be a new language interpreter

chainhelen/delve 0

Delve is a debugger for the Go programming language.

chainhelen/deno 0

A secure TypeScript runtime on V8

issue commentgo-delve/delve

Failure to read stdin on programs run under dlv.

Need to use --headless mode

Run server and client respectively on two terminals.

// server
dlv debug main.go --headless --listen 127.0.0.1:3100

// client 
 dlv connect 127.0.1:3100 
ianbruene

comment created time in 3 days

pull request commentgo-delve/delve

Only load floating point registers when needed [WIP]

En, about

Benchmark before:

BenchmarkConditionalBreakpoints-4   	       1	4327350142 ns/op

Benchmark after:

BenchmarkConditionalBreakpoints-4   	       1	3852642917 ns/op

When I do benchmark (previous similar optimizations), the fluctuation is very big every time and the value is inconspicuous. Of course it must be optimized in theory.

I just want to konw how to generate reliable data. Maybe remove the maximum, remove the minimum, take the average?

aarzilli

comment created time in 3 days

pull request commentgo-delve/delve

scripts/make: do not test buildmode=pie on linux/386

1. Not one cpu, in my local docker

// top
%Cpu0  :  3.9 us,  0.0 sy,  0.0 ni, 95.1 id,  0.0 wa,  0.0 hi,  1.0 si,  0.0 st
%Cpu1  :  2.9 us,  1.9 sy,  0.0 ni, 95.2 id,  0.0 wa,  0.0 hi,  0.0 si,  0.0 st
%Cpu2  :  2.9 us,  1.0 sy,  0.0 ni, 96.1 id,  0.0 wa,  0.0 hi,  0.0 si,  0.0 st
%Cpu3  :  1.9 us,  1.0 sy,  0.0 ni, 96.1 id,  0.0 wa,  0.0 hi,  1.0 si,  0.0 st
// main.go
package main

import (
  "fmt"
  "runtime"
)

func main() {
  fmt.Println(runtime.NumCPU())
}

go run main.go

4

2. And I bought the vps so that I could finish 386 linux debugger code, nothing else on that machine. I will send ip/user/passward to your email, you can also check what happend.

3. I think I need to build one full 386 linux vm on local pc and try to find more infomation.

aarzilli

comment created time in 3 days

pull request commentgo-delve/delve

scripts/make: do not test buildmode=pie on linux/386

Tow scripts 

for ((i=1; i<=100; i++)); do go test -v -test.run TestNextConcurrent$ ; done

for ((i=1; i<=100; i++)); do go test -test-buildmode=pie -v -test.run TestNextConcurrent$ ; done

(Not only TestNextConcurrent, other cases are same) In fact, I have tow 386 circumstances.

  1. I386/centos (2Core/4G) remote vps hang (exceed 10m) with high probability on both pie and nopie. I can see the generated exe didn't exit and output nothing when hanging.

  2. I386/centos local docker on amd64 is always ok.

I can't explain this , myabe need to collect more information to find mini reproduction-conditions.

aarzilli

comment created time in 3 days

issue commentgo-delve/delve

Stepping into interface method calls

It is implemented by #1915, so this issue can be closed.

aarzilli

comment created time in 4 days

startedphil-opp/blog_os

started time in 6 days

Pull request review commentgo-delve/delve

terminal/command: Add 'reload' command

 func (s *ServerImpl) Run() error { 		Foreground:           s.config.Foreground, 		DebugInfoDirectories: s.config.DebugInfoDirectories, 		CheckGoVersion:       s.config.CheckGoVersion,+		Packages:             s.config.Packages,

I have try it, this s.config.Packages always be []string{}. If i build and reload in other path, debugger can't not realod the right project.

Ex: dlv debug ~/main.go in other path.

alexsaezm

comment created time in 8 days

push eventchainhelen/delve

Derek Parker

commit sha 7571c9d7c7d14fad45dbce4368fafbb9079bf1ac

travis.yml: Add go_import_path This makes it so that CI will still work on forks, if necessary.

view details

Derek Parker

commit sha 697310fc296883780f1a57cf162a8c68ebb187d3

scripts: Update gen-travis.go

view details

Derek Parker

commit sha c4fd80fcd0c2cca60ae894ff38ec6460a9288231

pkg/proc: Clean up proc.go This patch moves out unrelated types, variables and functions from proc.go into a place where they make more sense.

view details

Derek Parker

commit sha 475551cf3d25ead83fc45a683bd71ae9883a0ac6

pkg/proc: Reset time after continue in benchmarks Reset the timer once we start executing the code paths we actually care to measure, since the setup and continue steps can be relatively expensive. Also, disable recording on benchmarks. This seems less useful, and also cuts out a lot of noise when benchmarks are run.

view details

Alessandro Arzilli

commit sha 8bb93e9ae19589b7ee665225c2206ef118928ae9

proc/gdbserial,debugger: allow clients to stop a recording (#1890) Allows Delve clients to stop a recording midway by sending a Command('halt') request. This is implemented by changing debugger.New to start recording the process on a separate goroutine while holding the processMutex locked. By locking the processMutex we ensure that almost all RPC requests will block until the recording is done, since we can not respond correctly to any of them. API calls that do not require manipulating or examining the target process, such as "IsMulticlient", "SetApiVersion" and "GetState(nowait=true)" will work while we are recording the process. Two other internal changes are made to the API: both GetState and Restart become asynchronous requests, like Command. Restart because this way it can be interrupted by a StopRecording request if the rerecord option is passed. GetState because clients need a call that will block until the recording is compelted and can also be interrupted with a StopRecording. Clients that are uninterested in allowing the user to stop a recording can ignore this change, since eventually they will make a request to Delve that will block until the recording is completed. Clients that wish to support this feature must: 1. call GetState(nowait=false) after connecting to Delve, before any call that would need to manipulate the target process 2. allow the user to send a StopRecording request during the initial GetState call 3. allow the user to send a StopRecording request during any subsequent Restart(rerecord=true) request (if supported). Implements #1747

view details

Derek Parker

commit sha ccd438c65f5054553f49c21f603ee4a677191020

pkg/proc: Move proc exec funcs to Target methods

view details

Derek Parker

commit sha 4abf4f1c1fd5c686c9075b21fd839b2b300272ee

pkg/proc: Rename proc.go -> target_exec.go

view details

Alessandro Arzilli

commit sha 2786e46c5ff1650f8a712b378d8a2f9a1590fc40

vendor: check vendor directory (#1962) Remove gitignore rule preventing vendor/modules.txt file from being commited, add check that the vendor directory is synchronized.

view details

Derek Parker

commit sha 5177c247f6f032042009d2f93e9dc2e625e846cc

pkg/proc: Remove unused parameter Also cleanup some typos.

view details

Derek Parker

commit sha 5d109cb197254aef802b186ba9c8b83155f1942e

*: Add some more documentation to exported types and methods

view details

Derek Parker

commit sha 85c34e47eeaceb6d3a1f8f32d274bae6ca70ad39

*: mv scripts _scripts Instead of selectively excluding this directory, hide it from the go tooling by applying the "_" prefix.

view details

push time in 8 days

push eventchainhelen/delve

Derek Parker

commit sha 7571c9d7c7d14fad45dbce4368fafbb9079bf1ac

travis.yml: Add go_import_path This makes it so that CI will still work on forks, if necessary.

view details

Derek Parker

commit sha 697310fc296883780f1a57cf162a8c68ebb187d3

scripts: Update gen-travis.go

view details

Derek Parker

commit sha c4fd80fcd0c2cca60ae894ff38ec6460a9288231

pkg/proc: Clean up proc.go This patch moves out unrelated types, variables and functions from proc.go into a place where they make more sense.

view details

Derek Parker

commit sha 475551cf3d25ead83fc45a683bd71ae9883a0ac6

pkg/proc: Reset time after continue in benchmarks Reset the timer once we start executing the code paths we actually care to measure, since the setup and continue steps can be relatively expensive. Also, disable recording on benchmarks. This seems less useful, and also cuts out a lot of noise when benchmarks are run.

view details

Alessandro Arzilli

commit sha 8bb93e9ae19589b7ee665225c2206ef118928ae9

proc/gdbserial,debugger: allow clients to stop a recording (#1890) Allows Delve clients to stop a recording midway by sending a Command('halt') request. This is implemented by changing debugger.New to start recording the process on a separate goroutine while holding the processMutex locked. By locking the processMutex we ensure that almost all RPC requests will block until the recording is done, since we can not respond correctly to any of them. API calls that do not require manipulating or examining the target process, such as "IsMulticlient", "SetApiVersion" and "GetState(nowait=true)" will work while we are recording the process. Two other internal changes are made to the API: both GetState and Restart become asynchronous requests, like Command. Restart because this way it can be interrupted by a StopRecording request if the rerecord option is passed. GetState because clients need a call that will block until the recording is compelted and can also be interrupted with a StopRecording. Clients that are uninterested in allowing the user to stop a recording can ignore this change, since eventually they will make a request to Delve that will block until the recording is completed. Clients that wish to support this feature must: 1. call GetState(nowait=false) after connecting to Delve, before any call that would need to manipulate the target process 2. allow the user to send a StopRecording request during the initial GetState call 3. allow the user to send a StopRecording request during any subsequent Restart(rerecord=true) request (if supported). Implements #1747

view details

Derek Parker

commit sha ccd438c65f5054553f49c21f603ee4a677191020

pkg/proc: Move proc exec funcs to Target methods

view details

Derek Parker

commit sha 4abf4f1c1fd5c686c9075b21fd839b2b300272ee

pkg/proc: Rename proc.go -> target_exec.go

view details

Alessandro Arzilli

commit sha 2786e46c5ff1650f8a712b378d8a2f9a1590fc40

vendor: check vendor directory (#1962) Remove gitignore rule preventing vendor/modules.txt file from being commited, add check that the vendor directory is synchronized.

view details

Derek Parker

commit sha 5177c247f6f032042009d2f93e9dc2e625e846cc

pkg/proc: Remove unused parameter Also cleanup some typos.

view details

Derek Parker

commit sha 5d109cb197254aef802b186ba9c8b83155f1942e

*: Add some more documentation to exported types and methods

view details

Derek Parker

commit sha 85c34e47eeaceb6d3a1f8f32d274bae6ca70ad39

*: mv scripts _scripts Instead of selectively excluding this directory, hide it from the go tooling by applying the "_" prefix.

view details

chainhelen

commit sha e6a1196d14f71a653bd16d74a639b814658568cd

pkg/config: add `disassemble-flavor` option for config Allow user to specify output syntax flavor of assembly in the disassemble command. Close #415

view details

push time in 8 days

pull request commentgo-delve/delve

test: Add check for generating .travis.yml

Oh, get it.

chainhelen

comment created time in 11 days

PR opened go-delve/delve

test: Add check for generating .travis.yml
+3 -0

0 comment

1 changed file

pr created time in 12 days

create barnchchainhelen/delve

branch : travis_check

created branch time in 12 days

Pull request review commentgo-delve/delve

pkg,service: add `-f` option for cmd disassemble

 func (c *Commands) sourceCommand(t *Term, ctx callContext, args string) error { 	return c.executeFile(t, args) } -var disasmUsageError = errors.New("wrong number of arguments: disassemble [-a <start> <end>] [-l <locspec>]")+func indexNextArgs(args string, splitStrs []string) int {+	for i := range args {+		for _, splitStr := range splitStrs {+			if i+len(splitStr) <= len(args) && args[i:i+len(splitStr)] == splitStr {+				return i+			}+		}+	}+	return -1+} -func disassCommand(t *Term, ctx callContext, args string) error {-	var cmd, rest string+func nextDisasmArgs(args string, splitStrs []string) (string, string, error) {

I think we should use split2PartsBySpace instead of doing this.

Do this because I notice go allow this expression main . main for -l. so

disasm   -l    main         .   main     -f    intel 

is ok.

Even

// main.go
package main

import (
	"fmt"
)

func main() {
	fmt   .       Printf("hello world\n")
}

go run main.go run ok.

chainhelen

comment created time in 13 days

issue commentgo-delve/delve

How to avoid print truncating output?

Two ways

  1. Re-config max-array-values by command config, here is doc
  2. print ccid[64:]
siddjain

comment created time in 13 days

push eventchainhelen/delve

chainhelen

commit sha 75cbd2aba0abca9dfaa26c8d424380fadf24c99b

Fix: deepsource

view details

push time in 14 days

push eventchainhelen/delve

chainhelen

commit sha 2116d3292b6ade9c0509b74c5bd627b54c88bd15

Fix: location

view details

push time in 14 days

push eventchainhelen/delve

chainhelen

commit sha 67c72e6b514fff3846751caaae2e1af5a65cbce1

Fix: recover integration2_test.go

view details

push time in 15 days

PR opened go-delve/delve

pkg,service: add -f for cmd disassemble

Allow user to specify output syntax flavor of assembly in the disassemble command.

Close #415

+89 -52

0 comment

4 changed files

pr created time in 16 days

create barnchchainhelen/delve

branch : flavor

created branch time in 16 days

push eventchainhelen/delve

Derek Parker

commit sha edc5436fe36de8c9f818db1ee29462afe297e194

*: Add DeepSource CI config

view details

Derek Parker

commit sha ad75f78c4e69c8c62c30e80aa9e47cf336534435

*: Fix go vet complaints (#1935) * *: Fix go vet struct complaints * *: Fix struct vet issue on linux * *: Ignore proc/native in go vet check We have to do some unsafe pointer manipulation that will never make go vet happy within the proc/native package. Ignore it for runs of go vet.

view details

chainhelen

commit sha 65d7f5c65f3f5bc73490771106e8a8bf4903129a

pkg/proc: Prevent program crash when called meanless expression (#1934) If we call one expression which is the fake method of meanless string, `delve` will panic. Strengthen the inspection of boundary conditions when supporting function calls on non-struct types. Update: #1871

view details

Derek Parker

commit sha 731a7fc125e3d9899d6b1f9c082cdec81a4714d2

*: consolidate appends where possible

view details

Sourya Vatsyayan

commit sha 5ef973c742287f323895dee46a121adfd054f0e3

*: Exclude `_fixtures` for DeepSource issues (#1941)

view details

Alessandro Arzilli

commit sha 4aee281520c6cdc3be9bbbf3cf27060d1ed1759d

travis-ci: autogenerate .travis.yml (#1939) Autogenerates .travis.yml configuration using the contents of pkg/goversion/compat.go. Also: - excludes testing of windows/amd64/tip and linux/arm64/tip (Travis-CI can't get Go at tip for those combinations). - Removes Go 1.11 from pkg/goversion/compat.go which we don't test anymore.

view details

Alessandro Arzilli

commit sha c6de961be886060dea50f5bfa1ee530bce00671b

terminal: add display command (#1917) Implements #1256

view details

Alessandro Arzilli

commit sha e1cfd727959d465314cc812be7c1576952c01ba7

proc: fix bad cached goroutines after a call injection (#1926) Inserts a call to ClearAllGCache into stepInstructionOut so that cached goroutine state is not inconsistent after an injected function call.\ Fixes #1925

view details

Alessandro Arzilli

commit sha 0cfd52787fd85e4fcdf229eaadef08e92c6e9aea

proc/gdbserial: add logging when Thread.Location will fail (#1937) Add logging for Issue #1927. The bug happens during the call to api.ConvertThread, returning an error will not suffice since ConvertThread will not surface it. Updates #1927

view details

Derek Parker

commit sha f80674979a1f645a8c1898c9fcde12c47981153d

*: Ignore scripts in deepsource analysis

view details

Derek Parker

commit sha f05ce0947318f0733d27868a8584e71644c50de6

*: Fix use of deprecated functionality

view details

Derek Parker

commit sha 5a83bdd6325d45963454236f5dfc7b1b398ede31

pkg/terminal: Use buffered channel for signal notify

view details

Derek Parker

commit sha ad2563f008058d8bbe638906a0db81827a092cb2

pkg/terminal: Remove unused slice and append

view details

Derek Parker

commit sha 3c683ae30ff36c670c47edf1d085b621141e1ef8

pkg/terminal: Use less permissive file permission on history file

view details

Alessandro Arzilli

commit sha 7cd12c34fd4dbbfe272685f692a269d9daaa8cea

proc,dwarf: cache debug.Entry objects (#1931) Instead of rescanning debug_info every time we want to read a function (either to find inlined calls or its variables) cache the tree of dwarf.Entry that we would generate and use that. Benchmark before: BenchmarkConditionalBreakpoints-4 1 5164689165 ns/op Benchmark after: BenchmarkConditionalBreakpoints-4 1 4817425836 ns/op Updates #1549

view details

Alessandro Arzilli

commit sha 112049176fce4324bd752460c2454d6d41562d8b

proc: fix switch to goroutine stack (#1948) When switching to the goroutine stack the stack iterator registers might not have any entry for the BP register, make sure to add it instead of just trying to change it.

view details

push time in 16 days

push eventchainhelen/delve

chainhelen

commit sha fa7b46f709847484ee27f0af3abaaee44733e900

pkg/proc: Prevent program crash when called meanless expression If we call one expression which is the fake method of meanless string, `delve` will panic. Strengthen the inspection of boundary conditions when supporting function calls on non-struct types. Update: #1871

view details

push time in 20 days

push eventchainhelen/delve

chainhelen

commit sha 5774066ddb5cafa01ce29b82aa437af01ac1007a

*: Transfer 32-bit test into travis from cirrus (#1932)

view details

MinJae Kwon

commit sha 25d8856cd1bb1f71d452ecc387e0cfb29c5bc93f

*: run gofmt (#1928)

view details

push time in 20 days

push eventchainhelen/delve

chainhelen

commit sha 5774066ddb5cafa01ce29b82aa437af01ac1007a

*: Transfer 32-bit test into travis from cirrus (#1932)

view details

MinJae Kwon

commit sha 25d8856cd1bb1f71d452ecc387e0cfb29c5bc93f

*: run gofmt (#1928)

view details

chainhelen

commit sha c0c9e1bc07a8fa4f94b8cdad0f4f6d15dfa4ee37

pkg/proc: Prevent program crash when called meanless expression If we call one expression which is the fake method of meanless string, `delve` will panic. Strengthen the inspection of boundary conditions when supporting function calls on non-struct types. Update: #1871

view details

push time in 20 days

push eventchainhelen/delve

chainhelen

commit sha a2f990079e3876a7e5c8c217072e1ad90d0eb8d5

pkg/proc: Prevent program crash when called meanless expression If we call one expression which is the fake method of meanless string, `delve` will panic. Strengthen the inspection of boundary conditions when supporting function calls on non-struct types. Update: #1871

view details

push time in 20 days

push eventchainhelen/delve

chainhelen

commit sha af9ecb1d7a128ab0944a9989e9b26df8fd5d69ec

pkg/proc: Prevent program crash when called meanless expression If we call one expression which is the fake method of meanless string, `delve` will panic. Strengthen the inspection of boundary conditions when supporting function calls on non-struct types. Update: #1871

view details

push time in 20 days

PR opened go-delve/delve

pkg/proc: Prevent program crash when callied meanless expression

If we call one expression which is the fake method of meanless string, delve will panic. Strengthen the inspection of boundary conditions when supporting function calls on non-struct types.

Update: #1871


Eg:

package main

import (
	"fmt"
)

func main() {
	fmt.Printf("hello world\n")  // set breakpoint
}
(dlv) call "delve".callme()
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x6f614f]

goroutine 51 [running]:
github.com/go-delve/delve/pkg/proc.(*Variable).findMethod(0xc0001945a0, 0xc000362548, 0x6, 0xc0001945a0, 0x0, 0x0)
	/media/chainhelen/LENOVO/xlworkspace/code/go/src/github.com/go-delve/delve/pkg/proc/eval.go:1829 +0x16f
github.com/go-delve/delve/pkg/proc.(*EvalScope).evalStructSelector(0xc0000ab2b0, 0xc0000c1240, 0x5, 0xc000362548, 0x6)
	/media/chainhelen/LENOVO/xlworkspace/code/go/src/github.com/go-delve/delve/pkg/proc/eval.go:1048 +0xbe
github.com/go-delve/delve/pkg/proc.(*EvalScope).evalAST(0xc0000ab2b0, 0xcb9440, 0xc0000c1240, 0xc000498000, 0x300, 0x0)
	/media/chainhelen/LENOVO/xlworkspace/code/go/src/github.com/go-delve/delve/pkg/proc/eval.go:639 +0x718
github.com/go-delve/delve/pkg/proc.funcCallEvalFuncExpr(0xc0000ab2b0, 0xc0003c3d30, 0x4abf00, 0x0, 0x0)
	/media/chainhelen/LENOVO/xlworkspace/code/go/src/github.com/go-delve/delve/pkg/proc/fncall.go:404 +0x87
github.com/go-delve/delve/pkg/proc.evalFunctionCall(0xc0000ab2b0, 0xc0000d5bc0, 0x0, 0x0, 0xc000068e50)
	/media/chainhelen/LENOVO/xlworkspace/code/go/src/github.com/go-delve/delve/pkg/proc/fncall.go:273 +0x25d
github.com/go-delve/delve/pkg/proc.(*EvalScope).evalAST(0xc0000ab2b0, 0xcb8d00, 0xc0000d5bc0, 0x1, 0x1, 0x40)
	/media/chainhelen/LENOVO/xlworkspace/code/go/src/github.com/go-delve/delve/pkg/proc/eval.go:606 +0x2c2
github.com/go-delve/delve/pkg/proc.(*EvalScope).EvalExpression(0xc0000ab2b0, 0xc00016c0a0, 0x10, 0x1, 0x1, 0x40, 0x40, 0xffffffffffffffff, 0x0, 0x0, ...)
	/media/chainhelen/LENOVO/xlworkspace/code/go/src/github.com/go-delve/delve/pkg/proc/eval.go:77 +0x56c
created by github.com/go-delve/delve/pkg/proc.EvalExpressionWithCalls
	/media/chainhelen/LENOVO/xlworkspace/code/go/src/github.com/go-delve/delve/pkg/proc/fncall.go:185 +0x35a

+7 -2

0 comment

2 changed files

pr created time in 20 days

create barnchchainhelen/delve

branch : fncall_string_panic

created branch time in 20 days

push eventchainhelen/delve

Derek Parker

commit sha 15509d4dc9c56f46c0ef7d2baf8d25c7355e46a6

*: Remove appveyor, use travis windows support (#1919) * *: Remove appveyor, use travis windows support * cmd/dlv: Update TestOutput for Travis on Windows * cmd/dlv: Skip TestGeneratedDoc in Travis on Windows * Reduce number of jobs in matrix We only really want to test the full arch/go matrix on Linux. For every other os/arch/go combination, only run Go tip and the latest supported version.

view details

Alessandro Arzilli

commit sha 88a0e1727ac84db6b443c2732a48acebb81ae859

terminal: preserve arguments and prefixes when repeating last command (#1930)

view details

push time in 20 days

PR closed go-delve/delve

DO_NOT_MERGE DO_NOT_COMMIT DO_NOT REVIEW

https://github.com/go-delve/delve/runs/501900175 https://github.com/go-delve/delve/runs/501900189 https://github.com/go-delve/delve/runs/498813610 https://github.com/go-delve/delve/runs/498813609 https://github.com/go-delve/delve/runs/498813615

There are similar probabilistic CI issue on cirrus, but I never reproduct locally (go1.14 go 1.12 go1.13 linux/386). Not sure why. I will close this pr after investigation.

Please do not merge, do not commit, do not review. Thx.

+87 -32

0 comment

5 changed files

chainhelen

pr closed time in 20 days

pull request commentgo-delve/delve

terminal: add display command

Ex:

package main

import (
	"fmt"
)

func main() {
	s := 100   
	p := 101         // set breakpoint and continue here.
	fmt.Printf("%d %d\n", s, p)
}
(dlv) display -a s
0: s = 100
(dlv) display -a p
1: p = 4236625  

(dlv) c
100 101
0: s = error Process 32653 has exited with status 0
1: p = error Process 32653 has exited with status 0
Process 32653 has exited with status 0

(dlv) n
Process 32653 has exited with status 0

(dlv) s
Process 32653 has exited with status 0

(dlv) si
0: s = error Process 32653 has exited with status 0
1: p = error Process 32653 has exited with status 0
Process 32653 has exited with status 0

(dlv) c
0: s = error Process 32653 has exited with status 0
1: p = error Process 32653 has exited with status 0
Process 32653 has exited with status 0

It looks some diffrents about c/s/si/n after process existed. And I think dlv could not display any info of cmd display when process existed.

aarzilli

comment created time in 21 days

push eventchainhelen/delve

Alessandro Arzilli

commit sha 88a0e1727ac84db6b443c2732a48acebb81ae859

terminal: preserve arguments and prefixes when repeating last command (#1930)

view details

chainhelen

commit sha fd7cb7f3b84cb0d009c83b09c4873c2869d3baa1

*: Transfer 32-bit test into travis from cirrus

view details

push time in 21 days

issue commentgo-delve/delve

Doesn't exit when binary fails with exit code 1

No exit and you can dlv connect then quit . Even restart for a new round of debugging, it depend on user.

But maybe it need to be written into FAQ.

mingrammer

comment created time in 21 days

issue commentgo-delve/delve

Still can't install on ARM

Armv7l is 32bit, delve only support 64bit for ARM linux at present.

kevindtimm

comment created time in 21 days

push eventchainhelen/delve

chainhelen

commit sha 9a6defbe92a18d406cf5c40a1668e09d039905c0

Test timeout

view details

push time in 22 days

PullRequestEvent

push eventchainhelen/delve

chainhelen

commit sha 0b5b1a1f50313c6340e5915cd61948f118d50c38

*: Transfer 32-bit test into travis from cirrus

view details

push time in 22 days

PR closed go-delve/delve

DO_NOT_MERGE DO_NOT_COMMIT DO_NOT REVIEW

https://github.com/go-delve/delve/runs/501900175 https://github.com/go-delve/delve/runs/501900189 https://github.com/go-delve/delve/runs/498813610 https://github.com/go-delve/delve/runs/498813609 https://github.com/go-delve/delve/runs/498813615

There are similar probabilistic CI issue on cirrus, but I never reproduct locally (go1.14 go 1.12 go1.13 linux/386). Not sure why. I will close this pr after investigation.

Please do not merge, do not commit, do not review. Thx.

+88 -32

0 comment

5 changed files

chainhelen

pr closed time in 22 days

Pull request review commentgo-delve/delve

*: Transfer 32-bit test into travis from cirrus

 before_install:   - export GOFLAGS=-mod=vendor   - if [ $TRAVIS_OS_NAME = "linux" ]; then sudo apt-get -qq update; sudo apt-get install -y dwz; echo "dwz version $(dwz --version)"; fi   - if [ $TRAVIS_OS_NAME = "windows" ]; then choco install procdump make; fi+++# 386 linux+jobs:+  include:+    -  os: linux+       services: docker+       env: go_32_version=1.14+    -  os: linux+       services: docker+       env: go_32_version=1.13+    -  os: linux+       services: docker+       env: go_32_version=1.12

Cann't use matrix, for example

env:
   - go_32_version=1.14
   - go_32_version=1.13
   - go_32_version=1.12

when in jobs.include, it have no effect.

chainhelen

comment created time in 22 days

PR opened go-delve/delve

*: Transfer 32-bit test into travis from cirrus
  1. 32bit docker on cicurs CI https://github.com/go-delve/delve/runs/501900175 https://github.com/go-delve/delve/runs/501900189 https://github.com/go-delve/delve/runs/498813610 https://github.com/go-delve/delve/runs/498813609 https://github.com/go-delve/delve/runs/498813615 ... This 32bit docker on cirrus CI failed frequency. The erros all have similar memory problems.
    Then I do for ((i=1; i<=300; i++)); do go test -v -test.run TestStepConcurrentDirect ; done on cirrus CI, see the error every CI period. (150 loops is enough in fact)

  2. Never reproduct on my local env. A. 386/centos go1.12.8/go1.13.6/go1.14 B. 386 docker on 64-bit ubunut (kernel 5.3) go1.12.8/go1.13.6/go1.14

  3. amd64 on cicurs CI Then I try test amd64 on cicurs CI and find same problem as 386. Also use for ((i=1; i<=300; i++)); do go test -v -test.run TestStepConcurrentDirect ; done on cirrus CI, see the same error every CI period. https://github.com/go-delve/delve/runs/507318116 https://github.com/go-delve/delve/runs/507318112 https://cirrus-ci.com/task/4921825478049792 https://github.com/go-delve/delve/runs/507957824 ...

  4. 64bit on travis CI All are ok. https://travis-ci.com/github/go-delve/delve/jobs/298173516 https://travis-ci.com/github/go-delve/delve/jobs/298173518 https://travis-ci.com/github/go-delve/delve/jobs/298174477 https://travis-ci.com/github/go-delve/delve/jobs/298174479 ... Also for ((i=1; i<=600; i++)); do go test -v -test.run TestStepConcurrentDirect ; done on cirrus CI, I use 600 loops here but all are ok.

  5. 32bit on travis CI https://travis-ci.com/github/go-delve/delve/jobs/298209841 https://travis-ci.com/github/go-delve/delve/jobs/298209842 ... All are ok, also 600 loops for teststepconcurrent

I think there maybe a problem about 386 docker with specify linux kernel for golang. (May be similar to https://github.com/golang/go/issues/35777 ?)

Sorry for this crude solution, I can't find a better way about ci failed of cicurs on 386 linux. Platform change is the only thing I can think of. Meanwhile I'm worried about complicating the .travis.yml file

Ps:

  1. It seem appveyor still work, need to stop.
  2. I will close #1924 once this pr merged or a better way.
  3. travis report display

image

+39 -26

0 comment

2 changed files

pr created time in 22 days

push eventchainhelen/delve

chainhelen

commit sha 4fd0b5c3fabc763ae6bba635149796e89b8467cd

*: Transfer 32-bit test into travis from cirrus

view details

push time in 22 days

create barnchchainhelen/delve

branch : travis_32

created branch time in 22 days

push eventchainhelen/delve

chainhelen

commit sha 744270636d75fd1d2bb22445a0b7fd2c5ca7697b

600 32bit

view details

push time in 22 days

push eventchainhelen/delve

chainhelen

commit sha 68fc7a5a1a62312ff051f7493c00d2a56105503d

go_32_version directly

view details

push time in 22 days

push eventchainhelen/delve

chainhelen

commit sha 7a46e9a1affd702f6e91bbda607f793312366673

remove setenforce

view details

push time in 22 days

push eventchainhelen/delve

chainhelen

commit sha 39d322d9362a47758f2f6482ed03d4916a207c22

Add --privileged

view details

push time in 22 days

push eventchainhelen/delve

chainhelen

commit sha 2bdf3df2c36baa5d219b19ecc246d51757759a77

Remove privileged and setenforce 0

view details

push time in 22 days

push eventchainhelen/delve

chainhelen

commit sha 68df553de996d345a5c755ecd560ddc3c3476b5f

sudo => privileged

view details

push time in 22 days

push eventchainhelen/delve

chainhelen

commit sha 0aae325715b532347562fe7c2c4368d36e861cd8

Add sudo Operation not permitted??

view details

push time in 22 days

push eventchainhelen/delve

chainhelen

commit sha b5b847538714ae88b70bdfe08b632a1f253a0c7c

ubuntu => centos

view details

push time in 22 days

push eventchainhelen/delve

chainhelen

commit sha 0665dbde96321e4ca42da83efd975451387108ee

try write instead -e

view details

push time in 22 days

push eventchainhelen/delve

chainhelen

commit sha d37ae850bdc0b073d19179b72b3d1f0c9793422c

E => v

view details

push time in 22 days

push eventchainhelen/delve

chainhelen

commit sha 8e072ca1ad9251b32b9655d70267a1c3787709cd

Add "docker help run"

view details

push time in 22 days

push eventchainhelen/delve

chainhelen

commit sha 99f4c76fec74a6384b4bcb23026e8ef8b82c6bb3

Fix

view details

push time in 22 days

push eventchainhelen/delve

chainhelen

commit sha f6ad682d74d89974f4b0e0df16ecd3ddfedcae72

Why? command not found

view details

chainhelen

commit sha c456ba512017c718fb08c3bd6cf8f1ce509c224f

Remove tip for 32

view details

push time in 22 days

push eventchainhelen/delve

chainhelen

commit sha cdbe02afb7f8db774fbf0962245677e4fe0d95d1

-e => --env

view details

push time in 22 days

push eventchainhelen/delve

chainhelen

commit sha 0b69c9b3a1de367718b3d963b5b7651246529389

Fix: go_version: command not found

view details

push time in 22 days

push eventchainhelen/delve

chainhelen

commit sha 1a5d526f522599a99b5a4586ddb0865eb039fc68

Fix judge

view details

push time in 22 days

push eventchainhelen/delve

chainhelen

commit sha ceb38c5b8433f13816dfc58893bfec97bdd4f71a

Fix: travis.test name

view details

push time in 22 days

push eventchainhelen/delve

chainhelen

commit sha 455650b2e1654128dc7e151bf283635693bd73eb

fix Permission Denied

view details

push time in 22 days

push eventchainhelen/delve

chainhelen

commit sha 02f9c6f37f40121e1d07e1a92b72d63bf8471286

Fix: 32bit

view details

push time in 22 days

push eventchainhelen/delve

chainhelen

commit sha f58d6c291efdc7a2de75540cdda8749b59bd91cf

Add .travis.test.sh

view details

push time in 22 days

push eventchainhelen/delve

chainhelen

commit sha d5c5486da97b9c542ee4ec50c966f9453457ebc0

Fix: 32bit

view details

push time in 22 days

push eventchainhelen/delve

chainhelen

commit sha cf6e62a3bbade1534d79d48159e07c3d5b5c597a

Add 32bit

view details

push time in 22 days

push eventchainhelen/test_cirrus

chainhelen

commit sha 2aa0c97da284b83a2d84bded47309262f2a36b01

Update .cirrus.yml

view details

push time in 23 days

push eventchainhelen/test_cirrus

chainhelen

commit sha c314654dcf7e237bcb4d3532f74f1989647a56ef

Update .cirrus.yml

view details

push time in 23 days

push eventchainhelen/test_cirrus

chainhelen

commit sha e6f7df7a1507e68547f7a98a7a65ab0c8ea59c43

Update .cirrus.yml

view details

push time in 23 days

push eventchainhelen/test_cirrus

chainhelen

commit sha b569165c3ab325a4b409cc004b60ff0edb76ef49

Update .cirrus.yml

view details

push time in 23 days

push eventchainhelen/test_cirrus

chainhelen

commit sha f782ba9c01dfdd0725a74c44f125d0d7d481a7f1

Update teststepconcurrent.go

view details

push time in 23 days

push eventchainhelen/delve

Derek Parker

commit sha 15509d4dc9c56f46c0ef7d2baf8d25c7355e46a6

*: Remove appveyor, use travis windows support (#1919) * *: Remove appveyor, use travis windows support * cmd/dlv: Update TestOutput for Travis on Windows * cmd/dlv: Skip TestGeneratedDoc in Travis on Windows * Reduce number of jobs in matrix We only really want to test the full arch/go matrix on Linux. For every other os/arch/go combination, only run Go tip and the latest supported version.

view details

chainhelen

commit sha aaa8724117e90f328b7988e8875e07a1a321f8d7

Merge branch 'master' into Test_Cirrus_CI

view details

chainhelen

commit sha 3ebdf91eaf7b53dc936645e595d64efd291dd66e

Add travis

view details

push time in 23 days

push eventchainhelen/delve

chainhelen

commit sha 706be39c8cdd3829919040ab7063c8a07ca047b7

Feture: remove all extra part

view details

push time in 23 days

push eventchainhelen/delve

chainhelen

commit sha bf155b690946e7b1aa076d85490ed539cce6b37a

Fix: remove =

view details

push time in 23 days

push eventchainhelen/delve

chainhelen

commit sha 541704d3c705bdfcccecc9e0664c1df9741d87f5

Feture: recover extra part

view details

push time in 23 days

push eventchainhelen/delve

chainhelen

commit sha 208694950d146b591bf88b93134ff3745ac51a6a

yum update && upgrade

view details

push time in 23 days

push eventchainhelen/delve

chainhelen

commit sha d29b3c2dc2477b8d91c24c367005edf94ad4b715

Change 6 into 7

view details

push time in 23 days

push eventchainhelen/delve

chainhelen

commit sha b2696df01d48fc68234ddcdffddd697836505b3c

Change 7 into 6

view details

push time in 23 days

push eventchainhelen/delve

chainhelen

commit sha 50a91937cf2d865d783ebe284db091c8110e2878

Add amd64

view details

push time in 23 days

push eventchainhelen/delve

chainhelen

commit sha eefdd16c86e1fffb2462ed0b17c7a631c1c19d25

Fix

view details

push time in 24 days

push eventchainhelen/delve

chainhelen

commit sha 73ae9631e426e0116d55fa3bb2f5f0848cbb1298

Remove vmread vmwrite

view details

push time in 24 days

push eventchainhelen/delve

chainhelen

commit sha 2b28728c06d564af9041929389e458eb31b059a7

wg.Add

view details

push time in 24 days

push eventchainhelen/delve

chainhelen

commit sha 42e41d183f61c932e5aa6c66c6a2d342e7caff9f

Foo => Foo2

view details

push time in 24 days

push eventchainhelen/delve

chainhelen

commit sha 52da4a2344471017eb89622b2e551756fb41d1e3

Add Disa

view details

push time in 24 days

push eventchainhelen/delve

chainhelen

commit sha e469d91a386c0123a237ae95d6b7ddaefb37aac7

Disassemble

view details

push time in 24 days

push eventchainhelen/delve

chainhelen

commit sha 98b497b6016bced29c022645aba5e6ef5c78bc54

Loop set int 1

view details

push time in 24 days

push eventchainhelen/delve

chainhelen

commit sha 69263e5f4e9001d053e5e3186950c6ea4a6bb519

Remove 100 loop for single goroutinue

view details

push time in 24 days

push eventchainhelen/delve

chainhelen

commit sha 5a3259103e62870140e3f70de7c94e263136ae24

remove print

view details

push time in 24 days

push eventchainhelen/delve

chainhelen

commit sha 344f98d4e347d69eeba254a05964108912362c91

Remove breakpoint

view details

push time in 24 days

push eventchainhelen/test_cirrus

chainhelen

commit sha b338913f0d5a78a69758ad41ae8fbf536f847a0a

Update .cirrus.yml

view details

push time in 24 days

push eventchainhelen/test_cirrus

chainhelen

commit sha 010429e6eae7425dd4eba165414b260c34e78899

Update .cirrus.yml

view details

push time in 24 days

push eventchainhelen/godbg

chainhelen

commit sha 366760b6ed86f4f4db57f153710b4115b3743158

Update .cirrus.yml

view details

push time in 24 days

push eventchainhelen/godbg

chainhelen

commit sha a2fd7c84cb71169d7d3652c96f41805ef09075f1

Update .cirrus.yml

view details

push time in 24 days

push eventchainhelen/godbg

chainhelen

commit sha e7a0a0dbc2194e158d660096b73d15652a84c745

Update .cirrus.yml

view details

push time in 24 days

push eventchainhelen/godbg

chainhelen

commit sha 19e33136496e635e673c5763aa69f9517d045bd9

Update .cirrus.yml

view details

push time in 24 days

push eventchainhelen/godbg

chainhelen

commit sha ff907c79fdd1e19db0e438cce04beb6c89321f3c

Update .cirrus.yml

view details

push time in 24 days

push eventchainhelen/godbg

chainhelen

commit sha 84392621b4451fac600ad236ec5d9a41849c2cf9

Update .cirrus.yml

view details

push time in 24 days

push eventchainhelen/delve

chainhelen

commit sha 6661210726f07a6d358ba202415f4356ed7fe861

fasle => false

view details

push time in 24 days

push eventchainhelen/delve

chainhelen

commit sha 22348e9a14488ccae8d17150c158942800daaa48

SwitchStack

view details

push time in 24 days

push eventchainhelen/delve

chainhelen

commit sha 64ca47108470fc62ba5f39611ecff5855d0f21b5

Fix

view details

push time in 24 days

push eventchainhelen/delve

chainhelen

commit sha 96ccf4597db6b40122a433a9a0ea06889a6987b9

Fix:

view details

push time in 24 days

push eventchainhelen/delve

chainhelen

commit sha a9cb78a435822caa7e9f7326103e1eddf563ac68

Fix

view details

push time in 24 days

more