summaryrefslogtreecommitdiffstats
path: root/drivers/staging/brcm80211/util/bcmutils.c
diff options
context:
space:
mode:
authorArend van Spriel <arend@broadcom.com>2011-03-02 21:18:46 +0100
committerGreg Kroah-Hartman <gregkh@suse.de>2011-03-02 19:48:05 -0500
commita30825a3c121a4bb4dd84a9522c875fb27172d9d (patch)
tree6efd37bc8892dd8d74adc7da820688d7123f611d /drivers/staging/brcm80211/util/bcmutils.c
parent3c4d93d42df252cb35f8f2cdec1671b14c685ca4 (diff)
downloadop-kernel-dev-a30825a3c121a4bb4dd84a9522c875fb27172d9d.zip
op-kernel-dev-a30825a3c121a4bb4dd84a9522c875fb27172d9d.tar.gz
staging: brcm80211: remove counting of allocated sk_buff packets
The function pkt_buf_get_skb and pkt_buf_free_skb were using struct osl_info field pktalloced to maintain counter of buffers in use in the driver. It was decided to remove this facility. The prototypes of these functions have been modified and the calling code adapted. Reviewed-by: Brett Rudley <brudley@broadcom.com> Reviewed-by: Henry Ptasinski <henryp@broadcom.com> Reviewed-by: Roland Vossen <rvossen@broadcom.com> Signed-off-by: Arend van Spriel <arend@broadcom.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/staging/brcm80211/util/bcmutils.c')
-rw-r--r--drivers/staging/brcm80211/util/bcmutils.c12
1 files changed, 4 insertions, 8 deletions
diff --git a/drivers/staging/brcm80211/util/bcmutils.c b/drivers/staging/brcm80211/util/bcmutils.c
index d066ed7..5bb4770 100644
--- a/drivers/staging/brcm80211/util/bcmutils.c
+++ b/drivers/staging/brcm80211/util/bcmutils.c
@@ -33,7 +33,7 @@
/* Global ASSERT type flag */
u32 g_assert_type;
-struct sk_buff *BCMFASTPATH pkt_buf_get_skb(struct osl_info *osh, uint len)
+struct sk_buff *BCMFASTPATH pkt_buf_get_skb(uint len)
{
struct sk_buff *skb;
@@ -41,16 +41,13 @@ struct sk_buff *BCMFASTPATH pkt_buf_get_skb(struct osl_info *osh, uint len)
if (skb) {
skb_put(skb, len);
skb->priority = 0;
-
- osh->pktalloced++;
}
return skb;
}
/* Free the driver packet. Free the tag if present */
-void BCMFASTPATH pkt_buf_free_skb(struct osl_info *osh,
- struct sk_buff *skb, bool send)
+void BCMFASTPATH pkt_buf_free_skb(struct sk_buff *skb)
{
struct sk_buff *nskb;
int nest = 0;
@@ -73,7 +70,6 @@ void BCMFASTPATH pkt_buf_free_skb(struct osl_info *osh,
*/
dev_kfree_skb(skb);
- osh->pktalloced--;
nest++;
skb = nskb;
}
@@ -245,7 +241,7 @@ void pktq_pflush(struct osl_info *osh, struct pktq *pq, int prec, bool dir)
while (p) {
q->head = p->prev;
p->prev = NULL;
- pkt_buf_free_skb(osh, p, dir);
+ pkt_buf_free_skb(p);
q->len--;
pq->len--;
p = q->head;
@@ -279,7 +275,7 @@ pktq_pflush(struct osl_info *osh, struct pktq *pq, int prec, bool dir,
else
prev->prev = p->prev;
p->prev = NULL;
- pkt_buf_free_skb(osh, p, dir);
+ pkt_buf_free_skb(p);
q->len--;
pq->len--;
p = (head ? q->head : prev->prev);
OpenPOWER on IntegriCloud