[Buildroot] [PATCH v3 3/8] postgresql: enable bfin compile

[Buildroot] [PATCH v3 3/8] postgresql: enable bfin compile

[Peter Seiderer]

- overwrite platform support with linux template
- disable spinlock support

Signed-off-by: Peter Seiderer <ps.report@gmx.net>
---
Changes v1 --> v3:
  - no changes
---
 package/postgresql/postgresql.mk | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/package/postgresql/postgresql.mk b/package/postgresql/postgresql.mk
index 7ebf795..76b19d5 100644
--- a/package/postgresql/postgresql.mk
+++ b/package/postgresql/postgresql.mk
@@ -13,6 +13,11 @@ POSTGRESQL_LICENSE_FILES = COPYRIGHT
 POSTGRESQL_INSTALL_STAGING = YES
 POSTGRESQL_CONFIG_SCRIPTS = pg_config
 
+# unsupported platform, overwrite manually
+ifeq ($(BR2_bfin),y)
+POSTGRESQL_CONF_OPTS += --with-template=linux
+endif
+
 ifeq ($(BR2_TOOLCHAIN_USES_UCLIBC),y)
 # overwrite IPV6 detection
 ifeq ($(BR2_INET_IPV6),y)
@@ -31,7 +36,7 @@ ifneq ($(BR2_TOOLCHAIN_HAS_THREADS),y)
 	POSTGRESQL_CONF_OPTS += --disable-thread-safety
 endif
 
-ifeq ($(BR2_microblazeel)$(BR2_microblazebe)$(BR2_nios2),y)
+ifeq ($(BR2_bfin)$(BR2_microblazebe)$(BR2_microblazeel)$(BR2_nios2),y)
 	POSTGRESQL_CONF_OPTS += --disable-spinlocks
 endif
 
-- 
2.1.4

[Buildroot] [PATCH v3 3/8] postgresql: enable bfin compile

[Thomas Petazzoni]

Dear Peter Seiderer,

On Tue, 10 Mar 2015 20:59:39 +0100, Peter Seiderer wrote:

> +# unsupported platform, overwrite manually
> +ifeq ($(BR2_bfin),y)
> +POSTGRESQL_CONF_OPTS += --with-template=linux
> +endif

Do we really need to make this conditional? I believe
--with-template=linux is fine in all cases since Buildroot only targets
Linux.

So what about having --with-template=linux unconditionally in
POSTGRESQL_CONF_OPTS instead?

If that's OK for you, I can adjust this when applying.

Thanks,

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

[Buildroot] [PATCH v3 3/8] postgresql: enable bfin compile

[Peter Seiderer]

Hello Thomas,

On Sun, Mar 15, 2015 at 07:47:53PM +0100, Thomas Petazzoni wrote:
> Dear Peter Seiderer,
> 
> On Tue, 10 Mar 2015 20:59:39 +0100, Peter Seiderer wrote:
> 
> > +# unsupported platform, overwrite manually
> > +ifeq ($(BR2_bfin),y)
> > +POSTGRESQL_CONF_OPTS += --with-template=linux
> > +endif
> 
> Do we really need to make this conditional? I believe
> --with-template=linux is fine in all cases since Buildroot only targets
> Linux.
> 

Did take a look at postgresql-9.4.1/configure.in, the decision is made on
a variable called $host_os, but used for (what I would call) $target_os:

- for the bfin test case:
checking build system type... x86_64-unknown-linux-gnu
checking host system type... bfin-buildroot-uclinux-uclibc

- for a buildroot toolchain:
checking build system type... x86_64-unknown-linux-gnu
checking host system type... arm-buildroot-linux-uclibcgnueabi

> So what about having --with-template=linux unconditionally in
> POSTGRESQL_CONF_OPTS instead?
> 

Yes, I think this is save...

> If that's OK for you, I can adjust this when applying.
>

Yes, thats fine with me...

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

