Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1283232 > unrolled thread

Re: [RFC2 PATCH v6 00/19] ILP32 for ARM64

Started byCatalin Marinas <catalin.marinas@arm.com>
First post2015-12-03 19:10 +0100
Last post2015-12-04 18:20 +0100
Articles 3 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [RFC2 PATCH v6 00/19] ILP32 for ARM64 Catalin Marinas <catalin.marinas@arm.com> - 2015-12-03 19:10 +0100
    Re: [RFC2 PATCH v6 00/19] ILP32 for ARM64 Yury Norov <ynorov@caviumnetworks.com> - 2015-12-04 16:40 +0100
      Re: [RFC2 PATCH v6 00/19] ILP32 for ARM64 Catalin Marinas <catalin.marinas@arm.com> - 2015-12-04 18:20 +0100

#1283232 — Re: [RFC2 PATCH v6 00/19] ILP32 for ARM64

FromCatalin Marinas <catalin.marinas@arm.com>
Date2015-12-03 19:10 +0100
SubjectRe: [RFC2 PATCH v6 00/19] ILP32 for ARM64
Message-ID<qBGum-98-21@gated-at.bofh.it>
On Wed, Nov 18, 2015 at 12:16:40AM +0300, Yury Norov wrote:
>  - ILP32 VDSO code excluded. It's not mandatory, and caused questions
>    during review process. We definitely make sure we will follow up
>    with a VDSO later on because it is needed for performance reasons;

I already asked in one of the patches, don't you need a VDSO for
sigreturn?

> The full regression table is:
> 		ILP32		LP64
> 
> float_bessel    FAIL	 134    PASSED	0
> float_exp_log   FAIL	 134    PASSED	0
> float_iperb     FAIL	 134    PASSED	0
> float_power     FAIL	 134    PASSED	0
> float_trigo     FAIL	 134    PASSED	0
> abort01         FAIL       2    FAIL 	2
> fcntl14         FAIL       2    FAIL 	2
> kill11          FAIL       2    FAIL 	2
> mmap16          FAIL       6    PASSED	0
> open12          FAIL       2    PASSED	0
> pause01         PASSED     0    FAIL 	9
> pipe07          FAIL       2    PASSED	0
> readdir01       FAIL       1    PASSED	0
> rename11        FAIL       2    PASSED	0
> rmdir02         FAIL       2    PASSED	0
> setregid02      FAIL       1    FAIL 	1
> settimeofday01  FAIL       1    FAIL 	1
> umount2_01      FAIL       2    PASSED	0
> umount2_02      FAIL       2    PASSED	0
> umount2_03      FAIL       2    PASSED	0
> utime06         FAIL       2    PASSED	0

Have you run any AArch32 LTP with these patches? The results should also
be included in your table above.

-- 
Catalin
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1283943

FromYury Norov <ynorov@caviumnetworks.com>
Date2015-12-04 16:40 +0100
Message-ID<qC0CK-4wp-23@gated-at.bofh.it>
In reply to#1283232
On Thu, Dec 03, 2015 at 05:59:55PM +0000, Catalin Marinas wrote:
> On Wed, Nov 18, 2015 at 12:16:40AM +0300, Yury Norov wrote:
> >  - ILP32 VDSO code excluded. It's not mandatory, and caused questions
> >    during review process. We definitely make sure we will follow up
> >    with a VDSO later on because it is needed for performance reasons;
> 
> I already asked in one of the patches, don't you need a VDSO for
> sigreturn?
> 

I didn't realise it's mandatory for ARMv8 when removed it. I'll back
it, but I think rework required anyway.

> 
> Have you run any AArch32 LTP with these patches? The results should also
> be included in your table above.
> 

The full regression table is:
		ILP32		LP64            AARCH32 (v4.3 & v4.3 + ILP32)

float_bessel    FAIL	 134    PASSED	0       PASSED	0
float_exp_log   FAIL	 134    PASSED	0       PASSED	0
float_iperb     FAIL	 134    PASSED	0       PASSED	0
float_power     FAIL	 134    PASSED	0       PASSED	0
float_trigo     FAIL	 134    PASSED	0       PASSED	0
abort01         FAIL       2    FAIL 	2       FAIL 	2
fcntl14         FAIL       2    FAIL 	2       PASSED	0
kill11          FAIL       2    FAIL 	2       FAIL 	2
mmap16          FAIL       6    PASSED	0       PASSED	0
open12          FAIL       2    PASSED	0       PASSED	0
pause01         PASSED     0    FAIL 	9       FAIL 	9
pipe07          FAIL       2    PASSED	0       PASSED	0
readdir01       FAIL       1    PASSED	0       PASSED	0
rename11        FAIL       2    PASSED	0       PASSED	0
rmdir02         FAIL       2    PASSED	0       PASSED	0
setregid02      FAIL       1    FAIL 	1       PASSED	0
settimeofday01  FAIL       1    FAIL 	1       FAIL 	5
umount2_01      FAIL       2    PASSED	0       PASSED	0
umount2_02      FAIL       2    PASSED	0       PASSED	0
umount2_03      FAIL       2    PASSED	0       PASSED	0
utime06         FAIL       2    PASSED	0       PASSED	0

Fortunately, there's no aarch32 regression. Note that pause01 and
setregid02 are not stable. 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1284064

FromCatalin Marinas <catalin.marinas@arm.com>
Date2015-12-04 18:20 +0100
Message-ID<qC2bx-5CT-35@gated-at.bofh.it>
In reply to#1283943
On Fri, Dec 04, 2015 at 06:35:50PM +0300, Yury Norov wrote:
> On Thu, Dec 03, 2015 at 05:59:55PM +0000, Catalin Marinas wrote:
> > On Wed, Nov 18, 2015 at 12:16:40AM +0300, Yury Norov wrote:
> > >  - ILP32 VDSO code excluded. It's not mandatory, and caused questions
> > >    during review process. We definitely make sure we will follow up
> > >    with a VDSO later on because it is needed for performance reasons;
> > 
> > I already asked in one of the patches, don't you need a VDSO for
> > sigreturn?
> 
> I didn't realise it's mandatory for ARMv8 when removed it. I'll back
> it, but I think rework required anyway.

The alternative is to use SA_RESTORER but AFAIK, its use has been
deprecated. I don't know what aarch64 glibc does if it doesn't have a
vdso.

-- 
Catalin
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web