Merge remote-tracking branch 'pali/win32-readeflags'

Merge remote-tracking branch 'pali/setpci-register-checks'

setpci: Check if standard register on device exist

Some standard registers are available only on device with header type 0,
some only on header type 1, some other only on header type 2 and some on
header type 0 and 1. Add definitions which registers are available on which
header type and add check to access only available registers.

Merge remote-tracking branch 'thesamesam/which-hunt'

Merge remote-tracking branch 'pali/win32-sysdbg'

lspci: Fix Virtual Channel VC# Caps: MaxTimeSlots

lib: Fix PCI_L1PM_SUBSTAT_CAP_L1PM_SUPP

lib: Fix PCI_HT_SEC_CMD_DN

Merge remote-tracking branch 'sthibaul/master'

CXL: Code clean-up

- We now pass capability length and revision to functions
  instead of reading them from config space again and again.

- Centralize fetching of the capability.

- Add checks for overrunning capability length.

- Avoid out-of-bounds reads from the array of register names.

- Sort capability list by ID.

Merge remote-tracking branch 'jphaws/cxl'

Merge remote-tracking branch 'pali/lspci-tree-filter'

lspci: Fix filter support (-s/-d) for subtrees in tree mode (-t)

cxl: Add placeholder for undecoded DVSECs

Signed-off-by: Ben Widawsky <ben.widawsky@intel.com>
Signed-off-by: Jaxon Haws <jaxon.haws@amd.com>

cxl: Add DVSEC Register Locator

Signed-off-by: Ben Widawsky <ben.widawsky@intel.com>
Co-authored-by: Jaxon Haws <jaxon.haws@amd.com>
Signed-off-by: Jaxon Haws <jaxon.haws@amd.com>

cxl: Add support for DVSEC port cap

Signed-off-by: Ben Widawsky <ben.widawsky@intel.com>
Co-authored-by: Jaxon Haws <jaxon.haws@amd.com>
Signed-off-by: Jaxon Haws <jaxon.haws@amd.com>
---
Add Viral Enable (Jonathan)
Add missing tab (Jonathan)
Add Alt Mem base/limit (Jonathan)

cxl: Implement more device DVSEC decoding

Signed-off-by: Ben Widawsky <ben.widawsky@intel.com>
Co-authored-by: Jaxon Haws <jaxon.haws@amd.com>
Signed-off-by: Jaxon Haws <jaxon.haws@amd.com>
---
Fix ranges (Pali): https://github.com/pciutils/pciutils/pull/59#discussion_r806335631

cxl: Rename caps to be device caps

Signed-off-by: Ben Widawsky <ben.widawsky@intel.com>
Co-authored-by: Jaxon Haws <jaxon.haws@amd.com>
Signed-off-by: Jaxon Haws <jaxon.haws@amd.com>

cxl: Rework caps to new function

This will help upcoming caps

Signed-off-by: Ben Widawsky <ben.widawsky@intel.com>
Co-authored-by: Jaxon Haws <jaxon.haws@amd.com>
Signed-off-by: Jaxon Haws <jaxon.haws@amd.com>

cxl: Collect all DVSEC Device fields

Signed-off-by: Ben Widawsky <ben.widawsky@intel.com>
Co-authored-by: Jaxon Haws <jaxon.haws@amd.com>
Signed-off-by: Jaxon Haws <jaxon.haws@amd.com>

cxl: Make id check more explicit

Currently only type 0 DVSEC caps are handled. Moving this check will
allow more robust type handling in the future.

Should be no functional change.

Signed-off-by: Ben Widawsky <ben.widawsky@intel.com>
Signed-off-by: Jaxon Haws <jaxon.haws@amd.com>

cxl: Rename variable to match other code

The current variable is word sized, and so this makes the CXL code match
the rest of the code.

Signed-off-by: Ben Widawsky <ben.widawsky@intel.com>
Signed-off-by: Jaxon Haws <jaxon.haws@amd.com>

hurd: fix filling information

The hurdish methods only implement the region and base information, the
rest should be taken from the generic method.

