Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   "Luck, Tony" <tony.luck@intel.com>
To:     "Williams, Dan J" <dan.j.williams@intel.com>,
        Bjorn Helgaas <helgaas@kernel.org>,
        "Liang, Kan" <kan.liang@linux.intel.com>
CC:     "bhelgaas@google.com" <bhelgaas@google.com>,
        "hdegoede@redhat.com" <hdegoede@redhat.com>,
        "kernelorg@undead.fr" <kernelorg@undead.fr>,
        "kjhambrick@gmail.com" <kjhambrick@gmail.com>,
        "2lprbe78@duck.com" <2lprbe78@duck.com>,
        "nicholas.johnson-opensource@outlook.com.au" 
        <nicholas.johnson-opensource@outlook.com.au>,
        "benoitg@coeus.ca" <benoitg@coeus.ca>,
        "mika.westerberg@linux.intel.com" <mika.westerberg@linux.intel.com>,
        "wse@tuxedocomputers.com" <wse@tuxedocomputers.com>,
        "mumblingdrunkard@protonmail.com" <mumblingdrunkard@protonmail.com>,
        "linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "Box, David E" <david.e.box@intel.com>,
        "Sun, Yunying" <yunying.sun@intel.com>,
        "Jiang, Dave" <dave.jiang@intel.com>
Subject: RE: Bug report: the extended PCI config space is missed with 6.2-rc2
Thread-Topic: Bug report: the extended PCI config space is missed with 6.2-rc2
Thread-Index: AQHZIVW2m6xt6ZXjPka32M949hA9Q66QewIAgAALXHA=
Date:   Fri, 6 Jan 2023 00:22:09 +0000
Message-ID: <SJ1PR11MB6083D724817B96F3B5EBA714FCFB9@SJ1PR11MB6083.namprd11.prod.outlook.com>
References: <ac2693d8-8ba3-72e0-5b66-b3ae008d539d@linux.intel.com>
 <20230105223257.GA1177387@bhelgaas>
 <63b75f8083b79_c81f02949b@dwillia2-xfh.jf.intel.com.notmuch>
