Re: [Qemu-devel] [PATCH v2 0/9] add failover feature for assigned network devices

2019-08-12 Thread Michael S. Tsirkin
On Fri, Aug 02, 2019 at 05:05:56PM +0200, Jens Freimann wrote: > This is implementing the host side of the net_failover concept > (https://www.kernel.org/doc/html/latest/networking/net_failover.html) Virtio bits look fine. Anyone else wants to comment on management/migration aspects? Which tree sh

Re: [Qemu-devel] [PATCH v2 0/9] add failover feature for assigned network devices

2019-08-07 Thread Jens Freimann
On Tue, Aug 06, 2019 at 04:43:35AM -0400, Michael S. Tsirkin wrote: On Mon, Aug 05, 2019 at 08:49:49PM +0200, Jens Freimann wrote: On Mon, Aug 05, 2019 at 10:22:25AM -0400, Michael S. Tsirkin wrote: > On Mon, Aug 05, 2019 at 03:12:15PM +0200, Jens Freimann wrote: > > On Fri, Aug 02, 2019 at 11:2

Re: [Qemu-devel] [PATCH v2 0/9] add failover feature for assigned network devices

2019-08-06 Thread Michael S. Tsirkin
On Mon, Aug 05, 2019 at 08:49:49PM +0200, Jens Freimann wrote: > On Mon, Aug 05, 2019 at 10:22:25AM -0400, Michael S. Tsirkin wrote: > > On Mon, Aug 05, 2019 at 03:12:15PM +0200, Jens Freimann wrote: > > > On Fri, Aug 02, 2019 at 11:22:10AM -0400, Michael S. Tsirkin wrote: > > > > On Fri, Aug 02, 2

Re: [Qemu-devel] [PATCH v2 0/9] add failover feature for assigned network devices

2019-08-05 Thread Jens Freimann
On Mon, Aug 05, 2019 at 10:22:25AM -0400, Michael S. Tsirkin wrote: On Mon, Aug 05, 2019 at 03:12:15PM +0200, Jens Freimann wrote: On Fri, Aug 02, 2019 at 11:22:10AM -0400, Michael S. Tsirkin wrote: > On Fri, Aug 02, 2019 at 05:05:56PM +0200, Jens Freimann wrote: > > This is implementing the hos

Re: [Qemu-devel] [PATCH v2 0/9] add failover feature for assigned network devices

2019-08-05 Thread Michael S. Tsirkin
On Mon, Aug 05, 2019 at 03:12:15PM +0200, Jens Freimann wrote: > On Fri, Aug 02, 2019 at 11:22:10AM -0400, Michael S. Tsirkin wrote: > > On Fri, Aug 02, 2019 at 05:05:56PM +0200, Jens Freimann wrote: > > > This is implementing the host side of the net_failover concept > > > (https://www.kernel.org/

Re: [Qemu-devel] [PATCH v2 0/9] add failover feature for assigned network devices

2019-08-05 Thread Jens Freimann
On Fri, Aug 02, 2019 at 11:22:10AM -0400, Michael S. Tsirkin wrote: On Fri, Aug 02, 2019 at 05:05:56PM +0200, Jens Freimann wrote: This is implementing the host side of the net_failover concept (https://www.kernel.org/doc/html/latest/networking/net_failover.html) Changes since v1: [...] Di

Re: [Qemu-devel] [PATCH v2 0/9] add failover feature for assigned network devices

2019-08-02 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20190802150605.5880-1-jfreim...@redhat.com/ Hi, This series failed the asan build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST SCRIPT BEGIN === #!/bin/bash mak

Re: [Qemu-devel] [PATCH v2 0/9] add failover feature for assigned network devices

2019-08-02 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20190802150605.5880-1-jfreim...@redhat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Subject: [Qemu-devel] [PATCH v2 0/9] add failover feature for assigned network devices Message-id

Re: [Qemu-devel] [PATCH v2 0/9] add failover feature for assigned network devices

2019-08-02 Thread Michael S. Tsirkin
On Fri, Aug 02, 2019 at 05:05:56PM +0200, Jens Freimann wrote: > This is implementing the host side of the net_failover concept > (https://www.kernel.org/doc/html/latest/networking/net_failover.html) > > Changes since v1: > - add new QMP events, >- one is send when the primary device is unplug

[Qemu-devel] [PATCH v2 0/9] add failover feature for assigned network devices

2019-08-02 Thread Jens Freimann
This is implementing the host side of the net_failover concept (https://www.kernel.org/doc/html/latest/networking/net_failover.html) Changes since v1: - add new QMP events, - one is send when the primary device is unplugged - one is send when VIRTIO_NET_F_STANDBY is not negotiated. This