[PATCH v4 4/5] whpx: Fixes include of whp-dispatch.h in whpx.h

Yonggang Luo posted 5 patches 5 years, 1 month ago
Maintainers: Richard Henderson <richard.henderson@linaro.org>, Eduardo Habkost <ehabkost@redhat.com>, Ed Maste <emaste@freebsd.org>, Paolo Bonzini <pbonzini@redhat.com>, Li-Wen Hsu <lwhsu@freebsd.org>, Sunil Muthuswamy <sunilmut@microsoft.com>
[PATCH v4 4/5] whpx: Fixes include of whp-dispatch.h in whpx.h
Posted by Yonggang Luo 5 years, 1 month ago
Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
---
 include/sysemu/whpx.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/sysemu/whpx.h b/include/sysemu/whpx.h
index 9346fd92e9..4f38784d7e 100644
--- a/include/sysemu/whpx.h
+++ b/include/sysemu/whpx.h
@@ -15,7 +15,7 @@
 
 #ifdef CONFIG_WHPX
 
-#include "whp-dispatch.h"
+#include <WinHvPlatformDefs.h>
 
 struct whpx_state {
     uint64_t mem_quota;
-- 
2.29.2.windows.3


Re: [PATCH v4 4/5] whpx: Fixes include of whp-dispatch.h in whpx.h
Posted by Marc-André Lureau 5 years, 1 month ago
Hi

On Thu, Jan 7, 2021 at 2:23 PM Yonggang Luo <luoyonggang@gmail.com> wrote:

> Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
> ---
>  include/sysemu/whpx.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/sysemu/whpx.h b/include/sysemu/whpx.h
> index 9346fd92e9..4f38784d7e 100644
> --- a/include/sysemu/whpx.h
> +++ b/include/sysemu/whpx.h
> @@ -15,7 +15,7 @@
>
>  #ifdef CONFIG_WHPX
>
> -#include "whp-dispatch.h"
> +#include <WinHvPlatformDefs.h>
>
>  struct whpx_state {
>      uint64_t mem_quota;
> --
> 2.29.2.windows.3
>

I submitted a similar patch, then Paolo did a different approach which
didn't land yet:
https://patchew.org/QEMU/20201219090637.1700900-1-pbonzini@redhat.com/


-- 
Marc-André Lureau
Re: [PATCH v4 4/5] whpx: Fixes include of whp-dispatch.h in whpx.h
Posted by Philippe Mathieu-Daudé 5 years, 1 month ago
On 1/7/21 11:19 AM, Yonggang Luo wrote:
> Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
> ---
>  include/sysemu/whpx.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>