<div dir="ltr"><div>r183098 should fix this.</div><div><br></div><div>Do you run both 32-bit and 64-bit tests? I.e., is 2304 still correct for 64 bit?</div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Sat, Jun 1, 2013 at 3:10 AM, Greg Fitzgerald <span dir="ltr"><<a href="mailto:garious@gmail.com" target="_blank">garious@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">> const uptr kThreadDescriptorSize = FIRST_32_SECOND_64(1168, 2304);<br>
<br>
</div>Yes, that change causes all tests to pass.<br>
<div class="im"><br>
<br>
> I need to figure out which ifdefs to put this under, so I might not be able to land the fix until Monday.<br>
<br>
</div>Okay, no worries, thanks for doing this.  I've moved over to<br>
release_33 for the short-term.  With the one change mentioned earlier<br>
(#include <stdint.h>), asan is working well for me when targeting<br>
either X86 Linux or ARM Android.<br>
<span class="HOEnZb"><font color="#888888"><br>
-Greg<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Fri, May 31, 2013 at 11:50 AM, Sergey Matveev <<a href="mailto:earthdok@google.com">earthdok@google.com</a>> wrote:<br>
> As a temporary fix, you can replace this line in sanitizer_linux_libcdep.c:<br>
><br>
> const uptr kThreadDescriptorSize = FIRST_32_SECOND_64(1216, 2304);<br>
><br>
> with<br>
><br>
> const uptr kThreadDescriptorSize = FIRST_32_SECOND_64(1168, 2304);<br>
><br>
> The tests should pass after that. I need to figure out which ifdefs to put<br>
> this under, so I might not be able to land the fix until Monday.<br>
><br>
><br>
> On Fri, May 31, 2013 at 8:48 PM, Greg Fitzgerald <<a href="mailto:garious@gmail.com">garious@gmail.com</a>> wrote:<br>
>><br>
>> > What version of glibc are you building with on x86?<br>
>><br>
>> 2.11.1 for 64-bit x86 linux<br>
>><br>
>> $ ldd --version<br>
>> ldd (Ubuntu EGLIBC 2.11.1-0ubuntu7.8) 2.11.1<br>
>><br>
>> On Fri, May 31, 2013 at 8:24 AM, Sergey Matveev <<a href="mailto:earthdok@google.com">earthdok@google.com</a>><br>
>> wrote:<br>
>> > Those changes shouldn't affect ARM at all, since everything is under #if<br>
>> > defined(__i386__) || defined(__x86_64__).<br>
>> ><br>
>> > What version of glibc are you building with on x86?<br>
>> ><br>
>> ><br>
>> > On Fri, May 31, 2013 at 7:16 PM, Greg Fitzgerald <<a href="mailto:garious@gmail.com">garious@gmail.com</a>><br>
>> > wrote:<br>
>> >><br>
>> >> The failures happen on x86 Linux, Ubuntu Lucid.  On ARM Android, my<br>
>> >> example code segfaults, whereas before it worked.  I wasn't planning to<br>
>> >> debug that though since there are unit test failures on x86 Linux.<br>
>> >><br>
>> >> Greg<br>
>> >><br>
>> >> On May 31, 2013, at 2:37 AM, Sergey Matveev <<a href="mailto:earthdok@google.com">earthdok@google.com</a>><br>
>> >> wrote:<br>
>> >><br>
>> >> I'm confused - do those test failures happen on x86 Android?<br>
>> >><br>
>> >><br>
>> >> On Fri, May 31, 2013 at 10:48 AM, Sergey Matveev <<a href="mailto:earthdok@google.com">earthdok@google.com</a>><br>
>> >> wrote:<br>
>> >>><br>
>> >>> r182853 introduced an assumption about the size of thread descriptor<br>
>> >>> in<br>
>> >>> glibc. On your system it's 48 bytes off, which is why the tests fail.<br>
>> >>> What<br>
>> >>> version of libc are you using?<br>
>> >>><br>
>> >>> (Repeating - my previous message got stuck in the moderation queue<br>
>> >>> because the quoted text was too long.)<br>
>> >><br>
>> >><br>
>> ><br>
><br>
><br>
</div></div></blockquote></div><br></div>