From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2607:f8b0:400e:c05::243; helo=mail-pg0-x243.google.com; envelope-from=heyi.guo@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-pg0-x243.google.com (mail-pg0-x243.google.com [IPv6:2607:f8b0:400e:c05::243]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id A235021E08283 for ; Wed, 14 Mar 2018 22:58:13 -0700 (PDT) Received: by mail-pg0-x243.google.com with SMTP id x2so2392342pgo.9 for ; Wed, 14 Mar 2018 23:04:37 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id; bh=bYiYDreMxmpwWT/8L3SUhs53LHVzy6q8IR3q9ieY1eo=; b=Jayf8gfx6pP53q+rTTMqvf2n4ZsNIaklu2oSn8Ldrf+2LeNCagL2mw9ejUP/iYyVly BaSYT28BtuLGsPSvnQXDJYV/H2jvtbSrMVgYfdwzCTOgsVqfr3itgbPyha1MHkRIznmy qmqhqiMB+8CErZO1oroaB9BLLkSoi2WaoP6T4= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=bYiYDreMxmpwWT/8L3SUhs53LHVzy6q8IR3q9ieY1eo=; b=TVky5wVm3s7rHDwGvY8e0fpmyY9BH4cxxS+LbrBXinle46BAYGQgFeZ0/Mx4HIMocZ UUp616s1HaY+1oCEfc62x6DFZrBQrWVVqI+UklOv/NH7PhmrjtQ8mCgEAUb3tGGcv1vk buvTGw5nV8pQ52OywGKqBr8XJ3fSiGtd/9i45UCYyr3gqMKV+1y73t24aTaT6/b6evDv kROiMiSb+TDa+dBXK9PYRyQ800/hEBMsiPVecnw28yOIOkjxzFmI8H3coyAK1dYdqctf dR9y4po1gLKO9JBNXrsTrKozaUWc78Am5wzyJ7CUP/NZIthlaoeBf1fUKK9u76bcRM1+ ugOQ== X-Gm-Message-State: AElRT7FmAc06u7QEEdifTGXXh9Wa2nA8WWjEx2+hFhZGLBEoZCF7iy1k WPODklRZ1A1Hoi3bPsz4BS3oKsnCIdg= X-Google-Smtp-Source: AG47ELuBhR27IMD4R8YR2JQJyu/+XNm17Mp0XXN6dbzDEWdvUQK+rv2DTk848hJknkGE85S16WA4Pg== X-Received: by 10.98.131.4 with SMTP id h4mr6673887pfe.156.1521093876605; Wed, 14 Mar 2018 23:04:36 -0700 (PDT) Received: from localhost.localdomain ([45.56.152.100]) by smtp.gmail.com with ESMTPSA id 70sm7436192pgb.86.2018.03.14.23.04.33 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Wed, 14 Mar 2018 23:04:35 -0700 (PDT) From: Heyi Guo To: edk2-devel@lists.01.org Cc: Heyi Guo , Ruiyu Ni , Ard Biesheuvel , Star Zeng , Eric Dong , Laszlo Ersek , Michael D Kinney , Maurice Ma , Prince Agyeman , Benjamin You , Jordan Justen , Anthony Perard , Julien Grall Date: Thu, 15 Mar 2018 14:03:57 +0800 Message-Id: <1521093843-48615-1-git-send-email-heyi.guo@linaro.org> X-Mailer: git-send-email 2.7.4 Subject: [PATCH v7 0/6] Add translation support to generic PciHostBridge X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 15 Mar 2018 05:58:14 -0000 Code can also be found here: https://github.com/iwishguo/edk2/tree/patch-pci-host-bridge-v7 v7: - Patch 4: implement 1 comments from Ray. v6: - Patch 1, 2: implement 3 comments from Laszlo. - Patch 4: implement 3 comments from Ray. Patch v5 inherits the code from RFC v4; we don't restart the version number for RFC to PATCH change. v5: - Patch 4/6: Modify the code according to the comments from Ray. - Patch 1/6 and 2/6 are totally new. They add initialization for all fields of PCI_ROOT_BRIDGE_APERTURE temporary variables in PciHostBridgeLib instances, so that they will not suffer from extension of PCI_ROOT_BRIDGE_APERTURE structure. - Generate a separate patch (3/6) for PciHostBridgeLib.h change. Though it is a prerequisite for patch 4/6, it does not change the code in PciHostBridge driver and won't cause any build failure or functional issue. v4: - Modify the code according to the comments from Ray, Laszlo and Ard (Please see the notes of Patch 1/3) - Ignore translation of bus in CreateRootBridge. v3: - Keep definition of Translation consistent in EDKII code: Translation = device address - host address. - Patch 2/2 is split into 2 patches (2/3 and 3/3). - Refine comments and commit messages to make the code easier to understand. v2: Changs are made according to the discussion on the mailing list, including: - PciRootBridgeIo->Configuration should return CPU view address, as well as PciIo->GetBarAttributes, and Translation Offset should be equal to PCI view address - CPU view address. - Add translation offset to PCI_ROOT_BRIDGE_APERTURE structure definition. - PciHostBridge driver internally used Base Address is still based on PCI view address, and translation offset = CPU view - PCI view, which follows the definition in ACPI, and not the same as that in UEFI spec. Cc: Ruiyu Ni Cc: Ard Biesheuvel Cc: Star Zeng Cc: Eric Dong Cc: Laszlo Ersek Cc: Michael D Kinney Cc: Maurice Ma Cc: Prince Agyeman Cc: Benjamin You Cc: Jordan Justen Cc: Anthony Perard Cc: Julien Grall Heyi Guo (6): CorebootPayloadPkg/PciHostBridgeLib: clear aperture vars for (re)init OvmfPkg/PciHostBridgeLib: clear PCI aperture vars for (re)init MdeModulePkg/PciHostBridgeLib.h: add address Translation MdeModulePkg/PciHostBridgeDxe: Add support for address translation MdeModulePkg/PciBus: convert host address to device address MdeModulePkg/PciBus: return CPU address for GetBarAttributes MdeModulePkg/Bus/Pci/PciHostBridgeDxe/PciHostBridge.h | 21 +++ MdeModulePkg/Bus/Pci/PciHostBridgeDxe/PciHostResource.h | 3 + MdeModulePkg/Include/Library/PciHostBridgeLib.h | 19 +++ CorebootPayloadPkg/Library/PciHostBridgeLib/PciHostBridgeSupport.c | 7 +- MdeModulePkg/Bus/Pci/PciBusDxe/PciIo.c | 12 +- MdeModulePkg/Bus/Pci/PciHostBridgeDxe/PciHostBridge.c | 129 ++++++++++++++++--- MdeModulePkg/Bus/Pci/PciHostBridgeDxe/PciRootBridgeIo.c | 135 ++++++++++++++++++-- OvmfPkg/Library/PciHostBridgeLib/PciHostBridgeLib.c | 4 + OvmfPkg/Library/PciHostBridgeLib/XenSupport.c | 7 +- 9 files changed, 306 insertions(+), 31 deletions(-) -- 2.7.4