From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([209.51.188.92]:35102) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gyx5L-00006M-Nm for qemu-devel@nongnu.org; Wed, 27 Feb 2019 06:09:12 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gyx5E-0002YP-BN for qemu-devel@nongnu.org; Wed, 27 Feb 2019 06:09:11 -0500 Received: from mx1.redhat.com ([209.132.183.28]:58982) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1gyx59-0002U6-Pl for qemu-devel@nongnu.org; Wed, 27 Feb 2019 06:09:01 -0500 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id A2AA7820E9 for ; Wed, 27 Feb 2019 11:08:56 +0000 (UTC) From: Juan Quintela Date: Wed, 27 Feb 2019 12:08:04 +0100 Message-Id: <20190227110812.2185-1-quintela@redhat.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Subject: [Qemu-devel] [PATCH v3 0/8] migration: Mark multifd not experimental List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Markus Armbruster , Eric Blake , "Dr. David Alan Gilbert" , Thomas Huth , Juan Quintela , Paolo Bonzini , Laurent Vivier v3: - answer all comments from Dave - updates to latest upstream. v2: - Measure packet size in bytes/not pages - Change the defalut value from 64KB to 512KB - rename used field to pages_used - rename size field to pages_alloc - Create nnext_packet_size field Will be used on compression series later. - Be flexible about what packet sizes we allow. Please review, In v1: - Change page_count default to 128. 16 was really small, and it makes much less contention on mutexes to just have bigger packets. - Drop multifd-page_count parameter This parameter was useful for testing, but in all my testing 128 is good enough, no need to have an extra knob. Libvirt don't want to expose this parameter because it is difficult to explain. - Drop experimental "x-" from multifd Code is stable, nothing big is happening here. - Multifd test And a test for multifd, this test has already been on other patch series. But now it uses the names without "-x". Please review. Juan Quintela (8): multifd: Only send pages when packet are not empty multifd: Rename "size" member to pages_alloc multifd: Create new next_packet_size field multifd: Drop x-multifd-page-count parameter multifd: Be flexible about packet size multifd: Change default packet size multifd: Drop x- tests: Add migration multifd test hmp.c | 17 +++------ migration/migration.c | 56 +++++++----------------------- migration/migration.h | 1 - migration/ram.c | 79 ++++++++++++++++++++++++++++++------------ migration/trace-events | 4 +-- qapi/migration.json | 45 +++++++++--------------- tests/migration-test.c | 48 +++++++++++++++++++++++++ 7 files changed, 142 insertions(+), 108 deletions(-) --=20 2.20.1