On Mon, Mar 30, 2015 at 12:13:00PM -0400, Doug Ledford wrote:
> On Fri, 2015-03-27 at 16:47 +0100, Michael Wang wrote:
> > Introduce helper has_iwarp() to help us check if an IB device
> > support IWARP protocol.
>
> This is a needless redirection. Just stick with the original
>
On 03/31/2015 03:42 PM, Tom Talpey wrote:
> On 3/31/2015 7:41 AM, Michael Wang wrote:
>> [snip]
> Yes, in fact the iWARP protocol does not preclude multiple read SGEs,
> even though most iWARP implementations have chosen to support just one.
>
> Even for multi-SGE-capable adapters, there is a
On 3/31/2015 7:41 AM, Michael Wang wrote:
Hi, Tom
Thanks for the comments :-)
On 03/31/2015 01:19 PM, Tom Talpey wrote:
[oops - repeating my reply with full cc's]
[snip]
Actually I'm thinking about Doug's idea to use rdma_transport_is_XX()
instead of the current basic helper, thus may
Hi, Tom
Thanks for the comments :-)
On 03/31/2015 01:19 PM, Tom Talpey wrote:
> [snip]
>
>>
>> Actually I'm thinking about Doug's idea to use rdma_transport_is_XX()
>> instead of the current basic helper, thus may be use
>> rdma_transport_is_iwarp()
>> in here could be better, since it's
On 3/31/2015 3:39 AM, Michael Wang wrote:
On 03/31/2015 12:35 AM, Jason Gunthorpe wrote:
On Mon, Mar 30, 2015 at 05:10:12PM +0200, Michael Wang wrote:
I found that actually we don't have to touch this one which
only used by HW driver currently.
I'm having a hard time understanding this, the
On 03/31/2015 12:35 AM, Jason Gunthorpe wrote:
> On Mon, Mar 30, 2015 at 05:10:12PM +0200, Michael Wang wrote:
>> I found that actually we don't have to touch this one which
>> only used by HW driver currently.
> I'm having a hard time understanding this, the code in question was in
>
>
On 03/31/2015 12:35 AM, Jason Gunthorpe wrote:
On Mon, Mar 30, 2015 at 05:10:12PM +0200, Michael Wang wrote:
I found that actually we don't have to touch this one which
only used by HW driver currently.
I'm having a hard time understanding this, the code in question was in
On 3/31/2015 3:39 AM, Michael Wang wrote:
On 03/31/2015 12:35 AM, Jason Gunthorpe wrote:
On Mon, Mar 30, 2015 at 05:10:12PM +0200, Michael Wang wrote:
I found that actually we don't have to touch this one which
only used by HW driver currently.
I'm having a hard time understanding this, the
Hi, Tom
Thanks for the comments :-)
On 03/31/2015 01:19 PM, Tom Talpey wrote:
[snip]
Actually I'm thinking about Doug's idea to use rdma_transport_is_XX()
instead of the current basic helper, thus may be use
rdma_transport_is_iwarp()
in here could be better, since it's actually a feature
On 3/31/2015 7:41 AM, Michael Wang wrote:
Hi, Tom
Thanks for the comments :-)
On 03/31/2015 01:19 PM, Tom Talpey wrote:
[oops - repeating my reply with full cc's]
[snip]
Actually I'm thinking about Doug's idea to use rdma_transport_is_XX()
instead of the current basic helper, thus may
On 03/31/2015 03:42 PM, Tom Talpey wrote:
On 3/31/2015 7:41 AM, Michael Wang wrote:
[snip]
Yes, in fact the iWARP protocol does not preclude multiple read SGEs,
even though most iWARP implementations have chosen to support just one.
Even for multi-SGE-capable adapters, there is a limit of
On Mon, Mar 30, 2015 at 12:13:00PM -0400, Doug Ledford wrote:
On Fri, 2015-03-27 at 16:47 +0100, Michael Wang wrote:
Introduce helper has_iwarp() to help us check if an IB device
support IWARP protocol.
This is a needless redirection. Just stick with the original
On Mon, Mar 30, 2015 at 05:10:12PM +0200, Michael Wang wrote:
> I found that actually we don't have to touch this one which
> only used by HW driver currently.
I'm having a hard time understanding this, the code in question was in
net/sunrpc/xprtrdma/svc_rdma_recvfrom.c
Which is the NFS ULP,
On 03/30/2015 06:13 PM, Doug Ledford wrote:
> On Fri, 2015-03-27 at 16:47 +0100, Michael Wang wrote:
>> Introduce helper has_iwarp() to help us check if an IB device
>> support IWARP protocol.
> This is a needless redirection. Just stick with the original
> rdma_transport_is_iwarp().
Agree, will
On Fri, 2015-03-27 at 16:47 +0100, Michael Wang wrote:
> Introduce helper has_iwarp() to help us check if an IB device
> support IWARP protocol.
This is a needless redirection. Just stick with the original
rdma_transport_is_iwarp().
> Cc: Jason Gunthorpe
> Cc: Doug Ledford
> Cc: Ira Weiny
>
On 03/27/2015 06:29 PM, Jason Gunthorpe wrote:
> On Fri, Mar 27, 2015 at 01:16:31PM -0400, ira.weiny wrote:
>> [snip]
> http://www.spinics.net/lists/linux-rdma/msg22565.html
>
> ''Unlike IB, the iWARP protocol only allows 1 target/sink SGE in an
> rdma read''
>
> It is one of those annoying verbs
On 03/27/2015 06:29 PM, Jason Gunthorpe wrote:
On Fri, Mar 27, 2015 at 01:16:31PM -0400, ira.weiny wrote:
[snip]
http://www.spinics.net/lists/linux-rdma/msg22565.html
''Unlike IB, the iWARP protocol only allows 1 target/sink SGE in an
rdma read''
It is one of those annoying verbs is
On 03/30/2015 06:13 PM, Doug Ledford wrote:
On Fri, 2015-03-27 at 16:47 +0100, Michael Wang wrote:
Introduce helper has_iwarp() to help us check if an IB device
support IWARP protocol.
This is a needless redirection. Just stick with the original
rdma_transport_is_iwarp().
Agree, will leave
On Mon, Mar 30, 2015 at 05:10:12PM +0200, Michael Wang wrote:
I found that actually we don't have to touch this one which
only used by HW driver currently.
I'm having a hard time understanding this, the code in question was in
net/sunrpc/xprtrdma/svc_rdma_recvfrom.c
Which is the NFS ULP, not
On Fri, 2015-03-27 at 16:47 +0100, Michael Wang wrote:
Introduce helper has_iwarp() to help us check if an IB device
support IWARP protocol.
This is a needless redirection. Just stick with the original
rdma_transport_is_iwarp().
Cc: Jason Gunthorpe jguntho...@obsidianresearch.com
Cc: Doug
On Fri, Mar 27, 2015 at 6:16 PM, ira.weiny wrote:
> On Fri, Mar 27, 2015 at 10:13:19AM -0600, Jason Gunthorpe wrote:
>> On Fri, Mar 27, 2015 at 04:47:36PM +0100, Michael Wang wrote:
>> >
>> > Introduce helper has_iwarp() to help us check if an IB device
>> > support IWARP protocol.
>>
>> Should
On Fri, Mar 27, 2015 at 01:16:31PM -0400, ira.weiny wrote:
> On Fri, Mar 27, 2015 at 10:13:19AM -0600, Jason Gunthorpe wrote:
> > On Fri, Mar 27, 2015 at 04:47:36PM +0100, Michael Wang wrote:
> > >
> > > Introduce helper has_iwarp() to help us check if an IB device
> > > support IWARP protocol.
>
On Fri, Mar 27, 2015 at 10:13:19AM -0600, Jason Gunthorpe wrote:
> On Fri, Mar 27, 2015 at 04:47:36PM +0100, Michael Wang wrote:
> >
> > Introduce helper has_iwarp() to help us check if an IB device
> > support IWARP protocol.
>
> Should probably be !has_rdma_read_sges()
>
> True if the device
On 03/27/2015 05:13 PM, Jason Gunthorpe wrote:
> On Fri, Mar 27, 2015 at 04:47:36PM +0100, Michael Wang wrote:
>> Introduce helper has_iwarp() to help us check if an IB device
>> support IWARP protocol.
> Should probably be !has_rdma_read_sges()
>
> True if the device can handle more than one SGE
On Fri, Mar 27, 2015 at 04:47:36PM +0100, Michael Wang wrote:
>
> Introduce helper has_iwarp() to help us check if an IB device
> support IWARP protocol.
Should probably be !has_rdma_read_sges()
True if the device can handle more than one SGE entry on a RDMA READ
work request.
Jason
--
To
Introduce helper has_iwarp() to help us check if an IB device
support IWARP protocol.
Cc: Jason Gunthorpe
Cc: Doug Ledford
Cc: Ira Weiny
Cc: Sean Hefty
Signed-off-by: Michael Wang
---
include/rdma/ib_verbs.h | 13 +
net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 2
On Fri, Mar 27, 2015 at 01:16:31PM -0400, ira.weiny wrote:
On Fri, Mar 27, 2015 at 10:13:19AM -0600, Jason Gunthorpe wrote:
On Fri, Mar 27, 2015 at 04:47:36PM +0100, Michael Wang wrote:
Introduce helper has_iwarp() to help us check if an IB device
support IWARP protocol.
Should
On Fri, Mar 27, 2015 at 6:16 PM, ira.weiny ira.we...@intel.com wrote:
On Fri, Mar 27, 2015 at 10:13:19AM -0600, Jason Gunthorpe wrote:
On Fri, Mar 27, 2015 at 04:47:36PM +0100, Michael Wang wrote:
Introduce helper has_iwarp() to help us check if an IB device
support IWARP protocol.
Introduce helper has_iwarp() to help us check if an IB device
support IWARP protocol.
Cc: Jason Gunthorpe jguntho...@obsidianresearch.com
Cc: Doug Ledford dledf...@redhat.com
Cc: Ira Weiny ira.we...@intel.com
Cc: Sean Hefty sean.he...@intel.com
Signed-off-by: Michael Wang
On Fri, Mar 27, 2015 at 10:13:19AM -0600, Jason Gunthorpe wrote:
On Fri, Mar 27, 2015 at 04:47:36PM +0100, Michael Wang wrote:
Introduce helper has_iwarp() to help us check if an IB device
support IWARP protocol.
Should probably be !has_rdma_read_sges()
True if the device can handle
On 03/27/2015 05:13 PM, Jason Gunthorpe wrote:
On Fri, Mar 27, 2015 at 04:47:36PM +0100, Michael Wang wrote:
Introduce helper has_iwarp() to help us check if an IB device
support IWARP protocol.
Should probably be !has_rdma_read_sges()
True if the device can handle more than one SGE entry on
On Fri, Mar 27, 2015 at 04:47:36PM +0100, Michael Wang wrote:
Introduce helper has_iwarp() to help us check if an IB device
support IWARP protocol.
Should probably be !has_rdma_read_sges()
True if the device can handle more than one SGE entry on a RDMA READ
work request.
Jason
--
To
32 matches
Mail list logo