Aren/NetBSD
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use a mutex instead of splvm() to protect the list of jubo-ready mbufs, as

Browse Source 
done with nfe(4) a while ago.

Issue reported by Gary Duzan, who kindly fixed the patch I had sent him.
This commit is contained in:
cube 2008-06-20 16:28:36 +00:00
parent 5e865a2f1b
commit 0ec953c973
2 changed files with 14 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
sys/dev/pci/if_sk.c
View File

@ -1,4 +1,4 @@
/* $NetBSD: if_sk.c,v 1.49 2008/04/28 20:23:55 martin Exp $ */
/* $NetBSD: if_sk.c,v 1.50 2008/06/20 16:28:36 cube Exp $ */
/*-
* Copyright (c) 2003 The NetBSD Foundation, Inc.
@ -115,7 +115,7 @@
*/
#include <sys/cdefs.h>
__KERNEL_RCSID(0, "$NetBSD: if_sk.c,v 1.49 2008/04/28 20:23:55 martin Exp $");
__KERNEL_RCSID(0, "$NetBSD: if_sk.c,v 1.50 2008/06/20 16:28:36 cube Exp $");
#include "bpfilter.h"
#include "rnd.h"
@ -125,6 +125,7 @@ __KERNEL_RCSID(0, "$NetBSD: if_sk.c,v 1.49 2008/04/28 20:23:55 martin Exp $");
#include <sys/sockio.h>
#include <sys/mbuf.h>
#include <sys/malloc.h>
#include <sys/mutex.h>
#include <sys/kernel.h>
#include <sys/socket.h>
#include <sys/device.h>
@ -858,6 +859,7 @@ sk_alloc_jumbo_mem(struct sk_if_softc *sc_if)
LIST_INIT(&sc_if->sk_jfree_listhead);
LIST_INIT(&sc_if->sk_jinuse_listhead);
mutex_init(&sc_if->sk_jpool_mtx, MUTEX_DEFAULT, IPL_NET);
/*
* Now divide it up into 9K pieces and save the addresses
@ -912,13 +914,17 @@ sk_jalloc(struct sk_if_softc *sc_if)
{
struct sk_jpool_entry *entry;
mutex_enter(&sc_if->sk_jpool_mtx);
entry = LIST_FIRST(&sc_if->sk_jfree_listhead);
if (entry == NULL)
if (entry == NULL) {
mutex_exit(&sc_if->sk_jpool_mtx);
return NULL;
}
LIST_REMOVE(entry, jpool_entries);
LIST_INSERT_HEAD(&sc_if->sk_jinuse_listhead, entry, jpool_entries);
mutex_exit(&sc_if->sk_jpool_mtx);
return sc_if->sk_cdata.sk_jslots[entry->slot];
}
@ -930,7 +936,7 @@ sk_jfree(struct mbuf *m, void *buf, size_t size, void *arg)
{
struct sk_jpool_entry *entry;
struct sk_if_softc *sc;
int i, s;
int i;
/* Extract the softc struct pointer. */
sc = (struct sk_if_softc *)arg;
@ -946,17 +952,17 @@ sk_jfree(struct mbuf *m, void *buf, size_t size, void *arg)
if ((i < 0) || (i >= SK_JSLOTS))
panic("sk_jfree: asked to free buffer that we don't manage!");
s = splvm();
mutex_enter(&sc->sk_jpool_mtx);
entry = LIST_FIRST(&sc->sk_jinuse_listhead);
if (entry == NULL)
panic("sk_jfree: buffer not in use!");
entry->slot = i;
LIST_REMOVE(entry, jpool_entries);
LIST_INSERT_HEAD(&sc->sk_jfree_listhead, entry, jpool_entries);
mutex_exit(&sc->sk_jpool_mtx);
if (__predict_true(m != NULL))
pool_cache_put(mb_cache, m);
splx(s);
}
/*

3
sys/dev/pci/if_skvar.h
View File

@ -1,4 +1,4 @@
/* $NetBSD: if_skvar.h,v 1.13 2008/04/28 20:23:55 martin Exp $ */
/* $NetBSD: if_skvar.h,v 1.14 2008/06/20 16:28:36 cube Exp $ */
/*-
* Copyright (c) 2003 The NetBSD Foundation, Inc.
@ -232,6 +232,7 @@ struct sk_if_softc {
struct sk_softc *sk_softc; /* parent controller */
int sk_tx_bmu; /* TX BMU register */
int sk_if_flags;
kmutex_t sk_jpool_mtx;
LIST_HEAD(__sk_jfreehead, sk_jpool_entry) sk_jfree_listhead;
LIST_HEAD(__sk_jinusehead, sk_jpool_entry) sk_jinuse_listhead;
SIMPLEQ_HEAD(__sk_txmaphead, sk_txmap_entry) sk_txmap_head;