Re: xen-netback hotplug-status regression bug

2021-04-13 Thread Michael Brown
On 13/04/2021 11:55, Paul Durrant wrote: Ok, so it sound like this was probably my misunderstanding of xenstore semantics in the first place (although I'm sure I remember watch registration failing for non-existent nodes at some point in the past... that may have been with a non-upstream versio

Re: xen-netback hotplug-status regression bug

2021-04-13 Thread Paul Durrant
On 13/04/2021 11:48, Michael Brown wrote: On 13/04/2021 08:12, Paul Durrant wrote: If the frontend subsequently disconnects and reconnects (e.g. transitions through Closed->Initialising->Connected) then: - Nothing recreates "hotplug-status" - When the frontend re-enters Connected state, conne

Re: xen-netback hotplug-status regression bug

2021-04-13 Thread Michael Brown
On 13/04/2021 08:12, Paul Durrant wrote: If the frontend subsequently disconnects and reconnects (e.g. transitions through Closed->Initialising->Connected) then: - Nothing recreates "hotplug-status" - When the frontend re-enters Connected state, connect() sets up a watch on "hotplug-status" a

Re: xen-netback hotplug-status regression bug

2021-04-13 Thread Paul Durrant
On 10/04/2021 19:25, Michael Brown wrote: Commit https://github.com/torvalds/linux/commit/1f25657 ("xen-netback: remove 'hotplug-status' once it has served its purpose") seems to have introduced a regression that prevents a vif frontend from transitioning more than once into Connected state.

xen-netback hotplug-status regression bug

2021-04-10 Thread Michael Brown
Commit https://github.com/torvalds/linux/commit/1f25657 ("xen-netback: remove 'hotplug-status' once it has served its purpose") seems to have introduced a regression that prevents a vif frontend from transitioning more than once into Connected state. As far as I can tell: - The defined vif sc