Re: [ptxdist] [BUG] commit 252cf4f1a0d239feb199f8dd89db9917a8840274 breaks udev

2011-08-31 Thread George McCollister

On 08/30/2011 03:40 PM, Marc Kleine-Budde wrote:

Moin,


20:11  georgem  commit 252cf4f1a0d239feb199f8dd89db9917a8840274
breaks udev for me. it errors on scsi.h.

Which toolchain and kernel-headers versions are you using?

cheers, Marc


The same problem happens on my i686 and armeb targets.

I'm using OSELAS.Toolchain-2011.03.0

i686-unknown-linux-gnu_gcc-4.5.2_glibc-2.13_binutils-2.21_kernel-2.6.33.7-sanitized
armeb-xscale-linux-gnueabi_gcc-4.5.2_glibc-2.13_binutils-2.21_kernel-2.6.33.7-sanitized

PTXCONF_KERNEL_HEADER_VERSION=2.6.33.7

I looked briefly and compared the version of scsi.h installed by 
kernel-header with the version in the toolchain. They looked quite 
different. The one in the toolchain says This file is a part of the GNU 
C Library. on the first line.


I was in the middle of getting a few other things working when I ran 
into this, so I reverted the change on my side to get passed the 
problem. If you need more information and want me to reapply the change 
and produce the problem again, let me know.


Regards,
George

--
ptxdist mailing list
ptxdist@pengutronix.de


Re: [ptxdist] [BUG] commit 252cf4f1a0d239feb199f8dd89db9917a8840274 breaks udev

2011-08-31 Thread George McCollister

On 08/31/2011 09:07 AM, Michael Olbrich wrote:

Why is PTXCONF_KERNEL_HEADER enabled at all? The package is mostly used
because the kernel header in the toolchain are too old. It seems this
version is too old. Newer kernel (2.6.39 here) don't install scsi.h.
I don't know which version changed this.
I usually use KERNEL_HEADER_VERSION == KERNEL_VERSION.

Michael

BUSYBOX and IPTABLES select KERNEL_HEADER. I recently upgraded my kernel 
from 2.6.33.7 to 2.6.33.9 but I haven't changed the kernel headers in 
the toolchain. I tried using 2.6.33.7 and 2.6.33.9 for 
KERNEL_HEADER_VERSION. I'm using a real-time kernel so upgrading isn't 
so easy. After we get this release out to customers I hope to start 
working with Linux 3.0+ real-time kernels. If you don't think this will 
be a problem for anyone else you can just leave it as is and I will put 
the old udev.make in my workspace/rules until we upgrade our kernel.


Regards,
George

--
ptxdist mailing list
ptxdist@pengutronix.de


Re: [ptxdist] [BUG] commit 252cf4f1a0d239feb199f8dd89db9917a8840274 breaks udev

2011-08-31 Thread Michael Olbrich
On Wed, Aug 31, 2011 at 09:28:05AM -0500, George McCollister wrote:
 On 08/31/2011 09:07 AM, Michael Olbrich wrote:
 Why is PTXCONF_KERNEL_HEADER enabled at all? The package is mostly used
 because the kernel header in the toolchain are too old. It seems this
 version is too old. Newer kernel (2.6.39 here) don't install scsi.h.
 I don't know which version changed this.
 I usually use KERNEL_HEADER_VERSION == KERNEL_VERSION.
 
 Michael
 
 BUSYBOX and IPTABLES select KERNEL_HEADER. I recently upgraded my
 kernel from 2.6.33.7 to 2.6.33.9 but I haven't changed the kernel
 headers in the toolchain. I tried using 2.6.33.7 and 2.6.33.9 for
 KERNEL_HEADER_VERSION. I'm using a real-time kernel so upgrading
 isn't so easy. After we get this release out to customers I hope to
 start working with Linux 3.0+ real-time kernels. If you don't think
 this will be a problem for anyone else you can just leave it as is
 and I will put the old udev.make in my workspace/rules until we
 upgrade our kernel.

You could also add this upstream kernel patch to the patch series.
Fixed in 2.6.35...

Regards,
Michael

commit 7407e5bba2cc821950344fd1391d9ad1b7e0b397
Author: Tom Rini tom_r...@mentor.com
Date:   Tue Apr 27 14:11:47 2010 -0700

[SCSI] Unexport scsi/scsi.h from headers_install

The scsi/scsi.h header is normally provided by the libc (and was not
exported by the kernel since 2.6.24) and has been until it was
re-exported with 2.6.31.  The kernel version is not userspace clean and
does not appear to provide anything useable in userland over the
(e)glibc version.

Signed-off-by: Tom Rini tom_r...@mentor.com
Signed-off-by: James Bottomley james.bottom...@suse.de

