Paul Durrant writes ("[PATCH v2 2/4] tools/hotplug: combine add/online and
remove/offline in vif-bridge..."):
> From: Paul Durrant <pdurr...@amazon.com>
>
> ... as they are in vif-route.
>
> The script is invoked with online/offline for vifs and add/remove for taps.
> The operations that are necessary, however, are the same in both cases. This
> patch therefore combines the cases.
This seems to newly add a "remove" case. Previously "remove" was a
no-op here. Is that right ? If so it needs to be discussed in the
commit message. We're not talking about a simple refactoring here!
Perhaps it would be best to move this bit
> + remove)
> + ;&
> offline)
which I think is the relevant change, into its own commit ?
Ian.
