| Date | Fri, 24 Jul 2020 11:22:58 +0200 | From | Ingo Molnar <> | Subject | Re: [PATCH v5 3/6] arch/x86: Implement text_alloc() and text_free() |
| |
* Jarkko Sakkinen <jarkko.sakkinen@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. */
Thanks,
Ingo
|