All of lore.kernel.org
 help / color / mirror / Atom feed
From: mfedyk@matchmail.com (Mike Fedyk)
To: sensors@stimpy.netroedge.com, linux-kernel@vger.kernel.org
Cc: greg@kroah.com, jamagallon@able.es, PrakashKC@gmx.de, akpm@osdl.org
Subject: 2.6.3-mm4
Date: Thu, 19 May 2005 06:24:44 +0000	[thread overview]
Message-ID: <40419A15.8030108@matchmail.com> (raw)
In-Reply-To: <20040227205922.6405eff7.khali@linux-fr.org>

Jean Delvare wrote:
>>You have to be kidding me.  Are you saying that with your patches to 
>>libsensors it won't support 2.6.3 style sensor sysfs names?
> 
> 
> Yes I am.  This isn't a fundamentally new problem.  Each Linux 2.6 has
> had its matching libsensors version that was not backward compatible
> (with earlier 2.6 kernels; it's still fully compatible with 2.4).
> 
> Keeping newer versions of libsensors compatible with all early 2.6
> kernel versions would make it incredibly more complex, with no
> significant benefit IMHO.
> 
> The facts are that the sysfs interface to i2c chips is just stabilizing.
> Greg's original naming scheme had many drawbacks (also we can't blame
> him for that, since nobody objected before me), as I have been
> explaining in a previous post on LKML. Also, many chip drivers did not
> comply with it in early 2.6 kernels, and new chip drivers wouldn't fit
> in it anyway.
> 
> The new interface is required if we want to write a chip-independant
> libsensors someday. I estimate that this is worth breaking the
> compatibility. The impacted kernels (later 2.5 and earlier 2.6) will
> obviously not be used anymore within a short time anyway.
> 
> I invite you to read my original post here:
> http://marc.theaimsgroup.com/?l=linux-kernel&m\x107715308608606
> 
> I explain the problems of the current interface and my goals with the
> new one. If you can think of a better solution to the problem, please
> speak up.

Working from the premise that there is a current (old-style with mostly 
chip dependent code), libsensors has 2.4 /proc support, and each 
specific release supports one of 2.6.[0123]...

