public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: David Brownell <david-b@pacbell.net>
To: Jeff Garzik <jgarzik@pobox.com>
Cc: Greg KH <greg@kroah.com>,
	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 14:45:12 -0800	[thread overview]
Message-ID: <3DD2D5F8.5030102@pacbell.net> (raw)
In-Reply-To: 3DD2BD4C.7060502@pobox.com

>> And it'd be handy if the text format for that information didn't change;
>> how it's stored in object modules doesn't matter.
> 
> Correction -- the tools that read the text format are buggy if they do 
> not transparently support changes to the text format.

That's not a correction, it's a tangent!   Either way, it's still
"handy" ... since supporting new formats requires new tools, and it's
clearly "handy" not to need to write, debug, and deploy new tools
(including teaching, documenting, etc).

> I am planning on adding PCI revision id to the information exported via 
> MODULE_DEVICE_TABLE(pci,...).  Tools which correctly read the 
> first-line-format-definition will continue to function as before, 
> regardless of additional fields I want to add.  Tools which make silly 
> assumptions will have those assumptions come back to bite them ;-)

The "silly assumption" seems to be that the current "modules.*map"
format can reasonably be extended ... when no rules for extending those
file formats were really defined, and the _only_ precedent for even
changing them involved incompatibility between "versions".

I'd far rather have a decent file format for that data, and move
away from that ungainly syntax, than try to retrofit rules about
how to extend that syntax.  Such a format should:

  - allow comments
  - not have as much garbage (zeroes, 0xffffffff, etc)
  - still be easily parsable from shell scripts (*)
  - be easier for site admins to edit (insert vi/emacs flamewar)
  - have an explicit version code, and rules about evolution

Alternatively, the tried'n'true approach of coupling the format
to the filename works:  don't reuse "modules.*map", use "*.modules"
or something.
- Dave

(*) This is the annoying requirement.  Maybe worth relaxing.
     For example, an XML format could easily support all the
     other requirements ... but not (AFAIK) that one.


  parent reply	other threads:[~2002-11-13 22:35 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 [this message]
2002-11-13 21:24 ` Jeff Garzik
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=3DD2D5F8.5030102@pacbell.net \
    --to=david-b@pacbell.net \
    --cc=greg@kroah.com \
    --cc=jgarzik@pobox.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox