Hi Sylvestre,

Thank you.

It seems that a part that refers to z_Linux_util.c is missing
in your patch.

The patch that contains dropped changes is attached.
With this additional patch I was able to build openmprtl 3.7.0-2 for mipsel.

Regards,
Dejan




________________________________________
From: Sylvestre Ledru [[email protected]]
Sent: Tuesday, October 20, 2015 1:32 PM
To: Dejan Latinovic
Subject: Re: Bug#790735: openmprtl: add MIPS support

It has been accepted.

It fails under mips/mipsel
cd /«PKGBUILDDIR»/obj-mipsel-linux-gnu/runtime/src && /usr/bin/cc  
-Domp_EXPORTS -fPIC -I/«PKGBUILDDIR»/obj-mipsel-linux-gnu/runtime/src 
-I/«PKGBUILDDIR»/runtime/src -I/«PKGBUILDDIR»/runtime/src/i18n 
-I/«PKGBUILDDIR»/runtime/src/include/41 
-I/«PKGBUILDDIR»/runtime/src/thirdparty/ittnotify    -D
USE_ITT_BUILD -D KMP_ARCH_STR="\"mips\"" -D BUILD_I8 -D 
KMP_LIBRARY_FILE=\"libomp.so.5\" -D KMP_VERSION_MAJOR=5 -D KMP_NESTED_HOT_TEAMS 
-D CACHE_LINE=64 -D KMP_ADJUST_BLOCKTIME=1 -D BUILD_PARALLEL_ORDERED -D 
KMP_ASM_INTRINS -D USE_ITT_NOTIFY=1 -D INTEL_ITTNOTIFY_PREFIX=__kmp_itt_ -D
KMP_USE_VERSION_SYMBOLS -D _GNU_SOURCE -D _REENTRANT -D BUILD_TV -D 
USE_CBLKDATA -D KMP_GOMP_COMPAT -D USE_LOAD_BALANCE -D KMP_TDATA_GTID -D 
KMP_USE_ASSERT -D KMP_DYNAMIC_LIB -D KMP_STATS_ENABLED=0 -D USE_DEBUGGER=0 -D 
OMPT_SUPPORT=0 -D OMPT_BLAME=1 -D OMPT_TRACE=1 -D OMP_50_ENABLED=0 -D
OMP_41_ENABLED=1 -D OMP_40_ENABLED=1 -D OMP_30_ENABLED=1 -D 
KMP_USE_ADAPTIVE_LOCKS=0 -D KMP_DEBUG_ADAPTIVE_LOCKS=0 -D 
KMP_USE_INTERNODE_ALIGNMENT=0 -std=c++11 -fno-exceptions -Wno-sign-compare 
-Wno-unused-function -Wno-unused-value -Wno-unused-variable -Wno-switch 
-Wno-unknown-pragmas
-Wno-missing-field-initializers -Wno-missing-braces -Wno-comment -x c++ -o 
CMakeFiles/omp.dir/z_Linux_util.c.o   -c 
/«PKGBUILDDIR»/runtime/src/z_Linux_util.c
/«PKGBUILDDIR»/runtime/src/z_Linux_util.c:176:5: error: #error Unknown or 
unsupported architecture
 #   error Unknown or unsupported architecture
     ^

but I guess this should be easy to fix (I can give it a try when I found the 
time)

S

Le 09/10/2015 17:05, Dejan Latinovic a écrit :
>
> Ok, thanks.
>
> D
>
> ________________________________________
> From: Sylvestre Ledru [[email protected]]
> Sent: Friday, October 09, 2015 2:20 PM
> To: Dejan Latinovic
> Subject: Re: Bug#790735: openmprtl: add MIPS support
>
> Le 09/10/2015 14:12, Sylvestre Ledru a écrit :
>> Le 09/10/2015 12:52, Dejan Latinovic a écrit :
>>>
>>>
>>> Hi Sylvestre,
>>> do we have any progress on this?
>> yes, in new for 3 weeks :(
>> https://ftp-master.debian.org/new.html
>>
>> I am waiting for this to be accepted and see the build results
>> before forwarding the patch upstream.
> I just pinged the ftp master team about it
>
>
> S
>
>
--- openmprtl-3.7.0.orig/runtime/src/z_Linux_util.c
+++ openmprtl-3.7.0/runtime/src/z_Linux_util.c
@@ -171,6 +171,29 @@ __kmp_print_cond( char *buffer, kmp_cond
 #    error Wrong code for getaffinity system call.
 #   endif /* __NR_sched_getaffinity */
 
+#  elif KMP_ARCH_MIPS
+#   ifndef __NR_sched_setaffinity
+#    define __NR_sched_setaffinity  4239
+#   elif __NR_sched_setaffinity != 4239
+#    error Wrong code for setaffinity system call.
+#   endif /* __NR_sched_setaffinity */
+#   ifndef __NR_sched_getaffinity
+#    define __NR_sched_getaffinity  4240
+#   elif __NR_sched_getaffinity != 4240
+#    error Wrong code for getaffinity system call.
+#   endif /* __NR_sched_getaffinity */
+
+#  elif KMP_ARCH_MIPS64
+#   ifndef __NR_sched_setaffinity
+#    define __NR_sched_setaffinity  5195
+#   elif __NR_sched_setaffinity != 5195
+#    error Wrong code for setaffinity system call.
+#   endif /* __NR_sched_setaffinity */
+#   ifndef __NR_sched_getaffinity
+#    define __NR_sched_getaffinity  5196
+#   elif __NR_sched_getaffinity != 5196
+#    error Wrong code for getaffinity system call.
+#   endif /* __NR_sched_getaffinity */
 
 #  else
 #   error Unknown or unsupported architecture

Reply via email to