[Buildroot] [PATCH v3 3/8] postgresql: enable bfin compile

[Thomas Petazzoni]

Dear Peter Seiderer,

On Mon, 16 Mar 2015 20:40:03 +0100, Peter Seiderer wrote:

> Did take a look at postgresql-9.4.1/configure.in, the decision is made on
> a variable called $host_os, but used for (what I would call) $target_os:
> 
> - for the bfin test case:
> checking build system type... x86_64-unknown-linux-gnu
> checking host system type... bfin-buildroot-uclinux-uclibc
> 
> - for a buildroot toolchain:
> checking build system type... x86_64-unknown-linux-gnu
> checking host system type... arm-buildroot-linux-uclibcgnueabi

Yes, the "linux" OS is recognized, but not "uclinux", I saw that. But
it means that passing --with-template=linux unconditionally is fine.

> > If that's OK for you, I can adjust this when applying.
> 
> Yes, thats fine with me...

Ok, thanks.

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

[Buildroot] [PATCH v3 3/8] postgresql: enable bfin compile

[Thomas Petazzoni]

Dear Peter Seiderer,

On Tue, 10 Mar 2015 20:59:39 +0100, Peter Seiderer wrote:
> - overwrite platform support with linux template
> - disable spinlock support
> 
> Signed-off-by: Peter Seiderer <ps.report@gmx.net>

Applied, after making --with-template=linux used unconditionally, as we
discussed.

Thanks!

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

[Buildroot] [PATCH v3 3/8] postgresql: enable bfin compile

[Thomas Petazzoni]

Dear Peter Seiderer,

On Tue, 10 Mar 2015 20:59:39 +0100, Peter Seiderer wrote:
> - overwrite platform support with linux template
> - disable spinlock support
> 
> Signed-off-by: Peter Seiderer <ps.report@gmx.net>

postgresql uses fork(), which is not available on Blackfin:
http://autobuild.buildroot.org/results/c9c/c9c93156fe64e89f35627aa592c47d8c5181ed7d/build-end.log.

So what was the point of enabling postgresql on Blackfin ?

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

[Buildroot] [PATCH v3 3/8] postgresql: enable bfin compile

[Peter Seiderer]

Hello Thomas,

On Mon, Mar 16, 2015 at 11:07:25PM +0100, Thomas Petazzoni wrote:
> Dear Peter Seiderer,
> 
> On Tue, 10 Mar 2015 20:59:39 +0100, Peter Seiderer wrote:
> > - overwrite platform support with linux template
> > - disable spinlock support
> > 
> > Signed-off-by: Peter Seiderer <ps.report@gmx.net>
> 
> postgresql uses fork(), which is not available on Blackfin:
> http://autobuild.buildroot.org/results/c9c/c9c93156fe64e89f35627aa592c47d8c5181ed7d/build-end.log.
> 

Will provide a patch for postgresql to check for fork/BR2_USE_MMU, is there any special string
for the comment besides 'needs threads'?

> So what was the point of enabling postgresql on Blackfin ?

No special point, had the build environment still from [1] around...

In retrospect (in case bfin is disabled because of fork) the patch subject should
have been something different...

Regards,
Peter

[1] http://git.buildroot.net/buildroot/commit/?id=fc2e91d5bd32047055a611ed0d62f050bc90377c

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

[Buildroot] [PATCH v3 3/8] postgresql: enable bfin compile

[Thomas Petazzoni]

Dear Peter Seiderer,

On Tue, 17 Mar 2015 20:52:03 +0100, Peter Seiderer wrote:

> Will provide a patch for postgresql to check for fork/BR2_USE_MMU, is there any special string
> for the comment besides 'needs threads'?

No there is no comment about a BR2_USE_MMU dependency, because it's not
something the user can "fix" : if your architecture doesn't have a MMU,
it's like this, there's nothing you can do about it.