Maint: Tag releases with my new PGP key

Avoid adding multiple version tags to the same symbol

This is apparently forbidden in most versions of binutils.

libpci: i386-io-windows.h: Do not define __readeflags() for GCC 4.9+

GCC header file <x86intrin.h> defines static inline function __readeflags()
which calls correct __builtin_ia32_readeflags_XX() builtin.

Header file <x86intrin.h> is included by MinGW-w64's <intrin.h> header file
in new versions of MinGW-w64 and <intrin.h> may be included transitionally
by some other header files automatically.

Defining __readeflags() as both macro and static inline function cause
compile errors.

Fix this compile error by not defining __readeflags() macro and instead
include GCC header file <x86intrin.h>.

lib/configure: respect $PKG_CONFIG completely

Signed-off-by: Sam James <sam@gentoo.org>

lib/configure: drop usage of which

'which' is not required by POSIX and is an external command which may not be
available, and 'command -v' does the job just fine.

Debian and Gentoo at least are both making efforts to drop which from
their base system package list.

Signed-off-by: Sam James <sam@gentoo.org>

setpci: Define CAPABILITIES register

pcilib.man: Include information about win32-sysdbg

libpci: Add new windows NT sysdbg implementation

NT SysDbg interface allow access to the PCI config space. Only devices on
the first domain are available and only first 256 bytes of the PCI config
space can be accessed. Compared to intel-conf1 access, this API is race
free as NT kernel serialize access to PCI I/O ports. This NT SysDbg API is
used by the !pci command of 32-bit WinDbg kernel debugger for displaying
PCI config space. Debug privilege is required to use this NT interface.

Released as 3.8.0.

Filters: Fixed symbol versioning

README and ChangeLog for 3.8.0.

Document new filter syntax in man pages

Filters now support partially specified classes and also prog-if's

Rewritten the filter parser, the old code was too twisted to extend.

pciutils.lsm: metalab.unc.edu is gone

Updated pci.ids to today's snapshot

lspci: Fix detection of memory space bar

intel-conf1 backend never show AtomicOpsCap: capabilities despite the fact
that is successfuly detects memory bars on device. But other backends show
this capability.

Error is in device_has_memory_space_bar() function, it expects that ->size
member is always filled. But size of the BAR is not available in PCI config
space and therefore raw backends cannot retrieve it.

Probably intention of the non-zero check was to verify that base address
was filled with non-zero size. So either base address is non-zero or length
is non-zero. Adjust check.

lspci: Fix detection of extended capabilities

Bus may report all-ones when trying to access non-existent extended space.
Same check is also in lib/caps.c extended space parser.

lspci: Fix detection of virtual regions

There are many issues with detection of virtual regions.

1. Variable for detecting virtual region is global and if one BAR is marked
   as virtual then all remaining BARs are treated as virtual too as this
   variable is not reset at next loop iteration.

2. Lower address is read from flg variable which on backends without config
   space is initialized from PCI flags, not from base address.

3. Code mixes at many places PCI flags, resource flags, PCI addresses and
   resource addresses. Some backends reports PCI flags in PCI addresses,
   some not.

Cleanup mess of ->base_addr, ->flags and PCI_BASE_ADDRESS. If backend
provides ->flags value (test via PCI_FILL_IO_FLAGS) then use it instead of
reading flags from pci config space.

Fix reading of PCI hw_lower and hw_upper addresses. If PCI_BASE_ADDRESS
reports different type than what is stored in base_addr then completely
ignore hw_lower and hw_upper values. It means that backend either provide
fiction information or provide resources in different order as they are
stored in hardware. In any case values from HW cannot be used as they do
not match values reported by backend.

And in the last case, make virtual variable local to the current BAR
processing and do not increment loop variable i when 64-bit MEM type is
detected via data from config space. It could miss some virtual resource
reported by the backend.

This change fixes displaying resources of PCI devices which have some
unset/unused BARs in the middle and OS reports virtual regions and
remaining regions without holes. E.g. HW BARs 0, 2, 5 are used and OS
reports base_addr 0, 1, 2, 3.

