Am 08.04.13 18:50 schrieb "Roland Dreier" unter <rol...@kernel.org>:
>On Mon, Apr 8, 2013 at 9:44 AM, Eric Dumazet <eduma...@google.com> wrote: >> Am empty page in the frag list ? You mean a frag with a zero length ? > >Yep... the code would do > > skb_fill_page_desc(skb, 0, page, 0, PAGE_SIZE); > >but then later on it did the equivalent of > > skb_frag_size_set(frag, 0); > skb->truesize += PAGE_SIZE; > >and somehow when data is flowing both directions with small packets, >that would lead the RX-side handling to return corrupt data to >userspace. > >Not sure if it's worth figuring this out entirely ... the more >efficient way for ipoib to do things is to reuse that page for next >time, not pass it up the stack unused. > > - R. First I thought perfect memory allocation inside ipoib_alloc_rx_skb() could be realized by passing the received packet size from function ipoib_ib_handle_rx_wc(). But I'm unable to estimate the requirements of the similar call inside ipoib_ib_post_receives(). Nevertheless thank you very much for spending time on this. Markus -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
