Re: [PATCH v5 2/2] mm: add a field to store names for private anonymous memory

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Kees,

On Fri, 21 Aug 2020 at 02:45, Kees Cook <keescook@xxxxxxxxxxxx> wrote:
>
> On Thu, Aug 20, 2020 at 06:00:14PM +0200, Oleg Nesterov wrote:
> > On 08/19, Sumit Semwal wrote:
> > >
> > > --- a/include/linux/mm.h
> > > +++ b/include/linux/mm.h
> > > @@ -2484,7 +2484,7 @@ static inline int vma_adjust(struct vm_area_struct *vma, unsigned long start,
> > >  extern struct vm_area_struct *vma_merge(struct mm_struct *,
> > >     struct vm_area_struct *prev, unsigned long addr, unsigned long end,
> > >     unsigned long vm_flags, struct anon_vma *, struct file *, pgoff_t,
> > > -   struct mempolicy *, struct vm_userfaultfd_ctx);
> > > +   struct mempolicy *, struct vm_userfaultfd_ctx, const char __user *);
> >
> > It seems that you forgot to update the callers in fs/userfaultfd.c ?
>
> (I recommend including "make allmodconfig && make" in your test workflow.)

Yes, indeed :| - that was stupid of me!
>
> --
> Kees Cook

Best,
Sumit.




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux