Released as v3.15.0

Update pci.ids to current snapshot

ABI: Forgot to assign versions to pci_filter_has_*

lspci: Show MSI routing

sysfs: Parse MSI routing if available

Bump ABI version of pci_fill_info to 3.15

lspci: Do not show IRQ routing for disabled INTx

Closes #193

sysfs: Fix bug in parsing of classes

Makefile: Fix building of man pages

The old pattern rule was subtly wrong and newer versions
of GNU Make started warning on it.

Makefile: We no longer use ctags

Man: Document what happens when multiple -s or -d options are given

lspci, setpci: Warn when -s or -d is used multiple times

filters: Added function for testing if a filter specifies slot/id

sysfs: Hot-unplug of devices should not produce hard errors

Closes #187.

Add README.DJGPP

Contributed by Pali.

lspci: add test with capabilities 32, 34, and 36

- h0032: Flit logging
- h0034: Flit error injection
- h0036: MMIO register block locator

FreeBSD supports 64b PCI addresses

setpci: Fix discards const from pointer target

Since glibc-2.43, and for ISO C23, the function strchr that return
pointer into their input arrays now have definitions as macros that
return a pointer to a const-qualified type when the input argument is
a pointer to a const-qualified type.

The char *e is only used as a pointer to const char **opts, and only
used for comparisons, so declare *e as const to address warning.

Fixes:
    setpci.c: In function 'parse_options':
    setpci.c:513:19: warning: assignment discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
      513 |             if (e = strchr(opts, *c))
          |                   ^

Signed-off-by: Rudi Heitbaum <rudi@heitbaum.com>

libpci: ecam: Fix discards const from pointer target

endptr is used as the return from strchr(addrs, ',') which is a const
char. endptr is subsequently used as a pointer to the token which is
char. Fix by not reusing endptr, but declare addrsptr as a const char *
pointer for use in the first case addressing the warning.

fixes:
    ecam.c: In function 'parse_next_addrs':
    ecam.c:620:10: warning: assignment discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
      620 |   endptr = strchr(addrs, ',');
          |          ^

Signed-off-by: Rudi Heitbaum <rudi@heitbaum.com>

dump: Support `lspci -F -` for stdin

It can be useful to pipe raw config registers (lspci -x) from one system
to another, so the latter system can do the parsing (lspci -vv -F ...).
For example, one might do this if the former has a more limited / older
version of lspci. Today, one has to write the contents to a file.

Support stdin via "-", so it's easier to run it through a pipeline, such
as:

  ssh ${remote_host} old_lspci -xxx | new_lspci -vv -F -

A dash (-) is a common convention used by many other tools. If one
really expected to access a file named "-", one can use "./-" or similar
to disambiguate.

Based on a patch by Brian Norris <briannorris@chromium.org>.

dump: Refactor file parsing

README: Download by FTP is no longer supported

libpci: win32-cfgmgr32: Fix handling of "\\??\\C:\\..." driver paths

libpci: physmem-windows: Fix check for length param of physmem_unmap()

VirtualQuery() does not have to return whole mapped range, but just region
subset. So call VirtualQuery() multiple times for each region and calculate
the total length of the mapping.

This change fixes unmapping of the BIOS 0xE0000-0x100000 memory for which
Windows creates region of one page and so VirtualQuery() for each region i
at virtual address ptr + i*4096 returns AllocationBase=ptr and RegionSize=4096.

windows: Mark more local win32 helpers functions as static

After movement of win32_call_func_with_tcb_privilege() function from
i386-io-windows.h to win32-helpers.c done in 3.11.0, lot of win32 helpers
functions are not used outside of win32-helpers.c anymore. Mark them as
static and remove from win32-helpers.h header file.

windows: Simplify win32_find_and_open_process_for_query()

Retrieve information about all processes in system via
NtQuerySystemInformation(SystemProcessInformation) call.
This method returns both process id and process name.
It is supported on all Windows NT based systems.

This replaces complicated code which uses different WinAPI functions
for specific Windows versions.

windows: Improve win32_change_token()

Function ImpersonateLoggedOnUser() is not available on older Windows
versions.

Instead of ImpersonateLoggedOnUser(), use SetThreadToken() on the
duplicated token of SecurityImpersonation type, same what is
ImpersonateLoggedOnUser() expected to do.

