Re: [PATCH v2 03/13] Provider Methods and Data Structures

2006-12-04 Thread Steve Wise
On Mon, 2006-12-04 at 08:45 -0800, Roland Dreier wrote: > > Roland, I think at one time we were talking about changing the Core to > > better handle this? Either with attributes/capabilities that the low > > level driver can set, or by set these method ptrs to NULL and the core > > should hand

Re: [PATCH v2 03/13] Provider Methods and Data Structures

2006-12-04 Thread Roland Dreier
> Roland, I think at one time we were talking about changing the Core to > better handle this? Either with attributes/capabilities that the low > level driver can set, or by set these method ptrs to NULL and the core > should handle it in the wrapper function... Yes, it would make sense to ch

Re: [PATCH v2 03/13] Provider Methods and Data Structures

2006-12-04 Thread Steve Wise
On Sun, 2006-12-03 at 13:07 +0100, Arjan van de Ven wrote: > On Sat, 2006-12-02 at 16:49 -0600, Steve Wise wrote: > > > + > > +static struct ib_ah *iwch_ah_create(struct ib_pd *pd, > > + struct ib_ah_attr *ah_attr) > > +{ > > + return ERR_PTR(-ENOSYS); > > +} > >

Re: [PATCH v2 03/13] Provider Methods and Data Structures

2006-12-03 Thread Arjan van de Ven
On Sat, 2006-12-02 at 16:49 -0600, Steve Wise wrote: > + > +static struct ib_ah *iwch_ah_create(struct ib_pd *pd, > + struct ib_ah_attr *ah_attr) > +{ > + return ERR_PTR(-ENOSYS); > +} -ENOSYS is just about ALWAYS a bug in that it's guaranteed to be the wrong

[PATCH v2 03/13] Provider Methods and Data Structures

2006-12-02 Thread Steve Wise
Provider methods to support the Linux RDMA verbs. Signed-off-by: Steve Wise <[EMAIL PROTECTED]> --- drivers/infiniband/hw/cxgb3/iwch_provider.c | 1170 +++ drivers/infiniband/hw/cxgb3/iwch_provider.h | 362 drivers/infiniband/hw/cxgb3/iwch_user.h | 68 ++