lspci: Fix handling of truncated lines for msvcrt.dll

msvcrt.dll's vsnprintf() implementation does not fill terminating null byte
when overflow occurs and buffer size is returned.

libpci: i386-io-windows.h: Fix error handling for GetProcessImageFileNameW() and GetModuleFileNameExW()

These functions may require buffer which is larger than MAX_PATH (wide)
characters and their error handling is more complicated. Fix it.

libpci: i386-io-windows.h: Fix MinGW build with pre-4.00 MSVCRT runtime library

Like CRTDLL library, pre-4.00 MSVCRT runtime library does not provide I/O port functions.

libpci: Always call pci_set_name_list_path() in pci_init_name_list_path()

If pci_init_name_list_path() does not call pci_set_name_list_path() then
a->id_file_name variable is NULL and pci_load_name_list() would crash as it
tries to do fopen(NULL, ...).

If libpci was configured at compile time to use current executable path for
locating pci.ids file and it is not possible to determinate current
executable path then call pci_set_name_list_path() with just filename
without path (this would fallback to the current working directory).

libpci: Compile windows i386-ports only for x86 CPU

There are already arm32 and arm64 port of MinGW-w64 toolchain.

fbsd-device: One more fix

Finishes the incomplete fix from 5c649bdcedfd823670dcbd74e9c38849d068db80.

Fix lspci: Power Management Control/Status PCI to PCI Bridge Support Extensions

README: Mention cfgmgr32 for Windows

README: Update information about Windows

Basic support for 64-bit systems is there.

pcilib.man: Include information about win32-cfgmgr32

lspci: Do not access config space when it is emulated

Emulated config space contains only few information so it could look like
some valid config space. libpci compose emulated config space either from
struct pci_dev or put some fake information (when struct pci_dev does not
have them).

To prevent showing to user fake/bogus information about PCI devices, show
only information which are directly stored in struct pci_dev when emulated
config space is used.

Do it via setting lspci's header type to invalid value (byte)-1, so lspci
code will handle device as unknown without trying to interpret values
config space. This header type is set only in lspci, not in libpci, so
other libpci applications would see valid config space.

lspci users are probably not interested in fake information provided by
libpci just for purpose to export syntactically valid config space.
Information stored in struct pci_dev are the correct one (or rather what OS
things that is correct).

libpci: Add windows cfgmgr32 implementation

Access via cfgmgr32.dll library allows to list PCI devices and retrieve
their basic properties and system resource configuration. Access is
available to all users and should not require special privileges, access
tokens, rights or permissions.

This cfgmgr32.dll library does not provide access to PCI config space.

libpci: Implement virtual config space for provides without access to config space

Add a new pci_dev member no_config_access which signals that there is no
access to config space for particular device. Reading operation in this
case should return data from emulated virtual config space. For provides
there is a new helper function pci_generic_read() which emulates config
spaces based on struct pci_dev members.

lspci: Show information also for devices with unknown header type

lspci sees header type with 0x7f value in case config space is not
accessible. It may be because of permission issues or missing backend
(e.g. on Windows) or because device itself does not respond to config
cycles and config space is inaccessible.

Inaccessible config space is common scenario for switchable PCIe GPU
cards. Some Nvidia and also some AMD cards when sleep then lspci sees
all-ones in their config space. But kernel has registered those cards and
some of their properties are cached ans available via sysfs or procfs (from
time when card was awake).

Currently lspci shows error message "Unknown header type 7f" and does not
parse any value neither from config space nor from sysfs backend.

So try to show at least information which kernel provided into libpci
struct pci_dev.

Set unknown values to zeros (instead of all-ones which are returned from
config space) and add a new function show_htype_unknown() for showing
additional information in case header type is unknown.

This change will also help new windows backend which has only emulated
config space and therefore valid data only in struct pci_dev.

libpci: Define some more PCI_IORESOURCE flags (compatible with ioport.h)

These flags define mapping between PCI config space and system resources.
So non-sysfs/procfs providers can fill these flags too.

