]> mj.ucw.cz Git - pciutils.git/blobdiff - lspci.c
lspci: Remove unnecessary !verbose check in show_range()
[pciutils.git] / lspci.c
diff --git a/lspci.c b/lspci.c
index f59ce9121e601d2a878e148c44edf1beb0a162aa..7418b072c0c23783e5763a4b8091b9cc24d6f793 100644 (file)
--- a/lspci.c
+++ b/lspci.c
@@ -1,9 +1,7 @@
 /*
- *     $Id: lspci.c,v 1.10 1998/03/31 21:02:16 mj Exp $
+ *     The PCI Utilities -- List All PCI Devices
  *
- *     Linux PCI Utilities -- List All PCI Devices
- *
- *     Copyright (c) 1997, 1998 Martin Mares <mj@atrey.karlin.mff.cuni.cz>
+ *     Copyright (c) 1997--2018 Martin Mares <mj@ucw.cz>
  *
  *     Can be freely distributed and used under the terms of the GNU GPL.
  */
 #include <stdio.h>
 #include <string.h>
 #include <stdlib.h>
-#include <fcntl.h>
-#include <unistd.h>
+#include <stdarg.h>
 
-#include "pciutils.h"
+#include "lspci.h"
 
 /* Options */
 
-static int verbose;                    /* Show detailed information */
-static int buscentric_view;            /* Show bus addresses/IRQ's instead of CPU-visible ones */
-static int show_hex;                   /* Show contents of config space as hexadecimal numbers */
-static struct pci_filter filter;       /* Device filter */
-static int show_tree;                  /* Show bus tree */
-static int machine_readable;           /* Generate machine-readable output */
-static char *pci_dir = PROC_BUS_PCI;
-
-static char options[] = "nvbxs:d:ti:p:m";
-
-static char help_msg[] = "\
-Usage: lspci [<switches>]\n\
-\n\
--v\t\tBe verbose\n\
--n\t\tShow numeric ID's\n\
--b\t\tBus-centric view (PCI addresses and IRQ's instead of those seen by the CPU)\n\
--x\t\tShow hex-dump of config space (-xx shows full 256 bytes)\n\
--s [[<bus>]:][<slot>][.[<func>]]\tShow only devices in selected slots\n\
--d [<vendor>]:[<device>]\tShow only selected devices\n\
--t\t\tShow bus tree\n\
--m\t\tProduce machine-readable output\n\
--i <file>\tUse specified ID database instead of " ETC_PCI_IDS "\n\
--p <dir>\tUse specified bus directory instead of " PROC_BUS_PCI "\n\
-";
-
-/* Format strings used for IRQ numbers */
-
-#ifdef ARCH_SPARC64
-#define IRQ_FORMAT "%08x"
-#else
-#define IRQ_FORMAT "%d"
+int verbose;                           /* Show detailed information */
+static int opt_hex;                    /* Show contents of config space as hexadecimal numbers */
+struct pci_filter filter;              /* Device filter */
+static int opt_filter;                 /* Any filter was given */
+static int opt_tree;                   /* Show bus tree */
+static int opt_path;                   /* Show bridge path */
+static int opt_machine;                        /* Generate machine-readable output */
+static int opt_map_mode;               /* Bus mapping mode enabled */
+static int opt_domains;                        /* Show domain numbers (0=disabled, 1=auto-detected, 2=requested) */
+static int opt_kernel;                 /* Show kernel drivers */
+static int opt_query_dns;              /* Query the DNS (0=disabled, 1=enabled, 2=refresh cache) */
+static int opt_query_all;              /* Query the DNS for all entries */
+char *opt_pcimap;                      /* Override path to Linux modules.pcimap */
+
+const char program_name[] = "lspci";
+
+static char options[] = "nvbxs:d:tPi:mgp:qkMDQ" GENERIC_OPTIONS ;
+
+static char help_msg[] =
+"Usage: lspci [<switches>]\n"
+"\n"
+"Basic display modes:\n"
+"-mm\t\tProduce machine-readable output (single -m for an obsolete format)\n"
+"-t\t\tShow bus tree\n"
+"\n"
+"Display options:\n"
+"-v\t\tBe verbose (-vv or -vvv for higher verbosity)\n"
+#ifdef PCI_OS_LINUX
+"-k\t\tShow kernel drivers handling each device\n"
 #endif
+"-x\t\tShow hex-dump of the standard part of the config space\n"
+"-xxx\t\tShow hex-dump of the whole config space (dangerous; root only)\n"
+"-xxxx\t\tShow hex-dump of the 4096-byte extended config space (root only)\n"
+"-b\t\tBus-centric view (addresses and IRQ's as seen by the bus)\n"
+"-D\t\tAlways show domain numbers\n"
+"-P\t\tDisplay bridge path in addition to bus and device number\n"
+"-PP\t\tDisplay bus path in addition to bus and device number\n"
+"\n"
+"Resolving of device ID's to names:\n"
+"-n\t\tShow numeric ID's\n"
+"-nn\t\tShow both textual and numeric ID's (names & numbers)\n"
+#ifdef PCI_USE_DNS
+"-q\t\tQuery the PCI ID database for unknown ID's via DNS\n"
+"-qq\t\tAs above, but re-query locally cached entries\n"
+"-Q\t\tQuery the PCI ID database for all ID's via DNS\n"
+#endif
+"\n"
+"Selection of devices:\n"
+"-s [[[[<domain>]:]<bus>]:][<slot>][.[<func>]]\tShow only devices in selected slots\n"
+"-d [<vendor>]:[<device>][:<class>]\t\tShow only devices with specified ID's\n"
+"\n"
+"Other options:\n"
+"-i <file>\tUse specified ID database instead of %s\n"
+#ifdef PCI_OS_LINUX
+"-p <file>\tLook up kernel modules in a given file instead of default modules.pcimap\n"
+#endif
+"-M\t\tEnable `bus mapping' mode (dangerous; root only)\n"
+"\n"
+"PCI access options:\n"
+GENERIC_HELP
+;
 
-/* Our view of the PCI bus */
-
-struct device {
-  struct device *next;
-  byte bus, devfn;
-  word vendid, devid;
-  unsigned int kernel_irq;
-  unsigned long kernel_base_addr[6];
-  byte config[256];
-};
-
-static struct device *first_dev, **last_dev = &first_dev;
+/*** Our view of the PCI bus ***/
 
-/* Miscellaneous routines */
+struct pci_access *pacc;
+struct device *first_dev;
+static int seen_errors;
+static int need_topology;
 
-void *
-xmalloc(unsigned int howmuch)
+int
+config_fetch(struct device *d, unsigned int pos, unsigned int len)
 {
-  void *p = malloc(howmuch);
-  if (!p)
+  unsigned int end = pos+len;
+  int result;
+
+  while (pos < d->config_bufsize && len && d->present[pos])
+    pos++, len--;
+  while (pos+len <= d->config_bufsize && len && d->present[pos+len-1])
+    len--;
+  if (!len)
+    return 1;
+
+  if (end > d->config_bufsize)
     {
-      fprintf(stderr, "lspci: Unable to allocate %d bytes of memory\n", howmuch);
-      exit(1);
+      int orig_size = d->config_bufsize;
+      while (end > d->config_bufsize)
+       d->config_bufsize *= 2;
+      d->config = xrealloc(d->config, d->config_bufsize);
+      d->present = xrealloc(d->present, d->config_bufsize);
+      memset(d->present + orig_size, 0, d->config_bufsize - orig_size);
     }
-  return p;
+  result = pci_read_block(d->dev, pos, d->config + pos, len);
+  if (result)
+    memset(d->present + pos, 1, len);
+  return result;
 }
 
