busybox neue version

[rhabarber1848]

Bevor eine neue Busybox-Version ins CVS kommt, sollte erstmal
dieser uralte Enigma-Bug gefixt werden, oder?
http://forum.tuxbox-cvs.sourceforge.net ... 54#p345854

[amiga23]

Hier zumindest mal ein diff für den diff der 1.12.1 Version von rhabarber1848

patch_actual-cvs_busybox-1.12.1.diff

[amiga23]

Oh, war ein Fehler drin jetzt sollte es gehen

patch_actual-cvs_busybox-1.12.1.diff

[dwilx]

Wieso machst du dafür einen Patch, wenn das doch mit

Code: Alles auswählen

--with-busybox-conf=[pfad zu deiner bb-m4config]

möglich wäre. Das geht doch jetzt so zu konfigurieren.

Bevor eine neue Busybox-Version ins CVS kommt, sollte erstmal
dieser uralte Enigma-Bug gefixt werden, oder?
http://forum.tuxbox-cvs.sourceforge.net ... 54#p345854

Weis denn jemand woher der Fehler kommt? Ist der überhaupt in der BB oder liegts anderswo?

[seife]

Die Syntax vom smb mount Befehl hat sich wohl geändert, was dazu führt, dass enigma kein samba mehr mounten kann.
Sollte wirklich trivial zu fixen sein, aber da ich keine Samba-Exports habe konnte ich es nicht machen.
Aber irgendeiner von denen, die immer die neueste Busybox verwenden wollen, sollte doch ein Samba-Setup haben, so dass er das auch mal schnell fixen könnte.

Wäre jedenfalls fair den eniga-Usern gegenüber. Und nein, enigma-Sourcecode beisst nicht

[dbt]

Busybox Update 1.13.2:

config und busybox.diff sind für das aktuelle CVS angepasst. Bitte testen!

busybox_1.13.2-diff-2009-02-25-10-32-40.patch

[mb405]

ah danke mal testen. die 1.13er reihe ging bei mir nie. die 1.12.4 geht ohne probleme hier

[mb405]

also genau das selbe wie ich schon vorher hatte

Code: Alles auswählen

init started: BusyBox v1.13.2 (2009-02-25 15:40:50 CET)
mount: using /etc/fstab                                                                                                                                     

und
insmod blabla
geht auch nicht mehr. ka warum
sobald ich den kompletten pfad
insmod /lib/modules/2.4.36-dbox2/misc/blabla.o
angebe gehts.

[dwilx]

Vermutlich weil in der neuen Version das nicht mehr drin ist:

Code: Alles auswählen

diff -Naur busybox-1.7.2/modutils/insmod.c busybox-1.7.2-tuxbox/modutils/insmod.c
--- busybox-1.7.2/modutils/insmod.c	2007-09-03 13:48:35.000000000 +0200
+++ busybox-1.7.2-tuxbox/modutils/insmod.c	2007-10-27 22:35:44.000000000 +0200
@@ -3959,6 +3959,7 @@
 	int len;
 	int k_crcs;
 	char *tmp, *tmp1;
+	char *tmp2;
 	unsigned long m_size;
 	ElfW(Addr) m_addr;
 	struct obj_file *f;
@@ -4031,6 +4032,14 @@
 	}
 
 	/* Get a filedesc for the module.  Check we we have a complete path */
