alloc_page(GFP_ATOMIC) is safe. it doesnt give you a highmempage Not sure about header splitting. If done, it probably should be smart as you said (allowing small frames to be in the first part)
On Wed, Apr 17, 2013 at 2:49 PM, Roland Dreier <[email protected]> wrote: > On Wed, Apr 17, 2013 at 12:10 PM, Eric Dumazet <[email protected]> wrote: >> + prefetch(page_address(frag->page.p)); > > I guess we would have to change our allocation to __get_free_page() > (instead of alloc_page()), so that we make sure never to get a highmem > page? > > Other than that, seems fine. > > By the way, is there any value to header splitting -- having the > adapter put the TCP/IP headers in the linear part and the payload in > the fragment page? How about in the case where we have a small enough > MTU that we could put the whole packet in the linear part? > > - R. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