-/* Interface for /proc/bus/pci */
-
-static void
-scan_dev_list(void)
+struct device *
+scan_device(struct pci_dev *p)
 {
-  FILE *f;
-  byte line[256];
-  byte name[256];
-
-  sprintf(name, "%s/devices", pci_dir);
-  if (! (f = fopen(name, "r")))
-    {
-      perror(name);
-      exit(1);
-    }
-  while (fgets(line, sizeof(line), f))
-    {
-      struct device *d = xmalloc(sizeof(struct device));
-      unsigned int dfn, vend;
-
-      sscanf(line, "%x %x %x %lx %lx %lx %lx %lx %lx",
-            &dfn,
-            &vend,
-            &d->kernel_irq,
-            &d->kernel_base_addr[0],
-            &d->kernel_base_addr[1],
-            &d->kernel_base_addr[2],
-            &d->kernel_base_addr[3],
-            &d->kernel_base_addr[4],
-            &d->kernel_base_addr[5]);
-      d->bus = dfn >> 8U;
-      d->devfn = dfn & 0xff;
-      d->vendid = vend >> 16U;
-      d->devid = vend & 0xffff;
-      if (filter_match(&filter, d->bus, d->devfn, d->vendid, d->devid))
-       {
-         *last_dev = d;
-         last_dev = &d->next;
-         d->next = NULL;
-       }
-    }
-  fclose(f);
-}
+  struct device *d;
 
-static inline void
-make_proc_pci_name(struct device *d, char *p)
-{
-  sprintf(p, "%s/%02x/%02x.%x",
-         pci_dir, d->bus, PCI_SLOT(d->devfn), PCI_FUNC(d->devfn));
+  if (p->domain && !opt_domains)
+    opt_domains = 1;
+  if (!pci_filter_match(&filter, p) && !need_topology)
+    return NULL;
+  d = xmalloc(sizeof(struct device));
+  memset(d, 0, sizeof(*d));
+  d->dev = p;
+  d->config_cached = d->config_bufsize = 64;
+  d->config = xmalloc(64);
+  d->present = xmalloc(64);
+  memset(d->present, 1, 64);
+  if (!pci_read_block(p, 0, d->config, 64))
+    {
+      fprintf(stderr, "lspci: Unable to read the standard configuration space header of device %04x:%02x:%02x.%d\n",
+             p->domain, p->bus, p->dev, p->func);
+      seen_errors++;
+      return NULL;
+    }
+  if ((d->config[PCI_HEADER_TYPE] & 0x7f) == PCI_HEADER_TYPE_CARDBUS)
+    {
+      /* For cardbus bridges, we need to fetch 64 bytes more to get the
+       * full standard header... */
+      if (config_fetch(d, 64, 64))
+       d->config_cached += 64;
+    }
+  pci_setup_cache(p, d->config, d->config_cached);
+  pci_fill_info(p, PCI_FILL_IDENT | PCI_FILL_CLASS);
+  return d;
 }
 
 static void
-scan_config(void)
+scan_devices(void)
 {
   struct device *d;
-  char name[64];
-  int fd, res;
-  int how_much = (show_hex > 1) ? 256 : 64;
+  struct pci_dev *p;
 
-  for(d=first_dev; d; d=d->next)
-    {
-      make_proc_pci_name(d, name);
-      if ((fd = open(name, O_RDONLY)) < 0)
-       {
-         fprintf(stderr, "lspci: Unable to open %s: %m\n", name);
-         exit(1);
-       }
-      res = read(fd, d->config, how_much);
-      if (res < 0)
-       {
-         fprintf(stderr, "lspci: Error reading %s: %m\n", name);
-         exit(1);
-       }
-      if (res != how_much)
-       {
-         fprintf(stderr, "lspci: Only %d bytes of config space available to you\n", res);
-         exit(1);
-       }
-      close(fd);
-    }
+  pci_scan_bus(pacc);
+  for (p=pacc->devices; p; p=p->next)
+    if (d = scan_device(p))
+      {
+       d->next = first_dev;
+       first_dev = d;
+      }
 }
 
+/*** Config space accesses ***/
+
 static void
-scan_proc(void)
+check_conf_range(struct device *d, unsigned int pos, unsigned int len)
 {
-  scan_dev_list();
-  scan_config();
+  while (len)
+    if (!d->present[pos])
+      die("Internal bug: Accessing non-read configuration byte at position %x", pos);
+    else
+      pos++, len--;
 }
 
-/* Config space accesses */
-
-static inline byte
+byte
 get_conf_byte(struct device *d, unsigned int pos)
 {
+  check_conf_range(d, pos, 1);
   return d->config[pos];
 }
 
-static word
+word
 get_conf_word(struct device *d, unsigned int pos)
 {
+  check_conf_range(d, pos, 2);
   return d->config[pos] | (d->config[pos+1] << 8);
 }
 
-static u32
+u32
 get_conf_long(struct device *d, unsigned int pos)
 {
+  check_conf_range(d, pos, 4);
   return d->config[pos] |
     (d->config[pos+1] << 8) |
     (d->config[pos+2] << 16) |
     (d->config[pos+3] << 24);
 }
 
-/* Sorting */
+/*** Sorting ***/
 
 static int
 compare_them(const void *A, const void *B)
 {
-  const struct device *a = *(const struct device **)A;
-  const struct device *b = *(const struct device **)B;
+  const struct pci_dev *a = (*(const struct device **)A)->dev;
+  const struct pci_dev *b = (*(const struct device **)B)->dev;
 
+  if (a->domain < b->domain)
+    return -1;
+  if (a->domain > b->domain)
+    return 1;
   if (a->bus < b->bus)
     return -1;
   if (a->bus > b->bus)
     return 1;
-  if (a->devfn < b->devfn)
+  if (a->dev < b->dev)
+    return -1;
+  if (a->dev > b->dev)
+    return 1;
+  if (a->func < b->func)
     return -1;
-  if (a->devfn > b->devfn)
+  if (a->func > b->func)
     return 1;
   return 0;
 }
@@ -212,15 +228,15 @@ compare_them(const void *A, const void *B)
 static void
 sort_them(void)
 {
-  struct device **index, **h;
+  struct device **index, **h, **last_dev;
   int cnt;
   struct device *d;
 
   cnt = 0;
-  for(d=first_dev; d; d=d->next)
+  for (d=first_dev; d; d=d->next)
     cnt++;
   h = index = alloca(sizeof(struct device *) * cnt);
-  for(d=first_dev; d; d=d->next)
+  for (d=first_dev; d; d=d->next)
     *h++ = d;
   qsort(index, cnt, sizeof(struct device *), compare_them);
   last_dev = &first_dev;
@@ -234,95 +250,277 @@ sort_them(void)
   *last_dev = NULL;
 }
 
