[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [MirageOS-devel] adding `replace_listener` function to Mirage_net.S


  • To: Hannes Mehnert <hannes@xxxxxxxxxxx>
  • From: Anil Madhavapeddy <anil@xxxxxxxxxx>
  • Date: Wed, 19 Jun 2019 14:00:00 +0100
  • Cc: mirageos-devel@xxxxxxxxxxxxxxxxxxxx
  • Delivery-date: Wed, 19 Jun 2019 13:27:03 +0000
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=recoil.org; h=content-type :mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; q=dns; s= selector1; b=Z4qvuK4j9H9Isbfj2ZFA2uiG7KheVymbQusskXFgZlNtPv1gG/w kpPaP+rhRvA1EJiHFBM7mDapjuYt7BYj3+6e3VtG4EVrXY5TyadL+sJZ64CALsrP Vu2ZgWmJPBwmO7NXBVFLNq+eduGSPGo9kqzxzHqwLLIuPs85PWFEntyI=
  • List-id: Developer list for MirageOS <mirageos-devel.lists.xenproject.org>


> On 18 Jun 2019, at 19:08, Hannes Mehnert <hannes@xxxxxxxxxxx> wrote:
> 
> Hello,
> 
> On 18/06/2019 17:33, Mindy Preston wrote:
>> In mirage_net.mli, the documentation for `val listen` says:
>> "this function can be stopped by calling disconnect in the device layer".
>> 
>> This is the only method mentioned to stop the call to `listen`. There
>> isn't any API call for replacing the callback established in `listen`. 
>> In mirage-vnetif, `disconnect` just stops the listener, and we can
>> continue to use the device after. This does not work for mirage-net-xen
>> (and maybe solo5), because disconnect cleans up the interface and we can
>> not continue using it.
> 
> As far as I can read both mirage-net-unix and mirage-net-solo5 are fine
> with sequences of connect followed by disconnect calls; maybe we should
> revise the documentation in mirage-net (that connect ; disconnect ;
> connect should work fine) and adapt mirage-net-xen to follow that behaviour?

Agreed; fixing mirage-net-xen sounds better here. It might become a
new interface (with a different vif number), but that shouldn't matter
to the unikernel I think.

regards,
Anil


_______________________________________________
MirageOS-devel mailing list
MirageOS-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/mirageos-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.