BusyBox Bug and Patch Tracking
BusyBox
  

Viewing Issue Simple Details Jump to Notes ] View Advanced ] Issue History ] Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0000279 [BusyBox] Networking Support major always 06-02-05 15:03 09-30-06 13:21
Reporter jean2 View Status public  
Assigned To BusyBox
Priority normal Resolution fixed  
Status closed   Product Version 1.00
Summary 0000279: [PATCH] On-demand net module loading fails with CONFIG_FEATURE_IFUPDOWN_IP
Description Kernel modules autoloading of network driver doesn't work when ifup use ip to configure the interface. This feature is useful because it allow to load the network driver only when it's configured in /etc/network/interfaces, to
minimise kernel memory footprint and power consumption.

The basic problem is that 'ip' is not designed to support module autoloading. This problem is most likely present upstream as well, however all distros I know use ifconfig from ifup (and not ip), which is why probably nobody noticed it.

The fix is pretty safe, if we don't find the interface in our cache, we explicitely request it from the kernel.

Patch is attached. I made the patch for busybox-1.00 and fully tested there, but I've verified that it applies smoothly and compile with the 02-june-05 snapshot.

Jean
Additional Information
Attached Files  busybox-ip-module-loading.patch [^] (1,597 bytes) 06-02-05 15:03

- Relationships

- Notes
(0000397)
mdeschamps
08-09-05 02:41

Please give away patches as patches not as exercices !
 
(0000470)
jean2
08-29-05 09:31

mdeschamps wrote :
>
> Please give away patches as patches not as exercices !

    Would you mind to clarify ? I can see that you have not tested the patch, neither understood what it does.
    The patch is fully functional and fully tested, and a very similar patch has been included upstream. The only exercise left to the reader is a minor optimisation thast does not matter in real life (especially in the context of memory constrained systems).
    Jean
 
(0001661)
vda
09-29-06 17:18

Committed as revision 16268
 

- Issue History
Date Modified Username Field Change
06-02-05 15:03 jean2 New Issue
06-02-05 15:03 jean2 File Added: busybox-ip-module-loading.patch
08-09-05 02:41 mdeschamps Note Added: 0000397
08-29-05 09:31 jean2 Note Added: 0000470
06-26-06 13:06 integrator Issue Monitored: integrator
07-12-06 05:17 integrator Issue End Monitor: integrator
09-29-06 17:18 vda Status assigned => resolved
09-29-06 17:18 vda Fixed in Version  => 1.2.x
09-29-06 17:18 vda Resolution open => fixed
09-29-06 17:18 vda Note Added: 0001661
09-30-06 13:21 vda Status resolved => closed


Copyright © 2000 - 2006 Mantis Group
Powered by Mantis Bugtracker