Skip to contentSkip to conversation

package/cairo: uses fork(), so requires an MMU

PATCH New Yann E. MORIN 3 participants 5 messages May 14, 2014 – May 15, 2014

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 package/cairo: uses fork(), so requires an MMU

From: "Yann E. MORIN" <[email protected]>

Fixes:

    http://autobuild.buildroot.net/results/038/038371cc8548b3d39d8174289bedc9aae4076303/
---
 package/cairo/Config.in | 1 +
 1 file changed, 1 insertion(+)
package/cairo/Config.in +1 -0
index 0c87a29..2742855 100644
@@ -4,6 +4,7 @@ config BR2_PACKAGE_CAIRO
 	select BR2_PACKAGE_FONTCONFIG
 	select BR2_PACKAGE_XLIB_LIBX11 if BR2_PACKAGE_XORG7
 	select BR2_PACKAGE_XLIB_LIBXEXT if BR2_PACKAGE_XORG7
+	depends on BR2_USE_MMU # fork()
 	help
 	  Cairo is a 2D graphics library with support for multiple
 	  output devices. Currently supported output targets include
Signature · 1 line
1.8.3.2
#1 lore raw

PATCH 1/1 package/cairo: uses fork(), so requires an MMU

On 2014-05-14 23:56 +0200, Yann E. MORIN spake thusly:

Quoted 4 lines

From: "Yann E. MORIN" <[email protected]>

Fixes:

    http://autobuild.buildroot.net/results/038/038371cc8548b3d39d8174289bedc9aae4076303/

Also fixes:

    http://autobuild.buildroot.net/results/d93/d9374c1bd08eae5db60e6939ce5e9e4c8ff8cbd5/
    http://autobuild.buildroot.net/results/574/5746e7e3b2bc36f31dee1acf71e59dc4d6c90af2/
    http://autobuild.buildroot.net/results/b66/b66d7125ea7e10313221462097ae6c22ede963e5/
    [...]
Quoted 20 lines
---
 package/cairo/Config.in | 1 +
 1 file changed, 1 insertion(+)
package/cairo/Config.in +1 -0
index 0c87a29..2742855 100644
@@ -4,6 +4,7 @@ config BR2_PACKAGE_CAIRO
 	select BR2_PACKAGE_FONTCONFIG
 	select BR2_PACKAGE_XLIB_LIBX11 if BR2_PACKAGE_XORG7
 	select BR2_PACKAGE_XLIB_LIBXEXT if BR2_PACKAGE_XORG7
+	depends on BR2_USE_MMU # fork()
 	help
 	  Cairo is a 2D graphics library with support for multiple
 	  output devices. Currently supported output targets include
Signature · 1 line
1.8.3.2
Signature · 6 lines
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
#2 lore raw

PATCH 1/1 package/cairo: uses fork(), so requires an MMU

Hi!

Quoted 11 lines · Yann E. MORIN

On 2014-05-14 23:56 +0200, Yann E. MORIN spake thusly:

Quoted 4 lines

From: "Yann E. MORIN" <[email protected]>

Fixes:

    http://autobuild.buildroot.net/results/038/038371cc8548b3d39d8174289bedc9aae4076303/

Also fixes:

    http://autobuild.buildroot.net/results/d93/d9374c1bd08eae5db60e6939ce5e9e4c8ff8cbd5/
    http://autobuild.buildroot.net/results/574/5746e7e3b2bc36f31dee1acf71e59dc4d6c90af2/
    http://autobuild.buildroot.net/results/b66/b66d7125ea7e10313221462097ae6c22ede963e5/
    [...]

A few weeks ago, I posted a fix to cairo for these issues:

http://patchwork.ozlabs.org/patch/342715/

The problem occurs on one of the test program, not on cairo itself, so cairo should be usable on Blackfin (I do not own one, so I have not tested it).

I opened a bug at freedesktop.org too:

https://bugs.freedesktop.org/show_bug.cgi?id=78709

Best regards,
ELB

#3 lore raw

PATCH 1/1 package/cairo: uses fork(), so requires an MMU

Dear Eric Le Bihan,

Quoted 15 lines · Eric Le Bihan
Quoted 11 lines · Yann E. MORIN

On 2014-05-14 23:56 +0200, Yann E. MORIN spake thusly:

Quoted 4 lines

From: "Yann E. MORIN" <[email protected]>

Fixes:

    http://autobuild.buildroot.net/results/038/038371cc8548b3d39d8174289bedc9aae4076303/

Also fixes:

    http://autobuild.buildroot.net/results/d93/d9374c1bd08eae5db60e6939ce5e9e4c8ff8cbd5/
    http://autobuild.buildroot.net/results/574/5746e7e3b2bc36f31dee1acf71e59dc4d6c90af2/
    http://autobuild.buildroot.net/results/b66/b66d7125ea7e10313221462097ae6c22ede963e5/
    [...]

A few weeks ago, I posted a fix to cairo for these issues:

Quoted 8 lines

http://patchwork.ozlabs.org/patch/342715/

The problem occurs on one of the test program, not on cairo itself, so cairo should be usable on Blackfin (I do not own one, so I have not tested it).

I opened a bug at freedesktop.org too:

https://bugs.freedesktop.org/show_bug.cgi?id=78709

Yep, I agree Eric's patch is better. I've therefore marked Yann's patch as rejected (hope you won't mind Yann).

Thanks,

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
#4 lore raw

PATCH 1/1 package/cairo: uses fork(), so requires an MMU

Thomas, All,

On 2014-05-15 10:02 +0200, Thomas Petazzoni spake thusly:

Quoted 21 lines · Eric Le Bihan
Quoted 20 lines · Yann E. MORIN
Quoted 11 lines

On 2014-05-14 23:56 +0200, Yann E. MORIN spake thusly:

Quoted 4 lines

From: "Yann E. MORIN" <[email protected]>

Fixes:

    http://autobuild.buildroot.net/results/038/038371cc8548b3d39d8174289bedc9aae4076303/

Also fixes:

    http://autobuild.buildroot.net/results/d93/d9374c1bd08eae5db60e6939ce5e9e4c8ff8cbd5/
    http://autobuild.buildroot.net/results/574/5746e7e3b2bc36f31dee1acf71e59dc4d6c90af2/
    http://autobuild.buildroot.net/results/b66/b66d7125ea7e10313221462097ae6c22ede963e5/
    [...]

A few weeks ago, I posted a fix to cairo for these issues:

http://patchwork.ozlabs.org/patch/342715/

The problem occurs on one of the test program, not on cairo itself, so cairo should be usable on Blackfin (I do not own one, so I have not tested it).

Quoted 6 lines
Quoted 3 lines

I opened a bug at freedesktop.org too:

https://bugs.freedesktop.org/show_bug.cgi?id=78709

Yep, I agree Eric's patch is better. I've therefore marked Yann's patch as rejected (hope you won't mind Yann).

He, of course not! I even was doubtful of that patch to begin with anyway. I just forgot to mention it in a '---' note.

Regards,
Yann E. MORIN.

Signature · 6 lines
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'