[ofa-general] Re: [PATCH 1/11] IB/ipoib: high dma support

[Or Gerlitz]

Eli Cohen wrote:
> On Sun, 2007-09-30 at 11:13 +0200, Or Gerlitz wrote: 
> If I understand correctly, there were two kinds of flags discussed in
> this thread. One of them is using __GFP_HIGHMEM for allocating buffers
> to be utilized in the receive flow of ipoib cm which it was agreed that
> will hurt performance.

> The second one is using a capability flag which devices use to declare
> that they support high dma - this one is not required too but implies
> that ipath will not work on 32 bit platforms (e.g. i386 with PAE).

What I would like to see agreement on is if we don't require this flag 
from the rdma device, how a user is being prevent from running into 
troubles when using ipoib/ipath/32bit/high-mem. If such config is 
impossible, that's fine.

> Saying that, Or, can you clarify how would you like the change-long
> comment to be changed?

something like:

This patch assumes all IB devices support DMA to high-mem pages

Or.