linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Michal Marek <mmarek@suse.cz>
To: "Fry, Donald H" <donald.h.fry@intel.com>
Cc: "John W. Linville" <linville@tuxdriver.com>,
	"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
	"kay@vrfy.org" <kay@vrfy.org>, "jcm@redhat.com" <jcm@redhat.com>,
	"linux-modules@vger.kernel.org" <linux-modules@vger.kernel.org>
Subject: Re: question on non-kernel patch
Date: Mon, 23 Apr 2012 20:56:22 +0200	[thread overview]
Message-ID: <4F95A5D6.7060500@suse.cz> (raw)
In-Reply-To: <193F82CA5D80C84A83D67BB5D5B9FDE548154FE8@ORSMSX101.amr.corp.intel.com>

Dne 20.4.2012 16:41, Fry, Donald H napsal(a):
> The base/core/common functionality is still called iwlwifi which
> interacts with the hardware.  On modprobe, the driver tries to find a
> microcode file to run based on the device/vendor id/sub-id.  While
> parsing the microcode file, it will indicate which software API it
> supports, which will indicate which specific module to use, module A
> (old) or module B (new).  This way the user still uses modprobe
> iwlwifi to install, and iwlwifi will request the appropriate specific
> module to make the hardware function.
> 
> However, since module A (for example) requires iwlwifi, an attempt to
> modprobe iwlwifi -r results in a message that iwlwifi is still in
> use.  Module A must be removed first followed by iwlwifi, etc.  While
> this may be obvious from looking at lsmod for a kernel developer, it
> is not obvious for most users.

Do users need to remove modules at all? I doubt it. And if they do, they
will use rmmod, because it is faster to type :). If your concern is
debugging bugreports from users, then simply instruct them to run
'modprobe -r iwlwifi_mod1; modprobe -r iwlwifi_mod2; modprobe -r
iwlwifi' instead of just 'modprobe -r iwlwifi'. Removing a module that
is not loaded will do nothing, it won't even print an error message.

Michal

  parent reply	other threads:[~2012-04-23 18:55 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-19 20:56 question on non-kernel patch Don Fry
2012-04-20 14:09 ` John W. Linville
2012-04-20 14:41   ` Fry, Donald H
2012-04-20 15:17     ` Larry Finger
2012-04-23 18:56     ` Michal Marek [this message]
2012-04-25 19:36     ` John W. Linville
2012-04-25 23:28       ` Kay Sievers

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=4F95A5D6.7060500@suse.cz \
    --to=mmarek@suse.cz \
    --cc=donald.h.fry@intel.com \
    --cc=jcm@redhat.com \
    --cc=kay@vrfy.org \
    --cc=linux-modules@vger.kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    /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).