svn commit: r253255 - head/sys/netgraph/bluetooth/drivers/ubt
Adrian Chadd
adrian at freebsd.org
Fri Jul 12 18:24:12 UTC 2013
Hm, I just found a patch in my local collection which works for all
netgraph nodes.
Maybe we should do that instead of this?
http://people.freebsd.org/~adrian/ath/20130712-netgraph-vimage.diff
That should solve it for all dynamic netgraph stuff, rather than just bluetooth.
-adrian
On 12 July 2013 01:03, Craig Rodrigues <rodrigc at freebsd.org> wrote:
> Author: rodrigc
> Date: Fri Jul 12 08:03:10 2013
> New Revision: 253255
> URL: http://svnweb.freebsd.org/changeset/base/253255
>
> Log:
> PR: kern/168520
> Submitted by: "YAMAMOTO, Shigeru" <shigeru at iij.ad.jp>
> Reviewed by: adrian
>
> In PC-BSD 9.1, VIMAGE is enabled in the kernel config.
> For laptops with Bluetooth capability, such as the HP Elitebook 8460p,
> the kernel will panic upon bootup, because curthread->td_vnet
> is not initialized.
>
> Properly initialize curthread->td_vnet when initializing the Bluetooth stack.
>
> This allows laptops such as the HP Elitebook 8460p laptop
> to properly boot with VIMAGE kernels.
>
> Modified:
> head/sys/netgraph/bluetooth/drivers/ubt/ng_ubt.c
>
> Modified: head/sys/netgraph/bluetooth/drivers/ubt/ng_ubt.c
> ==============================================================================
> --- head/sys/netgraph/bluetooth/drivers/ubt/ng_ubt.c Fri Jul 12 07:43:56 2013 (r253254)
> +++ head/sys/netgraph/bluetooth/drivers/ubt/ng_ubt.c Fri Jul 12 08:03:10 2013 (r253255)
> @@ -109,7 +109,9 @@
> #include <sys/unistd.h>
> #include <sys/callout.h>
> #include <sys/malloc.h>
> +#include <sys/jail.h>
> #include <sys/priv.h>
> +#include <sys/proc.h>
>
> #include "usbdevs.h"
> #include <dev/usb/usb.h>
> @@ -123,6 +125,7 @@
> #include <sys/mbuf.h>
> #include <sys/taskqueue.h>
>
> +#include <net/vnet.h>
> #include <netgraph/ng_message.h>
> #include <netgraph/netgraph.h>
> #include <netgraph/ng_parse.h>
> @@ -487,13 +490,14 @@ ubt_attach(device_t dev)
>
> sc->sc_dev = dev;
> sc->sc_debug = NG_UBT_WARN_LEVEL;
> -
> + CURVNET_SET(TD_TO_VNET(curthread));
> /*
> * Create Netgraph node
> */
>
> if (ng_make_node_common(&typestruct, &sc->sc_node) != 0) {
> UBT_ALERT(sc, "could not create Netgraph node\n");
> + CURVNET_RESTORE();
> return (ENXIO);
> }
>
> @@ -501,10 +505,12 @@ ubt_attach(device_t dev)
> if (ng_name_node(sc->sc_node, device_get_nameunit(dev)) != 0) {
> UBT_ALERT(sc, "could not name Netgraph node\n");
> NG_NODE_UNREF(sc->sc_node);
> + CURVNET_RESTORE();
> return (ENXIO);
> }
> NG_NODE_SET_PRIVATE(sc->sc_node, sc);
> NG_NODE_FORCE_WRITER(sc->sc_node);
> + CURVNET_RESTORE();
>
> /*
> * Initialize device softc structure
> @@ -631,8 +637,10 @@ ubt_detach(device_t dev)
> /* Destroy Netgraph node */
> if (node != NULL) {
> sc->sc_node = NULL;
> + CURVNET_SET(node->nd_vnet);
> NG_NODE_REALLY_DIE(node);
> ng_rmnode_self(node);
> + CURVNET_RESTORE();
> }
>
> /* Make sure ubt_task in gone */
More information about the svn-src-all
mailing list