-/* Normal output */
+/*** Normal output ***/
+
+static void
+show_slot_path(struct device *d)
+{
+  struct pci_dev *p = d->dev;
+
+  if (opt_path)
+    {
+      struct bus *bus = d->parent_bus;
+      struct bridge *br = bus->parent_bridge;
+
+      if (br && br->br_dev)
+       {
+         show_slot_path(br->br_dev);
+         if (opt_path > 1)
+           printf("/%02x:%02x.%d", p->bus, p->dev, p->func);
+         else
+           printf("/%02x.%d", p->dev, p->func);
+         return;
+       }
+    }
+  printf("%02x:%02x.%d", p->bus, p->dev, p->func);
+}
+
+static void
+show_slot_name(struct device *d)
+{
+  struct pci_dev *p = d->dev;
+
+  if (!opt_machine ? opt_domains : (p->domain || opt_domains >= 2))
+    printf("%04x:", p->domain);
+  show_slot_path(d);
+}
+
+void
+get_subid(struct device *d, word *subvp, word *subdp)
+{
+  byte htype = get_conf_byte(d, PCI_HEADER_TYPE) & 0x7f;
+
+  if (htype == PCI_HEADER_TYPE_NORMAL)
+    {
+      *subvp = get_conf_word(d, PCI_SUBSYSTEM_VENDOR_ID);
+      *subdp = get_conf_word(d, PCI_SUBSYSTEM_ID);
+    }
+  else if (htype == PCI_HEADER_TYPE_CARDBUS && d->config_cached >= 128)
+    {
+      *subvp = get_conf_word(d, PCI_CB_SUBSYSTEM_VENDOR_ID);
+      *subdp = get_conf_word(d, PCI_CB_SUBSYSTEM_ID);
+    }
+  else
+    *subvp = *subdp = 0xffff;
+}
 
 static void
 show_terse(struct device *d)
 {
   int c;
-
-  printf("%02x:%02x.%x %s: %s",
-        d->bus,
-        PCI_SLOT(d->devfn),
-        PCI_FUNC(d->devfn),
-        lookup_class(get_conf_word(d, PCI_CLASS_DEVICE)),
-        lookup_device_full(d->vendid, d->devid));
+  struct pci_dev *p = d->dev;
+  char classbuf[128], devbuf[128];
+
+  show_slot_name(d);
+  printf(" %s: %s",
+        pci_lookup_name(pacc, classbuf, sizeof(classbuf),
+                        PCI_LOOKUP_CLASS,
+                        p->device_class),
+        pci_lookup_name(pacc, devbuf, sizeof(devbuf),
+                        PCI_LOOKUP_VENDOR | PCI_LOOKUP_DEVICE,
+                        p->vendor_id, p->device_id));
   if (c = get_conf_byte(d, PCI_REVISION_ID))
     printf(" (rev %02x)", c);
-  if (verbose && (c = get_conf_byte(d, PCI_CLASS_PROG)))
-    printf(" (prog-if %02x)", c);
+  if (verbose)
+    {
+      char *x;
+      c = get_conf_byte(d, PCI_CLASS_PROG);
+      x = pci_lookup_name(pacc, devbuf, sizeof(devbuf),
+                         PCI_LOOKUP_PROGIF | PCI_LOOKUP_NO_NUMBERS,
+                         p->device_class, c);
+      if (c || x)
+       {
+         printf(" (prog-if %02x", c);
+         if (x)
+           printf(" [%s]", x);
+         putchar(')');
+       }
+    }
+  putchar('\n');
+
+  if (verbose || opt_kernel)
+    {
+      word subsys_v, subsys_d;
+      char ssnamebuf[256];
+
+      pci_fill_info(p, PCI_FILL_LABEL);
+
+      if (p->label)
+        printf("\tDeviceName: %s", p->label);
+      get_subid(d, &subsys_v, &subsys_d);
+      if (subsys_v && subsys_v != 0xffff)
+       printf("\tSubsystem: %s\n",
+               pci_lookup_name(pacc, ssnamebuf, sizeof(ssnamebuf),
+                       PCI_LOOKUP_SUBSYSTEM | PCI_LOOKUP_VENDOR | PCI_LOOKUP_DEVICE,
+                       p->vendor_id, p->device_id, subsys_v, subsys_d));
+    }
+}
+
+/*** Verbose output ***/
+
+static void
+show_size(u64 x)
+{
+  static const char suffix[][2] = { "", "K", "M", "G", "T" };
+  unsigned i;
+  if (!x)
+    return;
+  for (i = 0; i < (sizeof(suffix) / sizeof(*suffix) - 1); i++) {
+    if (x % 1024)
+      break;
+    x /= 1024;
+  }
+  printf(" [size=%u%s]", (unsigned)x, suffix[i]);
+}
+
+static void
+show_range(char *prefix, u64 base, u64 limit, int is_64bit)
+{
+  if (base > limit && verbose < 3)
+    {
+      printf("%s: None\n", prefix);
+      return;
+    }
+  printf("%s: ", prefix);
+  if (is_64bit)
+    printf("%016" PCI_U64_FMT_X "-%016" PCI_U64_FMT_X, base, limit);
+  else
+    printf("%08x-%08x", (unsigned) base, (unsigned) limit);
+  if (base <= limit)
+    show_size(limit - base + 1);
+  else
+    printf(" [empty]");
   putchar('\n');
 }
 
 static void
 show_bases(struct device *d, int cnt)
 {
+  struct pci_dev *p = d->dev;
   word cmd = get_conf_word(d, PCI_COMMAND);
   int i;
+  int virtual = 0;
 
-  for(i=0; i<6; i++)
+  for (i=0; i<cnt; i++)
     {
-      unsigned long pos;
-      unsigned int flg = get_conf_long(d, PCI_BASE_ADDRESS_0 + 4*i);
-      if (buscentric_view)
-       pos = flg;
-      else
-       pos = d->kernel_base_addr[i];
-      if (!pos || pos == 0xffffffff)
+      pciaddr_t pos = p->base_addr[i];
+      pciaddr_t len = (p->known_fields & PCI_FILL_SIZES) ? p->size[i] : 0;
+      pciaddr_t ioflg = (p->known_fields & PCI_FILL_IO_FLAGS) ? p->flags[i] : 0;
+      u32 flg = get_conf_long(d, PCI_BASE_ADDRESS_0 + 4*i);
+      if (flg == 0xffffffff)
+       flg = 0;
+      if (!pos && !flg && !len)
        continue;
+      if (verbose > 1)
+       printf("\tRegion %d: ", i);
+      else
+       putchar('\t');
+      if (ioflg & PCI_IORESOURCE_PCI_EA_BEI)
+         printf("[enhanced] ");
+      else if (pos && !(flg & ((flg & PCI_BASE_ADDRESS_SPACE_IO) ? PCI_BASE_ADDRESS_IO_MASK : PCI_BASE_ADDRESS_MEM_MASK)))
+       {
+         /* Reported by the OS, but not by the device */
+         printf("[virtual] ");
+         flg = pos;
+         virtual = 1;
+       }
       if (flg & PCI_BASE_ADDRESS_SPACE_IO)
        {
-         if (cmd & PCI_COMMAND_IO)
-           {
-             if (verbose > 1)
-               printf("\tRegion %d: ", i);
-             else
-               putchar('\t');
-             printf("I/O ports at %04lx\n", pos & PCI_BASE_ADDRESS_IO_MASK);
-           }
+         pciaddr_t a = pos & PCI_BASE_ADDRESS_IO_MASK;
+         printf("I/O ports at ");
+         if (a || (cmd & PCI_COMMAND_IO))
+           printf(PCIADDR_PORT_FMT, a);
+         else if (flg & PCI_BASE_ADDRESS_IO_MASK)
+           printf("<ignored>");
+         else
+           printf("<unassigned>");
+         if (!virtual && !(cmd & PCI_COMMAND_IO))
+           printf(" [disabled]");
        }
-      else if (cmd & PCI_COMMAND_MEMORY)
+      else
        {
          int t = flg & PCI_BASE_ADDRESS_MEM_TYPE_MASK;
-         if (verbose > 1)
-           printf("\tRegion %d: ", i);
-         else
-           putchar('\t');
+         pciaddr_t a = pos & PCI_ADDR_MEM_MASK;
+         int done = 0;
+         u32 z = 0;
+
          printf("Memory at ");
          if (t == PCI_BASE_ADDRESS_MEM_TYPE_64)
            {
-             if (i < cnt - 1)
+             if (i >= cnt - 1)
+               {
+                 printf("<invalid-64bit-slot>");
+                 done = 1;
+               }
+             else
                {
                  i++;
-                 if (!buscentric_view)
-                   printf("%08x", get_conf_long(d, PCI_BASE_ADDRESS_0 + 4*i));
+                 z = get_conf_long(d, PCI_BASE_ADDRESS_0 + 4*i);
                }
+           }
+         if (!done)
+           {
+             if (a)
+               printf(PCIADDR_T_FMT, a);
              else
-               printf("????????");
+               printf(((flg & PCI_BASE_ADDRESS_MEM_MASK) || z) ? "<ignored>" : "<unassigned>");
            }
-         printf("%08lx (%s, %sprefetchable)\n",
-                pos & PCI_BASE_ADDRESS_MEM_MASK,
+         printf(" (%s, %sprefetchable)",
                 (t == PCI_BASE_ADDRESS_MEM_TYPE_32) ? "32-bit" :
                 (t == PCI_BASE_ADDRESS_MEM_TYPE_64) ? "64-bit" :
-                (t == PCI_BASE_ADDRESS_MEM_TYPE_1M) ? "low-1M 32-bit" : "???",
+                (t == PCI_BASE_ADDRESS_MEM_TYPE_1M) ? "low-1M" : "type 3",
                 (flg & PCI_BASE_ADDRESS_MEM_PREFETCH) ? "" : "non-");
+         if (!virtual && !(cmd & PCI_COMMAND_MEMORY))
+           printf(" [disabled]");
        }
+      show_size(len);
+      putchar('\n');
     }
 }
 
 static void
-show_htype0(struct device *d)
+show_rom(struct device *d, int reg)
 {
-  u32 rom = get_conf_long(d, PCI_ROM_ADDRESS);
-
-  show_bases(d, 6);
-
-  if (rom & 1)
+  struct pci_dev *p = d->dev;
+  pciaddr_t rom = p->rom_base_addr;
+  pciaddr_t len = (p->known_fields & PCI_FILL_SIZES) ? p->rom_size : 0;
+  pciaddr_t ioflg = (p->known_fields & PCI_FILL_IO_FLAGS) ? p->rom_flags : 0;
+  u32 flg = get_conf_long(d, reg);
+  word cmd = get_conf_word(d, PCI_COMMAND);
+  int virtual = 0;
+
+  if (!rom && !flg && !len)
+    return;
+  putchar('\t');
+  if (ioflg & PCI_IORESOURCE_PCI_EA_BEI)
+      printf("[enhanced] ");
+  else if ((rom & PCI_ROM_ADDRESS_MASK) && !(flg & PCI_ROM_ADDRESS_MASK))
     {
-      word cmd = get_conf_word(d, PCI_COMMAND);
-      printf("\tExpansion ROM at %08x%s\n", rom & ~0xfff,
-            (cmd & PCI_COMMAND_MEMORY) ? "" : " [disabled]");
+      printf("[virtual] ");
+      flg = rom;
+      virtual = 1;
     }
+  printf("Expansion ROM at ");
+  if (rom & PCI_ROM_ADDRESS_MASK)
+    printf(PCIADDR_T_FMT, rom & PCI_ROM_ADDRESS_MASK);
+  else if (flg & PCI_ROM_ADDRESS_MASK)
+    printf("<ignored>");
+  else
+    printf("<unassigned>");
+  if (!(flg & PCI_ROM_ADDRESS_ENABLE))
+    printf(" [disabled]");
+  else if (!virtual && !(cmd & PCI_COMMAND_MEMORY))
+    printf(" [disabled by cmd]");
+  show_size(len);
+  putchar('\n');
+}
+
+static void
+show_htype0(struct device *d)
+{
+  show_bases(d, 6);
+  show_rom(d, PCI_ROM_ADDRESS);
+  show_caps(d, PCI_CAPABILITY_LIST);
 }
 
 static void
@@ -337,7 +535,7 @@ show_htype1(struct device *d)
   u32 pref_base = get_conf_word(d, PCI_PREF_MEMORY_BASE);
   u32 pref_limit = get_conf_word(d, PCI_PREF_MEMORY_LIMIT);
   u32 pref_type = pref_base & PCI_PREF_RANGE_TYPE_MASK;
-  u32 rom = get_conf_long(d, PCI_ROM_ADDRESS1);
+  word sec_stat = get_conf_word(d, PCI_SEC_STATUS);
   word brc = get_conf_word(d, PCI_BRIDGE_CONTROL);
 
   show_bases(d, 2);
@@ -359,159 +557,242 @@ show_htype1(struct device *d)
          io_base |= (get_conf_word(d, PCI_IO_BASE_UPPER16) << 16);
          io_limit |= (get_conf_word(d, PCI_IO_LIMIT_UPPER16) << 16);
        }