In-Reply-To: <63b75f8083b79_c81f02949b@dwillia2-xfh.jf.intel.com.notmuch>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?kN2vypxYC1+0sfPxHgj0JUUo3DcvlPnSatX4CC5DfaG1WNPfDFjSFbnGiwX8?=
 =?us-ascii?Q?iHRg7LNkD1/zD8QUCLs1GVuljrDSxSEjkMnlxisk8GFlmtMeKYBHR3j/3oAe?=
 =?us-ascii?Q?hmNXZcOLiiJsNqIYCfnBzcZSCRa0yWGiJ73yQ/Tloa888AJhpGduHmE9ZRVK?=
 =?us-ascii?Q?A6vc7pWPdIMiWRTM2TFa+KijBQHMNg8gsU6QLUxecshdleRVzCP3+XuPFiGs?=
 =?us-ascii?Q?z0R1ZlFh0cx5fVYI/23mkC+3KoHXX8xUsTly4d/10XA0jQl8giO/WzsIStcU?=
 =?us-ascii?Q?/0yqHJpM7OatLR/IH0XlbfGYRWbwnfCnu7haBUVAo1tWvTbisrsuSQfvVmD/?=
 =?us-ascii?Q?uNXZ2I4k7jrh85XbkL18APUpPyTxeZ5o+u/SMQ7uukXey8WVkUa6EpFmylkr?=
 =?us-ascii?Q?inwBhYIs9ngyNVVPWFhvanOkei8yJe3VE6E8s/T72Y8pwLrGwLjXBGcpskYu?=
 =?us-ascii?Q?nvAEtT0zHLsD7WJfLkaeJub21pqKhHOsxX7RLTY+Ad7AOYI41Kjt/wgbxiiB?=
 =?us-ascii?Q?iFtJ3ZjZnVeDLZoTEvVBDreVOsZqYJw/kbWrAoWBoJ54hUlZh1XfGPUZ1GBU?=
 =?us-ascii?Q?nFybx2/kDx2FmqNjQ2Pisnxlsky4zBPcFZzY/QFXzBYwGa1hOqLof+fz7RPm?=
 =?us-ascii?Q?GhPSUGq7l9JYeyXqKTCcFXovk1Sior5FLibJHcVLulVIOzRP6ZOox/xhJhS6?=
 =?us-ascii?Q?SUDFGw/yNedAMZQMHUQmsVsSvMnqipBwT5+4i0DC12SLcPJCglKdip7SGGj6?=
 =?us-ascii?Q?0XIl/L+dO9rh8+iYFUh93o8/NCT29Dmz+IsMsd5pT4fxCzG+yq0o6+Bdb1W5?=
 =?us-ascii?Q?UkmUliHiPjXgrQG+cw759i/giRKIYXiB+QSEhxhWyEkeGvrmWCMuXLaO/1j5?=
 =?us-ascii?Q?whhZ+lZCgskofWqJitxu8IYI6SLSBj72RzOxzKKX7v7V3mc2qWPWybqEIjhz?=
 =?us-ascii?Q?o+4FXtYNLBP6lQEEQdfyx+3EBSwHvIESkrZrVloaPbR0R6Y0k0qLcSPt5ZnB?=
 =?us-ascii?Q?8l+Pv00rhq23fIhhx1U11MXldZllAPIzqovXjS+MylpoVL5sKv/I+8Dr/ALi?=
 =?us-ascii?Q?HiqX+GA6Mkd/44Idy3w3676TMZTiPlMqIKEtxwSSCsgryXjQGR44orTn1FEh?=
 =?us-ascii?Q?eKoQULZC7lrzBIfA/YZnDEj36TnoVWMsMyHKCAFwUFPPcLuyJVXzxTG5wkcr?=
 =?us-ascii?Q?KcVXPSspglJvwZ76VtjG2XNwbA2ih4sLQgXGgR3tm28DVf7zXk61AuhNw66Z?=
 =?us-ascii?Q?8kWAvx9aCeV6+1p6jmpRpz9xriuTq1VxS37+ln7i0XqfFhiUEiuuaSY1LEqd?=
 =?us-ascii?Q?v1UHlqNOrLkLN75a/eG05Cv2WDOpx9gujJtCdoyTFWTYMSZp0/iJn/k9GCPr?=
 =?us-ascii?Q?ijJZlb4Ykyu601YMM2FV9zi5oZ5OzEiVdLRm0+3A0mkJA69MTky1YDFr+yP6?=
 =?us-ascii?Q?u/cdMeotLZIMZ4rcvnt9i9TtpN34uETrv+J46rW26Fu7fiJ9LL5/F7JsEmNS?=
 =?us-ascii?Q?iBKSz/YfHB61XZg4ci2Dln3yKE08eZZKc5pgBZFGlg91J2wQZYxzlyRZB9ns?=
 =?us-ascii?Q?+3GetS/9jOZphD+AKq4Mc2FBV/1GUm4t2b8YXlkY?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: SJ1PR11MB6083.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 95970787-662a-477d-4ee1-08daef7c0d0f
X-MS-Exchange-CrossTenant-originalarrivaltime: 06 Jan 2023 00:22:10.0124
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: Nwct3VG7SY7zHIJl8k5juvPriFyWrhh2mgazVC8TMgn5rqR4s5Y4VuI8rm8hy9F5UKpXLkVKHfjGxm3IX4Q/6Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW4PR11MB6618
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,
        RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_NONE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> ...and Dave, who reported that CXL enumeration was busted in -rc2, says
> this patch fixes that. So you can also add:
>
> Tested-by: Dave Jiang <dave.jiang@intel.com>

Also seems good for my Broadwell/EDAC system.

Boot messages mentioning MMCONFIG are:

$ dmesg | grep MMCONFIG
[   12.280360] PCI: MMCONFIG for domain 0000 [bus 00-ff] at [mem 0x80000000=
-0x8fffffff] (base 0x80000000)
[   12.291606] PCI: not using MMCONFIG
[   12.873676] PCI: MMCONFIG for domain 0000 [bus 00-ff] at [mem 0x80000000=
-0x8fffffff] (base 0x80000000)
[   12.893266] PCI: MMCONFIG at [mem 0x80000000-0x8fffffff] reserved in ACP=
I motherboard resources or EFI
[   12.903601] PCI: MMCONFIG for 0000 [bus00-7f] at [mem 0x80000000-0x87fff=
fff] (base 0x80000000) (size reduced!)
[   13.385616] acpi PNP0A03:00: fail to add MMCONFIG information, can't acc=
ess extended configuration space under this bridge
[   14.115617] acpi PNP0A03:01: fail to add MMCONFIG information, can't acc=
ess extended configuration space under this bridge
[   17.969639] acpi PNP0A08:02: fail to add MMCONFIG information, can't acc=
ess extended configuration space under this bridge
[   18.298640] acpi PNP0A08:03: fail to add MMCONFIG information, can't acc=
ess extended configuration space under this bridge

The EDAC driver works with this patch applied.

Tested-by: Tony Luck <tony.luck@intel.com>

-Tony