windows: Improve win32_change_error_mode()

Do not clear existing error bits when setting SEM_FAILCRITICALERRORS bit.
Just append SEM_FAILCRITICALERRORS to existing error bit set.

Use GetThreadErrorMode(), GetThreadErrorMode() or GetErrorMode() function
to retrive existing set error bits.

windows: Improve version detection

Move version detection code into helper functions, which can be reused.

For 32-bit x86 builds use GetVersion() which is available in all Windows
versions instead of GetVersionEx().

windows: Fix wideness of 16-bit constant VWIN32_DEVICE_ID

Update orthography of IDs

Originally, we spelled "ID's" with an apostrophe. This is still correct,
but somewhat niche spelling.

Update margin_hw.c

pcilmr: Fix incorrect device pair detection in margin_find_pair()

When finding the parent bridge of an endpoint device, margin_find_pair()
was reading PCI_SECONDARY_BUS (offset 0x19) from all devices in the
system without checking if they are actually bridge devices.

For non-bridge devices (header type 0), offset 0x19 contains arbitrary
data that could coincidentally match the target device's bus number,
causing the function to return an incorrect device as the downstream port.

This resulted in the misleading error:
  "Looks like you don't have enough privileges to access Device
   Configuration Space."

The error occurred because the incorrectly selected device (e.g., a SATA
controller) doesn't have a PCIe Express capability, causing pci_find_cap()
to correctly return NULL.

Fix by adding a margin_port_is_down(p) check before reading
PCI_SECONDARY_BUS. This ensures we only examine actual PCIe bridge
devices (Root Ports or Switch Downstream Ports) when searching for
the parent bridge.

Example failure case:
  - Target: 0000:60:00.0 (NVMe device, bus 0x60)
  - Actual parent: 0000:5d:02.0 (PCIe switch downstream port)
  - Incorrectly matched: 0000:00:18.0 (SATA controller with 0x60 at offset 0x19)

After fix, correctly identifies 0000:5d:02.0 as the downstream port.

setpci: add reg name for flit logging

Signed-off-by: Alex Martens <alex.martens@asteralabs.com>

Merge remote-tracking branch 'am/lspci-flit-log'

Merge remote-tracking branch 'github/master'

Merge remote-tracking branch 'am/lspci-flit-error'

ls-ecaps: Add XT and TEE-Limited support reporting

PCIe r6.1 added TDISP with TEE Limited bits.
PCIe r7.0 added XT mode for IDE TLPs.

Define new bits and update the test.

Signed-off-by: Alexey Kardashevskiy <aik@amd.com>

Fix indentation

lspci: decode MMIO Register Block Locator

MRBL is defined in the PCIe base specification 6.2.

Signed-off-by: Alex Martens <alex.martens@asteralabs.com>

lspci: decode flit logging

The flit logging extended capability is defined in the PCI express base
specification revision 6.4.

Signed-off-by: Alex Martens <alex.martens@asteralabs.com>

lspci: decode flit error injection

The flit error injection extended capability is defined in the PCI
express base specification revision 6.4.

Signed-off-by: Alex Martens <alex.martens@asteralabs.com>

ls-caps: Add support to decode Gen7 speed

Updated Link capabilities, Link status and Link capabilities 2 registers decode logic to support PCIe Gen7 speed.

Signed-off-by: Prabu Thangamuthu <prabut@synopsys.com>

header.h: Fix type 1 header comment

Should be 0x35-0x37 is reserved.

Signed-off-by: Rong Tao <rongtao@cestc.cn>

Maint: Another typo in maint/release

Maint: Fix typo in maint/release

Released as v3.14.0

Update pci.ids to the current snapshot

libpci: djgpp: Handle Windows NTVDM zero error codes

Windows NTVDM DPMI host clears the AX register for unsupported DPMI calls.
So handle zero error code as ENOSYS instead of default case EACCES.

Fix formatting of 64-bit integers

PCI_U64_FMT_X is needed.

Use snprintf instead of sprintf to prevent buffer overruns

In bitops.h, update the TABLE macro to call snprintf(buf, sizeof(buf),
...) rather than unbounded sprintf, ensuring that out-of-range indices
produce a bounded "??%d" string.

