Re: [PATCH v5 02/16] IB/pvrdma: Add user-level shared functions

2016-09-26 Thread Adit Ranadive
On Sun, Sep 25, 2016 at 23:13:38PM -0700, Leon Romanovsky wrote: > On Sun, Sep 25, 2016 at 09:22:11PM -0700, Adit Ranadive wrote: >> On Sun, Sep 25 2016 at 10:26:24AM +0300, Leon Romanovsky wrote: > > > > On Sat, Sep 24, 2016 at 04:21:26PM -0700, Adit Ranadive wrote: > > > > We share some common st

Re: [PATCH v5 02/16] IB/pvrdma: Add user-level shared functions

2016-09-25 Thread Leon Romanovsky
On Sun, Sep 25, 2016 at 09:22:11PM -0700, Adit Ranadive wrote: > On Sun, Sep 25 2016 at 10:26:24AM +0300, Leon Romanovsky wrote: > > > On Sat, Sep 24, 2016 at 04:21:26PM -0700, Adit Ranadive wrote: > > > We share some common structures with the user-level driver. This patch > > > adds > > > those

Re: [PATCH v5 02/16] IB/pvrdma: Add user-level shared functions

2016-09-25 Thread Adit Ranadive
On Sun, Sep 25 2016 at 10:26:24AM +0300, Leon Romanovsky wrote: > > On Sat, Sep 24, 2016 at 04:21:26PM -0700, Adit Ranadive wrote: > > We share some common structures with the user-level driver. This patch adds > > those structures and shared functions to traverse the QP/CQ rings. <...> > > + > >

Re: [PATCH v5 02/16] IB/pvrdma: Add user-level shared functions

2016-09-25 Thread Leon Romanovsky
On Sun, Sep 25, 2016 at 10:26:24AM +0300, Leon Romanovsky wrote: <...> > > + > > +/* PVRDMA send queue work request */ > > +struct pvrdma_sq_wqe_hdr { > > + __u64 wr_id;/* wr id */ > > + __u32 num_sge; /* size of s/g array */ > > + __u32 total_len;/* reserved */

Re: [PATCH v5 02/16] IB/pvrdma: Add user-level shared functions

2016-09-25 Thread Leon Romanovsky
On Sat, Sep 24, 2016 at 04:21:26PM -0700, Adit Ranadive wrote: > We share some common structures with the user-level driver. This patch adds > those structures and shared functions to traverse the QP/CQ rings. > > Reviewed-by: Yuval Shaia > Reviewed-by: Jorgen Hansen > Reviewed-by: George Zhang

[PATCH v5 02/16] IB/pvrdma: Add user-level shared functions

2016-09-24 Thread Adit Ranadive
We share some common structures with the user-level driver. This patch adds those structures and shared functions to traverse the QP/CQ rings. Reviewed-by: Yuval Shaia Reviewed-by: Jorgen Hansen Reviewed-by: George Zhang Reviewed-by: Aditya Sarwade Reviewed-by: Bryan Tan Signed-off-by: Adit R