On Sat, Aug 17, 2024 at 01:18:33AM GMT, Pedro Falcato wrote: > With no more users in the tree, we can finally remove can_modify_mm(). > > Signed-off-by: Pedro Falcato <[email protected]>
Reviewed-by: Lorenzo Stoakes <[email protected]> > --- > mm/internal.h | 14 -------------- > mm/mseal.c | 21 --------------------- > 2 files changed, 35 deletions(-) > > diff --git a/mm/internal.h b/mm/internal.h > index 1db320650539..3b738b0ad893 100644 > --- a/mm/internal.h > +++ b/mm/internal.h > @@ -1361,25 +1361,11 @@ static inline int can_do_mseal(unsigned long flags) > return 0; > } > > -bool can_modify_mm(struct mm_struct *mm, unsigned long start, > - unsigned long end); > #else > static inline int can_do_mseal(unsigned long flags) > { > return -EPERM; > } > - > -static inline bool can_modify_mm(struct mm_struct *mm, unsigned long start, > - unsigned long end) > -{ > - return true; > -} > - > -static inline bool can_modify_mm_madv(struct mm_struct *mm, unsigned long > start, > - unsigned long end, int behavior) > -{ > - return true; > -} > #endif > > #ifdef CONFIG_SHRINKER_DEBUG > diff --git a/mm/mseal.c b/mm/mseal.c > index fdd1666344fa..28cd17d7aaf2 100644 > --- a/mm/mseal.c > +++ b/mm/mseal.c > @@ -53,27 +53,6 @@ static bool is_ro_anon(struct vm_area_struct *vma) > return false; > } > > -/* > - * Check if the vmas of a memory range are allowed to be modified. > - * the memory ranger can have a gap (unallocated memory). > - * return true, if it is allowed. > - */ > -bool can_modify_mm(struct mm_struct *mm, unsigned long start, unsigned long > end) > -{ > - struct vm_area_struct *vma; > - > - VMA_ITERATOR(vmi, mm, start); > - > - /* going through each vma to check. */ > - for_each_vma_range(vmi, vma, end) { > - if (unlikely(!can_modify_vma(vma))) > - return false; > - } > - > - /* Allow by default. */ > - return true; > -} > - > /* > * Check if a vma is allowed to be modified by madvise. > */ > > -- > 2.46.0 >