> No special point, had the build environment still from [1] around...
> 
> In retrospect (in case bfin is disabled because of fork) the patch subject should
> have been something different...

Well, the patch by itself is then completely useless, because Blackfin
is a noMMU architecture, so postgresql will never work on it.

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

[Buildroot] [PATCH v3 3/8] postgresql: enable bfin compile

[Peter Seiderer]

Hello Thomas,

On Tue, Mar 17, 2015 at 08:59:23PM +0100, Thomas Petazzoni wrote:
> Dear Peter Seiderer,
> 
> On Tue, 17 Mar 2015 20:52:03 +0100, Peter Seiderer wrote:
> 
> > Will provide a patch for postgresql to check for fork/BR2_USE_MMU, is there any special string
> > for the comment besides 'needs threads'?
> 
> No there is no comment about a BR2_USE_MMU dependency, because it's not
> something the user can "fix" : if your architecture doesn't have a MMU,
> it's like this, there's nothing you can do about it.
> 
> > No special point, had the build environment still from [1] around...
> > 
> > In retrospect (in case bfin is disabled because of fork) the patch subject should
> > have been something different...
> 
> Well, the patch by itself is then completely useless, because Blackfin
> is a noMMU architecture, so postgresql will never work on it.
>

You are right (did not know that the 'uclinux' in 'bfin-buildroot-uclinux-uclibc' stands
for microcontroller Linux/noMMU)...

The only minor purpose leftover is as gcc ICE exampel for 'bfin-uclinux-gcc (ADI-2014R1-RC2) 4.3.5' (see [1])...

Revert the patch or keep it?

Regards,
Peter

[1] http://lists.busybox.net/pipermail/buildroot/2015-March/121094.html

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

[Buildroot] [PATCH v3 3/8] postgresql: enable bfin compile

[Thomas Petazzoni]

Dear Peter Seiderer,

On Tue, 17 Mar 2015 21:26:02 +0100, Peter Seiderer wrote:

> The only minor purpose leftover is as gcc ICE exampel for 'bfin-uclinux-gcc (ADI-2014R1-RC2) 4.3.5' (see [1])...

I don't understand, why do you say it's the "only minor purpose
leftover" ? 

If PostgreSQL uses fork(), it *cannot* work on noMMU architecture, so
there is no way to build PostgreSQL on Blackfin.

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

[Buildroot] [PATCH v3 3/8] postgresql: enable bfin compile

[Peter Seiderer]

Hello Thomas,

On Tue, Mar 17, 2015 at 09:29:39PM +0100, Thomas Petazzoni wrote:
> Dear Peter Seiderer,
> 
> On Tue, 17 Mar 2015 21:26:02 +0100, Peter Seiderer wrote:
> 
> > The only minor purpose leftover is as gcc ICE exampel for 'bfin-uclinux-gcc (ADI-2014R1-RC2) 4.3.5' (see [1])...
> 
> I don't understand, why do you say it's the "only minor purpose
> leftover" ? 
> 
> If PostgreSQL uses fork(), it *cannot* work on noMMU architecture, so
> there is no way to build PostgreSQL on Blackfin.
>

...should have added 'until Blackfin compile is disabled because of noMMU' as a footnote ;-)

But the remaining question: Revert the patch? Should I send a revert-patch?
Or can you revert/commit directly? Or leave as is?

Regards,
Peter

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

[Buildroot] [PATCH v3 3/8] postgresql: enable bfin compile

[Thomas Petazzoni]

Dear Peter Seiderer,

On Tue, 17 Mar 2015 21:45:48 +0100, Peter Seiderer wrote:

> ...should have added 'until Blackfin compile is disabled because of noMMU' as a footnote ;-)
> 
> But the remaining question: Revert the patch? Should I send a revert-patch?
> Or can you revert/commit directly? Or leave as is?

Well, please send a patch that adds the BR2_USE_MMU dependency, and as
part of it, removes the attempts at supporting Blackfin.

Thanks a lot!

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