diff options
| author | Arthur Hanson Rasmusson <arthur@vgpu.io> | 2026-09-29 16:32:03 -0400 |
|---|---|---|
| committer | Arthur Hanson Rasmusson <arthur@vgpu.io> | 2026-09-29 16:32:03 -0400 |
| commit | 26e6bc390cc9f466448a7470420d9de750e7611e (patch) | |
| tree | fba1cfb5268eed295f08459214794bd51e7aa068 /kernel/gsp_sema_service.c | |
| parent | a6cee384160b3210911838f79a8082d541da00f5 (diff) | |
| download | open-iov-26e6bc390cc9f466448a7470420d9de750e7611e.tar.gz open-iov-26e6bc390cc9f466448a7470420d9de750e7611e.zip | |
sema-svc: bounded diagnostics for GR counter writes; scanner guarantees counter-page delta
- rel-apply/publish now log the first 32/48 outcomes (written, skipped with
current value, no slot, read failure) so the live counter page can be
matched against host writes.
- guest pin scanner logs the counter page's pagemap state each loop and adds
it to the delta set explicitly, so the page is pinned the moment it is
mapped even if the window scan misses it.
Diffstat (limited to 'kernel/gsp_sema_service.c')
| -rw-r--r-- | kernel/gsp_sema_service.c | 42 |
1 files changed, 35 insertions, 7 deletions
diff --git a/kernel/gsp_sema_service.c b/kernel/gsp_sema_service.c index 21f0de0..e336277 100644 --- a/kernel/gsp_sema_service.c +++ b/kernel/gsp_sema_service.c @@ -351,6 +351,8 @@ struct sema_svc { u64 releases_jumped; /* applied with gap > 1 */ u64 releases_faked; /* skipped-entry fallback */ u64 counter_rel_applied; /* releases to registered counters */ + u64 counter_rel_dbg; /* rel-apply diagnostic budget */ + u64 counter_pub_dbg; /* counter-publish diagnostic budget */ u64 counter_pub_calls; /* publishes that found a registered counter */ u64 gr_rel_applied; /* release writes into the GR counter page */ u64 gr_rel_log; @@ -2107,6 +2109,7 @@ static void svc_counter_release_apply(struct nv_vgpu_device *vgpu, u32 h, u64 pa = 0; u32 cur = 0; int i; + u32 dbg = 0; if (!h || !payload) return; @@ -2119,21 +2122,43 @@ static void svc_counter_release_apply(struct nv_vgpu_device *vgpu, u32 h, } } spin_unlock_irqrestore(&svc.chan_lock, flags); - if (!pa) + if (!pa) { + if (svc.counter_rel_dbg < 32) { + svc.counter_rel_dbg++; + pr_info("sema-chan: rel-apply h=0x%x pay=%u -> no slot registered\n", + h, payload); + } return; + } - if (svc_read_ap(vgpu, pa, SVC_APERTURE_SYS, &cur, sizeof(cur))) + if (svc_read_ap(vgpu, pa, SVC_APERTURE_SYS, &cur, sizeof(cur))) { + if (svc.counter_rel_dbg < 32) { + svc.counter_rel_dbg++; + pr_info("sema-chan: rel-apply h=0x%x pa=0x%llx pay=%u -> read failed\n", + h, pa, payload); + } return; - if (payload <= cur || payload - cur > 0x100000) + } + if (payload <= cur || payload - cur > 0x100000) { + if (svc.counter_rel_dbg < 32) { + svc.counter_rel_dbg++; + pr_info("sema-chan: rel-apply h=0x%x pa=0x%llx pay=%u cur=%u -> skip\n", + h, pa, payload, cur); + } return; + } if (svc_write_ap(vgpu, pa, SVC_APERTURE_SYS, &payload, sizeof(payload))) return; svc.releases_applied++; svc.counter_rel_applied++; svc.post_hint_until = jiffies + 10 * HZ; - pr_info_ratelimited("sema-chan: counter release h=0x%x pa=0x%llx val=%u (was %u)\n", - h, pa, payload, cur); + if (svc.counter_rel_dbg < 32) { + svc.counter_rel_dbg++; + pr_info("sema-chan: rel-apply h=0x%x pa=0x%llx pay=%u cur=%u -> written\n", + h, pa, payload, cur); + } + (void)dbg; } /* @@ -3966,8 +3991,11 @@ static void svc_counter_publish(struct nv_vgpu_device *vgpu, &val, sizeof(val))) return; ch->counter_last = val; - pr_info_ratelimited("sema-chan: counter publish h=0x%x pa=0x%llx val=%u\n", - ch->h_object, ch->counter_pa, val); + if (svc.counter_pub_dbg < 48) { + svc.counter_pub_dbg++; + pr_info("sema-chan: counter publish h=0x%x pa=0x%llx val=%u (was %u)\n", + ch->h_object, ch->counter_pa, val, cur); + } } /* |
