All of lore.kernel.org
 help / color / mirror / Atom feed
From: mds4@verizon.net (Mark Studebaker)
To: lm-sensors@vger.kernel.org
Subject: [PATCH 2.4] i2c cleanups
Date: Thu, 19 May 2005 06:24:30 +0000	[thread overview]
Message-ID: <3FE1C1C3.44A1CEF@verizon.net> (raw)
In-Reply-To: <20031213191258.2d78a9f7.khali@linux-fr.org>

On 12/17/03 16:50 Ky?sti M?lkki wrote:
> 
> Long story short, as I recall it.
> 

> For kernel 2.4 : Replace MOD_INC_COUNT with try_inc_mod_count(), this
> may fail if the module in question is about to be removed from kernel.
> The callback in i2c_adapter and i2c_driver is void (*inc_use)() so the
> choice of exporting .owner=THIS_MODULE to i2c-core as was already
> done in 2.5 tree seemed like the correct choice. This is the essential
> question whether the cleanup is just an api change or a real bugfix.
> 
> A problem I see there is procfs allowing only one module reference per
> opened file. In terms of i2c architecture, we need to hold both adapter
> and driver modules in place. Currently neither is done and I think it
> will oops if you enter a directory under /proc/sys/dev/sensors and rmmod
> either of the two hw interface modules. The fill_inode from days of
> 2.2.x does not seem either.
> 

correct. see Ticket #331.
I don't see this as a big problem though.

Unfortunately, most of this is over my head so don't expect me
to contribute much to a review of Khali's patch.

mds

  parent reply	other threads:[~2005-05-19  6:24 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-12-13 18:12 [PATCH 2.4] i2c cleanups Jean Delvare
2005-05-19  6:24 ` Jean Delvare
2003-12-13 18:33 ` [PATCH 2.4] i2c cleanups (1/4) Jean Delvare
2005-05-19  6:24   ` Jean Delvare
2003-12-13 18:49 ` [PATCH 2.4] i2c cleanups (2/4) Jean Delvare
2005-05-19  6:24   ` Jean Delvare
2003-12-13 19:31 ` [PATCH 2.4] i2c cleanups (3/4) Jean Delvare
2005-05-19  6:24   ` Jean Delvare
2003-12-13 19:36 ` [PATCH 2.4] i2c cleanups (4/4) Jean Delvare
2005-05-19  6:24   ` Jean Delvare
2005-05-19  6:24 ` [PATCH 2.4] i2c cleanups Kyösti Mälkki
2005-05-19  6:24 ` Kyösti Mälkki
2005-05-19  6:24 ` Jean Delvare
2005-05-19  6:24 ` Jean Delvare
2005-05-19  6:24 ` Jean Delvare
2005-05-19  6:24 ` Mark Studebaker [this message]
2005-05-19  6:24 ` Greg KH
2005-05-19  6:24 ` Jean Delvare
2005-05-19  6:24 ` Jean Delvare
2005-05-19  6:24 ` Kyösti Mälkki
2005-05-19  6:24 ` Kyösti Mälkki
2005-05-19  6:24 ` Kyösti Mälkki
2005-05-19  6:24 ` Jean Delvare
2005-05-19  6:24 ` Jean Delvare
2005-05-19  6:24 ` Kyösti Mälkki
2005-05-19  6:24 ` Kyösti Mälkki
2005-05-19  6:24 ` Kyösti Mälkki
2005-05-19  6:24 ` Kyösti Mälkki
2005-05-19  6:24 ` Kyösti Mälkki

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=3FE1C1C3.44A1CEF@verizon.net \
    --to=mds4@verizon.net \
    --cc=lm-sensors@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 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.