Re: [RFC 00/11] QLogic RDMA Driver (qedr) RFC

On Tue, Sep 13, 2016 at 08:44:06AM +0000, Ram Amrani wrote:

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.

1. It will be great if you avoid from top-posting.
2. In the near future, these custom UAPI header files will be placed
under include/uapi/rdma/provider/.

Thanks,
Ram


-----Original Message-----
From: Jason Gunthorpe [mailto:<redacted>]
Sent: Monday, September 12, 2016 9:05 PM
To: Ram Amrani <redacted>
Cc: <redacted>; David Miller <redacted>; Yuval Mintz <redacted>; Ariel Elior <redacted>; Michal Kalderon <redacted>; Rajesh Borundia <redacted>; <redacted>; netdev <redacted>
Subject: Re: [RFC 00/11] QLogic RDMA Driver (qedr) RFC

On Mon, Sep 12, 2016 at 07:07:34PM +0300, Ram Amrani wrote:

quoted

 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
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html