Improve MSI and MSI-X display

There are several ways I think the MSI and MSI-X capabilities could be
reported better in lspci.

Enable is at the beginning of the MSI-X report, but the end of the
MSI report.  Because Enabled is perhaps the most important thing,
I moved it to the start.

The TabSize reported for MSI-X is perhaps a little obscure.  I feel
'Count' works better (and is then the same as MSI).  We can't readily
tell which of the MSI-X capabilities have been programmed, so we can't
report how many have been enabled.

Both capabilities report 'Mask'.  Unfortunately, in MSI, it indicates
whether the device supports per-vector masking, and in MSI-X it indicates
whether all the vectors are masked.  This patch changes both; MSI reports
'Maskable' and MSI-X reports 'Masked'.

All these changes combine to turn the report from this:

        Capabilities: [60] MSI: Mask+ 64bit+ Count=1/8 Enable-
        Capabilities: [78] MSI-X: Enable+ Mask- TabSize=8

to this:

        Capabilities: [60] MSI: Enable- Count=1/8 Maskable+ 64bit+
        Capabilities: [78] MSI-X: Enable+ Count=8 Masked-

which I find much less confusing when juxtaposed.

Signed-off-by: Matthew Wilcox <willy@linux.intel.com>

proc: Work around broken /proc/bus/pci/ on some architectures.

Several architectures (e.g., sparc64) use domains in proc directory
names, which is inconsistent, because /proc/bus/pci/devices is still
unable to represent domains.

Work around it, so that the proc back-end works at least on single-domain
machines.

Suggested by Matthew Wilcox.

sysfs: Avoid warnings when the slot address is missing.

lspci: Print vendor-/system-specific VPD item ids correctly

Vendor- and system-specific VPD items are matched only by their first
character, so we cannot read the item id from the matching structure.
There is a similar issue with unknown VPD items, which is handled by
creating a matching structure dynamically.

We can handle both cases using the static matching structure and the
actual item id, so do that.

Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>

Make MSI-X table size match the spec

MSI-X table size is 11 bits, not 10. See also PCI_MSIX_FLAGS_QSIZE
in linux/pci_regs.h. Make PCI_MSIX_TABSIZE match that.

Signed-off-by: Michael S. Tsirkin <mst@redhat.com>

Document that pci_dev->base_addr[] contains addresses with flags.

Correct Device Serial Number output

We are currently printing the Device Serial Number backwards.  Ben
Hutchings noticed that the DSN was an EUI-64 (derived from a MAC
address), but printed backwards.  Before:

        Capabilities: [140] Device Serial Number 97-62-84-ff-ff-3b-1f-00

After:
        Capabilities: [140] Device Serial Number 00-1f-3b-ff-ff-84-62-97

The MAC address is:

wlan0     Link encap:Ethernet  HWaddr 00:1f:3b:84:62:97

Signed-off-by: Matthew Wilcox <willy@linux.intel.com>

Support PM No_Soft_Reset bit in lspci

Signed-off-by: Yu Zhao <yu.zhao@intel.com>

Released as 3.1.2.

One more fix for the syntax error checks in setpci argument parser.

Released as 3.1.1.

Updated pci.ids to the today's snapshot.

configure: Set LC_ALL globally.