In setpci.c, change the device slot formatting from sprintf(slot, ...) to
snprintf(slot, sizeof(slot), ...), capping output to the 16-byte buffer
and avoiding overflow when printing PCI domain, bus, dev, and func
values.

Signed-off-by: Mingjie Shen <shen497@purdue.edu>

LMR: <string.h> is required for memset()

Tables of strings should have both pointers and the actual strings const

Remove trailing whitespace

Flit Mode and Device 3 Capability

header.h: Classes and capabilities from PCI Code and ID Assignment rev 1.18

Port to RT-Thread Smart DM PCI

Signed-off-by: GuEe-GUI <2991707448@qq.com>

lspci: Add test case for Physical Layer 16 GT/s and 32 GT/s extended capability registers

Update pci.ids to the current snapshot

pcilib.man: Update information about win32-kldbg

libpci: win32-kldbg: Implement registration driver from non-native process

It is common that 32-bit application is running on 64-bit host system, or
nowadays also that 64-bit AMD64 application is running on ARM64 system.

For all these cases the win32-kldbg.c code horrible fails with just generic
error message when trying to register kldbgdrv.sys driver.

Add code which detects machine type of running process, machine type of
kldbgdrv.sys driver and machine type of the host system. If machine type of
driver and system machines then allow to register driver. Otherwise print
debug verbose message why it is not possible to use kldbgdrv.sys driver
from kd.exe/windbg.exe binary. This could allow to debug issues via command
lspci -G why win32-kldbg refused to load driver from windbg.exe binary.

At the same time relax checks in win32_check_driver() to not depend on the
constants related to process type as process architecture is not relevant
here. Important is always only driver and native system architecture,
process may be running under WoW64 (e.g. i386 process or AMD64 system).

libpci: win32-kldbg: Fix calling IOCTL_KLDBG from 32-bit process on 64-bit system

32-bit process on 64-bit system needs to pass all pointers in 64-bit format
for IOCTL_KLDBG call.

So for 32-bit builds (determined by not defined _WIN64 macro) define new
SYSDBG_BUS_DATA64 and KLDBG64 structures with u64 type for pointer members.
Compiler will automatically align all structure members and inserts padding
between members as needed, in the same way as for 64-bit builds. Due to
alignment restrictions, adding just one dummy 32-bit member after each
pointer member does not work.

32-bit code then needs to figure out if the host system is 32-bit or 64-bit
and choose which structure (process native or 64-bit) needs to be passed to
the IOCTL_KLDBG call for successful execution. This is determined by the
win32_is_32bit_on_64bit_system() helper function.

With this change 32-bit i386 lspci.exe binary is working fine on 64-bit
AMD64 system and via win32-kldbg can access PCIe config space if the
AMD64 kldbgdrv.sys driver is registered in the host system.

libpci: win32-cfgmgr32: Improve parsing of driver path

Remove NT prefix "\\??\\" prefix only for drive letters when converting it
to Win32 path. And convert unhandled absolute NT path to Win32 path via
"GLOBALROOT" symlink (which points to NT root path) available in Win32
"\\\\?\\" path. With this change libpci's driver path on windows should be
always valid Win32 path.

libpci: win32-cfgmgr32: Define error messages for all CR_* error constants

windows: Fix declaration of GetSystemFirmwareTable function pointer

Function pointer attributes have to be specified on the left side of the *.

windows: Update comment about RtlNtStatusToDosError() side effect

windows: Check for SizeOfOptionalHeader before dereferencing OptionalHeader

offsetof(IMAGE_OPTIONAL_HEADER, DataDirectory) is the minimal size of
variable length OptionalHeader (IMAGE_OPTIONAL_HEADER) structure.

windows: Improve win32_change_error_mode()

Attempt to do two other thread safe methods before doing fallback to thread
unsafe SetErrorMode() method.

windows: Do not manually load ntdll.dll library

Per Windows Internals, Part 1, Image Loader section, the ntdll.dll library
is always loaded into every process on all NT systems.

So remove code which dynamically loads ntdll.dll library via LoadLibrary()
function and remove also code which changes error reporting mode (used just
for LoadLibrary) as both are not needed.

Also Microsoft C/C++ compilers optimize EXE applications in a way which
expects that the ntdll.dll library is loaded as the first module into the
process before the main EXE module itself.