Tried to fix the fbsd-device back-end

It was left broken by the fill_info reform.

pciutils: Add decode support for Data Object Exchange Extended Capability

PCI Data Object Exchange [1] provides a mailbox interface used as the
transport for various protocols defined by PCI-SIG and others. Make the
limited information in config space available. Note the Read/Write
Mailbox registers themselves are not currently parsed as the usefulness
of accessing one dword of a protocol is probably limited.

In future, operating systems may provide means to safely query the
supported protocols, but those have not yet been defined.

Example output:

Capabilities: [100 v1] Data Object Exchange
DOECap: IntSup+
Interrupt Message Number 001
DOECtl: IntEn+
DOESta: Busy- IntSta- Error- ObjectReady+

[1] PCIe r6.0, sections 6.30 and 7.9.24

Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

lspci: Decode PCIe 6.0 Slot Power Limit values

When the Slot Power Limit Scale field equals 00b (1.0x) and Slot
Power Limit Value exceeds EFh, the following alternative encodings
are used:

  F0h   > 239 W and ≤ 250 W Slot Power Limit
  F1h   > 250 W and ≤ 275 W Slot Power Limit
  F2h   > 275 W and ≤ 300 W Slot Power Limit
  F3h   > 300 W and ≤ 325 W Slot Power Limit
  F4h   > 325 W and ≤ 350 W Slot Power Limit
  F5h   > 350 W and ≤ 375 W Slot Power Limit
  F6h   > 375 W and ≤ 400 W Slot Power Limit
  F7h   > 400 W and ≤ 425 W Slot Power Limit
  F8h   > 425 W and ≤ 450 W Slot Power Limit
  F9h   > 450 W and ≤ 475 W Slot Power Limit
  FAh   > 475 W and ≤ 500 W Slot Power Limit
  FBh   > 500 W and ≤ 525 W Slot Power Limit
  FCh   > 525 W and ≤ 550 W Slot Power Limit
  FDh   > 550 W and ≤ 575 W Slot Power Limit
  FEh   > 575 W and ≤ 600 W Slot Power Limit
  FFh   Reserved for Slot Power Limit Values above 600 W

Previously only values F0h, F1h and F2h were covered.

Merge remote-tracking branch 'github/master'

lspci: Replace find_driver() via libpci PCI_FILL_DRIVER

And implement this show_kernel() and show_kernel_machine() for all
platforms.

libpci: sysfs: Implement support for PCI_FILL_DRIVER

In sysfs is driver name stored as symlink path of "driver" node.

libpci: proc: Implement support for PCI_FILL_DRIVER

File /proc/bus/pci/devices contains optional driver name in the last 18th field.

libpci: Define new string property PCI_FILL_DRIVER

This change extends libpci library and allows providers to fill
PCI_FILL_DRIVER via native system APIs. As it is string property there is
no need to increase ABI version.

Intended usage in application is just:

  const char *driver = pci_get_string_property(d->dev, PCI_FILL_DRIVER);

lspci: Do not show -[00]- bus in tree output

Secondary or subordinate bus cannot be zero. Zero value could indicate
either invalid secondary bus value or the fact that secondary bus value was
not filled or indicates non-compliant PCI-to-PCI bridge. This change makes
tree output better readable when bus numbers are not known or not provided.

lspci: Build tree based on PCI_FILL_PARENT information

Topology reported by system (libpci provider) may be different from
topology built based on primary/secondary/subordinate numbers from PCI
bridges by lspci.

This happens for example when some non-compliant PCI-to-PCI bridge
with Type 0 header (e.g. Marvell one) is available in the system.

So add additional edges reported by libpci when building tree in lspci.

libpci: sysfs: Implement support for PCI_FILL_PARENT

libpci: Add new option PCI_FILL_PARENT

This change extends libpci and allows providers to fill parent pci_dev.
This is useful to retrieve topology as it is reported by the system itself.

lspci: Retrieve prog if, subsystem ids and revision id via libpci

Use pci_fill_info with CLASS_EXT and SUBSYS to fill this information.

