Bug triage.

[Rob Landley]

On Friday 09 December 2005 17:50, Yann E. MORIN wrote:
> Hello All!
> Rob,
>
> On Thursday 08 December 2005 073, Rob Landley wrote:
> > 304  sometimes rmmod is unable to remove modules
> >   It's the - vs _ problem again.
>
> --> Fixed by revision 12546.

Got it.

> > 468  rmmod does not report failure
>
> --> Patch attached.
>     Easy.
>
> This is the patch from the bug system rediffed against rev 12789.
>   => busybox-0000468-rmmod-syscall.patch.bz2

Applied.

> > 467  rmmod does not handle -w and -f flags
>
> --> Fixed in revision 12551.

Got it.

> > 362  insmod chatter
>
> --> Patch attached.
>     Trivial.
>
> There was (IMHO) no need to update usage.h, as it already specifies both
> quiet and verbose options.
> This is the patch from the bug system rediffed against rev 12789.
>   => busybox-0000362-insmod-chatter.patch.bz2

Applied.

> > 276  Linux 2.6 module autoloading breaks when support for 2.4 modules is
> > disabled
> > 272  modprobe does not process parameters
>
> --> Patch attached.
>     Heavy.
> The attached patch about modprobe rework solves these two issues. Note that
> it still relies on my MODPROBE_MULTIPLE_OPTIONS. Rediffed against rev
> 12789. => busybox-modprobe.patch.bz2

This has unnecessary DEBUG() stuff in it.  I'll come back and give it a closer 
look this evening.

> --> Patch attached.
>     Easy.
> Another patch is attached that reworks the config options order and
> dependencies. Note that some options were used by either rmmod, insmod,
> lsmod or modprobe, but were available only when insmod was selected.
> Against rev 12789. => busybox-modutils-config-reworked.patch.bz2

Applied.  This fixes bug 272 by itself?

> Note that all these patches are not dependent one on the other, and so can
> be applied independently.

I applied all but one of them. :)

Rob
-- 
Steve Ballmer: Innovation!  Inigo Montoya: You keep using that word.
I do not think it means what you think it means.