Skip to content

Commit

Permalink
pkg/cover/backend: don't consider gvisor/starnix as linux
Browse files Browse the repository at this point in the history
  • Loading branch information
dvyukov committed Jul 3, 2024
1 parent ffe8393 commit 6f1d331
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions pkg/cover/backend/backend.go
Original file line number Diff line number Diff line change
Expand Up @@ -87,8 +87,8 @@ func Make(target *targets.Target, vm, objDir, srcDir, buildDir string, splitBuil
}

func GetPCBase(cfg *mgrconfig.Config) (uint64, error) {
if cfg.Target.OS == targets.Linux {
return getPCBase(cfg)
if cfg.Target.OS == targets.Linux && cfg.Type != targets.GVisor && cfg.Type != targets.Starnix {
return getLinuxPCBase(cfg)
}
return 0, nil
}
2 changes: 1 addition & 1 deletion pkg/cover/backend/elf.go
Original file line number Diff line number Diff line change
Expand Up @@ -245,7 +245,7 @@ func elfReadTextSec(module *KernelModule) (*elf.Section, error) {
return text, nil
}

func getPCBase(cfg *mgrconfig.Config) (uint64, error) {
func getLinuxPCBase(cfg *mgrconfig.Config) (uint64, error) {
bin := filepath.Join(cfg.KernelObj, cfg.SysTarget.KernelObject)
file, err := elf.Open(bin)
if err != nil {
Expand Down

0 comments on commit 6f1d331

Please sign in to comment.