Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 4 Apr 2022 19:11:58 GMT
From:      Colin Percival <cperciva@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: 5cc5cd739ef1 - stable/13 - uart(4): Add a concept of "unique" serial devices
Message-ID:  <202204041911.234JBw4W051402@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/13 has been updated by cperciva:

URL: https://cgit.FreeBSD.org/src/commit/?id=5cc5cd739ef15cc762d2f417bccb44daefc86d56

commit 5cc5cd739ef15cc762d2f417bccb44daefc86d56
Author:     Colin Percival <cperciva@FreeBSD.org>
AuthorDate: 2022-03-29 07:41:37 +0000
Commit:     Colin Percival <cperciva@FreeBSD.org>
CommitDate: 2022-04-04 19:11:41 +0000

    uart(4): Add a concept of "unique" serial devices
    
    FreeBSD detects serial ports twice: First, very early in the boot
    process, in order to obtain a usable console; and second, during
    the device probe/attach process.  When a UART is discovered during
    device probing, FreeBSD attempts to determine whether it is a
    device which was already being used as a console; without this,
    the console doesn't work in userland.
    
    Unfortunately it's possible for a UART to be mapped to a different
    location in memory when it is discovered on a bus than it has when
    it is announced via the ACPI SPCR table; this breaks the matching
    process, which relies on comparing bus addresses.
    
    To address this, we introduce a concept of "unique" serial devices,
    i.e. devices which are guaranteed to be present *only once* on any
    system.  If we discover one of these during device probing, we can
    match it to a same-PCI-vendor-and-device-numbers console which was
    announced via the ACPI SPCR table, regardless of the differing bus
    addresses.
    
    At present, the only unique serial device is the "Amazon PCI serial
    device" (vendor 0x1d0f, device 0x8250) found in some EC2 instances.
    This unbreaks the serial console on those systems.
    
    Reviewed by:    imp
    Sponsored by:   https://www.patreon.com/cperciva
    Differential Revision:  https://reviews.freebsd.org/D34703
    
    (cherry picked from commit ad93649d230428561db983153c546b39336fa4f1)
---
 sys/dev/uart/uart_bus_pci.c  | 51 ++++++++++++++++++++++++++++++++++++++++++++
 sys/dev/uart/uart_cpu.h      |  9 ++++++++
 sys/dev/uart/uart_cpu_acpi.c |  7 ++++++
 3 files changed, 67 insertions(+)

diff --git a/sys/dev/uart/uart_bus_pci.c b/sys/dev/uart/uart_bus_pci.c
index 92046261f544..b7612cda9f58 100644
--- a/sys/dev/uart/uart_bus_pci.c
+++ b/sys/dev/uart/uart_bus_pci.c
@@ -42,6 +42,7 @@ __FBSDID("$FreeBSD$");
 
 #include <dev/uart/uart.h>
 #include <dev/uart/uart_bus.h>
+#include <dev/uart/uart_cpu.h>
 
 #define	DEFAULT_RCLK	1843200
 
@@ -75,6 +76,11 @@ struct pci_id {
 	int		regshft;
 };
 
+struct pci_unique_id {
+	uint16_t	vendor;
+	uint16_t	device;
+};
+
 static const struct pci_id pci_ns8250_ids[] = {
 { 0x1028, 0x0008, 0xffff, 0, "Dell Remote Access Card III", 0x14,
 	128 * DEFAULT_RCLK },
@@ -209,6 +215,44 @@ uart_pci_match(device_t dev, const struct pci_id *id)
 	return ((id->vendor == vendor && id->device == device) ? id : NULL);
 }
 
+extern SLIST_HEAD(uart_devinfo_list, uart_devinfo) uart_sysdevs;
+
+/* PCI vendor/device pairs of devices guaranteed to be unique on a system. */
+static const struct pci_unique_id pci_unique_devices[] = {
+{ 0x1d0f, 0x8250 }	/* Amazon PCI serial device */
+};
+
+/* Match a UART to a console if it's a PCI device known to be unique. */
+static void
+uart_pci_unique_console_match(device_t dev)
+{
+	struct uart_softc *sc;
+	struct uart_devinfo * sysdev;
+	const struct pci_unique_id * id;
+	uint16_t vendor, device;
+
+	sc = device_get_softc(dev);
+	vendor = pci_get_vendor(dev);
+	device = pci_get_device(dev);
+
+	/* Is this a device known to exist only once in a system? */
+	for (id = pci_unique_devices; ; id++) {
+		if (id == &pci_unique_devices[nitems(pci_unique_devices)])
+			return;
+		if (id->vendor == vendor && id->device == device)
+			break;
+	}
+
+	/* If it matches a console, it must be the same device. */
+	SLIST_FOREACH(sysdev, &uart_sysdevs, next) {
+		if (sysdev->pci_info.vendor == vendor &&
+		    sysdev->pci_info.device == device) {
+			sc->sc_sysdev = sysdev;
+			sysdev->bas.rclk = sc->sc_bas.rclk;
+		}
+	}
+}
+
 static int
 uart_pci_probe(device_t dev)
 {
@@ -231,6 +275,13 @@ uart_pci_probe(device_t dev)
 	/* Bail out on error. */
 	if (result > 0)
 		return (result);
+	/*
+	 * If we haven't already matched this to a console, check if it's a
+	 * PCI device which is known to only exist once in any given system
+	 * and we can match it that way.
+	 */
+	if (sc->sc_sysdev == NULL)
+		uart_pci_unique_console_match(dev);
 	/* Set/override the device description. */
 	if (id->desc)
 		device_set_desc(dev, id->desc);
diff --git a/sys/dev/uart/uart_cpu.h b/sys/dev/uart/uart_cpu.h
index 4f34c767efc0..4eb7327fc788 100644
--- a/sys/dev/uart/uart_cpu.h
+++ b/sys/dev/uart/uart_cpu.h
@@ -52,6 +52,14 @@ struct uart_ops {
 extern bus_space_tag_t uart_bus_space_io;
 extern bus_space_tag_t uart_bus_space_mem;
 
+/*
+ * PCI ID used for matching "unique" devices to a console.
+ */
+struct uart_pci_info {
+	uint16_t vendor;
+	uint16_t device;
+};
+
 /*
  * Console and debug port device info.
  */
@@ -72,6 +80,7 @@ struct uart_devinfo {
 	void	*cookie;		/* Type dependent use. */
 	struct mtx *hwmtx;
 	struct uart_softc *sc;		/* valid only from start of attach */
+	struct uart_pci_info pci_info;
 };
 
 int uart_cpu_eqres(struct uart_bas *, struct uart_bas *);
diff --git a/sys/dev/uart/uart_cpu_acpi.c b/sys/dev/uart/uart_cpu_acpi.c
index 1e46462fc58a..68f743c8e8d2 100644
--- a/sys/dev/uart/uart_cpu_acpi.c
+++ b/sys/dev/uart/uart_cpu_acpi.c
@@ -36,6 +36,8 @@ __FBSDID("$FreeBSD$");
 
 #include <machine/bus.h>
 
+#include <dev/pci/pcireg.h>
+
 #include <dev/uart/uart.h>
 #include <dev/uart/uart_bus.h>
 #include <dev/uart/uart_cpu.h>
@@ -182,6 +184,11 @@ uart_cpu_acpi_spcr(int devtype, struct uart_devinfo *di)
 		    (int)spcr->BaudRate);
 		goto out;
 	}
+	if (spcr->PciVendorId != PCIV_INVALID &&
+	    spcr->PciDeviceId != PCIV_INVALID) {
+		di->pci_info.vendor = spcr->PciVendorId;
+		di->pci_info.device = spcr->PciDeviceId;
+	}
 
 	/* Apply device tweaks. */
 	if ((cd->cd_quirks & UART_F_IGNORE_SPCR_REGSHFT) ==



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