Fixes problems with character ranges in calls to `tr' spotted in some
locales by Mike Frysinger.

setpci: Fixed a typo in parsing of values, causing SIGSEGV on some bad inputs.

Patch by Andreas Wiese.

$(PCILIB): respect $(LDFLAGS) when linking

Signed-off-by: Mike Frysinger <vapier@gentoo.org>

Fixed a typo in the ChangeLog.

Released as 3.1.0.

Updated pci.ids to the snapshot of the day.

Updated ChangeLog.

Display some PCIe extended capabilities only when verbose level >= 2

Signed-off-by: Yu Zhao <yu.zhao@intel.com>

Don't display 'disabled' for a virtual base address

Don't display 'disabled' for the virtual base address since the resource
is reported by the OS not the hardware, which means the IO/MEM enable bits
in the command register may be invalid.

Signed-off-by: Yu Zhao <yu.zhao@intel.com>

Remove 64-bit BAR handling code for bus centric view

Remove the redundant 64-bit BAR handling code, since it's already taken
care of by generic/proc/sysfs methods code.

Signed-off-by: Yu Zhao <yu.zhao@intel.com>

Fix inconsistent bus centric view between the sysfs and proc methods

The vendor and device IDs may be different when using sysfs and proc
methods with bus centric view enabled. The sysfs method reads the IDs
from the configuration space rather than uses the values from sysfs
entries. But the proc method always uses values from the proc files.
This fix makes the proc method use the values from the configuration
space too for bus centric view.

Signed-off-by: Yu Zhao <yu.zhao@intel.com>

Fixed configure to work with Cygwin on Windows Vista.

Thanks to Jonathan Kotta.

Updated test cases from Yu Zhao.

PCIUtils: fix SR-IOV capability error in lspci

Fix wrong 'VF Migration' info when displaying SR-IOV capability.

Signed-off-by: Yu Zhao <yu.zhao@intel.com>

Fix base address flags when using sysfs method

The base address flags are ignored when using sysfs method, while
the proc and generic methods combined the flags into 'base_addr'.
This inconsistency may fail some applications using libpci.

Signed-off-by: Yu Zhao <yu.zhao@intel.com>

Released -alpha4.

A fix of the previous cleanup.

sysfs: Cleaned up caching of file descriptors.

The previous code opened the VPD descriptor even if no VPD access
was requested and it forgot to close it afterwards.

Cleaned up detection of physical slots.

Improved error handling. If the kernel does not support the slots directory at
all, we do not complain. Parse errors are now reported as warnings.

Also, the PCI_FILL_PHYS_SLOT is set for all devices whenever we try to detect
the slots, no matter how successful the detection was.

Slots: use strcpy instead of sprintf.

Updated the ChangeLog.

Released as -alpha3.

Man: Mention that the sysfs backend supports physical slots.

Display physical slot information in lspci -v

We've been exposing slot information in /sys/bus/pci/slots for a
long time now (as long as a hotplug driver or slot detection
driver like pci_slot is loaded).

Let's make life better for our users and display that information
in lspci. If slot entries appear in /sys/bus/pci/slots/,
correlate them to PCI devices, and display the information when
lspci -v is issued.

If no slot entries appear in sysfs (due to no modules loaded), do
nothing.

Now you'll see sample output like the following:

23:01.1 Class 0c04: Device 10df:fd00 (rev 01)
Subsystem: Device 10df:fd00
Physical Slot: 3
Flags: bus master, 66MHz, medium devsel, latency 248, IRQ 60
...

Signed-off-by: Alex Chiang <achiang@hp.com>

lspci: fix "suprise" typo

This patch corrects the spelling of "surprise."

Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com>

Fix ATS capability error in lspci

Fix the wrong position of the ATS enable bit.

Signed-off-by: Yu Zhao <yu.zhao@intel.com>

Ignore NUL character at the end of VPD strings.

Broadcom Ethernet controllers have been observed to use NUL-terminated strings.

Accompany VPD items with human-readable names.

Copyright is held by Solarflare, not Ben personally

(Changed at Ben's request.)

Updated Matthew's address.

Decode VPD fully only with -vv or higher.

This brings the behavior in line with the other capabilities.

Released as -alpha2.

Split lspci to multiple source files.

In verbose mode, display contents of VPD if possible

Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>

Add implementation of read_vpd() method for sysfs

Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>

Add PCI 2.2 VPD register definitions

Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>

Add pci_read_vpd() function and internal read_vpd() method

Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>

Added a test case for the ATS capability.

Released as 3.1.0-alpha1.

Updated library ABI with proper versioning.

setpci: Document the new syntax.

The table of known registers is gone. The man page refers to the --dumpregs
option instead.

setpci: Fixed a silly bug in parsing of options.

setpci: Cleaned up reporting of parse errors.

The full usage is no longer printed with each parse error,
only a suggestion to call `setpci --help' for help.

setpci: Decorate trace() with PCI_PRINTF.

setpci: Even better tracing messages.

setpci: Do not request read-write access when in demo mode.

setpci: Better tracing and reporting of runtime errors.

Use PCI_CAP_xxx internally in setpci.

Renumbered PCI_CAP_xxx to start with 1.

Implemented `setpci --dumpregs'.

Exit correctly after `setpci --version'.

Let setpci support capabilities.

Changed syntax of register addresses. They now consist of a base and an offset,
where the base is either a fixed address or a name/ID of a capability.

Needs lots of polish (see TODO).

Moved the new field at the end of struct pci_dev to keep the ABI.

Added library functions for accessing PCI capabilities.

Added xstrdup() to the common utility functions.

More splitting of the argument parser.

Coding style cleanup: `for(...)' wants a space.

Split parsing of options in setpci to multiple functions.

Added dumping of the Address Translation Services capability.

Let the maintainer script use plain Git instead of Cogito.

Released as 3.0.3.

Updated pci.ids to the current snapshot.

Let `lspci -k' display subsystem names.

Several users have requested a `quick overview' mode that would display
everything needed to identify a device and its drivers. I believe that it
is better to extend the `-k' mode to include subsystems than to introduce
yet another mode.

Fixed building on Cygwin.

It got broken by the introduction of WITH_LIBS.

Thanks to Steve Elliott for reporting that.

Decode HT 1.02 in the same way as HT 1.03.

AMD K8 processors claim that they're using HyperTransport 1.02, but the
publicly available documentation (specifically the BKDG) states that the
HT version is 1.03 in reality.

lspci will issue a warning "!!! Possibly incomplete decoding" and only
decode some basic HT properties for HT versions before 1.03.
Move that boundary down to HT version 1.02.

Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>

Fix printing of PCIE AER capability.

The cap_aer function in lspci.c which prints PCIE AER capabilities
labels the uncorrected error mask incorrectly as "status" and it
also has some typos. This patch fixes that.

Signed-off-by: Max Asbock <masbock@linux.vnet.ibm.com>

Better support for cross-compiling.

It is difficult to cross compile pciutils, as the host is not standard 3 or 4 components and one need to pass all tools cross compile prefixes.

The attached patch makes it easier:

1. Add CROSS_COMPILE prefix for all toolchain tools. I also changed default CC
   to gcc, I can fix this if you like.

2. Allow parsing standard HOST, such as i686-pc-linux-gnu.

3. Make rel be RELEASE, maybe the whole release thing can be removed as it is
   not actually used.

Signed-off-by: Alon Bar-Lev <alon.barlev@gmail.com>

Fix spelling of MSI.

Change the spelling of 'Signalled' to 'Signaled' to match the PCI spec.
Report the capability as 'MSI' instead of either 'Message Signaled
Interrupts' to fit better with MSI-X and bring the width of this line
below 80 columns.

Inspired-by: Randy Dunlap <randy.dunlap@oracle.com>
Signed-off-by: Matthew Wilcox <willy@linux.intel.com>

Updated pci.ids to the today's snapshot from the new web interface.

Forgot to bump the version number.

Fixed a minor bug in lib/configure and released as 3.0.2.

Released as v3.0.1.

Updated pci.ids to the most recent snapshot.

Added a Cygwin port.

Patch by Christopher Voltz, ported to the current tree by me.

Call wget with --no-timestamping.

If --timestamping was set in .wgetrc, wget fails, because it is unable to
handle --timestamping and -O simultaneously (for no apparent reason, but
the maintainers have decided that this bug has graduated to a feature,
see https://savannah.gnu.org/bugs/?22896).

Thanks to Ville Skytta for pointing that out.

Use the visibility attributes only when building a shared library.

Omair Eshkenazi has reported that libpci fails to build with gcc 3.x.
We require gcc 4.x for building shared libraries, but it should not
be needed when SHARED=no.

This patch changes internal.h to define PCI_ABI as the visibility
attribute only if we build a shared library.

However, this means that we need to know the configuration settings
before we include pci.h, so I had to include config.h explicitly
from internal.h and create a new symbol PCI_CONFIG_H to check whether
config.h has been already included.

libresolv is not available on BSD systems, but they have resolver in libc.

Suggested by Hasso Tepper.

Use h_errno instead of _res.h_errno.

The latter is apparently not available on BSD.

Suggested by Hasso Tepper.

Fix printing the MSI capability.

Contributed by Matthew Wilcox. His changelog says:

I'd like to present this change for your consideration.  I'm not sure
where the original idea comes from that MSI has a 'queue' of interrupts.
It has multiple independent interrupts.  Also, rather than just dumping
the raw values from the register, it makes sense to decode them into
the number of interrupts it supports.

Single Root I/O Virtualization capability support

Signed-off-by: Yu Zhao <yu.zhao@intel.com>
lib/header.h |   28 ++++++++++++++++++++++++++++
lspci.c      |   46 ++++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 74 insertions(+)

Alternative Routing-ID Interpretation capability support

Signed-off-by: Yu Zhao <yu.zhao@intel.com>
lib/header.h |   11 +++++++++++
lspci.c      |   22 ++++++++++++++++++++++
2 files changed, 33 insertions(+)

Access Control Services capability support

Signed-off-by: Yu Zhao <yu.zhao@intel.com>
lib/header.h |   20 ++++++++++++++++++++
lspci.c      |   25 ++++++++++++++++++++++++-
2 files changed, 44 insertions(+), 1 deletion(-)

Advanced Error Reporting capability support

Signed-off-by: Yu Zhao <yu.zhao@intel.com>
lib/header.h |    5 ++++-
lspci.c      |   47 ++++++++++++++++++++++++++++++++++++++++++++++-
2 files changed, 50 insertions(+), 2 deletions(-)

PCI Express Device, Link and Slot 2 capabilities support

Signed-off-by: Yu Zhao <yu.zhao@intel.com>
lib/header.h |   22 +++++++
lspci.c      |  166 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-
2 files changed, 186 insertions(+), 2 deletions(-)

Added test cases for new PCIE capabilities.

Contributed by Zhao Yu.

Let filters accept vendor and device 0xffff.

Thanks to Sebastian Herbszt for a bug report.

We have a new linux-pci mailing list.

By mistake, we were setting PCI_USE_DNS on Linux automatically.

Well spotted by Matthias Weidle.