All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tejun Heo <htejun@gmail.com>
To: Pedro Francisco <pedrogfrancisco@gmail.com>
Cc: Randy Dunlap <rdunlap@xenotime.net>,
	linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org
Subject: Re: ahci rmmodable and shouldn't
Date: Sun, 19 Sep 2010 14:32:36 +0200	[thread overview]
Message-ID: <4C9602E4.4090007@gmail.com> (raw)
In-Reply-To: <1284892106.2650.4.camel@sundae-v2>

Hello,

On 09/19/2010 12:28 PM, Pedro Francisco wrote:
> On Sex, 2010-09-17 at 11:53 +0200, Tejun Heo wrote:
>> Just don't rmmod a module which is serving a live filesystem.  Why
>> does it even matter?
> 
> ... Ok, let me argue another way. What if my mail was called: «lsmod
> shows false values ("Used by: 0") for module 'ahci'», would you agree it
> was an error and should therefore be fixed?

I think it works the same way for any other SCSI driver and it's
pretty convenient during development.  You can unload any network
driver module while the network is still up too.  Module reference
count doesn't have any specific meaning of 'use' preassigned other
than it's there to prevent unloading module while the code and data
are being actively used by the processor.  It's unfortunate that the
way module reference count is used for libata is causing discomfort to
you but I need much better reason to change it.

Thanks.

-- 
tejun

  reply	other threads:[~2010-09-19 12:31 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-16 21:56 ahci rmmodable and shouldn't Pedro Francisco
2010-09-16 21:58 ` Randy Dunlap
2010-09-17  9:53   ` Tejun Heo
2010-09-19 10:28     ` Pedro Francisco
2010-09-19 12:32       ` Tejun Heo [this message]
2010-09-20  9:25         ` Pedro Francisco
2010-09-20  9:31           ` Tejun Heo
2010-09-20  9:41             ` Tejun Heo
2010-09-20 20:44               ` Pedro Francisco
2010-09-20 22:09                 ` Tejun Heo
2010-09-20 22:10                   ` Tejun Heo
2010-09-21  6:14                     ` Michael Tokarev
2010-09-21  7:25                       ` [PATCH #upstream-fixes] ahci: fix module refcount breakage introduced by libahci split Tejun Heo
2010-09-21  7:32                         ` Michael Tokarev
2010-09-21  8:17                           ` Tejun Heo
2010-09-23 15:14                     ` ahci rmmodable and shouldn't Pedro Francisco

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=4C9602E4.4090007@gmail.com \
    --to=htejun@gmail.com \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pedrogfrancisco@gmail.com \
    --cc=rdunlap@xenotime.net \
    /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.