From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([209.51.188.92]:54302) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1goASk-0006Z5-5W for qemu-devel@nongnu.org; Mon, 28 Jan 2019 12:12:47 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1goASj-0002iB-58 for qemu-devel@nongnu.org; Mon, 28 Jan 2019 12:12:46 -0500 Received: from mx1.redhat.com ([209.132.183.28]:37908) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1goASh-0002Yq-8I for qemu-devel@nongnu.org; Mon, 28 Jan 2019 12:12:45 -0500 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 5E3FFA7E1 for ; Mon, 28 Jan 2019 17:12:31 +0000 (UTC) Date: Mon, 28 Jan 2019 12:12:29 -0500 From: "Michael S. Tsirkin" Message-ID: <20190128121142-mutt-send-email-mst@kernel.org> References: <20190128170321.16936-1-dgilbert@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190128170321.16936-1-dgilbert@redhat.com> Subject: Re: [Qemu-devel] [PATCH 0/9] Network announce changes List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Dr. David Alan Gilbert (git)" Cc: qemu-devel@nongnu.org, quintela@redhat.com, jasowang@redhat.com, eblake@redhat.com, armbru@redhat.com, berrange@redhat.com On Mon, Jan 28, 2019 at 05:03:12PM +0000, Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" > > Hi, > This is a reworking of a few sets of patches from 2017 > that were put together by myself, Germano and Vlad that make > the network announce system more flexible. > > Firstly, it's parameterised, so that you can change the number > of packets and the gap between them; the number can be set to 0 > to disable announce completely. > > Secondly, you can force an announce by a qmp or hmp command at > any time. This is useful if you need the guest to do an announce > for a different reason; for example if the management layer > has just juggled some bonding configuration around. > > The packet creation and timing also moves to net/ from migration/ > > The previous set was: > https://lists.gnu.org/archive/html/qemu-devel/2017-05/msg05594.html > > Dave Reviewed-by: Michael S. Tsirkin Who's applying this? Jason? > Dr. David Alan Gilbert (9): > net: Introduce announce timer > migration: Add announce parameters > virtio-net: Switch to using announce timer > migration: Switch to using announce timer > net: Add a network device specific self-announcement ability > virtio-net: Allow qemu_announce_self to trigger virtio announcements > qmp: Add announce-self command > hmp: Add hmp_announce_self > tests: Add a test for qemu self announcments > > hmp-commands.hx | 14 ++++ > hmp.c | 33 ++++++++ > hmp.h | 1 + > hw/net/trace-events | 8 ++ > hw/net/virtio-net.c | 69 ++++++++++++---- > include/hw/virtio/virtio-net.h | 4 +- > include/migration/misc.h | 12 +-- > include/net/announce.h | 41 ++++++++++ > include/net/net.h | 2 + > include/qemu/typedefs.h | 2 + > include/sysemu/sysemu.h | 2 - > migration/migration.c | 103 +++++++++++++++++++++++- > migration/migration.h | 4 + > migration/savevm.c | 72 +---------------- > migration/trace-events | 1 - > net/Makefile.objs | 1 + > net/announce.c | 141 +++++++++++++++++++++++++++++++++ > net/trace-events | 4 + > qapi/migration.json | 56 ++++++++++++- > qapi/net.json | 43 ++++++++++ > tests/Makefile.include | 2 + > tests/test-announce-self.c | 83 +++++++++++++++++++ > tests/test-hmp.c | 1 + > 23 files changed, 595 insertions(+), 104 deletions(-) > create mode 100644 include/net/announce.h > create mode 100644 net/announce.c > create mode 100644 tests/test-announce-self.c > > -- > 2.20.1