Currently,the ARM family assemblers only support vfp3.But the function isVFPPresent just determines if we want to use vfp by checking HWCAP_VFP. int fd = open("/proc/self/auxv", O_RDONLY); if (fd > 0) { Elf32_auxv_t aux; while (read(fd, &aux, sizeof(Elf32_auxv_t))) { if (aux.a_type == AT_HWCAP) { close(fd); return aux.a_un.a_val & HWCAP_VFP; } } close(fd); } This flag is on whenever the cpu has a vfp.But for armv5 family cpus,they just have vfp,not vfp3.So this will cause crash on these machines. You should change HWCAP_VFP to HWCAP_VFPv3. As the Linux kernel source arch/arm/vfp/vfpmodule.c descripts: #ifdef CONFIG_VFPv3 if (VFP_arch >= 2) { elf_hwcap |= HWCAP_VFPv3; /* * Check for VFPv3 D16. CPUs in this configuration * only have 16 x 64bit registers. */ if (((fmrx(MVFR0) & MVFR0_A_SIMD_MASK)) == 1) elf_hwcap |= HWCAP_VFPv3D16; } #endif
you don't want me to submit a patch,because I can't checkout the whole svn trunk.I am in China,where is "protected" by the Great Fire Wall.
well,seems a patch is mandatory here
Created attachment 194165 [details] change to vfp3
Created attachment 194166 [details] patch
Created attachment 194167 [details] patch
Comment on attachment 194167 [details] patch Rejecting attachment 194167 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-01', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 194167, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: /git.webkit.org/WebKit ed46a0b..9cd447a master -> origin/master Partial-rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ... Currently at 158402 = ed46a0b8b39c279bd312a1751a6d47cd897fafb8 r158404 = 9cd447a9654a49da4bb567a48886a88e9c6a144b Done rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc First, rewinding head to replay your work on top of it... Fast-forwarded master to refs/remotes/origin/master. Full output: http://webkit-queues.appspot.com/results/19498010
Hey. This patch has been sitting here for 2 years, with an r+ and just with a cq- because it don't has a changelog. The patch still applies, should we commit it? It only needs a changelog.
(In reply to comment #7) > Hey. This patch has been sitting here for 2 years, with an r+ and just with > a cq- because it don't has a changelog. > > The patch still applies, should we commit it? It only needs a changelog. I agree, let's add a changelog and commit it.