diff options
| -rw-r--r-- | kernel/gsp_sema_service.c | 19 |
1 files changed, 15 insertions, 4 deletions
diff --git a/kernel/gsp_sema_service.c b/kernel/gsp_sema_service.c index 96c9064..d73a5df 100644 --- a/kernel/gsp_sema_service.c +++ b/kernel/gsp_sema_service.c @@ -3542,9 +3542,20 @@ static int svc_op_translate(struct nv_vgpu_device *vgpu, u64 va, u64 *addr, * calling for this same page). Tell the pump so it can hold the entry * and retry it while the scan is in progress: a copy whose operand is * only resolvable by the scan needs several passes. - */ - if (svc_resolve_in_progress(va & ~0xfffULL)) - svc.op_tr_pending = 1; + * + * ISSUE-0002 (r58): hold on ANY unresolvable operand, not only while + * a targeted scan is already running. The guest's pma scrub submits + * memsets of freshly allocated vidmem on a GR channel; the operand + * translations (client buffer VAs pinned by the scanner) land ~1 s + * after the push, and the one-pass skip lost the memset forever - + * the guest then spun in scrubCheckAndWaitForSize (c3: 3 bigop + * memsets rc=-22 at 17:54:35, their pins logged at 17:54:36, the + * channel froze at get=89 with the client in pmaAllocatePages). + * The hold is bounded by the pump's retry counter, so a translation + * that never lands costs the bounded hold, then the entry is + * consumed exactly as before. + */ + svc.op_tr_pending = 1; return -ENOENT; } @@ -13032,7 +13043,7 @@ static void svc_pump_channels(struct nv_vgpu_device *vgpu) * channel. */ if (svc.op_tr_pending && - ch->op_retry < 24) { + ch->op_retry < 160) { ch->op_retry++; svc.gpfifo_nomap++; /* |