-      if (io_base)
-       printf("\tI/O behind bridge: %08x-%08x\n", io_base, io_limit+0xfff);
+      show_range("\tI/O behind bridge", io_base, io_limit+0xfff, 0);
     }
 
   if (mem_type != (mem_limit & PCI_MEMORY_RANGE_TYPE_MASK) ||
       mem_type)
     printf("\t!!! Unknown memory range types %x/%x\n", mem_base, mem_limit);
-  else if (mem_base)
+  else
     {
       mem_base = (mem_base & PCI_MEMORY_RANGE_MASK) << 16;
       mem_limit = (mem_limit & PCI_MEMORY_RANGE_MASK) << 16;
-      printf("\tMemory behind bridge: %08x-%08x\n", mem_base, mem_limit + 0xfffff);
+      show_range("\tMemory behind bridge", mem_base, mem_limit + 0xfffff, 0);
     }
 
   if (pref_type != (pref_limit & PCI_PREF_RANGE_TYPE_MASK) ||
       (pref_type != PCI_PREF_RANGE_TYPE_32 && pref_type != PCI_PREF_RANGE_TYPE_64))
     printf("\t!!! Unknown prefetchable memory range types %x/%x\n", pref_base, pref_limit);
-  else if (pref_base)
+  else
     {
-      pref_base = (pref_base & PCI_PREF_RANGE_MASK) << 16;
-      pref_limit = (pref_limit & PCI_PREF_RANGE_MASK) << 16;
-      if (pref_type == PCI_PREF_RANGE_TYPE_32)
-       printf("\tPrefetchable memory behind bridge: %08x-%08x\n", pref_base, pref_limit);
-      else
-       printf("\tPrefetchable memory behind bridge: %08x%08x-%08x%08x\n",
-              get_conf_long(d, PCI_PREF_BASE_UPPER32),
-              pref_base,
-              get_conf_long(d, PCI_PREF_LIMIT_UPPER32),
-              pref_limit);
+      u64 pref_base_64 = (pref_base & PCI_PREF_RANGE_MASK) << 16;
+      u64 pref_limit_64 = (pref_limit & PCI_PREF_RANGE_MASK) << 16;
+      if (pref_type == PCI_PREF_RANGE_TYPE_64)
+       {
+         pref_base_64 |= (u64) get_conf_long(d, PCI_PREF_BASE_UPPER32) << 32;
+         pref_limit_64 |= (u64) get_conf_long(d, PCI_PREF_LIMIT_UPPER32) << 32;
+       }
+      show_range("\tPrefetchable memory behind bridge", pref_base_64, pref_limit_64 + 0xfffff, (pref_type == PCI_PREF_RANGE_TYPE_64));
     }
 
-  if (get_conf_word(d, PCI_SEC_STATUS) & PCI_STATUS_SIG_SYSTEM_ERROR)
-    printf("\tSecondary status: SERR\n");
+  if (verbose > 1)
+    printf("\tSecondary status: 66MHz%c FastB2B%c ParErr%c DEVSEL=%s >TAbort%c <TAbort%c <MAbort%c <SERR%c <PERR%c\n",
+            FLAG(sec_stat, PCI_STATUS_66MHZ),
+            FLAG(sec_stat, PCI_STATUS_FAST_BACK),
+            FLAG(sec_stat, PCI_STATUS_PARITY),
+            ((sec_stat & PCI_STATUS_DEVSEL_MASK) == PCI_STATUS_DEVSEL_SLOW) ? "slow" :
+            ((sec_stat & PCI_STATUS_DEVSEL_MASK) == PCI_STATUS_DEVSEL_MEDIUM) ? "medium" :
+            ((sec_stat & PCI_STATUS_DEVSEL_MASK) == PCI_STATUS_DEVSEL_FAST) ? "fast" : "??",
+            FLAG(sec_stat, PCI_STATUS_SIG_TARGET_ABORT),
+            FLAG(sec_stat, PCI_STATUS_REC_TARGET_ABORT),
+            FLAG(sec_stat, PCI_STATUS_REC_MASTER_ABORT),
+            FLAG(sec_stat, PCI_STATUS_SIG_SYSTEM_ERROR),
+            FLAG(sec_stat, PCI_STATUS_DETECTED_PARITY));
+
+  show_rom(d, PCI_ROM_ADDRESS1);
 
-  if (rom & 1)
+  if (verbose > 1)
     {
-      word cmd = get_conf_word(d, PCI_COMMAND);
-      printf("\tExpansion ROM at %08x%s\n", rom & ~0xfff,
-            (cmd & PCI_COMMAND_MEMORY) ? "" : " [disabled]");
+      printf("\tBridgeCtl: Parity%c SERR%c NoISA%c VGA%c VGA16%c MAbort%c >Reset%c FastB2B%c\n",
+       FLAG(brc, PCI_BRIDGE_CTL_PARITY),
+       FLAG(brc, PCI_BRIDGE_CTL_SERR),
+       FLAG(brc, PCI_BRIDGE_CTL_NO_ISA),
+       FLAG(brc, PCI_BRIDGE_CTL_VGA),
+       FLAG(brc, PCI_BRIDGE_CTL_VGA_16BIT),
+       FLAG(brc, PCI_BRIDGE_CTL_MASTER_ABORT),
+       FLAG(brc, PCI_BRIDGE_CTL_BUS_RESET),
+       FLAG(brc, PCI_BRIDGE_CTL_FAST_BACK));
+      printf("\t\tPriDiscTmr%c SecDiscTmr%c DiscTmrStat%c DiscTmrSERREn%c\n",
+       FLAG(brc, PCI_BRIDGE_CTL_PRI_DISCARD_TIMER),
+       FLAG(brc, PCI_BRIDGE_CTL_SEC_DISCARD_TIMER),
+       FLAG(brc, PCI_BRIDGE_CTL_DISCARD_TIMER_STATUS),
+       FLAG(brc, PCI_BRIDGE_CTL_DISCARD_TIMER_SERR_EN));
     }
 
