mirror of
https://gitcode.com/JianFeeeee/HomeAgent.git
synced 2026-09-26 20:33:15 +00:00
b4fb254bf5
## 症状
全量 go test 偶发 SIGSEGV，整个测试二进制被杀（recover 捕不到 runtime
致命错误）。崩溃栈（2026-09-25 实测捕获，完整）：
readLoop (process.go:334)
→ markExited → once.Do
→ onExit → Plugin.handleExit (plugin.go:209)
→ Host.ReclaimOwner (host.go:342)
→ arenaRegion.ReclaimOwner → blockBase → getU32
→ SIGSEGV 读已 munmap 的内存
## 根因（两个叠加缺陷，同一后果）
**① markExited 里 close(exited) 早于 onExit**
close(p.exited) // :394 —— 先关闭，唤醒所有等待者
p.sup.untrack(p.name)
p.onExit(...) // :399 —— 回调里要读共享内存
onExit（内核侧 Plugin.handleExit）会调 Host.ReclaimOwner 回收该插件残留的
共享槽，那是要读共享内存区域的。而 exited 一关闭，Stop()/Kill() 就返回
（process.go:566/587），StopAll 随即返回，调用方（Host.Close）立刻
freeShm 解除映射 —— 此刻 onExit 还没跑完，ReclaimOwner 就成了读已 munmap
的内存。
修法：把 onExit 提到 close(p.exited) **之前**，并明确 exited 的语义是
「**完全**收尾完毕」而非「进程已死」——任何等待者看到它关闭后，都可安全
释放共享内存、卸载资源。
**② StopAll 超时分支 `go p.Kill()` 发射后不管**
go p.Kill() // 不等待
本函数返回后调用方就 unmap，而 Kill 内部要等 markExited 跑完（含 onExit）。
改为等全部 Kill 完成（Kill 自带 killReapTimeout 上限，不会无限拖住关停）。
**③ 同类的第三处：事件环订阅在关停时从不退订**
EventRing.Subscribe 注册到 Bus 的 handler 会 ring.WritePush（写共享内存），
而 Host.Close 会 munmap 整块区域。此前：
- handleEvents 把 EvtRingSubscribe 的取消函数**直接丢弃**（corehandler_runtime.go:103）
- EventsUnsubscribe 是 no-op，注释还写着「子进程 Stop 时由内核统一清理」，
但 closeProcHost 根本没有退订
于是每个订阅过的插件都在 Bus 上永久留了一个写共享内存的 handler，
munmap 后任意一条事件经过 Publish 就会写已解除映射的内存 ⇒ 同类 SIGSEGV。
修法：EventRing 记为 unsubs、新增 EvtRingSubscribeTracked（订阅即登记），
Host.Close 在 freeShm **之前**调用 evtCloser.Close 统一退订。
## 回归测试（3 个，均经变异验证「修复前判红」）
1. `TestProcess_OnExitCompletesBeforeExitedCloses`（proc）
断言 Exited() 关闭时 onExit 必须已返回。变异（把 close(exited) 挪回
onExit 之前）→ FAIL。这是本次崩溃的直接判据。
2. `TestHost_CloseUnsubscribesEventRing`（proc）
断言 Host.Close 调用了 evtCloser.Close。变异（撤掉退订）→ FAIL。
3. `TestEventRing_CloseUnsubscribesFromBus`（plugin）
端到端：订阅 → Publish 有写入 → Close → Publish 不再写入。变异
（Close 不做事）→ FAIL。
顺带给 EvtRing 加了 Written() 访问器（诊断 + 上述测试的可观察量）。
## 验证
- 三个新测试全绿；-race 下 ./internal/plugin/... 全绿
- proc 包连跑 12 轮、internal/plugins 连跑 20 轮：SIGSEGV 0 次
- 全量 go test -count=1 ./... → 38 ok / 0 FAIL
- go build ./... / go vet ./... 干净
## 另有两个**既有** flaky（本次未动，与 C 化无关，单独记录）
排查过程中用「我的树 20 轮 vs 干净树 20 轮」对照确认了归属：
- 测试间固定端口冲突（pluginmgr 9876 / remotedevice 9890 / webui 8080）：
并行或残留实例时报 bind: address already in use。干净树同样复现。
- TestRealPlugin_DeepSearchInvoke 依赖外部 SearXNG（127.0.0.1:8888）：
上游限流时（brave "too many requests"、duckduckgo/quark CAPTCHA）断言失败。
干净树同样复现。属外部依赖，不是代码缺陷。
两者都不属本次「排查崩溃」的范围，已记入 plan.md 待办。
101 lines
3.3 KiB
Go
101 lines
3.3 KiB
Go
package plugin
|
||
|
||
import (
|
||
"encoding/json"
|
||
"sync"
|
||
|
||
"gitcode.com/JianFeeeee/HomeAgent/internal/events"
|
||
"gitcode.com/JianFeeeee/HomeAgent/internal/plugin/proc"
|
||
pubsdk "gitcode.com/JianFeeeee/homeagent-sdk/sdk"
|
||
)
|
||
|
||
// EventRing 是 Bus 与 proc.EvtRing 之间的适配层。
|
||
//
|
||
// 把内核的事件总线接到共享内存事件环：Bus.Publish → handler
|
||
// 把事件序列化写入 EvtRing slot → eventfd 通知子进程。
|
||
// 不改 Bus 自身结构（保护零 API 变动）。
|
||
type EventRing struct {
|
||
ring *proc.EvtRing
|
||
bus *events.Bus
|
||
efd int
|
||
|
||
// unsubs 保存全部已注册订阅的取消函数。
|
||
//
|
||
// ★ 为什么必须留着：这些 handler 会 ring.WritePush（写共享内存）。
|
||
// 而 Host.Close() 会 freeShm 解除整块映射 —— 若那时 handler 还在 Bus 上，
|
||
// 一条事件就会让 handler 写已解除映射的内存：SIGSEGV。
|
||
// 注意 Bus.safeCall 的 recover **捕不到** SIGSEGV（它是 runtime 致命错误，
|
||
// 不是 panic），所以这不是「最坏情况只丢一条事件」，而是整个内核进程被杀。
|
||
//
|
||
// 此前 handleEvents 把 EvtRingSubscribe 返回的取消函数直接丢弃
|
||
// （且 EventsUnsubscribe 是空实现），于是每个订阅过的插件都在 Bus 上
|
||
// 永久留了一个写共享内存的 handler —— 内核关停时必炸。
|
||
// 现在改为在这里登记，由 Close 统一退订（内核关停、以及插件自己的
|
||
// events.unsubscribe 都走这里）。
|
||
mu sync.Mutex
|
||
unsubs []func()
|
||
}
|
||
|
||
func NewEventRing(ring *proc.EvtRing, efd int, bus *events.Bus) *EventRing {
|
||
return &EventRing{ring: ring, bus: bus, efd: efd}
|
||
}
|
||
|
||
// Close 退订本适配层注册到 Bus 的全部 handler。
|
||
//
|
||
// 必须在 Host.Close()（munmap 共享段）**之前**调用；见 unsubs 的说明。
|
||
// 幂等：重复调用安全（退订函数本身在 Bus 侧是「找不到就什么都不做」）。
|
||
func (er *EventRing) Close() {
|
||
er.mu.Lock()
|
||
unsubs := er.unsubs
|
||
er.unsubs = nil
|
||
er.mu.Unlock()
|
||
|
||
for _, fn := range unsubs {
|
||
if fn != nil {
|
||
fn()
|
||
}
|
||
}
|
||
}
|
||
|
||
// Subscribe 在 Bus 上注册一个把事件分发到事件环的 handler，返回取消函数。
|
||
//
|
||
// 不改 Bus 自身结构——handler 把事件序列化后写入环并 post eventfd，
|
||
// Bus 侧按 EventType 精确匹配分发（与现有逻辑完全一致）。
|
||
func (er *EventRing) Subscribe(eventType pubsdk.EventType) func() {
|
||
return er.bus.Subscribe(events.EventType(eventType), func(evt *events.Event) {
|
||
payload, err := json.Marshal(evt)
|
||
if err != nil {
|
||
return
|
||
}
|
||
er.ring.WritePush(pubsdk.EventType(evt.Type), payload)
|
||
proc.EvtfdNotify(er.efd)
|
||
})
|
||
}
|
||
|
||
// EvtRingSubscribe 实现 proc.EvtRingSubscriber 接口。
|
||
// 按事件类型列表订阅，返回统一取消函数。
|
||
func (er *EventRing) EvtRingSubscribe(types []pubsdk.EventType) func() {
|
||
unsubscribes := make([]func(), 0, len(types))
|
||
for _, t := range types {
|
||
unsubscribes = append(unsubscribes, er.Subscribe(t))
|
||
}
|
||
return func() {
|
||
for _, fn := range unsubscribes {
|
||
fn()
|
||
}
|
||
}
|
||
}
|
||
|
||
// EvtRingSubscribeTracked 与 EvtRingSubscribe 相同，但把取消函数登记到
|
||
// unsubs，供 Close 统一退订。内核的 events.subscribe 走这条。
|
||
func (er *EventRing) EvtRingSubscribeTracked(types []pubsdk.EventType) func() {
|
||
un := er.EvtRingSubscribe(types)
|
||
if un == nil {
|
||
return nil
|
||
}
|
||
er.mu.Lock()
|
||
er.unsubs = append(er.unsubs, un)
|
||
er.mu.Unlock()
|
||
return un
|
||
}
|