netdev - Re: [PATCH net-next v3 0/2] gve: Handle alternate miss-completions

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: <Y3R3PAwDe+6fFbBw@x130.lan>
Date:   Tue, 15 Nov 2022 21:38:04 -0800
From:   Saeed Mahameed <saeed@...nel.org>
To:     Jeroen de Borst <jeroendb@...gle.com>
Cc:     netdev@...r.kernel.org, davem@...emloft.net, kuba@...nel.org,
        jesse.brandeburg@...el.com
Subject: Re: [PATCH net-next v3 0/2] gve: Handle alternate miss-completions

On 14 Nov 15:35, Jeroen de Borst wrote:
>Some versions of the virtual NIC present miss-completions in
>an alternative way. Let the diver handle these alternate completions
>and announce this capability to the device.
>

nit: you missed to document the 1st "new AdminQ command" patch which has
some special logic to validate driver/os version compatibility with the
device and if it's related to this cover-letter title if at all.

>Changed in v3:
>- Rewording cover letter
>- Added 'Reviewed-by: Jesse Brandeburg <jesse.brandeburg@...el.com>'
>Changes in v2:
>- Changed the subject to include 'gve:'
>
>
>Jeroen de Borst (2):
>  gve: Adding a new AdminQ command to verify driver
>  gve: Handle alternate miss completions
>
> drivers/net/ethernet/google/gve/gve.h         |  1 +
> drivers/net/ethernet/google/gve/gve_adminq.c  | 19 +++++++
> drivers/net/ethernet/google/gve/gve_adminq.h  | 51 ++++++++++++++++++
> .../net/ethernet/google/gve/gve_desc_dqo.h    |  5 ++
> drivers/net/ethernet/google/gve/gve_main.c    | 52 +++++++++++++++++++
> drivers/net/ethernet/google/gve/gve_tx_dqo.c  | 18 ++++---
> 6 files changed, 140 insertions(+), 6 deletions(-)
>
>-- 
>2.38.1.431.g37b22c650d-goog
>