fixup

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>

Author: florianl

extensions/apmint/apmint.go

@@ -51,7 +51,7 @@ type apmProcessStorage struct {
 	TraceSocketPath string
 }
 
-// Loader implements plugins.Loader.
+// Loader implements extensions.Loader.
 func Loader(_ extensions.EbpfHandler, info *extensions.LoaderInfo) (extensions.Data, error) {
 	if !isPotentialAgentLib(info.FileName()) {
 		return nil, nil
@@ -158,7 +158,7 @@ type Instance struct {
 
 var _ extensions.Instance = &Instance{}
 
-// Detach implements the plugins.Instance interface.
+// Detach implements the extensions.Instance interface.
 func (i *Instance) Detach(ebpf extensions.EbpfHandler, pid libpf.PID) error {
 	return ebpf.DeleteProcData(libpf.APMInt, pid)
 }

extensions/config.go

@@ -67,7 +67,7 @@ type BEAMConfig struct {
 }
 
 // ExtensionsConfig holds configuration for all extensions.
-// Zero value means all plugins enabled.
+// By default all extensions are enabled.
 type ExtensionsConfig struct {
 	Python  PythonConfig  `mapstructure:"python"`
 	Perl    PerlConfig    `mapstructure:"perl"`
@@ -81,5 +81,5 @@ type ExtensionsConfig struct {
 	BEAM    BEAMConfig    `mapstructure:"beam"`
 }
 
-// AllExtensionsConfig returns a ExtensionsConfig with all extensions enabled (zero value).
+// AllExtensionsConfig returns a ExtensionsConfig with all extensions enabled.
 func AllExtensionsConfig() ExtensionsConfig { return ExtensionsConfig{} }

extensions/golabels/golabels.go

@@ -63,7 +63,7 @@ func Loader(_ extensions.EbpfHandler, info *extensions.LoaderInfo) (extensions.D
 		return nil, nil
 	}
 
-	// Go plugins are shared objects that share the runtime with the main
+	// Go extensions are shared objects that share the runtime with the main
 	// binary. The offsets we need are determined by the main binary so
 	// there is no reason to create a duplicate golabels instance for
 	// a plugin. A shared library is ET_DYN without a PT_INTERP segment

extensions/nodev8/v8.go

@@ -65,7 +65,7 @@ package nodev8 // import "go.opentelemetry.io/ebpf-profiler/extensions/nodev8"
 //        |    as ScopeInfo (normal case), it contains the file name, and the
 //        |       location of this scope along with other details.
 //        |- function_data
-//        |    can be BytecodeArray, plugins.Data, UncompiledFunctionData, WASM data etc.
+//        |    can be BytecodeArray, extensions.Data, UncompiledFunctionData, WASM data etc.
 //        \- script_or_debug_info
 //             as Script, it contains the text source code
 //             as DebugInfo, there is also debug info available
@@ -200,7 +200,7 @@ const (
 	// value to avoid huge malloc that could cause OOM crash.
 	maximumFixedTableSize = 512 * 1024
 
-	// lruSourceFileCacheSize is the LRU size for caching source files for an plugins.
+	// lruSourceFileCacheSize is the LRU size for caching source files for a extension.
 	// This should reflect the number of hot source files that are seen often in a trace.
 	lruSourceFileCacheSize = 128
 

extensions/nodev8/v8_test.go

@@ -1,7 +1,7 @@
 // Copyright The OpenTelemetry Authors
 // SPDX-License-Identifier: Apache-2.0
 
-package nodev8 // import "go.opentelemetry.io/ebpf-profiler/plugins/nodev8"
+package nodev8 // import "go.opentelemetry.io/ebpf-profiler/extensions/nodev8"
 
 import (
 	"testing"

extensions/perl/instance.go

@@ -347,7 +347,7 @@ func (i *perlInstance) getGV(gvAddr libpf.Address, nameOnly bool) (libpf.String,
 	return value, nil
 }
 
-// getCOP reads and caches a Control OP from remote plugins.
+// getCOP reads and caches a Control OP from remote extensions.
 // On success, the COP is returned. On error, the error.
 func (i *perlInstance) getCOP(copAddr libpf.Address, funcName libpf.String) (
 	*perlCOP, error,

extensions/php/opcache.go

@@ -32,7 +32,7 @@ package php // import "go.opentelemetry.io/ebpf-profiler/extensions/php"
 //
 // Before we begin it's illustrative to understand how PHP works internally.
 // PHP belongs to the class of interpreted languages that use bytecode: each PHP function is
-// decomposed into a sequence of bytecode instructions that are then executed by the PHP plugins.
+// decomposed into a sequence of bytecode instructions that are then executed by the PHP extension.
 // These instructions are known as zend_ops in the Zend compiler, and their internal structure
 // looks like this:
 //  struct _zend_op {
@@ -81,7 +81,7 @@ package php // import "go.opentelemetry.io/ebpf-profiler/extensions/php"
 // The implication of this are:
 // a) JIT'd code lives in shared memory, which means that all of the process-local work that the host-agent normally does doesn't really apply for PHP.
 // b) The PHP JIT doesn't even live in the same shared object as the PHP interpreter, so we can't
-//    find the JIT information from the PHP plugins.
+//    find the JIT information from the PHP extension.
 // c) Even if we could, PHP hides symbols by default and so recovering the relevant information isn't easy in this form[2].
 //
 // Note that we also can't use the approach used in the V8 interpreter

extensions/ruby/ruby.go

@@ -138,7 +138,7 @@ type rubyData struct {
 
 	// Address to global symbols, for id to string mappings
 	globalSymbolsAddr libpf.Address
-	// version of the currently used Ruby plugins.
+	// version of the currently used Ruby extension.
 	// major*0x10000 + minor*0x100 + release (e.g. 3.0.1 -> 0x30001)
 	version uint32
 

extensions/ruby/ruby_test.go

@@ -1,7 +1,7 @@
 // Copyright The OpenTelemetry Authors
 // SPDX-License-Identifier: Apache-2.0
 
-package ruby // import "go.opentelemetry.io/ebpf-profiler/plugins/ruby"
+package ruby // import "go.opentelemetry.io/ebpf-profiler/extensions/ruby"
 
 import (
 	"testing"

extensions/types.go

@@ -19,7 +19,7 @@ import (
 )
 
 const (
-	// LruFunctionCacheSize is the LRU size for caching functions for an plugins.
+	// LruFunctionCacheSize is the LRU size for caching functions for a extension.
 	// This should reflect the number of hot functions that are seen often in a trace.
 	LruFunctionCacheSize = 1024
 )