netdev - [linux-next-20121127 pcnet32] warning at check

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-next>] [day] [month] [year] [list]

Message-Id: <201211282325.BHI28612.SHJtOLOMOFFFVQ@I-love.SAKURA.ne.jp>
Date:	Wed, 28 Nov 2012 23:25:10 +0900
From:	Tetsuo Handa <penguin-kernel@...ove.SAKURA.ne.jp>
To:	pcnet32@...ntier.com
Cc:	netdev@...r.kernel.org
Subject: [linux-next-20121127 pcnet32] warning at check_unmap()

dma_mapping_error() check seems to be missing at

  static netdev_tx_t pcnet32_start_xmit(struct sk_buff *skb,
                                        struct net_device *dev)
  {
(...snipped...)
  	lp->tx_skbuff[entry] = skb;
  	lp->tx_dma_addr[entry] =
  	    pci_map_single(lp->pci_dev, skb->data, skb->len, PCI_DMA_TODEVICE);
// this line
  	lp->tx_ring[entry].base = cpu_to_le32(lp->tx_dma_addr[entry]);
  	wmb();                  /* Make sure owner changes after all others are visible */
  	lp->tx_ring[entry].status = cpu_to_le16(status);
(...snipped...)
  }

but I don't know how to fix.

[   20.908272] NET: Registered protocol family 10
[   21.627711] pcnet32 0000:02:01.0 eth0: link up
[   22.573436] ------------[ cut here ]------------
[   22.574952] WARNING: at lib/dma-debug.c:933 check_unmap+0x4a6/0x920()
[   22.575669] Hardware name: VMware Virtual Platform
[   22.575669] pcnet32 0000:02:01.0: DMA-API: device driver failed to check map error[device address=0x000000001a3607f2] [size=90 bytes] [mapped as single]
[   22.575669] Modules linked in: ipv6 binfmt_misc
[   22.575669] Pid: 0, comm: swapper/1 Not tainted 3.7.0-rc7-next-20121127 #91
[   22.575669] Call Trace:
[   22.575669]  [<c103370d>] warn_slowpath_common+0x6d/0xa0
[   22.575669]  [<c1249b46>] ? check_unmap+0x4a6/0x920
[   22.575669]  [<c1249b46>] ? check_unmap+0x4a6/0x920
[   22.575669]  [<c10337be>] warn_slowpath_fmt+0x2e/0x30
[   22.575669]  [<c1249b46>] check_unmap+0x4a6/0x920
[   22.575669]  [<c124a1bb>] debug_dma_unmap_page+0x5b/0x80
[   22.575669]  [<c13199be>] pcnet32_poll+0x45e/0x850
[   22.575669]  [<c105dc30>] ? __wake_up+0x40/0x50
[   22.575669]  [<c138dc70>] net_rx_action+0x90/0x140
[   22.575669]  [<c103b6f8>] __do_softirq+0x98/0x160
[   22.575669]  [<c103b660>] ? irq_enter+0x60/0x60
[   22.575669]  <IRQ>  [<c103b52d>] ? irq_exit+0x7d/0x80
[   22.575669]  [<c1023cb4>] ? smp_apic_timer_interrupt+0x54/0x90
[   22.575669]  [<c1230fb4>] ? trace_hardirqs_off_thunk+0xc/0x18
[   22.575669]  [<c144c47e>] ? apic_timer_interrupt+0x32/0x38
[   22.575669]  [<c100a071>] ? default_idle+0x21/0x40
[   22.575669]  [<c100a729>] ? cpu_idle+0x69/0xc0
[   22.575669]  [<c1444e7e>] ? start_secondary+0x1b5/0x1bc
[   22.575669] ---[ end trace d906affb5c70936d ]---
[   22.575669] Mapped at:
[   22.575669]  [<c124a4c8>] debug_dma_map_page+0x68/0x140
[   22.575669]  [<c131acbb>] pcnet32_start_xmit+0x12b/0x220
[   22.575669]  [<c138bea6>] dev_hard_start_xmit+0x1d6/0x3d0
[   22.575669]  [<c13a3bc9>] sch_direct_xmit+0x99/0x240
[   22.575669]  [<c138e003>] dev_queue_xmit+0x143/0x5f0
--
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