svn commit: r253255 - head/sys/netgraph/bluetooth/drivers/ubt

Craig Rodrigues rodrigc at
Sat Jul 13 19:45:15 UTC 2013

On Sat, Jul 13, 2013 at 11:33 AM, Mikolaj Golub <trociny at> wrote:
> Currently I don't have any pluggable device so my testing would not be
> very useful. I tried Marko's patch to test kldload/kldunload change,
> which is not in your patch.

OK, just to double check, based on reading your post: ,
with Marko's kldload/kldunload change, you encountered kernel panics in PF
and IPFW?
I also encountered kernel panics in PF with that patch.
Maybe we can clean up those modules in a separate pass,
and apply the kldload/kldunload fix some time in the future.

> Concerning your combined patch, as I said previously I don't like very
> much that dance with is_default_vnet. What is a purpose of it? As for
> me, it only makes the code less readable. I would just set curvnet to
> vnet0, as it is in Marko's patch.

Fine by me.  I have re-rolled my combined patch to remove the
is_default_vnet check
and go with Marko's code to set curvnet to vnet0:

(1)  Combine Adrian and Marko's patches, but omit Marko's changes to

(2)  Revert my Bluetooth patch:


More information about the svn-src-head mailing list