[PATCH 09/10] IB: remove the struct ib_phys_buf definition

2015-12-23 Thread Christoph Hellwig
Signed-off-by: Christoph Hellwig Reviewed-by: Sagi Grimberg Reviewed-by: Jason Gunthorpe [core] Reviewed-by: Steve Wise --- include/rdma/ib_verbs.h | 5 - 1 file changed, 5 deletions(-) diff --git a/include/rdma/ib_verbs.h b/include/rdma/ib_verbs.h index 1778442..197b620 100644 --- a/incl

Re: [PATCH 09/10] IB: remove the struct ib_phys_buf definition

2015-12-20 Thread Or Gerlitz
On 12/20/2015 5:41 PM, Christoph Hellwig wrote: I can't see a useful explanation for removing an unused structure. But if you really want one and come up with a coherent sentence or two I can add it. patch title - IB/core: Remove struct ib_phys_buf change log: Remove struct ib_phys_buf sinc

Re: [PATCH 09/10] IB: remove the struct ib_phys_buf definition

2015-12-20 Thread Christoph Hellwig
On Sun, Dec 20, 2015 at 09:37:38AM +0200, Or Gerlitz wrote: > On 12/18/2015 3:55 PM, Christoph Hellwig wrote: > >> Signed-off-by: Christoph Hellwig >> Reviewed-by: Sagi Grimberg >> Reviewed-by: Jason Gunthorpe [core] >> Reviewed-by: Steve Wise > > Here, too, please avoid empty change logs to IB

Re: [PATCH 09/10] IB: remove the struct ib_phys_buf definition

2015-12-19 Thread Or Gerlitz
On 12/18/2015 3:55 PM, Christoph Hellwig wrote: Signed-off-by: Christoph Hellwig Reviewed-by: Sagi Grimberg Reviewed-by: Jason Gunthorpe [core] Reviewed-by: Steve Wise Here, too, please avoid empty change logs to IB core patches. Or. --- include/rdma/ib_verbs.h | 5 - 1 file chang

[PATCH 09/10] IB: remove the struct ib_phys_buf definition

2015-12-18 Thread Christoph Hellwig
Signed-off-by: Christoph Hellwig Reviewed-by: Sagi Grimberg Reviewed-by: Jason Gunthorpe [core] Reviewed-by: Steve Wise --- include/rdma/ib_verbs.h | 5 - 1 file changed, 5 deletions(-) diff --git a/include/rdma/ib_verbs.h b/include/rdma/ib_verbs.h index ea093ee..284916d 100644 --- a/incl