From patchwork Wed Jan 20 20:45:06 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: stephen hemminger X-Patchwork-Id: 43371 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 2C165B7D43 for ; Thu, 21 Jan 2010 08:23:59 +1100 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754698Ab0ATVXA (ORCPT ); Wed, 20 Jan 2010 16:23:00 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753765Ab0ATVW7 (ORCPT ); Wed, 20 Jan 2010 16:22:59 -0500 Received: from suva.vyatta.com ([76.74.103.44]:53912 "EHLO suva.vyatta.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754594Ab0ATVW4 (ORCPT ); Wed, 20 Jan 2010 16:22:56 -0500 X-Greylist: delayed 1910 seconds by postgrey-1.27 at vger.kernel.org; Wed, 20 Jan 2010 16:22:54 EST Received: from suva.vyatta.com (suva [127.0.0.1]) by suva.vyatta.com (8.13.7/8.13.7) with ESMTP id o0KKpo5G001011; Wed, 20 Jan 2010 12:51:50 -0800 Received: (from shemminger@localhost) by suva.vyatta.com (8.13.7/8.13.7/Submit) id o0KKpjeb000985; Wed, 20 Jan 2010 12:51:45 -0800 Message-Id: <20100120204559.246153541@vyatta.com> References: <20100120204459.820265084@vyatta.com> User-Agent: quilt/0.46-1 Date: Wed, 20 Jan 2010 12:45:06 -0800 From: Stephen Hemminger To: David Miller , Jarek Poplawski Cc: netdev@vger.kernel.org Subject: [PATCH 07/11] sungem: fix DMA sync_single length error Content-Disposition: inline; filename=sungem.patch Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org The DMA api requires that the full mapping be sync'd when copying frame. First found by Jarek on sky2. Signed-off-by: Stephen Hemminger --- a/drivers/net/sungem.c 2010-01-20 11:43:58.779387706 -0800 +++ b/drivers/net/sungem.c 2010-01-20 11:52:50.398529616 -0800 @@ -847,9 +847,13 @@ static int gem_rx(struct gem *gp, int wo skb_reserve(copy_skb, 2); skb_put(copy_skb, len); - pci_dma_sync_single_for_cpu(gp->pdev, dma_addr, len, PCI_DMA_FROMDEVICE); + pci_dma_sync_single_for_cpu(gp->pdev, dma_addr, + RX_BUF_ALLOC_SIZE(gp), + PCI_DMA_FROMDEVICE); skb_copy_from_linear_data(skb, copy_skb->data, len); - pci_dma_sync_single_for_device(gp->pdev, dma_addr, len, PCI_DMA_FROMDEVICE); + pci_dma_sync_single_for_device(gp->pdev, dma_addr, + RX_BUF_ALLOC_SIZE(gp), + PCI_DMA_FROMDEVICE); /* We'll reuse the original ring buffer. */ skb = copy_skb;