[Toybox] Toybox post from lamiaworks at skymesh.com.au requires approval

Rob Landley rob at landley.net
Wed Jun 26 20:35:08 PDT 2013


This got caught in the spam filter due to the poster not being  
subscribed to the list...
======quoted message from lamiaworks <lamiaworks at skymesh.com.au>,  
subject "[PATCH] Re: modinfo..."======
> On 06/22/2013 06:49 AM, toybox-request at lists.landley.net wrote:
>> 
>> On Wed, Jun 19, 2013 at 11:44:08PM -0500, Rob Landley wrote:
>>> On 06/18/2013 08:55:42 AM, Isaac wrote:
...
>> Thanks for the answer; I missed that, probably due to trying to read
>> rather late.
>> I have a patch that assumes that the presence of the string ".ko"  
>> indicates
>> use of a path to a module (*.ko.xz and similar included, but not  
>> supported).
>> 
>> While I am not aware of any rules forbidding a module like  
>> "barf.ko.ko",
>> an strace of module-init-tools modinfo indicates that they If -b is  
>> given, use an image of the module tree. use similar logic.
>> 
>> It adds 6 lines, with 41 bytes increase in binary size (GCC 4.4,
>> glibc/dynamic and musl/static, according to make  
>> baseline/bloatcheck).
>> I'm not sure if the void->int change was needed, though it does keep  
>> GCC
>> quiet. But perror_exit; -> perror_msg; return; was needed.
>> 
>> Thanks,
>> Isaac Dunham
> 
> Looking from puppy linux, the -d is in module-init-tools version 3.16,

Except I thought he was talking about -b? (Base path?)

> I also think it's used within kmod (replaces module-init-tools)  
> especially when
> working with eudev so for those changing over it may be worth adding.

I'm definitely interested in making eudev work. That said, the  
submitter withdrew the original -b patch and I haven't got a test case  
for it so I dunno what doing it right looks like...

> regards
> scsijon

Rob


More information about the Toybox mailing list