Re: [PATCH v2] net: add skb allocation flags to pskb_copy

[Christoph Paasch <christoph.paasch@uclouvain.be>]

On 08/06/14 - 20:56:18, Octavian Purdila wrote:
> There are several instances where a pskb_copy or __pskb_copy is
> immediately followed by an skb_clone. Add a new parameter to allow the
> skb to be allocated from the fclone cache and thus speed up subsequent
> skb_clone calls.
> 
> Cc: Alexander Smirnov <alex.bluesman.smirnov@gmail.com>
> Cc: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>
> Cc: Marek Lindner <mareklindner@neomailbox.ch>
> Cc: Simon Wunderlich <sw@simonwunderlich.de>
> Cc: Antonio Quartulli <antonio@meshcoding.com>
> Cc: Marcel Holtmann <marcel@holtmann.org>
> Cc: Gustavo Padovan <gustavo@padovan.org>
> Cc: Johan Hedberg <johan.hedberg@gmail.com>
> Cc: Arvid Brodin <arvid.brodin@alten.se>
> Cc: Patrick McHardy <kaber@trash.net>
> Cc: Pablo Neira Ayuso <pablo@netfilter.org>
> Cc: Jozsef Kadlecsik <kadlec@blackhole.kfki.hu>
> Cc: Lauro Ramos Venancio <lauro.venancio@openbossa.org>
> Cc: Aloisio Almeida Jr <aloisio.almeida@openbossa.org>
> Cc: Samuel Ortiz <sameo@linux.intel.com>
> Cc: Jon Maloy <jon.maloy@ericsson.com>
> Cc: Allan Stephens <allan.stephens@windriver.com>
> Cc: Andrew Hendry <andrew.hendry@gmail.com>
> Cc: Eric Dumazet <edumazet@google.com>
> 
> Signed-off-by: Octavian Purdila <octavian.purdila@intel.com>
> ---
>  drivers/net/ieee802154/fakelb.c        | 2 +-
>  include/linux/skbuff.h                 | 8 ++++----
>  net/batman-adv/distributed-arp-table.c | 2 +-
>  net/batman-adv/network-coding.c        | 2 +-
>  net/bluetooth/hci_sock.c               | 5 +++--
>  net/core/skbuff.c                      | 9 ++++++---
>  net/hsr/hsr_device.c                   | 2 +-
>  net/hsr/hsr_main.c                     | 2 +-
>  net/ipv4/tcp_output.c                  | 6 +++---
>  net/netfilter/xt_TEE.c                 | 4 ++--
>  net/nfc/llcp_core.c                    | 2 +-
>  net/nfc/rawsock.c                      | 2 +-
>  net/tipc/bcast.c                       | 2 +-
>  net/x25/x25_forward.c                  | 4 ++--
>  14 files changed, 28 insertions(+), 24 deletions(-)

Reviewed-by: Christoph Paasch <christoph.paasch@uclouvain.be>


Cheers,
Christoph