2020-04-19 07:32:21

by Tianjia Zhang

Subject: [PATCH] KVM: X86: Fix compile error in svm/sev.c

The compiler reported the following compilation errors:

arch/x86/kvm/svm/sev.c: In function ‘sev_pin_memory’:
arch/x86/kvm/svm/sev.c:361:3: error: implicit declaration of function
‘release_pages’ [-Werror=implicit-function-declaration]
release_pages(pages, npinned);
^~~~~~~~~~~~~

The reason is that the 'pagemap.h' header file is not included.

Signed-off-by: Tianjia Zhang <[email protected]>
---
arch/x86/kvm/svm/sev.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/x86/kvm/svm/sev.c b/arch/x86/kvm/svm/sev.c
index 0e3fc311d7da..3ef99e87c1db 100644
--- a/arch/x86/kvm/svm/sev.c
+++ b/arch/x86/kvm/svm/sev.c
@@ -13,6 +13,7 @@
#include <linux/highmem.h>
#include <linux/psp-sev.h>
#include <linux/swap.h>
+#include <linux/pagemap.h>

#include "x86.h"
#include "svm.h"
--
2.17.1

2020-04-19 08:25:48

by Xiaoyao Li

Subject: Re: [PATCH] KVM: X86: Fix compile error in svm/sev.c

On 4/19/2020 3:30 PM, Tianjia Zhang wrote:
> The compiler reported the following compilation errors:
>
> arch/x86/kvm/svm/sev.c: In function ‘sev_pin_memory’:
> arch/x86/kvm/svm/sev.c:361:3: error: implicit declaration of function
> ‘release_pages’ [-Werror=implicit-function-declaration]
> release_pages(pages, npinned);
> ^~~~~~~~~~~~~
>
> The reason is that the 'pagemap.h' header file is not included.
>

FYI.

Boris has sent the Patch:
https://lkml.kernel.org/r/[email protected]

and it's already in kvm master/queue branch

2020-04-20 07:18:42

by Tianjia Zhang

Subject: Re: [PATCH] KVM: X86: Fix compile error in svm/sev.c

On 2020/4/19 16:24, Xiaoyao Li wrote:
> On 4/19/2020 3:30 PM, Tianjia Zhang wrote:
>> The compiler reported the following compilation errors:
>>
>> arch/x86/kvm/svm/sev.c: In function ‘sev_pin_memory’:
>> arch/x86/kvm/svm/sev.c:361:3: error: implicit declaration of function
>> ‘release_pages’ [-Werror=implicit-function-declaration]
>>     release_pages(pages, npinned);
>>     ^~~~~~~~~~~~~
>>
>> The reason is that the 'pagemap.h' header file is not included.
>>
>
> FYI.
>
> Boris has sent the Patch:
> https://lkml.kernel.org/r/[email protected]
>
> and it's already in kvm master/queue branch
>
>

This is the same fix, please ignore this patch.

Thanks and best,
Tianjia