On Mon, Feb 13, 2017 at 12:51 PM, Alexander Duyck <alexander.du...@gmail.com> wrote: > On Mon, Feb 13, 2017 at 11:58 AM, Eric Dumazet <eduma...@google.com> wrote:
>> + PAGE_SIZE, priv->dma_dir); >> page = page_alloc->page; >> /* Revert changes done by mlx4_alloc_pages */ >> - page_ref_sub(page, page_alloc->page_size / >> + page_ref_sub(page, PAGE_SIZE / >> priv->frag_info[i].frag_stride - 1); >> put_page(page); >> page_alloc->page = NULL; > > You can probably simplify this by using __page_frag_cache_drain since > that way you can just doe the sub and test for the whole set instead > of doing N - 1 and 1. Well, we remove this anyway in following patch ;)