[ovs-dev] openvswitch_handle_frame_hook

Thomas Graf tgraf at redhat.com
Sat Jun 29 00:19:49 UTC 2013


Jesse,

On 06/29/2013 12:44 AM, Jesse Gross wrote:
> Pravin pointed out that the RHEL6 openvswitch_handle_frame_hook
> unregister is happening from the RCU callback, which means that we
> immediately free the vport after removing the handler. This seems
> racy. I also noticed that we have asymmetric use of
> netdev_rx_handler_register/unregister (I don't think that this is an
> actual problem but it doesn't seem right). I think the ideal solution
> would be to put this and the code for kernels < 2.6.37 (which is very
> similar) into backports for netdev_rx_handler_register(). This would
> both hide the differences and ensure consistency.
>
> Can you take a look?

Makes sense. We will take a look and propose a fix. Thanks for
the pointer.



More information about the dev mailing list