netdev - Re: Bug#645308: tg3 broken for NetXtreme 5714S in squeeze 6.0.3 installer

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1319614904.11727.17.camel@deadeye>
Date:	Wed, 26 Oct 2011 09:41:44 +0200
From:	Ben Hutchings <ben@...adent.org.uk>
To:	Matt Carlson <mcarlson@...adcom.com>
Cc:	Michael Chan <mchan@...adcom.com>,
	"645308@...s.debian.org" <645308@...s.debian.org>,
	Marc Haber <mh+debian-bugs@...schlus.de>,
	netdev <netdev@...r.kernel.org>
Subject: Re: Bug#645308: tg3 broken for NetXtreme 5714S in squeeze 6.0.3
 installer

On Tue, 2011-10-25 at 17:20 -0700, Matt Carlson wrote:
> On Mon, Oct 24, 2011 at 04:47:54PM -0700, Ben Hutchings wrote:
> > On Mon, 2011-10-24 at 14:24 -0700, Matt Carlson wrote:
> > > On Fri, Oct 21, 2011 at 05:19:39AM -0700, Ben Hutchings wrote:
> > > > On Fri, 2011-10-21 at 11:08 +0200, Marc Haber wrote:
> > > > > On Fri, Oct 21, 2011 at 11:00:46AM +0200, Marc Haber wrote:
> > > > > > On Thu, Oct 20, 2011 at 05:28:34AM +0100, Ben Hutchings wrote:
> > > > > > > I don't see any changes that would obviously change the way this device
> > > > > > > is reconfigured during a down/up cycle.  There were some changes to
> > > > > > > power management that should just let the PCI core do some work that the
> > > > > > > driver used to, but it's possible that the result isn't quite the same.
> > > > > > > I built a module with those reverted; source and binary attached.  Could
> > > > > > > you test that?  I checked that d-i does include an insmod command.
> > > > > > 
> > > > > > The squeeze 6.0.3 installer with the shipped tg3.ko replaced with
> > > > > > yours boots and networks just fine without any workaround and without
> > > > > > manual interaction.
> > > > > 
> > > > > I was a bit fast on that. The interface now fails right in the middle
> > > > > of installation and needs the modprobe -r, modprobe stunt to network
> > > > > again.
> > > > 
> > > > Matt, Michael,
> > > > 
> > > > The tg3 driver has regressed for the 5714S since Linux 2.6.32.  Marc
> > > > Haber found this in the backported version included in our stable
> > > > update, but also confirmed it in Linux 3.0.
> > > > 
> > > > Bringing the interface down and then up again (which the installer does
> > > > for some reason) can leave it unable to pass traffic (possibly after
> > > > working for a few packets) until the module is reloaded.
> > > > 
> > > > I asked Marc to check whether reverting the power management changes
> > > > (071697e2bcd8dff2af4d6fdd6525c2324f89553b,
> > > > d237d9ecf06a00f0ebca657958cf2a1e92940796) made a difference, but it
> > > > doesn't seem to.
> > > > 
> > > > There is more information in the bug log at
> > > > <http://bugs.debian.org/645308>.
> > > 
> > > Where can I get the sources for this driver?  Commit
> > > 9e975cc291d80d5e4562d6bed15ec171e896d69b, entitled
> > > "tg3: Fix io failures after chip reset" has been a common source of
> > > problems.
> > 
> > Our current package has Linux 3.0.6 which includes the backport of that
> > change.  However, it is *not* included in my backport to 2.6.32 so it
> > doesn't explain the original report.
> > 
> > The backported version can be found in:
> > 
> > git://anonscm.debian.org/kernel/linux-2.6.git squeeze
> 
> The kernel version of that repository is 3.0.0-rc1.  Am I looking in the
> right place?

Look at the squeeze branch, not master.

Ben.

> But you're right.  The version of the driver in that repository does not
> have the change.
> 
> 

-- 
Ben Hutchings
Reality is just a crutch for people who can't handle science fiction.

Download attachment "signature.asc" of type "application/pgp-signature" (829 bytes)