+	/* first look in /var/lib/modules */
+	tmp2 = alloca(strlen(m_fullName) + sizeof "/var/lib/modules//");
+	strcpy(tmp2, "/var/lib/modules/");
+	strcat(tmp2, m_fullName);
+	if (stat(tmp2, &st) >= 0 && S_ISREG(st.st_mode) && (fp = fopen(tmp2, "r")) != NULL) {
+		m_filename = xstrdup(tmp2);
+		printf("insmod: preferring module %s\n", m_filename);
+	} else
 	if (stat(arg1, &st) < 0 || !S_ISREG(st.st_mode)
 	 || (fp = fopen(arg1, "r")) == NULL
 	) {

Sieht aus, als wäre das wegrationalisiert worden

[JtG-Riker]

Anbei mal mein diff

busbox-1.13.2.diff


Versucht ma bitte damit.

Gruß Riker

[mb405]

der insmod.c patch hats gebracht

[dbt]

So, noch mal nachgelegt. also booten tut's, wegen Kernel 2.6 müsst man nochmal schauen:
busybox_1.13.2-diff-2009-02-27-21-11-21.patch

[jojo]

Beim 2.6er muß die Kernel-Config noch angepasst werden

Code: Alles auswählen

* Linux Module Utilities
*
Default directory containing modules (DEFAULT_MODULES_DIR) [/lib/modules] /lib/modules
Default name of modules.dep (DEFAULT_DEPMOD_FILE) [modules.dep] modules.dep
Simplified modutils (MODPROBE_SMALL) [Y/n/?] y
  Accept module options on modprobe command line (FEATURE_MODPROBE_SMALL_OPTIONS_ON_CMDLINE) [N/y/?] (NEW) Y
  Skip loading of already loaded modules (FEATURE_MODPROBE_SMALL_CHECK_ALREADY_LOADED) [N/y/?] (NEW) Y
*
* Options common to multiple modutils
*

Oder kommt dies etwa nicht über den busybox-patch herein?

[dbt]

Ganz sauber ist das auf jeden Fall noch nicht. Wegen des 2.6er müsste das mal jemand versuchen, weil ich das gerade nicht mit dem 2.6er aufgesetzt habe.
Jetzt habe ich aber das Problem, das insmod beim bauen der rcS irgendwie nicht übergeben wird

Code: Alles auswählen

...
modprobe: applet not found
/proc/bus/tuxbox/vendor: No such file or directory
/proc/bus/tuxbox/vendor: No such file or directory
/proc/bus/tuxbox/model: No such file or directory
/proc/bus/tuxbox/model: No such file or directory
/proc/bus/tuxbox/submodel: No such file or directory
/proc/bus/tuxbox/submodel: No such file or directory
Detected STB:
        Vendor: Unknown
        Model: Unknown Unknown (2 flash chips)
modprobe: applet not found
modprobe: applet not found
modprobe: applet not found
modprobe: applet not found
modprobe: applet not found
modprobe: applet not found
modprobe: applet not found
modprobe: applet not found
modprobe: applet not found
modprobe: applet not found
modprobe: applet not found
modprobe: applet not found
modprobe: applet not found
modprobe: applet not found
modprobe: applet not found
modprobe: applet not found
modprobe: applet not found
modprobe: applet not found
modprobe: applet not found
modprobe: applet not found
ln: /dev/dvb/adapter0/demux1: No such file or directory
ln: /dev/dvb/adapter0/dvr1: No such file or directory
mkdir: cannot create directory '/var/log': File exists
Nov  2 21:54:10 dbox syslog.info syslogd started: BusyBox v1.13.2
Loading /share/keymaps/i386/qwertz/de-latin1-nodeadkeys.kmap.gz
Starting the automounter
...

[jojo]

Ich hab' mal busybox.config.m4 geändert (siehe übernächstes Posting):
<Unfug gelöscht>

Die Werte für y/n habe ich der defaultconfig entnommen - machen für mich Sinn.
Bezüglich der `' Orgien kann ich nichts sagen außer m4 is evil

[jojo]

<Unfug gelöscht>

[jojo]

argh...
Aber jetzt...

Code: Alles auswählen

--- ./cdk/Patches/busybox.config.m4.orig	2009-02-28 08:57:38.472005541 +0100
+++ ./cdk/Patches/busybox.config.m4	2009-02-28 15:51:50.776018692 +0100
@@ -440,7 +440,9 @@
 #
 ifdef(`kernel26',
 `option(`CONFIG_MODPROBE', `y', `y')
-option(`CONFIG_MODPROBE_SMALL', `y', `y')',
+option(`CONFIG_MODPROBE_SMALL', `y', `y')'
+option(`FEATURE_MODPROBE_SMALL_OPTIONS_ON_CMDLINE', `y', `y')
+option(`FEATURE_MODPROBE_SMALL_CHECK_ALREADY_LOADED', `y',`y'),
 `option(`CONFIG_MODPROBE', `n', `n')
 option(`CONFIG_MODPROBE_SMALL', `n', `n')'
 )

[Houdini]

wohl eher:

Code: Alles auswählen

