From: "Jörn Engel" <joern@wohnheim.fh-wedel.de>
To: Martin Schlemmer <azarah@gentoo.org>
Cc: James Bourne <jbourne@hardrock.org>,
Ken Brownfield <brownfld@irridia.com>,
Marc-Christian Petersen <m.c.p@wolk-project.de>,
KML <linux-kernel@vger.kernel.org>
Subject: Re: Oops: ptrace fix buggy
Date: Mon, 14 Apr 2003 20:58:06 +0200 [thread overview]
Message-ID: <20030414185806.GA12740@wohnheim.fh-wedel.de> (raw)
In-Reply-To: <1050343825.4757.17.camel@nosferatu.lan>
On Mon, 14 April 2003 20:10:25 +0200, Martin Schlemmer wrote:
>
> From include/linux/version.h:
>
> ---------------------------------------------------------------
> #define UTS_RELEASE "2.5.67-bk2"
> #define LINUX_VERSION_CODE 132419
> #define KERNEL_VERSION(a,b,c) (((a) << 16) + ((b) << 8) + (c))
> ---------------------------------------------------------------
>
> Your changes will definitely break things that do:
>
> #if ((LINUX_VERSION_CODE > KERNEL_VERSION(2.4.0)) && \
> (LINUX_VERSION_CODE < KERNEL_VERSION(2.5.0)))
>
> Also, most docs explaining the version format, refer to
>
> major.minor.micro
>
> Anyhow, I did not say it was set in stone, I just said if you
> start getting breakage when compiling/configuring things, do
> not wonder why =)
James Bourne:
> The problem comes in when you bump the FIXLEVEL for example, by one. At
> that point the LINUX_VERSION_CODE would not change even though you have a
> new kernel and modules compiled for the previous version may not load or
> may load and not work correctly (just one example).
>
> EXTRAVERSION on the other hand is not used for LINUX_VERSION_CODE
> calculation.
So basically, neither the existing EXTRAVERSION nor my new FIXLEVEL
are checked. Any code could potentially break with -ac1 to -ac2 or
with .1 to .2.
Did anyone experience such problems with -ac already? There are far
more changes in -ac than there are in your patch.
Driver compilation should not be an issue. Change the Makefile and
version.h should be changed as well, so any code depending on
version.h will be rebuild, whether necessary or not.
Module load sounds unrealistic for .[123...], as you shouldn't change
any interfaces with fixes. But it might be a real problem for -ac.
Jörn
PS: Or for -aa, -dj, -mm or whatever. It's just an example.
--
Do not stop an army on its way home.
-- Sun Tzu
next prev parent reply other threads:[~2003-04-14 18:47 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-04-07 12:22 Michael Lampe
2003-04-12 7:13 ` Ken Brownfield
2003-04-12 9:54 ` Marc-Christian Petersen
2003-04-14 13:34 ` James Bourne
2003-04-14 13:46 ` Jörn Engel
2003-04-14 13:56 ` James Bourne
2003-04-14 14:22 ` Jörn Engel
2003-04-14 14:31 ` Martin Schlemmer
2003-04-14 14:47 ` Jörn Engel
2003-04-14 17:09 ` James Bourne
2003-04-14 17:21 ` Jörn Engel
2003-04-14 18:16 ` James Bourne
2003-04-14 17:50 ` Russell King
2003-04-14 18:10 ` Martin Schlemmer
2003-04-14 18:58 ` Jörn Engel [this message]
2003-04-14 19:20 ` Valdis.Kletnieks
2003-04-14 21:19 ` James Bourne
2003-04-14 21:41 ` Jörn Engel
2003-04-15 6:10 ` Martin Schlemmer
2003-04-15 6:00 ` Martin Schlemmer
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=20030414185806.GA12740@wohnheim.fh-wedel.de \
--to=joern@wohnheim.fh-wedel.de \
--cc=azarah@gentoo.org \
--cc=brownfld@irridia.com \
--cc=jbourne@hardrock.org \
--cc=linux-kernel@vger.kernel.org \
--cc=m.c.p@wolk-project.de \
/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
all inboxes | Powered by JetHome®