lspci in some places reads class from what libpci provider fills in
dev->device_class and in some other places it reads directly from config
space. In dev->device_class is stored class possible different class as in
config space (e.g. if kernel is fixing class because device has bogus
information stored in config space).

With this change is class always read from dev->device_class which reflects
and respects lspci -b option (Bus-centric view). Same applies for subsystem
ids and revision id (note that prog if is part of class).

libpci: sysfs: Implement CLASS_EXT and SUBSYS support

In sysfs there are optional nodes with this information.

libpci: generic: Implement SUBSYS also for PCI_HEADER_TYPE_BRIDGE

Subsystem ids for PCI Bridges are stored in extended capability
PCI_CAP_ID_SSVID.

libpci: generic: Implement CLASS_EXT and SUBSYS support

PCI_FILL_SUBSYS is implemented only for PCI_HEADER_TYPE_NORMAL and
PCI_HEADER_TYPE_CARDBUS like in lspci.

libpci: Add new options for pci_fill_info: CLASS_EXT and SUBSYS

This change extends libpci library and allows providers to fill these
informations (Programming interface, Revision id and Subsystem ids) via
native system APIs, which sometimes may differs from what is stored in PCI
config space.

Programming interface is part of 24-bit Device Class number but apparently
libpci exports only high 16-bit of this number via device_class member.

lspci: Use PCI_FILL_BRIDGE_BASES to detect if range behind bridge is disabled or unsupported

Show resources behind bridge as reported by PCI_FILL_BRIDGE_BASES.

I/O or Prefetchable memory behind bridge is unsupported by bridge if both
base and limit bridge registers are read-only and returns zero. So if base
and limit registers returns zero (which is valid enabled range) and kernel
reports that particular resource is disabled it means that resource is
unsupported. Both I/O or Prefetchable memory resources are only optional.

libpci: Add support for filling bridge resources

Extend libpci API and ABI to fill bridge resources from sysfs.

lspci: Simplify printing range in show_range()

Use just one printf() call with width format argument based on number of bits.

lspci: Show 16/32/64 bit width for address ranges behind bridge

Type of address range is encoded in lower bits.

Merge branch 'fill-info'

libpci: i386-io-windows.h: Fix comment about CRTDLL

There is no 64-bit version of CRTDLL library. MinGW-w64 provided bogus
64-bit import library for non-existent runtime DLL library, which was
recently deleted.

lspci: Define PCI_U64_FMT_U format for printing u64

Windows CRTDLL and MSVCRT runtime system libraries do not support %llu
format string in printf. They support only %I64u format string. Fix this
problem by providing PCI_U64_FMT_U macro in the same way as existing
PCI_U64_FMT_X macro (for %llx).

For C99 systems this PCI_U64_FMT_U macro is defined to C99 PRIu64 constant.

This change fixes printing unsigned decimal 64-bit numbers by lspci on
Window systems independently of used compiler (MinGW or MSVC).

lspci: Replace unsigned long long type by u64 and %llx format by PCI_U64_FMT_X

pciutils already provides and uses u64 type together with PCI_U64_FMT_X
format macro for printing hex value of this type. So use u64 and
PCI_U64_FMT_X also on other few remaining places.

This change fixes printing hexadecimal 64-bit numbers by lspci on Window
systems independently of used compiler (MinGW or MSVC).

pciutils: Remove #include <unistd.h> from files which do not need it

In more files is <unistd.h> header file included without any usage.

MSVC does not provide this header file so remove this useless usage of
<unistd.h> to allow compilation under MSVC.

libpci: For PCI_OS_WINDOWS include windows.h instead of windef.h

According to Win32 API guidelines applications should include <windows.h>
instead of <windef.h>. This change fixes compilation under MSVC as MSVC
<windef.h> header file expects that some other Win32 header files from
<window.h> are already included.

libpci: For PCI_OS_WINDOWS define strncasecmp as alias for _strnicmp

CRTDLL, MSVCRT and UCRT runtimes provides strncasecmp()-like functionality
in _strnicmp() function. As opposite of strcasecmp() for which there are
_stricmp() and _strcmpi() variants, for strncasecmp() there is only
_strnicmp() function.

