netdev - Re: [mm PATCH 0/3] Page fragment updates

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 5 Dec 2016 09:01:12 -0800
From:   Alexander Duyck <alexander.duyck@...il.com>
To:     linux-mm <linux-mm@...ck.org>,
        Andrew Morton <akpm@...ux-foundation.org>
Cc:     Netdev <netdev@...r.kernel.org>,
        Eric Dumazet <edumazet@...gle.com>,
        David Miller <davem@...emloft.net>,
        Jeff Kirsher <jeffrey.t.kirsher@...el.com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [mm PATCH 0/3] Page fragment updates

On Tue, Nov 29, 2016 at 10:23 AM, Alexander Duyck
<alexander.duyck@...il.com> wrote:
> This patch series takes care of a few cleanups for the page fragments API.
>
> First we do some renames so that things are much more consistent.  First we
> move the page_frag_ portion of the name to the front of the functions
> names.  Secondly we split out the cache specific functions from the other
> page fragment functions by adding the word "cache" to the name.
>
> Second I did some minor clean-up on the function calls so that they are
> more inline with the existing __free_pages calls in terms of how they
> operate.
>
> Finally I added a bit of documentation that will hopefully help to explain
> some of this.  I plan to revisit this later as we get things more ironed
> out in the near future with the changes planned for the DMA setup to
> support eXpress Data Path.
>
> ---
>
> Alexander Duyck (3):
>       mm: Rename __alloc_page_frag to page_frag_alloc and __free_page_frag to page_frag_free
>       mm: Rename __page_frag functions to __page_frag_cache, drop order from drain
>       mm: Add documentation for page fragment APIs
>
>
>  Documentation/vm/page_frags               |   42 +++++++++++++++++++++++++++++
>  drivers/net/ethernet/intel/igb/igb_main.c |    6 ++--
>  include/linux/gfp.h                       |    9 +++---
>  include/linux/skbuff.h                    |    2 +
>  mm/page_alloc.c                           |   33 +++++++++++++----------
>  net/core/skbuff.c                         |    8 +++---
>  6 files changed, 73 insertions(+), 27 deletions(-)
>  create mode 100644 Documentation/vm/page_frags
>
> --

It's been about a week since I submitted this series.  Just wanted to
check in and see if anyone had any feedback or if this is good to be
accepted for 4.10-rc1 with the rest of the set?

Thanks.

- Alex