linux-kernel - Re: [PATCH 0/5] Switchtec Fixes and 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: <b4908c93-fb43-001b-b3eb-d3da0bb377c9@deltatee.com>
Date:   Fri, 24 Sep 2021 09:53:27 -0600
From:   Logan Gunthorpe <logang@...tatee.com>
To:     kelvin.cao@...rochip.com, kurt.schwemmer@...rosemi.com,
        bhelgaas@...gle.com, linux-pci@...r.kernel.org,
        linux-kernel@...r.kernel.org
Cc:     kelvincao@...look.com
Subject: Re: [PATCH 0/5] Switchtec Fixes and Improvements



On 2021-09-24 5:08 a.m., kelvin.cao@...rochip.com wrote:
> From: Kelvin Cao <kelvin.cao@...rochip.com>
> 
> Hi,
> 
> Please find a bunch of patches for the switchtec driver collected over the
> last few months.
> 
> The first 2 patches fix two minor bugs. Patch 3 updates the method of
> getting management VEP instance ID based on a new firmware change. Patch
> 4 replaces ENOTSUPP with EOPNOTSUPP to follow the preference of kernel.
> And the last patch adds check of event support to align with new firmware
> implementation.
> 
> This patchset is based on v5.15-rc2.
> 
> Thanks,
> Kelvin
> 
> Kelvin Cao (4):
>   PCI/switchtec: Error out MRPC execution when no GAS access
>   PCI/switchtec: Fix a MRPC error status handling issue
>   PCI/switchtec: Update the way of getting management VEP instance ID
>   PCI/switchtec: Replace ENOTSUPP with EOPNOTSUPP
> 
> Logan Gunthorpe (1):
>   PCI/switchtec: Add check of event support
> 
>  drivers/pci/switch/switchtec.c | 87 +++++++++++++++++++++++++++-------
>  include/linux/switchtec.h      |  1 +
>  2 files changed, 71 insertions(+), 17 deletions(-)

Thanks Kelvin! This all looks good to me.

For the entire series (except the last patch which I wrote):

Reviewed-by: Logan Gunthorpe <logang@...tatee.com>

Logan