{"id":808359,"url":"http://patchwork.ozlabs.org/api/1.0/patches/808359/?format=json","project":{"id":19,"url":"http://patchwork.ozlabs.org/api/1.0/projects/19/?format=json","name":"Linux IMX development","link_name":"linux-imx","list_id":"linux-imx-kernel.lists.patchwork.ozlabs.org","list_email":"linux-imx-kernel@lists.patchwork.ozlabs.org","web_url":null,"scm_url":null,"webscm_url":null},"msgid":"<1504198860-12951-5-git-send-email-Dave.Martin@arm.com>","date":"2017-08-31T17:00:36","name":"[v2,04/28] arm64: Port deprecated instruction emulation to new sysctl interface","commit_ref":null,"pull_url":null,"state":"new","archived":false,"hash":"bc94bcc9e69e94feb50ff2f1b8088e44b2f9d13c","submitter":{"id":26612,"url":"http://patchwork.ozlabs.org/api/1.0/people/26612/?format=json","name":"Dave Martin","email":"Dave.Martin@arm.com"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/linux-imx/patch/1504198860-12951-5-git-send-email-Dave.Martin@arm.com/mbox/","series":[{"id":883,"url":"http://patchwork.ozlabs.org/api/1.0/series/883/?format=json","date":"2017-08-31T17:00:33","name":"ARM Scalable Vector Extension (SVE)","version":2,"mbox":"http://patchwork.ozlabs.org/series/883/mbox/"}],"check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/808359/checks/","tags":{},"headers":{"Return-Path":"<linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org>","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=<UNKNOWN>)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"UGEcMc0w\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xjplL12nSz9sD5\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tFri,  1 Sep 2017 03:09:50 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dnSyS-0006EA-Ax; Thu, 31 Aug 2017 17:09:48 +0000","from foss.arm.com ([217.140.101.70])\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dnSqZ-0007FM-Mc for linux-arm-kernel@lists.infradead.org;\n\tThu, 31 Aug 2017 17:01:43 +0000","from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249])\n\tby usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 5CDD8164F;\n\tThu, 31 Aug 2017 10:01:22 -0700 (PDT)","from e103592.cambridge.arm.com (usa-sjc-imap-foss1.foss.arm.com\n\t[10.72.51.249])\n\tby usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id\n\tB0DF03F58F; Thu, 31 Aug 2017 10:01:20 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To:\n\tMessage-Id:Date:Subject:To:From:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=7xXHbN26fM82SREnjIwyT+1WRlBXQe4jg2inDwi35qU=;\n\tb=UGEcMc0wT1yRkJ\n\t63qVVNWwGZEytZE0Q4kQg96RJeeGQJpPKA1TLCTovj3BhMVKJwUP8HRUFRCR0tn8EG8TxctZ+W01z\n\tlFs7YIElEKAzef1UcY0cvcgixA5E4Uwdy37AGavGy53QVenwgDsewjco5hnm9+5js0wiVFTxHAgQX\n\tJg3Rdpe+gd5c5ZfrERgh3Z2LtbEQ6AGCKbSSl4fWfTdhcWSDzyAlRBB9ckt5CSfl73yBRYyJsXLav\n\t4PiNrDMrmG9Bav1mTwJbEcHcTPekBiGcXEZR5spOeIOcX7dPtuBIPOPb7HCt4fh1cIw+XN2iDplKc\n\tyLx/rJ75dYWeTcOVjuKQ==;","From":"Dave Martin <Dave.Martin@arm.com>","To":"linux-arm-kernel@lists.infradead.org","Subject":"[PATCH v2 04/28] arm64: Port deprecated instruction emulation to new\n\tsysctl interface","Date":"Thu, 31 Aug 2017 18:00:36 +0100","Message-Id":"<1504198860-12951-5-git-send-email-Dave.Martin@arm.com>","X-Mailer":"git-send-email 2.1.4","In-Reply-To":"<1504198860-12951-1-git-send-email-Dave.Martin@arm.com>","References":"<1504198860-12951-1-git-send-email-Dave.Martin@arm.com>","MIME-Version":"1.0","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170831_100140_036929_DDE36921 ","X-CRM114-Status":"GOOD (  12.51  )","X-Spam-Score":"-6.9 (------)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-6.9 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-5.0 RCVD_IN_DNSWL_HI RBL: Sender listed at http://www.dnswl.org/,\n\thigh trust [217.140.101.70 listed in list.dnswl.org]\n\t-0.0 SPF_PASS               SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":"<http://lists.infradead.org/mailman/options/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>","List-Archive":"<http://lists.infradead.org/pipermail/linux-arm-kernel/>","List-Post":"<mailto:linux-arm-kernel@lists.infradead.org>","List-Help":"<mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>","List-Subscribe":"<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>","Cc":"linux-arch@vger.kernel.org, libc-alpha@sourceware.org, Ard Biesheuvel\n\t<ard.biesheuvel@linaro.org>,  Szabolcs Nagy <szabolcs.nagy@arm.com>,\n\tCatalin Marinas\n\t<catalin.marinas@arm.com>, Will Deacon <will.deacon@arm.com>, Richard\n\tSandiford <richard.sandiford@arm.com>, =?utf-8?q?Alex_Benn=C3=A9e?=\n\t<alex.bennee@linaro.org>,  kvmarm@lists.cs.columbia.edu","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"\"linux-arm-kernel\" <linux-arm-kernel-bounces@lists.infradead.org>","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"},"content":"Currently, armv8_deprected.c takes charge of the \"abi\" sysctl\ndirectory, which makes life difficult for other code that wants to\nregister sysctls in the same directory.\n\nThere is a \"new\" [1] sysctl registration interface that removes the\nneed to define ctl_tables for parent directories explicitly, which\nis ideal here.\n\nThis patch ports register_insn_emulation_sysctl() over to the\nregister_sysctl() interface and removes the redundant ctl_table for\n\"abi\".\n\nSigned-off-by: Dave Martin <Dave.Martin@arm.com>\nReviewed-by: Alex Bennée <alex.bennee@linaro.org>\n\n[1] fea478d4101a (sysctl: Add register_sysctl for normal sysctl\nusers)\nThe commit message notes an intent to port users of the\npre-existing interfaces over to register_sysctl(), though the\nnumber of users of the new interface currently appears negligible.\n---\n arch/arm64/kernel/armv8_deprecated.c | 15 +++------------\n 1 file changed, 3 insertions(+), 12 deletions(-)","diff":"diff --git a/arch/arm64/kernel/armv8_deprecated.c b/arch/arm64/kernel/armv8_deprecated.c\nindex f0e6d71..e15eb2d 100644\n--- a/arch/arm64/kernel/armv8_deprecated.c\n+++ b/arch/arm64/kernel/armv8_deprecated.c\n@@ -228,15 +228,7 @@ static int emulation_proc_handler(struct ctl_table *table, int write,\n \treturn ret;\n }\n \n-static struct ctl_table ctl_abi[] = {\n-\t{\n-\t\t.procname = \"abi\",\n-\t\t.mode = 0555,\n-\t},\n-\t{ }\n-};\n-\n-static void __init register_insn_emulation_sysctl(struct ctl_table *table)\n+static void __init register_insn_emulation_sysctl(void)\n {\n \tunsigned long flags;\n \tint i = 0;\n@@ -262,8 +254,7 @@ static void __init register_insn_emulation_sysctl(struct ctl_table *table)\n \t}\n \traw_spin_unlock_irqrestore(&insn_emulation_lock, flags);\n \n-\ttable->child = insns_sysctl;\n-\tregister_sysctl_table(table);\n+\tregister_sysctl(\"abi\", insns_sysctl);\n }\n \n /*\n@@ -644,7 +635,7 @@ static int __init armv8_deprecated_init(void)\n \tcpuhp_setup_state_nocalls(CPUHP_AP_ARM64_ISNDEP_STARTING,\n \t\t\t\t  \"arm64/isndep:starting\",\n \t\t\t\t  run_all_insn_set_hw_mode, NULL);\n-\tregister_insn_emulation_sysctl(ctl_abi);\n+\tregister_insn_emulation_sysctl();\n \n \treturn 0;\n }\n","prefixes":["v2","04/28"]}