netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Ben Hutchings <ben@decadent.org.uk>
To: Stephen Hemminger <shemminger@linux-foundation.org>
Cc: 645589@bugs.debian.org, Antti Salmela <asalmela@iki.fi>,
	netdev <netdev@vger.kernel.org>
Subject: Re: Bug#645589: linux-image-3.0.0-2-amd64: sky2 rx errors on 3.0, 2.6.32 works
Date: Tue, 18 Oct 2011 04:27:55 +0100	[thread overview]
Message-ID: <1318908480.3340.79.camel@deadeye> (raw)
In-Reply-To: <20111017074016.6840.77265.reportbug@thor.viidakko.fi>

[-- Attachment #1: Type: text/plain, Size: 1851 bytes --]

On Mon, 2011-10-17 at 10:40 +0300, Antti Salmela wrote:
> Package: linux-2.6
> Version: 3.0.0-5
> Severity: normal
> 
> 
> sky2 loses packets on 3.0 (-3 and -5) and 3.1-rc7, 2.6.32-38 and
> setting interface to promiscuous works.
> 
> [   60.118244] sky2 0000:02:00.0: eth0: rx error, status 0xb92100 length 185
> [   62.664370] sky2 0000:02:00.0: eth0: rx error, status 0x602100 length 96
> [   63.370051] sky2 0000:02:00.0: eth0: rx error, status 0x422100 length 66
> [   63.714672] sky2 0000:02:00.0: eth0: rx error, status 0x722100 length 114
> [   64.513458] device eth0 entered promiscuous mode

It looks like this is a bug in accounting of VLAN tags, though I don't
see what difference promiscuous mode should make.

The log messages show that status has the VLAN flag (bit 13) set and the
length field (bits 16:28) equals the length passed into sky2_receive(),
but that function expects the length field to be greater by VLAN_HLEN.

This device is:

[...]
> 02:00.0 Ethernet controller [0200]: Marvell Technology Group Ltd. 88E8053 PCI-E Gigabit Ethernet Controller [11ab:4362] (rev 19)
> 	Subsystem: ASUSTeK Computer Inc. Marvell 88E8053 Gigabit Ethernet controller PCIe (Asus) [1043:8142]
> 	Control: I/O+ Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B- DisINTx+
> 	Status: Cap+ 66MHz- UDF- FastB2B- ParErr- DEVSEL=fast >TAbort- <TAbort- <MAbort- >SERR- <PERR- INTx-
> 	Latency: 0, Cache Line Size: 16 bytes
> 	Interrupt: pin A routed to IRQ 43
> 	Region 0: Memory at cdefc000 (64-bit, non-prefetchable) [size=16K]
> 	Region 2: I/O ports at c800 [size=256]
> 	Expansion ROM at cdec0000 [disabled] [size=128K]
> 	Capabilities: <access denied>
> 	Kernel driver in use: sky2
[...]

Ben.

-- 
Ben Hutchings
No political challenge can be met by shopping. - George Monbiot

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 828 bytes --]

       reply	other threads:[~2011-10-18  3:28 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20111017074016.6840.77265.reportbug@thor.viidakko.fi>
2011-10-18  3:27 ` Ben Hutchings [this message]
     [not found] ` <1318909386.3340.91.camel@deadeye>
2011-10-18 18:13   ` Bug#645589: linux-image-3.0.0-2-amd64: sky2 rx errors on 3.0, 2.6.32 works Stephen Hemminger
2011-10-19  4:09     ` Ben Hutchings

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=1318908480.3340.79.camel@deadeye \
    --to=ben@decadent.org.uk \
    --cc=645589@bugs.debian.org \
    --cc=asalmela@iki.fi \
    --cc=netdev@vger.kernel.org \
    --cc=shemminger@linux-foundation.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).