--- ./cdk/Patches/busybox.config.m4.orig   2009-02-28 08:57:38.472005541 +0100
+++ ./cdk/Patches/busybox.config.m4   2009-02-28 15:51:50.776018692 +0100
@@ -440,7 +440,9 @@
#
ifdef(`kernel26',
`option(`CONFIG_MODPROBE', `y', `y')
-option(`CONFIG_MODPROBE_SMALL', `y', `y'),
+option(`CONFIG_MODPROBE_SMALL', `y', `y')
+option(`FEATURE_MODPROBE_SMALL_OPTIONS_ON_CMDLINE', `y', `y')
+option(`FEATURE_MODPROBE_SMALL_CHECK_ALREADY_LOADED', `y',`y')',
`option(`CONFIG_MODPROBE', `n', `n')
option(`CONFIG_MODPROBE_SMALL', `n', `n')'
)

[jojo]

wohl eher:
<stuff deleted>

Vielen Dank - hatte ich einfach nicht mehr gesehen...

Allerdings hatte ich bei den Options den Präfix vergessen. Wie sollten denn die Werte für SMALL_OPTIONS_ON_CMDLINE bzw SMALL_CHECK_ALREADY_LOADED aussehen?
Ist 'y' in beiden Fällen korrekt?

Code: Alles auswählen

--- ./cdk/Patches/busybox.config.m4.orig   2009-02-28 08:57:38.472005541 +0100
+++ ./cdk/Patches/busybox.config.m4   2009-02-28 15:51:50.776018692 +0100
@@ -440,7 +440,9 @@
#
ifdef(`kernel26',
`option(`CONFIG_MODPROBE', `y', `y')
-option(`CONFIG_MODPROBE_SMALL', `y', `y'),
+option(`CONFIG_MODPROBE_SMALL', `y', `y')
+option(`CONFIG_FEATURE_MODPROBE_SMALL_OPTIONS_ON_CMDLINE', `y', `y')
+option(`CONFIG_FEATURE_MODPROBE_SMALL_CHECK_ALREADY_LOADED', `y',`y')',
`option(`CONFIG_MODPROBE', `n', `n')
option(`CONFIG_MODPROBE_SMALL', `n', `n')'
)

[jojo]

Mit der neuen Busybox gibt es doch einige kleine Probleme:

a) beim Laden der Treiber werden "Unknown symbol" in rauher Menge gemeldet, allerdings laufen die Treiber problemlos...

Code: Alles auswählen

[   20.648794] dboxide: Unknown symbol SELECT_DRIVE
[   20.704289] dboxide: Unknown symbol ide_device_add
[   20.759446] dboxide: Unknown symbol ide_unregister
[   20.817733] dboxide: Unknown symbol ide_hwifs
[   21.703093] Uniform Multi-Platform E-IDE driver
[   21.775595] ide: Assuming 50MHz system bus speed for PIO modes; override with idebus=xx
[   21.955329] dboxide: $Id: main.c,v 1.4.2.7 2009/02/12 12:41:51 rhabarber1848 Exp $
[   22.047132] dboxide: activating cs2
[   22.085558] dboxide: address space of DBox2 IDE CPLD is at: 0xc3050000
[   22.163815] dboxide: IDE-Interface detected, e000e000, Vendor: Unknown
[   22.894025] dboxide: sreset succeeded
[   22.974346] Probing IDE interface DBox2 ...
[   23.274143] hda: ST9160821AS, ATA DISK drive
[   24.059026] DBox2  at 0x010-0x017,0x04e on irq 47
[   24.487479] hda: max request size: 512KiB
[   26.494289] hda: 312581808 sectors (160041 MB) w/8192KiB Cache, CHS=19457/255/63                          
[   26.579614] hda: cache flushes supported
[   26.627670]  hda: hda1 hda2

Gleiches Problemchen bei dbox2_napi und nfs...

b) Der init-Prozess versucht virtuelle Konsolen auf vc2..vc6 zu öffnen, was jedoch bei den
Defaults vergebliche Liebesmüh' ist und in kontinuierlichen restarts mündet:

Code: Alles auswählen

starting pid 344, tty '': '/etc/init.d/start'

