Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 6 May 2016 23:46:35 +0000 (UTC)
From:      John Baldwin <jhb@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r299205 - head/sys/dev/pci
Message-ID:  <201605062346.u46NkZvB058339@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: jhb
Date: Fri May  6 23:46:35 2016
New Revision: 299205
URL: https://svnweb.freebsd.org/changeset/base/299205

Log:
  Restore name=value format of PCI location strings.
  
  When devctl was added, the location string for PCI devices was changed to
  use the PCI "selector" that pciconf and devctl accept.  However, devd
  assumes that location strings are formatted as a list of name=value pairs.
  As a result, devd is no longer parsing any of the values out of PCI
  device events.  Restore the previous format of the PCI location strings
  to restore the location and slot keywords in case any devd scripts are
  using this.  Add the "selector" as a new 'dbsf' location variable.
  
  Reviewed by:	imp
  MFC after:	3 days
  Differential Revision:	https://reviews.freebsd.org/D6253

Modified:
  head/sys/dev/pci/pci.c

Modified: head/sys/dev/pci/pci.c
==============================================================================
--- head/sys/dev/pci/pci.c	Fri May  6 23:44:24 2016	(r299204)
+++ head/sys/dev/pci/pci.c	Fri May  6 23:46:35 2016	(r299205)
@@ -5427,7 +5427,8 @@ pci_child_location_str_method(device_t d
     size_t buflen)
 {
 
-	snprintf(buf, buflen, "pci%d:%d:%d:%d", pci_get_domain(child),
+	snprintf(buf, buflen, "slot=%d function=%d dbsf=pci%d:%d:%d:%d",
+	    pci_get_slot(child), pci_get_function(child), pci_get_domain(child),
 	    pci_get_bus(child), pci_get_slot(child), pci_get_function(child));
 	return (0);
 }



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201605062346.u46NkZvB058339>