netdev - Re: [PATCH net-next 08/11] bnxt_en: Add partno and serialno to devlink info

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: <20200317105103.2e46da90@kicinski-fedora-pc1c0hjn.dhcp.thefacebook.com>
Date:   Tue, 17 Mar 2020 10:51:03 -0700
From:   Jakub Kicinski <kuba@...nel.org>
To:     Vasundhara Volam <vasundhara-v.volam@...adcom.com>
Cc:     davem@...emloft.net, netdev@...r.kernel.org,
        Michael Chan <michael.chan@...adcom.com>
Subject: Re: [PATCH net-next 08/11] bnxt_en: Add partno and serialno to
 devlink info_get cb

On Tue, 17 Mar 2020 20:47:23 +0530 Vasundhara Volam wrote:
> Add part number and serial number info from the vital product data
> to info_get command via devlink tool.
> 
> Some of the broadcom devices support both PCI extended config space
> for device serial number and VPD serial number. With this patch, both
> the information will be displayed via info_get cb.
> 
> Update bnxt.rst documentation as well.
> 
> Example display:
> 
> $ devlink dev info pci/0000:3b:00.1
> pci/0000:3b:00.1:
>   driver bnxt_en
>   serial_number B0-26-28-FF-FE-C8-85-20
>   versions:
>       fixed:
>         board.id BCM957508-P2100G
>         asic.id 1750
>         asic.rev 1
>         vpd.serialno P2100pm01920A0032CQ

This looks like it's a concatenation of multiple things. Isn't it?

>       running:
>         drv.spec 1.10.1.12
>         hw.addr b0:26:28:c8:85:21
>         hw.mh_base_addr b0:26:28:c8:85:20
>         fw 216.0.286.0
>         fw.psid 0.0.6
>         fw.app 216.0.251.0
> 
> Signed-off-by: Vasundhara Volam <vasundhara-v.volam@...adcom.com>
> Signed-off-by: Michael Chan <michael.chan@...adcom.com>