<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2013/10/29 Sebastian Dröge <span dir="ltr"><<a href="mailto:sebastian@centricular.com" target="_blank">sebastian@centricular.com</a>></span><br> <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>On Mo, 2013-10-28 at 13:33 -0700, Andoni Morales Alastruey wrote:<br> > Module: sdk/cerbero<br> > Branch: master<br> > Commit: 4c29d8bc5b2487984e0415376f312bfd137729d4<br> > URL: <a href="http://cgit.freedesktop.org/gstreamer/sdk/cerbero/commit/?id=4c29d8bc5b2487984e0415376f312bfd137729d4" target="_blank">http://cgit.freedesktop.org/gstreamer/sdk/cerbero/commit/?id=4c29d8bc5b2487984e0415376f312bfd137729d4</a><br> ><br> > Author: Andoni Morales Alastruey <<a href="mailto:ylatuya@gmail.com" target="_blank">ylatuya@gmail.com</a>><br> > Date: Mon Oct 28 21:32:06 2013 +0100<br> ><br> > pixman: disable TLS in iOS<br> ><br> > TLS is not supported in this platform but the configure<br> > check does not complain so it needs to be disabled manually<br> <br> </div>Do you have some more details about this? It compiled fine for me and<br> IIRC also worked on a device. How does it fail for you?<br> <br> What about Android, we had it disabled there too until recently but same<br> story. Worked for me.<br></blockquote><div><br></div><div>It's something thatI didn't expect failing. The check for TLS is in 2 steps:<br></div><div>1) Check for __thread or __declspec(thread)<br></div><div>2) Check for pthread_specific<br> <br></div><div>In master the firt one succeeds, although it's not supported in the platorm:<br><i><br>libtool: compile: llvm-gcc-4.2 -DHAVE_CONFIG_H -I. -I.. -arch armv7 -mcpu=cortex-a8 -pipe -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS6.1.sdk -arch armv7 -mcpu=cortex-a8 -pipe -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS6.1.sdk -miphoneos-version-min=5.1 -Wall -g -Os -Wall -fno-strict-aliasing -fvisibility=hidden -MT pixman-implementation.lo -MD -MP -MF .deps/pixman-implementation.Tpo -c pixman-implementation.c -fno-common -DPIC -o .libs/pixman-implementation.o<br> {standard input}:819:Rest of line ignored. 1st junk character valued 40 (().<br>{standard input}:823:Rest of line ignored. 1st junk character valued 40 (().<br>{standard input}:827:Rest of line ignored. 1st junk character valued 40 (().<br> </i></div><div><i><br>Using clang instead we get a most interesting result:<br><br>bash-3.2$ clang -DHAVE_CONFIG_H -I. -I.. -arch armv7 -mcpu=cortex-a8 -pipe -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS6.1.sdk -arch armv7 -mcpu=cortex-a8 -pipe -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS6.1.sdk -miphoneos-version-min=5.1 -Wall -g -Os -Wall -fno-strict-aliasing -fvisibility=hidden -MT pixman-implementation.lo -MD -MP -MF .deps/pixman-implementation.Tpo -c pixman-implementation.c -fno-common -DPIC -o .libs/pixman-implementation.o<br> pixman-implementation.c:66:1: error: thread-local storage is unsupported for the current target<br>PIXMAN_DEFINE_THREAD_LOCAL (cache_t, fast_path_cache);<br>^<br>In file included from pixman-implementation.c:28:<br>In file included from ./pixman-private.h:34:<br> ./pixman-compiler.h:117:12: note: instantiated from:<br> static TLS type name<br> ^<br>In file included from pixman-implementation.c:25:<br>../config.h:117:13: note: instantiated from:<br>#define TLS __thread</i><br> <br></div><div>But I have been comparing the results with the old branch which was working and at configure it does:<br><br><i>checking for thread local storage (TLS) support... (cached) none<br>checking for pthread_setspecific... yes</i><br> <br></div><div>I believe it's a regression introduced in:<br><i><br>commit 85eff088acb80643ce78316a5ba4d8b354bac4d4<br>Author: Sebastian Dröge <<a href="mailto:slomo@circular-chaos.org">slomo@circular-chaos.org</a>><br> Date: Sat Sep 21 16:21:26 2013 +0200<br><br> pixman: Build with TLS support on iOS and Android<br> <br> It works now<br><br>diff --git a/config/ios.config b/config/ios.config<br>index 800b780..071704a 100644<br> --- a/config/ios.config<br>+++ b/config/ios.config<br>@@ -102,6 +102,3 @@ os.environ['ac_cv_func_posix_getpwuid_r'] = 'yes'<br> os.environ['ac_cv_func_posix_getgrgid_r'] = 'yes'<br> os.environ['gt_cv_c_wchar_t'] = 'no'<br> <br>-# For pixman<br>-os.environ['ac_cv_tls'] = 'none'<br>-</i><br><br></div><div>Reverting both commits restores the situation.<br><br></div><div>Andoni<br></div><div><br></div><div> <br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span><font color="#888888"> --<br> Sebastian Dröge <<a href="mailto:sebastian@centricular.com" target="_blank">sebastian@centricular.com</a>><br> Centricular Ltd - <a href="http://www.centricular.com" target="_blank">http://www.centricular.com</a><br> Expertise, Straight from the Source<br> </font></span><br>_______________________________________________<br> gstreamer-devel mailing list<br> <a href="mailto:gstreamer-devel@lists.freedesktop.org" target="_blank">gstreamer-devel@lists.freedesktop.org</a><br> <a href="http://lists.freedesktop.org/mailman/listinfo/gstreamer-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/gstreamer-devel</a><br> <br></blockquote></div><br><br clear="all"><br>-- <br>Andoni Morales Alastruey<br><br>LongoMatch:The Digital Coach<br><a href="http://www.longomatch.ylatuya.es" target="_blank">http://www.longomatch.ylatuya.es</a> </div></div>