Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 6 Aug 2023 20:29:57 GMT
From:      Marius Strobl <marius@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: 96c76d930656 - main - exit1(): Revert sparc64 workaround
Message-ID:  <202308062029.376KTvj6025811@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by marius:

URL: https://cgit.FreeBSD.org/src/commit/?id=96c76d930656f13d3c041dc8d8f2fc3dd1e5e05a

commit 96c76d930656f13d3c041dc8d8f2fc3dd1e5e05a
Author:     Marius Strobl <marius@FreeBSD.org>
AuthorDate: 2023-08-06 19:54:57 +0000
Commit:     Marius Strobl <marius@FreeBSD.org>
CommitDate: 2023-08-06 20:26:01 +0000

    exit1(): Revert sparc64 workaround
    
    If this still is a problem on other architectures, it should be fixed
    properly.
    
    This reverts commit 5486ffc898503a846ecaf3f5ef9f9269beb4540e.
---
 sys/kern/kern_exit.c | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/sys/kern/kern_exit.c b/sys/kern/kern_exit.c
index cb0939ded6e1..e3345c14df26 100644
--- a/sys/kern/kern_exit.c
+++ b/sys/kern/kern_exit.c
@@ -238,13 +238,7 @@ exit1(struct thread *td, int rval, int signo)
 	TSPROCEXIT(td->td_proc->p_pid);
 
 	p = td->td_proc;
-	/*
-	 * XXX in case we're rebooting we just let init die in order to
-	 * work around an unsolved stack overflow seen very late during
-	 * shutdown on sparc64 when the gmirror worker process exists.
-	 * XXX what to do now that sparc64 is gone... remove if?
-	 */
-	if (p == initproc && rebooting == 0) {
+	if (p == initproc) {
 		printf("init died (signal %d, exit %d)\n", signo, rval);
 		panic("Going nowhere without my init!");
 	}



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202308062029.376KTvj6025811>