Re: [Qemu-devel] [PATCH for-2.12 1/2] rdma: fix up include directives

2018-03-21 Thread Yuval Shaia
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 Thanks. Reviewed-by: Yuval Shaia > --- >

Re: [Qemu-devel] [PATCH for-2.12 1/2] rdma: fix up include directives

2018-03-21 Thread Marcel Apfelbaum
On 21/03/2018 17:22, 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 Reviewed-by: Marcel Apfelbaum Thanks, Marcel > --- >

Re: [Qemu-devel] [PATCH for-2.12 1/2] rdma: fix up include directives

2018-03-21 Thread Daniel P . Berrangé
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 > --- > hw/rdma/rdma_backend.h| 2 +- >

[Qemu-devel] [PATCH for-2.12 1/2] rdma: fix up include directives

2018-03-21 Thread Michael S. Tsirkin
Our rule right now is to use <> for external headers only. RDMA code violates that, fix it up. Signed-off-by: Michael S. Tsirkin --- hw/rdma/rdma_backend.h| 2 +- hw/rdma/rdma_backend_defs.h | 2 +- hw/rdma/rdma_rm.h | 2 +- hw/rdma/rdma_utils.h