I've encountered several oops when adding and removing interfaces from bridges while using Xen. Most of the details are available [1]here. The short of it is the following sequence: CPU0 CPU1 add_de
* Stephen Hemminger <shemminger@xxxxxxxx> [2005-08-18 17:11]: unregister_netdevice asserts RTNL, add_del_if() in br_ioctl.c doesn't seem to do so. I don't see it down dev_get_by_index() path either.
But the pat to br_dev_ioctl() is via the socket ioctl and that should already have gotten RTNL. dev_ioctl rtnl_lock() dev_ifsioc() dev->do_ioctl --> br_dev_ioctl
* Stephen Hemminger <shemminger@xxxxxxxx> [2005-08-18 17:36]: Hrm. OK. It sounds like both paths are doing the right thing w.r.t asserting RTNL, but br_device_event() still gets called with: 1) dev->
* Stephen Hemminger <shemminger@xxxxxxxx> [2005-08-18 17:36]: Just to follow-up, the issue was a race between the call_rcu() callback for destroy_nbp() and an unregister_netdev() call. Sometimes the
Hello, I've encountered several oops when adding and removing interfaces from bridges while using Xen. Most of the details are available [1]here. The short of it is the following sequence: CPU0 CPU1
* Stephen Hemminger <shemminger@xxxxxxxx> [2005-08-18 17:11]: unregister_netdevice asserts RTNL, add_del_if() in br_ioctl.c doesn't seem to do so. I don't see it down dev_get_by_index() path either.
But the pat to br_dev_ioctl() is via the socket ioctl and that should already have gotten RTNL. dev_ioctl rtnl_lock() dev_ifsioc() dev->do_ioctl --> br_dev_ioctl
* Stephen Hemminger <shemminger@xxxxxxxx> [2005-08-18 17:36]: Hrm. OK. It sounds like both paths are doing the right thing w.r.t asserting RTNL, but br_device_event() still gets called with: 1) dev->
* Stephen Hemminger <shemminger@xxxxxxxx> [2005-08-18 17:36]: Just to follow-up, the issue was a race between the call_rcu() callback for destroy_nbp() and an unregister_netdev() call. Sometimes the