Skip to content

Commit

Permalink
r8169: use netdev_alloc_skb
Browse files Browse the repository at this point in the history
Use netdev_alloc_skb and remove the useless sk_buff * argument of
rtl8169_alloc_rx_skb.

Signed-off-by: Stephen Hemminger <shemminger@linux-foundation.org>
Signed-off-by: Francois Romieu <romieu@fr.zoreil.com>
Cc: Edward Hsu <edward_hsu@realtek.com.tw>
  • Loading branch information
Stephen Hemminger authored and Jeff Garzik committed Jul 9, 2007
1 parent a73b629 commit 15d3175
Showing 1 changed file with 14 additions and 13 deletions.
27 changes: 14 additions & 13 deletions drivers/net/r8169.c
Original file line number Diff line number Diff line change
Expand Up @@ -1997,31 +1997,28 @@ static inline void rtl8169_map_to_asic(struct RxDesc *desc, dma_addr_t mapping,
rtl8169_mark_to_asic(desc, rx_buf_sz);
}

static int rtl8169_alloc_rx_skb(struct pci_dev *pdev, struct sk_buff **sk_buff,
struct RxDesc *desc, int rx_buf_sz,
unsigned int align)
static struct sk_buff *rtl8169_alloc_rx_skb(struct pci_dev *pdev,
struct net_device *dev,
struct RxDesc *desc, int rx_buf_sz,
unsigned int align)
{
struct sk_buff *skb;
dma_addr_t mapping;
int ret = 0;

skb = dev_alloc_skb(rx_buf_sz + align);
skb = netdev_alloc_skb(dev, rx_buf_sz + align);
if (!skb)
goto err_out;

skb_reserve(skb, (align - 1) & (unsigned long)skb->data);
*sk_buff = skb;

mapping = pci_map_single(pdev, skb->data, rx_buf_sz,
PCI_DMA_FROMDEVICE);

rtl8169_map_to_asic(desc, mapping, rx_buf_sz);

out:
return ret;
return skb;

err_out:
ret = -ENOMEM;
rtl8169_make_unusable_by_asic(desc);
goto out;
}
Expand All @@ -2044,15 +2041,19 @@ static u32 rtl8169_rx_fill(struct rtl8169_private *tp, struct net_device *dev,
u32 cur;

for (cur = start; end - cur > 0; cur++) {
int ret, i = cur % NUM_RX_DESC;
struct sk_buff *skb;
unsigned int i = cur % NUM_RX_DESC;

if (tp->Rx_skbuff[i])
continue;

ret = rtl8169_alloc_rx_skb(tp->pci_dev, tp->Rx_skbuff + i,
tp->RxDescArray + i, tp->rx_buf_sz, tp->align);
if (ret < 0)
skb = rtl8169_alloc_rx_skb(tp->pci_dev, dev,
tp->RxDescArray + i,
tp->rx_buf_sz, tp->align);
if (!skb)
break;

tp->Rx_skbuff[i] = skb;
}
return cur - start;
}
Expand Down

0 comments on commit 15d3175

Please sign in to comment.