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

Re: [PATCH] libxl: force netback to wait for hotplug execution before connecting


  • To: Julien Grall <julien@xxxxxxx>
  • From: Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Date: Tue, 25 Jan 2022 17:09:21 +0100
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=5ljvA3kACGl8STvB+5+votPIQDy9fLJeYzt6cZLnXs0=; b=OOliIXODX04w7jobndCXhLRQh/gqpHam34Z6sE9kVEJg78cpGhmT31nfYZ6pcGy5XSV6h+tqHmUmNtdHMkyasSe1XJ4+gyjD4Im+U5/zBw+7cooHMAerLDQ6JqqA9/BK8Y40iV4x+MVbHE6IyDISx9M/9lslubaGkyOfmSe2L6NU0npDHPrfMNZU8cv6y0Jhjj6Q7HIemTn2dG4H/BtUfi4DVVDxn199ARj7ZxPKXj0mWrVVd2pwUqXqX0jmxsEYcOZxHCAOa0FUBxWNL64pQKBnKYjwG6bkrRQ7urAVROIWwbTHSMS257rhJpLu7J78W0gvhTJytWh5ZI+QyucEpA==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=U0IKKMc0MsGzrcbdx5HpalB3vUghNDgSi3+zeti43D9I3VHMBOOjqTpMdpPBRGMwBegTCdzWKjk1gNu0MH11jjGu0YVhNYoY17fdCLJafXEohDszF4INJbemryGqw+5IZadCvZhGaTbCyXkJciLMFzOxLgLKrJ49c9Ll/as1WARo6ZQfLfYMwFVPDFCNNURJ5KFHnpPeWmGZ4RA9RzPGvYqg5sgyJ+4mRLj+y07cp9PwdrRP8cqoTcT3XaGjBcM53Jv7wIWGXAMkuY22UKIJv6+WOgx1io3bTSUaEFWJvEjxkVX9ysOCrWq0Rt1Rj9G6/hq11QW1hohkNPhtlcZBOw==
  • Authentication-results: esa6.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Cc: <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>, Juergen Gross <jgross@xxxxxxxx>, James Dingwall <james-xen@xxxxxxxxxxxxxx>, Wei Liu <wei.liu@xxxxxxxxxx>, Paul Durrant <paul@xxxxxxx>
  • Delivery-date: Tue, 25 Jan 2022 16:10:37 +0000
  • Ironport-data: A9a23:fXAbhK9hxgnut9UIz8phDrUDKHmTJUtcMsCJ2f8bNWPcYEJGY0x3y zceWDrUM/+LYGGked51b4XnpxwAvpCHnNVrQAc6rS08E34SpcT7XtnIdU2Y0wF+jyHgoOCLy +1EN7Es+ehtFie0Si9AttENlFEkvU2ybuOU5NXsZ2YhFWeIdA970Ug5w7dj39Yy6TSEK1jlV e3a8pW31GCNg1aYAkpMg05UgEoy1BhakGpwUm0WPZinjneH/5UmJMt3yZWKB2n5WuFp8tuSH I4v+l0bElTxpH/BAvv9+lryn9ZjrrT6ZWBigVIOM0Sub4QrSoXfHc/XOdJFAXq7hQllkPhw5 fhSjKX3YzsvYJaLo9YPAylbEz5xaPguFL/veRBTsOSWxkzCNXDt3+9vHAc9OohwFuRfWD8Us 6ZCcXZUM07F17neLLGTE4GAguw5K8bmJsUHs2xIxjDFF/c2B5vERs0m4PcGhmls25gQQJ4yY eI6bhtfdB37TiYUMwYrItFlvuPwp2bwJmgwRFW9+vNsvjm7IBZK+LrkNdfOP8yHQ+1Zn0CXu WvC+yL8D1cHN7S31jeEtHatm8fMkDn9VYZUE6e3ntZyiUCX7nweDlsRT1TTifu2kEmlQPpEN lcZvCEpqMAa9lGvT9T7dw21pjiDpBF0c8pdFag25R+AzoLQ4h2FHS4UQzhZctskucQqAzsw2 TehgNfBFTFp9rqPRhqgGqy89G3of3JPdClbOHFCHVBtD8TfTJ8bjz/iUe9aV4SJiIPOXgrC0 WC6ryghiOBG5SIU7JmT8VfCijOqg5HGSA8p+wnaNl6YAhNFiJ2NPNLxtwWChRpUBMPAFwTa4 iBY8ySLxL1WVfmweDqxrPLh9V1Dz9KMK3XijFFmBPHNHBz9qif4Lei8DNyTTXqF0/romxe0O Cc/WisLvfe/2UdGi4ctP+pd7OxxlcDd+SzNDKy8Uza3SsEZmPW71C9vf1WM+GvmjVIhl6oyU b/CL5r3VSpLVvs5nWvuLwv47VPN7npirY80bcujpylLLJLEPCLFIVv7GAXmgh8FAFOs/1yOr oc32zqiwBRDSuzuChQ7AqZIRW3m2UMTXMisw+QOL7brClM/RAkJVqGNqZt8Jd0Nt/kFx4/go yDmMmcFmQWXuJEyAVjQApyVQOmxDc8XQLNSFXFEAGtELFB6Md/wt/9OLsVuFVTlncQ6pcNJo zA+U5zoKtxESyjd+iRbap/4rYd4cw+six7INC2gCAXTtbY7L+AQ0tO7LAbp6gcUCS+76Zk3r 7G6j1uJSpsfXQVySs3Rbav3nV+2uHEcnsN0XlfJfYYPKBm9rtAyJnyjlOIzLuENNQ7HmmmQ2 TGJDEpKvuLKuYI0roXE3PjWs4ezHuJiNUNGBG2Hv62uPCzX8zP7k49NWeqFZx7HU2bw9Pnwb OlZ1aikYvYGgExLo8x3FLMylfAy4N7mprl7yAV4HSqUMwT3W+04enTfhJtBrKxAwLNdqDCaY EPX94kII6iNNePkDEUVeFgvYNOc2KxGgTLV9/k0fhn3vXcl4LqdXExOFBCQkygBfqBtOYYoz Op96s4b7wuz1kgjPtqc130G8m2NKjoLUrk9t4FcC4ju01J5xlZHaJ3aKyn3/JDQNIkcbhh0e meZ1PjYmrBR5kveaH5iR3HC0N1UiYkKpB0XnkQJIE6Em4adi/I6tPGLHe/bkuiBIs177t9O
  • Ironport-hdrordr: A9a23:arCmVqkI7GZtAE6w8iDvqhlcltDpDfPOimdD5ihNYBxZY6Wkfp +V88jzhCWZtN9OYhwdcLC7WZVpQRvnhPlICK0qTM2ftWjdyRCVxeRZg7cKrAeQeREWmtQtsJ uINpIOdeEYbmIK8/oSgjPIaurIqePvmMvD5Za8854ud3ATV0gJ1XYHNu/xKDwReOApP+tcKH LKjfA32wZINE5nJviTNz0gZazuttfLnJXpbVovAAMm0hCHiXeN5KThGxaV8x8CW3cXqI1Su1 Ttokjc3OGOovu7whjT2yv66IlXosLozp9mCNaXgsYYBz3wgkKDZZhnWZeFoDcpydvfo2oCoZ 3pmVMNLs5z43TeciWcpgbs4RDp1HIU53rr2Taj8AzeiP28YAh/J9tKhIpffBecwVEnpstA3K VC2H/cn4ZLDDvb9R6NqOTgZlVPrA6ZsHAimekcgzh0So0FcoJcqoQZ4Qd8DIoAJiTn84oqed MeQP003MwmMG9yUkqp/lWGmLeXLzcO91a9MwU/U/WuonZrdCsT9Tpb+CQd9k1wgK7VBaM0ot gsCZ4Y5I2mePVmGJ6VNN1xMPdfNVa9Mi4kEFjiV2gPR5t3ck4klfbMkccIDaeRCdg18Kc=
  • Ironport-sdr: p5Ok9xtZGr28+I8VP3P+w5liICnK1Fjb2sBc963f1J4ehNsDEhU7g2RDSBm7VDDKIRY6/sPDkJ 5sLEDv1FjJANDsC1VRTuthtHo73a+stKQ6k7GIK1OyaTgiKP0U0kB47iP+k9dCREiWKZ4meNg5 vaw0FC0kXSQZ8ANtbmNHS9YAzL1bF+vAkrv9s+NYl87kmOT6rHI80RKHZEKDiwu6w6eXG1r3ZJ K57JHqi+RJKPBbqkSB0GK1UD76i9jnieBL4sOsvoqXu8sSAPchQDLCc8Y7RS+J8VcWMr2tB7MD XAQvjT3X1GMFffS9NjzhYF0T
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Tue, Jan 25, 2022 at 03:32:16PM +0000, Julien Grall wrote:
> Hi,
> 
> On 24/01/2022 16:02, Roger Pau Monne wrote:
> > By writing an empty "hotplug-status" xenstore node in the backend path
> > libxl can force Linux netback to wait for hotplug script execution
> > before proceeding to the 'connected' state.
> 
> I was actually chasing the same issue today :).
> 
> > 
> > This is required so that netback doesn't skip state 2 (InitWait) and
> 
> Technically netback never skip state 2 (otherwise it would always be
> reproducible). Instead, libxl may not be able to observe state 2 because
> receive a watch is asynchronous and doesn't contain the value of the node.
> So the backend may have moved to Connected before the state is read.

Right, might be more accurate to say it skips waiting for hotplug
script execution, and thus jumps from state 2 into 4. Note I think
it's also possible that by the time we setup the watch in libxl the
state has already been set to 4.

Thanks, Roger.



 


Rackspace

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