On Thu, Nov 14, 2024 at 06:36:15PM +0000, Lorenzo Stoakes wrote: > Refactor the map_deny_write_exec() to not unnecessarily require a VMA > parameter but rather to accept VMA flags parameters, which allows us to use > this function early in mmap_region() in a subsequent commit. > > While we're here, we refactor the function to be more readable and add some > additional documentation. > > Reported-by: Jann Horn <jannh@xxxxxxxxxx> > Fixes: deb0f6562884 ("mm/mmap: undo ->mmap() when arch_validate_flags() fails") > Cc: stable <stable@xxxxxxxxxx> > Reviewed-by: Liam R. Howlett <Liam.Howlett@xxxxxxxxxx> > Reviewed-by: Vlastimil Babka <vbabka@xxxxxxx> > Reviewed-by: Jann Horn <jannh@xxxxxxxxxx> > Signed-off-by: Lorenzo Stoakes <lorenzo.stoakes@xxxxxxxxxx> > --- > include/linux/mman.h | 21 ++++++++++++++++++--- > mm/mmap.c | 2 +- > mm/mprotect.c | 2 +- > 3 files changed, 20 insertions(+), 5 deletions(-) There's no clue here as to what the upstream git id is :( Also, you sent lots of patches for each branch, but not as a series, so we have no idea what order these go in :( Can you resend all of these, with the upstream git id in it, and as a patch series, so we know to apply them correctly? thanks, greg k-h