gmane.linux.busybox

Re: bb 1.17 causing subsequent driver problems

Peter Scheie — 2012-05-25T19:13:01

Thanks Tito.  CONFIG_DEFAULT_MODULES_DIR is set to the
default, /lib/modules.  I used the same config file with both versions
of bb.  I've posted it at http://pastebin.com/xNqvHnTm.

Peter

Re: bb 1.17 causing subsequent driver problems

Tito — 2012-05-25T18:58:10

Hi,
is CONFIG_DEFAULT_MODULES_DIR set correctly:   
Directory that contains kernel modules.   
Defaults to "/lib/modules" 

maybe also posting your config file could help.

Ciao,
Tito

Re: bb 1.17 causing subsequent driver problems

Peter Scheie — 2012-05-25T18:19:30

Correction: I've been using bb 1.15.2.  1.16 has mkfs_ext2 and works
with no problem.  But at some point I'll want to move to an
even-less-out-of-date version such as 1.17 or greater, so the question
still stands.

Peter

On Fri, 2012-05-25 at 11:27 -0500, Peter Scheie wrote:

bb 1.17 causing subsequent driver problems

Peter Scheie — 2012-05-25T16:27:22

I've been using bb 1.16.2 to build a custom initramfs, which we use to
replace the stock initrd in Centos 5 (we do some special things we need
in the initramfs).  Now I need mkfs_ext2, so I grabbed the bb 1.17.2
that I use in another project.  But I'm getting some odd behavior.  The
initramfs loads fine and does all the stuff it's supposed to, after
which it does a switch_root (so the initramfs & bb are out of the
picture) and mounts the file systems from the local disk, runs the
Centos rc scripts, all which seems to go fine.  But with 1.17, when udev
is started by Centos's rc.sysinit, I get an error that "DM multipath
kernel driver not loaded", and then later a modprobe warning, "Could not
open 'kernel/net/netfilter/x_tables.ko': no such file or directory"
followed by another modprobe error, "FATAL: Error inserting ip_tables
(kernel/net/ipv4/netfilter.ko: no such file or directory", even though
both those files are on the disk.  And of course, iptables then won't
run.

Any ideas as to what changed between 1

Re: [PATCH] Allow XZ streaming format in tar.

Denys Vlasenko — 2012-05-24T06:59:53

Patch is whitespace-damaged.


You did not add --xz long option, only short one, -J.



Shouldn't this be "unxz"?

Re: MAC address deny in udhcpd

Denys Vlasenko — 2012-05-24T06:27:29

No, you can't:

udhcpd: can't parse line 57 in udhcpd.conf



You can block incoming packets by MAC using iptables.
This way, all traffic can be ignored, not only DHCP one.

Problems with --sort-section=alignment

Rich Felker — 2012-05-24T00:21:51

Busybox is linked with --sort-section=alignment, and this seems to be
causing an obscure bugs one of my users experienced building busybox
on ARM/musl target. I've filed a bug report with binutils, but I'm not
sure where the issue should be addressed. See here:

http://sourceware.org/bugzilla/show_bug.cgi?id=14156

Basically, --sort-section=alignment wants to reorder the .init
section, which is not possible. The problem does not show up on other
targets, but I can't see anything that inhibits reordering .init, so
it seems to be working only by chance...

Rich

RE: MAC address deny in udhcpd

Tom Isaacson — 2012-05-23T10:22:28

-----Original Message-----

No thoughts at all?

Tom

swapon: implementing discard option.

Andy Lee — 2012-05-23T01:02:59

Hi,

This makes swapon to use discard option.
Enable support for discarding freed pages before they are reused.

Cheers,
Andy

---
 util-linux/Config.src  |    7 +++++++
 util-linux/swaponoff.c |   41 +++++++++++++++++++++++++++++++++--------
 2 files changed, 40 insertions(+), 8 deletions(-)

diff --git a/util-linux/Config.src b/util-linux/Config.src
index 57a52ce..84652eb 100644
--- a/util-linux/Config.src
+++ b/util-linux/Config.src
< at >< at > -606,6 +606,13 < at >< at > config FEATURE_SWAPON_PRI
 help
   Enable support for setting swap device priority in swapon.

+config FEATURE_SWAPON_DISCARD
+bool "Support discard option -d"
+default y
+depends on SWAPONOFF
+help
+  Enable support for discarding freed pages before they are reused.
+
 config SWITCH_ROOT
 bool "switch_root"
 default y
diff --git a/util-linux/swaponoff.c b/util-linux/swaponoff.c
index 54867ec..d911b31 100644
--- a/util-linux/swaponoff.c
+++ b/util-linux/swaponoff.c
< at >< at > -8,10 +8,13 < at >< at >
  */

 //usage:#define swapon_trivial_usage
-//usage:       "[-a]

Re: patches for building with Android NDK

Rob Walker — 2012-05-22T17:01:51

Certainly.  I very much agree with that.

I was didn't mean for the above question to be rhetorical.  I'm quite
new to BusyBox, and am therefore curious.

-Rob

[PATCH] unzip: ignore chmod errors

Natanael Copa — 2012-05-22T15:11:46

This makes unzip to FAT filesystems not exit with error.

This similar to how the "normal" unzip works.

Signed-off-by: Natanael Copa <ncopa< at >alpinelinux.org>
---
 archival/unzip.c       |    2 +-
 include/libbb.h        |    1 +
 libbb/make_directory.c |    7 ++++++-
 3 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/archival/unzip.c b/archival/unzip.c
index 3c76cda..c1b945a 100644
--- a/archival/unzip.c
+++ b/archival/unzip.c
< at >< at > -596,7 +596,7 < at >< at > int unzip_main(int argc, char **argv)
 printf("   creating: %s\n", dst_fn);
 }
 unzip_create_leading_dirs(dst_fn);