-  if (verbose > 1)
-    printf("\tBridgeCtl: Parity%c SERR%c NoISA%c VGA%c MAbort%c >Reset%c FastB2B%c\n",
-          (brc & PCI_BRIDGE_CTL_PARITY) ? '+' : '-',
-          (brc & PCI_BRIDGE_CTL_SERR) ? '+' : '-',
-          (brc & PCI_BRIDGE_CTL_NO_ISA) ? '+' : '-',
-          (brc & PCI_BRIDGE_CTL_VGA) ? '+' : '-',
-          (brc & PCI_BRIDGE_CTL_MASTER_ABORT) ? '+' : '-',
-          (brc & PCI_BRIDGE_CTL_BUS_RESET) ? '+' : '-',
-          (brc & PCI_BRIDGE_CTL_FAST_BACK) ? '+' : '-');
+  show_caps(d, PCI_CAPABILITY_LIST);
 }
 
 static void
 show_htype2(struct device *d)
 {
+  int i;
+  word cmd = get_conf_word(d, PCI_COMMAND);
+  word brc = get_conf_word(d, PCI_CB_BRIDGE_CONTROL);
+  word exca;
+  int verb = verbose > 2;
+
+  show_bases(d, 1);
+  printf("\tBus: primary=%02x, secondary=%02x, subordinate=%02x, sec-latency=%d\n",
+        get_conf_byte(d, PCI_CB_PRIMARY_BUS),
+        get_conf_byte(d, PCI_CB_CARD_BUS),
+        get_conf_byte(d, PCI_CB_SUBORDINATE_BUS),
+        get_conf_byte(d, PCI_CB_LATENCY_TIMER));
+  for (i=0; i<2; i++)
+    {
+      int p = 8*i;
+      u32 base = get_conf_long(d, PCI_CB_MEMORY_BASE_0 + p);
+      u32 limit = get_conf_long(d, PCI_CB_MEMORY_LIMIT_0 + p);
+      limit = limit + 0xfff;
+      if (base <= limit || verb)
+       printf("\tMemory window %d: %08x-%08x%s%s\n", i, base, limit,
+              (cmd & PCI_COMMAND_MEMORY) ? "" : " [disabled]",
+              (brc & (PCI_CB_BRIDGE_CTL_PREFETCH_MEM0 << i)) ? " (prefetchable)" : "");
+    }
+  for (i=0; i<2; i++)
+    {
+      int p = 8*i;
+      u32 base = get_conf_long(d, PCI_CB_IO_BASE_0 + p);
+      u32 limit = get_conf_long(d, PCI_CB_IO_LIMIT_0 + p);
+      if (!(base & PCI_IO_RANGE_TYPE_32))
+       {
+         base &= 0xffff;
+         limit &= 0xffff;
+       }
+      base &= PCI_CB_IO_RANGE_MASK;
+      limit = (limit & PCI_CB_IO_RANGE_MASK) + 3;
+      if (base <= limit || verb)
+       printf("\tI/O window %d: %08x-%08x%s\n", i, base, limit,
+              (cmd & PCI_COMMAND_IO) ? "" : " [disabled]");
+    }
+
+  if (get_conf_word(d, PCI_CB_SEC_STATUS) & PCI_STATUS_SIG_SYSTEM_ERROR)
+    printf("\tSecondary status: SERR\n");
+  if (verbose > 1)
+    printf("\tBridgeCtl: Parity%c SERR%c ISA%c VGA%c MAbort%c >Reset%c 16bInt%c PostWrite%c\n",
+          FLAG(brc, PCI_CB_BRIDGE_CTL_PARITY),
+          FLAG(brc, PCI_CB_BRIDGE_CTL_SERR),
+          FLAG(brc, PCI_CB_BRIDGE_CTL_ISA),
+          FLAG(brc, PCI_CB_BRIDGE_CTL_VGA),
+          FLAG(brc, PCI_CB_BRIDGE_CTL_MASTER_ABORT),
+          FLAG(brc, PCI_CB_BRIDGE_CTL_CB_RESET),
+          FLAG(brc, PCI_CB_BRIDGE_CTL_16BIT_INT),
+          FLAG(brc, PCI_CB_BRIDGE_CTL_POST_WRITES));
+
+  if (d->config_cached < 128)
+    {
+      printf("\t<access denied to the rest>\n");
+      return;
+    }
+
+  exca = get_conf_word(d, PCI_CB_LEGACY_MODE_BASE);
+  if (exca)
+    printf("\t16-bit legacy interface ports at %04x\n", exca);
+  show_caps(d, PCI_CB_CAPABILITY_LIST);
 }
 
 static void
 show_verbose(struct device *d)
 {
+  struct pci_dev *p = d->dev;
   word status = get_conf_word(d, PCI_STATUS);
   word cmd = get_conf_word(d, PCI_COMMAND);
-  word class = get_conf_word(d, PCI_CLASS_DEVICE);
+  word class = p->device_class;
   byte bist = get_conf_byte(d, PCI_BIST);
   byte htype = get_conf_byte(d, PCI_HEADER_TYPE) & 0x7f;
   byte latency = get_conf_byte(d, PCI_LATENCY_TIMER);
   byte cache_line = get_conf_byte(d, PCI_CACHE_LINE_SIZE);
   byte max_lat, min_gnt;
   byte int_pin = get_conf_byte(d, PCI_INTERRUPT_PIN);
-  byte int_line = get_conf_byte(d, PCI_INTERRUPT_LINE);
   unsigned int irq;
-  word subsys_v, subsys_d;
+  char *dt_node;
 
   show_terse(d);
 
+  pci_fill_info(p, PCI_FILL_IRQ | PCI_FILL_BASES | PCI_FILL_ROM_BASE | PCI_FILL_SIZES |
+    PCI_FILL_PHYS_SLOT | PCI_FILL_NUMA_NODE | PCI_FILL_DT_NODE);
+  irq = p->irq;
+
   switch (htype)
     {
     case PCI_HEADER_TYPE_NORMAL:
       if (class == PCI_CLASS_BRIDGE_PCI)
-       {
-       badhdr:
-         printf("\t!!! Header type %02x doesn't match class code %04x\n", htype, class);
-         return;
-       }
+       printf("\t!!! Invalid class %04x for header type %02x\n", class, htype);
       max_lat = get_conf_byte(d, PCI_MAX_LAT);
       min_gnt = get_conf_byte(d, PCI_MIN_GNT);
-      subsys_v = get_conf_word(d, PCI_SUBSYSTEM_VENDOR_ID);
-      subsys_d = get_conf_word(d, PCI_SUBSYSTEM_ID);
       break;
     case PCI_HEADER_TYPE_BRIDGE:
-      if (class != PCI_CLASS_BRIDGE_PCI)
-       goto badhdr;
-      irq = int_line = int_pin = min_gnt = max_lat = 0;
-      subsys_v = subsys_d = 0;
+      if ((class >> 8) != PCI_BASE_CLASS_BRIDGE)
+       printf("\t!!! Invalid class %04x for header type %02x\n", class, htype);
+      min_gnt = max_lat = 0;
       break;
     case PCI_HEADER_TYPE_CARDBUS:
       if ((class >> 8) != PCI_BASE_CLASS_BRIDGE)
-       goto badhdr;
-      irq = int_line = int_pin = min_gnt = max_lat = 0;
-      subsys_v = subsys_d = 0;
+       printf("\t!!! Invalid class %04x for header type %02x\n", class, htype);
+      min_gnt = max_lat = 0;
       break;
     default:
       printf("\t!!! Unknown header type %02x\n", htype);
       return;
     }
 
-  if (buscentric_view)
-    irq = int_line;
-  else
-    irq = d->kernel_irq;
+  if (p->phy_slot)
+    printf("\tPhysical Slot: %s\n", p->phy_slot);
+
+  if (dt_node = pci_get_string_property(p, PCI_FILL_DT_NODE))
+    printf("\tDevice tree node: %s\n", dt_node);
 
   if (verbose > 1)
     {
-      if (subsys_v)
-       printf("\tSubsystem ID: %04x:%04x\n", subsys_v, subsys_d);
-      printf("\tControl: I/O%c Mem%c BusMaster%c SpecCycle%c MemWINV%c VGASnoop%c ParErr%c Stepping%c SERR%c FastB2B%c\n",
-            (cmd & PCI_COMMAND_IO) ? '+' : '-',
-            (cmd & PCI_COMMAND_MEMORY) ? '+' : '-',
-            (cmd & PCI_COMMAND_MASTER) ? '+' : '-',
-            (cmd & PCI_COMMAND_SPECIAL) ? '+' : '-',
-            (cmd & PCI_COMMAND_INVALIDATE) ? '+' : '-',
-            (cmd & PCI_COMMAND_VGA_PALETTE) ? '+' : '-',
-            (cmd & PCI_COMMAND_PARITY) ? '+' : '-',
-            (cmd & PCI_COMMAND_WAIT) ? '+' : '-',
-            (cmd & PCI_COMMAND_SERR) ? '+' : '-',
-            (cmd & PCI_COMMAND_FAST_BACK) ? '+' : '-');
-      printf("\tStatus: 66Mhz%c UDF%c FastB2B%c ParErr%c DEVSEL=%s >TAbort%c <TAbort%c <MAbort%c >SERR%c <PERR%c\n",
-            (status & PCI_STATUS_66MHZ) ? '+' : '-',
-            (status & PCI_STATUS_UDF) ? '+' : '-',
-            (status & PCI_STATUS_FAST_BACK) ? '+' : '-',
-            (status & PCI_STATUS_PARITY) ? '+' : '-',
+      printf("\tControl: I/O%c Mem%c BusMaster%c SpecCycle%c MemWINV%c VGASnoop%c ParErr%c Stepping%c SERR%c FastB2B%c DisINTx%c\n",
+            FLAG(cmd, PCI_COMMAND_IO),
+            FLAG(cmd, PCI_COMMAND_MEMORY),
+            FLAG(cmd, PCI_COMMAND_MASTER),
+            FLAG(cmd, PCI_COMMAND_SPECIAL),
+            FLAG(cmd, PCI_COMMAND_INVALIDATE),
+            FLAG(cmd, PCI_COMMAND_VGA_PALETTE),
+            FLAG(cmd, PCI_COMMAND_PARITY),
+            FLAG(cmd, PCI_COMMAND_WAIT),
+            FLAG(cmd, PCI_COMMAND_SERR),
+            FLAG(cmd, PCI_COMMAND_FAST_BACK),
+            FLAG(cmd, PCI_COMMAND_DISABLE_INTx));
+      printf("\tStatus: Cap%c 66MHz%c UDF%c FastB2B%c ParErr%c DEVSEL=%s >TAbort%c <TAbort%c <MAbort%c >SERR%c <PERR%c INTx%c\n",
+            FLAG(status, PCI_STATUS_CAP_LIST),
+            FLAG(status, PCI_STATUS_66MHZ),
+            FLAG(status, PCI_STATUS_UDF),
+            FLAG(status, PCI_STATUS_FAST_BACK),
+            FLAG(status, PCI_STATUS_PARITY),
             ((status & PCI_STATUS_DEVSEL_MASK) == PCI_STATUS_DEVSEL_SLOW) ? "slow" :
             ((status & PCI_STATUS_DEVSEL_MASK) == PCI_STATUS_DEVSEL_MEDIUM) ? "medium" :
             ((status & PCI_STATUS_DEVSEL_MASK) == PCI_STATUS_DEVSEL_FAST) ? "fast" : "??",
-            (status & PCI_STATUS_SIG_TARGET_ABORT) ? '+' : '-',
-            (status & PCI_STATUS_REC_TARGET_ABORT) ? '+' : '-',
-            (status & PCI_STATUS_REC_MASTER_ABORT) ? '+' : '-',
-            (status & PCI_STATUS_SIG_SYSTEM_ERROR) ? '+' : '-',
-            (status & PCI_STATUS_DETECTED_PARITY) ? '+' : '-');
+            FLAG(status, PCI_STATUS_SIG_TARGET_ABORT),
+            FLAG(status, PCI_STATUS_REC_TARGET_ABORT),
+            FLAG(status, PCI_STATUS_REC_MASTER_ABORT),
+            FLAG(status, PCI_STATUS_SIG_SYSTEM_ERROR),
+            FLAG(status, PCI_STATUS_DETECTED_PARITY),
+            FLAG(status, PCI_STATUS_INTx));
       if (cmd & PCI_COMMAND_MASTER)
        {
-         printf("\tLatency: ");
-         if (min_gnt)
-           printf("%d min, ", min_gnt);
-         if (max_lat)
-           printf("%d max, ", max_lat);
-         printf("%d set", latency);
+         printf("\tLatency: %d", latency);
+         if (min_gnt || max_lat)
+           {
+             printf(" (");
+             if (min_gnt)
+               printf("%dns min", min_gnt*250);
+             if (min_gnt && max_lat)
+               printf(", ");
+             if (max_lat)
+               printf("%dns max", max_lat*250);
+             putchar(')');
+           }
          if (cache_line)
-           printf(", cache line size %02x", cache_line);
+           printf(", Cache Line Size: %d bytes", cache_line * 4);
          putchar('\n');
        }
-      if (int_pin)
-       printf("\tInterrupt: pin %c routed to IRQ " IRQ_FORMAT "\n", 'A' + int_pin - 1, irq);
+      if (int_pin || irq)
+       printf("\tInterrupt: pin %c routed to IRQ " PCIIRQ_FMT "\n",
+              (int_pin ? 'A' + int_pin - 1 : '?'), irq);
+      if (p->numa_node != -1)
+       printf("\tNUMA node: %d\n", p->numa_node);
     }
   else
     {
@@ -525,7 +806,7 @@ show_verbose(struct device *d)
       if (cmd & PCI_COMMAND_FAST_BACK)
        printf("fast Back2Back, ");
       if (status & PCI_STATUS_66MHZ)
-       printf("66Mhz, ");
+       printf("66MHz, ");
       if (status & PCI_STATUS_UDF)
        printf("user-definable features, ");
       printf("%s devsel",
@@ -534,11 +815,10 @@ show_verbose(struct device *d)
             ((status & PCI_STATUS_DEVSEL_MASK) == PCI_STATUS_DEVSEL_FAST) ? "fast" : "??");
       if (cmd & PCI_COMMAND_MASTER)
        printf(", latency %d", latency);
-      if (int_pin)
-       if (d->kernel_irq)
-         printf(", IRQ " IRQ_FORMAT, irq);
-       else
-         printf(", IRQ ?");
+      if (irq)
+       printf(", IRQ " PCIIRQ_FMT, irq);
+      if (p->numa_node != -1)
+       printf(", NUMA node %d", p->numa_node);
       putchar('\n');
     }
 
@@ -564,13 +844,22 @@ show_verbose(struct device *d)
     }
 }
 
