Skip to content

Commit cf815f9

Browse files
committed
parca specific fixups
1 parent 333ac10 commit cf815f9

File tree

5 files changed

+6
-7
lines changed

5 files changed

+6
-7
lines changed

interpreter/customlabels/customlabels.go

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ import (
88

99
"go.opentelemetry.io/ebpf-profiler/interpreter"
1010
"go.opentelemetry.io/ebpf-profiler/libpf"
11-
"go.opentelemetry.io/ebpf-profiler/libpf/pfelf"
1211
"go.opentelemetry.io/ebpf-profiler/remotememory"
1312
"go.opentelemetry.io/ebpf-profiler/support"
1413
)
@@ -44,7 +43,7 @@ func Loader(_ interpreter.EbpfHandler, info *interpreter.LoaderInfo) (interprete
4443
}
4544
abiVersionSym, err := ef.LookupSymbol(abiVersionExport)
4645
if err != nil {
47-
if errors.Is(err, pfelf.ErrSymbolNotFound) {
46+
if errors.Is(err, libpf.ErrSymbolNotFound) {
4847
return nil, nil
4948
}
5049

interpreter/golabels/golabels.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ func Loader(_ interpreter.EbpfHandler, info *interpreter.LoaderInfo) (interprete
5858
log.Debugf("file %s detected as go version %s", info.FileName(), goVersion)
5959

6060
offsets := getOffsets(goVersion)
61-
tlsOffset, err := extractTLSGOffset(file, info.FileName())
61+
tlsOffset, err := extractTLSGOffset(file)
6262
if err != nil {
6363
return nil, fmt.Errorf("failed to extract TLS offset: %w", err)
6464
}

interpreter/golabels/tls_amd64.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,6 @@ func extractTLSGOffset(f *pfelf.File) (int32, error) {
6363
}
6464
}
6565
exit:
66-
log.Warnf("Failed to decode stackcheck symbol, Go label collection might not work %s", path)
66+
log.Warnf("Failed to decode stackcheck symbol, Go label collection might not work")
6767
return -8, nil
6868
}

interpreter/golabels/tls_arm64.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ import (
2121
// 0x000000000007f270 <+16>: mov x27, #0x30 // #48
2222
// 0x000000000007f274 <+20>: ldr x28, [x0, x27]
2323
// 0x000000000007f278 <+24>: ret
24-
func extractTLSGOffset(f *pfelf.File, path string) (int32, error) {
24+
func extractTLSGOffset(f *pfelf.File) (int32, error) {
2525
iscgo, err := f.IsCgoEnabled()
2626
if err != nil || !iscgo {
2727
return 0, err
@@ -55,6 +55,6 @@ func extractTLSGOffset(f *pfelf.File, path string) (int32, error) {
5555
}
5656
}
5757
}
58-
log.Warnf("Failed to decode load_g symbol, Go label collection might not work with CGO frames (%s)", path)
58+
log.Warnf("Failed to decode load_g symbol, Go label collection might not work with CGO frames")
5959
return 0, nil
6060
}

interpreter/luajit/offsets.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -358,7 +358,7 @@ func (o *offsetData) readSym(sym *libpf.Symbol) ([]byte, error) {
358358

359359
func (o *offsetData) lookupSymbol(name libpf.SymbolName) (s *libpf.Symbol, err error) {
360360
s, err = o.f.LookupSymbol(name)
361-
if err == pfelf.ErrSymbolNotFound && o.syms != nil {
361+
if err == libpf.ErrSymbolNotFound && o.syms != nil {
362362
s, err = o.syms.LookupSymbol(name)
363363
}
364364
if s == nil && o.dsyms != nil {

0 commit comments

Comments
 (0)