-if (bb_make_directory(dst_fn, dir_mode, 0)) {
+if (bb_make_directory(dst_fn, dir_mode, FILEUTILS_IGNORE_CHMOD_ERR)) {
 xfunc_die();
 }
 } else {
diff --git a/include/libbb.h b/include/libbb.h
index f12800f..5e5c8c7 100644
--- a/include/libbb.h
+++ b/include/libbb.h
< at >< at > -333,6 +333,7 < at >< at > enum {/* DO NOT CHANGE THESE VALUES!  cp.c, mv.c, install.c depend on them. */
 FILEUTILS_PRESERVE_SECURITY_CONTEX

Re: [PATCH] unzip: ignore chmod errors

Natanael Copa — 2012-05-22T15:11:19

I forgot the signed-off...

I will resend. sorry


On Tue, May 22, 2012 at 4:56 PM, Natanael Copa <natanael.copa< at >gmail.com> wrote:

[PATCH] unzip: ignore chmod errors

Natanael Copa — 2012-05-22T14:56:22

This makes unzip to FAT filesystems not exit with error.

This similar to how the "normal" unzip works.
---
 archival/unzip.c       |    2 +-
 include/libbb.h        |    1 +
 libbb/make_directory.c |    7 ++++++-
 3 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/archival/unzip.c b/archival/unzip.c
index 3c76cda..c1b945a 100644
--- a/archival/unzip.c
+++ b/archival/unzip.c
< at >< at > -596,7 +596,7 < at >< at > int unzip_main(int argc, char **argv)
 printf("   creating: %s\n", dst_fn);
 }
 unzip_create_leading_dirs(dst_fn);
-if (bb_make_directory(dst_fn, dir_mode, 0)) {
+if (bb_make_directory(dst_fn, dir_mode, FILEUTILS_IGNORE_CHMOD_ERR)) {
 xfunc_die();
 }
 } else {
diff --git a/include/libbb.h b/include/libbb.h
index f12800f..5e5c8c7 100644
--- a/include/libbb.h
+++ b/include/libbb.h
< at >< at > -333,6 +333,7 < at >< at > enum {/* DO NOT CHANGE THESE VALUES!  cp.c, mv.c, install.c depend on them. */
 FILEUTILS_PRESERVE_SECURITY_CONTEXT = 1 << 9, /* -c */
 FILEUTILS_SET_SECURITY_CONTEXT

Re: patches for building with Android NDK

Tias — 2012-05-22T07:56:38

I'm not trying to imply that all applets should be supported on android...

Its just that a number of 'basic' applets such as bbconfig, date, df, 
fsck, kill, mke2fs, mount, ipv6 support etc. can currently not be built.
For many of these applets rather simple patches are already floating 
around, so it makes sense to integrate them upstream, no?


Kind regards,
Tias

Re: patches for building with Android NDK

Rob Walker — 2012-05-22T06:23:55

I get the feeling that Android's deliberately different in many of these
areas and that a fair number of applets are gonna be broken no matter
what.  The ROI's just too low to emulate or fix.  However, isn't this
true of a lot of other configurations that BusyBox supports?

What kind of system has FULL support?  I.e. every applet compiles and works?

Thanks,
Rob

On 5/21/12 10:54 PM, Tito wrote:

Re: patches for building with Android NDK

Tito — 2012-05-22T05:54:47

Hi,
at a first glance a few of this patches seems to add missing syscalls
and IMO they should be added to bionic libc and not to busybox
or at least not to the applets' files, maybe they could be added to
a platform.h file. 
Had not time to look at all of them tough.

Ciao,
Tito

patches for building with Android NDK

Tias — 2012-05-21T22:03:54

Dear Busybox developers,

Since Rob Walker's patch[1], building busybox with the Android NDK has 
really become dead simple; no special hacks or external libraries are 
needed anymore.

However, a large number of applets can currently not be built.

I started documenting why applets don't build, as well as collecting 
patches from different sources, that fix the issues:
https://github.com/tias/android-busybox-ndk
https://github.com/tias/android-busybox-ndk/tree/master/patches

I would love to see the ideas behind these patches integrated, so that a 
standard busybox build can be used in the many android open-source 
projects that depend on it (and currently use a statically built 
third-party blob).


However, I am unfamiliar with the coding practices of the busybox 
community, and many of these patches were written to fix the build and 
not necessarily to get merged upstream.
I consider my work in collecting these patches as just a first step.
Hence, I would like to solicit feedback on what can and can not

[PATCH] Allow XZ streaming format in tar.

Boris Reisig — 2012-05-21T19:24:17

Here's a patch that will allow tar to uncompress XZ file formats when it 
can't properly detect XZ data formats when the input data comes from the 
standard input.  I have added the (-J) option to force tar to recognize 
XZ file format. Without this patch, it won't work with data streams.

Example: wget -O - http://192.168.0.1/1.xz | tar -xvJf -

diff -uNr busybox-original/archival/tar.c busybox/archival/tar.c
--- busybox-original/archival/tar.c     2012-05-21 00:19:38.876360998 -0500
+++ busybox/archival/tar.c      2012-05-21 00:05:08.299222265 -0500
< at >< at > -693,6 +693,7 < at >< at >
  //usage:       "-[" IF_FEATURE_TAR_CREATE("c") "xt"
  //usage:       IF_FEATURE_SEAMLESS_Z("Z")
  //usage:       IF_FEATURE_SEAMLESS_GZ("z")
+//usage:       IF_FEATURE_SEAMLESS_XZ("J")
  //usage:       IF_FEATURE_SEAMLESS_BZ2("j")
  //usage:       IF_FEATURE_SEAMLESS_LZMA("a")
  //usage:       IF_FEATURE_TAR_CREATE("h")
< at >< at > -719,6 +720,9 < at >< at >
  //usage:       IF_FEATURE_SEAMLESS_GZ(
  //usage:     "\n       z       (De)compress using gzip"

Re: mdev with USB wifi adapters

Laurent Bercot — 2012-05-21T11:56:20

 To avoid this kind of race condition (due to the hotplug interface
spawning helpers in parallel instead of sequentially), you can use
s6-devd along with mdev. s6-devd just listens to the netlink interface
and spawns helpers sequentially.

 http://www.skarnet.org/software/s6-linux-utils/s6-devd.html

Re: mdev with USB wifi adapters

Walter Dnes — 2012-05-21T06:47:57

On Fri, May 18, 2012 at 10:49:05AM +0200, Denys Vlasenko wrote


  What I've been doing is to insert generic "print" code into the mdev
scripts. E.g....

exec 3>> /dev/shm/logfile.txt
printenv >&3
echo "===separator===" >&3
exec 3>&-

  This works in in my sh and bash.  Of course, in linux sh is merely a
symlink to bash.  Can anybody confirm this works in "real sh"?  See
http://tldp.org/LDP/abs/html/io-redirection.html for details.  Note that
bash may use file descriptor 5 internally, so you're limited to
3,4,6,7,8,9.  If you get desparate, close 0, 1, and 2 and re-use them.

  Anyhow, I got an important clue re some problems I'm having with
automount and unmount.  Inserting a USB key or drive generates multiple
events, e.g.

one event for /dev/sdb
one event for /dev/sdb1

and presumably one event for each and every partition.  I also assume
when receiving an event to create /dev/sdb1, I should put a sleep loop
in the script that gets activated if /dev/sdb doesn't exist yet <G>.
Now that I know what I'm up a

Re: [PATCH] Fix link time error for functions calling check_signature16()

Denys Vlasenko — 2012-05-20T10:57:40

On Fri, May 18, 2012 at 3:34 AM, Anthony G. Basile
<basile< at >opensource.dyc.edu> wrote:

Thanks! I reproduced the problem and committed a minimal fix.