Without this change linking final setpci.exe executable undef MSVC fails.

libpci: For MSVC < 19.00 define snprintf outside of endian section

snprintf() macro is not endian specific and therefore should be declared
outside of the endian section.

This also fixes snprintf() function for new MinGW-w64 toolchains where
snprintf() is defined as wrapper around _snprintf() which do not return
negative value on overflow. libpci would call MinGW-w64 patched snprintf()
function and not broken system function _snprintf().

libpci: For MSVC < 19.00 define vsnprintf as alias for _vsnprintf

MSVC prior version 19.00 do not have vsnprintf() function, only
_vsnprintf().

Install lspci to /usr/bin on Linux systems

On Linux, lspci is useful even for ordinary users, although only
a subset of features is available.

README: Added a note on dynamic linking and derived works

Simplified pci_fill_info() and friends

Previously, we kept track of which fields were already filled, which
was quite brittle.

Now we keep only the set of already known fields in struct pci_dev.
We check if the current field is needed against this information.

Not only this simplifies the whole thing, but it also enables future
back-ends to call pci_fill_info() recursively as needed.

Another try at choosing the default compiler

Cross-compilers often provide only "gcc" and not "cc".

So let's try "cc" when building natively and "gcc" when cross-compiling.

lspci: Improvements to PCIe link speed downgrade reporting

Do not report PCIe link downgrades for downstream ports.

Changed wording so that "overdriven" is reported instead of
"strange" for speeds greater than the maximum supported one.

Also report nothing instead of "ok".

Inspired by patches by Bjorn Helgaas and Matthew Wilcox.

pci.h: The error callback is now declared with PCI_NONRET

Users of the repeatedly complain that the library crashes, which is
usually caused by providing an error hook which returns to the library.
Let's try warning them more explicitly.

types.h: Introduced PCI_NONRET

lspci: Update tests files with VF 10-Bit Tag Requester

Update the tests files with the new field 10BitTagReq
in SR-IOV Capabilities Register.

Signed-off-by: Dongdong Liu <liudongdong3@huawei.com>

lspci: Decode VF 10-Bit Tag Requester

Decode VF 10-Bit Tag Requester Supported and Enable bit
in SR-IOV Capabilities Register.

Sample output:
  IOVCap: Migration- 10BitTagReq- Interrupt Message Number: 000
  IOVCtl: Enable+ Migration- Interrupt- MSE+ ARIHierarchy- 10BitTagReq-

Signed-off-by: Dongdong Liu <liudongdong3@huawei.com>

lspci: Bus mapping works in non-zero domain

Makefile: don't hardcode gcc

I believe "cc" is a much better default nowadays.
Another approach would be to (also) use "?=" for these variables.

libpci: i386-io-windows.h: Mute false-positive warning

  i386-ports.c: In function ‘conf12_setup_io’:
  i386-io-windows.h:1021: warning: ‘old_token’ may be used uninitialized in this function
  i386-io-windows.h:1021: note: ‘old_token’ was declared here

It is always properly initialized when accessed, just gcc compiler does not see it.

i386-ports: Do not mark pci_access in conf12_cleanup() as unused

It is used as argument for intel_cleanup_io() function.

lib: Fix definition of strcasecmp() for PCI_OS_WINDOWS

UCRT, MSVCRT and CRTDLL runtime libraries provides only _strcmpi()
function and not strcmpi().

MinGW32 has static libraries libcoldname.a and libmoldname.a which provides
strcmpi() function (as link-time redirect to _strcmpi()). libcoldname.a is
automatically linked when compiling for CRTDLL runtime and libmoldname.a
for MSVCRT runtime.

MinGW-w64 has only libmoldname.a library with strcmpi() function and it is
linked to final executable only when compiling for MSVCRT runtime.
when linking with MSVCRT.

To prevent dependency on particular linking configuration and MinGW
toolchain, use set strcasecmp() as alias to _strcmpi() function which is
provided by any runtime library.