On 2002.03.07 Thierry Vignaud wrote:
>"J.A. Magallon" <[EMAIL PROTECTED]> writes:
>
>> > ie, if we detect a tv card at install time, we alter /etc/modules
>> > so that bttv is automagically loaded when you boot.
>>
>> You shouldn't do that. That implies I am wasting memory all the
>> time,
>
>sure, 30kb ...

Yes, just negligible, but...

>
>better spent your time in fixing memory leaks in userland apps or
>optimize their memory usage patterns.
>
>> just to see tv or hear radio a short amount of time.
>
>having support for tv/radio is better than no support at all :-)
>

...what I do not understand is why you throw away all the kernel
autoload of modules, that was invented to not have to do what you are
doing.

>> What you shoul really do is to modify /etc/modules.conf like
>>
>> # TV
>> alias char-major-81 bttv
>
>we may do this later. as for now, we fully support quite a number of
>tv cards. i won't change something that seems "just an easy trick"
>just before release.
>

What looks like an 'easy trick' is to preload everything. If you have the
autoloader, you can use it. See scsi_hostadapter. You could define
a generic module in shipped /etc/modules.conf, something like

alias radio-tv-card off
alias char-major-81 radio-tv-card

and then, at install time, detect the card and

s/radio-tv-card off/radio-tv-card the_chosen_one

>for now, we only fix bugs. not enhance something that just works
>smoothly.
>

>you see the point ?
>

Really, I don't. But 8.2 is just there.

-- 
J.A. Magallon                           #  Let the source be with you...        
mailto:[EMAIL PROTECTED]
Mandrake Linux release 8.2 (Cooker) for i586
Linux werewolf 2.4.19-pre2-jam2 #1 SMP Tue Mar 5 01:21:55 CET 2002 i686

Reply via email to