diff options
| author | Dmitry Vyukov <dvyukov@google.com> | 2021-03-17 09:51:20 +0100 |
|---|---|---|
| committer | Dmitry Vyukov <dvyukov@google.com> | 2021-03-18 09:17:51 +0100 |
| commit | 3d67c6a0759e6360bdcc0d7a8716b428354ca6bd (patch) | |
| tree | 8839f0e0a40f2442e0a7b299c5f979e3f2fc011c /pkg/cover/report.go | |
| parent | 4fd7e7cf6192443813368e9bb984e2b517c5ac17 (diff) | |
pkg/cover: refactor module discovery interface
Make module discovery convert host.KernelModule to backend.Module.
Also error if we have modules on non-Linux
and make it possible to return errors from module discovery.
Diffstat (limited to 'pkg/cover/report.go')
| -rw-r--r-- | pkg/cover/report.go | 20 |
1 files changed, 2 insertions, 18 deletions
diff --git a/pkg/cover/report.go b/pkg/cover/report.go index b622af4a4..8f805fe10 100644 --- a/pkg/cover/report.go +++ b/pkg/cover/report.go @@ -5,7 +5,6 @@ package cover import ( "fmt" - "path/filepath" "sort" "github.com/google/syzkaller/pkg/cover/backend" @@ -30,23 +29,8 @@ type Prog struct { var RestorePC = backend.RestorePC func MakeReportGenerator(target *targets.Target, vm, objDir, srcDir, buildDir string, subsystem []mgrconfig.Subsystem, - moduleObj []string, hostModules []host.KernelModule) (*ReportGenerator, error) { - if objDir == "" { - return nil, fmt.Errorf("kernel obj directory is not specified") - } - moduleObj = append([]string{objDir}, moduleObj...) - modules := []*backend.Module{ - { - Path: filepath.Join(objDir, target.KernelObject), - }, - } - for _, mod := range hostModules { - modules = append(modules, &backend.Module{ - Name: mod.Name, - Addr: mod.Addr, - }) - } - impl, err := backend.Make(target, vm, srcDir, buildDir, moduleObj, modules) + moduleObj []string, modules []host.KernelModule) (*ReportGenerator, error) { + impl, err := backend.Make(target, vm, objDir, srcDir, buildDir, moduleObj, modules) if err != nil { return nil, err } |
