]> git.baikalelectronics.ru Git - kernel.git/commit
xen-netback: switch state to InitWait at the end of netback_probe()...
authorPaul Durrant <pdurrant@amazon.com>
Tue, 17 Dec 2019 13:32:17 +0000 (13:32 +0000)
committerDavid S. Miller <davem@davemloft.net>
Wed, 18 Dec 2019 07:03:33 +0000 (23:03 -0800)
commit518716323f228e846b2f28cbc6be86e234f5a2e9
treeb145c129fa3e750393c250aab873111544ba5c84
parent18082b541f39ee088ae83590509fe763df735d8a
xen-netback: switch state to InitWait at the end of netback_probe()...

...as the comment above the function states.

The switch to Initialising at the start of the function is somewhat bogus
as the toolstack will have set that initial state anyway. To behave
correctly, a backend should switch to InitWait once it has set up all
xenstore values that may be required by a initialising frontend. This
patch calls backend_switch_state() to make the transition at the
appropriate point.

NOTE: backend_switch_state() ignores errors from xenbus_switch_state()
      and so this patch removes an error path from netback_probe(). This
      means a failure to change state at this stage (in the absence of
      other failures) will leave the device instantiated. This is highly
      unlikley to happen as a failure to change state would indicate a
      failure to write to xenstore, and that will trigger other error
      paths. Also, a 'stuck' device can still be cleaned up using 'unbind'
      in any case.

Signed-off-by: Paul Durrant <pdurrant@amazon.com>
Acked-by: Wei Liu <wei.liu@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/xen-netback/xenbus.c