+/*** Machine-readable dumps ***/
+
 static void
 show_hex_dump(struct device *d)
 {
-  int i;
-  int limit = (show_hex > 1) ? 256 : 64;
+  unsigned int i, cnt;
 
-  for(i=0; i<limit; i++)
+  cnt = d->config_cached;
+  if (opt_hex >= 3 && config_fetch(d, cnt, 256-cnt))
+    {
+      cnt = 256;
+      if (opt_hex >= 4 && config_fetch(d, 256, 4096-256))
+       cnt = 4096;
+    }
+
+  for (i=0; i<cnt; i++)
     {
       if (! (i & 15))
        printf("%02x:", i);
@@ -580,278 +869,113 @@ show_hex_dump(struct device *d)
     }
 }
 
+static void
+print_shell_escaped(char *c)
+{
+  printf(" \"");
+  while (*c)
+    {
+      if (*c == '"' || *c == '\\')
+       putchar('\\');
+      putchar(*c++);
+    }
+  putchar('"');
+}
+
 static void
 show_machine(struct device *d)
 {
+  struct pci_dev *p = d->dev;
   int c;
+  word sv_id, sd_id;
+  char classbuf[128], vendbuf[128], devbuf[128], svbuf[128], sdbuf[128];
+  char *dt_node;
+
+  get_subid(d, &sv_id, &sd_id);
 
   if (verbose)
     {
-      printf("Device:\t%02x:%02x.%x\n", d->bus, PCI_SLOT(d->devfn), PCI_FUNC(d->devfn));
-      printf("Class:\t%s\n", lookup_class(get_conf_word(d, PCI_CLASS_DEVICE)));
-      printf("Vendor:\t%s\n", lookup_vendor(d->vendid));
-      printf("Device:\t%s\n", lookup_device(d->vendid, d->devid));
+      pci_fill_info(p, PCI_FILL_PHYS_SLOT | PCI_FILL_NUMA_NODE | PCI_FILL_DT_NODE);
+      printf((opt_machine >= 2) ? "Slot:\t" : "Device:\t");
+      show_slot_name(d);
+      putchar('\n');
+      printf("Class:\t%s\n",
+            pci_lookup_name(pacc, classbuf, sizeof(classbuf), PCI_LOOKUP_CLASS, p->device_class));
+      printf("Vendor:\t%s\n",
+            pci_lookup_name(pacc, vendbuf, sizeof(vendbuf), PCI_LOOKUP_VENDOR, p->vendor_id, p->device_id));
+      printf("Device:\t%s\n",
+            pci_lookup_name(pacc, devbuf, sizeof(devbuf), PCI_LOOKUP_DEVICE, p->vendor_id, p->device_id));
+      if (sv_id && sv_id != 0xffff)
+       {
+         printf("SVendor:\t%s\n",
+                pci_lookup_name(pacc, svbuf, sizeof(svbuf), PCI_LOOKUP_SUBSYSTEM | PCI_LOOKUP_VENDOR, sv_id));
+         printf("SDevice:\t%s\n",
+                pci_lookup_name(pacc, sdbuf, sizeof(sdbuf), PCI_LOOKUP_SUBSYSTEM | PCI_LOOKUP_DEVICE, p->vendor_id, p->device_id, sv_id, sd_id));
+       }
+      if (p->phy_slot)
+       printf("PhySlot:\t%s\n", p->phy_slot);
       if (c = get_conf_byte(d, PCI_REVISION_ID))
        printf("Rev:\t%02x\n", c);
       if (c = get_conf_byte(d, PCI_CLASS_PROG))
        printf("ProgIf:\t%02x\n", c);
+      if (opt_kernel)
+       show_kernel_machine(d);
+      if (p->numa_node != -1)
+       printf("NUMANode:\t%d\n", p->numa_node);
+      if (dt_node = pci_get_string_property(p, PCI_FILL_DT_NODE))
+        printf("DTNode:\t%s\n", dt_node);
     }
   else
     {
-      printf("%02x:%02x.%x ", d->bus, PCI_SLOT(d->devfn), PCI_FUNC(d->devfn));
-      printf("\"%s\" \"%s\" \"%s\"",
-            lookup_class(get_conf_word(d, PCI_CLASS_DEVICE)),
-            lookup_vendor(d->vendid),
-            lookup_device(d->vendid, d->devid));
+      show_slot_name(d);
+      print_shell_escaped(pci_lookup_name(pacc, classbuf, sizeof(classbuf), PCI_LOOKUP_CLASS, p->device_class));
+      print_shell_escaped(pci_lookup_name(pacc, vendbuf, sizeof(vendbuf), PCI_LOOKUP_VENDOR, p->vendor_id, p->device_id));
+      print_shell_escaped(pci_lookup_name(pacc, devbuf, sizeof(devbuf), PCI_LOOKUP_DEVICE, p->vendor_id, p->device_id));
       if (c = get_conf_byte(d, PCI_REVISION_ID))
        printf(" -r%02x", c);
       if (c = get_conf_byte(d, PCI_CLASS_PROG))
        printf(" -p%02x", c);
-      putchar('\n');
-    }
-}
-
-static void
-show(void)
-{
-  struct device *d;
-
-  for(d=first_dev; d; d=d->next)
-    {
-      if (machine_readable)
-       show_machine(d);
-      else if (verbose)
-       show_verbose(d);
-      else
-       show_terse(d);
-      if (show_hex)
-       show_hex_dump(d);
-      if (verbose || show_hex)
-       putchar('\n');
-    }
-}
-
-/* Tree output */
-
-struct bridge {
-  struct bridge *chain;                        /* Single-linked list of bridges */
-  struct bridge *next, *child;         /* Tree of bridges */
-  struct bus *first_bus;               /* List of busses connected to this bridge */
-  unsigned int primary, secondary, subordinate;        /* Bus numbers */
-  struct device *br_dev;
-};
-
-struct bus {
-  unsigned int number;
-  struct bus *sibling;
-  struct device *first_dev, **last_dev;
-};
-
-static struct bridge host_bridge = { NULL, NULL, NULL, NULL, ~0, 0, ~0, NULL };
-
-static struct bus *
-find_bus(struct bridge *b, unsigned int n)
-{
-  struct bus *bus;
-
-  for(bus=b->first_bus; bus; bus=bus->sibling)
-    if (bus->number == n)
-      break;
-  return bus;
-}
-
-static struct bus *
-new_bus(struct bridge *b, unsigned int n)
-{
-  struct bus *bus = xmalloc(sizeof(struct bus));
-
-  bus = xmalloc(sizeof(struct bus));
-  bus->number = n;
-  bus->sibling = b->first_bus;
-  bus->first_dev = NULL;
-  bus->last_dev = &bus->first_dev;
-  b->first_bus = bus;
-  return bus;
-}
-
-static void
-insert_dev(struct device *d, struct bridge *b)
-{
-  struct bus *bus;
-
-  if (! (bus = find_bus(b, d->bus)))
-    {
-      struct bridge *c;
-      for(c=b->child; c; c=c->next)
-       if (c->secondary <= d->bus && d->bus <= c->subordinate)
-         return insert_dev(d, c);
-      bus = new_bus(b, d->bus);
-    }
-  /* Simple insertion at the end _does_ guarantee the correct order as the
-   * original device list was sorted by (bus, devfn) lexicographically
-   * and all devices on the new list have the same bus number.
-   */
-  *bus->last_dev = d;
-  bus->last_dev = &d->next;
-  d->next = NULL;
-}
-
-static void
-grow_tree(void)
-{
-  struct device *d, *d2;
-  struct bridge **last_br, *b;
-
-  /* Build list of bridges */
-
-  last_br = &host_bridge.chain;
-  for(d=first_dev; d; d=d->next)
-    {
-      word class = get_conf_word(d, PCI_CLASS_DEVICE);
-      if (class == PCI_CLASS_BRIDGE_PCI && (get_conf_byte(d, PCI_HEADER_TYPE) & 0x7f) == 1)
+      if (sv_id && sv_id != 0xffff)
        {
-         b = xmalloc(sizeof(struct bridge));
-         b->primary = get_conf_byte(d, PCI_PRIMARY_BUS);
-         b->secondary = get_conf_byte(d, PCI_SECONDARY_BUS);
-         b->subordinate = get_conf_byte(d, PCI_SUBORDINATE_BUS);
-         *last_br = b;
-         last_br = &b->chain;
-         b->next = b->child = NULL;
-         b->first_bus = NULL;
-         b->br_dev = d;
+         print_shell_escaped(pci_lookup_name(pacc, svbuf, sizeof(svbuf), PCI_LOOKUP_SUBSYSTEM | PCI_LOOKUP_VENDOR, sv_id));
+         print_shell_escaped(pci_lookup_name(pacc, sdbuf, sizeof(sdbuf), PCI_LOOKUP_SUBSYSTEM | PCI_LOOKUP_DEVICE, p->vendor_id, p->device_id, sv_id, sd_id));
        }
-    }
-  *last_br = NULL;
-
-  /* Create a bridge tree */
-
-  for(b=&host_bridge; b; b=b->chain)
-    {
-      struct bridge *c, *best;
-      best = NULL;
-      for(c=&host_bridge; c; c=c->chain)
-       if (c != b && b->primary >= c->secondary && b->primary <= c->subordinate &&
-           (!best || best->subordinate - best->primary > c->subordinate - c->primary))
-         best = c;
-      if (best)
-       {
-         b->next = best->child;
-         best->child = b;
-       }
-    }
-
-  /* Insert secondary bus for each bridge */
-
-  for(b=&host_bridge; b; b=b->chain)
-    if (!find_bus(b, b->secondary))
-      new_bus(b, b->secondary);
-
-  /* Create bus structs and link devices */
-
-  for(d=first_dev; d;)
-    {
-      d2 = d->next;
-      insert_dev(d, &host_bridge);
-      d = d2;
+      else
+       printf(" \"\" \"\"");
+      putchar('\n');
     }
 }
 
