Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] glib 2.40 on xtensa
@ 2014-07-18 22:40 Thomas Petazzoni
  2014-07-19  0:28 ` Max Filippov
  0 siblings, 1 reply; 6+ messages in thread
From: Thomas Petazzoni @ 2014-07-18 22:40 UTC (permalink / raw)
  To: buildroot

Baruch, Max,

We have recently update glib to version 2.40, and this is causing
issues on Xtensa:

  http://autobuild.buildroot.org/results/a1a/a1a1f97a2c54ba4f7f7e44cc094a55bd23ca0aa0/build-end.log

  CC       gspawn.lo
gthread-posix.c: In function 'g_cond_impl_new':
gthread-posix.c:648:2: error: #error Cannot support GCond on your platform.
 #error Cannot support GCond on your platform.
  ^
gthread-posix.c: In function 'g_cond_wait_until':
gthread-posix.c:896:2: error: #error Cannot support GCond on your platform.
 #error Cannot support GCond on your platform.
  ^
gthread-posix.c:859:19: warning: unused variable 'ts' [-Wunused-variable]
   struct timespec ts;
                   ^

Could you have a look and see what can be done to fix this problem?

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Buildroot] glib 2.40 on xtensa
  2014-07-18 22:40 [Buildroot] glib 2.40 on xtensa Thomas Petazzoni
@ 2014-07-19  0:28 ` Max Filippov
  2014-07-19  9:33   ` Thomas Petazzoni
  0 siblings, 1 reply; 6+ messages in thread
From: Max Filippov @ 2014-07-19  0:28 UTC (permalink / raw)
  To: buildroot

On Sat, Jul 19, 2014 at 2:40 AM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Baruch, Max,
>
> We have recently update glib to version 2.40, and this is causing
> issues on Xtensa:
>
>   http://autobuild.buildroot.org/results/a1a/a1a1f97a2c54ba4f7f7e44cc094a55bd23ca0aa0/build-end.log
>
>   CC       gspawn.lo
> gthread-posix.c: In function 'g_cond_impl_new':
> gthread-posix.c:648:2: error: #error Cannot support GCond on your platform.
>  #error Cannot support GCond on your platform.
>   ^
> gthread-posix.c: In function 'g_cond_wait_until':
> gthread-posix.c:896:2: error: #error Cannot support GCond on your platform.
>  #error Cannot support GCond on your platform.
>   ^
> gthread-posix.c:859:19: warning: unused variable 'ts' [-Wunused-variable]
>    struct timespec ts;
>                    ^
>
> Could you have a look and see what can be done to fix this problem?

It requires pthread_cond_timedwait_relative_np (not available on uclibc)
or pthread_condattr_setclock (only available with NPTL). Looks like we
need NPTL to support this glib version.

-- 
Thanks.
-- Max

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Buildroot] glib 2.40 on xtensa
  2014-07-19  0:28 ` Max Filippov
@ 2014-07-19  9:33   ` Thomas Petazzoni
  2014-07-20  0:17     ` Max Filippov
  2014-07-20  0:20     ` Max Filippov
  0 siblings, 2 replies; 6+ messages in thread
From: Thomas Petazzoni @ 2014-07-19  9:33 UTC (permalink / raw)
  To: buildroot

Dear Max Filippov,

On Sat, 19 Jul 2014 04:28:11 +0400, Max Filippov wrote:
> On Sat, Jul 19, 2014 at 2:40 AM, Thomas Petazzoni
> <thomas.petazzoni@free-electrons.com> wrote:
> > Baruch, Max,
> >
> > We have recently update glib to version 2.40, and this is causing
> > issues on Xtensa:
> >
> >   http://autobuild.buildroot.org/results/a1a/a1a1f97a2c54ba4f7f7e44cc094a55bd23ca0aa0/build-end.log
> >
> >   CC       gspawn.lo
> > gthread-posix.c: In function 'g_cond_impl_new':
> > gthread-posix.c:648:2: error: #error Cannot support GCond on your platform.
> >  #error Cannot support GCond on your platform.
> >   ^
> > gthread-posix.c: In function 'g_cond_wait_until':
> > gthread-posix.c:896:2: error: #error Cannot support GCond on your platform.
> >  #error Cannot support GCond on your platform.
> >   ^
> > gthread-posix.c:859:19: warning: unused variable 'ts' [-Wunused-variable]
> >    struct timespec ts;
> >                    ^
> >
> > Could you have a look and see what can be done to fix this problem?
> 
> It requires pthread_cond_timedwait_relative_np (not available on uclibc)

Well, libglib builds fine on ARM/uClibc.

> or pthread_condattr_setclock (only available with NPTL). Looks like we
> need NPTL to support this glib version.

Argh. There's no NPTL support for:

 - ARC (but I believe it's coming)
 - AVR32 (but we're deprecating this architecture, so we don't care)
 - Blackfin (and I don't think there's much hope to have NPTL one day,
   but glib is anyway disabled on !MMU platforms, so we don't care)
 - Xtensa (don't know)
 - i386 (because NPTL requires at least i486)

Should we patch glib to make the support of the GCond stuff optional,
and hope not too many programs will use it? Or simply add the NPTL
dependency to glib, but that would mean a huge number of packages
suddenly become unavailable for some architectures.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Buildroot] glib 2.40 on xtensa
  2014-07-19  9:33   ` Thomas Petazzoni
