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>
CC:     "Liang, Kan" <kan.liang@linux.intel.com>,
        "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>
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: AQHZIS5W8o9BItAAdUOkB8l+e5XlU66QHQ6QgAAH+QCAABTaAIAAAkWA
Date:   Thu, 5 Jan 2023 19:58:47 +0000
Message-ID: <SJ1PR11MB608347C8A95717A8BEE3BF2DFCFA9@SJ1PR11MB6083.namprd11.prod.outlook.com>
References: <SJ1PR11MB6083C504335B2DE1B31C440CFCFA9@SJ1PR11MB6083.namprd11.prod.outlook.com>
 <20230105182950.GA1158496@bhelgaas>
 <63b7289cb7439_5174129445@dwillia2-xfh.jf.intel.com.notmuch>
In-Reply-To: <63b7289cb7439_5174129445@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?eoUynHebrSSdLz4IOEVo4jhIhQBNCuBUsPrXq1uL0D7pBQwCrPMjXnhqL2KT?=
 =?us-ascii?Q?hDGl/8U9gudn4lqmFMgzrWHf889q/plwkKwuMY3SYqO2bumul2Ehhat/pgwW?=
 =?us-ascii?Q?gOfsk0vEE9YtyTfeS/agHktxzkgg5fxDkJmgmIhbY8zJy71MKBzFqDZ4m6KF?=
 =?us-ascii?Q?0Q1DZSVGltz7xkieN52VNCL4MMuLJnJZcjztdW35UWFsxFDbCeOHkBgd/U0H?=
 =?us-ascii?Q?zvON7cqp1cYu56vDFad+bSo+T+YvPN2jGFf7CbcAcYb1Zdn2s/P6/HIFcFYw?=
 =?us-ascii?Q?7LE2UXN3A5BgzqEXFx5nPBTnkn437vH/wi/ZD9EzQPDkxWspP7LYHxi5Nbd1?=
 =?us-ascii?Q?+cYKKGf2FQ7KUrYChlVP/qq04C6cB/xXPyNuf+9o4Y90QzWrQVd6E7D04AgM?=
 =?us-ascii?Q?DN0vpIDok3j6NdcB/Tmu5pz29UaIWtBGQar8bFd4bJHOP1P3+YtAUgikJe4d?=
 =?us-ascii?Q?VBTUuQE8KHe8W/E41n+/N00sANATzzrSMw4Gp3rW7Dzu9nnl0JDFkTrHnU29?=
 =?us-ascii?Q?7RyxgULdquu9aUyj8EOIvGhbW1tFxCEm+XOX3/uejQ2yPLqtci+5fk9HDW8C?=
 =?us-ascii?Q?h03cwhpWtT5/7+KGkyY3aajyCE7n1AB5+fxi+c8GY44avr+bJ+cFL+42xJ5d?=
 =?us-ascii?Q?EVnoE4MABAPDPmdknfa8ZeTVj6zCdRfBOyzv38ybTrJ3Tf/P7OzpImPajurO?=
 =?us-ascii?Q?EsEzI6gFTL3vDUgbOU+xN4D3fJwmvkcN69OUrsw31PSTzMDUVw+yMsRTb6zc?=
 =?us-ascii?Q?qCbDY59LIKEn/Y3wRAE+giMfTmMxnjJOvlymblIPQtfo+RCri0CJ4hdm/bSy?=
 =?us-ascii?Q?bUeGh5D/pzA/4ldv0OPIAbZsBqJiLlyzV30tF0BcSFOqLrELyD6+zJfsLNA7?=
 =?us-ascii?Q?CMB0UHdaAvZ725R1ftXGle/n9OWj99KrqZKuOVP2Ui+TN3cHoYhYVyqCNYK4?=
 =?us-ascii?Q?VFdLLk/YtH5iU64hG9qLrxbTGNyf4U2bMqhcdlT8r9png0O8XFGFMEt0h1qX?=
 =?us-ascii?Q?fAA3xgOSOu3D2xmjSxsor0pXKwm4t+Y9VBZUQQOumRhCoFmrl5Sc0IBKmXb7?=
 =?us-ascii?Q?oXGqYZOKlaTq8CozSOadzID2puVdU7COUjFi9kbKsaft/NJHq8QcFSHd/IdK?=
 =?us-ascii?Q?HiwWz7Qgz+7rVTRA0ZsJ8a4i6bXRJY7Ha5DODQ3Fie0Ays5n+8Q/p0VSWMA6?=
 =?us-ascii?Q?9DkZ1YZ74Br0sPL9T10haBhCKDJ7e1N09FhqU2tJGSHQqwdyjxfMp1Kj4udP?=
 =?us-ascii?Q?gT3f4/V5/H6vMFxK69QN1M6Nlwwh8Iphyjh2mwf/0LNO1z47FVPWUE5mBfHR?=
 =?us-ascii?Q?kIkaYFLDqmYYFCY+PemAtPu5xgMwgfgh4VueEb+RAZhaPujOgXiA5quT+bpe?=
 =?us-ascii?Q?CQMjSlUL+YFczwxYv2HXIQ9RjMO5pWBpVQCWN+tV3hlG3T9jjDw6rKtu4fCj?=
 =?us-ascii?Q?Xcv6j3n/vTAHC21AyTG9ifH4NpMiWiyTQxf8DCxUKCHrgnzWtBoNuSYNNhCn?=
 =?us-ascii?Q?Z9l2pLHzUG2nRFGeTgh5pEUnlhGpJ66GkJyxBS3v0d9LkevivbUsHeZZ/tzu?=
 =?us-ascii?Q?W/SH+AZNhQjQk9X7KUc=3D?=
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: 192ff7f3-6f10-4eee-741e-08daef574201
X-MS-Exchange-CrossTenant-originalarrivaltime: 05 Jan 2023 19:58:47.4306
 (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: ET/wDxS9yidgRAHgquCirUDtn2Hf/cXLnZVg0lOvx4eUeSm6sDX5Rj6DBrMmbWXFQ34IfU3pTgaBjK3gi9dLQw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR11MB6213
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,
        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

> Definitely an ambiguity / conflict, but not sure it is a bug when you
> look at from the perspective of how would an EFI runtime service use
> ECAM/MMCONFIG space?=20
>
> Would it be enough to add this clarification in "EFI 2.9 Table 7-6
> Memory Type Usage after ExitBootServices()"?
>
> s/This memory is not used by the OS./This memory is not used by the OS,
> unless ACPI declares it for another purpose./

In the case of the EDAC driver there isn't any ACPI declaration.  It just d=
oes
pci_get_device() to find the devices it needs to use with a lookup based on
vendor-id and device-id.

and then uses	pci_read_config_dword() to read various offsets from config
space, some of which are in PCIe extended config space.

-Tony