netdev - Re: [PATCH v5 intel-next 0/9] XDP

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 for Android: free password hash cracker in your pocket
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20210818075256.GA16780@ranger.igk.intel.com>
Date:   Wed, 18 Aug 2021 09:52:56 +0200
From:   Maciej Fijalkowski <maciej.fijalkowski@...el.com>
To:     "Nguyen, Anthony L" <anthony.l.nguyen@...el.com>
Cc:     "intel-wired-lan@...ts.osuosl.org" <intel-wired-lan@...ts.osuosl.org>,
        "toke@...hat.com" <toke@...hat.com>,
        "Karlsson, Magnus" <magnus.karlsson@...el.com>,
        "davem@...emloft.net" <davem@...emloft.net>,
        "Lobakin, Alexandr" <alexandr.lobakin@...el.com>,
        "bjorn@...nel.org" <bjorn@...nel.org>,
        "Brandeburg, Jesse" <jesse.brandeburg@...el.com>,
        "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
        "bpf@...r.kernel.org" <bpf@...r.kernel.org>,
        "kuba@...nel.org" <kuba@...nel.org>,
        "Creeley, Brett" <brett.creeley@...el.com>,
        "joamaki@...il.com" <joamaki@...il.com>
Subject: Re: [PATCH v5 intel-next 0/9] XDP_TX improvements for ice

On Tue, Aug 17, 2021 at 09:59:01PM +0100, Nguyen, Anthony L wrote:
> On Sat, 2021-08-14 at 16:08 +0200, Maciej Fijalkowski wrote:
> > With the v5, I think it's time for a proper change log.
> 
> This isn't applying to the Intel-wired-LAN tree. If you want it to go
> through there, could you base the patches on that tree?

Interestingly this is the first time that happens to me and I always based
my XDP related driver work on bpf-next.

iwl tree is some standalone tree or is it just the net-next ?

> 
> Also, looking at NIPA, it looks like patches 2 and 3 have kdoc issues.

Yeah I saw kdoc issue on patch 3 and wanted to ask you to fix this if you
would be applying that set but given that you're asking for a re-submit
i'll fix those by myself.

> 
> Thanks,
> Tony
> 
> > v4->v5:
> > * fix issues pointed by lkp; variables used for updating ring stats
> >   could be un-inited
> > * s/ice_ring/ice_rx_ring; it looks now symmetric given that we have
> >   ice_tx_ring struct dedicated for Tx ring
> > * go through the code and use ice_for_each_* macros; it was spotted
> > by
> >   Brett that there was a place around that code that this set is
> >   touching that was not using the ice_for_each_txq. Turned out that
> > there
> >   were more such places
> > * take care of coalesce related code; carry the info about type of
> > ring
> >   container in ice_ring_container
> > * pull out getting rid of @ring_active onto separate patch, as
> > suggested
> >   by Brett
> >
> > v3->v4:
> > * fix lkp issues;
> >
> > v2->v3:
> > * improve XDP_TX in a proper way
> > * split ice_ring
> > * propagate XDP ring pointer to Rx ring
> >
> > v1->v2:
> > * try to improve XDP_TX processing
> >
> > v4 :
> > https://lore.kernel.org/bpf/20210806095539.34423-1-maciej.fijalkowski@intel.com/
> > v3 :
> > https://lore.kernel.org/bpf/20210805230046.28715-1-maciej.fijalkowski@intel.com/
> > v2 :
> > https://lore.kernel.org/bpf/20210705164338.58313-1-maciej.fijalkowski@intel.com/
> > v1 :
> > https://lore.kernel.org/bpf/20210601113236.42651-1-maciej.fijalkowski@intel.com/
> >
> > Thanks!
> > Maciej
> >
> > Maciej Fijalkowski (9):
> >   ice: remove ring_active from ice_ring
> >   ice: move ice_container_type onto ice_ring_container
> >   ice: split ice_ring onto Tx/Rx separate structs
> >   ice: unify xdp_rings accesses
> >   ice: do not create xdp_frame on XDP_TX
> >   ice: propagate xdp_ring onto rx_ring
> >   ice: optimize XDP_TX workloads
> >   ice: introduce XDP_TX fallback path
> >   ice: make use of ice_for_each_* macros
> >
> >  drivers/net/ethernet/intel/ice/ice.h          |  41 +++-
> >  drivers/net/ethernet/intel/ice/ice_arfs.c     |   2 +-
> >  drivers/net/ethernet/intel/ice/ice_base.c     |  51 ++---
> >  drivers/net/ethernet/intel/ice/ice_base.h     |   8 +-
> >  drivers/net/ethernet/intel/ice/ice_dcb_lib.c  |   9 +-
> >  drivers/net/ethernet/intel/ice/ice_dcb_lib.h  |  10 +-
> >  drivers/net/ethernet/intel/ice/ice_ethtool.c  |  93 +++++----
> >  drivers/net/ethernet/intel/ice/ice_lib.c      |  88 +++++----
> >  drivers/net/ethernet/intel/ice/ice_lib.h      |   6 +-
> >  drivers/net/ethernet/intel/ice/ice_main.c     | 142 +++++++++-----
> >  drivers/net/ethernet/intel/ice/ice_ptp.c      |   2 +-
> >  drivers/net/ethernet/intel/ice/ice_ptp.h      |   4 +-
> >  drivers/net/ethernet/intel/ice/ice_trace.h    |  28 +--
> >  drivers/net/ethernet/intel/ice/ice_txrx.c     | 183 +++++++++++-----
> > --
> >  drivers/net/ethernet/intel/ice/ice_txrx.h     | 126 +++++++-----
> >  drivers/net/ethernet/intel/ice/ice_txrx_lib.c |  98 ++++++++--
> >  drivers/net/ethernet/intel/ice/ice_txrx_lib.h |  14 +-
> >  .../net/ethernet/intel/ice/ice_virtchnl_pf.c  |   2 +-
> >  drivers/net/ethernet/intel/ice/ice_xsk.c      |  70 ++++---
> >  drivers/net/ethernet/intel/ice/ice_xsk.h      |  20 +-
> >  20 files changed, 607 insertions(+), 390 deletions(-)
> >