From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS31976 209.132.180.0/23 X-Spam-Status: No, score=-4.1 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED,RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL,SPF_HELO_PASS,SPF_PASS shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dcvr.yhbt.net (Postfix) with ESMTPS id 474221F5A2 for ; Sun, 9 Feb 2020 18:57:58 +0000 (UTC) DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:from:to:cc:subject:date:message-id :mime-version:content-transfer-encoding; q=dns; s=default; b=OGw AK7FGkfj4V9WY4Wgrji+Jf/guIm57Xn7+VzIXQoUSEIqpRpTH7oJvrpTheMZpr7e 9RTVR4kUn1h4eedr2wAGbf8klwTpVAnBI+iS/Dl+kguUcatmHKPn2gqNs8NRpxuL cgTk2gCN8OhtltGyUKkgvk9DJozf2msEWcpwlV2M= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:from:to:cc:subject:date:message-id :mime-version:content-transfer-encoding; s=default; bh=Sh8zPGRJ5 tejbDKKCwVTfzzw3Mo=; b=vWiq8HxSvjc7I1bCc4rRzq1oiS4Swh4XxmZnZAVcP l4BKsJjwTI63hUjxp+u4piEvDYFVZNbJAwsxGLKv9TK7Kk15ydLVG09cAt7Wlz86 dQ/J4hPXqXxSSbIY3EridzE37DUcmiooZBqN1Wzqf2pkizFAolmVvKWt/XtcNaOe qU= Received: (qmail 79907 invoked by alias); 9 Feb 2020 18:57:55 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Received: (qmail 79899 invoked by uid 89); 9 Feb 2020 18:57:55 -0000 Authentication-Results: sourceware.org; auth=none X-HELO: mailbox.box.xen0n.name DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=xen0n.name; s=mail; t=1581274666; bh=LtXvZTVbChpPW0uSECB13j1u5L0QSWcczGZYFqpxQxo=; h=From:To:Cc:Subject:Date:From; b=cGYS5MjT2YoVy/0nwhtbswiDuLhjFMtn9xnY+GFVsUwBoWUl1PkEI8DbCkcVGFEyC rgYfyYYYOO8GP+TD+QGBIFrmAeidlpPJX+ZoPO/5gRSRQeOuDG3ZqmEI4AXu/0Jq2U CHoAInk7todIDCTR+SF6ZZfHretlpzMrZRZOuyWU= From: WANG Xuerui To: libc-alpha@sourceware.org Cc: WANG Xuerui Subject: [PATCH] mips: Fix argument passing for inlined syscalls on Linux [BZ #25523] Date: Mon, 10 Feb 2020 02:57:29 +0800 Message-Id: <20200209185729.15896-1-git@xen0n.name> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit According to [gcc documentation][1], temporary variables must be used for the desired content to not be call-clobbered. Fix the Linux inline syscall templates by adding temporary variables, much like what x86 did before (commit 381a0c26d73e0f074c962e0ab53b99a6c327066d). Tested with gcc 9.2.0, both cross-compiled and natively on Loongson 3A4000. [1]: https://gcc.gnu.org/onlinedocs/gcc/Local-Register-Variables.html --- sysdeps/unix/sysv/linux/mips/mips32/sysdep.h | 30 ++++++--- .../unix/sysv/linux/mips/mips64/n32/sysdep.h | 63 ++++++++++++------- .../unix/sysv/linux/mips/mips64/n64/sysdep.h | 63 ++++++++++++------- 3 files changed, 104 insertions(+), 52 deletions(-) diff --git a/sysdeps/unix/sysv/linux/mips/mips32/sysdep.h b/sysdeps/unix/sysv/linux/mips/mips32/sysdep.h index beefcf284b..c275d63f67 100644 --- a/sysdeps/unix/sysv/linux/mips/mips32/sysdep.h +++ b/sysdeps/unix/sysv/linux/mips/mips32/sysdep.h @@ -178,10 +178,11 @@ union __mips_syscall_return long _sys_result; \ \ { \ + long _arg1 = (long) (arg1); \ register long __s0 asm ("$16") __attribute__ ((unused)) \ = (number); \ register long __v0 asm ("$2"); \ - register long __a0 asm ("$4") = (long) (arg1); \ + register long __a0 asm ("$4") = _arg1; \ register long __a3 asm ("$7"); \ __asm__ volatile ( \ ".set\tnoreorder\n\t" \ @@ -202,11 +203,13 @@ union __mips_syscall_return long _sys_result; \ \ { \ + long _arg1 = (long) (arg1); \ + long _arg2 = (long) (arg2); \ register long __s0 asm ("$16") __attribute__ ((unused)) \ = (number); \ register long __v0 asm ("$2"); \ - register long __a0 asm ("$4") = (long) (arg1); \ - register long __a1 asm ("$5") = (long) (arg2); \ + register long __a0 asm ("$4") = _arg1; \ + register long __a1 asm ("$5") = _arg2; \ register long __a3 asm ("$7"); \ __asm__ volatile ( \ ".set\tnoreorder\n\t" \ @@ -228,12 +231,15 @@ union __mips_syscall_return long _sys_result; \ \ { \ + long _arg1 = (long) (arg1); \ + long _arg2 = (long) (arg2); \ + long _arg3 = (long) (arg3); \ register long __s0 asm ("$16") __attribute__ ((unused)) \ = (number); \ register long __v0 asm ("$2"); \ - register long __a0 asm ("$4") = (long) (arg1); \ - register long __a1 asm ("$5") = (long) (arg2); \ - register long __a2 asm ("$6") = (long) (arg3); \ + register long __a0 asm ("$4") = _arg1; \ + register long __a1 asm ("$5") = _arg2; \ + register long __a2 asm ("$6") = _arg3; \ register long __a3 asm ("$7"); \ __asm__ volatile ( \ ".set\tnoreorder\n\t" \ @@ -255,13 +261,17 @@ union __mips_syscall_return long _sys_result; \ \ { \ + long _arg1 = (long) (arg1); \ + long _arg2 = (long) (arg2); \ + long _arg3 = (long) (arg3); \ + long _arg4 = (long) (arg4); \ register long __s0 asm ("$16") __attribute__ ((unused)) \ = (number); \ register long __v0 asm ("$2"); \ - register long __a0 asm ("$4") = (long) (arg1); \ - register long __a1 asm ("$5") = (long) (arg2); \ - register long __a2 asm ("$6") = (long) (arg3); \ - register long __a3 asm ("$7") = (long) (arg4); \ + register long __a0 asm ("$4") = _arg1; \ + register long __a1 asm ("$5") = _arg2; \ + register long __a2 asm ("$6") = _arg3; \ + register long __a3 asm ("$7") = _arg4; \ __asm__ volatile ( \ ".set\tnoreorder\n\t" \ v0_init \ diff --git a/sysdeps/unix/sysv/linux/mips/mips64/n32/sysdep.h b/sysdeps/unix/sysv/linux/mips/mips64/n32/sysdep.h index f96636538a..958a889147 100644 --- a/sysdeps/unix/sysv/linux/mips/mips64/n32/sysdep.h +++ b/sysdeps/unix/sysv/linux/mips/mips64/n32/sysdep.h @@ -138,10 +138,11 @@ long _sys_result; \ \ { \ + long long _arg1 = ARGIFY (arg1); \ register long long __s0 asm ("$16") __attribute__ ((unused)) \ = (number); \ register long long __v0 asm ("$2"); \ - register long long __a0 asm ("$4") = ARGIFY (arg1); \ + register long long __a0 asm ("$4") = _arg1; \ register long long __a3 asm ("$7"); \ __asm__ volatile ( \ ".set\tnoreorder\n\t" \ @@ -162,11 +163,13 @@ long _sys_result; \ \ { \ + long long _arg1 = ARGIFY (arg1); \ + long long _arg2 = ARGIFY (arg2); \ register long long __s0 asm ("$16") __attribute__ ((unused)) \ = (number); \ register long long __v0 asm ("$2"); \ - register long long __a0 asm ("$4") = ARGIFY (arg1); \ - register long long __a1 asm ("$5") = ARGIFY (arg2); \ + register long long __a0 asm ("$4") = _arg1; \ + register long long __a1 asm ("$5") = _arg2; \ register long long __a3 asm ("$7"); \ __asm__ volatile ( \ ".set\tnoreorder\n\t" \ @@ -188,12 +191,15 @@ long _sys_result; \ \ { \ + long long _arg1 = ARGIFY (arg1); \ + long long _arg2 = ARGIFY (arg2); \ + long long _arg3 = ARGIFY (arg3); \ register long long __s0 asm ("$16") __attribute__ ((unused)) \ = (number); \ register long long __v0 asm ("$2"); \ - register long long __a0 asm ("$4") = ARGIFY (arg1); \ - register long long __a1 asm ("$5") = ARGIFY (arg2); \ - register long long __a2 asm ("$6") = ARGIFY (arg3); \ + register long long __a0 asm ("$4") = _arg1; \ + register long long __a1 asm ("$5") = _arg2; \ + register long long __a2 asm ("$6") = _arg3; \ register long long __a3 asm ("$7"); \ __asm__ volatile ( \ ".set\tnoreorder\n\t" \ @@ -215,13 +221,17 @@ long _sys_result; \ \ { \ + long long _arg1 = ARGIFY (arg1); \ + long long _arg2 = ARGIFY (arg2); \ + long long _arg3 = ARGIFY (arg3); \ + long long _arg4 = ARGIFY (arg4); \ register long long __s0 asm ("$16") __attribute__ ((unused)) \ = (number); \ register long long __v0 asm ("$2"); \ - register long long __a0 asm ("$4") = ARGIFY (arg1); \ - register long long __a1 asm ("$5") = ARGIFY (arg2); \ - register long long __a2 asm ("$6") = ARGIFY (arg3); \ - register long long __a3 asm ("$7") = ARGIFY (arg4); \ + register long long __a0 asm ("$4") = _arg1; \ + register long long __a1 asm ("$5") = _arg2; \ + register long long __a2 asm ("$6") = _arg3; \ + register long long __a3 asm ("$7") = _arg4; \ __asm__ volatile ( \ ".set\tnoreorder\n\t" \ v0_init \ @@ -242,14 +252,19 @@ long _sys_result; \ \ { \ + long long _arg1 = ARGIFY (arg1); \ + long long _arg2 = ARGIFY (arg2); \ + long long _arg3 = ARGIFY (arg3); \ + long long _arg4 = ARGIFY (arg4); \ + long long _arg5 = ARGIFY (arg5); \ register long long __s0 asm ("$16") __attribute__ ((unused)) \ = (number); \ register long long __v0 asm ("$2"); \ - register long long __a0 asm ("$4") = ARGIFY (arg1); \ - register long long __a1 asm ("$5") = ARGIFY (arg2); \ - register long long __a2 asm ("$6") = ARGIFY (arg3); \ - register long long __a3 asm ("$7") = ARGIFY (arg4); \ - register long long __a4 asm ("$8") = ARGIFY (arg5); \ + register long long __a0 asm ("$4") = _arg1; \ + register long long __a1 asm ("$5") = _arg2; \ + register long long __a2 asm ("$6") = _arg3; \ + register long long __a3 asm ("$7") = _arg4; \ + register long long __a4 asm ("$8") = _arg5; \ __asm__ volatile ( \ ".set\tnoreorder\n\t" \ v0_init \ @@ -270,15 +285,21 @@ long _sys_result; \ \ { \ + long long _arg1 = ARGIFY (arg1); \ + long long _arg2 = ARGIFY (arg2); \ + long long _arg3 = ARGIFY (arg3); \ + long long _arg4 = ARGIFY (arg4); \ + long long _arg5 = ARGIFY (arg5); \ + long long _arg6 = ARGIFY (arg6); \ register long long __s0 asm ("$16") __attribute__ ((unused)) \ = (number); \ register long long __v0 asm ("$2"); \ - register long long __a0 asm ("$4") = ARGIFY (arg1); \ - register long long __a1 asm ("$5") = ARGIFY (arg2); \ - register long long __a2 asm ("$6") = ARGIFY (arg3); \ - register long long __a3 asm ("$7") = ARGIFY (arg4); \ - register long long __a4 asm ("$8") = ARGIFY (arg5); \ - register long long __a5 asm ("$9") = ARGIFY (arg6); \ + register long long __a0 asm ("$4") = _arg1; \ + register long long __a1 asm ("$5") = _arg2; \ + register long long __a2 asm ("$6") = _arg3; \ + register long long __a3 asm ("$7") = _arg4; \ + register long long __a4 asm ("$8") = _arg5; \ + register long long __a5 asm ("$9") = _arg6; \ __asm__ volatile ( \ ".set\tnoreorder\n\t" \ v0_init \ diff --git a/sysdeps/unix/sysv/linux/mips/mips64/n64/sysdep.h b/sysdeps/unix/sysv/linux/mips/mips64/n64/sysdep.h index 9d30291f84..f47613deaf 100644 --- a/sysdeps/unix/sysv/linux/mips/mips64/n64/sysdep.h +++ b/sysdeps/unix/sysv/linux/mips/mips64/n64/sysdep.h @@ -134,10 +134,11 @@ long _sys_result; \ \ { \ + long _arg1 = (long) (arg1); \ register long __s0 asm ("$16") __attribute__ ((unused)) \ = (number); \ register long __v0 asm ("$2"); \ - register long __a0 asm ("$4") = (long) (arg1); \ + register long __a0 asm ("$4") = _arg1; \ register long __a3 asm ("$7"); \ __asm__ volatile ( \ ".set\tnoreorder\n\t" \ @@ -158,11 +159,13 @@ long _sys_result; \ \ { \ + long _arg1 = (long) (arg1); \ + long _arg2 = (long) (arg2); \ register long __s0 asm ("$16") __attribute__ ((unused)) \ = (number); \ register long __v0 asm ("$2"); \ - register long __a0 asm ("$4") = (long) (arg1); \ - register long __a1 asm ("$5") = (long) (arg2); \ + register long __a0 asm ("$4") = _arg1; \ + register long __a1 asm ("$5") = _arg2; \ register long __a3 asm ("$7"); \ __asm__ volatile ( \ ".set\tnoreorder\n\t" \ @@ -184,12 +187,15 @@ long _sys_result; \ \ { \ + long _arg1 = (long) (arg1); \ + long _arg2 = (long) (arg2); \ + long _arg3 = (long) (arg3); \ register long __s0 asm ("$16") __attribute__ ((unused)) \ = (number); \ register long __v0 asm ("$2"); \ - register long __a0 asm ("$4") = (long) (arg1); \ - register long __a1 asm ("$5") = (long) (arg2); \ - register long __a2 asm ("$6") = (long) (arg3); \ + register long __a0 asm ("$4") = _arg1; \ + register long __a1 asm ("$5") = _arg2; \ + register long __a2 asm ("$6") = _arg3; \ register long __a3 asm ("$7"); \ __asm__ volatile ( \ ".set\tnoreorder\n\t" \ @@ -211,13 +217,17 @@ long _sys_result; \ \ { \ + long _arg1 = (long) (arg1); \ + long _arg2 = (long) (arg2); \ + long _arg3 = (long) (arg3); \ + long _arg4 = (long) (arg4); \ register long __s0 asm ("$16") __attribute__ ((unused)) \ = (number); \ register long __v0 asm ("$2"); \ - register long __a0 asm ("$4") = (long) (arg1); \ - register long __a1 asm ("$5") = (long) (arg2); \ - register long __a2 asm ("$6") = (long) (arg3); \ - register long __a3 asm ("$7") = (long) (arg4); \ + register long __a0 asm ("$4") = _arg1; \ + register long __a1 asm ("$5") = _arg2; \ + register long __a2 asm ("$6") = _arg3; \ + register long __a3 asm ("$7") = _arg4; \ __asm__ volatile ( \ ".set\tnoreorder\n\t" \ v0_init \ @@ -238,14 +248,19 @@ long _sys_result; \ \ { \ + long _arg1 = (long) (arg1); \ + long _arg2 = (long) (arg2); \ + long _arg3 = (long) (arg3); \ + long _arg4 = (long) (arg4); \ + long _arg5 = (long) (arg5); \ register long __s0 asm ("$16") __attribute__ ((unused)) \ = (number); \ register long __v0 asm ("$2"); \ - register long __a0 asm ("$4") = (long) (arg1); \ - register long __a1 asm ("$5") = (long) (arg2); \ - register long __a2 asm ("$6") = (long) (arg3); \ - register long __a3 asm ("$7") = (long) (arg4); \ - register long __a4 asm ("$8") = (long) (arg5); \ + register long __a0 asm ("$4") = _arg1; \ + register long __a1 asm ("$5") = _arg2; \ + register long __a2 asm ("$6") = _arg3; \ + register long __a3 asm ("$7") = _arg4; \ + register long __a4 asm ("$8") = _arg5; \ __asm__ volatile ( \ ".set\tnoreorder\n\t" \ v0_init \ @@ -266,15 +281,21 @@ long _sys_result; \ \ { \ + long _arg1 = (long) (arg1); \ + long _arg2 = (long) (arg2); \ + long _arg3 = (long) (arg3); \ + long _arg4 = (long) (arg4); \ + long _arg5 = (long) (arg5); \ + long _arg6 = (long) (arg6); \ register long __s0 asm ("$16") __attribute__ ((unused)) \ = (number); \ register long __v0 asm ("$2"); \ - register long __a0 asm ("$4") = (long) (arg1); \ - register long __a1 asm ("$5") = (long) (arg2); \ - register long __a2 asm ("$6") = (long) (arg3); \ - register long __a3 asm ("$7") = (long) (arg4); \ - register long __a4 asm ("$8") = (long) (arg5); \ - register long __a5 asm ("$9") = (long) (arg6); \ + register long __a0 asm ("$4") = _arg1; \ + register long __a1 asm ("$5") = _arg2; \ + register long __a2 asm ("$6") = _arg3; \ + register long __a3 asm ("$7") = _arg4; \ + register long __a4 asm ("$8") = _arg5; \ + register long __a5 asm ("$9") = _arg6; \ __asm__ volatile ( \ ".set\tnoreorder\n\t" \ v0_init \ -- 2.24.1