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

["Ramiro Aceves via gnats" <gnats-admin%NetBSD.org@localhost>]

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

From: Ramiro Aceves <ea1abz%gmail.com@localhost>
To: gnats-bugs%netbsd.org@localhost, gnats-admin%netbsd.org@localhost, netbsd-bugs%netbsd.org@localhost,
 emmankoko519%gmail.com@localhost
Cc: 
Subject: Re: PR/59773 CVS commit: src/sys/external/bsd/drm2/dist/drm/nouveau
Date: Sun, 23 Nov 2025 12:37:43 +0100

 El 22/11/25 a las 23:35, Taylor R Campbell escribió:
 > The following reply was made to PR bin/59773; it has been noted by GNATS.
 > 
 > From: "Taylor R Campbell" <riastradh%netbsd.org@localhost>
 > To: gnats-bugs%gnats.NetBSD.org@localhost
 > Cc:
 > Subject: PR/59773 CVS commit: src/sys/external/bsd/drm2/dist/drm/nouveau
 > Date: Sat, 22 Nov 2025 22:32:48 +0000
 > 
 >   Module Name:	src
 >   Committed By:	riastradh
 >   Date:		Sat Nov 22 22:32:48 UTC 2025
 >   
 >   Modified Files:
 >   	src/sys/external/bsd/drm2/dist/drm/nouveau: nouveau_fence.c
 >   
 >   Log Message:
 >   nouveau(4): Fix return value of fence wait.
 >   
 >   kpause(9) returns EAGAIN/EWOULDBLOCK when it times out, which is
 >   perfectly normal and expected here with an internal one-tick timeout
 >   which should just go through another loop iteration.  This function
 >   is expected to return the number of ticks left to wait when done, so
 >   it reports timeout by returning zero, and is forbidden to fail with
 >   EAGAIN/EWOULDBLOCK.
 >   
 >   This is also forbidden to fail with EINTR -- on interruption it must
 >   fail with the Linux ERESTARTSYS instead.  So map EINTR to that too
 >   while here.
 >   
 >   PR kern/59773: i386 11.0_BETA system crash (Xorg nouveau ?)
 >   
 >   
 >   To generate a diff of this commit:
 >   cvs rdiff -u -r1.17 -r1.18 \
 >       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.
 >   
 
 Thanks so much .