summaryrefslogtreecommitdiffhomepage
path: root/pkg/sentry/strace
diff options
context:
space:
mode:
authorFabricio Voznika <fvoznika@google.com>2020-04-10 11:17:59 -0700
committergVisor bot <gvisor-bot@google.com>2020-04-10 11:19:12 -0700
commit1798d6cbee3360b09d3736069e15fd746e863bd2 (patch)
tree36534478cc1624185626c14c6e94f730454554f0 /pkg/sentry/strace
parent12b00c815638a28943c23d3be6ef09b955c6149e (diff)
Remove TODO from kernel.Stracer
The dependency strace=>kernel grew over time. strace also depends on task's FD table and FSContext. It could be fixed with some interfaces the other way, but then we're trading an interface for another, and kernel.Stracer is likely cleaner. Closes #155 PiperOrigin-RevId: 305909678
Diffstat (limited to 'pkg/sentry/strace')
-rw-r--r--pkg/sentry/strace/strace.go3
1 files changed, 0 insertions, 3 deletions
diff --git a/pkg/sentry/strace/strace.go b/pkg/sentry/strace/strace.go
index 77655558e..b94c4fbf5 100644
--- a/pkg/sentry/strace/strace.go
+++ b/pkg/sentry/strace/strace.go
@@ -778,9 +778,6 @@ func (s SyscallMap) Name(sysno uintptr) string {
//
// N.B. This is not in an init function because we can't be sure all syscall
// tables are registered with the kernel when init runs.
-//
-// TODO(gvisor.dev/issue/155): remove kernel package dependencies from this
-// package and have the kernel package self-initialize all syscall tables.
func Initialize() {
for _, table := range kernel.SyscallTables() {
// Is this known?