summaryrefslogtreecommitdiffstats
path: root/kernel/gsp_sema_service.c
diff options
context:
space:
mode:
authorArthur Hanson Rasmusson <arthur@vgpu.io>2026-09-29 16:32:03 -0400
committerArthur Hanson Rasmusson <arthur@vgpu.io>2026-09-29 16:32:03 -0400
commit5b80f3d5e1b9a597a7e7e0d761a5fa8b5804dbc1 (patch)
treed9dde0e9f59099edda901b4117b8e076eddc1260 /kernel/gsp_sema_service.c
parent09a729ac2a8c6f1865d5ef3bfd2b4df55d43764a (diff)
downloadopen-iov-5b80f3d5e1b9a597a7e7e0d761a5fa8b5804dbc1.tar.gz
open-iov-5b80f3d5e1b9a597a7e7e0d761a5fa8b5804dbc1.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.c42
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);
+ }
}
/*