[v5] network announce; interface selection & IDs

[Qemu-devel] [PATCH v5 0/5] network announce; interface selection & IDs

Dr. David Alan Gilbert (git) posted 5 patches 4 years, 10 months ago

Diff against v3 v4
Download series mbox

Test s390x passed

Test checkpatch passed

Test asan passed

Test docker-mingw@fedora passed

Test docker-clang@ubuntu passed

Test FreeBSD passed

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

Maintainers: Jason Wang <jasowang@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, "Michael S. Tsirkin" <mst@redhat.com>, "Dr. David Alan Gilbert" <dgilbert@redhat.com>, Markus Armbruster <armbru@redhat.com>, Laurent Vivier <lvivier@redhat.com>, Thomas Huth <thuth@redhat.com>, Eric Blake <eblake@redhat.com>

hmp-commands.hx         |  7 +++-
hw/net/virtio-net.c     |  4 +-
include/net/announce.h  |  8 +++-
monitor/hmp-cmds.c      | 41 ++++++++++++++++++-
net/announce.c          | 89 +++++++++++++++++++++++++++++++++++------
net/trace-events        |  3 +-
qapi/net.json           | 16 ++++++--
tests/virtio-net-test.c | 57 ++++++++++++++++++++++++--
8 files changed, 198 insertions(+), 27 deletions(-)

Expand all Fold all

[Qemu-devel] [PATCH v5 0/5] network announce; interface selection & IDs

Posted by Dr. David Alan Gilbert (git) 4 years, 10 months ago

From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>

Up until now, the 'announce' feature has mainly been used
for migration where we announce on all interfaces.  Another
use for 'announce' is in cases of network topology changes.

Since network topology changes may only affect a subset
of the interfaces, we add an 'interface list' to announce
to restrict the announcment to the interfaces we're interested
in.

Multiple topology changes might happen in close succession,
so we allow multiple timers, each with their own parameters
(including the interface list).

Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>

v5
  Minor review fixes [Jason] 

Dr. David Alan Gilbert (5):
  net/announce: Allow optional list of interfaces
  net/announce: Add HMP optional interface list
  net/announce: Add optional ID
  net/announce: Add HMP optional ID
  net/announce: Expand test for stopping self announce

 hmp-commands.hx         |  7 +++-
 hw/net/virtio-net.c     |  4 +-
 include/net/announce.h  |  8 +++-
 monitor/hmp-cmds.c      | 41 ++++++++++++++++++-
 net/announce.c          | 89 +++++++++++++++++++++++++++++++++++------
 net/trace-events        |  3 +-
 qapi/net.json           | 16 ++++++--
 tests/virtio-net-test.c | 57 ++++++++++++++++++++++++--
 8 files changed, 198 insertions(+), 27 deletions(-)

-- 
2.21.0

Re: [Qemu-devel] [PATCH v5 0/5] network announce; interface selection & IDs

Posted by Jason Wang 4 years, 10 months ago

On 2019/6/21 上午2:47, Dr. David Alan Gilbert (git) wrote:
> From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
>
> Up until now, the 'announce' feature has mainly been used
> for migration where we announce on all interfaces.  Another
> use for 'announce' is in cases of network topology changes.
>
> Since network topology changes may only affect a subset
> of the interfaces, we add an 'interface list' to announce
> to restrict the announcment to the interfaces we're interested
> in.
>
> Multiple topology changes might happen in close succession,
> so we allow multiple timers, each with their own parameters
> (including the interface list).
>
> Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
>
> v5
>    Minor review fixes [Jason]
>
> Dr. David Alan Gilbert (5):
>    net/announce: Allow optional list of interfaces
>    net/announce: Add HMP optional interface list
>    net/announce: Add optional ID
>    net/announce: Add HMP optional ID
>    net/announce: Expand test for stopping self announce
>
>   hmp-commands.hx         |  7 +++-
>   hw/net/virtio-net.c     |  4 +-
>   include/net/announce.h  |  8 +++-
>   monitor/hmp-cmds.c      | 41 ++++++++++++++++++-
>   net/announce.c          | 89 +++++++++++++++++++++++++++++++++++------
>   net/trace-events        |  3 +-
>   qapi/net.json           | 16 ++++++--
>   tests/virtio-net-test.c | 57 ++++++++++++++++++++++++--
>   8 files changed, 198 insertions(+), 27 deletions(-)


Applied.

Thanks