Re: Fix quilt merge error in acpi-cpufreq.c

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Ingo Molnar
Date: Wednesday, April 15, 2009 - 5:08 pm

* Linus Torvalds <torvalds@linux-foundation.org> wrote:


Ok, i see your argument.

Is there any other way for us to get the benefits of the impact 
line, without actually adding one?

They are:

 - Better split-up patches from contributors.

 - Increased maintainer and developer attention on the effects of
   patches.

 - Less time we have to spend on patches we get with impact-lines.
   Both hpa and me reported this.

 - easy regression post-mortems

What i dont understand is how you can dismiss these positive points 
so easily and only concentrate on the negative points - these 
effects are mostly visible to those creating impact lines - not to 
you. You cannot really have seen any of these effects without having 
done impact lines for a few days.

(
  Okay, you are perhaps an exception, you _do_ write fantastic
  commit logs that generally need no 'stinkin impact line.

  There's exceptions though, even with your commits. I wish you had 
  added one to ea34f43a for example, which you committed today.

  It is not clear from that commit log at all what the practical
  relevance of your fix is.

  I suspect it fixes Ali Gholami Rudi's problem of his CPU hitting
  50^C till the fan turns on.

  I'd probably have added the following impact line (although i'd
  have first asked Ali whether this is the precise impact the fix
  had on him - it's not 100% clear from the discussion):

    Impact: fix cpufreq misbehavior causing high CPU temperature

  Does this information matter? I think it does. Does it matter 
  _more_ than the rest of the commit log? I think, to most Linux
  users, it does. That's one of the reasons why we are experimenting 
  with formalized this kind of information. It's important 
  information and it should not be forgotten from commit logs.

  As a developer, while writing up a commit log it is _so_ easy to
  get lost in the details of the 'how' and 'why' - and not emit
  basic information: why do people care? What were the bad
  _practical_ effects of the bug that are fixed here?

  It is basic human nature to get lost in that - even for the best.
)

	Ingo
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: Fix quilt merge error in acpi-cpufreq.c, Ingo Molnar, (Mon Apr 13, 7:05 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Ingo Molnar, (Tue Apr 14, 10:44 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Rusty Russell, (Wed Apr 15, 3:44 am)
Re: Fix quilt merge error in acpi-cpufreq.c, Linus Torvalds, (Wed Apr 15, 8:05 am)
Re: Fix quilt merge error in acpi-cpufreq.c, Ali Gholami Rudi, (Wed Apr 15, 8:22 am)
Re: Fix quilt merge error in acpi-cpufreq.c, Linus Torvalds, (Wed Apr 15, 8:28 am)
Re: Fix quilt merge error in acpi-cpufreq.c, Ingo Molnar, (Wed Apr 15, 9:26 am)
Re: Fix quilt merge error in acpi-cpufreq.c, Ingo Molnar, (Wed Apr 15, 9:41 am)
Re: Fix quilt merge error in acpi-cpufreq.c, H. Peter Anvin, (Wed Apr 15, 9:46 am)
Re: Fix quilt merge error in acpi-cpufreq.c, H. Peter Anvin, (Wed Apr 15, 10:00 am)
Re: Fix quilt merge error in acpi-cpufreq.c, Linus Torvalds, (Wed Apr 15, 10:19 am)
Re: Fix quilt merge error in acpi-cpufreq.c, H. Peter Anvin, (Wed Apr 15, 11:47 am)
Re: Fix quilt merge error in acpi-cpufreq.c, Linus Torvalds, (Wed Apr 15, 12:43 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Ingo Molnar, (Wed Apr 15, 1:07 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Andrew Morton, (Wed Apr 15, 1:32 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Ingo Molnar, (Wed Apr 15, 2:03 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Linus Torvalds, (Wed Apr 15, 2:15 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Andrew Morton, (Wed Apr 15, 2:17 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, David Miller, (Wed Apr 15, 2:23 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Ingo Molnar, (Wed Apr 15, 3:40 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Ingo Molnar, (Wed Apr 15, 3:48 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Ingo Molnar, (Wed Apr 15, 4:04 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Linus Torvalds, (Wed Apr 15, 4:08 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Linus Torvalds, (Wed Apr 15, 4:11 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, David Miller, (Wed Apr 15, 4:49 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Ingo Molnar, (Wed Apr 15, 5:08 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Linus Torvalds, (Wed Apr 15, 5:23 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Linus Torvalds, (Wed Apr 15, 5:38 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Ingo Molnar, (Wed Apr 15, 5:44 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Ingo Molnar, (Wed Apr 15, 5:50 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Linus Torvalds, (Wed Apr 15, 6:03 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Rusty Russell, (Wed Apr 15, 6:27 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Ingo Molnar, (Wed Apr 15, 6:46 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Rusty Russell, (Wed Apr 15, 7:00 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Paul Gortmaker, (Wed Apr 15, 7:22 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Linus Torvalds, (Wed Apr 15, 7:22 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Theodore Tso, (Wed Apr 15, 7:31 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Linus Torvalds, (Wed Apr 15, 7:34 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Ray Lee, (Wed Apr 15, 8:10 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Theodore Tso, (Wed Apr 15, 8:55 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, H. Peter Anvin, (Wed Apr 15, 9:33 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Ingo Molnar, (Thu Apr 16, 12:14 am)
Re: Fix quilt merge error in acpi-cpufreq.c, Ingo Molnar, (Thu Apr 16, 12:23 am)
Re: Fix quilt merge error in acpi-cpufreq.c, Ingo Molnar, (Thu Apr 16, 12:44 am)
Re: Fix quilt merge error in acpi-cpufreq.c, Ingo Molnar, (Thu Apr 16, 12:56 am)
Re: Fix quilt merge error in acpi-cpufreq.c, Ingo Molnar, (Thu Apr 16, 1:02 am)
Re: Fix quilt merge error in acpi-cpufreq.c, Christoph Hellwig, (Thu Apr 16, 4:00 am)
Re: Fix quilt merge error in acpi-cpufreq.c, Theodore Tso, (Thu Apr 16, 4:57 am)
Re: Fix quilt merge error in acpi-cpufreq.c, Valdis.Kletnieks, (Thu Apr 16, 6:04 am)
Re: Fix quilt merge error in acpi-cpufreq.c, Jonathan Corbet, (Thu Apr 16, 6:55 am)
Re: Fix quilt merge error in acpi-cpufreq.c, Valdis.Kletnieks, (Thu Apr 16, 8:24 am)
Re: Fix quilt merge error in acpi-cpufreq.c, Valdis.Kletnieks, (Thu Apr 16, 8:41 am)
Re: Fix quilt merge error in acpi-cpufreq.c, Rusty Russell, (Mon Apr 20, 1:14 am)
Re: Fix quilt merge error in acpi-cpufreq.c, Ingo Molnar, (Mon Apr 20, 3:38 am)
Re: Fix quilt merge error in acpi-cpufreq.c, Jonathan Corbet, (Tue Apr 21, 12:37 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Rusty Russell, (Tue Apr 21, 6:58 pm)
Re: Fix quilt merge error in acpi-cpufreq.c, Rusty Russell, (Tue Apr 21, 9:18 pm)