-static void
-print_it(byte *line, byte *p)
-{
-  *p++ = '\n';
-  *p = 0;
-  fputs(line, stdout);
-  for(p=line; *p; p++)
-    if (*p == '+' || *p == '|')
-      *p = '|';
-    else
-      *p = ' ';
-}
+/*** Main show function ***/
 
-static void show_tree_bridge(struct bridge *, byte *, byte *);
-
-static void
-show_tree_dev(struct device *d, byte *line, byte *p)
+void
+show_device(struct device *d)
 {
-  struct bridge *b;
-
-  p += sprintf(p, "%02x.%x", PCI_SLOT(d->devfn), PCI_FUNC(d->devfn));
-  for(b=&host_bridge; b; b=b->chain)
-    if (b->br_dev == d)
-      {
-       if (b->secondary == b->subordinate)
-         p += sprintf(p, "-[%02x]-", b->secondary);
-       else
-         p += sprintf(p, "-[%02x-%02x]-", b->secondary, b->subordinate);
-        show_tree_bridge(b, line, p);
-        return;
-      }
-  if (verbose)
-    p += sprintf(p, "  %s", lookup_device_full(d->vendid, d->devid));
-  print_it(line, p);
-}
-
-static void
-show_tree_bus(struct bus *b, byte *line, byte *p)
-{
-  if (!b->first_dev)
-    print_it(line, p);
-  else if (!b->first_dev->next)
-    {
-      *p++ = '-';
-      *p++ = '-';
-      show_tree_dev(b->first_dev, line, p);
-    }
+  if (opt_machine)
+    show_machine(d);
   else
     {
-      struct device *d = b->first_dev;
-      while (d->next)
-       {
-         p[0] = '+';
-         p[1] = '-';
-         show_tree_dev(d, line, p+2);
-         d = d->next;
-       }
-      p[0] = '\\';
-      p[1] = '-';
-      show_tree_dev(d, line, p+2);
-    }
-}
-
-static void
-show_tree_bridge(struct bridge *b, byte *line, byte *p)
-{
-  *p++ = '-';
-  if (!b->first_bus->sibling)
-    {
-      if (b == &host_bridge)
-        p += sprintf(p, "[%02x]-", b->first_bus->number);
-      show_tree_bus(b->first_bus, line, p);
-    }
-  else
-    {
-      struct bus *u = b->first_bus;
-      byte *k;
-
-      while (u->sibling)
-        {
-          k = p + sprintf(p, "+-[%02x]-", u->number);
-          show_tree_bus(u, line, k);
-          u = u->sibling;
-        }
-      k = p + sprintf(p, "\\-[%02x]-", u->number);
-      show_tree_bus(u, line, k);
+      if (verbose)
+       show_verbose(d);
+      else
+       show_terse(d);
+      if (opt_kernel || verbose)
+       show_kernel(d);
     }
+  if (opt_hex)
+    show_hex_dump(d);
+  if (verbose || opt_hex)
+    putchar('\n');
 }
 
 static void