windows: Do not cast FARPROC to LPVOID and then to some function pointer

FARPROC is function pointer type intptr_t(__stdcall*)() and LPVOID is data
pointer type void*. Casting from function pointer to data pointer and back
is undefined in C, and moreover in all cases it is not needed. In all cases
it is just needed to cast FARPROC function pointer type to some specific
function pointer type, and casting via intermediate LPVOID was there just
to mute compiler warnings about casting between two incompatible function
pointer types. To mute that compiler warning, do casting via intermediate
generic function pointer type void(*)(void) which is preferred according to
gcc documentation and does not throw any compiler warnings neither by gcc,
nor by msvc compilers.

libpci: add some missing PCI_CLASS constants

The names and values are taken from the pci.ids file.

Allow pci.h to be included from C++

Add CXL DVSEC GPF time_scale 10s support

Ref CXL Spec chapter 8.1.6 and 8.1.7, Add time_scale 10s support.

When time_scale is 7h

Old code shows:
GPF Phase 1 Timeout: 20<?>

New code shows
GPF Phase 1 Timeout: 20s

The changes do not affect Reserved time scale encoding shows <?>.

Signed-off-by: yeeli <seven.yi.lee@gmail.com>

lspci: Decode Physical Layer 64 GT/s extended capability register

lspci: Decode Physical Layer 16 GT/s and 32 GT/s extended capability registers

Extend the GNU/Hurd back-end to work on 64-bit systems

lspci: add VirtIO SharedMemory capability support

This patch adds the support for VirtIO share memory capability [1].
A shared memory region is defined in a `struct virtio_pci_cap64`
where the highest 32 bits of `offset` and `size` are appened to the
original `struct virtio_pci_cap`.

With this patch, a VirtIO PMEM device (ID 27) shows like the
following:

```
00:02.0 Class ffff: Device 1af4:105b (rev 01)
        Control: I/O+ Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B- DisINTx+
        Status: Cap+ 66MHz- UDF- FastB2B- ParErr- DEVSEL=fast >TAbort- <TAbort- <MAbort- >SERR- <PERR- INTx-
        Latency: 0
        Region 0: Memory at 100001000 (64-bit, non-prefetchable) [size=4K]
        Region 2: Memory at 101000000 (64-bit, non-prefetchable) [size=16M]
        Capabilities: [40] Vendor Specific Information: VirtIO: CommonCfg
                BAR=0 offset=00000000 size=0000003c
        Capabilities: [50] Vendor Specific Information: VirtIO: ISR
                BAR=0 offset=0000003c size=00000001
        Capabilities: [60] Vendor Specific Information: VirtIO: Notify
                BAR=0 offset=00000040 size=00000002 multiplier=00000002
        Capabilities: [78] MSI-X: Enable+ Count=2 Masked-
                Vector table: BAR=0 offset=00000058
                PBA: BAR=0 offset=00000078
        Capabilities: [88] Vendor Specific Information: VirtIO: DeviceCfg
                BAR=0 offset=00000044 size=00000010
        Capabilities: [98] Vendor Specific Information: VirtIO: SharedMemory
                BAR=2 offset=0000000000000000 size=0000000001000000 id=0
        Kernel driver in use: virtio-pci
```

[1] Sec 4.1.4.7 https://docs.oasis-open.org/virtio/virtio/v1.2/csd01/virtio-v1.2-csd01.html#x1-1240004

Signed-off-by: Changyuan Lyu <changyuan.lv@gmail.com>

ECAM: Fix memory leak

ECAM: Spell "ACPI" in capitals in debug messages

lspci: Parse Intel Vendor specific capabilities

Intel Vendor specific capabilities are present on following devices:
* Intel Host Bridge / DRAM Controller
* Intel Integrated Graphics Controller
* Intel LPC Controller

Intel Host Bridge / DRAM Controller is on BDF address 00:00.0, so following
command should should parse and show Intel Vendor specific capabilities:

  lspci -s 00:00.0 -vv

Currently only Intel Capabilities Version 1 is supported for now. This is
used since second generation of the Intel Core processors (Sandy Bridge).

libpci: Update manpage documentation for devmem.path

libpci: Document physmem API

libpci: Add Windows physmem support for PCIe ECAM access

