ACK/Cmnt: [PATCH] [Groovy] UBUNTU: SAUCE: export __get_vm_area_caller() and map_kernel_range()

[Andrea Righi]

On Fri, Jul 31, 2020 at 11:55:40AM +0200, Paolo Pisati wrote:
> As explained here:
> 
> https://www.virtualbox.org/ticket/19644
> 
> and in the recent virtualbox DKMS upload changelog:
> 
> https://metadata.ftp-master.debian.org/changelogs/contrib/v/virtualbox/virtualbox_6.1.12-dfsg-8_changelog
> 
> Virtualbox DKMS now requires access to these two kernel functions to fix a FTBFS
> against Linux 5.8, so until they rewrite their VM code (something that is not
> happening anytime soon, see comment 8 in the first link), export these two
> symbols.
> 
> Signed-off-by: Paolo Pisati <paolo.pisati at canonical.com>

Makes sense to me, but, as mentioned by Colin, we should drop this patch
once virtualbox fixes this upstream.

Acked-by: Andrea Righi <andrea.righi at canonical.com>

> ---
>  mm/vmalloc.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/mm/vmalloc.c b/mm/vmalloc.c
> index 317afdc7d247..dffac67ba7f7 100644
> --- a/mm/vmalloc.c
> +++ b/mm/vmalloc.c
> @@ -326,6 +326,7 @@ int map_kernel_range(unsigned long start, unsigned long size, pgprot_t prot,
>  	flush_cache_vmap(start, start + size);
>  	return ret;
>  }
> +EXPORT_SYMBOL(map_kernel_range);
>  
>  int is_vmalloc_or_module_addr(const void *x)
>  {
> @@ -2130,6 +2131,7 @@ struct vm_struct *__get_vm_area_caller(unsigned long size, unsigned long flags,
>  	return __get_vm_area_node(size, 1, flags, start, end, NUMA_NO_NODE,
>  				  GFP_KERNEL, caller);
>  }
> +EXPORT_SYMBOL(__get_vm_area_caller);
>  
>  /**
>   * get_vm_area - reserve a contiguous kernel virtual area
> -- 
> 2.25.1
> 
> 
> -- 
> kernel-team mailing list
> kernel-team at lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team