-show_forest(void)
+show(void)
 {
-  char line[256];
+  struct device *d;
 
-  grow_tree();
-  show_tree_bridge(&host_bridge, line, line);
+  for (d=first_dev; d; d=d->next)
+    if (pci_filter_match(&filter, d->dev))
+      show_device(d);
 }
 
 /* Main */
@@ -862,62 +986,120 @@ main(int argc, char **argv)
   int i;
   char *msg;
 
-  filter_init(&filter);
+  if (argc == 2 && !strcmp(argv[1], "--version"))
+    {
+      puts("lspci version " PCIUTILS_VERSION);
+      return 0;
+    }
+
+  pacc = pci_alloc();
+  pacc->error = die;
+  pci_filter_init(pacc, &filter);
+
   while ((i = getopt(argc, argv, options)) != -1)
     switch (i)
       {
       case 'n':
-       show_numeric_ids = 1;
+       pacc->numeric_ids++;
        break;
       case 'v':
        verbose++;
        break;
       case 'b':
-       buscentric_view = 1;
+       pacc->buscentric = 1;
        break;
       case 's':
-       if (msg = filter_parse_slot(&filter, optarg))
-         {
-           fprintf(stderr, "lspci: -f: %s\n", msg);
-           return 1;
-         }
+       if (msg = pci_filter_parse_slot(&filter, optarg))
+         die("-s: %s", msg);
+       opt_filter = 1;
        break;
       case 'd':
-       if (msg = filter_parse_id(&filter, optarg))
-         {
-           fprintf(stderr, "lspci: -d: %s\n", msg);
-           return 1;
-         }
+       if (msg = pci_filter_parse_id(&filter, optarg))
+         die("-d: %s", msg);
+       opt_filter = 1;
        break;
       case 'x':
-       show_hex++;
+       opt_hex++;
+       break;
+      case 'P':
+       opt_path++;
+       need_topology = 1;
        break;
       case 't':
-       show_tree++;
+       opt_tree++;
+       need_topology = 1;
        break;
       case 'i':
-       pci_ids = optarg;
+        pci_set_name_list_path(pacc, optarg, 0);
+       break;
+      case 'm':
+       opt_machine++;
        break;
       case 'p':
-       pci_dir = optarg;
+       opt_pcimap = optarg;
        break;
-      case 'm':
-       machine_readable++;
+#ifdef PCI_OS_LINUX
+      case 'k':
+       opt_kernel++;
+       break;
+#endif
+      case 'M':
+       opt_map_mode++;
+       break;
+      case 'D':
+       opt_domains = 2;
+       break;
+#ifdef PCI_USE_DNS
+      case 'q':
+       opt_query_dns++;
        break;
+      case 'Q':
+       opt_query_all = 1;
+       break;
+#else
+      case 'q':
+      case 'Q':
+       die("DNS queries are not available in this version");
+#endif
       default:
+       if (parse_generic_option(i, pacc, optarg))
+         break;
       bad:
-       fprintf(stderr, help_msg);
+       fprintf(stderr, help_msg, pacc->id_file_name);
        return 1;
       }
   if (optind < argc)
     goto bad;
 
-  scan_proc();
-  sort_them();
-  if (show_tree)
-    show_forest();
+  if (opt_query_dns)
+    {
+      pacc->id_lookup_mode |= PCI_LOOKUP_NETWORK;
+      if (opt_query_dns > 1)
+       pacc->id_lookup_mode |= PCI_LOOKUP_REFRESH_CACHE;
+    }
+  if (opt_query_all)
+    pacc->id_lookup_mode |= PCI_LOOKUP_NETWORK | PCI_LOOKUP_SKIP_LOCAL;
+
+  pci_init(pacc);
+  if (opt_map_mode)
+    {
+      if (need_topology)
+       die("Bus mapping mode does not recognize bus topology");
+      map_the_bus();
+    }
   else
-    show();
+    {
+      scan_devices();
+      sort_them();
+      if (need_topology)
+       grow_tree();
+      if (opt_tree)
+       show_forest(opt_filter ? &filter : NULL);
+      else
+       show();
+    }
+  show_kernel_cleanup();
+  pci_cleanup(pacc);
 
-  return 0;
+  return (seen_errors ? 2 : 0);
 }