can't open /dev/vc/2: No such file or directory
can't open /dev/vc/3: No such file or directory
can't open /dev/vc/4: No such file or directory
can't open /dev/vc/5: No such file or directory
can't open /dev/vc/6: No such file or directory
Thu Jan  1 01:00:00 CET 1970
process '-/bin/sh' (pid 347) exited. Scheduling for restart.
process '-/bin/sh' (pid 348) exited. Scheduling for restart.
process '-/bin/sh' (pid 350) exited. Scheduling for restart.
process '-/bin/sh' (pid 352) exited. Scheduling for restart.
process '-/bin/sh' (pid 353) exited. Scheduling for restart.
can't open /dev/vc/2: No such file or directory
can't open /dev/vc/3: No such file or directory
can't open /dev/vc/4: No such file or directory
... etc...

Der Fall b) läßt sich zwar durch die inittab korrigieren, allerdings differiert das Verhalten deutlich zur alten busybox. Anscheinend wurde bei dieser geprüft, ob das entsprechende tty auch vorhanden ist, bevor ein Prozess darauf gesetzt wurde.

[dwilx]

Anbei mal mein diff

busbox-1.13.2.diff


Versucht ma bitte damit.

Gruß Riker

Da ist in dem insmod Teil auch noch ein Leak drin, ganz am Schluß wird fp nicht wieder freigegeben, wenn ich das richtig sehe. Da müsste galube ich noch ein close(fp) drangehängt werden:

Code: Alles auswählen

diff -ur busybox-1.13.2.orig/modutils/insmod.c busybox-1.13.2/modutils/insmod.c
--- busybox-1.13.2.orig/modutils/insmod.c	2008-11-09 18:28:03.000000000 +0100
+++ busybox-1.13.2/modutils/insmod.c	2009-01-26 21:39:32.000000000 +0100
@@ -10,11 +10,42 @@
 #include "libbb.h"
 #include "modutils.h"
 
