From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id D1B45C4332F for ; Fri, 3 Nov 2023 12:53:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Subject:CC:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=vp/QsPGlclELoQaK4HH7Znbpdt18aKXrWk5dzF3MR1U=; b=1pblTVZT3kFqaf Y8V+gqAuLdaU5Nr285/+tOiiSdwYoYBNVBO/OQWbzKnQFC35r6R/q43nQaCpxW0QQQZn412ag8Xmi pmZgWgF08ZanpCRcRb6UOtFszFDniR+pTisatOoVbTOWuEvf2TquhJmARpv+VlSHJixxbuf6ErLv3 z610v1YLF6/3ZLo8jMZhDO3Yi/kk9LKCzZr7jHXAoBQBHQNX3PmdCVZcNv9TYZGzA3a1sDx4X8NzG Hu35X7yjiOk91uZqdH7dH/rSskTFk31LhX3FbheUFe0d1QdzEos+ISjeC6cCZBj4SD/9f3mNOyOX5 VQI2PllrqyrI2zJBINzg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qytfW-00BR1W-3A; Fri, 03 Nov 2023 12:52:58 +0000 Received: from frasgout.his.huawei.com ([185.176.79.56]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qytfS-00BQzd-2z; Fri, 03 Nov 2023 12:52:57 +0000 Received: from lhrpeml500005.china.huawei.com (unknown [172.18.147.207]) by frasgout.his.huawei.com (SkyGuard) with ESMTP id 4SMLGZ168Hz6K982; Fri, 3 Nov 2023 20:51:34 +0800 (CST) Received: from localhost (10.202.227.76) by lhrpeml500005.china.huawei.com (7.191.163.240) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.31; Fri, 3 Nov 2023 12:52:29 +0000 Date: Fri, 3 Nov 2023 12:52:28 +0000 From: Jonathan Cameron To: "Russell King (Oracle)" CC: James Morse , , , , , , , , , , Salil Mehta , Jean-Philippe Brucker , , Subject: Re: [RFC PATCH v2 15/35] ACPI: processor: Add support for processors described as container packages Message-ID: <20231103125228.00005c94@Huawei.com> In-Reply-To: References: <20230913163823.7880-1-james.morse@arm.com> <20230913163823.7880-16-james.morse@arm.com> <20230914145353.000072e2@Huawei.com> Organization: Huawei Technologies Research and Development (UK) Ltd. X-Mailer: Claws Mail 4.1.0 (GTK 3.24.33; x86_64-w64-mingw32) MIME-Version: 1.0 X-Originating-IP: [10.202.227.76] X-ClientProxiedBy: lhrpeml100002.china.huawei.com (7.191.160.241) To lhrpeml500005.china.huawei.com (7.191.163.240) X-CFilter-Loop: Reflected X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231103_055255_268608_2364449A X-CRM114-Status: GOOD ( 31.21 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Fri, 3 Nov 2023 10:43:14 +0000 "Russell King (Oracle)" wrote: > On Thu, Sep 14, 2023 at 02:53:53PM +0100, Jonathan Cameron wrote: > > On Wed, 13 Sep 2023 16:38:03 +0000 > > James Morse wrote: > > > > > ACPI has two ways of describing processors in the DSDT. Either as a device > > > object with HID ACPI0007, or as a type 'C' package inside a Processor > > > Container. The ACPI processor driver probes CPUs described as devices, but > > > not those described as packages. > > > > > > > Specification reference needed... > > > > Terminology wise, I'd just refer to Processor() objects as I think they > > are named objects rather than data terms like a package (Which include > > a PkgLength etc) > > I'm not sure what kind of reference you want for the above. Looking in > ACPI 6.5, I've found in 5.2.12: > > "Starting with ACPI Specification 6.3, the use of the Processor() object > was deprecated. Only legacy systems should continue with this usage. On > the Itanium architecture only, a _UID is provided for the Processor() > that is a string object. This usage of _UID is also deprecated since it > can preclude an OSPM from being able to match a processor to a > non-enumerable device, such as those defined in the MADT. From ACPI > Specification 6.3 onward, all processor objects for all architectures > except Itanium must now use Device() objects with an _HID of ACPI0007, > and use only integer _UID values." > > Also, there is: > > https://uefi.org/specs/ACPI/6.5/08_Processor_Configuration_and_Control.html#declaring-processors That pair of refs, just as 'where to look if you care' cross references, seem to cover it as well as possible. > > Unfortunately, using the search facility on that site to try and find > Processor() doesn't work - it appears to strip the "()" characters from > the search (which is completely dumb, why do search facilities do that?) Yeah. Not great. > > > > The missing probe for CPUs described as packages creates a problem for > > > moving the cpu_register() calls into the acpi_processor driver, as CPUs > > > described like this don't get registered, leading to errors from other > > > subsystems when they try to add new sysfs entries to the CPU node. > > > (e.g. topology_sysfs_init()'s use of topology_add_dev() via cpuhp) > > > > > > To fix this, parse the processor container and call acpi_processor_add() > > > for each processor that is discovered like this. The processor container > > > handler is added with acpi_scan_add_handler(), so no detach call will > > > arrive. > > > > > > Qemu TCG describes CPUs using packages in a processor container. > > > > processor terms in a processor container. > > Are you wanting this to be: > > "Qemu TCG describes CPUs using processor terms in a processor > container." > > ? Searching the ACPI spec for "processor terms" (with or without quotes) > only brings up results for "terms" - yet another reason to hate site- > provided search facilities, I don't know why sites bother. :( Yup. I just use the PDFs partly for that reason. Not possible to find in 6.5 because as it's deprecated they removed the information.. Look at ACPI 6.3 and there is 19.6.108 Processor (Declare Processor) deep in the ASL operator reference Wording wise I'm not sure exactly what they should be other than they aren't packages (if my rough ASL understanding is right). Different byte encoding. Jonathan > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel