-
Notifications
You must be signed in to change notification settings - Fork 137
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
POWER VSX PowerPC64LE Linux support #313
Comments
it's already ported and It was tested on wide versions of GCC and CLANG during working on gh-288
That will require a full review of all existing kernels. I think it's better to merge #291(still need improvements similar to what happened on gh-288) and then review both VSX/zVX in the same time.
please could you set |
36/39 Test #36: roundtriptest2dsp_4_4 ............***Failed 64.36 sec
37/39 Test #37: roundtriptest2dsp_8_8 ............***Failed 0.55 sec
38/39 Test #38: roundtriptest2dsp_10_10 ..........***Failed 0.25 sec
39/39 Test #39: roundtriptest2dsp_5_15 ...........***Failed 0.34 sec |
@edelsohn, Could please you provides gcc version? /opt/at12.0/bin/gcc --version
gcc (GCC) 8.3.1 20190304 (Advance-Toolchain-at12.0) [revision 269374]
Copyright (C) 2018 Free Software Foundation, Inc.
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
ldd ./bin/roundtriptest2dsp
linux-vdso64.so.1 (0x00007fffa8ed0000)
libsleefdft.so.3 => /opencv/sleef/build_gcc8/lib/libsleefdft.so.3 (0x00007fffa8ce0000)
libsleef.so.3 => /opencv/sleef/build_gcc8/lib/libsleef.so.3 (0x00007fffa8bc0000)
libm.so.6 => /opt/at12.0/lib64/power9/libm.so.6 (0x00007fffa8a80000)
libgomp.so.1 => /opt/at12.0/lib64/power9/libgomp.so.1 (0x00007fffa8a20000)
libpthread.so.0 => /opt/at12.0/lib64/power9/libpthread.so.0 (0x00007fffa89d0000)
libc.so.6 => /opt/at12.0/lib64/power9/libc.so.6 (0x00007fffa87a0000)
/opt/at12.0/lib64/ld64.so.2 (0x00007fffa8ef0000)
libdl.so.2 => /opt/at12.0/lib64/libdl.so.2 (0x00007fffa8770000) |
I'm testing Power on GNU Compile Farm gcc135. I set CC= before running cmake. Is there something else necessary to properly configure SLEEF, especially for Power VSX? [dje@gcc135 bin]$ ldd ./roundtriptest2dsp |
as I have access to the Cfarm.
I am using locally build cmake. as I dont have sudoer access
|
It requires some changes to cmake file related to root path and searches dir. I'm going to release a patch adding CMake toolchain for AT. |
@edelsohn It does not reproduce on my environment. |
This bug is actually not just for POWER. It can be reproduced on other architectures using GCC. FFTW needs to be uninstalled to have these tests running. |
The current master (f50b231) is not building on power9:
Tested on Power9 with GCC10 (debian11). The previous release (3.5.1) was building and all tests are passing. |
I confirmed the problem. |
The bounty mostly is complete. We continue to work with @shibatch to adjust some issues, but the majority of the work already has been completed by Shibata-san. |
All the tests are passing in POWER9 with AT14.0. |
Sure. I will start working on it. |
I made PR #360. |
This patch adds POWER9 support as mentioned in issue #313. Co-authored-by: shibatch <[email protected]>
This has been accomplished with the latest pull request for P9 support. |
Port Sleef to IBM POWER VSX architecture supporting PowerPC64LE Linux compiled with both GCC and Clang. Achieve equivalent speedup to x86-64 and AArch64 appropriate for the VSX 128 bit vector width.
With Github clone from today on CentOS8 built with GCC AT 12.0, I see the following failures:
The following tests FAILED:
36 - roundtriptest2dsp_4_4 (Failed)
37 - roundtriptest2dsp_8_8 (Failed)
38 - roundtriptest2dsp_10_10 (Failed)
39 - roundtriptest2dsp_5_15 (Failed)
The text was updated successfully, but these errors were encountered: