Re: 2.6.29-rc8: Reported regressions from 2.6.28

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Linus Torvalds
Date: Monday, March 16, 2009 - 12:57 pm

On Mon, 16 Mar 2009, Jeff Chua wrote:

Taking a bisect log is repeatable, but pointless.

If you made any mistakes in bisecting (marking a kernel that was good as 
being bad, or the other way around), the log will always replay to the 
same thing, but it will still be wrong.

In other words, "git bisect" is only as reliable as the data you feed it, 
and if the behavior isn't 100% repeatable and unambiguous (or if you 
simply made a mistake), you need to double-check things.

So after bisecting a commit, if there is any question what-so-ever whether 
the commit makes sense as a result, you need to double-check it. The best 
way to double-check it is to go back to a known-bad state (preferably the 
tip of the branch) and revert the presumed-bad commit, and verify that it 
really fixes the behavior.

But if that is impossible (for example, because the commit no longer 
reverts cleanly), at least make 100% sure that the state at the commit is 
bad, and then go to (all) parents of that commit and make 100% sure that 
the state at those points is _good_. 

IOW, if you've pinpointed 71c11fb57b924c160297ccd9e1761db598d00ac2 as 
being bad, then you should go back and double-check that its parent 
(in this case 4607816f608b42a5379aca97ceed08378804c99f) is good.

Because if it's parent is also bad, then that just means that you made 
some mistake in "git bisect".

The thing about bisecting is that it is _extremely_ efficient. It takes 
essentially the minimal number of answers to get to the end result. But 
that very efficiency also means that getting even just _one_ of those 
answers wrong will take you _way_ off base. There's no room for error, 
because bisect will take each bit and use it to maximally split the error 
space.

In this case, it really sounds like maybe you marked the parent good, even 
though you should have marked it bad.

			Linus
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
2.6.29-rc8: Reported regressions from 2.6.28, Rafael J. Wysocki, (Sat Mar 14, 12:01 pm)
[Bug #12805] QinQ vlan trunking regression, Rafael J. Wysocki, (Sat Mar 14, 12:05 pm)
Re: [Bug #12805] QinQ vlan trunking regression, David Miller, (Sat Mar 14, 3:04 pm)
Re: [Bug #12805] QinQ vlan trunking regression, Rafael J. Wysocki, (Sat Mar 14, 3:26 pm)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Jeff Chua, (Sat Mar 14, 7:58 pm)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Jeff Chua, (Sat Mar 14, 8:06 pm)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Rafael J. Wysocki, (Sun Mar 15, 3:41 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Johannes Berg, (Sun Mar 15, 11:11 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Linus Torvalds, (Sun Mar 15, 11:44 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Johannes Berg, (Sun Mar 15, 12:01 pm)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Ingo Molnar, (Sun Mar 15, 1:26 pm)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Zhang Rui, (Sun Mar 15, 6:02 pm)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Jeff Chua, (Mon Mar 16, 6:24 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Linus Torvalds, (Mon Mar 16, 12:57 pm)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Jeff Chua, (Mon Mar 16, 4:55 pm)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Johannes Berg, (Tue Mar 17, 12:50 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, John W. Linville, (Tue Mar 17, 7:48 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, John W. Linville, (Tue Mar 17, 8:28 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Ingo Molnar, (Tue Mar 17, 8:39 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, John W. Linville, (Tue Mar 17, 9:05 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Jeff Chua, (Tue Mar 17, 9:24 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, John W. Linville, (Tue Mar 17, 10:10 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Jeff Chua, (Tue Mar 17, 10:21 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Jeff Chua, (Tue Mar 17, 10:27 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Jeff Chua, (Tue Mar 17, 10:31 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Jeff Chua, (Tue Mar 17, 11:26 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Johannes Berg, (Tue Mar 17, 12:22 pm)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Jeff Chua, (Wed Mar 18, 7:58 pm)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Jeff Chua, (Wed Mar 18, 8:25 pm)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Jeff Chua, (Wed Mar 18, 9:23 pm)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Jeff Chua, (Wed Mar 18, 9:49 pm)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Johannes Berg, (Thu Mar 19, 2:38 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, John W. Linville, (Thu Mar 19, 7:13 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Johannes Berg, (Thu Mar 19, 9:59 am)
Re: Required sequence to set wireless parameters?, Frans Pop, (Thu Mar 19, 12:24 pm)
Re: Required sequence to set wireless parameters?, Johannes Berg, (Thu Mar 19, 12:27 pm)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Jeff Chua, (Thu Mar 19, 9:55 pm)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Jeff Chua, (Thu Mar 19, 10:20 pm)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Johannes Berg, (Fri Mar 20, 1:32 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Jeff Chua, (Fri Mar 20, 3:04 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Johannes Berg, (Fri Mar 20, 3:13 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Jeff Chua, (Fri Mar 20, 9:14 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Jeff Chua, (Fri Mar 20, 10:19 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Johannes Berg, (Sat Mar 21, 5:09 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Jeff Chua, (Sat Mar 21, 8:08 am)
Re: 2.6.29-rc8: Reported regressions from 2.6.28, Johannes Berg, (Sat Mar 21, 8:11 am)