netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jeff Garzik <jeff@garzik.org>
To: Lennert Buytenhek <buytenh@wantstofly.org>
Cc: netdev@vger.kernel.org, Nicolas Pitre <nico@cam.org>,
	Ashish Karkare <akarkare@marvell.com>,
	Dale Farnsworth <dale@farnsworth.org>
Subject: Re: mv643xx_eth pull request for 2.6.28
Date: Wed, 24 Sep 2008 20:41:14 -0400	[thread overview]
Message-ID: <48DADE2A.3010206@garzik.org> (raw)
In-Reply-To: <20080922031931.GA7678@xi.wantstofly.org>

Lennert Buytenhek wrote:
> Hi Jeff,
> 
> This is the set of mv643xx_eth patches intended for 2.6.28.  It has
> most of the patches posted to the list in the last couple of weeks,
> plus three more fixes:
> - Unless we tell the hardware to include the pseudo-header when
>   checksumming, all packets will be received with the hardware
>   indicating a bad checksum, causing the driver to set CHECKSUM_NONE
>   on all packets, i.e. hardware receive checksum offload won't be
>   used at all.  We need to set the bit that tells the hardware to
>   include the pseudo-header when checksumming.
> - The driver would corrupt the packet on transmit if the L3 header
>   didn't start 14, 18, 22 or 26 bytes into the packet.  Now it'll
>   just call skb_checksum_help() to do checksumming in software if
>   this happens.
> - Due to a hardware bug, the 64 bit hardware byte counters are
>   effectively only 32 bit, which necessitates polling them at least
>   once every ~30 seconds to prevent them from overflowing.
> 
> I've also included the mv643xx_eth phylib conversion, as Andy Fleming
> ACKed it.  The skb recycling bits aren't included (need more discussion),
> and neither are the Pegasos SRAM bits, since I didn't receive any tester
> feedback on them.
> 
> I've tested this series extensively on all mv643xx_eth hardware I have
> available, and it all looks good.  Please pull.
> 
> 
> thanks,
> Lennert
> 
> 
> 
> The following changes since commit 24342c34a022ee90839873d91396045e12ef1090:
>   Linus Torvalds (1):
>         Linux 2.6.27-rc5
> 
> are available in the git repository at:
> 
>   git://git.marvell.com/mv643xx_eth.git for-2.6.28
>   http://git.marvell.com/?p=mv643xx_eth.git;a=shortlog;h=for-2.6.28
> 
> Lennert Buytenhek (28):
>       mv643xx_eth: fix 'netdev_priv(dev) == dev->priv' assumption
>       mv643xx_eth: fix receive buffer DMA unmapping
>       mv643xx_eth: get rid of IRQF_SAMPLE_RANDOM
>       mv643xx_eth: get rid of modulo operations
>       mv643xx_eth: switch ->phy_lock from a spinlock to a mutex
>       mv643xx_eth: use the SMI done interrupt to wait for SMI access completion
>       mv643xx_eth: make napi unconditional
>       mv643xx_eth: get rid of receive-side locking
>       mv643xx_eth: get rid of compile-time configurable transmit checksumming
>       mv643xx_eth: require contiguous receive and transmit queue numbering
>       mv643xx_eth: smi sharing is a per-unit property, not a per-port one
>       mv643xx_eth: remove force_phy_addr field
>       mv643xx_eth: get rid of netif_{stop,wake}_queue() calls on link down/up
>       mv643xx_eth: delete unused and uninteresting interrupt source mask bits
>       mv643xx_eth: transmit multiqueue support
>       mv643xx_eth: move all work to the napi poll handler
>       mv643xx_eth: switch to netif tx queue lock, get rid of private spinlock
>       mv643xx_eth: avoid dropping tx lock during transmit reclaim
>       mv643xx_eth: replace array of skbs awaiting transmit completion with a queue
>       mv643xx_eth: shrink default receive and transmit queue sizes
>       mv643xx_eth: avoid reading ->byte_cnt twice during receive processing
>       mv643xx_eth: add support for chips without transmit bandwidth control
>       mv643xx_eth: fix receive checksumming
>       mv643xx_eth: deal with unexpected ethernet header sizes
>       mv643xx_eth: enforce frequent hardware statistics polling
>       phylib: allow incremental scanning of an mii bus
>       mv643xx_eth: convert to phylib
>       mv643xx_eth: bump version to 1.4
> 
>  arch/arm/mach-kirkwood/db88f6281-bp-setup.c  |    2 +-
>  arch/arm/mach-kirkwood/rd88f6192-nas-setup.c |    2 +-
>  arch/arm/mach-kirkwood/rd88f6281-setup.c     |    2 +-
>  arch/arm/mach-loki/lb88rc8480-setup.c        |    2 +-
>  arch/arm/mach-mv78xx0/common.c               |    6 +-
>  arch/arm/mach-mv78xx0/db78x00-bp-setup.c     |    8 +-
>  arch/arm/mach-orion5x/db88f5281-setup.c      |    2 +-
>  arch/arm/mach-orion5x/dns323-setup.c         |    2 +-
>  arch/arm/mach-orion5x/kurobox_pro-setup.c    |    2 +-
>  arch/arm/mach-orion5x/mss2-setup.c           |    2 +-
>  arch/arm/mach-orion5x/mv2120-setup.c         |    2 +-
>  arch/arm/mach-orion5x/rd88f5181l-fxo-setup.c |    2 +-
>  arch/arm/mach-orion5x/rd88f5181l-ge-setup.c  |    2 +-
>  arch/arm/mach-orion5x/rd88f5182-setup.c      |    2 +-
>  arch/arm/mach-orion5x/ts78xx-setup.c         |    3 +-
>  arch/arm/mach-orion5x/tsx09-common.c         |    2 +-
>  arch/arm/mach-orion5x/wnr854t-setup.c        |    2 +-
>  arch/arm/mach-orion5x/wrt350n-v2-setup.c     |    2 +-
>  arch/powerpc/sysdev/mv64x60_dev.c            |    6 +-
>  drivers/net/Kconfig                          |    2 +-
>  drivers/net/mv643xx_eth.c                    | 1438 +++++++++++++-------------
>  drivers/net/phy/mdio_bus.c                   |   89 +-
>  include/linux/mv643xx_eth.h                  |   13 +-
>  include/linux/phy.h                          |    2 +

pulled



      reply	other threads:[~2008-09-25  0:41 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-09-22  3:19 mv643xx_eth pull request for 2.6.28 Lennert Buytenhek
2008-09-25  0:41 ` Jeff Garzik [this message]

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=48DADE2A.3010206@garzik.org \
    --to=jeff@garzik.org \
    --cc=akarkare@marvell.com \
    --cc=buytenh@wantstofly.org \
    --cc=dale@farnsworth.org \
    --cc=netdev@vger.kernel.org \
    --cc=nico@cam.org \
    /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).