linux-hotplug.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Alexander E. Patrakov" <patrakov@ums.usu.ru>
To: linux-hotplug@vger.kernel.org
Subject: Re: [patch] pnp rc
Date: Sat, 09 Oct 2004 05:36:21 +0000	[thread overview]
Message-ID: <416778D5.2090409@ums.usu.ru> (raw)
In-Reply-To: <416695D2.2050405@free.fr>

matthieu castet wrote:
> Hi,
> this patch clean the /etc/hotplug/isapnp.rc.
> It is based against debian hotplug version.
> 
> I saw that you add recently in your cvs a pnp.rc.
> 
> You will see that your version also try to load anolog for PNPB02F 
> instead of the gameport driver ns558. And it is not right because anolog 
> is a joystick driver not the generic gameport driver we want.

This bug applies to both versions of the script. In both pnp.distmap and 
isapnp.aliases, please remove the "analog" line completely (as opposed 
to what your patch does). The reason is that the "ns558" driver already 
contains this alias without our help, see

/lib/modules/`uname -r`/modules.alias

Thanks for the report. Don't forget to forward that to Debian 
bugtracker, since the bug is also there.

> pnp.rc it is near 4 time longer than the debian version :

well - could you please improve the Debian version in order for it to be 
able to load e.g. the snd-es18xx module?

> Also ./pnp.rc list even modules that are compile with the kernel : here 
> pcspkr, psmouse, atkdb...

The strange thing is that you don't see this with isapnp.rc - this may 
happen if you forgot to add the contents of isapnp.aliases to your 
modprobe.conf file.

The recent Debian versions of isapnp.rc actually try to resolve these 
module names:
         MODULE=$(modprobe --show-depends -q pnp:d$MODULE | sed -e 
'$!d;s/.*\/\(.*\)\.ko .*/\1/')
         [ "$MODULE" ] || continue

So they do try to modprobe --show-depends e.g. the "atkbd" driver, but 
fail silently and continue (as opposed to pnp.rc when used with Debian's 
"functions" file -- that fails with some noise and continues).

As for Greg's statement concerning the maintainership: please give me a 
week to decide if I want to be a maintainer. The reasons for such a long 
delay is explained below.

1) I have to coordinate my actions with Marco d'Itri (from Debian). I 
remember his words about the idea with aliases being modern, progressive 
and right. The question is whether it is possible and good to apply this 
idea to pci.rc also. I think it is, but I want to know his opinion.

2) Currently with 2.6 kernels module-init-tools don't convert 
MODULE_DEVICE_TABLE(isapnp, ...) to modules.isapnpmap entries. They 
understand only pnp and pnp_card device maps. Because of this 
limitation, any isapnp hardware detection script that relies only on 
modules.isapnpmap (or, equivalently, aliases) is less useful that it 
might be. E.g. 3com network cards are not detectable. What is the 
correct list to discuss that limitation?

-- 
Alexander E. Patrakov


-------------------------------------------------------
This SF.net email is sponsored by: IT Product Guide on ITManagersJournal
Use IT products in your business? Tell us what you think of them. Give us
Your Opinions, Get Free ThinkGeek Gift Certificates! Click to find out more
http://productguide.itmanagersjournal.com/guidepromo.tmpl
_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel

  parent reply	other threads:[~2004-10-09  5:36 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-10-08 13:27 [patch] pnp rc matthieu castet
2004-10-08 22:10 ` Greg KH
2004-10-09  5:36 ` Alexander E. Patrakov [this message]
2004-10-09  8:51 ` Alexander E. Patrakov
2004-10-10 20:38 ` matthieu castet

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=416778D5.2090409@ums.usu.ru \
    --to=patrakov@ums.usu.ru \
    --cc=linux-hotplug@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;
as well as URLs for NNTP newsgroup(s).