It requires either access to NT Section \Device\PhysicalMemory (or
compatible) or to have available kernel32.dll VxDCall2 function or
w32skrnl.dll DPMI function.

libpci: Add DJGPP physmem support for PCIe ECAM access

It requires either Device Mapping support on DPMI host or Physical Address
Mapping support together with support for changing DS descriptor limit to
maximal size 4 GB which enables address wrapping and so access to addresses
below the process base address.

Maintainer scripts: Create GitHub releases

Released as v3.13.0

Updated pci.ids

ABI version bump for pci_fill_info()

We have new flags and new fields in struct pci_dev.

ChangeLog was missing a 3.12 release

RCD: Cleanup

Add display function for cxl1.1 device link status information. (#183)

* Add display function for cxl1.1 device link status information.

* Modifications according to comments. Adding a description, Fix odd indent, and remove unnecessary statement.

pcilmr: Fix margining for ports with Lane reversal

Current implementation interacts only with first Negotiated Link Width
lanes even when Maximum Link Width for the port is bigger than that and
Lane reversal is used. Utility in such situation may try to margin lane
which is not used right now and erroneously fail with
'Error during caps reading' message. Fix that behaviour.

Signed-off-by: Nikita Proshkin <n.proshkin@yadro.com>

pcilmr.man: Fix whitespace errors

pcilmr: Update usage and man: new arguments format and grading

Signed-off-by: Nikita Proshkin <n.proshkin@yadro.com>

pcilmr: Apply grading quirk for Ice Lake RC ports

Ice Lake RC ports don't support two side independent timing margining,
however the entire margin across the eye is what is reported by one side
margining. Utility already has quirks for Ice Lake RC, so expand them
based on this grading information.

Signed-off-by: Nikita Proshkin <n.proshkin@yadro.com>

pcilmr: Add option to configure margining dwell time

Signed-off-by: Nikita Proshkin <n.proshkin@yadro.com>

pcilmr: Add new grading option

Original version of the utility used values from the Table 8-11 of the
PCIe Base Spec Rev 5.0 to evaluate lanes. But it seems that these values
relate only to the margining equipment and are not relevant to evaluating
the quality of connections.

The PCIe Base Spec Rev 5.0 sets the minimum values for the eye in the
section 8.4.2. Change default grading values in the utility according to
this section.

The specification uses the values of the full width and height of the eye,
so add these values to the output of the utility.

In addition, manufacturers can provide criteria for their devices that
differ from the standard ones. Usually this information falls under the
NDA, so add an option to the utility that will allow the user to set
necessary criteria for evaluating the quality of lanes.

Implement the following syntax for the -g(rading) option:
-g 1t=15ps,f | -g 6v=20

Use passed per link receiver criteria for the eye width (timing - t) or
height (voltage - v) in the utility results.

Additional flag f is for situations when port doesn't support two side
independent margining. In such cases by default calculate EW or EH as a
double one side result. User can add f flag for -g option to tell the
utility that the result in one direction is actually the measurement of
the full eye (for example, Ice Lake RC ports work in this way) and it does
not need to be multiplied.

Signed-off-by: Nikita Proshkin <n.proshkin@yadro.com>

pcilmr: Move most of pcilmr arguments parsing logic to the separate file

Also change arguments parsing logic: now link parameters (selected lane
numbers, timing or voltage steps, etc) need to be specified after link port
and will affect only this link margining (previously, one option was
applied to all links).

See updated man for syntax and example.

Signed-off-by: Nikita Proshkin <n.proshkin@yadro.com>

pcilmr: Ensure that utility can accept either Downstream or Upstream link port

Previously, the utility expected only the Upstream Port to be input and,
in fact, passing the Downstream Port led to strange and buggy error
messages. Improve arguments parsing logic to accept any side of the link.

It seems that the only use case that will not be available now is margining
the internal links of the switch, but this scenario looks as strange as
possible.

Signed-off-by: Nikita Proshkin <n.proshkin@yadro.com>

sysfs: Include <libgen.h> for basename()

Otherwise, it fails to build with musl libc.

dump: allow 6-digit domains

The SPDK VMD driver assigns domains for the devices behind a VMD by
concatenating bus/device/function of the VMD, each on a separate byte.
For instance, a device behind a VMD with an address of 5d:05.5 would be
assigned domain 5d0505.

Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com>