[openib-general] oops at device removal

[Michael S. Tsirkin]

> We have observed the following crash:

OK, I think I see a reason for this.

I notice the following in code, file multicast.c, function mcast_add_one:

        ib_set_client_data(device, &mcast_client, dev);

	        INIT_IB_EVENT_HANDLER(&event_handler, device,
				      mcast_event_handler);
        ib_register_event_handler(&event_handler);

So it seems like if I have 2 devices, &event_handler will be registered twice.
This will trigger data corruption as same entry will be added to list twice.

Or so it seems. Sean, what's the idea here?

-- 
MST