Content-Disposition: inline
Content-Transfer-Encoding: 8bit
Content-Type: text/plain; charset="US-ASCII"
MIME-Version: 1.0
Date: Thu, 18 Apr 2024 18:17:06 +0000
From: "Martin Husemann" <martin@netbsd.org>
Subject: CVS commit: [netbsd-10] src/sys/arch/aarch64/aarch64
To: source-changes@NetBSD.org
Message-Id: <20240418181706.52353FA2C@cvs.NetBSD.org>
Sender: source-changes-owner@NetBSD.org
Precedence: bulk
Reply-To: source-changes-d@NetBSD.org
Mail-Reply-To: "Martin Husemann" <martin@netbsd.org>
Mail-Followup-To: source-changes-d@NetBSD.org

Module Name:	src
Committed By:	martin
Date:		Thu Apr 18 18:17:06 UTC 2024

Modified Files:
	src/sys/arch/aarch64/aarch64 [netbsd-10]: cpu_machdep.c sig_machdep.c

Log Message:
Pull up following revision(s) (requested by skrll in ticket #667):

	sys/arch/aarch64/aarch64/sig_machdep.c: revision 1.9
	sys/arch/aarch64/aarch64/cpu_machdep.c: revision 1.15

kern/58149: aarch64: Cannot return from a signal handler if SP was
misaligned when the signal arrived

Apply the kernel diff from the PR
1. sendsig_siginfo() previously assumed that user SP was always aligned to
    16 bytes and could call signal handlers with SP misaligned. This is a
    wrong assumption because aarch64 demands that SP is aligned *only while*
    it's being used to access memory. Now it properly aligns it before
    pusing anything on the stack.
2. cpu_mcontext_validate() used to check if _REG_SP was aligned and
    considered the ucontext invalid otherwise. This meant if a signal was
    sent to a process whose SP was misaligned, the signal handler would fail
    to return because the ucontext passed from the kernel was an invalid
    one. Now setcontext(2) doesn't complain about misaligned SP.


To generate a diff of this commit:
cvs rdiff -u -r1.13 -r1.13.4.1 src/sys/arch/aarch64/aarch64/cpu_machdep.c
cvs rdiff -u -r1.8 -r1.8.4.1 src/sys/arch/aarch64/aarch64/sig_machdep.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.