+#include <sys/utsname.h>
+static char *m_filename;
+static char *m_fullName;
+
+static int FAST_FUNC check_module_name_match(const char *filename,
+		struct stat *statbuf UNUSED_PARAM,
+		void *userdata, int depth UNUSED_PARAM)
+{
+	char *fullname = (char *) userdata;
+	char *tmp;
+
+	if (fullname[0] == '\0')
+		return FALSE;
+
+	tmp = bb_get_last_path_component_nostrip(filename);
+	if (strcmp(tmp, fullname) == 0) {
+		/* Stop searching if we find a match */
+		m_filename = xstrdup(filename);
+		return FALSE;
+	}
+	return TRUE;
+}
+
 int insmod_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
 int insmod_main(int argc UNUSED_PARAM, char **argv)
 {
 	char *filename;
 	int rc;
+int len;
+char *tmp;
+char *tmp2;
+int k_version = 0;
+struct stat st;
+struct utsname uts;
+char *m_name;
+FILE *fp;
 
 	USE_FEATURE_2_4_MODULES(
 		getopt32(argv, INSMOD_OPTS INSMOD_ARGS);
@@ -25,9 +56,107 @@
 	if (!filename)
 		bb_show_usage();
 
-	rc = bb_init_module(filename, parse_cmdline_module_options(argv));
+	/* Grab the module name */
+	tmp = xstrdup(filename);
+	len = strlen(filename);
+
+	if (uname(&uts) == 0) {
+		if (uts.release[0] == '2') {
+			k_version = uts.release[2] - '0';
+		}
+	}
+
+#if ENABLE_FEATURE_2_6_MODULES
+	if (k_version > 4 && len > 3 && tmp[len - 3] == '.'
+	 && tmp[len - 2] == 'k' && tmp[len - 1] == 'o'
+	) {
+		len -= 3;
+		tmp[len] = '\0';
+	} else
+#endif
+		if (len > 2 && tmp[len - 2] == '.' && tmp[len - 1] == 'o') {
+			len -= 2;
+			tmp[len] = '\0';
+		}
+
+
+#if ENABLE_FEATURE_2_6_MODULES
+	if (k_version > 4)
+		m_fullName = xasprintf("%s.ko", tmp);
+	else
+#endif
+		m_fullName = xasprintf("%s.o", tmp);
+
+	if (!m_name) {
+		m_name = tmp;
+	}
+	free(tmp);
+
+	/* first look in /var/lib/modules */
+	tmp2 = alloca(strlen(m_fullName) + sizeof "/var/lib/modules//");
+	strcpy(tmp2, "/var/lib/modules/");
+	strcat(tmp2, m_fullName);
+	if (stat(tmp2, &st) >= 0 && S_ISREG(st.st_mode) && (fp = fopen(tmp2, "r")) != NULL) {
+		m_filename = xstrdup(tmp2);
+		printf("insmod: preferring module %s\n", m_filename);
+	} else
+	/* Get a filedesc for the module.  Check that we have a complete path */
+	if (stat(filename, &st) < 0 || !S_ISREG(st.st_mode)
+	 || (fp = fopen_for_read(filename)) == NULL
+	) {
+		/* Hmm.  Could not open it.  First search under /lib/modules/`uname -r`,
+		 * but do not error out yet if we fail to find it... */
+		if (k_version) {	/* uname succeedd */
+			char *module_dir;
+			char *tmdn;
+
+			tmdn = concat_path_file(CONFIG_DEFAULT_MODULES_DIR, uts.release);
+			/* Jump through hoops in case /lib/modules/`uname -r`
+			 * is a symlink.  We do not want recursive_action to
+			 * follow symlinks, but we do want to follow the
+			 * /lib/modules/`uname -r` dir, So resolve it ourselves
+			 * if it is a link... */
+			module_dir = xmalloc_readlink(tmdn);
+			if (!module_dir)
+				module_dir = xstrdup(tmdn);
+			recursive_action(module_dir, ACTION_RECURSE,
+					check_module_name_match, NULL, m_fullName, 0);
+			free(module_dir);
+			free(tmdn);
+		}
+
+		/* Check if we have found anything yet */
+		if (!m_filename || ((fp = fopen_for_read(m_filename)) == NULL)) {
+			int r;
+			char *module_dir;
+
+			free(m_filename);
+			m_filename = NULL;
+			module_dir = xmalloc_readlink(CONFIG_DEFAULT_MODULES_DIR);
+			if (!module_dir)
+				module_dir = xstrdup(CONFIG_DEFAULT_MODULES_DIR);
+			/* No module found under /lib/modules/`uname -r`, this
+			 * time cast the net a bit wider.  Search /lib/modules/ */
+			r = recursive_action(module_dir, ACTION_RECURSE,
+					check_module_name_match, NULL, m_fullName, 0);
+			if (r)
+				bb_error_msg_and_die("%s: module not found", m_fullName);
+			free(module_dir);
+			if (m_filename == NULL
+			 || ((fp = fopen_for_read(m_filename)) == NULL)
+			) {
+				bb_error_msg_and_die("%s: module not found", m_fullName);
+			}
+		}
+	} else 
+		m_filename = xstrdup(filename);
+
+
+
+
+	rc = bb_init_module(m_filename, parse_cmdline_module_options(argv));
 	if (rc)
-		bb_error_msg("cannot insert '%s': %s", filename, moderror(rc));
+		bb_error_msg("cannot insert '%s': %s", m_filename, moderror(rc));

 	return rc;
 }

[seife]

Das ist bei busybox egal (und IIRC teilweise Absicht), dass da die FDs nicht explizit geschlossen werden und der Speicher nicht explizit freigegeben wird - exit() bzw. der Kernle macht das automatisch, also muss man keinen Platz im Binary dafür verschwenden
Gilt natürlich nur für die Programme, die nicht als daemon länger laufen.

[rhabarber1848]

Busybox 1.13.3 wurde veröffentlicht, enthält einen Großteil
der hier besprochenen Patches und läuft im Yadd.

Hier mein Patch: busybox13.diff

Den wget-long_options-Patch, der vor kurzem hier irgendwo
gepostet wurde, konnte ich nicht portieren.

[dbt]

Ich habe 1.13.3 inzwischen auch zum laufen gebracht, aber mit dem insmod-Patch kann ich mich nicht so recht anfreunden. Jedenfalls nimmt er nicht die Pfadangaben für die Module wie sie in der rcS stehen, sondern nur die Modulnamen. Dort habe ich $MD lahm gelegt damit das geht. Und überhaupt ist die Originale Lösung für insmod irgendwie etwas hier vorbeigegangen. Evtl. Sollte man das mal direkt bei busybox anpingen, damit das hier nicht immer so auseinandergenommen werden muß. Es geht zwar, aber ist halt immer wieder umständlich.

[rhabarber1848]

Kurze Anmerkung: Busybox 1.13.3 ist im Flashimage 14kb größer als 1.7.2.