From patchwork Tue Jun 21 11:57:25 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Zhang Jian(Bamvor)" X-Patchwork-Id: 13280 Received: (qmail 117388 invoked by alias); 21 Jun 2016 11:59:06 -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 Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 117296 invoked by uid 89); 21 Jun 2016 11:59:05 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.6 required=5.0 tests=AWL, BAYES_00, RP_MATCHES_RCVD, SPF_PASS autolearn=ham version=3.3.2 spammy= X-HELO: szxga05-in.huawei.com Subject: Re: [PATCH 11/27] [AARCH64] Syscalls for ILP32 are passed always via 64bit values. To: Andreas Schwab , Yury Norov References: <1466485631-3532-1-git-send-email-ynorov@caviumnetworks.com> <1466485631-3532-13-git-send-email-ynorov@caviumnetworks.com> <5769281D.6090109@huawei.com> CC: , , , , , , , , , , , Andrew Pinski , , "Zhangjian (Bamvor)" From: "Zhangjian (Bamvor)" Message-ID: <57692BA5.5050002@huawei.com> Date: Tue, 21 Jun 2016 19:57:25 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0 MIME-Version: 1.0 In-Reply-To: <5769281D.6090109@huawei.com> X-CFilter-Loop: Reflected X-Mirapoint-Virus-RAPID-Raw: score=unknown(0), refid=str=0001.0A020206.57692BB8.00CA, ss=1, re=0.000, recu=0.000, reip=0.000, cl=1, cld=1, fgs=0, ip=0.0.0.0, so=2014-11-16 11:51:01, dmn=2013-03-21 17:37:32 X-Mirapoint-Loop-Id: 11bc6b85de004f4b64ba1c69552c9c5f Hi, On 2016/6/21 19:42, Zhangjian (Bamvor) wrote: > Hi, > > On 2016/6/21 15:56, Andreas Schwab wrote: >> Yury Norov writes: >> >>> diff --git a/sysdeps/unix/sysv/linux/aarch64/sysdep.h b/sysdeps/unix/sysv/linux/aarch64/sysdep.h >>> index 1ffabc2..42f89c8 100644 >>> --- a/sysdeps/unix/sysv/linux/aarch64/sysdep.h >>> +++ b/sysdeps/unix/sysv/linux/aarch64/sysdep.h >>> @@ -161,11 +161,11 @@ >>> call. */ >>> # undef INLINE_SYSCALL >>> # define INLINE_SYSCALL(name, nr, args...) \ >>> - ({ unsigned long _sys_result = INTERNAL_SYSCALL (name, , nr, args); \ >>> + ({ unsigned long long _sys_result = INTERNAL_SYSCALL (name, , nr, args); \ >>> if (__builtin_expect (INTERNAL_SYSCALL_ERROR_P (_sys_result, ), 0))\ >>> { \ >>> __set_errno (INTERNAL_SYSCALL_ERRNO (_sys_result, )); \ >>> - _sys_result = (unsigned long) -1; \ >>> + _sys_result = (unsigned long long) -1; \ >>> } \ >>> (long) _sys_result; }) >> >> The cast should be (long long). > My understanding that we need more conversion, reference the follow patch: Sorry, I used the wrong version of patch, here is the version could apply on yury'. For vdso part, maybe we should fix in this patch instead of "[PATCH 19/27] [AARCH64] Add typesizes.h for ILP32". From 8c5f27c19407ba3ba8c400f0645cb51f91a5b379 Mon Sep 17 00:00:00 2001 From: gaoyongliang Date: Thu, 19 May 2016 18:19:05 +0800 Subject: [PATCH] ilp32 fix syscall return value to use long long Signed-off-by: Yongliang Gao Signed-off-by: Bamvor Jian Zhang --- sysdeps/unix/sysdep.h | 2 +- sysdeps/unix/sysv/linux/aarch64/sysdep.h | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/sysdeps/unix/sysdep.h b/sysdeps/unix/sysdep.h index 94a2ce0..a12456b 100644 --- a/sysdeps/unix/sysdep.h +++ b/sysdeps/unix/sysdep.h @@ -53,7 +53,7 @@ #define SYSCALL_CANCEL(...) \ ({ \ - long int sc_ret; \ + long long sc_ret; \ if (SINGLE_THREAD_P) \ sc_ret = __SYSCALL_CALL (__VA_ARGS__); \ else \ diff --git a/sysdeps/unix/sysv/linux/aarch64/sysdep.h b/sysdeps/unix/sysv/linux/aarch64/sysdep.h index 42f89c8..82de012 100644 --- a/sysdeps/unix/sysv/linux/aarch64/sysdep.h +++ b/sysdeps/unix/sysv/linux/aarch64/sysdep.h @@ -167,7 +167,7 @@ __set_errno (INTERNAL_SYSCALL_ERRNO (_sys_result, )); \ _sys_result = (unsigned long long) -1; \ } \ - (long) _sys_result; }) + (long long) _sys_result; }) # undef INTERNAL_SYSCALL_DECL # define INTERNAL_SYSCALL_DECL(err) do { } while (0) @@ -194,7 +194,7 @@ # undef INTERNAL_SYSCALL_ERROR_P # define INTERNAL_SYSCALL_ERROR_P(val, err) \ - ((unsigned long) (val) >= (unsigned long) -4095) + ((unsigned long long) (val) >= (unsigned long long) -4095) # undef INTERNAL_SYSCALL_ERRNO # define INTERNAL_SYSCALL_ERRNO(val, err) (-(val))