Re: [Xen-devel] netfront leaking two pages on unload?

[Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>]

On 16/1/08 21:16, "Andy Grover" <andy.grover@xxxxxxxxxx> wrote:

> Should it instead be:
> 
> case XenbusStateClosed:
> if (xenbus_dev_is_online(dev)) {
> struct backend_info *be = dev->dev.driver_data;
> netif_disconnect(be->netif);
> break;
> }
> xenbus_switch_state(dev, XenbusStateClosed);
> /* fall through if not online */
> 
> ?
> 
> Otherwise the pages cannot be freed properly in the guest, right?

Well, I think you've chosen slightly the wrong place, so I've just committed
a change that makes the close-down logic more like in blkback. Possibly the
reason we didn't follow that same logic before is that netif_disconnect()
also makes the network device in dom0 also go away, which might cause
unwanted chnages to routing and firewall rules, execution of hotplug
scripts, etc. But we can separate the concepts of relinquishing frontend
resources and destroying the backend netdev later if it turns out to be
necessary.

 -- Keir



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel