All of lore.kernel.org
 help / color / mirror / Atom feed
From: Russell King <rmk+pcmcia@arm.linux.org.uk>
To: linux-pcmcia@lists.infradead.org, linux-ide@vger.kernel.org,
	alan@lxorguk.ukuu.org.uk
Subject: Re: [PATCH 1/8] pcmcia: Add an id to ide-cs.c
Date: Mon, 20 Nov 2006 21:07:03 +0000	[thread overview]
Message-ID: <20061120210702.GD26791@flint.arm.linux.org.uk> (raw)
In-Reply-To: <20061120174333.GA18660@isilmar.linta.de>

On Mon, Nov 20, 2006 at 12:43:33PM -0500, Dominik Brodowski wrote:
> As the matching by func_id is fuzzy and gives false positives, this is a
> multiple-step process:
> 
> a) the kernel checks all built-in and previously loaded modules for
>    prod_id and manf_id matches
> 
> b) userspace (udev/hotplug + modprobe) loads appropriate modules (including
>    those which are only matched by func_id
> 
> c) during the module initialization (e.g. modprobe hasn't returned yet) the
>    kernel checks the modules based on prod_id and manf_id matches
> 
> d) after all these modprobe calls return, userspace writes "1" into
>    /sys/$devpath/allow_func_id_match. Then, the kernel re-checks all
>    built-in and previously loaded modules for func_id "fuzzy" matches.
> 
> It is self-evident that steps b)-d) only work once userspace is ready. As
> PCMCIA drivers should be able to work even before that, manf_id and prod_id
> table entries do make sense even if func_id matching works.

However, it doesn't scale.  You're going to be forever adding entry
after entry after entry to drivers.  It's a never-ending job.

Of course, it's really up to you whether you want this task. 8)

-- 
Russell King
 Linux kernel    2.6 ARM Linux   - http://www.arm.linux.org.uk/
 maintainer of:  2.6 Serial core

  reply	other threads:[~2006-11-20 21:07 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-11-19 16:34 [git pull] PCMCIA fixes for 2.6.19-rc6 Dominik Brodowski
2006-11-19 16:38 ` [PATCH 1/8] pcmcia: Add an id to ide-cs.c Dominik Brodowski
2006-11-20  9:53   ` Russell King
2006-11-20 17:43     ` Dominik Brodowski
2006-11-20 21:07       ` Russell King [this message]
2006-11-19 16:39 ` [PATCH 2/8] pcmcia: yet another IDE ID Dominik Brodowski
2006-11-19 17:05   ` Alan
2006-11-19 20:36 ` [git pull] PCMCIA fixes for 2.6.19-rc6 Andrew Morton
2006-11-20  4:24   ` Dominik Brodowski

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=20061120210702.GD26791@flint.arm.linux.org.uk \
    --to=rmk+pcmcia@arm.linux.org.uk \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-pcmcia@lists.infradead.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.