Re: [PATCH v8 0/7] mseal system mappings

Kees Cook
2025-03-03T16:33:57Z

On Mon, Mar 03, 2025 at 09:59:13AM -0500, Liam R. Howlett wrote:
> * jeffxu@chromium.org <jeffxu@chromium.org> [250303 00:09]:
>
> ...
>
> >
> > Link: https://lore.kernel.org/all/20240415163527.626541-1-jeffxu@chromium.org/ [1]
> > Link: Documentation/userspace-api/mseal.rst [2]
> > Link: https://lore.kernel.org/all/CABi2SkU9BRUnqf70-nksuMCQ+yyiWjo3fM4XkRkL-NrCZxYAyg@mail.gmail.com/ [3]
> > Link: https://lore.kernel.org/all/CABi2SkV6JJwJeviDLsq9N4ONvQ=EFANsiWkgiEOjyT9TQSt+HA@mail.gmail.com/ [4]
> > Link: https://lore.kernel.org/all/202502251035.239B85A93@keescook/ [5]
> >
> > -------------------------------------------
> > History:
> >
> > V8:
> > - Change ARCH_SUPPORTS_MSEAL_X to ARCH_SUPPORTS_MSEAL_X (Liam R. Howlett)
>
> It looks like this was captured wrong in just this update. Change
> ARCH_HAS => ARCH_SUPPORTS. Code looks correct.
>
> Kees also ran down the meaning of the two more than I did, so thanks
> again Kees!

You're welcome; I'm glad I did it -- I really need to capture that
research somewhere. Perhaps a blog post ... I can't really find a good
place in Documentation/ ? Hmmm

--
Kees Cook