On 6/16/21 1:07 AM, Taylor Simpson wrote:
+    /*
+     * The on-stack signal trampoline is no longer executed;
+     * however, the libgcc signal frame unwinding code checks
+     * for the presence of these two numeric magic values.
+     */

Hexagon uses musl, not libgcc.  So, I'm not sure if this is needed.  The 
signals.c test passes for me without this change.  Are you seeing it fail?

I copied the comment from the kernel source.


+void setup_sigtramp(abi_ulong sigtramp_page) {
+    uint32_t *tramp = lock_user(VERIFY_WRITE, sigtramp_page, 4 * 2, 0);
+    assert(tramp != NULL);
+
+    default_rt_sigreturn = sigtramp_page;
+    install_sigtramp(tramp);
+
+    unlock_user(tramp, sigtramp_page, 4 * 2); }

Put the closing curly on a new line.

That's your mailer.  It's correct in the original.


r~

Reply via email to