git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Sean <seanlkml@sympatico.ca>
To: Larry Finger <larry.finger@lwfinger.net>
Cc: git@vger.kernel.org
Subject: Re: Problem with bisect
Date: Mon, 6 Aug 2007 14:12:33 -0400	[thread overview]
Message-ID: <20070806141233.3d8a43cc.seanlkml@sympatico.ca> (raw)
In-Reply-To: <46B623D6.7070809@lwfinger.net>

On Sun, 05 Aug 2007 14:24:06 -0500
Larry Finger <larry.finger@lwfinger.net> wrote:

> Sean wrote:
> > On Sun, 05 Aug 2007 11:02:21 -0500
> > Larry Finger <Larry.Finger@lwfinger.net> wrote:
> > 
> >> I'm helping someone find what looks like a regression in bcm43xx-mac80211 between v2.6.22 and 
> >> v2.6.23-rc1. This driver is not in the mainstream kernel, but is found in John Linville's 
> >> wireless-dev git tree. When we do the first bisection between the current state and v2.6.22, we 
> >> obtain a kernel whose Makefile says it is v2.6.22; however, it's code is based on a state before 
> >> bcm43xx-mac80211 was introduced into this tree. My memory isn't what it used to be, but I think this 
> >> code was put into this tree during 2.6.19 or .20. When I used visualize to see the tree, the bottom 
> >> is all the way to v2.6.16, which I think is the origin of the git process.
> >>
> >> Is this a git bug, or is it some flaw in this particular tree? We have worked around the problem by 
> >> arbitrarily calling each bisection that does not have the bcm43xx-mac80211 code as "good". It has 
> >> been a source of confusion for the guy I'm helping as it is his first bisection. Unfortunately, the 
> >> bug doesn't show on my machine.
> >>
> The git repo is git://git.kernel.org/pub/scm/linux/kernel/git/linville/wireless-dev.git.
> 
> The commands were:
> 
> git bisect start
> git bisect bad
> git bisect good v2.6.22
> 
> I'm using git version 1.4.4.2.g04509

The directory "drivers/net/wireless/bcm43xx-mac80211" is only introduced in
commit v2.6.23-rc1-1621-gd05daff.   It didn't exist in v2.6.22.

You can see this with the command:

  $ git log -- drivers/net/wireless/bcm43xx-mac80211

Where the last listed commit is d05daff.  So of course there will be many
bisection points back to v2.6.22 where that directory just doesn't exist.
A bit of digging with Git shows this history for most of the files in
that directory:

  renamed in v2.6.23-rc1-1621 as bcm43xx-mac80211
  renamed in v2.6.21-rc1-809 as mac80211
  renamed in v2.6.17-rc2-357 as d80211/bcm43xx
 Imported in v2.6.16-1725 as bcm43xx-d80211

HTH,
Sean

  parent reply	other threads:[~2007-08-06 18:13 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-08-05 16:02 Problem with bisect Larry Finger
     [not found] ` <20070805145234.554bf671.seanlkml@sympatico.ca>
2007-08-05 19:24   ` Larry Finger
2007-08-05 19:44     ` Thomas Glanzmann
2007-08-05 20:33       ` Larry Finger
2007-08-06 18:12     ` Sean [this message]
2007-08-06 18:37       ` Larry Finger
2007-08-07  1:50 ` Christian Couder
2007-08-07  2:53   ` Larry Finger
2007-08-07  5:26     ` Christian Couder

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=20070806141233.3d8a43cc.seanlkml@sympatico.ca \
    --to=seanlkml@sympatico.ca \
    --cc=git@vger.kernel.org \
    --cc=larry.finger@lwfinger.net \
    /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;
as well as URLs for NNTP newsgroup(s).