[BusyBox 0000661]: mdev doesn't work with uClibc-0.9.28

bugs at busybox.net bugs at busybox.net
Sun Jan 29 09:18:31 UTC 2006


A NOTE has been added to this issue. 
====================================================================== 
http://busybox.net/bugs/view.php?id=661 
====================================================================== 
Reported By:                gavinl
Assigned To:                BusyBox
====================================================================== 
Project:                    BusyBox
Issue ID:                   661
Category:                   Other
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     assigned
====================================================================== 
Date Submitted:             01-26-2006 10:23 PST
Last Modified:              01-29-2006 01:18 PST
====================================================================== 
Summary:                    mdev doesn't work with uClibc-0.9.28
Description: 
1. if read() file return less than 1, we should close the fd and then goto
end.
2. In uClibc-0.9.28, the entry->d_type is always DT_UNKNOWN, so we can't
use d_type to check if it is a directory.
3. We need to omit "device" and "bridge" in searchin /proc directory, they
are links, and will cause endless loop.

Patch based on busybox-1.1.0 and a snapshot is attached.
====================================================================== 

---------------------------------------------------------------------- 
 gavinl - 01-27-06 06:03  
---------------------------------------------------------------------- 
In Description 3, it should be "in searching /sys directory" 

---------------------------------------------------------------------- 
 landley - 01-29-06 01:18  
---------------------------------------------------------------------- 
http://busybox.net/bugs/view.php?id=1 fixed.
http://busybox.net/bugs/view.php?id=2: why isn't this a uClibc bug?
http://busybox.net/bugs/view.php?id=3: This is related to
http://busybox.net/bugs/view.php?id=2: we're only recursing into directories,
not into
symlinks.  This problem only happens when we can't tell the difference,
because the C library is broken.

Now proposed changes to /sys break this by turning everything into
symlinks, but udev keeps breaking so I suppose they feel they must break
mdev to maintain parity. :)  But hardwiring in knowledge about the names
of specific branches it's dangerous do go down is _exteremely_ ugly...

http://lwn.net/Articles/162242/ 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
01-26-06 10:23  gavinl         New Issue                                    
01-26-06 10:23  gavinl         Status                   new => assigned     
01-26-06 10:23  gavinl         Assigned To               => BusyBox         
01-26-06 10:23  gavinl         File Added: busybox-1.1.0-mdev.patch             
      
01-27-06 06:03  gavinl         Note Added: 0001014                          
01-29-06 01:18  landley        Note Added: 0001018                          
======================================================================




More information about the busybox-cvs mailing list