[{"id":1768053,"web_url":"http://patchwork.ozlabs.org/comment/1768053/","msgid":"<20170913172911.3ca2h6cpju7etifi@localhost>","list_archive_url":null,"date":"2017-09-13T17:29:11","subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","submitter":{"id":938,"url":"http://patchwork.ozlabs.org/api/people/938/","name":"Catalin Marinas","email":"catalin.marinas@arm.com"},"content":"On Thu, Aug 31, 2017 at 06:00:46PM +0100, Dave P Martin wrote:\n> This patch implements the core logic for changing a task's vector\n> length on request from userspace.  This will be used by the ptrace\n> and prctl frontends that are implemented in later patches.\n> \n> The SVE architecture permits, but does not require, implementations\n> to support vector lengths that are not a power of two.  To handle\n> this, logic is added to check a requested vector length against a\n> possibly sparse bitmap of available vector lengths at runtime, so\n> that the best supported value can be chosen.\n> \n> Signed-off-by: Dave Martin <Dave.Martin@arm.com>\n> Cc: Alex Bennée <alex.bennee@linaro.org>\n\nCan this be merged with patch 20? It seems to add the PR_ definitions\nwhich get actually used later when the prctl interface is added.","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=\"jordT1xs\"; 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 3xspZF4X3sz9ryT\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tThu, 14 Sep 2017 03:29:41 +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 1dsBTn-0000Fk-0V; Wed, 13 Sep 2017 17:29:39 +0000","from usa-sjc-mx-foss1.foss.arm.com ([217.140.101.70]\n\thelo=foss.arm.com)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dsBTj-0000E2-Eb for linux-arm-kernel@lists.infradead.org;\n\tWed, 13 Sep 2017 17:29:36 +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 4A7A31435;\n\tWed, 13 Sep 2017 10:29:15 -0700 (PDT)","from localhost (usa-sjc-mx-foss1.foss.arm.com [217.140.101.70])\n\tby usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id\n\t368593F483; Wed, 13 Sep 2017 10:29:15 -0700 (PDT)","from cmarinas by localhost with local (Exim 4.89)\n\t(envelope-from <catalin.marinas@arm.com>)\n\tid 1dsBTM-000060-23; Wed, 13 Sep 2017 10:29:12 -0700"],"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:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=P1daZJvzCODGAwsq9BhBiT8q/lyHDtQcSqlpn3IG8+4=;\n\tb=jordT1xskj3YxJ\n\t1mT2YjVTnoPgDU1/b6Wtmh0M/gD0Q/99dZuiqlmWIuMggRKyUM71voxFcni8KmZfWhTud/oL+F4mU\n\tmMu5zdpsrnN6tXMwo4GwbyVjuE3z0g1UdCqwJRs8a7esYEaqZKwc6gqTxxvgKcXtC3xQsT+HN1si4\n\tNJic3lcn7sWTz021eZL3ML2aV9hPeCCPYqBwC1NdRELl7LhHqdkOo4E74VskVfhBn7u7qJjmpcP/J\n\t5fG5C7u4OkbHoQVJCi85ToGKoiMMKAQckx1yTKTIOlelQpmftyFuzdJ1wf+cSoFVOFCi8Mq7elC5a\n\tyY1lbXNOlrD6pzPUSdbQ==;","Date":"Wed, 13 Sep 2017 10:29:11 -0700","From":"Catalin Marinas <catalin.marinas@arm.com>","To":"Dave Martin <Dave.Martin@arm.com>","Subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","Message-ID":"<20170913172911.3ca2h6cpju7etifi@localhost>","References":"<1504198860-12951-1-git-send-email-Dave.Martin@arm.com>\n\t<1504198860-12951-15-git-send-email-Dave.Martin@arm.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<1504198860-12951-15-git-send-email-Dave.Martin@arm.com>","User-Agent":"NeoMutt/20170113 (1.7.2)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170913_102935_507707_7C99624B ","X-CRM114-Status":"GOOD (  11.39  )","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\tRichard Sandiford <richard.sandiford@arm.com>, Yao Qi <Yao.Qi@arm.com>, \n\tAlan Hayward <alan.hayward@arm.com>, Will Deacon <will.deacon@arm.com>,\n\tgdb@sourceware.org, Alex =?iso-8859-1?q?Benn=E9e?=\n\t<alex.bennee@linaro.org>, kvmarm@lists.cs.columbia.edu,\n\tlinux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"iso-8859-1\"","Content-Transfer-Encoding":"quoted-printable","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"}},{"id":1768090,"web_url":"http://patchwork.ozlabs.org/comment/1768090/","msgid":"<20170913190611.GC23415@e103592.cambridge.arm.com>","list_archive_url":null,"date":"2017-09-13T19:06:12","subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","submitter":{"id":26612,"url":"http://patchwork.ozlabs.org/api/people/26612/","name":"Dave Martin","email":"Dave.Martin@arm.com"},"content":"On Wed, Sep 13, 2017 at 10:29:11AM -0700, Catalin Marinas wrote:\n> On Thu, Aug 31, 2017 at 06:00:46PM +0100, Dave P Martin wrote:\n> > This patch implements the core logic for changing a task's vector\n> > length on request from userspace.  This will be used by the ptrace\n> > and prctl frontends that are implemented in later patches.\n> > \n> > The SVE architecture permits, but does not require, implementations\n> > to support vector lengths that are not a power of two.  To handle\n> > this, logic is added to check a requested vector length against a\n> > possibly sparse bitmap of available vector lengths at runtime, so\n> > that the best supported value can be chosen.\n> > \n> > Signed-off-by: Dave Martin <Dave.Martin@arm.com>\n> > Cc: Alex Bennée <alex.bennee@linaro.org>\n> \n> Can this be merged with patch 20? It seems to add the PR_ definitions\n> which get actually used later when the prctl interface is added.\n\nThis patch is used both by patch 19 and by patch 20, which I preferred\nnot to merge with each other: ptrace and prctl are significantly\ndifferent things.\n\nThe prctl bit definitions are added here because they are the canonical\ndefinitions used by both interfaces.  The ptrace #defines are based on\nthem.\n\nDoes it make sense if I merge patch 20 into this one and apply patch 19\non top?  This avoide the appearance of prctl #defines with no prctl\nimplementation.\n\nCheers\n---Dave","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=\"VhgXgP/7\"; 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 3xsrkF3154z9sDB\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tThu, 14 Sep 2017 05:06:45 +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 1dsCzi-0006Ax-Qt; Wed, 13 Sep 2017 19:06:42 +0000","from usa-sjc-mx-foss1.foss.arm.com ([217.140.101.70]\n\thelo=foss.arm.com)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dsCzf-00067j-Uc for linux-arm-kernel@lists.infradead.org;\n\tWed, 13 Sep 2017 19:06:41 +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 D5CD91529;\n\tWed, 13 Sep 2017 12:06:16 -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 ESMTPSA id\n\t9AA173F483; Wed, 13 Sep 2017 12:06:14 -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:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=4qmSC4tJy49tYm4NBYHE+QlRY1OA4OHNnIVQL5D7+z4=;\n\tb=VhgXgP/77UEf4v\n\t4EmhsLdP647Oz03x6uC7mLuLs/I1eDt+mrrJDoUtBsXns9yxWtmT4h2xIibe908qj7wifq+WP46W9\n\tkian8eO8aG2S+xplT+jbK/ERjM56rHHMRb3UE3fQK0IfCr80Z7l24bzygT0lfv8WbrwZmMwcad5/O\n\tGNRMlvkQJ4p3SGHVQlndsCykXI1gXiuYfAoe0l3bL/RW7SSsIftkEgg9c9qauYQMr0p/ykhopXZ8Y\n\txJYi7hM2wRRyVCJxUdD+V9XuVMJR51yC8OaqtjwiZsYSYpwSSHhgC8lBv5dZF3re/RoLni1GMsp92\n\toQFLxuwxzJw6m9PGPqBg==;","Date":"Wed, 13 Sep 2017 20:06:12 +0100","From":"Dave Martin <Dave.Martin@arm.com>","To":"Catalin Marinas <catalin.marinas@arm.com>","Subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","Message-ID":"<20170913190611.GC23415@e103592.cambridge.arm.com>","References":"<1504198860-12951-1-git-send-email-Dave.Martin@arm.com>\n\t<1504198860-12951-15-git-send-email-Dave.Martin@arm.com>\n\t<20170913172911.3ca2h6cpju7etifi@localhost>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20170913172911.3ca2h6cpju7etifi@localhost>","User-Agent":"Mutt/1.5.23 (2014-03-12)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170913_120639_998199_A61C1F3E ","X-CRM114-Status":"GOOD (  17.57  )","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, gdb@sourceware.org,\n\tArd Biesheuvel <ard.biesheuvel@linaro.org>, Szabolcs Nagy\n\t<szabolcs.nagy@arm.com>, Richard Sandiford <richard.sandiford@arm.com>,\n\tYao Qi <Yao.Qi@arm.com>, Will Deacon <will.deacon@arm.com>,\n\tAlan Hayward <alan.hayward@arm.com>, Alex =?iso-8859-1?q?Benn=E9e?=\n\t<alex.bennee@linaro.org>, kvmarm@lists.cs.columbia.edu,\n\tlinux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"iso-8859-1\"","Content-Transfer-Encoding":"quoted-printable","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"}},{"id":1768221,"web_url":"http://patchwork.ozlabs.org/comment/1768221/","msgid":"<20170913221123.y4znytmxtplx24m4@localhost>","list_archive_url":null,"date":"2017-09-13T22:11:23","subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","submitter":{"id":938,"url":"http://patchwork.ozlabs.org/api/people/938/","name":"Catalin Marinas","email":"catalin.marinas@arm.com"},"content":"On Wed, Sep 13, 2017 at 08:06:12PM +0100, Dave P Martin wrote:\n> On Wed, Sep 13, 2017 at 10:29:11AM -0700, Catalin Marinas wrote:\n> > On Thu, Aug 31, 2017 at 06:00:46PM +0100, Dave P Martin wrote:\n> > > This patch implements the core logic for changing a task's vector\n> > > length on request from userspace.  This will be used by the ptrace\n> > > and prctl frontends that are implemented in later patches.\n> > > \n> > > The SVE architecture permits, but does not require, implementations\n> > > to support vector lengths that are not a power of two.  To handle\n> > > this, logic is added to check a requested vector length against a\n> > > possibly sparse bitmap of available vector lengths at runtime, so\n> > > that the best supported value can be chosen.\n> > > \n> > > Signed-off-by: Dave Martin <Dave.Martin@arm.com>\n> > > Cc: Alex Bennée <alex.bennee@linaro.org>\n> > \n> > Can this be merged with patch 20? It seems to add the PR_ definitions\n> > which get actually used later when the prctl interface is added.\n> \n> This patch is used both by patch 19 and by patch 20, which I preferred\n> not to merge with each other: ptrace and prctl are significantly\n> different things.\n> \n> The prctl bit definitions are added here because they are the canonical\n> definitions used by both interfaces.  The ptrace #defines are based on\n> them.\n> \n> Does it make sense if I merge patch 20 into this one and apply patch 19\n> on top?  This avoide the appearance of prctl #defines with no prctl\n> implementation.\n\nThat's fine, you can bring patch 20 forward. If there are other\nnon-trivial issues, feel free to ignore my comment.","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=\"LcyjGo8P\"; 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 3xswqs4xG7z9sRm\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tThu, 14 Sep 2017 08:11:53 +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 1dsFss-0005Ix-FG; Wed, 13 Sep 2017 22:11:50 +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 1dsFso-00058I-L9 for linux-arm-kernel@lists.infradead.org;\n\tWed, 13 Sep 2017 22:11:48 +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 9D18B1529;\n\tWed, 13 Sep 2017 15:11:25 -0700 (PDT)","from localhost (usa-sjc-mx-foss1.foss.arm.com [217.140.101.70])\n\tby usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id\n\t899243F578; Wed, 13 Sep 2017 15:11:25 -0700 (PDT)","from cmarinas by localhost with local (Exim 4.89)\n\t(envelope-from <catalin.marinas@arm.com>)\n\tid 1dsFsR-0002og-Dy; Wed, 13 Sep 2017 15:11:23 -0700"],"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:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=YBYprBFknXwQMycI1ZJnJLsPBs7tF6VRAl03Iy09Y9E=;\n\tb=LcyjGo8PbWpy8B\n\tZBTyDEWGevUE3Pcz314IVNpPlQXzhHeFurbcmyNa1X37mRUCSYD5/60DaZzU+nqU5idR2gqKwy+XP\n\tpSovrhGxp6o5x1YpyRtKs7inJrCcz0jHcrBNT9UcpUObNiKryqtJW27UvSqFw91GHTNEQ2m23sbyY\n\tWtNpUuPN+c8A18Am9XvVKT8e7J3QDS6ONp29phhNYOp9qwlghXNDNiJis5bF3pb53mBuiwZnI3GxP\n\tEWA7nf19h2Qyqmwy+WnyHXx34UFVFpoaX7n1p2BiRPd/9Uovik1Rs9LdKPv8sPMiph0JhxtK0XHBR\n\toh6NG+VVp7edIFLH4WGw==;","Date":"Wed, 13 Sep 2017 15:11:23 -0700","From":"Catalin Marinas <catalin.marinas@arm.com>","To":"Dave Martin <Dave.Martin@arm.com>","Subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","Message-ID":"<20170913221123.y4znytmxtplx24m4@localhost>","References":"<1504198860-12951-1-git-send-email-Dave.Martin@arm.com>\n\t<1504198860-12951-15-git-send-email-Dave.Martin@arm.com>\n\t<20170913172911.3ca2h6cpju7etifi@localhost>\n\t<20170913190611.GC23415@e103592.cambridge.arm.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20170913190611.GC23415@e103592.cambridge.arm.com>","User-Agent":"NeoMutt/20170113 (1.7.2)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170913_151146_711606_6D523B55 ","X-CRM114-Status":"GOOD (  20.09  )","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\tgdb@sourceware.org, Yao Qi <Yao.Qi@arm.com>,\n\tAlan Hayward <alan.hayward@arm.com>, Will Deacon <will.deacon@arm.com>, \n\tRichard Sandiford <richard.sandiford@arm.com>, Alex\n\t=?iso-8859-1?q?Benn=E9e?= <alex.bennee@linaro.org>,\n\tkvmarm@lists.cs.columbia.edu, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"iso-8859-1\"","Content-Transfer-Encoding":"quoted-printable","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"}},{"id":1771735,"web_url":"http://patchwork.ozlabs.org/comment/1771735/","msgid":"<D72E3AB8-6FBE-42B8-8F3D-B67B314B9542@arm.com>","list_archive_url":null,"date":"2017-09-20T10:57:10","subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","submitter":{"id":65004,"url":"http://patchwork.ozlabs.org/api/people/65004/","name":"Alan Hayward","email":"alan.hayward@arm.com"},"content":"> On 31 Aug 2017, at 18:00, Dave Martin <Dave.Martin@arm.com> wrote:\r\n\r\n>\r\n> +int sve_set_vector_length(struct task_struct *task,\r\n> +  unsigned long vl, unsigned long flags)\r\n> +{\r\n> +WARN_ON(task == current && preemptible());\r\n> +\r\n> +if (flags & ~(unsigned long)(PR_SVE_VL_INHERIT |\r\n> +     PR_SVE_SET_VL_ONEXEC))\r\n> +return -EINVAL;\r\n> +\r\n> +if (!sve_vl_valid(vl))\r\n> +return -EINVAL;\r\n> +\r\n> +/*\r\n> + * Clamp to the maximum vector length that VL-agnostic SVE code can\r\n> + * work with.  A flag may be assigned in the future to allow setting\r\n> + * of larger vector lengths without confusing older software.\r\n> + */\r\n> +if (vl > SVE_VL_ARCH_MAX)\r\n> +vl = SVE_VL_ARCH_MAX;\r\n> +\r\n> +vl = find_supported_vector_length(vl);\r\n> +\r\n\r\n\r\nGiven, sve_set_vector_length is called when setting the vector length in\r\nPTRACE_SETREGSET, it looks to me like if you set VL to a value that’s not\r\nsupported by the hardware, then it’s going to round down to the previous value.\r\nIs that correct? I’m not sure if that’s explained in the docs?\r\n\r\nWhat happens if you give a vl value lower than the min supported value in the\r\nhardware?\r\n\r\n\r\n> +/*\r\n> + * All vector length selection from userspace comes through here.\r\n> + * We're on a slow path, so some sanity-checks are included.\r\n> + * If things go wrong there's a bug somewhere, but try to fall back to a\r\n> + * safe choice.\r\n> + */\r\n> +static unsigned int find_supported_vector_length(unsigned int vl)\r\n> +{\r\n> +int bit;\r\n> +int max_vl = sve_max_vl;\r\n> +\r\n> +if (WARN_ON(!sve_vl_valid(vl)))\r\n> +vl = SVE_VL_MIN;\r\n> +\r\n> +if (WARN_ON(!sve_vl_valid(max_vl)))\r\n> +max_vl = SVE_VL_MIN;\r\n> +\r\n> +if (vl > max_vl)\r\n> +vl = max_vl;\r\n> +\r\n> +bit = find_next_bit(sve_vq_map, SVE_VQ_MAX,\r\n> +    vq_to_bit(sve_vq_from_vl(vl)));\r\n> +return sve_vl_from_vq(bit_to_vq(bit));\r\n> +}\r\n> +\r\n\r\n\r\nThanks,\r\nAlan.\r\nIMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.","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 header.b=\"IHERpATu\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=armh.onmicrosoft.com\n\theader.i=@armh.onmicrosoft.com\n\theader.b=\"r0N8mxq3\"; dkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=Alan.Hayward@arm.com; "],"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 3xxxXm15qRz9s7c\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tWed, 20 Sep 2017 20:57:44 +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 1duchI-0005w7-DR; Wed, 20 Sep 2017 10:57:40 +0000","from mail-db5eur01on0043.outbound.protection.outlook.com\n\t([104.47.2.43] helo=EUR01-DB5-obe.outbound.protection.outlook.com)\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1duchB-0005oq-Mo for linux-arm-kernel@lists.infradead.org;\n\tWed, 20 Sep 2017 10:57:37 +0000","from AM3PR08MB0101.eurprd08.prod.outlook.com (10.160.211.19) by\n\tAM3PR08MB0166.eurprd08.prod.outlook.com (10.161.35.142) with\n\tMicrosoft SMTP Server (version=TLS1_2,\n\tcipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id\n\t15.20.56.11; Wed, 20 Sep 2017 10:57:10 +0000","from AM3PR08MB0101.eurprd08.prod.outlook.com\n\t([fe80::916d:2b75:34f2:da29]) by\n\tAM3PR08MB0101.eurprd08.prod.outlook.com\n\t([fe80::916d:2b75:34f2:da29%17]) with mapi id 15.20.0056.018;\n\tWed, 20 Sep 2017 10:57:10 +0000"],"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:Content-ID:In-Reply-To:\n\tReferences:Message-ID:Date:Subject:To:From:Reply-To:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=lLYd+qpTXoARqifaN0CcmegOH25OewFTG0vfjbUg4oQ=;\n\tb=IHERpATu0GrCMz\n\tqPQmy930dY2Tkft3L34cKIGJePpvVoivYA54NzDogO6CXhbKM+r4Y80uZf9c+uekpZEKDkCSPOQ12\n\te1ZtWNo9EqkHpL4mMUkpuZWlE+f3CsEFMUF5VHAfty/RGLrW6j40W6393iIpezFyZCjv2jjQV+5e1\n\tW+EGRH5msU180o66GL9AOaySUOOEIc8Ym/4CFP/anmcSwVzVJfH2mSgMo7ixCSlG20eVsqgXzD87s\n\tydcEWlxHizh2VjVk29ByRfLoTmH7t70b74Vabga99eyXGmZmsvJOwZs20F4UTTTnJAU/BkKb4QfW1\n\t60uBt9jxODeoU+GeIWHg==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=armh.onmicrosoft.com;\n\ts=selector1-arm-com;\n\th=From:Date:Subject:Message-ID:Content-Type:MIME-Version;\n\tbh=IJX1yBGGcp44iLKOXGZ/BTuTwU1ozbLiSRCabpdD7PI=;\n\tb=r0N8mxq38OO/2K7PGNoJaaHCJwnXCrJrW5pSfDqs2jmJk1sL8OvZf30dTLkNKWDQUXsrwMlnlmEJMPZDRoCtv0HQCt4WcCEh/Woaq+7Nut1farAvFEVFvwFFi+s7SpbM5Po2Mnrs8+qnvnVXeHruJptqcce66IcdxL7pCEsX9pc="],"From":"Alan Hayward <Alan.Hayward@arm.com>","To":"Dave P Martin <Dave.Martin@arm.com>","Subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","Thread-Topic":"[PATCH v2 14/28] arm64/sve: Backend logic for setting the\n\tvector length","Thread-Index":"AQHTInrT0RE6i7ldhE+RC9J8O4DYcaK9uXOA","Date":"Wed, 20 Sep 2017 10:57:10 +0000","Message-ID":"<D72E3AB8-6FBE-42B8-8F3D-B67B314B9542@arm.com>","References":"<1504198860-12951-1-git-send-email-Dave.Martin@arm.com>\n\t<1504198860-12951-15-git-send-email-Dave.Martin@arm.com>","In-Reply-To":"<1504198860-12951-15-git-send-email-Dave.Martin@arm.com>","Accept-Language":"en-GB, en-US","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","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 header.b=\"IHERpATu\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=armh.onmicrosoft.com\n\theader.i=@armh.onmicrosoft.com\n\theader.b=\"r0N8mxq3\"; dkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=Alan.Hayward@arm.com; "],"x-originating-ip":"[217.140.96.140]","x-ms-publictraffictype":"Email","x-microsoft-exchange-diagnostics":"1; AM3PR08MB0166;\n\t6:e+vyMbqamopNj/8TY3XIbWlSvm+SXMZS82z7UTVJBBeWBfmWubjY3IJfvKO3MFMQcp1V6nRSfjp9RYD+xw5L5KQihBJwl64HY1zwMdevz/m8iYSgwBewkKRnlOjcwng7NPUQydLnvGgcorUSW48ACeXB1wIjwOHCBQxX05puHCG9rxZs+GG+7UQIDMoUDOkPoYR+PSyEz7/WmFbkuM+Y8TYrNS1+lVBs1M6cgx11XLr70dZXcm4ckZXcmn/9cQWq037PlJaVjUYeCcug9eGkhCSwnWzIf2uxOIxjSNUTeKnqwY0k3neKrdw8LBJgUpuBoa0tOk6K6NL1lXgPUkIveg==;\n\t5:2vhsuQJgANOLlNQ7j5PU2hGjVTRQfoW85eovJqaNGL8b9hky+OpG9FLf+Q2X+8+JAei25qT4QzGQg9nJOXIQhpXKvJtjlc+QJHeq3cIsnbQkLToKRDZzeOobVqI5ZJJEp5IZUr52gAwbEIYjkcH0yg==;\n\t24:gP1ahPA4QSZH8FGREXkJhaEGiLdNvEXQmL8wiXGgFG/iXcNBGrvUVkzr0QKkijcRgj8Zo9+qnX9gl6HaUDZ57WlRFfV4GhVqsBqRYZ42uqc=;\n\t7:xIY9qmzOjLjIlq4e1I6E4DHFS4vKbxPla/KRqTL18jfHASfw2LfTOYhvEMH2Eb2aopAd80Cu4j9Jt9nJDMpVzuBySmpb26IQHWY1uR7QlUnJcMsmUnFYHtYjxztv9RpK7IIjhTOCw1t/Gs8h/8mGKBI/wTVzMefQ40J69gmCAcAifq941lI9Jw+7eC3KITL/Yol/JVbIZw27huNeBDymoLwMxfM9PFxOvEDdqQQHeSU=","x-ms-exchange-antispam-srfa-diagnostics":"SSOS;","x-ms-office365-filtering-correlation-id":"b01cdd91-bdf7-489c-1782-08d5001657fa","x-ms-office365-filtering-ht":"Tenant","x-microsoft-antispam":"UriScan:; BCL:0; PCL:0;\n\tRULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(2017030254152)(48565401081)(300000503095)(300135400095)(2017052603199)(201703131423075)(201703031133081)(201702281549075)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095);\n\tSRVR:AM3PR08MB0166; ","x-ms-traffictypediagnostic":"AM3PR08MB0166:","x-exchange-antispam-report-test":"UriScan:(180628864354917);","x-microsoft-antispam-prvs":"<AM3PR08MB0166EB029E6FFEBED23F29C697610@AM3PR08MB0166.eurprd08.prod.outlook.com>","x-exchange-antispam-report-cfa-test":"BCL:0; PCL:0;\n\tRULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(8121501046)(5005006)(10201501046)(3002001)(100000703101)(100105400095)(93006095)(93001095)(6055026)(6041248)(20161123555025)(20161123560025)(20161123558100)(20161123564025)(20161123562025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095);\n\tSRVR:AM3PR08MB0166; BCL:0; PCL:0;\n\tRULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095);\n\tSRVR:AM3PR08MB0166; ","x-forefront-prvs":"04362AC73B","x-forefront-antispam-report":"SFV:NSPM;\n\tSFS:(10009020)(6009001)(39860400002)(346002)(376002)(189002)(199003)(40434004)(24454002)(229853002)(6436002)(54906003)(6506006)(82746002)(36756003)(316002)(8676002)(81156014)(81166006)(37006003)(53546010)(7736002)(5250100002)(305945005)(6486002)(5890100001)(101416001)(6862004)(5660300001)(68736007)(189998001)(2906002)(4326008)(86362001)(3280700002)(3660700001)(8936002)(50986999)(76176999)(54356999)(478600001)(2900100001)(97736004)(72206003)(66066001)(6246003)(25786009)(14454004)(83716003)(6512007)(105586002)(6116002)(106356001)(2950100002)(6636002)(53936002)(102836003)(3846002)(33656002)(99286003);\n\tDIR:OUT; SFP:1101; SCL:1; SRVR:AM3PR08MB0166;\n\tH:AM3PR08MB0101.eurprd08.prod.outlook.com; FPR:; SPF:None;\n\tPTR:InfoNoRecords; MX:1; A:1; LANG:en; ","received-spf":"None (protection.outlook.com: arm.com does not designate\n\tpermitted sender hosts)","spamdiagnosticoutput":"1:99","spamdiagnosticmetadata":"NSPM","Content-ID":"<251BF33EED722C488B29338E21C2A12E@eurprd08.prod.outlook.com>","MIME-Version":"1.0","X-OriginatorOrg":"arm.com","X-MS-Exchange-CrossTenant-originalarrivaltime":"20 Sep 2017 10:57:10.7144\n\t(UTC)","X-MS-Exchange-CrossTenant-fromentityheader":"Hosted","X-MS-Exchange-CrossTenant-id":"f34e5979-57d9-4aaa-ad4d-b122a662184d","X-MS-Exchange-Transport-CrossTenantHeadersStamped":"AM3PR08MB0166","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170920_035734_196556_EBF30493 ","X-CRM114-Status":"GOOD (  15.68  )","X-Spam-Score":"-2.9 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-2.9 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno trust [104.47.2.43 listed in list.dnswl.org]\n\t-1.0 RCVD_IN_MSPIKE_H5      RBL: Excellent reputation (+5)\n\t[104.47.2.43 listed in wl.mailspike.net]\n\t-0.0 SPF_PASS               SPF: sender matches SPF record\n\t-0.0 SPF_HELO_PASS          SPF: HELO matches SPF record\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED            Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.0 RCVD_IN_MSPIKE_WL      Mailspike good senders","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\" <linux-arch@vger.kernel.org>,\n\t\"libc-alpha@sourceware.org\" <libc-alpha@sourceware.org>, \n\t\"gdb@sourceware.org\" <gdb@sourceware.org>, Ard Biesheuvel\n\t<ard.biesheuvel@linaro.org>,  Szabolcs Nagy <Szabolcs.Nagy@arm.com>,\n\tCatalin Marinas <Catalin.Marinas@arm.com>, Yao Qi <Yao.Qi@arm.com>,\n\tWill Deacon <Will.Deacon@arm.com>, Richard Sandiford\n\t<Richard.Sandiford@arm.com>, =?utf-8?q?Alex_Benn=C3=A9e?=\n\t<alex.bennee@linaro.org>, \"kvmarm@lists.cs.columbia.edu\"\n\t<kvmarm@lists.cs.columbia.edu>, \"linux-arm-kernel@lists.infradead.org\"\n\t<linux-arm-kernel@lists.infradead.org>","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"}},{"id":1771738,"web_url":"http://patchwork.ozlabs.org/comment/1771738/","msgid":"<FC5BA359-D37C-493C-87CD-146B83D3CCB5@arm.com>","list_archive_url":null,"date":"2017-09-20T10:59:55","subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","submitter":{"id":65004,"url":"http://patchwork.ozlabs.org/api/people/65004/","name":"Alan Hayward","email":"alan.hayward@arm.com"},"content":"(Resending without disclaimer)\r\n\r\n> On 31 Aug 2017, at 18:00, Dave Martin <Dave.Martin@arm.com> wrote:\r\n\r\n> \r\n> +int sve_set_vector_length(struct task_struct *task,\r\n> +\t\t\t  unsigned long vl, unsigned long flags)\r\n> +{\r\n> +\tWARN_ON(task == current && preemptible());\r\n> +\r\n> +\tif (flags & ~(unsigned long)(PR_SVE_VL_INHERIT |\r\n> +\t\t\t\t     PR_SVE_SET_VL_ONEXEC))\r\n> +\t\treturn -EINVAL;\r\n> +\r\n> +\tif (!sve_vl_valid(vl))\r\n> +\t\treturn -EINVAL;\r\n> +\r\n> +\t/*\r\n> +\t * Clamp to the maximum vector length that VL-agnostic SVE code can\r\n> +\t * work with.  A flag may be assigned in the future to allow setting\r\n> +\t * of larger vector lengths without confusing older software.\r\n> +\t */\r\n> +\tif (vl > SVE_VL_ARCH_MAX)\r\n> +\t\tvl = SVE_VL_ARCH_MAX;\r\n> +\r\n> +\tvl = find_supported_vector_length(vl);\r\n> +\r\n\r\n\r\nGiven, sve_set_vector_length is called when setting the vector length in\r\nPTRACE_SETREGSET, it looks to me like if you set VL to a value that’s not\r\nsupported by the hardware, then it’s going to round down to the previous value.\r\nIs that correct? I’m not sure if that’s explained in the docs?\r\n\r\nWhat happens if you give a vl value lower than the min supported value in the\r\nhardware?\r\n\r\n\r\n> +/*\r\n> + * All vector length selection from userspace comes through here.\r\n> + * We're on a slow path, so some sanity-checks are included.\r\n> + * If things go wrong there's a bug somewhere, but try to fall back to a\r\n> + * safe choice.\r\n> + */\r\n> +static unsigned int find_supported_vector_length(unsigned int vl)\r\n> +{\r\n> +\tint bit;\r\n> +\tint max_vl = sve_max_vl;\r\n> +\r\n> +\tif (WARN_ON(!sve_vl_valid(vl)))\r\n> +\t\tvl = SVE_VL_MIN;\r\n> +\r\n> +\tif (WARN_ON(!sve_vl_valid(max_vl)))\r\n> +\t\tmax_vl = SVE_VL_MIN;\r\n> +\r\n> +\tif (vl > max_vl)\r\n> +\t\tvl = max_vl;\r\n> +\r\n> +\tbit = find_next_bit(sve_vq_map, SVE_VQ_MAX,\r\n> +\t\t\t    vq_to_bit(sve_vq_from_vl(vl)));\r\n> +\treturn sve_vl_from_vq(bit_to_vq(bit));\r\n> +}\r\n> +\r\n\r\n\r\nThanks,\r\nAlan.","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 header.b=\"r9Y3/4m8\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=armh.onmicrosoft.com\n\theader.i=@armh.onmicrosoft.com\n\theader.b=\"Kj22nC3x\"; dkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=Alan.Hayward@arm.com; "],"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 3xxxcV6gsbz9s7c\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tWed, 20 Sep 2017 21:00:58 +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 1duckQ-0000Er-MF; Wed, 20 Sep 2017 11:00:54 +0000","from mail-eopbgr50062.outbound.protection.outlook.com\n\t([40.107.5.62] helo=EUR03-VE1-obe.outbound.protection.outlook.com)\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1ducjq-00072Y-Nv for linux-arm-kernel@lists.infradead.org;\n\tWed, 20 Sep 2017 11:00:26 +0000","from AM3PR08MB0101.eurprd08.prod.outlook.com (10.160.211.19) by\n\tAM3PR08MB0166.eurprd08.prod.outlook.com (10.161.35.142) with\n\tMicrosoft SMTP Server (version=TLS1_2,\n\tcipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id\n\t15.20.56.11; Wed, 20 Sep 2017 10:59:55 +0000","from AM3PR08MB0101.eurprd08.prod.outlook.com\n\t([fe80::916d:2b75:34f2:da29]) by\n\tAM3PR08MB0101.eurprd08.prod.outlook.com\n\t([fe80::916d:2b75:34f2:da29%17]) with mapi id 15.20.0056.018;\n\tWed, 20 Sep 2017 10:59:55 +0000"],"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:Content-ID:In-Reply-To:\n\tReferences:Message-ID:Date:Subject:To:From:Reply-To:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=n88ECYMfHcb9kxhY3ID4j5+HZqb74ium2vhMgT+SRe4=;\n\tb=r9Y3/4m8ELYzBa\n\tgtKGaAZ50++rEhNdFCh4AVRi8WLJ6GT6FJepSRMEfU6meRSX9sIm6z7RIP3vGe1n/kpRUxdxZxkjv\n\tsyMabj2jROc/Vob79NwHARBRNNAIe+CTsDrjMhekvxo/2C28a73LT3LOgpf4ghVhWJIrrDGCupqPV\n\t3HjpMCV7tPXN//8oNvNvcAyDE4UyQBZdyH3bvCuEUZytDw1YhXGBFgae+J6QYE+x7/+I8sCx1wm72\n\tk974qO4dJQCOMqOWFibqzEnR1Sb6FVb2Vo3I3PM5RE/WT9DWAavF+MuYoQnRie2yb+uPmQgfuofu2\n\tYajlv0LU4OvrQ6eSGSvQ==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=armh.onmicrosoft.com;\n\ts=selector1-arm-com;\n\th=From:Date:Subject:Message-ID:Content-Type:MIME-Version;\n\tbh=Zu1E9qpm3BeX4SlQlJ6ZM2KzpiFe8ot1pDoeErskRZs=;\n\tb=Kj22nC3xM3EkhZQJasKhRj8RXBEQajq8+4SMe+LmVu04oBnQL4yelCFBTqF4U1EeHQvUP1NsrGgHqXWYx1zo9FmVrzactkkUkuVKLFHqS2TJggRkod70b0PKHY2JZNIaBgf5lPddd9yLoYabdg5v3xwfOSTTU+w6A23Z5gPj0sM="],"From":"Alan Hayward <Alan.Hayward@arm.com>","To":"Dave P Martin <Dave.Martin@arm.com>","Subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","Thread-Topic":"[PATCH v2 14/28] arm64/sve: Backend logic for setting the\n\tvector length","Thread-Index":"AQHTInrT0RE6i7ldhE+RC9J8O4DYcaK9ujmA","Date":"Wed, 20 Sep 2017 10:59:55 +0000","Message-ID":"<FC5BA359-D37C-493C-87CD-146B83D3CCB5@arm.com>","References":"<1504198860-12951-1-git-send-email-Dave.Martin@arm.com>\n\t<1504198860-12951-15-git-send-email-Dave.Martin@arm.com>","In-Reply-To":"<1504198860-12951-15-git-send-email-Dave.Martin@arm.com>","Accept-Language":"en-GB, en-US","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","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 header.b=\"r9Y3/4m8\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=armh.onmicrosoft.com\n\theader.i=@armh.onmicrosoft.com\n\theader.b=\"Kj22nC3x\"; dkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=Alan.Hayward@arm.com; "],"x-originating-ip":"[217.140.96.140]","x-ms-publictraffictype":"Email","x-microsoft-exchange-diagnostics":"1; AM3PR08MB0166;\n\t6:xcNdcj4GfeGn4lp+/qz81028oMDHHJ1BSM4PiUe/6uraIhRvAnC7KYbP6qSClIr5u/NGvQNlbseKsSurQNLVD7OdQnq9Znw+IRFJXR8QXERHUhzuvLVCIRBHIhxKGM/R4CIQ2JKE7S13qx99iMPLmj20wiHG3UF9y8eoThwwOPqPrAF9bF4vchTewFRJoIVEoaCcsQujKXxjgH7r8CsapsaFjrwu0UpzqNjKaTlCWh8JA0iXx5Wx+mwYPec+EKpjD1hN5DXRNXydK4bJ3NMugbyLtCLYhqnpMeYzKzrX4dCZ49iJ3JAoaQLPPcztcduPcsJNBcDAD8XX8mq9tFGOlw==;\n\t5:zs4z02AvNY8cGWwPselDc/qpaHLDvdCc7KNAAjfNpAn9W7QdKQismnYM7H4LhLIypdBgNHjaRWnH12pPw3/lvgG9gYii+MJFynUPbflrbLFm6rhuYJeEH7L7l9KEGvkb519ayYCaep54SMaHemFyJw==;\n\t24:63MLlhR+BNArWjYPLaO4vR/Om8UGCqDB/sC1MM7cNvMiXbsQJPeTY0yp6283qOHqv/I9OUOn3X5DKcGIC3ZM+sGtft37M3y23oHkyZwhsEE=;\n\t7:zJrmOAWOyPp9IfRdKsiY7zIQ9KqcoIwOjt/VdP5dCFult8QQprhE9F/SGCUZV7ie4Py02S0bDlmR2G26lERX8PbvH4NztkrBH2pKxqaaxHg3TsvT1K8yzcH8POBqlDqn5HudTv4iF/SLVM1GGM/fmgLwAOE9Z4xroWtcLPoAha16PKmxqMA9tJTuWsReLZOFsRMKy4hoTL517+OsB3imp9uHY8lKS7BJHu7ygQf1G3U=","x-ms-exchange-antispam-srfa-diagnostics":"SSOS;","x-ms-office365-filtering-correlation-id":"a9531a05-b192-4357-f491-08d50016ba63","x-ms-office365-filtering-ht":"Tenant","x-microsoft-antispam":"UriScan:; BCL:0; PCL:0;\n\tRULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(2017030254152)(48565401081)(300000503095)(300135400095)(2017052603199)(201703131423075)(201703031133081)(201702281549075)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095);\n\tSRVR:AM3PR08MB0166; ","x-ms-traffictypediagnostic":"AM3PR08MB0166:","nodisclaimer":"True","x-exchange-antispam-report-test":"UriScan:(180628864354917);","x-microsoft-antispam-prvs":"<AM3PR08MB016673535C86D45B3FAB176B97610@AM3PR08MB0166.eurprd08.prod.outlook.com>","x-exchange-antispam-report-cfa-test":"BCL:0; PCL:0;\n\tRULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(8121501046)(5005006)(10201501046)(3002001)(100000703101)(100105400095)(93006095)(93001095)(6055026)(6041248)(20161123555025)(20161123560025)(20161123558100)(20161123564025)(20161123562025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095);\n\tSRVR:AM3PR08MB0166; BCL:0; PCL:0;\n\tRULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095);\n\tSRVR:AM3PR08MB0166; ","x-forefront-prvs":"04362AC73B","x-forefront-antispam-report":"SFV:NSPM;\n\tSFS:(10009020)(6009001)(39860400002)(346002)(376002)(189002)(199003)(24454002)(229853002)(6436002)(54906003)(6506006)(82746002)(36756003)(316002)(8676002)(81156014)(81166006)(37006003)(53546010)(7736002)(5250100002)(305945005)(6486002)(101416001)(6862004)(5660300001)(68736007)(189998001)(2906002)(4326008)(86362001)(3280700002)(3660700001)(8936002)(50986999)(76176999)(54356999)(478600001)(2900100001)(97736004)(72206003)(66066001)(6246003)(25786009)(14454004)(83716003)(6512007)(105586002)(6116002)(106356001)(2950100002)(6636002)(53936002)(102836003)(3846002)(33656002)(99286003);\n\tDIR:OUT; SFP:1101; SCL:1; SRVR:AM3PR08MB0166;\n\tH:AM3PR08MB0101.eurprd08.prod.outlook.com; FPR:; SPF:None;\n\tPTR:InfoNoRecords; MX:1; A:1; LANG:en; ","received-spf":"None (protection.outlook.com: arm.com does not designate\n\tpermitted sender hosts)","spamdiagnosticoutput":"1:99","spamdiagnosticmetadata":"NSPM","Content-ID":"<6B3111D0F66877459DFF3822341B37B7@eurprd08.prod.outlook.com>","MIME-Version":"1.0","X-OriginatorOrg":"arm.com","X-MS-Exchange-CrossTenant-originalarrivaltime":"20 Sep 2017 10:59:55.8092\n\t(UTC)","X-MS-Exchange-CrossTenant-fromentityheader":"Hosted","X-MS-Exchange-CrossTenant-id":"f34e5979-57d9-4aaa-ad4d-b122a662184d","X-MS-Exchange-Transport-CrossTenantHeadersStamped":"AM3PR08MB0166","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170920_040019_460109_F98B08DA ","X-CRM114-Status":"GOOD (  14.53  )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-1.9 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno trust [40.107.5.62 listed in list.dnswl.org]\n\t-0.0 RCVD_IN_MSPIKE_H3      RBL: Good reputation (+3)\n\t[40.107.5.62 listed in wl.mailspike.net]\n\t-0.0 SPF_PASS               SPF: sender matches SPF record\n\t-0.0 SPF_HELO_PASS          SPF: HELO matches SPF record\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED            Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.0 RCVD_IN_MSPIKE_WL      Mailspike good senders","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\" <linux-arch@vger.kernel.org>,\n\t\"libc-alpha@sourceware.org\" <libc-alpha@sourceware.org>, \n\t\"gdb@sourceware.org\" <gdb@sourceware.org>, Ard Biesheuvel\n\t<ard.biesheuvel@linaro.org>,  Szabolcs Nagy <Szabolcs.Nagy@arm.com>,\n\tCatalin Marinas <Catalin.Marinas@arm.com>, Yao Qi <Yao.Qi@arm.com>,\n\tWill Deacon <Will.Deacon@arm.com>, Richard Sandiford\n\t<Richard.Sandiford@arm.com>, nd <nd@arm.com>, =?utf-8?q?Alex_Benn?=\n\t=?utf-8?b?w6ll?= <alex.bennee@linaro.org>, \"kvmarm@lists.cs.columbia.edu\"\n\t<kvmarm@lists.cs.columbia.edu>, \"linux-arm-kernel@lists.infradead.org\"\n\t<linux-arm-kernel@lists.infradead.org>","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"}},{"id":1771742,"web_url":"http://patchwork.ozlabs.org/comment/1771742/","msgid":"<20170920110902.GG24231@e103592.cambridge.arm.com>","list_archive_url":null,"date":"2017-09-20T11:09:04","subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","submitter":{"id":72402,"url":"http://patchwork.ozlabs.org/api/people/72402/","name":"Dave Martin","email":"dave.martin@foss.arm.com"},"content":"On Wed, Sep 20, 2017 at 10:59:55AM +0000, Alan Hayward wrote:\n> (Resending without disclaimer)\n> \n> > On 31 Aug 2017, at 18:00, Dave Martin <Dave.Martin@arm.com> wrote:\n> \n> > \n> > +int sve_set_vector_length(struct task_struct *task,\n> > +\t\t\t  unsigned long vl, unsigned long flags)\n> > +{\n> > +\tWARN_ON(task == current && preemptible());\n> > +\n> > +\tif (flags & ~(unsigned long)(PR_SVE_VL_INHERIT |\n> > +\t\t\t\t     PR_SVE_SET_VL_ONEXEC))\n> > +\t\treturn -EINVAL;\n> > +\n> > +\tif (!sve_vl_valid(vl))\n> > +\t\treturn -EINVAL;\n> > +\n> > +\t/*\n> > +\t * Clamp to the maximum vector length that VL-agnostic SVE code can\n> > +\t * work with.  A flag may be assigned in the future to allow setting\n> > +\t * of larger vector lengths without confusing older software.\n> > +\t */\n> > +\tif (vl > SVE_VL_ARCH_MAX)\n> > +\t\tvl = SVE_VL_ARCH_MAX;\n> > +\n> > +\tvl = find_supported_vector_length(vl);\n> > +\n> \n> \n> Given, sve_set_vector_length is called when setting the vector length in\n> PTRACE_SETREGSET, it looks to me like if you set VL to a value that’s not\n> supported by the hardware, then it’s going to round down to the previous value.\n> Is that correct? I’m not sure if that’s explained in the docs?\n\nDoes this cover it?\n\n\"On success, the calling thread's vector length is changed to the\nlargest value supported by the system that is less than or equal to vl.\"\n\n(For ptrace, I just cross-reference the PR_SVE_SET_VL behaviour, above.)\n\n> What happens if you give a vl value lower than the min supported value in the\n> hardware?\n\nThis is impossible, unless vl < SVE_VL_MIN (which is rejected explicitly\nby the !sve_vl_valid() check in sve_set_vector_length()).\n\nThe architecture required support for all power-of-two vector lengths\nless than the maximum supported vector length, so by construction\nSVE_VL_MIN is supported by all hardware.\n\nTo be defensive, if we fail to detect support for SVE_VL_MIN, I set the\ncorresponding bit in sve_vq_map and WARN.  This is just to help ensure\nfind_supported_vector_length doesn't fall off the end of sve_vq_map.\n\n\nDoes that sounds correct?  There may be a clearer way of achieving this.\n\nCheers\n---Dave\n\n> \n> \n> > +/*\n> > + * All vector length selection from userspace comes through here.\n> > + * We're on a slow path, so some sanity-checks are included.\n> > + * If things go wrong there's a bug somewhere, but try to fall back to a\n> > + * safe choice.\n> > + */\n> > +static unsigned int find_supported_vector_length(unsigned int vl)\n> > +{\n> > +\tint bit;\n> > +\tint max_vl = sve_max_vl;\n> > +\n> > +\tif (WARN_ON(!sve_vl_valid(vl)))\n> > +\t\tvl = SVE_VL_MIN;\n> > +\n> > +\tif (WARN_ON(!sve_vl_valid(max_vl)))\n> > +\t\tmax_vl = SVE_VL_MIN;\n> > +\n> > +\tif (vl > max_vl)\n> > +\t\tvl = max_vl;\n> > +\n> > +\tbit = find_next_bit(sve_vq_map, SVE_VQ_MAX,\n> > +\t\t\t    vq_to_bit(sve_vq_from_vl(vl)));\n> > +\treturn sve_vl_from_vq(bit_to_vq(bit));\n> > +}\n> > +\n> \n> \n> Thanks,\n> Alan.\n> _______________________________________________\n> linux-arm-kernel mailing list\n> linux-arm-kernel@lists.infradead.org\n> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel","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=\"jXf8emKx\"; 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 3xxxpS3ZJLz9s7f\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tWed, 20 Sep 2017 21:09:36 +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 1ducsn-0004zE-7n; Wed, 20 Sep 2017 11:09:33 +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 1ducsk-0004oj-4i for linux-arm-kernel@lists.infradead.org;\n\tWed, 20 Sep 2017 11:09:32 +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 72B0780D;\n\tWed, 20 Sep 2017 04:09:09 -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 ESMTPSA id\n\tD64C83F578; Wed, 20 Sep 2017 04:09:06 -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:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=4Yd094Nz131Q00s4VNrNWtj3+o3k8OeESCczF5u1JfU=;\n\tb=jXf8emKx7Vkbt+\n\tgaJdwa9ksLpetQPvaKdkhXH3Wzx5mKWGrXqeIL2bUiBD44GeiGssHbXa2p6VCtVHH0cr+Gr6k9ZNv\n\tmBtOjBXGXkouXjgV0Z2tpYYSkuXsNm2UmDDC5HRx+SQPB81dGxkC8/c953J91vr0EwxeqKD3SRP1l\n\twzUo7hmr0i0ZCGrw8k03UogTzRMWWcDPxdk3BJt6HKxmtPAnhoiKeKwHhXjBp/f5vIatRmCmwfel6\n\tHhKqTF4XlZOOLaGmyjQIEvXeFowDMaeIEfGTyRwB43LkOplM9prJVw36BTerHz2Hi6z2Ie02Akd6p\n\tF3EOCm8ZtTV+PZXjDm/g==;","Date":"Wed, 20 Sep 2017 12:09:04 +0100","From":"Dave Martin <dave.martin@foss.arm.com>","To":"Alan Hayward <Alan.Hayward@arm.com>","Subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","Message-ID":"<20170920110902.GG24231@e103592.cambridge.arm.com>","References":"<1504198860-12951-1-git-send-email-Dave.Martin@arm.com>\n\t<1504198860-12951-15-git-send-email-Dave.Martin@arm.com>\n\t<FC5BA359-D37C-493C-87CD-146B83D3CCB5@arm.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<FC5BA359-D37C-493C-87CD-146B83D3CCB5@arm.com>","User-Agent":"Mutt/1.5.23 (2014-03-12)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170920_040930_225255_B8EBDDF8 ","X-CRM114-Status":"GOOD (  20.42  )","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\" <linux-arch@vger.kernel.org>,\n\t\"libc-alpha@sourceware.org\" <libc-alpha@sourceware.org>, Ard Biesheuvel\n\t<ard.biesheuvel@linaro.org>,  Szabolcs Nagy <Szabolcs.Nagy@arm.com>,\n\t\"gdb@sourceware.org\" <gdb@sourceware.org>, Yao Qi <Yao.Qi@arm.com>, \n\tWill Deacon <Will.Deacon@arm.com>, \"kvmarm@lists.cs.columbia.edu\"\n\t<kvmarm@lists.cs.columbia.edu>, Richard Sandiford\n\t<Richard.Sandiford@arm.com>, Catalin Marinas <Catalin.Marinas@arm.com>,\n\tnd <nd@arm.com>, Alex =?iso-8859-1?q?Benn=E9e?= <alex.bennee@linaro.org>,\n\tDave P Martin <Dave.Martin@arm.com>,\n\t\"linux-arm-kernel@lists.infradead.org\"\n\t<linux-arm-kernel@lists.infradead.org>","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"}},{"id":1772134,"web_url":"http://patchwork.ozlabs.org/comment/1772134/","msgid":"<E82895B2-3F95-4A54-8703-D654221F8CBA@arm.com>","list_archive_url":null,"date":"2017-09-20T18:08:21","subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","submitter":{"id":65004,"url":"http://patchwork.ozlabs.org/api/people/65004/","name":"Alan Hayward","email":"alan.hayward@arm.com"},"content":"> On 20 Sep 2017, at 12:09, Dave Martin <dave.martin@foss.arm.com> wrote:\r\n> \r\n> On Wed, Sep 20, 2017 at 10:59:55AM +0000, Alan Hayward wrote:\r\n>> (Resending without disclaimer)\r\n>> \r\n>>> On 31 Aug 2017, at 18:00, Dave Martin <Dave.Martin@arm.com> wrote:\r\n>> \r\n>>> \r\n>>> +int sve_set_vector_length(struct task_struct *task,\r\n>>> +\t\t\t  unsigned long vl, unsigned long flags)\r\n>>> +{\r\n>>> +\tWARN_ON(task == current && preemptible());\r\n>>> +\r\n>>> +\tif (flags & ~(unsigned long)(PR_SVE_VL_INHERIT |\r\n>>> +\t\t\t\t     PR_SVE_SET_VL_ONEXEC))\r\n>>> +\t\treturn -EINVAL;\r\n>>> +\r\n>>> +\tif (!sve_vl_valid(vl))\r\n>>> +\t\treturn -EINVAL;\r\n>>> +\r\n>>> +\t/*\r\n>>> +\t * Clamp to the maximum vector length that VL-agnostic SVE code can\r\n>>> +\t * work with.  A flag may be assigned in the future to allow setting\r\n>>> +\t * of larger vector lengths without confusing older software.\r\n>>> +\t */\r\n>>> +\tif (vl > SVE_VL_ARCH_MAX)\r\n>>> +\t\tvl = SVE_VL_ARCH_MAX;\r\n>>> +\r\n>>> +\tvl = find_supported_vector_length(vl);\r\n>>> +\r\n>> \r\n>> \r\n>> Given, sve_set_vector_length is called when setting the vector length in\r\n>> PTRACE_SETREGSET, it looks to me like if you set VL to a value that’s not\r\n>> supported by the hardware, then it’s going to round down to the previous value.\r\n>> Is that correct? I’m not sure if that’s explained in the docs?\r\n> \r\n> Does this cover it?\r\n> \r\n> \"On success, the calling thread's vector length is changed to the\r\n> largest value supported by the system that is less than or equal to vl.\"\r\n> \r\n> (For ptrace, I just cross-reference the PR_SVE_SET_VL behaviour, above.)\r\n\r\nFor ptrace is it worth mentioning user should do a GET after a SET to confirm\r\nwhat VL value was actually set?\r\n\r\n> \r\n>> What happens if you give a vl value lower than the min supported value in the\r\n>> hardware?\r\n> \r\n> This is impossible, unless vl < SVE_VL_MIN (which is rejected explicitly\r\n> by the !sve_vl_valid() check in sve_set_vector_length()).\r\n> \r\n> The architecture required support for all power-of-two vector lengths\r\n> less than the maximum supported vector length, so by construction\r\n> SVE_VL_MIN is supported by all hardware.\r\n\r\nOk, I’m happy with that.\r\n\r\n> \r\n> To be defensive, if we fail to detect support for SVE_VL_MIN, I set the\r\n> corresponding bit in sve_vq_map and WARN.  This is just to help ensure\r\n> find_supported_vector_length doesn't fall off the end of sve_vq_map.\r\n> \r\n> \r\n> Does that sounds correct?  There may be a clearer way of achieving this.\r\n> \r\n> Cheers\r\n> ---Dave\r\n> \r\n>> \r\n>> \r\n>>> +/*\r\n>>> + * All vector length selection from userspace comes through here.\r\n>>> + * We're on a slow path, so some sanity-checks are included.\r\n>>> + * If things go wrong there's a bug somewhere, but try to fall back to a\r\n>>> + * safe choice.\r\n>>> + */\r\n>>> +static unsigned int find_supported_vector_length(unsigned int vl)\r\n>>> +{\r\n>>> +\tint bit;\r\n>>> +\tint max_vl = sve_max_vl;\r\n>>> +\r\n>>> +\tif (WARN_ON(!sve_vl_valid(vl)))\r\n>>> +\t\tvl = SVE_VL_MIN;\r\n>>> +\r\n>>> +\tif (WARN_ON(!sve_vl_valid(max_vl)))\r\n>>> +\t\tmax_vl = SVE_VL_MIN;\r\n>>> +\r\n>>> +\tif (vl > max_vl)\r\n>>> +\t\tvl = max_vl;\r\n>>> +\r\n>>> +\tbit = find_next_bit(sve_vq_map, SVE_VQ_MAX,\r\n>>> +\t\t\t    vq_to_bit(sve_vq_from_vl(vl)));\r\n>>> +\treturn sve_vl_from_vq(bit_to_vq(bit));\r\n>>> +}\r\n>>> +\r\n>> \r\n>> \r\n>> Thanks,\r\n>> Alan.\r\n>> _______________________________________________\r\n>> linux-arm-kernel mailing list\r\n>> linux-arm-kernel@lists.infradead.org\r\n>> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel","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 header.b=\"RIUtzQbI\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=armh.onmicrosoft.com\n\theader.i=@armh.onmicrosoft.com\n\theader.b=\"hWXjAAF+\"; dkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=Alan.Hayward@arm.com; "],"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 3xy76F6mntz9s8J\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tThu, 21 Sep 2017 04:08:53 +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 1dujQY-0002eX-3o; Wed, 20 Sep 2017 18:08:50 +0000","from mail-ve1eur01on0073.outbound.protection.outlook.com\n\t([104.47.1.73] helo=EUR01-VE1-obe.outbound.protection.outlook.com)\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dujQT-0002aL-Gn for linux-arm-kernel@lists.infradead.org;\n\tWed, 20 Sep 2017 18:08:48 +0000","from AM3PR08MB0101.eurprd08.prod.outlook.com (10.160.211.19) by\n\tAM3PR08MB0708.eurprd08.prod.outlook.com (10.163.189.26) with\n\tMicrosoft SMTP Server (version=TLS1_2,\n\tcipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id\n\t15.20.13.10; Wed, 20 Sep 2017 18:08:21 +0000","from AM3PR08MB0101.eurprd08.prod.outlook.com\n\t([fe80::916d:2b75:34f2:da29]) by\n\tAM3PR08MB0101.eurprd08.prod.outlook.com\n\t([fe80::916d:2b75:34f2:da29%17]) with mapi id 15.20.0056.018;\n\tWed, 20 Sep 2017 18:08:21 +0000"],"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:Content-ID:In-Reply-To:\n\tReferences:Message-ID:Date:Subject:To:From:Reply-To:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=sz00qGTMG7wTAfoYaTWxI3w3/1+3mURNtbPlpugZGd8=;\n\tb=RIUtzQbIqZFiFx\n\tCIymiCIdB1Rz4m1lKF6+FbxV+nd+7QLObXvCbVF8lh+JpH8WODOxa0mXqdWOcisILlbSZe23ZBIKb\n\txhERuVP/ApT7f4DWUhF11hsSQ8fnY9iLRgpX2bzhon/QVQgaPZeKEobJNJ2IVcBg1oeujeNtPn8Oy\n\tRyZP8RA3LR38NxgB3ca55dgpYNaEdWuWspZDbSoFPZRiJdGkXeKtDSmVqZIHrLk9KRDY8ptA2phEG\n\tJR6REhCLc86f4RITdvnRBH7gLtVKeEIn2LMiaidj3DI+wDFKAI0YqnVHrWXKFTpavlj1T8Fo9EkZK\n\t6b6wtg6n50KENNhh+NZQ==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=armh.onmicrosoft.com;\n\ts=selector1-arm-com;\n\th=From:Date:Subject:Message-ID:Content-Type:MIME-Version;\n\tbh=8wJ/KSftoplVMRCPZywGPTdJKyhV2GOr5TeRXHVE/Jw=;\n\tb=hWXjAAF+Yz+Xg7jE1nseGNOniVE2M/Dzr2PqTVLF3giQpPc4VV9i2FE34MeErdptF81mFJAseopY0aysJRP8AjnPqxKbFUXMpjMJJrHy+knzir8u9u1ClxZQKlqEekkY3V4QpRX9JfVx8GzOc40w4QqIeY+qYYdMXHL/0vIF+K4="],"From":"Alan Hayward <Alan.Hayward@arm.com>","To":"Dave Martin <dave.martin@foss.arm.com>","Subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","Thread-Topic":"[PATCH v2 14/28] arm64/sve: Backend logic for setting the\n\tvector length","Thread-Index":"AQHTInrT0RE6i7ldhE+RC9J8O4DYcaK9ujmAgAACjgCAAHUkAA==","Date":"Wed, 20 Sep 2017 18:08:21 +0000","Message-ID":"<E82895B2-3F95-4A54-8703-D654221F8CBA@arm.com>","References":"<1504198860-12951-1-git-send-email-Dave.Martin@arm.com>\n\t<1504198860-12951-15-git-send-email-Dave.Martin@arm.com>\n\t<FC5BA359-D37C-493C-87CD-146B83D3CCB5@arm.com>\n\t<20170920110902.GG24231@e103592.cambridge.arm.com>","In-Reply-To":"<20170920110902.GG24231@e103592.cambridge.arm.com>","Accept-Language":"en-GB, en-US","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","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 header.b=\"RIUtzQbI\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=armh.onmicrosoft.com\n\theader.i=@armh.onmicrosoft.com\n\theader.b=\"hWXjAAF+\"; dkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=Alan.Hayward@arm.com; "],"x-originating-ip":"[217.140.96.140]","x-ms-publictraffictype":"Email","x-microsoft-exchange-diagnostics":"1; AM3PR08MB0708;\n\t6:M+lvd3PYDg53KyuXWW5RElj/4fnCoDuP7Qr6BjzmiKns/mDqhCmhoL4OCMb5X44ED7ZuajYI42uMqfKq99RO/bYYzCVnuFzP5MXdskfJw32saaVTwIhTpFBYKB2kkeTn6a3O661tRNO76WflASKOHxNoKYrMETrFpBHpN7v548zu6jUfVUhnwZqVkAiY+vscZnE7pAe5ghTneNY3/VVjBjKClnzgzOtiileTu28BnT814FBak1qfdEM8rjtJHDBiIB0GaFpPuHORg6GDmxsdVbLcnDvVO6laKjIp0+PbpTw/iyi+KLBrWXo5tEdMNTueoM76kVwDmtXhCzeSOBV9kg==;\n\t5:/Rut767rX1k+lBgStvMar+ZC8ZfX6CYQO+8O7ZVQIiehIpH9l6kbV+oUkgyIBTkcn2jwEoLDysk8LfDZBNqyfecfOq5LiCj+9I9DGfuANbRf6a5zW20kxXDQXO5DEEexK3ftbKEWaip2LZXgz1s+QQ==;\n\t24:tq6m3fnBi+ah0hN1ylCnsuz1GVxA5lvJXPt6PFliKvD6Vn46KeldX3mnf2BB/vRUsbbJKOeryFokmVZKtFalUw9EIzRbiE75DR8keAq8HQw=;\n\t7:nrsu5sCoQV1b5lDHo53MAhuhI9tchvdaC/YvrSv9J/DBmV/oXm3CByDGJ7Ye7QOvNgjJBcbp0tcW6g4g9HBmUkk9a28d0ErCvTwJ8P+NnkRgMPXqGA7e3t1ARZ8kYFfq4wQ/iy6KKE2Oi6vJa3c95pIsdR9Z6sNUOzUBxEP+sPNy/BSmsJir33jKOg/4PmZd6MPwRtUEnlog4ZOOw51K5u9wtnjfkep1QSNumvNmwj8=","x-ms-exchange-antispam-srfa-diagnostics":"SSOS;","x-ms-office365-filtering-correlation-id":"ee48ca93-4ecc-4793-877a-08d500529453","x-ms-office365-filtering-ht":"Tenant","x-microsoft-antispam":"UriScan:; BCL:0; PCL:0;\n\tRULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(2017030254152)(48565401081)(300000503095)(300135400095)(2017052603199)(201703131423075)(201703031133081)(201702281549075)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095);\n\tSRVR:AM3PR08MB0708; ","x-ms-traffictypediagnostic":"AM3PR08MB0708:","nodisclaimer":"True","x-exchange-antispam-report-test":"UriScan:(180628864354917)(258649278758335); ","x-microsoft-antispam-prvs":"<AM3PR08MB07088602FA659F5BA8BDC30F97610@AM3PR08MB0708.eurprd08.prod.outlook.com>","x-exchange-antispam-report-cfa-test":"BCL:0; PCL:0;\n\tRULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(5005006)(8121501046)(100000703101)(100105400095)(93006095)(93001095)(3002001)(10201501046)(6055026)(6041248)(20161123558100)(20161123562025)(20161123555025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123564025)(20161123560025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095);\n\tSRVR:AM3PR08MB0708; BCL:0; PCL:0;\n\tRULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095);\n\tSRVR:AM3PR08MB0708; ","x-forefront-prvs":"04362AC73B","x-forefront-antispam-report":"SFV:NSPM;\n\tSFS:(10009020)(6009001)(346002)(376002)(39860400002)(199003)(24454002)(189002)(2950100002)(5250100002)(93886005)(68736007)(4326008)(25786009)(5660300001)(14454004)(33656002)(6246003)(106356001)(72206003)(53546010)(99286003)(6512007)(1720100001)(2900100001)(76176999)(6306002)(54356999)(53936002)(50986999)(105586002)(82746002)(478600001)(6862004)(966005)(101416001)(8936002)(81156014)(6436002)(81166006)(3846002)(102836003)(6116002)(316002)(83716003)(86362001)(8676002)(189998001)(2906002)(66066001)(6486002)(36756003)(7736002)(305945005)(3660700001)(97736004)(6506006)(54906003)(229853002)(3280700002);\n\tDIR:OUT; SFP:1101; SCL:1; SRVR:AM3PR08MB0708;\n\tH:AM3PR08MB0101.eurprd08.prod.outlook.com; FPR:; SPF:None;\n\tPTR:InfoNoRecords; A:1; MX:1; LANG:en; ","received-spf":"None (protection.outlook.com: arm.com does not designate\n\tpermitted sender hosts)","spamdiagnosticoutput":"1:99","spamdiagnosticmetadata":"NSPM","Content-ID":"<E10FDC6A16F67C418BEF1FD9A3874438@eurprd08.prod.outlook.com>","MIME-Version":"1.0","X-OriginatorOrg":"arm.com","X-MS-Exchange-CrossTenant-originalarrivaltime":"20 Sep 2017 18:08:21.6426\n\t(UTC)","X-MS-Exchange-CrossTenant-fromentityheader":"Hosted","X-MS-Exchange-CrossTenant-id":"f34e5979-57d9-4aaa-ad4d-b122a662184d","X-MS-Exchange-Transport-CrossTenantHeadersStamped":"AM3PR08MB0708","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170920_110845_978938_835F467F ","X-CRM114-Status":"GOOD (  22.10  )","X-Spam-Score":"-2.9 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-2.9 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno trust [104.47.1.73 listed in list.dnswl.org]\n\t-1.0 RCVD_IN_MSPIKE_H5      RBL: Excellent reputation (+5)\n\t[104.47.1.73 listed in wl.mailspike.net]\n\t-0.0 SPF_PASS               SPF: sender matches SPF record\n\t-0.0 SPF_HELO_PASS          SPF: HELO matches SPF record\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED            Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.0 RCVD_IN_MSPIKE_WL      Mailspike good senders","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\" <linux-arch@vger.kernel.org>,\n\t\"libc-alpha@sourceware.org\" <libc-alpha@sourceware.org>, Ard Biesheuvel\n\t<ard.biesheuvel@linaro.org>, Szabolcs Nagy <Szabolcs.Nagy@arm.com>, \n\t\"gdb@sourceware.org\" <gdb@sourceware.org>, Yao Qi <Yao.Qi@arm.com>, \n\tWill Deacon <Will.Deacon@arm.com>, \"kvmarm@lists.cs.columbia.edu\"\n\t<kvmarm@lists.cs.columbia.edu>, Richard Sandiford\n\t<Richard.Sandiford@arm.com>, Catalin Marinas <Catalin.Marinas@arm.com>,\n\tnd <nd@arm.com>, =?utf-8?q?Alex_Benn=C3=A9e?= <alex.bennee@linaro.org>,\n\tDave P Martin <Dave.Martin@arm.com>,\n\t\"linux-arm-kernel@lists.infradead.org\"\n\t<linux-arm-kernel@lists.infradead.org>","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"}},{"id":1772710,"web_url":"http://patchwork.ozlabs.org/comment/1772710/","msgid":"<20170921111937.GA17434@e103592.cambridge.arm.com>","list_archive_url":null,"date":"2017-09-21T11:19:45","subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","submitter":{"id":26612,"url":"http://patchwork.ozlabs.org/api/people/26612/","name":"Dave Martin","email":"Dave.Martin@arm.com"},"content":"On Wed, Sep 20, 2017 at 06:08:21PM +0000, Alan Hayward wrote:\n> \n> > On 20 Sep 2017, at 12:09, Dave Martin <dave.martin@foss.arm.com> wrote:\n\n[...]\n\n> >> Given, sve_set_vector_length is called when setting the vector length in\n> >> PTRACE_SETREGSET, it looks to me like if you set VL to a value that’s not\n> >> supported by the hardware, then it’s going to round down to the previous value.\n> >> Is that correct? I’m not sure if that’s explained in the docs?\n> > \n> > Does this cover it?\n> > \n> > \"On success, the calling thread's vector length is changed to the\n> > largest value supported by the system that is less than or equal to vl.\"\n> > \n> > (For ptrace, I just cross-reference the PR_SVE_SET_VL behaviour, above.)\n> \n> For ptrace is it worth mentioning user should do a GET after a SET to confirm\n> what VL value was actually set?\n\nThis seems worth a clarification -- I'd thought this was already\nmentioned, but it isn't.\n\nHow about:\n\n  The caller must make a further GETREGSET call if it needs to know what VL is\n  actually set by SETREGSET, unless is it known in advance that the requested\n  VL is supported.\n\n\n[...]\n\nCheers\n---Dave","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=\"Uwk3tZJq\"; 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 3xyZ0P6Vmjz9t3v\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tThu, 21 Sep 2017 21:20:21 +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 1duzWk-00060A-6m; Thu, 21 Sep 2017 11:20:18 +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 1duzWf-0004jY-Fp for linux-arm-kernel@lists.infradead.org;\n\tThu, 21 Sep 2017 11:20:16 +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 13B4880D;\n\tThu, 21 Sep 2017 04:19:53 -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 ESMTPSA id\n\t86C3F3F58C; Thu, 21 Sep 2017 04:19:50 -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:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=NQYqcwoa83BWuh7rcUGd5ojPaK6OX4xliAjztFcm9Ns=;\n\tb=Uwk3tZJqr+7NDX\n\t5uUfFr7Lg3JhGiiy4ncRXCtzR1Z0hN8KK8LDphayvgBNx6fKrBdfvi61ToM/LhOWg8ohk4L12Wk/+\n\tszFljpBcc7CeH+gIAx7rxjTZxJHpjjKifbDE5UKm1C1ZnLfV2AQdiiYA0F11tzSrJk1N/K2NikTHk\n\tAgUMtxYrNbjldmx950aiciZo2utxe3iWPAQ14yl0oerYKUunreg1HvoZwYYN507dG78c3M7rVHH2D\n\teiYaU4E8ZdygadsG5yLVblfcnqYZzN9FH0c3V4b5WWoiEqWpDydHhIsSCdo9/72iKLkvgmqvjHmnM\n\tuXGmk2WQtkAqJzjJ/HHw==;","Date":"Thu, 21 Sep 2017 12:19:45 +0100","From":"Dave Martin <Dave.Martin@arm.com>","To":"Alan Hayward <Alan.Hayward@arm.com>","Subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","Message-ID":"<20170921111937.GA17434@e103592.cambridge.arm.com>","References":"<1504198860-12951-1-git-send-email-Dave.Martin@arm.com>\n\t<1504198860-12951-15-git-send-email-Dave.Martin@arm.com>\n\t<FC5BA359-D37C-493C-87CD-146B83D3CCB5@arm.com>\n\t<20170920110902.GG24231@e103592.cambridge.arm.com>\n\t<E82895B2-3F95-4A54-8703-D654221F8CBA@arm.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<E82895B2-3F95-4A54-8703-D654221F8CBA@arm.com>","User-Agent":"Mutt/1.5.23 (2014-03-12)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170921_042013_577362_FEC3DACA ","X-CRM114-Status":"GOOD (  14.04  )","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\" <linux-arch@vger.kernel.org>,\n\t\"libc-alpha@sourceware.org\" <libc-alpha@sourceware.org>, Ard Biesheuvel\n\t<ard.biesheuvel@linaro.org>,  Szabolcs Nagy <Szabolcs.Nagy@arm.com>,\n\t\"gdb@sourceware.org\" <gdb@sourceware.org>, Yao Qi <Yao.Qi@arm.com>, \n\tWill Deacon\n\t<Will.Deacon@arm.com>, Dave Martin <dave.martin@foss.arm.com>,\n\tRichard Sandiford <Richard.Sandiford@arm.com>, Catalin Marinas\n\t<Catalin.Marinas@arm.com>, nd <nd@arm.com>, Alex =?iso-8859-1?q?Benn?=\n\t=?iso-8859-1?q?=E9e?= <alex.bennee@linaro.org>,\n\t\"kvmarm@lists.cs.columbia.edu\" <kvmarm@lists.cs.columbia.edu>, \n\t\"linux-arm-kernel@lists.infradead.org\"\n\t<linux-arm-kernel@lists.infradead.org>","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"}},{"id":1772740,"web_url":"http://patchwork.ozlabs.org/comment/1772740/","msgid":"<E7AEE685-0814-4308-97D2-4AC095A52ED2@arm.com>","list_archive_url":null,"date":"2017-09-21T11:57:24","subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","submitter":{"id":65004,"url":"http://patchwork.ozlabs.org/api/people/65004/","name":"Alan Hayward","email":"alan.hayward@arm.com"},"content":"> On 21 Sep 2017, at 12:19, Dave Martin <Dave.Martin@arm.com> wrote:\r\n> \r\n> On Wed, Sep 20, 2017 at 06:08:21PM +0000, Alan Hayward wrote:\r\n>> \r\n>>> On 20 Sep 2017, at 12:09, Dave Martin <dave.martin@foss.arm.com> wrote:\r\n> \r\n> [...]\r\n> \r\n>>>> Given, sve_set_vector_length is called when setting the vector length in\r\n>>>> PTRACE_SETREGSET, it looks to me like if you set VL to a value that’s not\r\n>>>> supported by the hardware, then it’s going to round down to the previous value.\r\n>>>> Is that correct? I’m not sure if that’s explained in the docs?\r\n>>> \r\n>>> Does this cover it?\r\n>>> \r\n>>> \"On success, the calling thread's vector length is changed to the\r\n>>> largest value supported by the system that is less than or equal to vl.\"\r\n>>> \r\n>>> (For ptrace, I just cross-reference the PR_SVE_SET_VL behaviour, above.)\r\n>> \r\n>> For ptrace is it worth mentioning user should do a GET after a SET to confirm\r\n>> what VL value was actually set?\r\n> \r\n> This seems worth a clarification -- I'd thought this was already\r\n> mentioned, but it isn't.\r\n> \r\n> How about:\r\n> \r\n>  The caller must make a further GETREGSET call if it needs to know what VL is\r\n>  actually set by SETREGSET, unless is it known in advance that the requested\r\n>  VL is supported.\r\n> \r\n\r\nLooks good to me.\r\n\r\n\r\nAlan.","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 header.b=\"c9NL9u/0\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=armh.onmicrosoft.com\n\theader.i=@armh.onmicrosoft.com\n\theader.b=\"sKw7z1Sq\"; dkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=Alan.Hayward@arm.com; "],"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 3xyZql6VFcz9s5L\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tThu, 21 Sep 2017 21:57:55 +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 1dv076-0006gd-DG; Thu, 21 Sep 2017 11:57:52 +0000","from mail-ve1eur03on0627.outbound.protection.outlook.com\n\t([2a01:111:f400:fe09::627]\n\thelo=EUR03-VE1-obe.outbound.protection.outlook.com)\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dv072-0006c1-PK for linux-arm-kernel@lists.infradead.org;\n\tThu, 21 Sep 2017 11:57:50 +0000","from DB3PR08MB0106.eurprd08.prod.outlook.com (10.161.56.20) by\n\tDB3PR08MB0074.eurprd08.prod.outlook.com (10.161.56.12) with Microsoft\n\tSMTP Server (version=TLS1_2,\n\tcipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id\n\t15.20.56.11; Thu, 21 Sep 2017 11:57:24 +0000","from DB3PR08MB0106.eurprd08.prod.outlook.com\n\t([fe80::20a0:44dc:3c35:6cce]) by\n\tDB3PR08MB0106.eurprd08.prod.outlook.com\n\t([fe80::20a0:44dc:3c35:6cce%18]) with mapi id 15.20.0056.018;\n\tThu, 21 Sep 2017 11:57:24 +0000"],"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:Content-ID:In-Reply-To:\n\tReferences:Message-ID:Date:Subject:To:From:Reply-To:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=skGhtDpN/gIHnveE07povCNktjyuZKJ9E/KKnY1AGgw=;\n\tb=c9NL9u/0e0FYb9\n\twI71l/UhAtibKxTDsfWxZnC6ZkNtSsQgApvQpIUZwpV3vOZRLwgzkIgtRm8IEXvfcUM5GXWpob4gL\n\t6MTJRWVBUKShUOasN9ed6Qm9edTF6IerUgluaJWqHm+fCjt3t2cIMJQr69F707Uz4k/JpmjrG+U1G\n\t4F5alnnIJriAb8JL9b8tWq3Zt2NB1hjlcre/isEUctpC8v+/NkRRXsAO2ZtRyPitB0PTcJH0qPx9J\n\t7g86Mxxu/+fh1yUu+f/cLLtxQ5uw/cik0KY7it26Kzif8KAlY+gS05sU2449Fm9u9Pnqr6N5P8msZ\n\tg9D8BZBIYsdOP2kYkBbg==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=armh.onmicrosoft.com;\n\ts=selector1-arm-com;\n\th=From:Date:Subject:Message-ID:Content-Type:MIME-Version;\n\tbh=7YzZjsUh2FF10/ae/9hAZT939jBgKfChbgCv7bOT/Jc=;\n\tb=sKw7z1SqZPL4gY9MK1zknQ+xhzFFJ62/ATG154Ewi6LvN6NdxUYJMEkJxhGasip96BNvTd8wkSIneZGS2j1EUUGzLKcxQFpQDytN95A7FlQRcjGGcVefZUW+4cmWAfEFLGkcJzfTEX8bsOBpffzAlaxRCH83pZ1A10rVUW1QCE4="],"From":"Alan Hayward <Alan.Hayward@arm.com>","To":"Dave P Martin <Dave.Martin@arm.com>","Subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","Thread-Topic":"[PATCH v2 14/28] arm64/sve: Backend logic for setting the\n\tvector length","Thread-Index":"AQHTInrT0RE6i7ldhE+RC9J8O4DYcaK9ujmAgAACjgCAAHUkAIABIC2AgAAKhIA=","Date":"Thu, 21 Sep 2017 11:57:24 +0000","Message-ID":"<E7AEE685-0814-4308-97D2-4AC095A52ED2@arm.com>","References":"<1504198860-12951-1-git-send-email-Dave.Martin@arm.com>\n\t<1504198860-12951-15-git-send-email-Dave.Martin@arm.com>\n\t<FC5BA359-D37C-493C-87CD-146B83D3CCB5@arm.com>\n\t<20170920110902.GG24231@e103592.cambridge.arm.com>\n\t<E82895B2-3F95-4A54-8703-D654221F8CBA@arm.com>\n\t<20170921111937.GA17434@e103592.cambridge.arm.com>","In-Reply-To":"<20170921111937.GA17434@e103592.cambridge.arm.com>","Accept-Language":"en-GB, en-US","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","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 header.b=\"c9NL9u/0\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=armh.onmicrosoft.com\n\theader.i=@armh.onmicrosoft.com\n\theader.b=\"sKw7z1Sq\"; dkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=Alan.Hayward@arm.com; "],"x-originating-ip":"[217.140.96.140]","x-ms-publictraffictype":"Email","x-microsoft-exchange-diagnostics":"1; DB3PR08MB0074;\n\t6:NGN8xq7adtu72IMzy6bV3mBTi9AckRJZamUzxSW1Vq0gzFN2JAfSfM0bRjnDodO4fICkpbyVOuAERCgJeH8eRXp2Tl+c+7CrGtaxlaiEOogDADNfdgtIOmqEuYrDJHBXMDs+fdDWvWhMtDlARJ7UU0C0lB096HhLU4lyasn7yQM+cvSzcNI1e9ESB6uwhdtFRfmNQoOn+sP5/n5lZjOasz+BFiH44J/Vf+f2LWDEbPy7JICLsXHdeZFMmKhyvwgGW5MEqvvIowOob4+Lmda2FjjEEFqlu69ObOCxtN34z3aHu4K7m5VPbKkcPrLLozIvsLcAjzMx7klHiBmGGhUemA==;\n\t5:7P/NTEMAze1JG/m2jfRBdVI48WFLxA89Dvcb6P+4ReDTeBhRX8/yCuSOpSNBRdmsbr9MEacnN6pSw++zHVFQbxbkRyF90EfBHih6/kf/V8uUQpEoYamtBLwTTW+4k07BHgBCy1E/DgojZQtA+vUGeQ==;\n\t24:XNIBaid82JEqIwj0kOSDalyb2Bm8FC9psJD0GhzFzXWJ7gWfqk75lCym1LCkgmXwUyJa5566ujvE0X4vWSQlMh+q+znKwcj822r4IsFBTuQ=;\n\t7:utQ0O+1FweucMaWFjDQz3jRL0+2zzqngkW3cC9OUK46o1D/GyT7P8fHVLcVS1+1R3TOlQ0nsrxPxu5Rkf+HCSgJlFumJ8cylsWF6OFgisYi1srmli85Nw+gosQ7B92J4eu+lFxG0D3untAxqS3wqBzlLp3vBjTem1PzrRCWl0WO9/3iewDLKFtv3R4tqw026PjSpvwaAH/3ySw36b3GWoL8huQuTLewIsHvVbNJ55yc=","x-ms-exchange-antispam-srfa-diagnostics":"SSOS;","x-ms-office365-filtering-correlation-id":"d333b1cb-2bbe-40e2-171f-08d500e7ec93","x-ms-office365-filtering-ht":"Tenant","x-microsoft-antispam":"UriScan:; BCL:0; PCL:0;\n\tRULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(2017030254152)(48565401081)(300000503095)(300135400095)(2017052603199)(201703131423075)(201703031133081)(201702281549075)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095);\n\tSRVR:DB3PR08MB0074; ","x-ms-traffictypediagnostic":"DB3PR08MB0074:","nodisclaimer":"True","x-exchange-antispam-report-test":"UriScan:(180628864354917);","x-microsoft-antispam-prvs":"<DB3PR08MB007444768278657CB0ECA6A897660@DB3PR08MB0074.eurprd08.prod.outlook.com>","x-exchange-antispam-report-cfa-test":"BCL:0; PCL:0;\n\tRULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(5005006)(8121501046)(3002001)(10201501046)(100000703101)(100105400095)(93006095)(93001095)(6055026)(6041248)(20161123560025)(20161123558100)(20161123564025)(20161123555025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123562025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095);\n\tSRVR:DB3PR08MB0074; BCL:0; PCL:0;\n\tRULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095);\n\tSRVR:DB3PR08MB0074; ","x-forefront-prvs":"04371797A5","x-forefront-antispam-report":"SFV:NSPM;\n\tSFS:(10009020)(6009001)(39860400002)(376002)(346002)(189002)(199003)(24454002)(6486002)(6506006)(316002)(3660700001)(3280700002)(36756003)(93886005)(189998001)(6436002)(81166006)(81156014)(8936002)(76176999)(53546010)(8676002)(305945005)(7736002)(83716003)(54906003)(97736004)(66066001)(14454004)(5250100002)(86362001)(2906002)(37006003)(54356999)(50986999)(99286003)(102836003)(478600001)(6246003)(4326008)(72206003)(53936002)(68736007)(25786009)(33656002)(101416001)(2950100002)(6512007)(6636002)(6116002)(3846002)(82746002)(105586002)(2900100001)(6862004)(106356001)(5660300001)(229853002);\n\tDIR:OUT; SFP:1101; SCL:1; SRVR:DB3PR08MB0074;\n\tH:DB3PR08MB0106.eurprd08.prod.outlook.com; FPR:; SPF:None;\n\tPTR:InfoNoRecords; MX:1; A:1; LANG:en; ","received-spf":"None (protection.outlook.com: arm.com does not designate\n\tpermitted sender hosts)","spamdiagnosticoutput":"1:99","spamdiagnosticmetadata":"NSPM","Content-ID":"<C72B5FFA24A00B4E816F0BD2F22B9E01@eurprd08.prod.outlook.com>","MIME-Version":"1.0","X-OriginatorOrg":"arm.com","X-MS-Exchange-CrossTenant-originalarrivaltime":"21 Sep 2017 11:57:24.8207\n\t(UTC)","X-MS-Exchange-CrossTenant-fromentityheader":"Hosted","X-MS-Exchange-CrossTenant-id":"f34e5979-57d9-4aaa-ad4d-b122a662184d","X-MS-Exchange-Transport-CrossTenantHeadersStamped":"DB3PR08MB0074","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170921_045749_194545_1B5F6029 ","X-CRM114-Status":"GOOD (  11.66  )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-1.9 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno\n\ttrust [2a01:111:f400:fe09:0:0:0:627 listed in] [list.dnswl.org]\n\t-0.0 SPF_PASS               SPF: sender matches SPF record\n\t-0.0 SPF_HELO_PASS          SPF: HELO matches SPF record\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED            Message has a DKIM or DK signature,\n\tnot necessarily valid","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\" <linux-arch@vger.kernel.org>,\n\t\"libc-alpha@sourceware.org\" <libc-alpha@sourceware.org>, Ard Biesheuvel\n\t<ard.biesheuvel@linaro.org>, Szabolcs Nagy <Szabolcs.Nagy@arm.com>, \n\t\"gdb@sourceware.org\" <gdb@sourceware.org>, Yao Qi <Yao.Qi@arm.com>, \n\tWill Deacon\n\t<Will.Deacon@arm.com>, Dave Martin <dave.martin@foss.arm.com>,\n\tRichard Sandiford <Richard.Sandiford@arm.com>, Catalin Marinas\n\t<Catalin.Marinas@arm.com>, nd <nd@arm.com>, =?utf-8?q?Alex_Benn=C3=A9?=\n\t=?utf-8?q?e?= <alex.bennee@linaro.org>, \"kvmarm@lists.cs.columbia.edu\"\n\t<kvmarm@lists.cs.columbia.edu>, \"linux-arm-kernel@lists.infradead.org\"\n\t<linux-arm-kernel@lists.infradead.org>","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"}},{"id":1780798,"web_url":"http://patchwork.ozlabs.org/comment/1780798/","msgid":"<20171005164229.GX3611@e103592.cambridge.arm.com>","list_archive_url":null,"date":"2017-10-05T16:42:29","subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","submitter":{"id":26612,"url":"http://patchwork.ozlabs.org/api/people/26612/","name":"Dave Martin","email":"Dave.Martin@arm.com"},"content":"On Wed, Sep 13, 2017 at 03:11:23PM -0700, Catalin Marinas wrote:\n> On Wed, Sep 13, 2017 at 08:06:12PM +0100, Dave P Martin wrote:\n> > On Wed, Sep 13, 2017 at 10:29:11AM -0700, Catalin Marinas wrote:\n> > > On Thu, Aug 31, 2017 at 06:00:46PM +0100, Dave P Martin wrote:\n> > > > This patch implements the core logic for changing a task's vector\n> > > > length on request from userspace.  This will be used by the ptrace\n> > > > and prctl frontends that are implemented in later patches.\n> > > > \n> > > > The SVE architecture permits, but does not require, implementations\n> > > > to support vector lengths that are not a power of two.  To handle\n> > > > this, logic is added to check a requested vector length against a\n> > > > possibly sparse bitmap of available vector lengths at runtime, so\n> > > > that the best supported value can be chosen.\n> > > > \n> > > > Signed-off-by: Dave Martin <Dave.Martin@arm.com>\n> > > > Cc: Alex Bennée <alex.bennee@linaro.org>\n> > > \n> > > Can this be merged with patch 20? It seems to add the PR_ definitions\n> > > which get actually used later when the prctl interface is added.\n> > \n> > This patch is used both by patch 19 and by patch 20, which I preferred\n> > not to merge with each other: ptrace and prctl are significantly\n> > different things.\n> > \n> > The prctl bit definitions are added here because they are the canonical\n> > definitions used by both interfaces.  The ptrace #defines are based on\n> > them.\n> > \n> > Does it make sense if I merge patch 20 into this one and apply patch 19\n> > on top?  This avoide the appearance of prctl #defines with no prctl\n> > implementation.\n> \n> That's fine, you can bring patch 20 forward. If there are other\n> non-trivial issues, feel free to ignore my comment.\n\nI've had a go at this, but I think it's going to be more trouble than\nit's worth -- there are other interdependencies between the patches\nwhich make them tricky to reorder.\n\nI could add a note in the commit message for this patch explaining why\nthe prctl flag #defines are being added here.  What do you think?\n\nCheers\n---Dave","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=\"BdtTvCKW\"; 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 3y7JVG1FnZz9t2V\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tFri,  6 Oct 2017 03:43:02 +1100 (AEDT)","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 1e09Eg-000572-NX; Thu, 05 Oct 2017 16:42:58 +0000","from usa-sjc-mx-foss1.foss.arm.com ([217.140.101.70]\n\thelo=foss.arm.com)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e09Ec-0004tX-Dt for linux-arm-kernel@lists.infradead.org;\n\tThu, 05 Oct 2017 16:42:56 +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 31FFB80D;\n\tThu,  5 Oct 2017 09:42:34 -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 ESMTPSA id\n\tEE2263F53D; Thu,  5 Oct 2017 09:42:31 -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:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=xkr2yXL2clkRsbuRcOGvifKo/vxV/HA7qTu+Z7t9xic=;\n\tb=BdtTvCKWI69XTX\n\tG6it0mgYMAjiZ703NOqOiri6pUCy2oBbVMv0Kd5KGTTGJR4NkCXGg9jPUkPxGpmRDJCCnZ8RVyFfJ\n\tS5yK4bfiIZIICk1XAfDSvh2vr1qzlLZ2TuO+123taCK0vSiNUjKQssa2kawCRufOFDZ3Di0+OnyU1\n\tMEnQws0qDMR8XsaET+K67ImNpqhDXU54iej3I6qWCNk7gbY9EwTH6lSGen4kPckYX2YAS0BtD+A8L\n\tFSPNiXoX30Dahxt9eyePbnUtqHpP34FS7Ytp8D5Mfm9t9s5tDNL1AEHOwq0ZEWaReUzVOVh6URngP\n\teCAkkxlpGfyLvHiftGIA==;","Date":"Thu, 5 Oct 2017 17:42:29 +0100","From":"Dave Martin <Dave.Martin@arm.com>","To":"Catalin Marinas <catalin.marinas@arm.com>","Subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","Message-ID":"<20171005164229.GX3611@e103592.cambridge.arm.com>","References":"<1504198860-12951-1-git-send-email-Dave.Martin@arm.com>\n\t<1504198860-12951-15-git-send-email-Dave.Martin@arm.com>\n\t<20170913172911.3ca2h6cpju7etifi@localhost>\n\t<20170913190611.GC23415@e103592.cambridge.arm.com>\n\t<20170913221123.y4znytmxtplx24m4@localhost>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20170913221123.y4znytmxtplx24m4@localhost>","User-Agent":"Mutt/1.5.23 (2014-03-12)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171005_094254_479196_279A2937 ","X-CRM114-Status":"GOOD (  24.85  )","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\tgdb@sourceware.org, Yao Qi <Yao.Qi@arm.com>,\n\tWill Deacon <will.deacon@arm.com>, Richard Sandiford\n\t<richard.sandiford@arm.com>,  Alan Hayward <alan.hayward@arm.com>, Alex\n\t=?iso-8859-1?q?Benn=E9e?= <alex.bennee@linaro.org>,\n\tkvmarm@lists.cs.columbia.edu, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"iso-8859-1\"","Content-Transfer-Encoding":"quoted-printable","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"}},{"id":1780819,"web_url":"http://patchwork.ozlabs.org/comment/1780819/","msgid":"<20171005165334.cstx6fszl7shaudg@armageddon.cambridge.arm.com>","list_archive_url":null,"date":"2017-10-05T16:53:34","subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","submitter":{"id":938,"url":"http://patchwork.ozlabs.org/api/people/938/","name":"Catalin Marinas","email":"catalin.marinas@arm.com"},"content":"On Thu, Oct 05, 2017 at 05:42:29PM +0100, Dave P Martin wrote:\n> On Wed, Sep 13, 2017 at 03:11:23PM -0700, Catalin Marinas wrote:\n> > On Wed, Sep 13, 2017 at 08:06:12PM +0100, Dave P Martin wrote:\n> > > On Wed, Sep 13, 2017 at 10:29:11AM -0700, Catalin Marinas wrote:\n> > > > On Thu, Aug 31, 2017 at 06:00:46PM +0100, Dave P Martin wrote:\n> > > > > This patch implements the core logic for changing a task's vector\n> > > > > length on request from userspace.  This will be used by the ptrace\n> > > > > and prctl frontends that are implemented in later patches.\n> > > > > \n> > > > > The SVE architecture permits, but does not require, implementations\n> > > > > to support vector lengths that are not a power of two.  To handle\n> > > > > this, logic is added to check a requested vector length against a\n> > > > > possibly sparse bitmap of available vector lengths at runtime, so\n> > > > > that the best supported value can be chosen.\n> > > > > \n> > > > > Signed-off-by: Dave Martin <Dave.Martin@arm.com>\n> > > > > Cc: Alex Bennée <alex.bennee@linaro.org>\n> > > > \n> > > > Can this be merged with patch 20? It seems to add the PR_ definitions\n> > > > which get actually used later when the prctl interface is added.\n> > > \n> > > This patch is used both by patch 19 and by patch 20, which I preferred\n> > > not to merge with each other: ptrace and prctl are significantly\n> > > different things.\n> > > \n> > > The prctl bit definitions are added here because they are the canonical\n> > > definitions used by both interfaces.  The ptrace #defines are based on\n> > > them.\n> > > \n> > > Does it make sense if I merge patch 20 into this one and apply patch 19\n> > > on top?  This avoide the appearance of prctl #defines with no prctl\n> > > implementation.\n> > \n> > That's fine, you can bring patch 20 forward. If there are other\n> > non-trivial issues, feel free to ignore my comment.\n> \n> I've had a go at this, but I think it's going to be more trouble than\n> it's worth -- there are other interdependencies between the patches\n> which make them tricky to reorder.\n> \n> I could add a note in the commit message for this patch explaining why\n> the prctl flag #defines are being added here.  What do you think?\n\nAs I said, it's up to you. A line in the commit message would do.","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=\"h7P0D5GI\"; 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 3y7Jl82tBxz9t2h\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tFri,  6 Oct 2017 03:54:12 +1100 (AEDT)","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 1e09PU-0003Lh-Pj; Thu, 05 Oct 2017 16:54:08 +0000","from usa-sjc-mx-foss1.foss.arm.com ([217.140.101.70]\n\thelo=foss.arm.com)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e09PM-00036C-4J for linux-arm-kernel@lists.infradead.org;\n\tThu, 05 Oct 2017 16:54:06 +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 BD3B180D;\n\tThu,  5 Oct 2017 09:53:39 -0700 (PDT)","from armageddon.cambridge.arm.com (armageddon.cambridge.arm.com\n\t[10.1.206.84])\n\tby usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id\n\t8610D3F53D; Thu,  5 Oct 2017 09:53:37 -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:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=BenFz3slxZOZSYKcUk87kLVpv+HJCoI4Iy5MHoxsirU=;\n\tb=h7P0D5GIISA/1D\n\tjeOSanKQd6ETePERRzrVG2nr1sBDSrdqJxKaP66jTNiLjPQDu2AbB/BzKJk6N/6WKXvUMsFRvw7eF\n\tC3gUZrmWVsYYjBZrKMZSXasePCQds8QtUVrXTueH9N1nCGYlf22MReDzgEaRncqz3xwlzYGNh+bgv\n\t1oCMo753CP6J7gtSCxAvt+VR9XSvC5JKTfttj6Fb5snFuFpdJQuet4x6dCIpibf9SPJ3jFDdChi3m\n\thAfr2VUMNeGK+Q0Rv6TyNz49TodgniYugqzm55O9WkyFbQkOR+kubDH+S0YkydltfYnLX9oUvJeJh\n\tOddL0DcBcMa5YPMV22mQ==;","Date":"Thu, 5 Oct 2017 17:53:34 +0100","From":"Catalin Marinas <catalin.marinas@arm.com>","To":"Dave Martin <Dave.Martin@arm.com>","Subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","Message-ID":"<20171005165334.cstx6fszl7shaudg@armageddon.cambridge.arm.com>","References":"<1504198860-12951-1-git-send-email-Dave.Martin@arm.com>\n\t<1504198860-12951-15-git-send-email-Dave.Martin@arm.com>\n\t<20170913172911.3ca2h6cpju7etifi@localhost>\n\t<20170913190611.GC23415@e103592.cambridge.arm.com>\n\t<20170913221123.y4znytmxtplx24m4@localhost>\n\t<20171005164229.GX3611@e103592.cambridge.arm.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20171005164229.GX3611@e103592.cambridge.arm.com>","User-Agent":"NeoMutt/20170113 (1.7.2)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171005_095401_053050_7D18A6D3 ","X-CRM114-Status":"GOOD (  26.35  )","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\tgdb@sourceware.org, Yao Qi <Yao.Qi@arm.com>,\n\tAlan Hayward <alan.hayward@arm.com>, Will Deacon <will.deacon@arm.com>, \n\tRichard Sandiford <richard.sandiford@arm.com>, Alex\n\t=?iso-8859-1?q?Benn=E9e?= <alex.bennee@linaro.org>,\n\tkvmarm@lists.cs.columbia.edu, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"iso-8859-1\"","Content-Transfer-Encoding":"quoted-printable","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"}},{"id":1780830,"web_url":"http://patchwork.ozlabs.org/comment/1780830/","msgid":"<20171005170446.GZ3611@e103592.cambridge.arm.com>","list_archive_url":null,"date":"2017-10-05T17:04:47","subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","submitter":{"id":26612,"url":"http://patchwork.ozlabs.org/api/people/26612/","name":"Dave Martin","email":"Dave.Martin@arm.com"},"content":"On Thu, Oct 05, 2017 at 05:53:34PM +0100, Catalin Marinas wrote:\n> On Thu, Oct 05, 2017 at 05:42:29PM +0100, Dave P Martin wrote:\n> > On Wed, Sep 13, 2017 at 03:11:23PM -0700, Catalin Marinas wrote:\n> > > On Wed, Sep 13, 2017 at 08:06:12PM +0100, Dave P Martin wrote:\n> > > > On Wed, Sep 13, 2017 at 10:29:11AM -0700, Catalin Marinas wrote:\n\n[...]\n\n> > > > > Can this be merged with patch 20? It seems to add the PR_ definitions\n> > > > > which get actually used later when the prctl interface is added.\n> > > > \n> > > > This patch is used both by patch 19 and by patch 20, which I preferred\n> > > > not to merge with each other: ptrace and prctl are significantly\n> > > > different things.\n> > > > \n> > > > The prctl bit definitions are added here because they are the canonical\n> > > > definitions used by both interfaces.  The ptrace #defines are based on\n> > > > them.\n> > > > \n> > > > Does it make sense if I merge patch 20 into this one and apply patch 19\n> > > > on top?  This avoide the appearance of prctl #defines with no prctl\n> > > > implementation.\n> > > \n> > > That's fine, you can bring patch 20 forward. If there are other\n> > > non-trivial issues, feel free to ignore my comment.\n> > \n> > I've had a go at this, but I think it's going to be more trouble than\n> > it's worth -- there are other interdependencies between the patches\n> > which make them tricky to reorder.\n> > \n> > I could add a note in the commit message for this patch explaining why\n> > the prctl flag #defines are being added here.  What do you think?\n> \n> As I said, it's up to you. A line in the commit message would do.\n\nOK, I think I'll stick with this then.\n\nCheers\n---Dave","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=\"I++Y3lCC\"; 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 3y7K076BW6z9t2l\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tFri,  6 Oct 2017 04:05:20 +1100 (AEDT)","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 1e09aF-0001Xy-Nd; Thu, 05 Oct 2017 17:05:15 +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 1e09aC-0000HW-4s for linux-arm-kernel@lists.infradead.org;\n\tThu, 05 Oct 2017 17:05:13 +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 BA44E80D;\n\tThu,  5 Oct 2017 10:04:51 -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 ESMTPSA id\n\t829DF3F53D; Thu,  5 Oct 2017 10:04:49 -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:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=b+PMpYVfu2ghjhuZOoaF9M4VTUUGwnrnPqdg5Nj5dUM=;\n\tb=I++Y3lCC0/+nCz\n\t7CG9DKLNSpDksPkehCEkNupDA6HYJI75wpSy9gSAUu2b0MmhVgOp7rpEBKweiIwLkwFaT0Zpmr6Ih\n\t/NkYCS2FdHSxgfjIP5LhEFLGM5e4vuQUh/g9QQC5su3qr9O0uNyB7Zkm2ojma6qqGUcS7oO/2ks2n\n\tK6g1tVmuRccWFmMLg78xFVHdIVRYezsk4UHvJhAXo7lcoPFR/aqBl+32bZ7BjxTJURRlg7pFhXU/p\n\tDVpFJjNkbNW4I0E35nKdyjSrlhmK9rMP2QPU5cCymJFVngLhbB93XIQQrGOMwpjJo4Ppn4Zidpra6\n\tnCoTxgKD6HGbCugudjqQ==;","Date":"Thu, 5 Oct 2017 18:04:47 +0100","From":"Dave Martin <Dave.Martin@arm.com>","To":"Catalin Marinas <catalin.marinas@arm.com>","Subject":"Re: [PATCH v2 14/28] arm64/sve: Backend logic for setting the vector\n\tlength","Message-ID":"<20171005170446.GZ3611@e103592.cambridge.arm.com>","References":"<1504198860-12951-1-git-send-email-Dave.Martin@arm.com>\n\t<1504198860-12951-15-git-send-email-Dave.Martin@arm.com>\n\t<20170913172911.3ca2h6cpju7etifi@localhost>\n\t<20170913190611.GC23415@e103592.cambridge.arm.com>\n\t<20170913221123.y4znytmxtplx24m4@localhost>\n\t<20171005164229.GX3611@e103592.cambridge.arm.com>\n\t<20171005165334.cstx6fszl7shaudg@armageddon.cambridge.arm.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20171005165334.cstx6fszl7shaudg@armageddon.cambridge.arm.com>","User-Agent":"Mutt/1.5.23 (2014-03-12)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171005_100512_208715_18A0E7BF ","X-CRM114-Status":"GOOD (  23.36  )","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\tgdb@sourceware.org, Yao Qi <Yao.Qi@arm.com>,\n\tWill Deacon <will.deacon@arm.com>, Richard Sandiford\n\t<richard.sandiford@arm.com>,  Alan Hayward <alan.hayward@arm.com>, Alex\n\t=?iso-8859-1?q?Benn=E9e?= <alex.bennee@linaro.org>,\n\tkvmarm@lists.cs.columbia.edu, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","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"}}]