aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--pkg/build/akaros.go2
-rw-r--r--pkg/build/build.go6
-rw-r--r--pkg/build/darwin.go2
-rw-r--r--pkg/build/freebsd.go2
-rw-r--r--pkg/build/fuchsia.go2
-rw-r--r--pkg/build/gvisor.go2
-rw-r--r--pkg/build/linux.go10
-rw-r--r--pkg/build/linux_linux.go2
-rw-r--r--pkg/build/linux_nolinux.go2
-rw-r--r--pkg/build/netbsd.go2
-rw-r--r--pkg/build/openbsd.go2
-rw-r--r--pkg/build/test.go2
-rw-r--r--pkg/instance/instance.go2
-rw-r--r--syz-ci/manager.go2
-rw-r--r--tools/syz-build/build.go2
15 files changed, 21 insertions, 21 deletions
diff --git a/pkg/build/akaros.go b/pkg/build/akaros.go
index 73f4e9ac8..046096708 100644
--- a/pkg/build/akaros.go
+++ b/pkg/build/akaros.go
@@ -16,7 +16,7 @@ import (
type akaros struct{}
-func (ctx akaros) build(params *Params) error {
+func (ctx akaros) build(params Params) error {
configFile := filepath.Join(params.KernelDir, ".config")
if err := osutil.WriteFile(configFile, params.Config); err != nil {
return fmt.Errorf("failed to write config file: %v", err)
diff --git a/pkg/build/build.go b/pkg/build/build.go
index 9b0f9ded7..3b3387f4d 100644
--- a/pkg/build/build.go
+++ b/pkg/build/build.go
@@ -51,7 +51,7 @@ type Params struct {
// behavior. Binary equal builds, or builds that differ only in e.g. debug info,
// have the same ID. The ID may be empty if OS implementation does not have
// a way to calculate such IDs.
-func Image(params *Params) (string, error) {
+func Image(params Params) (string, error) {
builder, err := getBuilder(params.TargetOS, params.TargetArch, params.VMType)
if err != nil {
return "", err
@@ -104,12 +104,12 @@ func (err *KernelError) Error() string {
}
type builder interface {
- build(params *Params) error
+ build(params Params) error
clean(kernelDir, targetArch string) error
}
type signer interface {
- sign(params *Params) (string, error)
+ sign(params Params) (string, error)
}
func getBuilder(targetOS, targetArch, vmType string) (builder, error) {
diff --git a/pkg/build/darwin.go b/pkg/build/darwin.go
index 41cfad7a6..fa4007eb5 100644
--- a/pkg/build/darwin.go
+++ b/pkg/build/darwin.go
@@ -9,7 +9,7 @@ import (
type darwin struct{}
-func (ctx darwin) build(params *Params) error {
+func (ctx darwin) build(params Params) error {
// TODO(HerrSpace): Implement this.
return fmt.Errorf("pkg/build: darwin.build not implemented")
}
diff --git a/pkg/build/freebsd.go b/pkg/build/freebsd.go
index d002b8ad0..f8ac801ea 100644
--- a/pkg/build/freebsd.go
+++ b/pkg/build/freebsd.go
@@ -16,7 +16,7 @@ import (
type freebsd struct{}
-func (ctx freebsd) build(params *Params) error {
+func (ctx freebsd) build(params Params) error {
confDir := fmt.Sprintf("%v/sys/%v/conf/", params.KernelDir, params.TargetArch)
confFile := "SYZKALLER"
diff --git a/pkg/build/fuchsia.go b/pkg/build/fuchsia.go
index 7e9be5dff..8ad06f1af 100644
--- a/pkg/build/fuchsia.go
+++ b/pkg/build/fuchsia.go
@@ -27,7 +27,7 @@ func syzRoot() (string, error) {
return filepath.Abs(filepath.Join(filepath.Dir(selfPath), "../.."))
}
-func (fu fuchsia) build(params *Params) error {
+func (fu fuchsia) build(params Params) error {
syzDir, err := syzRoot()
if err != nil {
return err
diff --git a/pkg/build/gvisor.go b/pkg/build/gvisor.go
index f26a09346..0925a7b7c 100644
--- a/pkg/build/gvisor.go
+++ b/pkg/build/gvisor.go
@@ -19,7 +19,7 @@ type gvisor struct{}
var bazelTargetPath = regexp.MustCompile(`(?sm:.*^)\s*Outputs: \[(.*)\](?sm:$.*)`)
-func (gvisor gvisor) build(params *Params) error {
+func (gvisor gvisor) build(params Params) error {
if params.Compiler == "" {
params.Compiler = "bazel"
}
diff --git a/pkg/build/linux.go b/pkg/build/linux.go
index be2341311..7aedaef16 100644
--- a/pkg/build/linux.go
+++ b/pkg/build/linux.go
@@ -26,7 +26,7 @@ type linux struct{}
var _ signer = linux{}
-func (linux linux) build(params *Params) error {
+func (linux linux) build(params Params) error {
if err := linux.buildKernel(params); err != nil {
return err
}
@@ -47,11 +47,11 @@ func (linux linux) build(params *Params) error {
return embedLinuxKernel(params, kernelPath)
}
-func (linux linux) sign(params *Params) (string, error) {
+func (linux linux) sign(params Params) (string, error) {
return elfBinarySignature(filepath.Join(params.OutputDir, "obj", "vmlinux"))
}
-func (linux linux) buildKernel(params *Params) error {
+func (linux linux) buildKernel(params Params) error {
configFile := filepath.Join(params.KernelDir, ".config")
if err := linux.writeFile(configFile, params.Config); err != nil {
return fmt.Errorf("failed to write config file: %v", err)
@@ -96,7 +96,7 @@ func (linux linux) buildKernel(params *Params) error {
return nil
}
-func (linux) createImage(params *Params, kernelPath string) error {
+func (linux) createImage(params Params, kernelPath string) error {
tempDir, err := ioutil.TempDir("", "syz-build")
if err != nil {
return err
@@ -162,7 +162,7 @@ func runMakeImpl(arch, compiler, ccache, kernelDir string, addArgs ...string) er
return err
}
-func runMake(params *Params, addArgs ...string) error {
+func runMake(params Params, addArgs ...string) error {
return runMakeImpl(params.TargetArch, params.Compiler, params.Ccache, params.KernelDir, addArgs...)
}
diff --git a/pkg/build/linux_linux.go b/pkg/build/linux_linux.go
index af34d6b22..0a59f3fd7 100644
--- a/pkg/build/linux_linux.go
+++ b/pkg/build/linux_linux.go
@@ -22,7 +22,7 @@ import (
// - ssh works without password (we don't copy the key)
// - cmdline file is not supported (should be moved to kernel config)
// - the kernel is stored in the image in /vmlinuz file.
-func embedLinuxKernel(params *Params, kernelPath string) error {
+func embedLinuxKernel(params Params, kernelPath string) error {
if params.CmdlineFile != "" {
return fmt.Errorf("cmdline file is not supported for linux images")
}
diff --git a/pkg/build/linux_nolinux.go b/pkg/build/linux_nolinux.go
index 7353bb275..0210627f5 100644
--- a/pkg/build/linux_nolinux.go
+++ b/pkg/build/linux_nolinux.go
@@ -10,6 +10,6 @@ import (
"errors"
)
-func embedLinuxKernel(params *Params, kernelPath string) error {
+func embedLinuxKernel(params Params, kernelPath string) error {
return errors.New("building linux image is only supported on linux")
}
diff --git a/pkg/build/netbsd.go b/pkg/build/netbsd.go
index 309a90299..89b22cf82 100644
--- a/pkg/build/netbsd.go
+++ b/pkg/build/netbsd.go
@@ -22,7 +22,7 @@ import (
type netbsd struct{}
-func (ctx netbsd) build(params *Params) error {
+func (ctx netbsd) build(params Params) error {
const kernelName = "GENERIC_SYZKALLER"
confDir := fmt.Sprintf("%v/sys/arch/%v/conf", params.KernelDir, params.TargetArch)
compileDir := fmt.Sprintf("%v/sys/arch/%v/compile/obj/%v", params.KernelDir, params.TargetArch, kernelName)
diff --git a/pkg/build/openbsd.go b/pkg/build/openbsd.go
index a47579035..1446d1e18 100644
--- a/pkg/build/openbsd.go
+++ b/pkg/build/openbsd.go
@@ -15,7 +15,7 @@ import (
type openbsd struct{}
-func (ctx openbsd) build(params *Params) error {
+func (ctx openbsd) build(params Params) error {
const kernelName = "SYZKALLER"
confDir := fmt.Sprintf("%v/sys/arch/%v/conf", params.KernelDir, params.TargetArch)
compileDir := fmt.Sprintf("%v/sys/arch/%v/compile/%v", params.KernelDir, params.TargetArch, kernelName)
diff --git a/pkg/build/test.go b/pkg/build/test.go
index e80afcc86..93ae27f96 100644
--- a/pkg/build/test.go
+++ b/pkg/build/test.go
@@ -5,7 +5,7 @@ package build
type test struct{}
-func (tb test) build(params *Params) error {
+func (tb test) build(params Params) error {
return nil
}
diff --git a/pkg/instance/instance.go b/pkg/instance/instance.go
index e0c1e3ce8..3e28f2170 100644
--- a/pkg/instance/instance.go
+++ b/pkg/instance/instance.go
@@ -108,7 +108,7 @@ func (env *env) BuildSyzkaller(repoURL, commit string) error {
func (env *env) BuildKernel(compilerBin, ccacheBin, userspaceDir, cmdlineFile, sysctlFile string, kernelConfig []byte) (
string, string, error) {
imageDir := filepath.Join(env.cfg.Workdir, "image")
- params := &build.Params{
+ params := build.Params{
TargetOS: env.cfg.TargetOS,
TargetArch: env.cfg.TargetVMArch,
VMType: env.cfg.Type,
diff --git a/syz-ci/manager.go b/syz-ci/manager.go
index 7d548ff7b..90e43d472 100644
--- a/syz-ci/manager.go
+++ b/syz-ci/manager.go
@@ -304,7 +304,7 @@ func (mgr *Manager) build(kernelCommit *vcs.Commit) error {
if err := config.SaveFile(filepath.Join(tmpDir, "tag"), info); err != nil {
return fmt.Errorf("failed to write tag file: %v", err)
}
- params := &build.Params{
+ params := build.Params{
TargetOS: mgr.managercfg.TargetOS,
TargetArch: mgr.managercfg.TargetVMArch,
VMType: mgr.managercfg.Type,
diff --git a/tools/syz-build/build.go b/tools/syz-build/build.go
index 4e4eff035..59dd339ba 100644
--- a/tools/syz-build/build.go
+++ b/tools/syz-build/build.go
@@ -34,7 +34,7 @@ func main() {
if err != nil {
tool.Fail(err)
}
- params := &build.Params{
+ params := build.Params{
TargetOS: *flagOS,
TargetArch: *flagArch,
VMType: "gce",