public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Rusty Russell <rusty@rustcorp.com.au>
To: David Brownell <david-b@pacbell.net>
Cc: "Adam J. Richter" <adam@yggdrasil.com>,
	greg@kroah.com, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Module alias and table support
Date: Fri, 29 Nov 2002 14:26:44 +1100	[thread overview]
Message-ID: <20021129040120.5F0B72C239@lists.samba.org> (raw)
In-Reply-To: Your message of "Thu, 28 Nov 2002 14:01:50 -0800." <3DE6924E.9060609@pacbell.net>

In message <3DE6924E.9060609@pacbell.net> you write:
> This is a multi-part message in MIME format.
> 
> --Boundary_(ID_oTV6oYegTuXHnGSwqC+1Lw)
> Content-type: text/plain; charset=us-ascii; format=flowed
> Content-transfer-encoding: 7BIT
> 
> 
> > Hmm, with 2.5.50 and module-init-tools 0.8a two "modules.*map" files
> > are created -- but they're empty.  That's with the latest 2.5 modutils
> > 
> >   http://www.kernel.org/pub/linux/kernel/people/rusty/modules/
> > 
> > So that's not quite working yet. ...
> 
> OK -- good news!
> 
> I now have USB hotplugging behaving again, with three patches on
> top of 2.5.50 and modutils 0.8a :
> 
> - The <linux/module.h> patch included with your 0.8a announce:
>     http://marc.theaimsgroup.com/?l=linux-kernel&m=103845080926386&w=2
> 
> - A tweak to that patch to still include <linux/elf.h> (to compile)

Good!  I think I'll release a 2.5.50 modules megapatch while I'm
waiting for Linus to merge.

> - The attached patch to your "modprobe", implementing "-n" and (so
>    I can see it works at least partially) using "-v".

And the compulsory spelling fix (Adam put that in, I'm sure it was to
check that people were paying attention).

Thanks, merged for 0.9 with the globals changed to locals and the
"wrong arg" fprintf removed (getopt prints a message) rather than
fixed.

Thanks very much for the testing and the patch!
Rusty.
--
  Anyone who quotes me in their sig is an idiot. -- Rusty Russell.

  reply	other threads:[~2002-11-29  3:54 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-11-27 20:54 [PATCH] Module alias and table support Adam J. Richter
2002-11-27 21:53 ` David Brownell
     [not found]   ` <20021127142006.A24246@adam.yggdrasil.com>
2002-11-27 22:59     ` David Brownell
2002-11-28 23:39       ` Rusty Russell
2002-11-28  3:14   ` Rusty Russell
2002-11-28  6:44     ` David Brownell
2002-11-28 22:01       ` David Brownell
2002-11-29  3:26         ` Rusty Russell [this message]
2002-11-29 19:56           ` Gerd Knorr
2002-11-29  1:28       ` Rusty Russell
2002-11-29  3:40     ` Greg KH
2002-11-27 22:40 ` 2.5 modutils getting back device " David Brownell
  -- strict thread matches above, loose matches on Subject: below --
2002-11-28  0:22 [PATCH] Module alias and " Adam J. Richter
2002-11-28  1:48 ` David Brownell
2002-11-28  4:45 ` Keith Owens
2002-11-27  6:25 David Brownell
2002-11-26  4:52 Adam J. Richter
2002-11-26  3:26 Adam J. Richter
2002-11-26  3:49 ` Greg KH
2002-11-14  1:19 Rusty Russell
2002-11-20  8:23 ` Greg KH
2002-11-24 23:34   ` Rusty Russell
2002-11-25 21:36     ` Greg KH
2002-11-25 23:42       ` Rusty Russell

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=20021129040120.5F0B72C239@lists.samba.org \
    --to=rusty@rustcorp.com.au \
    --cc=adam@yggdrasil.com \
    --cc=david-b@pacbell.net \
    --cc=greg@kroah.com \
    --cc=linux-kernel@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox