[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL v4 08/11] rdma: core logic
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PULL v4 08/11] rdma: core logic |
Date: |
Thu, 18 Apr 2013 16:12:42 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130402 Thunderbird/17.0.5 |
On 04/17/2013 05:07 PM, address@hidden wrote:
> From: "Michael R. Hines" <address@hidden>
>
> Code that does need to be visible is kept
> well contained inside this file and this is the only
> new additional file to the entire patch - good
> progress.
>
> This file includes the entire protocol and interfaces
> required to perform RDMA migration.
>
> Also, the configure and Makefile modifications to link
> this file are included.
>
> Full documentation is in docs/rdma.txt
Which doesn't appear until later in the series? If a git bisect lands
on this patch, I have to go out of my way to find the later commit that
adds the docs.
I personally like series that put the docs FIRST. On initial review,
that gives the docs a chance for a clean-room review untainted by the
implementation choices; and leaves the docs fresh in reviewers' minds
during the rest of the series to validate that the implementation
matches docs. On later review (such as git bisect landing here), it
means the docs are in-tree for any other commit that references them.
If it were me, I'd rebase things to put docs in patch 1 on the v5 series.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-devel] [PULL v4 08/11] rdma: core logic, (continued)
- Re: [Qemu-devel] [PULL v4 08/11] rdma: core logic, Michael R. Hines, 2013/04/18
- Re: [Qemu-devel] [PULL v4 08/11] rdma: core logic, Michael S. Tsirkin, 2013/04/18
- Re: [Qemu-devel] [PULL v4 08/11] rdma: core logic, Michael R. Hines, 2013/04/18
- Re: [Qemu-devel] [PULL v4 08/11] rdma: core logic, Michael S. Tsirkin, 2013/04/18
- Re: [Qemu-devel] [PULL v4 08/11] rdma: core logic, Michael R. Hines, 2013/04/18
- Re: [Qemu-devel] [PULL v4 08/11] rdma: core logic, Michael S. Tsirkin, 2013/04/18
- Re: [Qemu-devel] [PULL v4 08/11] rdma: core logic, Anthony Liguori, 2013/04/18
- [Qemu-devel] licensing of IBM contributions to QEMU (was Re: [PULL v4 08/11] rdma: core logic), Paolo Bonzini, 2013/04/18
- Re: [Qemu-devel] licensing of IBM contributions to QEMU (was Re: [PULL v4 08/11] rdma: core logic), Michael R. Hines, 2013/04/18
- Re: [Qemu-devel] licensing of IBM contributions to QEMU (was Re: [PULL v4 08/11] rdma: core logic), Anthony Liguori, 2013/04/18
Re: [Qemu-devel] [PULL v4 08/11] rdma: core logic,
Eric Blake <=
Re: [Qemu-devel] [PULL v4 00/11] rdma: migration support, Anthony Liguori, 2013/04/17
Re: [Qemu-devel] [PULL v4 00/11] rdma: migration support, Michael S. Tsirkin, 2013/04/18
- Re: [Qemu-devel] [PULL v4 00/11] rdma: migration support, Michael R. Hines, 2013/04/18
- Re: [Qemu-devel] [PULL v4 00/11] rdma: migration support, Michael S. Tsirkin, 2013/04/18
- Re: [Qemu-devel] [PULL v4 00/11] rdma: migration support, Michael R. Hines, 2013/04/18
- Re: [Qemu-devel] [PULL v4 00/11] rdma: migration support, Michael S. Tsirkin, 2013/04/18
- Re: [Qemu-devel] [PULL v4 00/11] rdma: migration support, Michael R. Hines, 2013/04/18
- Re: [Qemu-devel] [PULL v4 00/11] rdma: migration support, Michael S. Tsirkin, 2013/04/18