[v3] net/virtio: fixes for failover

[PATCH v3 0/4] net/virtio: fixes for failover

Jens Freimann posted 4 patches 4 years, 5 months ago

Diff against v2
Download series mbox

Test asan passed

Test checkpatch passed

Test FreeBSD passed

Test docker-mingw@fedora passed

Test docker-clang@ubuntu passed

Test docker-quick@centos7 passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20191120154951.27877-1-jfreimann@redhat.com

Maintainers: Juan Quintela <quintela@redhat.com>, "Dr. David Alan Gilbert" <dgilbert@redhat.com>, "Michael S. Tsirkin" <mst@redhat.com>, Jason Wang <jasowang@redhat.com>

hw/net/virtio-net.c | 58 +++++++++++++++++++++++++++++----------------
migration/savevm.c  |  3 ++-
2 files changed, 40 insertions(+), 21 deletions(-)

Expand all Fold all

[PATCH v3 0/4] net/virtio: fixes for failover

Posted by Jens Freimann 4 years, 5 months ago

This series fixes bugs found by coverity and one reported by David
Gilbert.

v2->v3:
 * change patch description and subject of patch 3/4

Jens Freimann (4):
  net/virtio: fix dev_unplug_pending
  net/virtio: return early when failover primary alread added
  net/virtio: fix re-plugging of primary device
  net/virtio: return error when device_opts arg is NULL

 hw/net/virtio-net.c | 58 +++++++++++++++++++++++++++++----------------
 migration/savevm.c  |  3 ++-
 2 files changed, 40 insertions(+), 21 deletions(-)

-- 
2.21.0

Re: [PATCH v3 0/4] net/virtio: fixes for failover

Posted by Michael S. Tsirkin 4 years, 5 months ago

On Wed, Nov 20, 2019 at 04:49:47PM +0100, Jens Freimann wrote:
> This series fixes bugs found by coverity and one reported by David
> Gilbert.


Looks good.  Jason can you merge this pls?

> v2->v3:
>  * change patch description and subject of patch 3/4
> 
> Jens Freimann (4):
>   net/virtio: fix dev_unplug_pending
>   net/virtio: return early when failover primary alread added
>   net/virtio: fix re-plugging of primary device
>   net/virtio: return error when device_opts arg is NULL
> 
>  hw/net/virtio-net.c | 58 +++++++++++++++++++++++++++++----------------
>  migration/savevm.c  |  3 ++-
>  2 files changed, 40 insertions(+), 21 deletions(-)
> 
> -- 
> 2.21.0

Re: [PATCH v3 0/4] net/virtio: fixes for failover

Posted by Jason Wang 4 years, 5 months ago

On 2019/11/20 下午11:56, Michael S. Tsirkin wrote:
> On Wed, Nov 20, 2019 at 04:49:47PM +0100, Jens Freimann wrote:
>> This series fixes bugs found by coverity and one reported by David
>> Gilbert.
>
> Looks good.  Jason can you merge this pls?


Yes, applied.

Thanks


>
>> v2->v3:
>>   * change patch description and subject of patch 3/4
>>
>> Jens Freimann (4):
>>    net/virtio: fix dev_unplug_pending
>>    net/virtio: return early when failover primary alread added
>>    net/virtio: fix re-plugging of primary device
>>    net/virtio: return error when device_opts arg is NULL
>>
>>   hw/net/virtio-net.c | 58 +++++++++++++++++++++++++++++----------------
>>   migration/savevm.c  |  3 ++-
>>   2 files changed, 40 insertions(+), 21 deletions(-)
>>
>> -- 
>> 2.21.0