From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Dirk Hohndel <hohndel@infradead.org>, Len Brown <lenb@kernel.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: Linux 2.6.32-rc3
Date: Wed, 07 Oct 2009 08:33:23 +1100 [thread overview]
Message-ID: <1254864803.6035.25.camel@pasglop> (raw)
In-Reply-To: <alpine.LFD.2.01.0910060705240.3432@localhost.localdomain>
>
> It doesn't help, for several reasons:
>
> - the step function of the Makefile change happens once per release, and
> if you compile anything but releases, you can never rely on just the
> revision. Was it a plain -rc, a plain release, or something in
> between? You'll never know, just looking at the 2.6.x.y thing.
>
> In other words, you fundamentally have three choices:
>
> (a) be confused. Adding an "-rc0" won't help. You'll still be confused
> in between releases about exactly what you're running.
Well, thing is, it's not us who are confused in general, it's users who
do reports with confusing versions. Yes, I agree that anything but a
release warrants a proper commit ID, but that's exactly what I asked for
when I wanted -rc0 here :-)
IE. That the -only- kernel that is called "2.6.x" is the release,
everything else has some kind of -rc in front of it, which allows me to
bug the user for a commit ID and know right away that this isn't a
release kernel.
> (b) use CONFIG_LOCALVERSION_AUTO=y
>
> Now, if 'uname -r' says 2.6.31, then you _know_ it's exactly
> 2.6.31 and nothing else. If it's a few commits after 2.6.31, it
> will say something like '2.6.31-1-g752015d', and you know that
> it's one commit after 2.6.31, and you'll know _which_ commit it is!
>
> (c) Don't compile anything but releases.
>
> Those are the choices.
Sure, when doing the stuff ourselves. Again, the problem is user
reports. Being able to distinguish between a 2.6.x "release" kernel and
anything else would be of value, at least to me. For anything else, I
can say "heh, you've been compiling non-release stuff, you should be
able to get me a git commit ID.
Difference boils down to users falling into two categories: Those who
compile non-release stuff, and should be able to figure out what the ID
is or recompile with LOCAL_VERSION set propertly etc... and those who
don't, didn't always compile the kernel themselves, and fortunately in
-most- cases are only running a "release".
> - An even _more_ fundamental reason: Linux development isn't linear.
> There is not one "first commit" after a release, and there never will
> be. Sure, there's a first commit that I do, but that has absolutely
> zero relevance.
it would be easy enough for you to push a change to the Makefile just
after you tag a release and before you merge anything else.
> Learn this. Until you do, you'll be confused, and you'll show your
> confusion by saying "I want a 2.6.n+1-rc0". You'll _also_ show your
> confusion by things like "I was bisecting a bug that happened between
> 2.6.30 and 2.6.31, and suddenly git was asking me to test a kernel that
> said it was version 2.6.29-rc1 - so I stopped bisecting because git was
> confused".
>
> Who was confused? Was it git, or was it the person who thought that the
> Makefile version could be consistent in a non-linear world?
>
> So no. I'm not going to do -rc0. Because doing that is _stupid_. And until
> you understand _why_ it's stupid, it's pointless talking about it, and
> when you _do_ understand that it's stupid, you'll agree with me.
I disagree. I understand the linearity problem. My point isn't about
having the Makefile provide with any kind of precise "pointer" into that
tree for non-release, but really only to differenciate a release from
anything else.
I know for somebody who uses git everyday, the concept of release even
becomes a bit fuzzy, but there's a lot of folks out there who really
don't see anything else :-)
Cheers,
Ben.
> Linus
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
next prev parent reply other threads:[~2009-10-06 21:43 UTC|newest]
Thread overview: 131+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-10-05 0:44 Linux 2.6.32-rc3 Linus Torvalds
2009-10-05 18:55 ` James Cloos
2009-10-06 1:57 ` Len Brown
2009-10-06 2:51 ` Dirk Hohndel
2009-10-06 14:18 ` Linus Torvalds
2009-10-06 14:38 ` Dirk Hohndel
2009-10-06 15:13 ` Linus Torvalds
2009-10-06 15:34 ` Dirk Hohndel
2009-10-06 15:43 ` Linus Torvalds
[not found] ` <4ACBB7D7.10207@urpla.net>
2009-10-06 22:13 ` Linus Torvalds
2009-10-06 16:36 ` Frans Pop
2009-10-07 1:09 ` Bryan Donlan
2009-10-07 5:56 ` Frans Pop
2009-10-06 14:44 ` Ingo Molnar
2009-10-06 15:24 ` Linus Torvalds
2009-10-06 15:36 ` Ingo Molnar
2009-10-06 15:51 ` Linus Torvalds
2009-10-06 16:29 ` Ingo Molnar
2009-10-06 16:35 ` Ingo Molnar
2009-10-06 16:31 ` Linus Torvalds
2009-10-06 16:40 ` Ingo Molnar
2009-10-06 18:12 ` Theodore Tso
2009-10-06 18:24 ` Ingo Molnar
2009-10-06 21:19 ` Stefan Richter
2009-10-06 17:15 ` Stefan Richter
2009-10-06 18:16 ` Ingo Molnar
2009-10-06 17:22 ` Frans Pop
2009-10-06 17:32 ` Linus Torvalds
2009-10-06 18:29 ` Frans Pop
2009-10-07 0:51 ` Florian Mickler
2009-10-06 17:35 ` [patch] kbuild: Improve version string logic Ingo Molnar
2009-10-06 18:37 ` Johannes Berg
2009-10-06 18:49 ` Ingo Molnar
2009-10-06 18:55 ` Johannes Berg
2009-10-06 19:03 ` Theodore Tso
2009-10-06 19:45 ` Frans Pop
2009-10-06 19:48 ` Johannes Berg
2009-10-06 20:25 ` Frans Pop
2009-10-07 2:43 ` David Rientjes
2009-10-12 19:57 ` [PATCH, v2] " Ingo Molnar
2009-10-12 22:04 ` Frans Pop
2009-10-13 7:05 ` Ingo Molnar
2009-10-13 17:51 ` Frans Pop
2009-10-13 18:01 ` Linus Torvalds
2009-10-13 23:59 ` David Rientjes
2009-10-14 6:59 ` Ingo Molnar
2009-10-14 7:24 ` David Rientjes
2009-10-14 7:33 ` Ingo Molnar
2009-10-14 7:42 ` David Rientjes
2009-10-14 23:43 ` Frans Pop
2009-10-15 7:37 ` David Rientjes
2009-10-15 14:13 ` Frans Pop
2009-10-15 20:38 ` David Rientjes
2009-10-15 21:01 ` Frans Pop
2009-10-15 9:03 ` Ingo Molnar
2009-10-15 14:42 ` Frans Pop
2009-10-15 20:45 ` David Rientjes
2009-10-15 8:01 ` David Rientjes
2009-10-15 8:59 ` Ingo Molnar
2009-10-14 21:55 ` Frans Pop
2009-10-13 2:00 ` David Rientjes
2009-10-13 7:07 ` Ingo Molnar
2009-10-13 7:59 ` David Rientjes
2010-06-07 17:18 ` [PATCH, v2] kbuild: Improve version string logic - two for the price of one - No thanks Boaz Harrosh
2010-06-07 19:45 ` David Rientjes
2010-06-08 5:52 ` Boaz Harrosh
2010-06-08 6:18 ` David Rientjes
2010-06-08 6:34 ` Paul Mundt
2010-06-08 6:39 ` Boaz Harrosh
2010-06-08 7:16 ` Boaz Harrosh
2010-06-08 7:21 ` Paul Mundt
2010-06-08 7:21 ` Boaz Harrosh
2010-06-08 7:32 ` Paul Mundt
2010-06-08 7:52 ` Boaz Harrosh
2010-06-08 9:17 ` David Rientjes
2010-06-08 6:37 ` Boaz Harrosh
2010-06-08 8:31 ` kbuild: Fix the breakage caused by "improve version string logic" Boaz Harrosh
2010-06-08 9:13 ` David Rientjes
2010-06-08 10:14 ` Boaz Harrosh
2010-06-08 10:19 ` Boaz Harrosh
2010-06-09 6:55 ` David Rientjes
2010-06-09 7:54 ` Boaz Harrosh
2010-06-09 8:18 ` Mike Galbraith
2009-10-06 17:40 ` Linux 2.6.32-rc3 Len Brown
2009-10-06 18:16 ` Linus Torvalds
2009-10-07 22:33 ` Len Brown
2009-10-06 17:45 ` Dirk Hohndel
2009-10-06 19:22 ` Joel Becker
[not found] ` <4ACB77ED.6060104@grm.uci.cu>
2009-10-06 18:00 ` Herlin R. Matos Lastres
2009-10-15 15:51 ` Frans Pop
2009-10-06 15:42 ` Linus Torvalds
2009-10-06 17:09 ` Frans Pop
2009-10-06 17:34 ` Stefan Richter
2009-10-06 17:41 ` Linus Torvalds
2009-10-06 18:56 ` david
2009-10-06 18:23 ` Frans Pop
2009-10-06 19:23 ` Stefan Richter
2009-10-06 17:44 ` Theodore Tso
2009-10-06 18:14 ` Theodore Tso
2009-10-06 18:20 ` Linus Torvalds
2009-10-06 16:40 ` Frans Pop
2009-10-06 18:35 ` Linus Torvalds
2009-10-06 19:37 ` Frans Pop
2009-10-07 21:39 ` Steven Rostedt
2009-10-08 15:20 ` Frans Pop
2009-10-06 15:29 ` Stefan Richter
2009-10-06 17:08 ` Ingo Molnar
2009-10-06 17:20 ` Stefan Richter
2009-10-06 21:33 ` Benjamin Herrenschmidt [this message]
2009-10-06 22:19 ` Linus Torvalds
2009-10-07 1:22 ` Dave Airlie
2009-10-07 2:31 ` Theodore Tso
2009-10-07 2:45 ` Benjamin Herrenschmidt
2009-10-10 12:09 ` Pavel Machek
2009-10-10 12:18 ` Felipe Contreras
2009-10-07 3:23 ` Linus Torvalds
2009-10-07 3:31 ` Linus Torvalds
2009-10-07 13:52 ` Theodore Tso
2009-10-07 14:52 ` Mike Galbraith
2009-10-07 17:44 ` david
2009-10-07 18:13 ` Mike Galbraith
2009-10-07 4:02 ` Justin P. Mattock
2009-10-07 10:41 ` 2.6.32-rc3: floating-point build failure (undefined reference to `__udivdi3' in menu governor) Andreas Mohr
2009-10-07 14:23 ` Arjan van de Ven
2009-10-07 17:34 ` Andreas Mohr
2009-10-07 17:45 ` Arjan van de Ven
2009-10-07 17:45 ` Kyle McMartin
2009-10-09 16:01 ` Andreas Mohr
2009-10-09 16:32 ` Arjan van de Ven
2009-10-09 17:08 ` Kyle McMartin
2009-10-09 17:12 ` Arjan van de Ven
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=1254864803.6035.25.camel@pasglop \
--to=benh@kernel.crashing.org \
--cc=hohndel@infradead.org \
--cc=lenb@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@linux-foundation.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.