{"id":807802,"url":"http://patchwork.ozlabs.org/api/patches/807802/?format=json","web_url":"http://patchwork.ozlabs.org/project/ubuntu-kernel/patch/20170830174128.32541-5-dann.frazier@canonical.com/","project":{"id":15,"url":"http://patchwork.ozlabs.org/api/projects/15/?format=json","name":"Ubuntu Kernel","link_name":"ubuntu-kernel","list_id":"kernel-team.lists.ubuntu.com","list_email":"kernel-team@lists.ubuntu.com","web_url":null,"scm_url":null,"webscm_url":null,"list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<20170830174128.32541-5-dann.frazier@canonical.com>","list_archive_url":null,"date":"2017-08-30T17:41:26","name":"[4/6,SRU,Zesty] clocksource/drivers/arm_arch_timer: Fix read and iounmap of incorrect variable","commit_ref":null,"pull_url":null,"state":"new","archived":false,"hash":"35b76ffbebed6d9f8137fb126fdc467bec6beaaa","submitter":{"id":9337,"url":"http://patchwork.ozlabs.org/api/people/9337/?format=json","name":"dann frazier","email":"dann.frazier@canonical.com"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/ubuntu-kernel/patch/20170830174128.32541-5-dann.frazier@canonical.com/mbox/","series":[{"id":670,"url":"http://patchwork.ozlabs.org/api/series/670/?format=json","web_url":"http://patchwork.ozlabs.org/project/ubuntu-kernel/list/?series=670","date":"2017-08-30T17:41:23","name":"[1/6,SRU,Zesty] Revert \"UBUNTU: SAUCE: arm64: arch_timer: Enable CNTVCT_EL0 trap if workaround is enabled\"","version":1,"mbox":"http://patchwork.ozlabs.org/series/670/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/807802/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/807802/checks/","tags":{},"related":[],"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 3xjCWP46C1z9sN7;\n\tThu, 31 Aug 2017 03:42:25 +1000 (AEST)","from localhost ([127.0.0.1] helo=huckleberry.canonical.com)\n\tby huckleberry.canonical.com with esmtp (Exim 4.76)\n\t(envelope-from )\n\tid 1dn70Q-0002t9-Nx; Wed, 30 Aug 2017 17:42:22 +0000","from complete.lackof.org ([198.49.126.79])\n\tby huckleberry.canonical.com with esmtps\n\t(TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32)\n\t(Exim 4.76) (envelope-from ) id 1dn700-0002jM-Nm\n\tfor kernel-team@lists.ubuntu.com; Wed, 30 Aug 2017 17:41:56 +0000","from localhost (c-107-2-141-92.hsd1.co.comcast.net [107.2.141.92])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (Client did not present a certificate)\n\tby complete.lackof.org (Postfix) with ESMTPSA id 0083D33E0258\n\tfor ;\n\tWed, 30 Aug 2017 11:41:54 -0600 (MDT)"],"From":"dann frazier ","To":"kernel-team@lists.ubuntu.com","Subject":"[PATCH 4/6][SRU Zesty] clocksource/drivers/arm_arch_timer: Fix read\n\tand iounmap of incorrect variable","Date":"Wed, 30 Aug 2017 11:41:26 -0600","Message-Id":"<20170830174128.32541-5-dann.frazier@canonical.com>","X-Mailer":"git-send-email 2.14.1","In-Reply-To":"<20170830174128.32541-1-dann.frazier@canonical.com>","References":"<20170830174128.32541-1-dann.frazier@canonical.com>","X-Spam-Status":"No, score=0.0 required=5.0 tests=UNPARSEABLE_RELAY\n\tautolearn=unavailable version=3.3.2","X-Spam-Checker-Version":"SpamAssassin 3.3.2 (2011-06-06) on\n\tcomplete.lackof.org","X-BeenThere":"kernel-team@lists.ubuntu.com","X-Mailman-Version":"2.1.14","Precedence":"list","List-Id":"Kernel team discussions ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","MIME-Version":"1.0","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Errors-To":"kernel-team-bounces@lists.ubuntu.com","Sender":"kernel-team-bounces@lists.ubuntu.com"},"content":"From: Frank Rowand \n\nBugLink: https://bugs.launchpad.net/bugs/1713821\n\nFix boot warning 'Trying to vfree() nonexistent vm area'\nfrom arch_timer_mem_of_init().\n\nRefactored code attempts to read and iounmap using address frame\ninstead of address ioremap(frame->cntbase).\n\nFixes: c389d701dfb70 (\"clocksource: arm_arch_timer: split MMIO timer probing.\")\n\nSigned-off-by: Frank Rowand \nReviewed-by: Fu Wei \nAcked-by: Marc Zyngier \nSigned-off-by: Daniel Lezcano \n(cherry picked from commit 3db1200ca21f3c63c9044185dc5762ef996848cb)\nSigned-off-by: dann frazier \n---\n drivers/clocksource/arm_arch_timer.c | 4 ++--\n 1 file changed, 2 insertions(+), 2 deletions(-)","diff":"diff --git a/drivers/clocksource/arm_arch_timer.c b/drivers/clocksource/arm_arch_timer.c\nindex 3f13ad936c74..1c46d9ac6f21 100644\n--- a/drivers/clocksource/arm_arch_timer.c\n+++ b/drivers/clocksource/arm_arch_timer.c\n@@ -1219,9 +1219,9 @@ arch_timer_mem_frame_get_cntfrq(struct arch_timer_mem_frame *frame)\n \t\treturn 0;\n \t}\n \n-\trate = readl_relaxed(frame + CNTFRQ);\n+\trate = readl_relaxed(base + CNTFRQ);\n \n-\tiounmap(frame);\n+\tiounmap(base);\n \n \treturn rate;\n }\n","prefixes":["4/6","SRU","Zesty"]}