All of lore.kernel.org
 help / color / mirror / Atom feed
From: Martin Dalecki <dalecki@evision-ventures.com>
To: Martin Mares <mj@suse.cz>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>,
	Vojtech Pavlik <vojtech@suse.cz>,
	Jeff Garzik <jgarzik@mandrakesoft.com>,
	David Woodhouse <dwmw2@infradead.org>,
	Dan Hollis <goemon@anime.net>,
	Oliver Xymoron <oxymoron@waste.org>,
	Keith Owens <kaos@ocs.com.au>,
	linux-kernel@vger.kernel.org
Subject: Re: Persistent module storage [was Linux 2.4 Status / TODO page]
Date: Tue, 07 Nov 2000 11:59:17 +0100	[thread overview]
Message-ID: <3A07E085.3661EB6D@evision-ventures.com> (raw)
In-Reply-To: <3A06A053.56F09ACB@mandrakesoft.com> <E13sphu-0006O4-00@the-village.bc.nu> <20001106221254.A1196@albireo.ucw.cz>

Martin Mares wrote:
> 
> Hi Alan!
> 
> > If the sound card is only used some of the time or setup and then used
> > for TV its nice to get the 60K + 128K DMA buffer back when you dont need it
> > especially on a low end box
> 
> So why don't we allocate / free the DMA buffer on device open / close instead
> of module insert / remove?  If the reason lies in problems with allocation
> of large chunks of contiguous memory, we're going to have exactly the same
> problems when autoloading the module.
> 
> I think that automatic loading / unloading of modules has been a terrible hack
> since its first days (although back in these times a useful one) and that the
> era of its usefulness is over. There are zillions of problems with this
> mechanism, the most important ones being:

Amen.

>    o  It would have to preserve _complete_ device state over module reload.
>       For the sound card mixer settings discussed it's close to trivial, but
>       for example consider a tape drive and the problem of preserving tape
>       position after reload (probably including device reset causing tape rewind).
>       And what about textures loaded to memory of a 3D video card?
> 
>    o  For many drivers, the "device currently open" concept makes no sense.
>       Consider a mouse driver whose only activity is to feed mouse events
>       to an event device. The mouse driver can be unloaded in any time (either
>       manually or perhaps automatically after the mouse gets unplugged), hence
>       it should have a use count == zero, but even if it seems to be unused,
>       it must not be unloaded just because of some timeout since the mouse
>       will cease to work.
> 
>    o  It interferes with hotplug in nasty ways. Let's have a USB host controller
>       driver with currently no devices on its bus. It's also an example of a zero
>       use count driver, but it also must not be unloaded as it's needed for
>       recognizing newly plugged in devices.

Plese add power-saving devices like in notebooks to the list as well.
For example in my notebook the PC speaker loops through the maestro-3e.
The BIOS is initializing the maestro with some sane mixer values but
after
a suspend cycle the pc speaker is compleatly off due to suspension of
the
maestro-3e chip and the leak of a *permanent* driver sitting around to
handle
the wakeup event.

> I don't argue whether we need or need not some kind of persistent storage for
> the modules (it might be a good idea when it comes to hotplug, but it should
> be probably taken care of by the userspace hotplug helpers), but I think that
> it has no chance to solve the problems with automatic unloading.
> 
> We could of course attempt to circumvent the problems listed above by adding
> some hints to module state which will say whether it's possible to auto-unload
> the module or not even if it has zero use count, but it means another thing
> to handle in all the drivers (well, at least another thing to think of whether
> it's needed or not for each driver) and I think that the total effect of
> the autounloading mechanism (a minimum amount of memory saved) in no way
> outweighs the cost of implementing it right.

And the pain for the user of the whole: Take the example of ALSA
over-modularisation...
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/

  parent reply	other threads:[~2000-11-07 10:07 UTC|newest]

Thread overview: 113+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <3A06A053.56F09ACB@mandrakesoft.com>
2000-11-06 12:29 ` Persistent module storage [was Linux 2.4 Status / TODO page] Keith Owens
2000-11-06 17:07   ` Alan Cox
2000-11-06 18:09     ` Martin Dalecki
2000-11-06 17:30       ` Alan Cox
2000-11-06 17:05 ` Alan Cox
2000-11-06 18:30   ` Paul Jakma
2000-11-06 21:12   ` Martin Mares
2000-11-07  1:17     ` Horst von Brand
2000-11-07  9:59       ` Martin Mares
2000-11-07 10:59     ` Martin Dalecki [this message]
2000-11-07 12:27       ` Alan Cox
2000-11-07  7:55 David Feuer
  -- strict thread matches above, loose matches on Subject: below --
2000-11-06 22:48 Wayne.Brown
2000-11-06  0:54 Linux 2.4 Status / TODO page (Updated as of 2.4.0-test10) David Woodhouse
2000-11-06  1:28 ` Persistent module storage [was Linux 2.4 Status / TODO page] Keith Owens
2000-11-06  6:39   ` David Woodhouse
2000-11-06  7:12   ` Oliver Xymoron
2000-11-06  7:17     ` David Woodhouse
2000-11-06  7:25       ` Jeff Garzik
2000-11-06  7:29         ` David Woodhouse
2000-11-06 10:53         ` Alan Cox
2000-11-06 11:03           ` Dan Hollis
2000-11-06 11:04             ` Jeff Garzik
2000-11-06 11:35               ` Alan Cox
2000-11-06 11:36                 ` Jeff Garzik
2000-11-06 11:06             ` David Woodhouse
2000-11-06 11:09               ` Jeff Garzik
2000-11-06 11:20               ` Jeff Garzik
2000-11-06 11:37               ` David Woodhouse
2000-11-06 11:40                 ` Jeff Garzik
2000-11-06 11:47                 ` David Woodhouse
2000-11-06 11:57                   ` Jeff Garzik
2000-11-06 12:03                     ` Alan Cox
2000-11-06 13:12                   ` David Woodhouse
2000-11-06 13:38                     ` Jeff Garzik
2000-11-06 13:56                     ` David Woodhouse
2000-11-06 13:21                   ` David Woodhouse
2000-11-06 13:35                   ` James A. Sutherland
2000-11-06 17:12                     ` Alan Cox
2000-11-06 17:38                       ` James A. Sutherland
2000-11-06 18:39                       ` Paul Jakma
2000-11-06 21:28                         ` Alan Cox
2000-11-06 18:55                     ` Dan Hollis
2000-11-07  0:18                       ` James A. Sutherland
2000-11-07  0:27                         ` Alan Cox
2000-11-07  0:38                           ` James A. Sutherland
2000-11-07 12:07                             ` Alan Cox
2000-11-07 12:13                               ` James A. Sutherland
2000-11-07 12:35                                 ` Alan Cox
2000-11-07 12:49                                   ` James A. Sutherland
2000-11-07 12:52                                     ` Alan Cox
2000-11-07 12:51                                   ` Petko Manolov
2000-11-06 13:40                   ` David Woodhouse
2000-11-06 15:23                     ` James A. Sutherland
2000-11-06 15:34                     ` David Woodhouse
2000-11-06 16:31                       ` Horst von Brand
2000-11-06 17:06                         ` David Woodhouse
2000-11-06 17:25                           ` Alon Ziv
2000-11-06 17:34                             ` Alan Cox
2000-11-06 19:49                               ` Rogier Wolff
2000-11-06 21:34                                 ` Alan Cox
2000-11-06 17:25                           ` David Woodhouse
2000-11-06 19:27                             ` Tim Riker
2000-11-06 21:33                               ` Alan Cox
2000-11-06 23:57                           ` Horst von Brand
2000-11-06 17:23                         ` Alan Cox
2000-11-08 14:56                           ` Jamie Lokier
2000-11-06 18:00                         ` Martin Dalecki
2000-11-06 17:29                           ` Alan Cox
2000-11-06 16:42                       ` James A. Sutherland
2000-11-06 16:57                         ` Horst von Brand
2000-11-06 17:01                           ` James A. Sutherland
2000-11-06 23:54                             ` Horst von Brand
2000-11-07  8:44                               ` James A. Sutherland
2000-11-06 17:12                           ` David Woodhouse
2000-11-06 17:45                             ` James A. Sutherland
2000-11-06 18:37                             ` Paul Jakma
2000-11-07  0:04                             ` Horst von Brand
2000-11-06 17:08                       ` David Woodhouse
2000-11-06 17:33                         ` James A. Sutherland
2000-11-06 23:28                           ` Gerhard Mack
2000-11-07  0:34                             ` James A. Sutherland
2000-11-07  0:42                               ` Gerhard Mack
2000-11-07  0:43                                 ` James A. Sutherland
2000-11-07  1:20                                   ` Gerhard Mack
2000-11-07  8:41                                     ` James A. Sutherland
2000-11-07  1:44                               ` Horst von Brand
2000-11-06 17:44                         ` David Woodhouse
2000-11-06 17:53                           ` James A. Sutherland
2000-11-06 20:46                             ` Evan Jeffrey
2000-11-07  0:23                               ` James A. Sutherland
2000-11-06 15:15                 ` Martin Dalecki
2000-11-06 17:19                   ` Alan Cox
2000-11-06 17:34                     ` David Woodhouse
2000-11-06 18:22                       ` Oliver Xymoron
2000-11-06 18:37                         ` Jeff Garzik
2000-11-06 19:09                           ` Oliver Xymoron
2000-11-07  0:32                             ` Horst von Brand
2000-11-06 21:19                           ` Alan Cox
2000-11-06 18:22                 ` Paul Jakma
2000-11-06 21:18                   ` Alan Cox
2000-11-06 23:00                     ` Paul Jakma
2000-11-07  2:11                       ` Keith Owens
2000-11-06  7:28       ` Oliver Xymoron
2000-11-06  7:32         ` David Woodhouse
2000-11-06  7:45           ` Jeff Garzik
2000-11-06  8:00             ` David Woodhouse
2000-11-06 13:44               ` Andrew Pimlott
2000-11-06  7:48           ` Oliver Xymoron
2000-11-06  8:02             ` David Woodhouse
2000-11-06 18:09               ` Eric W. Biederman
2000-11-06 21:17                 ` Alan Cox
2000-11-07  9:55                   ` Helge Hafting
2000-11-07  2:09                 ` Keith Owens

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3A07E085.3661EB6D@evision-ventures.com \
    --to=dalecki@evision-ventures.com \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=dwmw2@infradead.org \
    --cc=goemon@anime.net \
    --cc=jgarzik@mandrakesoft.com \
    --cc=kaos@ocs.com.au \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mj@suse.cz \
    --cc=oxymoron@waste.org \
    --cc=vojtech@suse.cz \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.