On 06/09/12 09:48 +0400, Виноградов Василий wrote: > Hi! > > I propose the following patch for compiling libqb under Cygwin. Nice. > Though, I'm not sure about memory barriers. They are disabled by setting ac_cv_atomic_need_memory_barrier=no in configure.ac What compiler are you using? I assume gcc. If so I think you will still need ac_cv_atomic_need_memory_barrier. > because Cygwin doesn't have dl_phdr_info structure and so on. > Can anyone speak something about it? You probably need to turn off "ac_cv_link_attribute_section=no" to work around the missing dl_phdr_info. Regards Angus > _______________________________________________ > quarterback-devel mailing list > quarterback-devel(a)lists.fedorahosted.org > https://lists.fedorahosted.org/mailman/listinfo/quarterback-devel _______________________________________________ quarterback-devel mailing list quarterback-devel(a)lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/quarterback-devel

On 06/09/12 15:46 +0400, Виноградов Василий wrote: > I've made a mistake. > > Of course, dl_phdr_info structure doesn't depend on ac_cv_atomic_need_memory_barrier. > For this purpose I turn off "ac_cv_link_attribute_section=no" in the patch, indeed. > > Under Cygwin GCC doesn't' have built-in atomic operations (undefined references to __sync_synchronize(), > __sync_bool_compare_and_swap(), __sync_fetch_and_add()) even for IA. But we still need memory barriers. > For this purpose we need to turn on "ac_cv_atomic_need_memory_barrier=yes" or "nongcc_memory_barrier_needed=yes". > I like the last more. Moreover, it is already used in configure.ac. > > New patch is attached. Hi I can commit something for you but just remembered something from looking at the configure.ac file. Did you try the following at all? ( CFLAGS="-march=<your arch> -mtune=native" ./configure )

-Angus > 06.09.2012, 14:09, "Angus Salkeld" <asalkeld(a)redhat.com&gt;: >>  On 06/09/12 09:48 +0400, Виноградов Василий wrote: >>>  Hi! >>> >>>  I propose the following patch for compiling libqb under Cygwin. >>  Nice. >>>  Though, I'm not sure about memory barriers. They are disabled by setting ac_cv_atomic_need_memory_barrier=no in configure.ac >>  What compiler are you using? I assume gcc. If so I think you will still need ac_cv_atomic_need_memory_barrier. >>>  because Cygwin doesn't have dl_phdr_info structure and so on. >>>  Can anyone speak something about it? >>  You probably need to turn off "ac_cv_link_attribute_section=no" to work >>  around the missing dl_phdr_info. >> >>  Regards >>  Angus >>>  _______________________________________________ >>>  quarterback-devel mailing list >>>  quarterback-devel(a)lists.fedorahosted.org >>>  https://lists.fedorahosted.org/mailman/listinfo/quarterback-devel >>  _______________________________________________ >>  quarterback-devel mailing list >>  quarterback-devel(a)lists.fedorahosted.org >>  https://lists.fedorahosted.org/mailman/listinfo/quarterback-devel > _______________________________________________ > quarterback-devel mailing list > quarterback-devel(a)lists.fedorahosted.org > https://lists.fedorahosted.org/mailman/listinfo/quarterback-devel _______________________________________________ quarterback-devel mailing list quarterback-devel(a)lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/quarterback-devel

On 07/09/12 10:43 +0400, Виноградов Василий wrote: > 07.09.2012, 03:12, "Angus Salkeld" <asalkeld(a)redhat.com&gt;: >>  On 06/09/12 15:46 +0400, Виноградов Василий wrote: >>>  I've made a mistake. >>> >>>  Of course, dl_phdr_info structure doesn't depend on ac_cv_atomic_need_memory_barrier. >>>  For this purpose I turn off "ac_cv_link_attribute_section=no" in the patch, indeed. >>> >>>  Under Cygwin GCC doesn't' have built-in atomic operations (undefined references to __sync_synchronize(), >>>  __sync_bool_compare_and_swap(), __sync_fetch_and_add()) even for IA. But we still need memory barriers. >>>  For this purpose we need to turn on "ac_cv_atomic_need_memory_barrier=yes" or "nongcc_memory_barrier_needed=yes". >>>  I like the last more. Moreover, it is already used in configure.ac. >>> >>>  New patch is attached. >>  Hi >> >>  I can commit something for you but just remembered something from >>  looking at the configure.ac file. >> >>  Did you try the following at all? >> >>  ( CFLAGS="-march=<your arch> -mtune=native" ./configure ) > I've just tried to recompile. It doesn't work. Therefore we have to enable memory barriers explicitly in configure.ac. OK, well thanks for trying. Do you need any more changes?

-A >>  -Angus >>>  06.09.2012, 14:09, "Angus Salkeld" <asalkeld(a)redhat.com&gt;: >>>>   On 06/09/12 09:48 +0400, Виноградов Василий wrote: >>>>>   Hi! >>>>> >>>>>   I propose the following patch for compiling libqb under Cygwin. >>>>   Nice. >>>>>   Though, I'm not sure about memory barriers. They are disabled by setting ac_cv_atomic_need_memory_barrier=no in configure.ac >>>>   What compiler are you using? I assume gcc. If so I think you will still need ac_cv_atomic_need_memory_barrier. >>>>>   because Cygwin doesn't have dl_phdr_info structure and so on. >>>>>   Can anyone speak something about it? >>>>   You probably need to turn off "ac_cv_link_attribute_section=no" to work >>>>   around the missing dl_phdr_info. >>>> >>>>   Regards >>>>   Angus >>>>>   _______________________________________________ >>>>>   quarterback-devel mailing list >>>>>   quarterback-devel(a)lists.fedorahosted.org >>>>>   https://lists.fedorahosted.org/mailman/listinfo/quarterback-devel >>>>   _______________________________________________ >>>>   quarterback-devel mailing list >>>>   quarterback-devel(a)lists.fedorahosted.org >>>>   https://lists.fedorahosted.org/mailman/listinfo/quarterback-devel >>>  _______________________________________________ >>>  quarterback-devel mailing list >>>  quarterback-devel(a)lists.fedorahosted.org >>>  https://lists.fedorahosted.org/mailman/listinfo/quarterback-devel >>  _______________________________________________ >>  quarterback-devel mailing list >>  quarterback-devel(a)lists.fedorahosted.org >>  https://lists.fedorahosted.org/mailman/listinfo/quarterback-devel > _______________________________________________ > quarterback-devel mailing list > quarterback-devel(a)lists.fedorahosted.org > https://lists.fedorahosted.org/mailman/listinfo/quarterback-devel _______________________________________________ quarterback-devel mailing list quarterback-devel(a)lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/quarterback-devel