netdev - Re: [git patches] net driver updates for .26

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]

Date:	Wed, 26 Mar 2008 00:07:21 -0400
From:	Jeff Garzik <jeff@...zik.org>
To:	David Miller <davem@...emloft.net>
CC:	netdev@...r.kernel.org, Chris Snook <csnook@...hat.com>,
	Jay Cliburn <jacliburn@...lsouth.net>
Subject: Re: [git patches] net driver updates for .26

David Miller wrote:
> From: Jeff Garzik <jeff@...zik.org>
> Date: Tue, 18 Mar 2008 03:59:02 -0400
> 
>> Just sent this upstream to davem for 2.6.26...  snipped the patch I
>> usually post along with a git push, due to mailing list restrictions.
>>
>> Please pull from 'upstream-net26' branch of
>> master.kernel.org:/pub/scm/linux/kernel/git/jgarzik/netdev-2.6.git upstream-net26
> 
> This fails an allmodconfig build:
> 
> drivers/net/8390.c: In function '__inittest':
> drivers/net/8390.c:60: error: 'ns8390_init_module' undeclared (first use in this function)
> drivers/net/8390.c:60: error: (Each undeclared identifier is reported only once
> drivers/net/8390.c:60: error: for each function it appears in.)
> 
> The fix was easy so I took care of it, this time...

hrm, indeed.  sorry about that.  Something clearly screwed up in my 
process, because part of that process is an x86-64 allmodconfig before 
pushing.


> There was also a merge conflict in drivers/s390/net/qeth_main.c
> (against a net-2.6.26 change, which is why you really should work
> against that for 2.6.26 bound changes not against Linus's head as you
> did here), which is easy to resolve by putting the conflicting change
> to qeth_main.c into the inetdev notifier in qeth_l3_main.c, which
> I took care of as well.
> 
> This time....

I mentioned this one on IRC ahead of time, when I asked if it was OK, as 
an exception, to base off Linus's HEAD.  This was because I had started 
my for-2.6.26 queue a while before you did.

And just to state the obvious, unless you and I agree on an exception 
(which I thought we had done this time), all changes from me will be 
based on top of your repo(s) rather than Linus's.


> There are also some new build warnings such as:
> 
> drivers/net/atlx/atl1.c:506: warning: 'atl1_phy_enter_power_saving' defined but not used

<checks>  doesn't appear in allmodconfig, so it looks like 
atl1_phy_enter_power_saving() just needs some CONFIG_PM wrappers

(hi Jay, Chris)


> Which I fixed up as well, but that atl1_phy_enter_power_saving()
> function is just a jumble of commented out code, yuck!  Please
> review some of this stuff :-/

I did not see anything wrong with a maintainer leaving in some 
commented-out placeholder code.

Some of this stuff should get filled in once Chris and Jay add atl2 
support to the existing atl1 stuff you see here.

That was a big impetus for this set of atl1 changes -- move things 
around and massage things so that it is trivial to drop in atl2 support.

	Jeff


--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html