af_packet: Don't use skb after dev_queue_xmit()
in [Kernel]
|
Prev: New HID device Philips Remote RC 153_Vista
Next: tpm_infineon: Fix suspend/resume handler for pnp_driver
From: Jarek Poplawski on 19 Jan 2010 05:50 On Tue, Jan 19, 2010 at 12:46:24AM -0500, Michael Breuer wrote: > Ok - one last update for a while ...not sure what's next... I put some > printk's into sky2.c xmit logic - the packets are being sent to the .... Btw, could you try if this patch can makes difference in triggering the "lib/dma-debug.c:898" warning? Jarek P. --- drivers/net/sky2.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/drivers/net/sky2.c b/drivers/net/sky2.c index 7650f73..e02e9e9 100644 --- a/drivers/net/sky2.c +++ b/drivers/net/sky2.c @@ -2479,6 +2479,9 @@ static int sky2_status_intr(struct sky2_hw *hw, int to_do, u16 idx) port = le->css & CSS_LINK_BIT; dev = hw->dev[port]; + if (!netif_running(dev)) + continue; + sky2 = netdev_priv(dev); length = le16_to_cpu(le->length); status = le32_to_cpu(le->status); -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Michael Breuer on 19 Jan 2010 10:30 On 1/19/2010 3:41 AM, Jarek Poplawski wrote: > On Tue, Jan 19, 2010 at 12:46:24AM -0500, Michael Breuer wrote: > >> So net-net, as far as my TX packet loss issue, sky2 is in the clear. If >> something on the linux side should be informing the switch about >> something then there may still be an issue. If the wifi router should be >> doing something differently, then it's unfortunately likely a 2.4.37 >> kernel issue (That's what dd-wrt is using). >> > IMHO until there is no proof from a sniffer or some regs dumps the > switch and the router are more suspicious than your NIC or linux box. > Then debugging these other things isn't so much interesting from my > POV ;-) > > Anyway, if you only want to get it working (instead of debugging), > it seems you might try moving the dhcp server to the router or maybe > even using two separate servers with their pools - unless I missed > something in your config. > > Jarek P. > I'm going to take a stab at debugging before creating a new subnet. Thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Michael Breuer on 19 Jan 2010 10:50 On 1/19/2010 5:47 AM, Jarek Poplawski wrote: > On Tue, Jan 19, 2010 at 12:46:24AM -0500, Michael Breuer wrote: > >> Ok - one last update for a while ...not sure what's next... I put some >> printk's into sky2.c xmit logic - the packets are being sent to the >> > ... > > Btw, could you try if this patch can makes difference in triggering > the "lib/dma-debug.c:898" warning? > > Jarek P. > --- > > drivers/net/sky2.c | 3 +++ > 1 files changed, 3 insertions(+), 0 deletions(-) > > diff --git a/drivers/net/sky2.c b/drivers/net/sky2.c > index 7650f73..e02e9e9 100644 > --- a/drivers/net/sky2.c > +++ b/drivers/net/sky2.c > @@ -2479,6 +2479,9 @@ static int sky2_status_intr(struct sky2_hw *hw, int to_do, u16 idx) > > port = le->css& CSS_LINK_BIT; > dev = hw->dev[port]; > + if (!netif_running(dev)) > + continue; > + > sky2 = netdev_priv(dev); > length = le16_to_cpu(le->length); > status = le32_to_cpu(le->status); > Still get the warning... but now 60 bytes. Jan 19 10:43:50 mail kernel: ------------[ cut here ]------------ Jan 19 10:43:50 mail kernel: WARNING: at lib/dma-debug.c:902 check_sync+0xc1/0x43f() Jan 19 10:43:50 mail kernel: Hardware name: System Product Name Jan 19 10:43:50 mail kernel: sky2 0000:04:00.0: DMA-API: device driver tries to sync DMA memory it has not allocated [device address=0x0000000320a0b022] [size=60 bytes] Jan 19 10:43:50 mail kernel: Modules linked in: microcode(+) ip6table_filter ip6table_mangle ip6_tables iptable_raw iptable_mangle ipt_MASQUERADE iptable_nat nf_nat appletalk psnap llc nfsd lockd nfs_acl auth_rpcgss exportfs hwmon_vid coretemp sunrpc acpi_cpufreq sit tunnel4 ipt_LOG nf_conntrack_netbios_ns nf_conntrack_ftp xt_DSCP xt_dscp xt_MARK nf_conntrack_ipv6 xt_multiport ipv6 dm_multipath kvm_intel kvm snd_hda_codec_analog snd_ens1371 gameport snd_rawmidi snd_ac97_codec snd_hda_intel ac97_bus snd_hda_codec snd_hwdep snd_seq snd_seq_device snd_pcm gspca_spca505 snd_timer gspca_main firewire_ohci snd soundcore videodev i2c_i801 firewire_core v4l1_compat snd_page_alloc v4l2_compat_ioctl32 pcspkr wmi crc_itu_t iTCO_wdt iTCO_vendor_support asus_atk0110 sky2 hwmon fbcon tileblit font bitblit softcursor raid456 async_raid6_recov async_pq raid6_pq async_xor xor async_memcpy async_tx raid1 ata_generic pata_acpi pata_marvell nouveau ttm drm_kms_helper drm agpgart fb i2c_algo_bit cfbcopyarea i2c_core cfbimgblt cfbf Jan 19 10:43:50 mail kernel: illrect [last unloaded: ip6_tables] Jan 19 10:43:50 mail kernel: Pid: 0, comm: swapper Not tainted 2.6.32.4MMAPNODMARAF3SKY2PSKBMAYPULL-00912-g914160d-dirty #3 Jan 19 10:43:50 mail kernel: Call Trace: Jan 19 10:43:50 mail kernel: <IRQ> [<ffffffff810536ee>] warn_slowpath_common+0x7c/0x94 Jan 19 10:43:50 mail kernel: [<ffffffff8105375d>] warn_slowpath_fmt+0x41/0x43 Jan 19 10:43:50 mail kernel: [<ffffffff8127b871>] check_sync+0xc1/0x43f Jan 19 10:43:50 mail kernel: [<ffffffff8107a683>] ? timekeeping_get_ns+0x1b/0x3d Jan 19 10:43:50 mail kernel: [<ffffffff813c6770>] ? __netdev_alloc_skb+0x34/0x50 Jan 19 10:43:50 mail kernel: [<ffffffff8127bf42>] debug_dma_sync_single_for_cpu+0x42/0x44 Jan 19 10:43:50 mail kernel: [<ffffffff812792c7>] ? swiotlb_sync_single+0x2a/0xb6 Jan 19 10:43:50 mail kernel: [<ffffffff81279423>] ? swiotlb_sync_single_for_cpu+0xc/0xe Jan 19 10:43:50 mail kernel: [<ffffffffa0155ee0>] sky2_poll+0x4d0/0xaeb [sky2] Jan 19 10:43:50 mail kernel: [<ffffffff813cd40e>] net_rx_action+0xb5/0x1f3 Jan 19 10:43:50 mail kernel: [<ffffffff8105af0f>] __do_softirq+0xf8/0x1cd Jan 19 10:43:50 mail kernel: [<ffffffff810a3006>] ? handle_IRQ_event+0x119/0x12b Jan 19 10:43:50 mail kernel: [<ffffffff81012e1c>] call_softirq+0x1c/0x30 Jan 19 10:43:50 mail kernel: [<ffffffff810143a3>] do_softirq+0x4b/0xa6 Jan 19 10:43:50 mail kernel: [<ffffffff8105aaef>] irq_exit+0x4a/0x8c Jan 19 10:43:50 mail kernel: [<ffffffff8146c095>] do_IRQ+0xa5/0xbc Jan 19 10:43:50 mail kernel: [<ffffffff81012613>] ret_from_intr+0x0/0x16 Jan 19 10:43:50 mail kernel: <EOI> [<ffffffff812c2f6e>] ? acpi_idle_enter_bm+0x256/0x28a Jan 19 10:43:50 mail kernel: [<ffffffff812c2f67>] ? acpi_idle_enter_bm+0x24f/0x28a Jan 19 10:43:50 mail kernel: [<ffffffff813a279c>] ? cpuidle_idle_call+0x9e/0xfa Jan 19 10:43:50 mail kernel: [<ffffffff81010c90>] ? cpu_idle+0xb4/0xf6 Jan 19 10:43:50 mail kernel: [<ffffffff814616d2>] ? start_secondary+0x201/0x242 Jan 19 10:43:50 mail kernel: ---[ end trace 310dbc8966610916 ]--- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Jarek Poplawski on 19 Jan 2010 15:00 On Tue, Jan 19, 2010 at 10:47:27AM -0500, Michael Breuer wrote: .... > Still get the warning... but now 60 bytes. > Jan 19 10:43:50 mail kernel: ------------[ cut here ]------------ > Jan 19 10:43:50 mail kernel: WARNING: at lib/dma-debug.c:902 Thanks for trying this. Jarek P. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Michael Breuer on 19 Jan 2010 15:10
On 1/19/2010 2:59 PM, Jarek Poplawski wrote: > On Tue, Jan 19, 2010 at 10:47:27AM -0500, Michael Breuer wrote: > ... > >> Still get the warning... but now 60 bytes. >> Jan 19 10:43:50 mail kernel: ------------[ cut here ]------------ >> Jan 19 10:43:50 mail kernel: WARNING: at lib/dma-debug.c:902 >> > Thanks for trying this. > Jarek P. > NP. Let me know what else I can do to help with this one. Note that it seems innocuous. I'll continue with the older set and no dmar for now and see if I get another crash. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ |