On Fri, Jul 24, 2020 at 11:22:58AM +0200, Ingo Molnar wrote:
> 
> * Jarkko Sakkinen <jarkko.sakki...@linux.intel.com> wrote:
> 
> > +void text_free(void *region)
> > +{
> > +   /*
> > +    * This memory may be RO, and freeing RO memory in an interrupt is not
> > +    * supported by vmalloc.
> > +    */
> > +   lockdep_assert_irqs_enabled();
> > +
> > +   vfree(region);
> 
> Had to dig around a bit to find the source of this restriction. Might 
> make sense to clarify this comment to:
> 
>       /*
>        * This memory may be read-only, and freeing VM_FLUSH_RESET_PERMS memory
>        * in an interrupt is not supported by vmalloc.
>        */

This definitely clarifies and is a better explanation. I updated the
commit accordingly. Thank you.

> 
> Thanks,
> 
>       Ingo

/Jarkko

Reply via email to