netdev - Re: [PATCH net-next 0/5] ionic: more driver fixes

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]

Message-ID: <87ttoxuv51.fsf@nvidia.com>
Date: Mon, 04 Dec 2023 20:32:26 -0800
From: Rahul Rameshbabu <rrameshbabu@...dia.com>
To: Shannon Nelson <shannon.nelson@....com>
Cc: <netdev@...r.kernel.org>,  <davem@...emloft.net>,  <kuba@...nel.org>,
  <edumazet@...gle.com>,  <pabeni@...hat.com>,  <f.fainelli@...il.com>,
  <brett.creeley@....com>,  <drivers@...sando.io>
Subject: Re: [PATCH net-next 0/5] ionic: more driver fixes

On Mon, 04 Dec, 2023 13:09:31 -0800 Shannon Nelson <shannon.nelson@....com> wrote:
> These are a few code cleanup items that appeared first in a
> separate net patchset,
>     https://lore.kernel.org/netdev/20231201000519.13363-1-shannon.nelson@amd.com/
> but are now aimed for net-next.
>
> Brett Creeley (4):
>   ionic: Use cached VF attributes

Was on the fence about whether this patch should be considered a "fix"
or not. I think it makes sense that it is a performance enhancement.

>   ionic: Don't check null when calling vfree()
>   ionic: Make the check for Tx HW timestamping more obvious
>   ionic: Re-arrange ionic_intr_info struct for cache perf
>
> Shannon Nelson (1):
>   ionic: set ionic ptr before setting up ethtool ops
>
>  drivers/net/ethernet/pensando/ionic/ionic.h   |   2 -
>  .../net/ethernet/pensando/ionic/ionic_dev.c   |  40 -------
>  .../net/ethernet/pensando/ionic/ionic_dev.h   |   7 +-
>  .../net/ethernet/pensando/ionic/ionic_lif.c   | 107 +++---------------
>  .../net/ethernet/pensando/ionic/ionic_lif.h   |   5 +
>  .../net/ethernet/pensando/ionic/ionic_main.c  |  22 ----
>  .../net/ethernet/pensando/ionic/ionic_txrx.c  |  10 +-
>  7 files changed, 28 insertions(+), 165 deletions(-)

Reviewed-by: Rahul Rameshbabu <rrameshbabu@...dia.com>