netdev - Re: [PATCH net-next 00/15] bnxt_en: Error recovery improvements.

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: <CAF=yD-KFe+QAb5JkK1xYUTzjgL32cOWUEqsX3qJrbg3ky-ZPrQ@mail.gmail.com>
Date:   Mon, 25 Jan 2021 20:37:52 -0500
From:   Willem de Bruijn <willemdebruijn.kernel@...il.com>
To:     Michael Chan <michael.chan@...adcom.com>
Cc:     David Miller <davem@...emloft.net>,
        Network Development <netdev@...r.kernel.org>,
        Jakub Kicinski <kuba@...nel.org>, gospo@...adcom.com
Subject: Re: [PATCH net-next 00/15] bnxt_en: Error recovery improvements.

On Mon, Jan 25, 2021 at 3:36 AM Michael Chan <michael.chan@...adcom.com> wrote:
>
> This series contains a number of improvements in the area of error
> recovery.  Most error recovery scenarios are tightly coordinated with
> the firmware.  A number of patches add retry logic to establish
> connection with the firmware if there are indications that the
> firmware is still alive and will likely transition back to the
> normal state.  Some patches speed up the recovery process and make
> it more reliable.  There are some cleanup patches as well.
>
> Edwin Peer (3):
>   bnxt_en: handle CRASH_NO_MASTER during bnxt_open()
>   bnxt_en: log firmware debug notifications
>   bnxt_en: attempt to reinitialize after aborted reset
>
> Michael Chan (9):
>   bnxt_en: Update firmware interface to 1.10.2.11.
>   bnxt_en: Define macros for the various health register states.
>   bnxt_en: Retry sending the first message to firmware if it is under
>     reset.
>   bnxt_en: Add bnxt_fw_reset_timeout() helper.
>   bnxt_en: Add a new BNXT_STATE_NAPI_DISABLED flag to keep track of NAPI
>     state.
>   bnxt_en: Modify bnxt_disable_int_sync() to be called more than once.
>   bnxt_en: Improve firmware fatal error shutdown sequence.
>   bnxt_en: Consolidate firmware reset event logging.
>   bnxt_en: Do not process completion entries after fatal condition
>     detected.
>
> Vasundhara Volam (3):
>   bnxt_en: Move reading VPD info after successful handshake with fw.
>   bnxt_en: Add an upper bound for all firmware command timeouts.
>   bnxt_en: Retry open if firmware is in reset.
>
>  drivers/net/ethernet/broadcom/bnxt/bnxt.c     | 228 ++++++++++++----
>  drivers/net/ethernet/broadcom/bnxt/bnxt.h     |  22 ++
>  .../net/ethernet/broadcom/bnxt/bnxt_devlink.c |   7 +-
>  drivers/net/ethernet/broadcom/bnxt/bnxt_hsi.h | 249 ++++++++++++++----
>  4 files changed, 393 insertions(+), 113 deletions(-)

For netdrv:

Acked-by: Willem de Bruijn <willemb@...gle.com>