svn commit: r253255 - head/sys/netgraph/bluetooth/drivers/ubt
Craig Rodrigues
rodrigc at FreeBSD.org
Fri Jul 12 18:39:37 UTC 2013
I like your patch. It may be worth having one more Netgraph person review
it and give a sanity check, but
I think we should commit your patch ASAP.
I am new to VNET, but I am wondering if in your patch we can KASSERT if
node->vnet is NULL. That would make
things easier to debug.
The part of my patch which which does CURVNET_SET is still necessary upon
attach in Bluetooth.
However, with your patch, we can get rid of the part of my patch which does
CURVNET_SET upon Bluetooth detach.
--
Craig
On Fri, Jul 12, 2013 at 11:24 AM, Adrian Chadd <adrian at freebsd.org> wrote:
> 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