Hi Jason, I see that "include/uapi/rdma" contains API that is common. The qedr_user.h, that I assume you are referring to, is a qedr specific API. For example, we issue the ib_copy_to_udata() on structures defined in the file. So per my understanding it is in place.
Thanks, Ram -----Original Message----- From: Jason Gunthorpe [mailto:jguntho...@obsidianresearch.com] Sent: Monday, September 12, 2016 9:05 PM To: Ram Amrani <ram.amr...@qlogic.com> Cc: dledf...@redhat.com; David Miller <da...@davemloft.net>; Yuval Mintz <yuval.mi...@qlogic.com>; Ariel Elior <ariel.el...@qlogic.com>; Michal Kalderon <michal.kalde...@qlogic.com>; Rajesh Borundia <rajesh.borun...@qlogic.com>; linux-r...@vger.kernel.org; netdev <netdev@vger.kernel.org> Subject: Re: [RFC 00/11] QLogic RDMA Driver (qedr) RFC On Mon, Sep 12, 2016 at 07:07:34PM +0300, Ram Amrani wrote: > drivers/infiniband/hw/qedr/main.c | 907 ++++++ > drivers/infiniband/hw/qedr/qedr.h | 494 ++++ > drivers/infiniband/hw/qedr/qedr_cm.c | 626 +++++ > drivers/infiniband/hw/qedr/qedr_cm.h | 61 + > drivers/infiniband/hw/qedr/qedr_hsi.h | 56 + > drivers/infiniband/hw/qedr/qedr_hsi_rdma.h | 748 +++++ > drivers/infiniband/hw/qedr/qedr_user.h | 80 + We are requiring new uAPI headers are placed under include/uapi/rdma/, please coordinate with Leon on the path. Jason