[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-2.12 1/2] rdma: fix up include directives
From: |
Daniel P . Berrangé |
Subject: |
Re: [Qemu-devel] [PATCH for-2.12 1/2] rdma: fix up include directives |
Date: |
Wed, 21 Mar 2018 15:38:09 +0000 |
User-agent: |
Mutt/1.9.2 (2017-12-15) |
On Wed, Mar 21, 2018 at 05:22:07PM +0200, Michael S. Tsirkin wrote:
> Our rule right now is to use <> for external headers only.
> RDMA code violates that, fix it up.
>
> Signed-off-by: Michael S. Tsirkin <address@hidden>
> ---
> hw/rdma/rdma_backend.h | 2 +-
> hw/rdma/rdma_backend_defs.h | 2 +-
> hw/rdma/rdma_rm.h | 2 +-
> hw/rdma/rdma_utils.h | 6 +++---
> hw/rdma/vmw/pvrdma.h | 8 ++++----
> hw/rdma/vmw/pvrdma_dev_ring.h | 2 +-
> hw/rdma/rdma_backend.c | 6 +++---
> hw/rdma/rdma_rm.c | 6 +++---
> hw/rdma/vmw/pvrdma_cmd.c | 8 ++++----
> hw/rdma/vmw/pvrdma_dev_ring.c | 8 ++++----
> hw/rdma/vmw/pvrdma_main.c | 24 ++++++++++++------------
> hw/rdma/vmw/pvrdma_qp_ops.c | 4 ++--
> 12 files changed, 39 insertions(+), 39 deletions(-)
Reviewed-by: Daniel P. Berrangé <address@hidden>
I'm assuming these were detected by build failures caused by the 2nd
patch in this series, so other areas of the codebase are all correct
already.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
Re: [Qemu-devel] [PATCH for-2.12 1/2] rdma: fix up include directives,
Daniel P . Berrangé <=
Re: [Qemu-devel] [PATCH for-2.12 1/2] rdma: fix up include directives, Marcel Apfelbaum, 2018/03/21
Re: [Qemu-devel] [PATCH for-2.12 1/2] rdma: fix up include directives, Yuval Shaia, 2018/03/21