From nobody Tue Jan 11 18:18:11 2022 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 6608B194136D; Tue, 11 Jan 2022 18:18:13 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4JYJpX35sCz4TZ2; Tue, 11 Jan 2022 18:18:12 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 1141B1DE32; Tue, 11 Jan 2022 18:18:12 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.16.1/8.16.1) with ESMTP id 20BIIBVj000571; Tue, 11 Jan 2022 18:18:11 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.16.1/8.16.1/Submit) id 20BIIBxr000570; Tue, 11 Jan 2022 18:18:11 GMT (envelope-from git) Date: Tue, 11 Jan 2022 18:18:11 GMT Message-Id: <202201111818.20BIIBxr000570@gitrepo.freebsd.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org From: Ed Maste Subject: git: d5f4d9d52458 - releng/12.3 - Hyper-V: vPCI: Prepopulate device bars List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-all@freebsd.org X-BeenThere: dev-commits-src-all@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: emaste X-Git-Repository: src X-Git-Refname: refs/heads/releng/12.3 X-Git-Reftype: branch X-Git-Commit: d5f4d9d5245813f89a64ec6f84eebc025f3e9df9 Auto-Submitted: auto-generated ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1641925092; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=glPc1IW4UIeqVCLGtrDTR5n1Lv8CnfcdsE2kZXX1mx8=; b=tJiyZdkSWio+Qg1069l5gQj6ucAqtKnEWZzmPCKrunwYUOfWCg5yyB729EU78CEkFZLa+5 BrjxWj0iHTkqxt9s38qUdr6i6Cq/XUiWkves9NpUEgqnQYPbSkm7YUcUkwDX7WdJ2H+TfY oDuSFWCQWCqDgomhztlBkIGT6NScfm5vswCt84Z0g0hFUxQIOh5K76PtoB5zaC/gPf6/fo cKqAA1LMzjRpnOlj53ta3crTFBST69lJpIlDAHW4m2LodBiEc4FP9sUwLQwIlebT5/X/+E lvz7mIzl18CPfo+hNOQTPChdLoZ0d/+S+oOhrpL9XCpeTvM6AZbyCDFysVtTzw== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1641925092; a=rsa-sha256; cv=none; b=Cn7y3dVGrc5WHXhGHevvWAUCv276d7LhNDwzQzsJEqE0tLboq4QNCeUVhnY6HV3k0W8z80 1/3Shvdpj/Khu8nEmUQrlUvl3qp4LVQqT1ccNH0GupFI8MbTVA1AHZWP+TePR2DEhkv1OS lR8SAw6SG1q80OgnOU3RwjapWGc3QK5yVWYlLAHkdiUXGNDLuZWvin2/2khQmgSxZDX9o2 MJVGj1v77v/Oxvp9hS/B10yEb7OAyjB9TOALNw7a4r3EBt6bAQMGKaX9KIxFWr6MasfhrL bpn0xXEx0A6wBiSPbhm9gU7vYnOQFguXJ4X6ueA11lptCXp2RZmB8fXlWFBoSA== ARC-Authentication-Results: i=1; mx1.freebsd.org; none X-ThisMailContainsUnwantedMimeParts: N The branch releng/12.3 has been updated by emaste: URL: https://cgit.FreeBSD.org/src/commit/?id=d5f4d9d5245813f89a64ec6f84eebc025f3e9df9 commit d5f4d9d5245813f89a64ec6f84eebc025f3e9df9 Author: Wei Hu AuthorDate: 2021-11-27 06:42:34 +0000 Commit: Ed Maste CommitDate: 2022-01-10 00:42:55 +0000 Hyper-V: vPCI: Prepopulate device bars In recent Hyper-V releases on Windows Server 2022, vPCI code does not initialize the last 4 bit of device bar registers. This behavior change could result weird problems cuasing PCI code failure when configuring bars. Just write all 1's to those bars whose probed values are not the same as current read ones. This seems to make Hyper-V vPCI and pci_write_bar() to cooperate correctly on these releases. Reported by: khng@freebsd.org Tested by: khng@freebsd.org MFC after: 2 weeks Sponsored by: Microsoft (cherry picked from commit 75412a521f60d4b0393c730ffb284e7c6ff9d2de) (cherry picked from commit eabea1c700ad8eacb8dc780d8620b59ce72b2cf2) Approved by: so Errata: FreeBSD-EN-22:03.hyperv --- sys/dev/hyperv/pcib/vmbus_pcib.c | 43 ++++++++++++++++++++++++++++++++++++++++ 1 file changed, 43 insertions(+) diff --git a/sys/dev/hyperv/pcib/vmbus_pcib.c b/sys/dev/hyperv/pcib/vmbus_pcib.c index 72e430c946db..c7df32044678 100644 --- a/sys/dev/hyperv/pcib/vmbus_pcib.c +++ b/sys/dev/hyperv/pcib/vmbus_pcib.c @@ -1356,6 +1356,47 @@ _hv_pcifront_write_config(struct hv_pci_dev *hpdev, int where, int size, } } +/* + * The vPCI in some Hyper-V releases do not initialize the last 4 + * bit of BAR registers. This could result weird problems causing PCI + * code fail to configure BAR correctly. + * + * Just write all 1's to those BARs whose probed values are not zero. + * This seems to make the Hyper-V vPCI and pci_write_bar() to cooperate + * correctly. + */ + +static void +vmbus_pcib_prepopulate_bars(struct hv_pcibus *hbus) +{ + struct hv_pci_dev *hpdev; + int i; + + mtx_lock(&hbus->device_list_lock); + TAILQ_FOREACH(hpdev, &hbus->children, link) { + for (i = 0; i < 6; i++) { + /* Ignore empty bar */ + if (hpdev->probed_bar[i] == 0) + continue; + + uint32_t bar_val = 0; + + _hv_pcifront_read_config(hpdev, PCIR_BAR(i), + 4, &bar_val); + + if (hpdev->probed_bar[i] != bar_val) { + if (bootverbose) + printf("vmbus_pcib: initialize bar %d " + "by writing all 1s\n", i); + + _hv_pcifront_write_config(hpdev, PCIR_BAR(i), + 4, 0xffffffff); + } + } + } + mtx_unlock(&hbus->device_list_lock); +} + static void vmbus_pcib_set_detaching(void *arg, int pending __unused) { @@ -1479,6 +1520,8 @@ vmbus_pcib_attach(device_t dev) if (ret) goto vmbus_close; + vmbus_pcib_prepopulate_bars(hbus); + hbus->pci_bus = device_add_child(dev, "pci", -1); if (!hbus->pci_bus) { device_printf(dev, "failed to create pci bus\n");