netdev - Re: [PATCH net-next v1 1/2] net: introduce and use skb_frag_fill_page

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: Tue, 9 May 2023 16:43:11 +0300
From: Leon Romanovsky <leon@...nel.org>
To: Yunsheng Lin <linyunsheng@...wei.com>
Cc: davem@...emloft.net, kuba@...nel.org, pabeni@...hat.com,
	netdev@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH net-next v1 1/2] net: introduce and use
 skb_frag_fill_page_desc()

On Tue, May 09, 2023 at 07:43:36PM +0800, Yunsheng Lin wrote:
> Most users use __skb_frag_set_page()/skb_frag_off_set()/
> skb_frag_size_set() to fill the page desc for a skb frag.
> 
> Introduce skb_frag_fill_page_desc() to do that.
> 
> net/bpf/test_run.c does not call skb_frag_off_set() to
> set the offset, "copy_from_user(page_address(page), ...)"
> suggest that it is assuming offset to be initialized as
> zero, so call skb_frag_fill_page_desc() with offset being
> zero for this case.
> 
> Also, skb_frag_set_page() is not used anymore, so remove
> it.
> 
> Signed-off-by: Yunsheng Lin <linyunsheng@...wei.com>
> ---
>  .../net/ethernet/aquantia/atlantic/aq_ring.c  |  6 ++--
>  drivers/net/ethernet/broadcom/bnxt/bnxt.c     |  5 ++-
>  drivers/net/ethernet/chelsio/cxgb3/sge.c      |  5 ++-
>  drivers/net/ethernet/emulex/benet/be_main.c   | 32 ++++++++++---------
>  drivers/net/ethernet/freescale/enetc/enetc.c  |  5 ++-
>  .../net/ethernet/fungible/funeth/funeth_rx.c  |  5 ++-
>  drivers/net/ethernet/marvell/mvneta.c         |  5 ++-
>  .../net/ethernet/mellanox/mlx5/core/en_rx.c   |  4 +--
>  drivers/net/ethernet/sun/cassini.c            |  8 ++---
>  drivers/net/virtio_net.c                      |  4 +--
>  drivers/net/vmxnet3/vmxnet3_drv.c             |  4 +--
>  drivers/net/xen-netback/netback.c             |  4 +--
>  include/linux/skbuff.h                        | 27 ++++++----------
>  net/bpf/test_run.c                            |  3 +-
>  net/core/gro.c                                |  4 +--
>  net/core/pktgen.c                             | 13 +++++---
>  net/core/skbuff.c                             |  7 ++--
>  net/tls/tls_device.c                          | 10 +++---
>  net/xfrm/xfrm_ipcomp.c                        |  5 +--
>  19 files changed, 64 insertions(+), 92 deletions(-)
> 

Thanks,
Reviewed-by: Leon Romanovsky <leonro@...dia.com>