diff --git a/include/scsi/Kbuild b/include/scsi/Kbuild
index b3a0ee6..f2b9491 100644
--- a/include/scsi/Kbuild
+++ b/include/scsi/Kbuild
@@ -1,4 +1,3 @@
-header-y += scsi.h
 header-y += scsi_netlink.h
 header-y += scsi_netlink_fc.h
 header-y += scsi_bsg_fc.h

-- 
Pengutronix e.K.   | |
Industrial Linux Solutions | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0|
Amtsgericht Hildesheim, HRA 2686   | Fax:   +49-5121-206917- |

-- 
ptxdist mailing list
ptxdist@pengutronix.de


Re: [ptxdist] [PATCH 1/2] Add host-bison package

2011-08-31 Thread Bernhard Walle
Hi,

thanks for the review. I'll post the updated patch.

Am 29.08.11 12:04, schrieb Michael Olbrich:
 --- /dev/null
  +++ b/rules/pre/bison.make
  @@ -0,0 +1,5 @@
  +# -*-makefile-*-
  +
  +ifdef PTXCONF_HOST_BISON
  +export BISON_PKGDATADIR=$(PTXCONF_SYSROOT_HOST)/share/bison
  +endif
 What is this for?

Without this, bison doesn't find its runtime files
($(PTXCONF_SYSROOT_HOST)/share/bison/m4sugar/m4sugar.m4). It uses PREFIX
(DATADIR) which is /usr/share.

I tried to set --prefix to $(PTXCONF_SYSROOT_HOST) and DESTDIR to '',
but it didn't work. Don't remember the cause.

Setting the environment variable was the easiest solution without
patching the source code, but if you don't accept that, I can
investigate the isse again and find a better solution. But give me some
days as I'm quite busy...


Regards,
Bernhard

-- 
ptxdist mailing list
ptxdist@pengutronix.de


[ptxdist] [PATCH 1/2] Add host-bison package

2011-08-31 Thread Bernhard Walle
From: Bernhard Walle wa...@corscience.de

Since we have host-flex, it makes also sense to get independent of
the bison installation of the host.

Signed-off-by: Bernhard Walle wa...@corscience.de
---
 rules/host-bison.in   |8 
 rules/host-bison.make |   32 
 rules/pre/bison.make  |5 +
 3 files changed, 45 insertions(+), 0 deletions(-)
 create mode 100644 rules/host-bison.in
 create mode 100644 rules/host-bison.make
 create mode 100644 rules/pre/bison.make

diff --git a/rules/host-bison.in b/rules/host-bison.in
new file mode 100644
index 000..41bb930
--- /dev/null
+++ b/rules/host-bison.in
@@ -0,0 +1,8 @@
+## SECTION=hosttools_noprompt
+
+config HOST_BISON
+   tristate
+   help
+ The parser generator (replaces yacc).
+
+# vim: syntax=kconfig
diff --git a/rules/host-bison.make b/rules/host-bison.make
new file mode 100644
index 000..d62b0b4
--- /dev/null
+++ b/rules/host-bison.make
@@ -0,0 +1,32 @@
+# -*-makefile-*-
+#
+# Copyright (C) 2011 by Bernhard Walle wa...@corscience.de
+#
+# See CREDITS for details about who has contributed to this project.
+#
+# For further information about the PTXdist project and license conditions
+# see the README file.
+#
+
+#
+# We provide this package
+#
+HOST_PACKAGES-$(PTXCONF_HOST_BISON) += host-bison
+
+#
+# Paths and names
+#
+HOST_BISON_VERSION := 2.5
+HOST_BISON_MD5 := 9dba20116b13fc61a0846b0058fbe004
+HOST_BISON := bison-$(HOST_BISON_VERSION)
+HOST_BISON_SUFFIX  := tar.bz2
+HOST_BISON_URL := 
http://ftp.gnu.org/gnu/bison/$(HOST_BISON).$(HOST_BISON_SUFFIX)
+HOST_BISON_SOURCE  := $(SRCDIR)/$(HOST_BISON).$(HOST_BISON_SUFFIX)
+HOST_BISON_DIR := $(HOST_BUILDDIR)/$(HOST_BISON)
+
+#
+# autoconf
+#
+HOST_BISON_CONF_TOOL   := autoconf
+
+# vim: syntax=make
diff --git a/rules/pre/bison.make b/rules/pre/bison.make
new file mode 100644
index 000..cfbee38
--- /dev/null
+++ b/rules/pre/bison.make
@@ -0,0 +1,5 @@
+# -*-makefile-*-
+
+ifdef PTXCONF_HOST_BISON
+export BISON_PKGDATADIR=$(PTXCONF_SYSROOT_HOST)/share/bison
+endif
-- 
1.7.6.1


-- 
ptxdist mailing list
ptxdist@pengutronix.de