summaryrefslogtreecommitdiff
path: root/src/pardes.zig
diff options
context:
space:
mode:
authorGabriel Schneider <[email protected]>2026-09-27 17:52:31 -0300
committerGabriel Schneider <[email protected]>2026-10-01 00:12:14 -0300
commitf4412d6dfdb1e2bd8549a7edba785ee242fd38f2 (patch)
treea48a67c60ce0ee55be8db3691f08f36c1986d7f0 /src/pardes.zig
parent542dd489149e33915d19878a277e23b3a9c0b070 (diff)
downloadpardes-f4412d6dfdb1e2bd8549a7edba785ee242fd38f2.tar.gz
pardes-f4412d6dfdb1e2bd8549a7edba785ee242fd38f2.zip
Wake reads waiting on /log, event and pty/data; one reader per consuming file; pty/ctl reads back
A read parked on a queue was retried only when some unrelated write had to wait, so a follower of /log (and a reader of event or pty/data) slept until then. Pushing a record now marks the turn parked, and giving the turn up wakes them (measured: stuck past 3 s before, 0 s after). event and pty/data consume what they read, so a second open for reading is refused with rio's "file in use" (EBUSY through 9ns); writers still get in, and pty/data queues output only for an actual reader. pty/ctl reads back "winsize C R", in the words it takes. /log fixes from review: a record longer than a read comes in pieces (a shell read loop failed on long lines), every repeated message is logged, a record bigger than the ring is cut to fit instead of emptying it, the ring is reserved at boot so recording never allocates, and panes present at boot are recorded first. Co-Authored-By: Claude Opus 5.5 <[email protected]>
Diffstat (limited to 'src/pardes.zig')
-rw-r--r--src/pardes.zig9
1 files changed, 8 insertions, 1 deletions
diff --git a/src/pardes.zig b/src/pardes.zig
index 6d294230..06bc81ed 100644
--- a/src/pardes.zig
+++ b/src/pardes.zig
@@ -6604,6 +6604,9 @@ pub const Pardes = struct {
// A config init that opens a file with `Look …` armed the pulse before
// anyone touched anything. Nobody asked for that, so boot is silent.
_ = p.takeHaptic();
+ // The ring is reserved whole now, so recording never allocates later.
+ try p.fs.log.buf.ensureTotalCapacityPrecise(p.gpa, p.fs.log.cap);
+ ctlfs.events.announce(p); // /log starts with the panes it booted with
return p;
}
@@ -7438,6 +7441,9 @@ pub const Pardes = struct {
if (text.len == 0) return;
const serial: u32 = if (id < MAX_PANES) if (p.panes[id]) |pane| pane.serial else 0 else 0;
const kept = text[0..@min(text.len, LoggedMessage.cap)];
+ // /log hears every one: a client that retried and failed the same way
+ // is waiting on that second line. Only the +Messages view collapses.
+ ctlfs.events.noteMessage(p, serial, kept);
if (p.messages_len > 0) {
const last = &p.messages[(p.messages_head + limits.message_log - 1) % limits.message_log];
if (last.serial == serial and
@@ -7451,7 +7457,6 @@ pub const Pardes = struct {
return;
}
}
- ctlfs.events.noteMessage(p, serial, kept);
const slot = &p.messages[p.messages_head];
slot.len = @intCast(kept.len);
@memcpy(slot.text[0..slot.len], kept);
@@ -13635,6 +13640,8 @@ pub const Pardes = struct {
p.sync();
p.presentation.enabled = true;
_ = p.takeHaptic(); // see init: a restored session is not a gesture
+ try p.fs.log.buf.ensureTotalCapacityPrecise(p.gpa, p.fs.log.cap);
+ ctlfs.events.announce(p);
return p;
}