[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] rdma: rename 'x-rdma' => 'rdma'
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH] rdma: rename 'x-rdma' => 'rdma' |
Date: |
Tue, 22 Oct 2013 21:20:38 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.0 |
On 10/22/2013 05:59 PM, address@hidden wrote:
> From: "Michael R. Hines" <address@hidden>
>
> As far as we can tell, all known bugs have been fixed,
> there as been very good participation in testing and running.
>
> 1. Parallel RDMA migrations are working
> 2. IPv6 migration is working
> 3. Libvirt patches are ready
> 4. virt-test is working
>
> Any objections to removing the experimental tag?
>
> There is one remaining bug: qemu-system-i386 does not compile
> with RDMA: I have very zero access to 32-bit hardware
> using RDMA, so this hasn't been much of a priority. It seems
> safer to *not* submit non-testable patch rather than submit
> submit a fix just for the sake of compiling =)
>
> Signed-off-by: Michael R. Hines <address@hidden>
> ---
>
> TODO:
> =====
> -1. 'migrate x-rdma:host:port' and '-incoming x-rdma' options will be
> +1. 'migrate rdma:host:port' and '-incoming rdma' options will be
> renamed to 'rdma' after the experimental phase of this work has
> completed upstream.
Shouldn't you remove step 1 and renumber the rest of the list
altogether, rather than just altering the comment to make it out-of-date?
> +++ b/qapi-schema.json
> @@ -615,7 +615,7 @@
> # This feature allows us to minimize migration traffic for certain
> work
> # loads, by sending compressed difference of the pages
> #
> -# @x-rdma-pin-all: Controls whether or not the entire VM memory footprint is
> +# @rdma-pin-all: Controls whether or not the entire VM memory footprint is
> # mlock()'d on demand or all at once. Refer to docs/rdma.txt for
> usage.
> # Disabled by default. Experimental: may (or may not) be renamed
> after
> # further testing is complete. (since 1.6)
I'd also recommend tweaking this to say 'since 1.7', since the spelling
'rdma-pin-all' is new to this release.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
Re: [Qemu-devel] [PATCH] rdma: rename 'x-rdma' => 'rdma', Michael R. Hines, 2013/10/25