<div dir="ltr"><div>Greetings;<br><br></div><div>I recently noticed this change via the linux-next tree.</div><div><br></div><div>It may not be possible to edit at this late date, but the change description refers to PROT_KERNEL, which is a symbol which does not appear to exist; perhaps PAGE_KERNEL was meant? The mismatch caused me and a couple other folks some confusion briefly until we decided it was supposed to be PAGE_KERNEL; if it's not too late, editing the description to clarify so would be nice.<br><br></div><div>Many thanks.<br><br></div><div>John Dorminy<br></div><div><br><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 14, 2020 at 11:15 AM Wei Liu <<a href="mailto:wei.liu@kernel.org">wei.liu@kernel.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Tue, Apr 14, 2020 at 03:13:40PM +0200, Christoph Hellwig wrote:<br>
> The pgprot argument to __vmalloc is always PROT_KERNEL now, so remove<br>
> it.<br>
> <br>
> Signed-off-by: Christoph Hellwig <<a href="mailto:hch@lst.de" target="_blank">hch@lst.de</a>><br>
> Reviewed-by: Michael Kelley <<a href="mailto:mikelley@microsoft.com" target="_blank">mikelley@microsoft.com</a>> [hyperv]<br>
> Acked-by: Gao Xiang <<a href="mailto:xiang@kernel.org" target="_blank">xiang@kernel.org</a>> [erofs]<br>
> Acked-by: Peter Zijlstra (Intel) <<a href="mailto:peterz@infradead.org" target="_blank">peterz@infradead.org</a>><br>
> ---<br>
>  arch/x86/hyperv/hv_init.c              |  3 +--<br>
[...]<br>
> <br>
> diff --git a/arch/x86/hyperv/hv_init.c b/arch/x86/hyperv/hv_init.c<br>
> index 5a4b363ba67b..a3d689dfc745 100644<br>
> --- a/arch/x86/hyperv/hv_init.c<br>
> +++ b/arch/x86/hyperv/hv_init.c<br>
> @@ -95,8 +95,7 @@ static int hv_cpu_init(unsigned int cpu)<br>
>        * not be stopped in the case of CPU offlining and the VM will hang.<br>
>        */<br>
>       if (!*hvp) {<br>
> -             *hvp = __vmalloc(PAGE_SIZE, GFP_KERNEL | __GFP_ZERO,<br>
> -                              PAGE_KERNEL);<br>
> +             *hvp = __vmalloc(PAGE_SIZE, GFP_KERNEL | __GFP_ZERO);<br>
>       }<br>
<br>
Acked-by: Wei Liu <<a href="mailto:wei.liu@kernel.org" target="_blank">wei.liu@kernel.org</a>><br>
<br>
</blockquote></div>