Review feedback

Author: gnurizen

interpreter/golang/golang.go

@@ -70,7 +70,7 @@ func Loader(_ interpreter.EbpfHandler, info *interpreter.LoaderInfo) (interprete
 		// are our best bet.
 		log.Warnf("version %s unknown; using offsets for latest known Go version %s."+
 			"If Go traceID integration and other custom labels support is buggy,"+
-			" try upgrading to the latest version.", goVersion, latestVersion)
+			" try upgrading to the latest profiler version.", goVersion, latestVersion)
 		return data{
 			goVersion: goVersion,
 			offsets:   allOffsets[latestVersion],

interpreter/golang/runtime_data.go

@@ -8,6 +8,11 @@ import "C"
 // Consider bumping this whenever a new version of Go is released.
 var latestVersion = "go1.24"
 
+// Offsets come from DWARF debug information:
+// https://github.com/parca-dev/parca-agent/blob/2815ded21704934c/pkg/runtime/golang/golang.go#L85
+// However since DWARF information can be stripped we record them here.
+// TODO: Should we look for DWARF information to support new versions
+// automatically when available?
 var allOffsets = map[string]C.GoCustomLabelsOffsets{
 	"go1.13": {
 		m_offset:               48,