Date:   Tue, 31 Jan 2023 13:14:00 +0000
From:   Mark Brown <broonie@kernel.org>
To:     Catalin Marinas <catalin.marinas@arm.com>
Cc:     Will Deacon <will@kernel.org>,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 0/6] arm64/signal: Signal handling cleanups
Message-ID: <Y9kUGGYpWvbb1fZC@sirena.org.uk>
References: <20221212-arm64-signal-cleanup-v2-0-14a8f3e088b7@kernel.org>
 <Y9kO1egKjKNqE0zg@arm.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="pEWZi37D5UvUp+u6"
Content-Disposition: inline
In-Reply-To: <Y9kO1egKjKNqE0zg@arm.com>
Precedence: bulk


--pEWZi37D5UvUp+u6
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Tue, Jan 31, 2023 at 12:51:33PM +0000, Catalin Marinas wrote:
> On Tue, Jan 03, 2023 at 08:25:15PM +0000, Mark Brown wrote:

> >       arm64/signal: Avoid rereading context frame sizes
> >       arm64/signal: Only read new data when parsing the SVE context
> >       arm64/signal: Only read new data when parsing the ZA context

> I'm not sure these add much to the code readability (and the performance
> improvement I guess is negligible). We avoid some copy_from_user() into
> the context structures but rely on data read previously or some
> get_user() into local variables. Personally I'd make the
> restore_fpsimd_context() also do a copy_from_user() for consistency with
> the current sve and za frames restoring.

> Personal preference, not sure whether Will has the same view.

I don't particularly care about those changs either, Will seemed to be
asking for something like that.

Note that I should at some point today send a version of this series
rebased on for-next/core due to the TPIDR2 and SME2 changes.

--pEWZi37D5UvUp+u6
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmPZFBcACgkQJNaLcl1U
h9CM0Qf/Q4kwXZ5gT5659flGrSzy49DIrxrJyXlQYg6be5W2GMKo+isJeXnSQW72
YZ7ROEKBRMC6Fnv62XGwiSWXtETgEm3xWK49QMnUd2Xd91P6uRdM5hB0EvvBZ+8r
rKTJ4EpDaIP8+kooqrBz0JBFDxfiR5uUo8/Vy1YuFIyDbK39hJnwyS1Dq6rqZd5+
wJewvo8Jef/oqYsB+kGeiv5WIeEUz/CQKAWkpvHFYcOCP1WwTe1W0vsMjF8tmjxi
5tHKzWqFo/O03dyJf+5C03h2ud+1p9V35bKP3OloKSKTPDUXIC44Pem0u3wQ+yH6
rVz3iF8vy7zfylgHM3IgTsCYgyDbYw==
=u87Z
-----END PGP SIGNATURE-----

--pEWZi37D5UvUp+u6--