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] [day] [month] [year] [list]
Date:   Wed, 6 Feb 2019 09:20:10 -0800
From:   David Ahern <dsahern@...il.com>
To:     Jakub Kicinski <jakub.kicinski@...ronome.com>, jiri@...nulli.us
Cc:     stephen@...workplumber.org, netdev@...r.kernel.org,
        oss-drivers@...ronome.com
Subject: Re: [PATCH iproute2-next v5] devlink: add info subcommand

On 2/4/19 8:10 AM, Jakub Kicinski wrote:
> Add support for reading the device serial number, driver name
> and various versions.  Example:
> 
> $ devlink dev info pci/0000:82:00.0
> pci/0000:82:00.0:
>   driver nfp
>   serial_number 16240145
>   versions:
>       fixed:
>         board.id AMDA0081-0001
>         board.rev 15
>         board.vendor SMA
>         board.model hydrogen
>       running:
>         fw.mgmt 010181.010181.0101d4
>         fw.cpld 0x1030000
>         fw.app abm-d372b6
>         fw.undi 0.0.2
>         chip.init AMDA-0081-0001  20160318164536
>       stored:
>         fw.mgmt 010181.010181.0101d4
>         fw.app abm-d372b6
>         fw.undi 0.0.2
>         chip.init AMDA-0081-0001  20160318164536
> 
> $ devlink -jp dev info pci/0000:82:00.0
> {
>     "info": {
>         "pci/0000:82:00.0": {
>             "driver": "nfp",
>             "serial_number": "16240145",
>             "versions": {
>                 "fixed": {
>                     "board.id": "AMDA0081-0001",
>                     "board.rev": "15",
>                     "board.vendor": "SMA",
>                     "board.model": "hydrogen"
>                 },
>                 "running": {
>                     "fw.mgmt": "010181.010181.0101d4",
>                     "fw.cpld": "0x1030000",
>                     "fw.app": "abm-d372b6",
>                     "fw.undi": "0.0.2",
>                     "chip.init": "AMDA-0081-0001  20160318164536"
>                 },
>                 "stored": {
>                     "fw.mgmt": "010181.010181.0101d4",
>                     "fw.app": "abm-d372b6",
>                     "fw.undi": "0.0.2",
>                     "chip.init": "AMDA-0081-0001  20160318164536"
>                 }
>             }
>         }
>     }
> }
> 
> v5:
>  - remove spurious new line.
> v4:
>  - more commit message improvements.
> v3:
>  - show up-to-date output in the commit message.
> v2 (Jiri):
>  - remove filtering;
>  - add example in the commit message.
> RFCv2:
>  - make info subcommand of dev.
> 
> Signed-off-by: Jakub Kicinski <jakub.kicinski@...ronome.com>
> ---
>  devlink/devlink.c      | 171 +++++++++++++++++++++++++++++++++++++++++
>  man/man8/devlink-dev.8 |  26 +++++++
>  2 files changed, 197 insertions(+)
> 
applied to iproute2-next.
Thanks for adding the example output

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