From mboxrd@z Thu Jan 1 00:00:00 1970 From: Geert Uytterhoeven Subject: Re: [PATCH v9 00/12] Support PPTT for ARM64 Date: Tue, 29 May 2018 12:48:12 +0200 Message-ID: References: <20180511235807.30834-1-jeremy.linton@arm.com> <20180517170523.h7tuvbzdfluuidcz@armageddon.cambridge.arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: In-Reply-To: <20180517170523.h7tuvbzdfluuidcz@armageddon.cambridge.arm.com> Sender: linux-kernel-owner@vger.kernel.org To: Catalin Marinas , Jeremy Linton Cc: ACPI Devel Maling List , Mark Rutland , austinwc@codeaurora.org, tnowicki@caviumnetworks.com, Palmer Dabbelt , Will Deacon , linux-riscv@lists.infradead.org, Morten.Rasmussen@arm.com, vkilari@codeaurora.org, Lorenzo Pieralisi , jhugo@codeaurora.org, Al Stone , Len Brown , John Garry , wangxiongfeng2@huawei.com, Dietmar Eggemann , Linux ARM , Ard Biesheuvel , Greg KH , "Rafael J. Wysocki" , Linux Kernel Mailing List , Hanju List-Id: linux-acpi@vger.kernel.org Hi Catalin, Jeremy, On Thu, May 17, 2018 at 7:05 PM, Catalin Marinas wrote: > On Fri, May 11, 2018 at 06:57:55PM -0500, Jeremy Linton wrote: >> Jeremy Linton (12): >> drivers: base: cacheinfo: move cache_setup_of_node() >> drivers: base: cacheinfo: setup DT cache properties early >> cacheinfo: rename of_node to fw_token >> arm64/acpi: Create arch specific cpu to acpi id helper >> ACPI/PPTT: Add Processor Properties Topology Table parsing >> ACPI: Enable PPTT support on ARM64 >> drivers: base cacheinfo: Add support for ACPI based firmware tables >> arm64: Add support for ACPI based firmware tables >> arm64: topology: rename cluster_id >> arm64: topology: enable ACPI/PPTT based CPU topology >> ACPI: Add PPTT to injectable table list >> arm64: topology: divorce MC scheduling domain from core_siblings > > Queued for 4.18 (without Sudeep's latest property_read_u64 cacheinfo > patch - http://lkml.kernel.org/r/20180517154701.GA20281@e107155-lin; I > can add it separately). This is now commit 37c3ec2d810f87ea ("arm64: topology: divorce MC scheduling domain from core_siblings") in arm64/for-next/core, causing system suspend on big.LITTLE systems to hang after shutting down the first CPU: $ echo mem > /sys/power/state PM: suspend entry (deep) PM: Syncing filesystems ... done. Freezing user space processes ... (elapsed 0.001 seconds) done. OOM killer disabled. Freezing remaining freezable tasks ... (elapsed 0.001 seconds) done. Disabling non-boot CPUs ... CPU1: shutdown psci: CPU1 killed. For me, it fails on the following big.LITTLE systems: R-Car H3 ES2.0 (4xCA57 + 4xCA53) R-Car M3-W (2xCA57 + 4xCA53) System supend still works fine on systems with big cores only: R-Car H3 ES1.0 (4xCA57 (4xCA53 disabled in firmware)) R-Car M3-N (2xCA57) Reverting this commit fixes the issue for me. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933228AbeE2KsR (ORCPT ); Tue, 29 May 2018 06:48:17 -0400 Received: from mail-vk0-f68.google.com ([209.85.213.68]:41736 "EHLO mail-vk0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932847AbeE2KsO (ORCPT ); Tue, 29 May 2018 06:48:14 -0400 X-Google-Smtp-Source: ADUXVKJUAv7Y0AFigMaIPV4leCt0w+OCUD1So0q34msEv2/2WMK8hz74LgzY7//OU2WqDSIzrEvTDJgeoF7rjE/ZBIc= MIME-Version: 1.0 In-Reply-To: <20180517170523.h7tuvbzdfluuidcz@armageddon.cambridge.arm.com> References: <20180511235807.30834-1-jeremy.linton@arm.com> <20180517170523.h7tuvbzdfluuidcz@armageddon.cambridge.arm.com> From: Geert Uytterhoeven Date: Tue, 29 May 2018 12:48:12 +0200 X-Google-Sender-Auth: QtL6nt0OgAEXZ4KuIxy3cCn9v0E Message-ID: Subject: Re: [PATCH v9 00/12] Support PPTT for ARM64 To: Catalin Marinas , Jeremy Linton Cc: ACPI Devel Maling List , Mark Rutland , austinwc@codeaurora.org, tnowicki@caviumnetworks.com, Palmer Dabbelt , Will Deacon , linux-riscv@lists.infradead.org, Morten.Rasmussen@arm.com, vkilari@codeaurora.org, Lorenzo Pieralisi , jhugo@codeaurora.org, Al Stone , Len Brown , John Garry , wangxiongfeng2@huawei.com, Dietmar Eggemann , Linux ARM , Ard Biesheuvel , Greg KH , "Rafael J. Wysocki" , Linux Kernel Mailing List , Hanjun Guo , Sudeep Holla , Linux-Renesas Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Catalin, Jeremy, On Thu, May 17, 2018 at 7:05 PM, Catalin Marinas wrote: > On Fri, May 11, 2018 at 06:57:55PM -0500, Jeremy Linton wrote: >> Jeremy Linton (12): >> drivers: base: cacheinfo: move cache_setup_of_node() >> drivers: base: cacheinfo: setup DT cache properties early >> cacheinfo: rename of_node to fw_token >> arm64/acpi: Create arch specific cpu to acpi id helper >> ACPI/PPTT: Add Processor Properties Topology Table parsing >> ACPI: Enable PPTT support on ARM64 >> drivers: base cacheinfo: Add support for ACPI based firmware tables >> arm64: Add support for ACPI based firmware tables >> arm64: topology: rename cluster_id >> arm64: topology: enable ACPI/PPTT based CPU topology >> ACPI: Add PPTT to injectable table list >> arm64: topology: divorce MC scheduling domain from core_siblings > > Queued for 4.18 (without Sudeep's latest property_read_u64 cacheinfo > patch - http://lkml.kernel.org/r/20180517154701.GA20281@e107155-lin; I > can add it separately). This is now commit 37c3ec2d810f87ea ("arm64: topology: divorce MC scheduling domain from core_siblings") in arm64/for-next/core, causing system suspend on big.LITTLE systems to hang after shutting down the first CPU: $ echo mem > /sys/power/state PM: suspend entry (deep) PM: Syncing filesystems ... done. Freezing user space processes ... (elapsed 0.001 seconds) done. OOM killer disabled. Freezing remaining freezable tasks ... (elapsed 0.001 seconds) done. Disabling non-boot CPUs ... CPU1: shutdown psci: CPU1 killed. For me, it fails on the following big.LITTLE systems: R-Car H3 ES2.0 (4xCA57 + 4xCA53) R-Car M3-W (2xCA57 + 4xCA53) System supend still works fine on systems with big cores only: R-Car H3 ES1.0 (4xCA57 (4xCA53 disabled in firmware)) R-Car M3-N (2xCA57) Reverting this commit fixes the issue for me. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds From mboxrd@z Thu Jan 1 00:00:00 1970 From: geert@linux-m68k.org (Geert Uytterhoeven) Date: Tue, 29 May 2018 12:48:12 +0200 Subject: [PATCH v9 00/12] Support PPTT for ARM64 In-Reply-To: <20180517170523.h7tuvbzdfluuidcz@armageddon.cambridge.arm.com> References: <20180511235807.30834-1-jeremy.linton@arm.com> <20180517170523.h7tuvbzdfluuidcz@armageddon.cambridge.arm.com> Message-ID: To: linux-riscv@lists.infradead.org List-Id: linux-riscv.lists.infradead.org Hi Catalin, Jeremy, On Thu, May 17, 2018 at 7:05 PM, Catalin Marinas wrote: > On Fri, May 11, 2018 at 06:57:55PM -0500, Jeremy Linton wrote: >> Jeremy Linton (12): >> drivers: base: cacheinfo: move cache_setup_of_node() >> drivers: base: cacheinfo: setup DT cache properties early >> cacheinfo: rename of_node to fw_token >> arm64/acpi: Create arch specific cpu to acpi id helper >> ACPI/PPTT: Add Processor Properties Topology Table parsing >> ACPI: Enable PPTT support on ARM64 >> drivers: base cacheinfo: Add support for ACPI based firmware tables >> arm64: Add support for ACPI based firmware tables >> arm64: topology: rename cluster_id >> arm64: topology: enable ACPI/PPTT based CPU topology >> ACPI: Add PPTT to injectable table list >> arm64: topology: divorce MC scheduling domain from core_siblings > > Queued for 4.18 (without Sudeep's latest property_read_u64 cacheinfo > patch - http://lkml.kernel.org/r/20180517154701.GA20281 at e107155-lin; I > can add it separately). This is now commit 37c3ec2d810f87ea ("arm64: topology: divorce MC scheduling domain from core_siblings") in arm64/for-next/core, causing system suspend on big.LITTLE systems to hang after shutting down the first CPU: $ echo mem > /sys/power/state PM: suspend entry (deep) PM: Syncing filesystems ... done. Freezing user space processes ... (elapsed 0.001 seconds) done. OOM killer disabled. Freezing remaining freezable tasks ... (elapsed 0.001 seconds) done. Disabling non-boot CPUs ... CPU1: shutdown psci: CPU1 killed. For me, it fails on the following big.LITTLE systems: R-Car H3 ES2.0 (4xCA57 + 4xCA53) R-Car M3-W (2xCA57 + 4xCA53) System supend still works fine on systems with big cores only: R-Car H3 ES1.0 (4xCA57 (4xCA53 disabled in firmware)) R-Car M3-N (2xCA57) Reverting this commit fixes the issue for me. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds From mboxrd@z Thu Jan 1 00:00:00 1970 From: geert@linux-m68k.org (Geert Uytterhoeven) Date: Tue, 29 May 2018 12:48:12 +0200 Subject: [PATCH v9 00/12] Support PPTT for ARM64 In-Reply-To: <20180517170523.h7tuvbzdfluuidcz@armageddon.cambridge.arm.com> References: <20180511235807.30834-1-jeremy.linton@arm.com> <20180517170523.h7tuvbzdfluuidcz@armageddon.cambridge.arm.com> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Catalin, Jeremy, On Thu, May 17, 2018 at 7:05 PM, Catalin Marinas wrote: > On Fri, May 11, 2018 at 06:57:55PM -0500, Jeremy Linton wrote: >> Jeremy Linton (12): >> drivers: base: cacheinfo: move cache_setup_of_node() >> drivers: base: cacheinfo: setup DT cache properties early >> cacheinfo: rename of_node to fw_token >> arm64/acpi: Create arch specific cpu to acpi id helper >> ACPI/PPTT: Add Processor Properties Topology Table parsing >> ACPI: Enable PPTT support on ARM64 >> drivers: base cacheinfo: Add support for ACPI based firmware tables >> arm64: Add support for ACPI based firmware tables >> arm64: topology: rename cluster_id >> arm64: topology: enable ACPI/PPTT based CPU topology >> ACPI: Add PPTT to injectable table list >> arm64: topology: divorce MC scheduling domain from core_siblings > > Queued for 4.18 (without Sudeep's latest property_read_u64 cacheinfo > patch - http://lkml.kernel.org/r/20180517154701.GA20281 at e107155-lin; I > can add it separately). This is now commit 37c3ec2d810f87ea ("arm64: topology: divorce MC scheduling domain from core_siblings") in arm64/for-next/core, causing system suspend on big.LITTLE systems to hang after shutting down the first CPU: $ echo mem > /sys/power/state PM: suspend entry (deep) PM: Syncing filesystems ... done. Freezing user space processes ... (elapsed 0.001 seconds) done. OOM killer disabled. Freezing remaining freezable tasks ... (elapsed 0.001 seconds) done. Disabling non-boot CPUs ... CPU1: shutdown psci: CPU1 killed. For me, it fails on the following big.LITTLE systems: R-Car H3 ES2.0 (4xCA57 + 4xCA53) R-Car M3-W (2xCA57 + 4xCA53) System supend still works fine on systems with big cores only: R-Car H3 ES1.0 (4xCA57 (4xCA53 disabled in firmware)) R-Car M3-N (2xCA57) Reverting this commit fixes the issue for me. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds