On Tue, Feb 02, 2016 at 11:31:08AM +0000, Paul Durrant wrote: [...] > +static int xen_register_mcast_ctrl_watch(struct xenbus_device *dev, > + struct xenvif *vif) > +{ > + int err = 0; > + char *node; > + unsigned maxlen = strlen(dev->otherend) + > + sizeof("/request-multicast-control"); > + > + if (vif->mcast_ctrl_watch.node) > + return -EADDRINUSE; > +
Sorry if I didn't make myself clear enough. I think EADDRINUSE should also be logged. Wei.