PR/58745 CVS commit: src/sys/external/bsd/drm2/dist/drm/nouveau

["Taylor R Campbell" <riastradh%netbsd.org@localhost>]

The following reply was made to PR kern/58745; it has been noted by GNATS.

From: "Taylor R Campbell" <riastradh%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/58745 CVS commit: src/sys/external/bsd/drm2/dist/drm/nouveau
Date: Sat, 7 Nov 2026 11:04:16 +0000

 Module Name:	src
 Committed By:	riastradh
 Date:		Sat Nov  7 11:04:16 UTC 2026
 
 Modified Files:
 	src/sys/external/bsd/drm2/dist/drm/nouveau: nouveau_fence.c
 
 Log Message:
 nouveau: Don't set dma fence error if already signaled.
 
 This happens asynchronously on hardware error.  Upstream Linux trips
 WARN_ON in this case; our dma_fence_set_error trips KASSERT instead.
 We could maybe switch dma_fence_set_error to use WARN_ON too but it's
 probably better not to take this path anyway -- this assertion trips
 on logic that is incoherent, if not outright wrong.
 
 PR kern/58745: nouveau triggered assert in linux_dma_fence.c
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.18 -r1.19 \
     src/sys/external/bsd/drm2/dist/drm/nouveau/nouveau_fence.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.