@ 2014-07-20  0:17     ` Max Filippov
  2014-07-20  8:26       ` Thomas Petazzoni
  2014-07-20  0:20     ` Max Filippov
  1 sibling, 1 reply; 6+ messages in thread
From: Max Filippov @ 2014-07-20  0:17 UTC (permalink / raw)
  To: buildroot

Hi Thomas,

On Sat, Jul 19, 2014 at 1:33 PM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Dear Max Filippov,
>
> On Sat, 19 Jul 2014 04:28:11 +0400, Max Filippov wrote:
>> On Sat, Jul 19, 2014 at 2:40 AM, Thomas Petazzoni
>> <thomas.petazzoni@free-electrons.com> wrote:
>> > Baruch, Max,
>> >
>> > We have recently update glib to version 2.40, and this is causing
>> > issues on Xtensa:
>> >
>> >   http://autobuild.buildroot.org/results/a1a/a1a1f97a2c54ba4f7f7e44cc094a55bd23ca0aa0/build-end.log
>> >
>> >   CC       gspawn.lo
>> > gthread-posix.c: In function 'g_cond_impl_new':
>> > gthread-posix.c:648:2: error: #error Cannot support GCond on your platform.
>> >  #error Cannot support GCond on your platform.
>> >   ^
>> > gthread-posix.c: In function 'g_cond_wait_until':
>> > gthread-posix.c:896:2: error: #error Cannot support GCond on your platform.
>> >  #error Cannot support GCond on your platform.
>> >   ^
>> > gthread-posix.c:859:19: warning: unused variable 'ts' [-Wunused-variable]
>> >    struct timespec ts;
>> >                    ^
>> >
>> > Could you have a look and see what can be done to fix this problem?
>>
>> It requires pthread_cond_timedwait_relative_np (not available on uclibc)
>
> Well, libglib builds fine on ARM/uClibc.
>
>> or pthread_condattr_setclock (only available with NPTL). Looks like we
>> need NPTL to support this glib version.
>
> Argh. There's no NPTL support for:
>
>  - ARC (but I believe it's coming)
>  - AVR32 (but we're deprecating this architecture, so we don't care)
>  - Blackfin (and I don't think there's much hope to have NPTL one day,
>    but glib is anyway disabled on !MMU platforms, so we don't care)
>  - Xtensa (don't know)
>  - i386 (because NPTL requires at least i486)
>
> Should we patch glib to make the support of the GCond stuff optional,
> and hope not too many programs will use it? Or simply add the NPTL
> dependency to glib, but that would mean a huge number of packages
> suddenly become unavailable for some architectures.

There's a patch in glib mainline that implements GMutex natively on linux.
It applies cleanly to glib-2.40 and with it xtensa builds successfully.
I can post a buildroot patch with it.

-- 
Thanks.
-- Max

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Buildroot] glib 2.40 on xtensa
  2014-07-19  9:33   ` Thomas Petazzoni
  2014-07-20  0:17     ` Max Filippov
@ 2014-07-20  0:20     ` Max Filippov
  1 sibling, 0 replies; 6+ messages in thread
From: Max Filippov @ 2014-07-20  0:20 UTC (permalink / raw)
  To: buildroot

On Sat, Jul 19, 2014 at 1:33 PM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Argh. There's no NPTL support for:
>
>  - ARC (but I believe it's coming)
>  - AVR32 (but we're deprecating this architecture, so we don't care)
>  - Blackfin (and I don't think there's much hope to have NPTL one day,
>    but glib is anyway disabled on !MMU platforms, so we don't care)
>  - Xtensa (don't know)

We're planning to complete NPTL implementation this year.

-- 
Thanks.
-- Max

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Buildroot] glib 2.40 on xtensa
  2014-07-20  0:17     ` Max Filippov
@ 2014-07-20  8:26       ` Thomas Petazzoni
  0 siblings, 0 replies; 6+ messages in thread
From: Thomas Petazzoni @ 2014-07-20  8:26 UTC (permalink / raw)
  To: buildroot

Dear Max Filippov,

On Sun, 20 Jul 2014 04:17:53 +0400, Max Filippov wrote:

> > Argh. There's no NPTL support for:
> >
> >  - ARC (but I believe it's coming)
> >  - AVR32 (but we're deprecating this architecture, so we don't care)
> >  - Blackfin (and I don't think there's much hope to have NPTL one day,
> >    but glib is anyway disabled on !MMU platforms, so we don't care)
> >  - Xtensa (don't know)
> >  - i386 (because NPTL requires at least i486)
> >
> > Should we patch glib to make the support of the GCond stuff optional,
> > and hope not too many programs will use it? Or simply add the NPTL
> > dependency to glib, but that would mean a huge number of packages
> > suddenly become unavailable for some architectures.
> 
> There's a patch in glib mainline that implements GMutex natively on linux.
> It applies cleanly to glib-2.40 and with it xtensa builds successfully.
> I can post a buildroot patch with it.

Ah, thanks, sounds good!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2014-07-20  8:26 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-07-18 22:40 [Buildroot] glib 2.40 on xtensa Thomas Petazzoni
2014-07-19  0:28 ` Max Filippov
2014-07-19  9:33   ` Thomas Petazzoni
2014-07-20  0:17     ` Max Filippov
2014-07-20  8:26       ` Thomas Petazzoni
2014-07-20  0:20     ` Max Filippov

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox