All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeff Garzik <jgarzik@pobox.com>
To: Greg KH <greg@kroah.com>
Cc: David Brownell <david-b@pacbell.net>,
	rusty@rustcorp.com.au, kaos <kaos@ocs.com.au>,
	linux-kernel@vger.kernel.org
Subject: Re: 2.5.47bk2 + current modutils == broken hotplug
Date: Wed, 13 Nov 2002 16:24:27 -0500	[thread overview]
Message-ID: <3DD2C30B.9000404@pobox.com> (raw)
In-Reply-To: <3DD2B1D5.7020903@pacbell.net>

Greg KH wrote:

> On Wed, Nov 13, 2002 at 03:59:56PM -0500, Jeff Garzik wrote:
>
> >(tangent warning!)
> >Another long term idea I would eventually like to realize is the removal
> >of device ids from the C source code.  I don't care where they go --
> >drivers/net/pci_ids [per directory ids?], drivers/net/3c59x.meta,
> >whereever.  Anywhere but the C source code.  It's quite silly to require
> >a driver rebuild just to add a single PCI id, and further, embedding
> >metadata in C source is rarely a good idea in the long term.  [reference
> >some of Linus's counter-arguments when it was mentioned that Donald
> >Becker's method of including Config.{in,help} data in C source might be
> >useful]
>
>
> True, this would be nice, but how would the driver know to bind to a new
> device, if it isn't rebuilt, and doesn't know about the new id that was
> just added?  In the current scheme of driver matching to devices, I
> don't see how this could be done.


I think that truly seamless rebinding is doable but would require too 
much additional complexity in the kernel.  Rebinding to a new id table 
between unregister() ... register() pairs, or between mod unload and mod 
load, should be enough to be useable for 98% of the cases.

It should be noted that David Hinds' pcmcia-cs package stores id in an 
external database.  There are both positive and negative lessons that 
have been learned from that experience (WRT external id tables only! 
:):)) too.


> Not to say I would not want to see this changed to allow this to happen,
> I'm very tired of telling USB Palm users to get a new kernel version
> just because a single device id was added which their new device has.


Indeed :/  There are also issues like vendors who want to GPG-sign 
drivers, and updating the PCI id table makes the GnuPG signature and the 
certification that goes along with it invalid, requiring a vendor update.

	Jeff



  parent reply	other threads:[~2002-11-13 21:18 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-11-13 20:11 2.5.47bk2 + current modutils == broken hotplug David Brownell
2002-11-13 20:17 ` Greg KH
2002-11-13 20:40   ` David Brownell
2002-11-13 20:59 ` Jeff Garzik
2002-11-13 21:07   ` Greg KH
2002-11-13 22:45   ` David Brownell
2002-11-13 21:24 ` Jeff Garzik [this message]
2002-11-13 23:00   ` David Brownell
2002-11-14  3:46 ` Rusty Russell
2002-11-14  8:02   ` David Brownell
2002-11-14 10:01     ` Rusty Russell
2002-11-14 16:19       ` David Brownell
2002-11-14 17:42         ` Rusty Russell
2002-11-14 10:41   ` Gerd Knorr

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=3DD2C30B.9000404@pobox.com \
    --to=jgarzik@pobox.com \
    --cc=david-b@pacbell.net \
    --cc=greg@kroah.com \
    --cc=kaos@ocs.com.au \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rusty@rustcorp.com.au \
    /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.