Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 19 Sep 2012 12:28:51 -0700
From:      Jim Harris <jim.harris@gmail.com>
To:        Gavin Atkinson <gavin@freebsd.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r240699 - in head: sys/dev/pci usr.sbin/pciconf
Message-ID:  <CAJP=Hc-uEeYKHpXLmXpr4FpMqupF=FhYRq-OQo=4F2K9NHvWOQ@mail.gmail.com>
In-Reply-To: <201209191822.q8JIMFEi043012@svn.freebsd.org>
References:  <201209191822.q8JIMFEi043012@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Sep 19, 2012 at 11:22 AM, Gavin Atkinson <gavin@freebsd.org> wrote:
> Author: gavin
> Date: Wed Sep 19 18:22:14 2012
> New Revision: 240699
> URL: http://svn.freebsd.org/changeset/base/240699
>
> Log:
>   Recognise NVM Express devices and pretty-print their name.
>
>   MFC after:    1 week
>
> Modified:
>   head/sys/dev/pci/pci.c
>   head/usr.sbin/pciconf/pciconf.c
>
> Modified: head/sys/dev/pci/pci.c
> ==============================================================================
> --- head/sys/dev/pci/pci.c      Wed Sep 19 16:28:38 2012        (r240698)
> +++ head/sys/dev/pci/pci.c      Wed Sep 19 18:22:14 2012        (r240699)
> @@ -3661,6 +3661,7 @@ static struct
>         {PCIC_STORAGE,          PCIS_STORAGE_ATA_ADMA,  "ATA (ADMA)"},
>         {PCIC_STORAGE,          PCIS_STORAGE_SATA,      "SATA"},
>         {PCIC_STORAGE,          PCIS_STORAGE_SAS,       "SAS"},
> +       {PCIC_STORAGE,          PCIS_STORAGE_NVM,       "NVM Express"},

Hi Gavin,

This should really read "NVM", not "NVM Express".  NVM Express should
be reserved for
NVM subclass devices with the NVM Express PI (Programming Interface).

PCI SIG refers to this PI as "Enterprise NVMHCI", but it's the same thing.

Regards,

-Jim


>         {PCIC_NETWORK,          -1,                     "network"},
>         {PCIC_NETWORK,          PCIS_NETWORK_ETHERNET,  "ethernet"},
>         {PCIC_NETWORK,          PCIS_NETWORK_TOKENRING, "token ring"},
>
> Modified: head/usr.sbin/pciconf/pciconf.c
> ==============================================================================
> --- head/usr.sbin/pciconf/pciconf.c     Wed Sep 19 16:28:38 2012        (r240698)
> +++ head/usr.sbin/pciconf/pciconf.c     Wed Sep 19 18:22:14 2012        (r240699)
> @@ -345,6 +345,7 @@ static struct
>         {PCIC_STORAGE,          PCIS_STORAGE_ATA_ADMA,  "ATA (ADMA)"},
>         {PCIC_STORAGE,          PCIS_STORAGE_SATA,      "SATA"},
>         {PCIC_STORAGE,          PCIS_STORAGE_SAS,       "SAS"},
> +       {PCIC_STORAGE,          PCIS_STORAGE_NVM,       "NVM Express"},
>         {PCIC_NETWORK,          -1,                     "network"},
>         {PCIC_NETWORK,          PCIS_NETWORK_ETHERNET,  "ethernet"},
>         {PCIC_NETWORK,          PCIS_NETWORK_TOKENRING, "token ring"},



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJP=Hc-uEeYKHpXLmXpr4FpMqupF=FhYRq-OQo=4F2K9NHvWOQ>