[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <6FA8AEB5634BC9F0+99411846-6783-44bf-80c4-8b46ad9d046b@uniontech.com>
Date: Tue, 15 Apr 2025 23:09:37 +0800
From: WangYuli <wangyuli@...ontech.com>
To: Lorenzo Stoakes <lorenzo.stoakes@...cle.com>
Cc: corbet@....net, chenhuacai@...nel.org, kernel@...0n.name,
akpm@...ux-foundation.org, jeffxu@...omium.org, Liam.Howlett@...cle.com,
kees@...nel.org, hca@...ux.ibm.com, linux-doc@...r.kernel.org,
linux-kernel@...r.kernel.org, loongarch@...ts.linux.dev, xry111@...111.site,
tglx@...utronix.de, thomas.weissschuh@...utronix.de, Jason@...c4.com,
zhanjun@...ontech.com, niecheng1@...ontech.com, guanwentao@...ontech.com
Subject: Re: [PATCH] mseal sysmap: enable LoongArch
Hi Lorenzo Stoakes,
On 2025/4/15 21:58, Lorenzo Stoakes wrote:
> I'm sure it's fine, but I am yet to encounter somebody enabling this for an
> arch and saying 'I have checked to ensure we in no way rely on remapping
> the VDSO, VVAR or any other special mapping'.
>
> Because if you haven't, doing this breaks your arch.
>
> I hate to sound like a grumpy maintainer, but could I just ask to check
> whether you have, indeed, confirmed this? :)
I've done some real-world testing before submitting.
Before enabling MSEAL_SYSTEM_MAPPINGS, vdso/vvar is not sealed; after
enabling MSEAL_SYSTEM_MAPPINGS, vdso/vvar is sealed.
I tested the boot and shutdown of the Deepin 23 distribution, web Browse
with Firefox, and document editing with LibreOffice both before and
after enabling it, and I didn't encounter any issues
(MSEAL_SYSTEM_MAPPINGS does cause a noticeable performance drop when
outputting images via simple_drm, but I think this is expected).
Of course, I also ran the self-tests provided by
tools/testing/selftests/mseal_system_mappings, and all tests passed.
I wanted to confirm if "breaks your arch" refers to the above?
Do I need to perform any other kinds of tests to ensure the validity of
this submission?
> If so then all good.
>
> I also think we have a table somewhere in a doc that needs updating then?
> Jeff, can you confirm?
>
> Thanks!
>
Thanks!
--
WangYuli
Download attachment "OpenPGP_0xC5DA1F3046F40BEE.asc" of type "application/pgp-keys" (633 bytes)
Download attachment "OpenPGP_signature.asc" of type "application/pgp-signature" (237 bytes)
Powered by blists - more mailing lists