[{"id":1763292,"web_url":"http://patchwork.ozlabs.org/comment/1763292/","msgid":"<7e14955f-874c-0c40-8733-4563783a1c60@canonical.com>","list_archive_url":null,"date":"2017-09-05T12:26:19","subject":"ACK: [PATCH 0/6][SRU Zesty] Miscellaneous arm_arch_timer fixes","submitter":{"id":2898,"url":"http://patchwork.ozlabs.org/api/people/2898/","name":"Stefan Bader","email":"stefan.bader@canonical.com"},"content":"On 30.08.2017 19:41, dann frazier wrote:\n> BugLink: https://bugs.launchpad.net/bugs/1713821\n> \n> I submitted an early version of the arm_arch_timer errata workaround series\n> back before zesty and, as promised[*], I've reviewed the changes that were\n> made to that series before landing to look for fixes we need to pick up.\n> One such fix was in this patch:\n> \"arm64: arch_timer: Enable CNTVCT_EL0 trap if workaround is enabled\"\n> which could break the trapping if the user access bit was set at reset.\n> For that, I've reverted the early version and replaced it with the upstream\n> cherry-pick.\n> \n> Another was in this patch:\n> \"arm64: arch_timer: Allows a CPU-specific erratum to only affect a subset of CPUs\"\n> Which, while it doesn't impact Ubuntu today, could if we were to enable\n> CNOFIG_PREEMPT_TRACER.\n> \n> We're also carrying a backport of the series that enables the SBSA\n> watchdog driver - there have been a few fixes to it since it was\n> merged, and those are included here as well.\n> \n> [*] https://lists.ubuntu.com/archives/kernel-team/2017-March/083134.html\n> \n> The following changes since commit 1bfb6eb7ecd64066f0ff2e2a5990ac6bbded1816:\n> \n> UBUNTU: Ubuntu-4.10.0-34.38 (2017-08-28 10:41:02 -0300)\n> \n> are available in the git repository at:\n> \n> git://git.launchpad.net/~dannf/ubuntu/+source/linux/+git/linux arm64-arch-timer-fixes-z\n> \n> for you to fetch changes up to 5c7cce8036222550a599ed584dce0aa5c6f314a1:\n> \n> clocksource/drivers/arm_arch_timer: Avoid infinite recursion when ftrace is enabled (2017-08-29 16:57:42 -0600)\n> \n> ----------------------------------------------------------------\n> Ding Tianhong (1):\n> clocksource/drivers/arm_arch_timer: Avoid infinite recursion when ftrace is enabled\n> \n> Frank Rowand (1):\n> clocksource/drivers/arm_arch_timer: Fix read and iounmap of incorrect variable\n> \n> Marc Zyngier (1):\n> arm64: arch_timer: Enable CNTVCT_EL0 trap if workaround is enabled\n> \n> Matthias Kaehlcke (1):\n> clocksource/drivers/arm_arch_timer: Fix mem frame loop initialization\n> \n> Sudeep Holla (1):\n> clocksource/arm_arch_timer: Fix arch_timer_mem_find_best_frame()\n> \n> dann frazier (1):\n> Revert \"UBUNTU: SAUCE: arm64: arch_timer: Enable CNTVCT_EL0 trap if workaround is enabled\"\n> \n> arch/arm64/include/asm/arch_timer.h | 4 ++--\n> drivers/clocksource/arm_arch_timer.c | 11 ++++++-----\n> 2 files changed, 8 insertions(+), 7 deletions(-)\n> \nAcked-by: Stefan Bader \n\nChanges restricted to specific arch code which can be verified.","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.ubuntu.com\n\t(client-ip=91.189.94.19; helo=huckleberry.canonical.com;\n\tenvelope-from=kernel-team-bounces@lists.ubuntu.com;\n\treceiver=)","Received":["from huckleberry.canonical.com (huckleberry.canonical.com\n\t[91.189.94.19])\n\tby ozlabs.org (Postfix) with ESMTP id 3xmmD93Kvxz9sRm;\n\tTue, 5 Sep 2017 22:26:33 +1000 (AEST)","from localhost ([127.0.0.1] helo=huckleberry.canonical.com)\n\tby huckleberry.canonical.com with esmtp (Exim 4.86_2)\n\t(envelope-from )\n\tid 1dpCw1-0004Ke-9I; Tue, 05 Sep 2017 12:26:29 +0000","from youngberry.canonical.com ([91.189.89.112])\n\tby huckleberry.canonical.com with esmtps\n\t(TLS1.0:DHE_RSA_AES_128_CBC_SHA1:128)\n\t(Exim 4.86_2) (envelope-from )\n\tid 1dpCvz-0004KR-95\n\tfor kernel-team@lists.ubuntu.com; Tue, 05 Sep 2017 12:26:27 +0000","from 1.general.smb.uk.vpn ([10.172.193.28])\n\tby youngberry.canonical.com with esmtpsa\n\t(TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16)\n\t(Exim 4.76) (envelope-from )\n\tid 1dpCvz-0004mG-0y\n\tfor kernel-team@lists.ubuntu.com; Tue, 05 Sep 2017 12:26:27 +0000"],"Subject":"ACK: [PATCH 0/6][SRU Zesty] Miscellaneous arm_arch_timer fixes","To":"kernel-team@lists.ubuntu.com","References":"<20170830174128.32541-1-dann.frazier@canonical.com>","From":"Stefan Bader ","Message-ID":"<7e14955f-874c-0c40-8733-4563783a1c60@canonical.com>","Date":"Tue, 5 Sep 2017 14:26:19 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"<20170830174128.32541-1-dann.frazier@canonical.com>","X-BeenThere":"kernel-team@lists.ubuntu.com","X-Mailman-Version":"2.1.20","Precedence":"list","List-Id":"Kernel team discussions ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"multipart/mixed;\n\tboundary=\"===============5883544678708932758==\"","Errors-To":"kernel-team-bounces@lists.ubuntu.com","Sender":"\"kernel-team\" "}},{"id":1763306,"web_url":"http://patchwork.ozlabs.org/comment/1763306/","msgid":"<551132c8-e029-16c8-ee07-bb220b5e7321@canonical.com>","list_archive_url":null,"date":"2017-09-05T12:43:31","subject":"ACK: [PATCH 0/6][SRU Zesty] Miscellaneous arm_arch_timer fixes","submitter":{"id":2900,"url":"http://patchwork.ozlabs.org/api/people/2900/","name":"Colin Ian King","email":"colin.king@canonical.com"},"content":"On 30/08/17 18:41, dann frazier wrote:\n> BugLink: https://bugs.launchpad.net/bugs/1713821\n> \n> I submitted an early version of the arm_arch_timer errata workaround series\n> back before zesty and, as promised[*], I've reviewed the changes that were\n> made to that series before landing to look for fixes we need to pick up.\n> One such fix was in this patch:\n> \"arm64: arch_timer: Enable CNTVCT_EL0 trap if workaround is enabled\"\n> which could break the trapping if the user access bit was set at reset.\n> For that, I've reverted the early version and replaced it with the upstream\n> cherry-pick.\n> \n> Another was in this patch:\n> \"arm64: arch_timer: Allows a CPU-specific erratum to only affect a subset of CPUs\"\n> Which, while it doesn't impact Ubuntu today, could if we were to enable\n> CNOFIG_PREEMPT_TRACER.\n> \n> We're also carrying a backport of the series that enables the SBSA\n> watchdog driver - there have been a few fixes to it since it was\n> merged, and those are included here as well.\n> \n> [*] https://lists.ubuntu.com/archives/kernel-team/2017-March/083134.html\n> \n> The following changes since commit 1bfb6eb7ecd64066f0ff2e2a5990ac6bbded1816:\n> \n> UBUNTU: Ubuntu-4.10.0-34.38 (2017-08-28 10:41:02 -0300)\n> \n> are available in the git repository at:\n> \n> git://git.launchpad.net/~dannf/ubuntu/+source/linux/+git/linux arm64-arch-timer-fixes-z\n> \n> for you to fetch changes up to 5c7cce8036222550a599ed584dce0aa5c6f314a1:\n> \n> clocksource/drivers/arm_arch_timer: Avoid infinite recursion when ftrace is enabled (2017-08-29 16:57:42 -0600)\n> \n> ----------------------------------------------------------------\n> Ding Tianhong (1):\n> clocksource/drivers/arm_arch_timer: Avoid infinite recursion when ftrace is enabled\n> \n> Frank Rowand (1):\n> clocksource/drivers/arm_arch_timer: Fix read and iounmap of incorrect variable\n> \n> Marc Zyngier (1):\n> arm64: arch_timer: Enable CNTVCT_EL0 trap if workaround is enabled\n> \n> Matthias Kaehlcke (1):\n> clocksource/drivers/arm_arch_timer: Fix mem frame loop initialization\n> \n> Sudeep Holla (1):\n> clocksource/arm_arch_timer: Fix arch_timer_mem_find_best_frame()\n> \n> dann frazier (1):\n> Revert \"UBUNTU: SAUCE: arm64: arch_timer: Enable CNTVCT_EL0 trap if workaround is enabled\"\n> \n> arch/arm64/include/asm/arch_timer.h | 4 ++--\n> drivers/clocksource/arm_arch_timer.c | 11 ++++++-----\n> 2 files changed, 8 insertions(+), 7 deletions(-)\n> \n\nThese are limited to a single arch and are verifiable. All seem\nreasonable fixes to me. Thanks Dann.\n\nAcked-by: Colin Ian King ","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.ubuntu.com\n\t(client-ip=91.189.94.19; helo=huckleberry.canonical.com;\n\tenvelope-from=kernel-team-bounces@lists.ubuntu.com;\n\treceiver=)","Received":["from huckleberry.canonical.com (huckleberry.canonical.com\n\t[91.189.94.19])\n\tby ozlabs.org (Postfix) with ESMTP id 3xmmbv1Tjdz9sRV;\n\tTue, 5 Sep 2017 22:43:39 +1000 (AEST)","from localhost ([127.0.0.1] helo=huckleberry.canonical.com)\n\tby huckleberry.canonical.com with esmtp (Exim 4.86_2)\n\t(envelope-from )\n\tid 1dpDCZ-0005uO-9c; Tue, 05 Sep 2017 12:43:35 +0000","from youngberry.canonical.com ([91.189.89.112])\n\tby huckleberry.canonical.com with esmtps\n\t(TLS1.0:DHE_RSA_AES_128_CBC_SHA1:128)\n\t(Exim 4.86_2) (envelope-from )\n\tid 1dpDCW-0005uG-TT\n\tfor kernel-team@lists.ubuntu.com; Tue, 05 Sep 2017 12:43:32 +0000","from 1.general.cking.uk.vpn ([10.172.193.212])\n\tby youngberry.canonical.com with esmtpsa\n\t(TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16)\n\t(Exim 4.76) (envelope-from )\n\tid 1dpDCW-0006c5-Kc; Tue, 05 Sep 2017 12:43:32 +0000"],"Subject":"ACK: [PATCH 0/6][SRU Zesty] Miscellaneous arm_arch_timer fixes","To":"dann frazier , kernel-team@lists.ubuntu.com","References":"<20170830174128.32541-1-dann.frazier@canonical.com>","From":"Colin Ian King ","Message-ID":"<551132c8-e029-16c8-ee07-bb220b5e7321@canonical.com>","Date":"Tue, 5 Sep 2017 13:43:31 +0100","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101\n\tThunderbird/45.8.0","MIME-Version":"1.0","In-Reply-To":"<20170830174128.32541-1-dann.frazier@canonical.com>","X-BeenThere":"kernel-team@lists.ubuntu.com","X-Mailman-Version":"2.1.20","Precedence":"list","List-Id":"Kernel team discussions ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"kernel-team-bounces@lists.ubuntu.com","Sender":"\"kernel-team\" "}},{"id":1769153,"web_url":"http://patchwork.ozlabs.org/comment/1769153/","msgid":"<707c2edc-29c9-91d9-5509-92f07b901e5f@canonical.com>","list_archive_url":null,"date":"2017-09-15T12:34:51","subject":"APPLIED Z: [PATCH 0/6][SRU Zesty] Miscellaneous arm_arch_timer fixes","submitter":{"id":2898,"url":"http://patchwork.ozlabs.org/api/people/2898/","name":"Stefan Bader","email":"stefan.bader@canonical.com"},"content":"On 30.08.2017 19:41, dann frazier wrote:\n> BugLink: https://bugs.launchpad.net/bugs/1713821\n> \n> I submitted an early version of the arm_arch_timer errata workaround series\n> back before zesty and, as promised[*], I've reviewed the changes that were\n> made to that series before landing to look for fixes we need to pick up.\n> One such fix was in this patch:\n> \"arm64: arch_timer: Enable CNTVCT_EL0 trap if workaround is enabled\"\n> which could break the trapping if the user access bit was set at reset.\n> For that, I've reverted the early version and replaced it with the upstream\n> cherry-pick.\n> \n> Another was in this patch:\n> \"arm64: arch_timer: Allows a CPU-specific erratum to only affect a subset of CPUs\"\n> Which, while it doesn't impact Ubuntu today, could if we were to enable\n> CNOFIG_PREEMPT_TRACER.\n> \n> We're also carrying a backport of the series that enables the SBSA\n> watchdog driver - there have been a few fixes to it since it was\n> merged, and those are included here as well.\n> \n> [*] https://lists.ubuntu.com/archives/kernel-team/2017-March/083134.html\n> \n> The following changes since commit 1bfb6eb7ecd64066f0ff2e2a5990ac6bbded1816:\n> \n> UBUNTU: Ubuntu-4.10.0-34.38 (2017-08-28 10:41:02 -0300)\n> \n> are available in the git repository at:\n> \n> git://git.launchpad.net/~dannf/ubuntu/+source/linux/+git/linux arm64-arch-timer-fixes-z\n> \n> for you to fetch changes up to 5c7cce8036222550a599ed584dce0aa5c6f314a1:\n> \n> clocksource/drivers/arm_arch_timer: Avoid infinite recursion when ftrace is enabled (2017-08-29 16:57:42 -0600)\n> \n> ----------------------------------------------------------------\n> Ding Tianhong (1):\n> clocksource/drivers/arm_arch_timer: Avoid infinite recursion when ftrace is enabled\n> \n> Frank Rowand (1):\n> clocksource/drivers/arm_arch_timer: Fix read and iounmap of incorrect variable\n> \n> Marc Zyngier (1):\n> arm64: arch_timer: Enable CNTVCT_EL0 trap if workaround is enabled\n> \n> Matthias Kaehlcke (1):\n> clocksource/drivers/arm_arch_timer: Fix mem frame loop initialization\n> \n> Sudeep Holla (1):\n> clocksource/arm_arch_timer: Fix arch_timer_mem_find_best_frame()\n> \n> dann frazier (1):\n> Revert \"UBUNTU: SAUCE: arm64: arch_timer: Enable CNTVCT_EL0 trap if workaround is enabled\"\n> \n> arch/arm64/include/asm/arch_timer.h | 4 ++--\n> drivers/clocksource/arm_arch_timer.c | 11 ++++++-----\n> 2 files changed, 8 insertions(+), 7 deletions(-)\n> \nApplied to master-next","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.ubuntu.com\n\t(client-ip=91.189.94.19; helo=huckleberry.canonical.com;\n\tenvelope-from=kernel-team-bounces@lists.ubuntu.com;\n\treceiver=)","Received":["from huckleberry.canonical.com (huckleberry.canonical.com\n\t[91.189.94.19])\n\tby ozlabs.org (Postfix) with ESMTP id 3xtvxG0YVGz9s7C;\n\tFri, 15 Sep 2017 22:34:58 +1000 (AEST)","from localhost ([127.0.0.1] helo=huckleberry.canonical.com)\n\tby huckleberry.canonical.com with esmtp (Exim 4.86_2)\n\t(envelope-from )\n\tid 1dsppe-00031o-2u; Fri, 15 Sep 2017 12:34:54 +0000","from youngberry.canonical.com ([91.189.89.112])\n\tby huckleberry.canonical.com with esmtps\n\t(TLS1.0:DHE_RSA_AES_128_CBC_SHA1:128)\n\t(Exim 4.86_2) (envelope-from )\n\tid 1dsppc-00031S-Ch\n\tfor kernel-team@lists.ubuntu.com; Fri, 15 Sep 2017 12:34:52 +0000","from 1.general.smb.uk.vpn ([10.172.193.28])\n\tby youngberry.canonical.com with esmtpsa\n\t(TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16)\n\t(Exim 4.76) (envelope-from )\n\tid 1dsppc-0002Gl-4i\n\tfor kernel-team@lists.ubuntu.com; Fri, 15 Sep 2017 12:34:52 +0000"],"Subject":"APPLIED Z: [PATCH 0/6][SRU Zesty] Miscellaneous arm_arch_timer fixes","To":"kernel-team@lists.ubuntu.com","References":"<20170830174128.32541-1-dann.frazier@canonical.com>","From":"Stefan Bader ","Message-ID":"<707c2edc-29c9-91d9-5509-92f07b901e5f@canonical.com>","Date":"Fri, 15 Sep 2017 14:34:51 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"<20170830174128.32541-1-dann.frazier@canonical.com>","X-BeenThere":"kernel-team@lists.ubuntu.com","X-Mailman-Version":"2.1.20","Precedence":"list","List-Id":"Kernel team discussions ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"multipart/mixed;\n\tboundary=\"===============4583304588185350376==\"","Errors-To":"kernel-team-bounces@lists.ubuntu.com","Sender":"\"kernel-team\" "}}]