All of lore.kernel.org
 help / color / mirror / Atom feed
From: Doug Ledford <dledford@redhat.com>
To: Douglas Gilbert <dougg@torque.net>
Cc: linux-scsi@vger.kernel.org
Subject: Re: lk 2.5.49 module frustration
Date: Fri, 22 Nov 2002 22:26:16 -0500	[thread overview]
Message-ID: <20021123032616.GD19547@redhat.com> (raw)
In-Reply-To: <3DDEF2A2.8050701@torque.net>

On Sat, Nov 23, 2002 at 02:14:42PM +1100, Douglas Gilbert wrote:
> Hopefully others are having more success with the new
> module loader than I am. My system is basically RH8.0
> with the new module-init-tools (version 0.7) loaded
> and the recent 2.5.49 kernel.
> 
> During system load there is noise (failure?) on calls
> to modprobe since the new command doesn't support the same
> switches as the old. [harmless?]

It doesn't yet.  It's intended to.

> My lsmod looks like this:
> # lsmod
> Module                  Size  Used by
> scsi_debug             29423  0 [unsafe]
> ohci1394               21561  0
> ieee1394               41773  1 ohci1394 [unsafe]
> parport_pc             18883  0 [unsafe]
> parport                30228  1 parport_pc [unsafe]
> ehci_hcd               32801  0
> usbcore                90237  3 ehci_hcd
> 
> Those "unsafe" dependencies wedge in a module, making
> it hard to remove them. In the case of scsi_debug about
> 30 seconds after it is loaded I get this on my console:
> # Module scsi_debug cannot be unloaded due to unsafe
> # usage in fs/proc/inode.c:204
> That is procfs calling __MOD_INC_USE_COUNT. Is that our
> problem (i.e. shouldn't RR fix that before he lets this
> stuff loose on us)?

I posted a patch for this almost a week back.  It didn't get picked up, so 
I put this particular fix into my tree that I uploaded and announced 
today.

> Are all the MODULE_* macros now dead? I can no longer
> get any load time options/parameters through to the
> scsi_debug. Nothing useful comes out of modinfo.

Yes, it's very much a work in process, not a completed change over.  Rusty 
has publically stated that he thinks he bit off more than he can chew and 
I know he's taken a lot of flak over this change.  However, there *are* 
positive parts to Rusty's work and it does in fact solve the races he says 
it solves (at least once you convert things like fs/proc/inode.c to know 
about the fact that getting a module *can* fail) and as soon as you get 
the remaining parts of the kernel to properly grab module references when 
calling into a module.  Myself and Cristoph already did that basic job for 
the SCSI layer.

> Where is the ..... documentation?
> $ ls -l modules.txt
> -rw-r--r--    1 1046     101          9441 Feb 17  2001 modules.txt

Yes, that's another thing that isn't complete.

-- 
  Doug Ledford <dledford@redhat.com>     919-754-3700 x44233
         Red Hat, Inc. 
         1801 Varsity Dr.
         Raleigh, NC 27606
  

  reply	other threads:[~2002-11-23  3:26 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-11-23  3:14 lk 2.5.49 module frustration Douglas Gilbert
2002-11-23  3:26 ` Doug Ledford [this message]
2002-11-23  4:44   ` Randy.Dunlap
2002-11-23 12:42   ` Douglas Gilbert

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=20021123032616.GD19547@redhat.com \
    --to=dledford@redhat.com \
    --cc=dougg@torque.net \
    --cc=linux-scsi@vger.kernel.org \
    /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.