On Thu, Feb 9, 2017 at 4:28 AM, Tariq Toukan <[email protected]> wrote:
>
>
> On 07/02/2017 5:02 AM, Eric Dumazet wrote:
>>
>> + for (nr = 0;;) {
>> + frag_size = min_t(int, length, frag_info->frag_size);
>> +
>> if (unlikely(!frags[nr].page))
>> goto fail;
>> @@ -603,15 +602,16 @@ static int mlx4_en_complete_rx_desc(struct
>> mlx4_en_priv *priv,
>> __skb_fill_page_desc(skb, nr, frags[nr].page,
>> frags[nr].page_offset,
>> - frag_info->frag_size);
>> + frag_size);
>
> Same for dma_sync_single (two lines above).
Oh right, although I was not sure it this really matter on a part of a page.