Re: [GIT v3] softdep: now with testing cheese, bugfixes and docs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


On 10/13/09, Andreas Robinson <andr345@xxxxxxxxx> wrote:
> On Tue, Oct 13, 2009 at 7:57 PM, Alan Jenkins
> <sourcejedi.lkml@xxxxxxxxxxxxxx> wrote:
>> On 10/13/09, Alan Jenkins <sourcejedi.lkml@xxxxxxxxxxxxxx> wrote:
>>> On 10/13/09, Andreas Robinson <andr345@xxxxxxxxx> wrote:

>>>> commit 5fd23bcc409045894152bc2e58cd978dcba5140a
>>>> Author: Andreas Robinson <andr345@xxxxxxxxx>
>>>> Date:   Sat Oct 10 20:00:22 2009 +0200
>>>>
>>>>     modprobe: softdep now handles mock modules
>>>>
>>>>     do_softdep skips the main module if it does not exist, following
>>>>     the behaviour of the install and remove commands.
>>>>
>>>>     Signed-off-by: Andreas Robinson <andr345@xxxxxxxxx>
>>>
>>> Hum.  Why?
>>>
>>> Install / remove commands are stupidly powerful.
>>
>> Also - this affects _all_ softdep rules, right?
>>
>> If you write an install command for a real module e.g. eeepc-laptop,
>> which includes "modprobe --ignore-install eeepc-laptop", you still get
>> a "FATAL" message if the module doesn't exist.  The exit code will
>> even be correct if you write the rule correctly :-).  But this commit
>> means that if you add softdeps for a module, it won't complain if the
>> module doesn't exist when you try to load it.  That seems like a step
>> backwards
>
> *cough*. It appears I didn't think this through properly... :-)
>
> Should I rip it out and rebase right away or do you want to review it
> the way it stands?

Go ahead.  I've fetched your git tree, so I can review it locally
whatever you do with it in the mean time :).

Thanks
Alan
--
To unsubscribe from this list: send the line "unsubscribe linux-modules" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Home]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Video Projectors]     [PDAs]     [Free Online Dating]     [Hacking TiVo]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Devices]     [Big List of Linux Books]     [16.7MP]

Add to Google Powered by Linux