internal/system: file-based big lock

Simultaneous commits or reverts can race if they have overlapping
targets. This big lock synchronises entry into the Commit and Revert
critical sections across all cmd/hakurei processes.

Signed-off-by: Ophestra <cat@gensokyo.uk>

1 files changed, 17 insertions, 17 deletions

diff --git a/internal/system/system_test.go b/internal/system/system_test.go
index e466c171..8d863826 100644
--- a/internal/system/system_test.go
+++ b/internal/system/system_test.go
@@ -82,7 +82,7 @@ func TestNew(t *testing.T) {
 					t.Errorf("recover: %v, want %v", r, want)
 				}
 			}()
-			New(nil, message.New(nil), 0)
+			New(nil, message.New(nil), 0, nil)
 		})
 
 		t.Run("msg", func(t *testing.T) {
@@ -92,7 +92,7 @@ func TestNew(t *testing.T) {
 					t.Errorf("recover: %v, want %v", r, want)
 				}
 			}()
-			New(t.Context(), nil, 0)
+			New(t.Context(), nil, 0, nil)
 		})
 
 		t.Run("uid", func(t *testing.T) {
@@ -102,11 +102,11 @@ func TestNew(t *testing.T) {
 					t.Errorf("recover: %v, want %v", r, want)
 				}
 			}()
-			New(t.Context(), message.New(nil), -1)
+			New(t.Context(), message.New(nil), -1, nil)
 		})
 	})
 
-	sys := New(t.Context(), message.New(nil), 0xbeef)
+	sys := New(t.Context(), message.New(nil), 0xbeef, nil)
 	if sys.ctx == nil {
 		t.Error("New: ctx = nil")
 	}
@@ -125,51 +125,51 @@ func TestEqual(t *testing.T) {
 		want bool
 	}{
 		{"simple UID",
-			New(t.Context(), message.New(nil), 150),
-			New(t.Context(), message.New(nil), 150),
+			New(t.Context(), message.New(nil), 150, nil),
+			New(t.Context(), message.New(nil), 150, nil),
 			true},
 
 		{"simple UID differ",
-			New(t.Context(), message.New(nil), 150),
-			New(t.Context(), message.New(nil), 151),
+			New(t.Context(), message.New(nil), 150, nil),
+			New(t.Context(), message.New(nil), 151, nil),
 			false},
 
 		{"simple UID nil",
-			New(t.Context(), message.New(nil), 150),
+			New(t.Context(), message.New(nil), 150, nil),
 			nil,
 			false},
 
 		{"op length mismatch",
-			New(t.Context(), message.New(nil), 150).
+			New(t.Context(), message.New(nil), 150, nil).
 				ChangeHosts("chronos"),
-			New(t.Context(), message.New(nil), 150).
+			New(t.Context(), message.New(nil), 150, nil).
 				ChangeHosts("chronos").
 				Ensure(m("/run"), 0755, false),
 			false},
 
 		{"op value mismatch",
-			New(t.Context(), message.New(nil), 150).
+			New(t.Context(), message.New(nil), 150, nil).
 				ChangeHosts("chronos").
 				Ensure(m("/run"), 0644, false),
-			New(t.Context(), message.New(nil), 150).
+			New(t.Context(), message.New(nil), 150, nil).
 				ChangeHosts("chronos").
 				Ensure(m("/run"), 0755, false),
 			false},
 
 		{"op type mismatch",
-			New(t.Context(), message.New(nil), 150).
+			New(t.Context(), message.New(nil), 150, nil).
 				ChangeHosts("chronos").
 				Wayland(m("/proc/nonexistent/dst"), m("/proc/nonexistent/src"), "\x00", "\x00"),
-			New(t.Context(), message.New(nil), 150).
+			New(t.Context(), message.New(nil), 150, nil).
 				ChangeHosts("chronos").
 				Ensure(m("/run"), 0755, false),
 			false},
 
 		{"op equals",
-			New(t.Context(), message.New(nil), 150).
+			New(t.Context(), message.New(nil), 150, nil).
 				ChangeHosts("chronos").
 				Ensure(m("/run"), 0755, false),
-			New(t.Context(), message.New(nil), 150).
+			New(t.Context(), message.New(nil), 150, nil).
 				ChangeHosts("chronos").
 				Ensure(m("/run"), 0755, false),
 			true},