I'm glad I'm not the maintainer of libsensors for a distribution.  Since 
you have effectively pushed the compatibility work to them.  Just think 
of angry customer complaints about this. :(

Since there is going to be an effective libsensors-new library with chip 
independent code, I suggest you put the compat code into the old library.

Mike

WARNING: multiple messages have this Message-ID (diff)
From: Mike Fedyk <mfedyk@matchmail.com>
To: sensors@stimpy.netroedge.com, linux-kernel@vger.kernel.org
Cc: greg@kroah.com, jamagallon@able.es, PrakashKC@gmx.de, akpm@osdl.org
Subject: Re: 2.6.3-mm4
Date: Sat, 28 Feb 2004 23:51:49 -0800	[thread overview]
Message-ID: <40419A15.8030108@matchmail.com> (raw)
In-Reply-To: <20040227205922.6405eff7.khali@linux-fr.org>

Jean Delvare wrote:
>>You have to be kidding me.  Are you saying that with your patches to 
>>libsensors it won't support 2.6.3 style sensor sysfs names?
> 
> 
> Yes I am.  This isn't a fundamentally new problem.  Each Linux 2.6 has
> had its matching libsensors version that was not backward compatible
> (with earlier 2.6 kernels; it's still fully compatible with 2.4).
> 
> Keeping newer versions of libsensors compatible with all early 2.6
> kernel versions would make it incredibly more complex, with no
> significant benefit IMHO.
> 
> The facts are that the sysfs interface to i2c chips is just stabilizing.
> Greg's original naming scheme had many drawbacks (also we can't blame
> him for that, since nobody objected before me), as I have been
> explaining in a previous post on LKML. Also, many chip drivers did not
> comply with it in early 2.6 kernels, and new chip drivers wouldn't fit
> in it anyway.
> 
> The new interface is required if we want to write a chip-independant
> libsensors someday. I estimate that this is worth breaking the
> compatibility. The impacted kernels (later 2.5 and earlier 2.6) will
> obviously not be used anymore within a short time anyway.
> 
> I invite you to read my original post here:
> http://marc.theaimsgroup.com/?l=linux-kernel&m=107715308608606
> 
> I explain the problems of the current interface and my goals with the
> new one. If you can think of a better solution to the problem, please
> speak up.

Working from the premise that there is a current (old-style with mostly 
chip dependent code), libsensors has 2.4 /proc support, and each 
specific release supports one of 2.6.[0123]...

I'm glad I'm not the maintainer of libsensors for a distribution.  Since 
you have effectively pushed the compatibility work to them.  Just think 
of angry customer complaints about this. :(

Since there is going to be an effective libsensors-new library with chip 
independent code, I suggest you put the compat code into the old library.

Mike

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

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-02-26  2:55 2.6.3-mm4 Andrew Morton
2004-02-26  8:22 ` 2.6.3-mm4 Alexander Hoogerhuis
2004-02-26  8:48   ` 2.6.3-mm4 Marc-Christian Petersen
2004-02-26  8:51 ` 2.6.3-mm4 Nuno Silva
2004-02-27  0:48   ` 2.6.3-mm4 Greg KH
2004-02-26 15:50 ` 2.6.3-mm4 David Martínez Moreno
2004-02-26 15:59   ` 2.6.3-mm4 Christoph Hellwig
2004-02-26 16:30 ` 2.6.3-mm4 (compile stats) John Cherry
2004-02-26 18:59   ` Mike Fedyk
2004-02-26 19:26     ` John Cherry
2004-02-26 18:50 ` 2.6.3-mm4 Matthias Hentges
2004-02-26 23:35 ` 2.6.3-mm4, sensors broken Prakash K. Cheemplavam
2004-02-27  0:03   ` Andrew Morton
2004-02-27  8:58     ` Prakash K. Cheemplavam
2004-02-27  0:11 ` 2.6.3-mm4 J.A. Magallon
2004-02-27  0:46   ` 2.6.3-mm4 Greg KH
2005-05-19  6:24     ` 2.6.3-mm4 Greg KH
2004-02-27  8:35     ` 2.6.3-mm4 Jean Delvare
2005-05-19  6:24       ` 2.6.3-mm4 Jean Delvare
2004-02-27 18:16       ` 2.6.3-mm4 Mike Fedyk
2005-05-19  6:24         ` 2.6.3-mm4 Mike Fedyk
2004-02-27 19:59         ` 2.6.3-mm4 Jean Delvare
2005-05-19  6:24           ` 2.6.3-mm4 Jean Delvare
2004-02-29  7:51           ` Mike Fedyk [this message]
2005-05-19  6:24             ` 2.6.3-mm4 Mike Fedyk
2004-02-29 10:11             ` 2.6.3-mm4 Jean Delvare
2005-05-19  6:24               ` 2.6.3-mm4 Jean Delvare
2004-02-27 16:48     ` 2.6.3-mm4 Prakash K. Cheemplavam
2005-05-19  6:24       ` 2.6.3-mm4 Prakash K. Cheemplavam
2004-02-27  9:00   ` 2.6.3-mm4 Prakash K. Cheemplavam
2004-02-27 23:51 ` 2.6.3-mm4 Adrian Bunk
2004-03-01  8:25 ` MM VM patches was: 2.6.3-mm4 Mike Fedyk
2004-03-01  8:44   ` Nick Piggin
2004-03-01  9:05     ` Mike Fedyk
2004-03-01  9:27       ` Nick Piggin
2004-03-01  9:47         ` Mike Fedyk
2004-03-01  9:10     ` Nick Piggin
2004-03-01  9:52       ` [PATCH] 2.6.4-rc1-mm1: vm-kswapd-incremental-min (was Re: MM VM patches was: 2.6.3-mm4) Nick Piggin
2004-03-01 10:18         ` Andrew Morton
2004-03-01 10:29           ` Nick Piggin
     [not found] <A6974D8E5F98D511BB910002A50A6647615F33F5@hdsmsx402.hd.intel.com>
2004-02-26 22:40 ` 2.6.3-mm4 Len Brown

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=40419A15.8030108@matchmail.com \
    --to=mfedyk@matchmail.com \
    --cc=PrakashKC@gmx.de \
    --cc=akpm@osdl.org \
    --cc=greg@kroah.com \
    --cc=jamagallon@able.es \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sensors@stimpy.netroedge.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 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.