Skip to content

mm, mpx: add "vm_flags_t vm_flags" arg to do_mmap_pgoff()

Oleg Nesterov linux-mm linux-next lkml 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest1 patch 2 repliesIn review

Changes from the cover letter

- s/__do_mmap_pgoff/do_mmap/

- update mm/nommu.c too

- make do_mmap_pgoff() inline (perhaps we should kill it), this also avoids another change in nommu.c

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/1mm, mpx: add "vm_flags_t vm_flags" arg to do_mmap_pgoff()arch/x86/mm/mpx.c include/linux/mm.h mm/mmap.c +1 more+29 -592 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/1] mm, mpx: add "vm_flags_t vm_flags" arg to do_mmap_pgoff()
Quoted 5 lines · Kirill A. Shutemov
Quoted 3 lines · Dave Hansen
Quoted 2 lines · Kirill A. Shutemov
Quoted 2 lines
Quoted 2 lines

These both look nice to me (and they both cull specialty MPX code which is excellent). I'll run them through a quick test.

Quoted 8 lines
Quoted 4 lines
Quoted 1 line

Any update?

Both patches look fine to me and test OK. Feel free to add my acked/tested/etc...

Oleg, could you prepare a proper patch with description/signed-off-by?

I'll send updated patchset with all changes to Andrew.

With pleasure, please see 1/1.

Changes:

- s/__do_mmap_pgoff/do_mmap/

- update mm/nommu.c too

- make do_mmap_pgoff() inline (perhaps we should kill it), this also avoids another change in nommu.c

Oleg.