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

2025-02-28 Thread Lorenzo Stoakes
On Thu, Feb 27, 2025 at 03:43:07PM -0800, Jeff Xu wrote: > On Tue, Feb 25, 2025 at 10:01 PM Lorenzo Stoakes > wrote: > > > > On Tue, Feb 25, 2025 at 04:17:01PM -0800, Jeff Xu wrote: > > > On Tue, Feb 25, 2025 at 2:32 AM Lorenzo Stoakes > > > wrote: > > > > > > > > BTW can we please drop the 'msea

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

2025-02-27 Thread Jeff Xu
On Tue, Feb 25, 2025 at 10:01 PM Lorenzo Stoakes wrote: > > On Tue, Feb 25, 2025 at 04:17:01PM -0800, Jeff Xu wrote: > > On Tue, Feb 25, 2025 at 2:32 AM Lorenzo Stoakes > > wrote: > > > > > > BTW can we please drop the 'mseal, system mappings' prefixes on this > > > series, it's really weird and

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

2025-02-25 Thread Lorenzo Stoakes
On Tue, Feb 25, 2025 at 04:17:01PM -0800, Jeff Xu wrote: > On Tue, Feb 25, 2025 at 2:32 AM Lorenzo Stoakes > wrote: > > > > BTW can we please drop the 'mseal, system mappings' prefixes on this > > series, it's really weird and makes it really hard for me to actually read > > the individual summary

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

2025-02-25 Thread Jeff Xu
On Tue, Feb 25, 2025 at 2:32 AM Lorenzo Stoakes wrote: > > BTW can we please drop the 'mseal, system mappings' prefixes on this > series, it's really weird and makes it really hard for me to actually read > the individual summary lines for each commit. 'mseal:' will do. > I am not sure. I had com

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

2025-02-25 Thread Lorenzo Stoakes
Jeff - looking further in this series, I asked for a couple things for this series which you've not provided: 1. Some assurance based on code that the kernel-side code doesn't rely on VDSO/VVAR etc. mapping. I gave up waiting for this and went and checked myself, it looks fine for arm64, x86

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

2025-02-25 Thread Lorenzo Stoakes
BTW can we please drop the 'mseal, system mappings' prefixes on this series, it's really weird and makes it really hard for me to actually read the individual summary lines for each commit. 'mseal:' will do. I mean really you could argue it's 'mm: mseal: ...' but I'm not quite _that_ pedantic :)

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

2025-02-24 Thread Lorenzo Stoakes
On Mon, Feb 24, 2025 at 03:07:03PM -0800, Jeff Xu wrote: > On Mon, Feb 24, 2025 at 3:03 PM Pedro Falcato wrote: > > > > On Mon, Feb 24, 2025 at 10:52 PM wrote: > > > > > > From: Jeff Xu > > > > > > This is V7 version, addressing comments from V6, without code logic > > > change. > > > > > >

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

2025-02-24 Thread Jeff Xu
On Mon, Feb 24, 2025 at 3:03 PM Pedro Falcato wrote: > > On Mon, Feb 24, 2025 at 10:52 PM wrote: > > > > From: Jeff Xu > > > > This is V7 version, addressing comments from V6, without code logic > > change. > > > > -- > > > > History: > > V7: > >

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

2025-02-24 Thread Pedro Falcato
On Mon, Feb 24, 2025 at 10:52 PM wrote: > > From: Jeff Xu > > This is V7 version, addressing comments from V6, without code logic > change. > > -- > > History: > V7: > - Remove cover letter from the first patch (Liam R. Howlett) > - Change macro n

[PATCH v7 0/7] mseal system mappings

2025-02-24 Thread jeffxu
From: Jeff Xu This is V7 version, addressing comments from V6, without code logic change. -- History: V7: - Remove cover letter from the first patch (Liam R. Howlett) - Change macro name to VM_SEALED_SYSMAP (Liam R. Howlett) - logging and fclos