summaryrefslogtreecommitdiffstats
path: root/kernel/gsp_sema_service.c
diff options
context:
space:
mode:
Diffstat (limited to 'kernel/gsp_sema_service.c')
-rw-r--r--kernel/gsp_sema_service.c36
1 files changed, 32 insertions, 4 deletions
diff --git a/kernel/gsp_sema_service.c b/kernel/gsp_sema_service.c
index ced815d..6f4e387 100644
--- a/kernel/gsp_sema_service.c
+++ b/kernel/gsp_sema_service.c
@@ -43992,8 +43992,18 @@ static bool svc_userd_ddx_reprobe(struct nv_vgpu_device *vgpu,
* validated block inside the client's ~10 s DMA-test window. The
* pump's confirm/reject/defer ladder still owns the latch's lifetime
* (defer bound 32, then reject -> the RAM scan re-adopts fresh).
- */
- if (ch->submissions != ch->gr_ram_userd_latch_sub)
+ *
+ * r1780 (eng5 c171): EXCEPT when the served latch has been RESET to
+ * put==0 - the client recycled the control block (the DDX's next
+ * push-on-demand generation). A block at put==0 is not "serving"
+ * anything, so the switch to a freshly validated live block must be
+ * allowed; without this the served-refusal kept the zeroed block and
+ * the channel stayed dark (c171: "Failed to idle DMA" t=88.3). The
+ * keep path in the pump's judge re-arms latch_put to 0 on the reset,
+ * so this exception only fires for recycled blocks.
+ */
+ if (ch->submissions != ch->gr_ram_userd_latch_sub &&
+ ch->gr_ram_userd_latch_put != 0)
return false;
if (!svc_gr_ram_userd_scan(vgpu, ch, &npa, &nput, &nget))
return false;
@@ -44952,8 +44962,9 @@ static void svc_pump_channels(struct nv_vgpu_device *vgpu)
pr_info("sema-chan: GR USERD h=0x%x pa=0x%llx speculative latch CONFIRMED (put %u -> %u)\n",
ch->h_object, ch->userd_pa,
ch->gr_ram_userd_latch_put, put);
- } else if (kfam && (gsp_sema_kick_latch_keep ||
- ch->gr_ram_userd_confirmed) &&
+ } else if ((kfam || svc_userd_ddx_family(ch)) &&
+ (gsp_sema_kick_latch_keep ||
+ ch->gr_ram_userd_confirmed) &&
put == 0 &&
ch->gr_ram_userd_latch_put != 0 &&
ch->gr_ram_userd_keeps < 64) {
@@ -44975,6 +44986,23 @@ static void svc_pump_channels(struct nv_vgpu_device *vgpu)
* after; the ADOPT itself was ring-entry-
* validated). sema_kick_latch_keep=0 restores
* the pre-fix condition for A/B.
+ *
+ * r1780 (eng5 c171, later-DMA instance): the
+ * same put-reset applies to the DDX family's
+ * served latch. The X channel 0x1000013 had
+ * served 7698 slots, the DDX's next push-on-
+ * demand generation recycled the control block
+ * (put 7698 -> 0), and because the reset sets
+ * `moved` the DEFERRED clause below did not
+ * apply -> REJECT -> the FB shape scan adopted
+ * a garbage block ("found by VRAM shape scan
+ * at fb 0x960000 (put=1 get=917535, idle,
+ * ring-validated)", reads 0/0) -> "Failed to
+ * idle DMA" at Xorg t=88.3. Keep the reset
+ * block re-armed; the 2 s re-probe (with the
+ * zeroed-latch exception in
+ * svc_userd_ddx_reprobe) finds the real new
+ * generation.
*/
ch->gr_ram_userd_keeps++;
if (ch->gr_ram_userd_keep_log < 8) {