diff mbox series

tools/xenstored: Remove unnecessary define XC_WANT_COMPAT_MAP_FOREIGN_API

Message ID 20210325114243.12284-1-julien@xen.org (mailing list archive)
State New, archived
Headers show
Series tools/xenstored: Remove unnecessary define XC_WANT_COMPAT_MAP_FOREIGN_API | expand

Commit Message

Julien Grall March 25, 2021, 11:42 a.m. UTC 
From: Julien Grall <jgrall@amazon.com>

The last use of the compat foreign API was dropped in commit
38eeb3864de4 "tools/xenstored: Drop mapping of the ring via foreign
map".

Therefore, we don't need to define XC_WANT_COMPAT_MAP_FOREIGN_API.

Signed-off-by: Julien Grall <jgrall@amazon.com>
---
 tools/xenstore/xenstored_core.h | 1 -
 1 file changed, 1 deletion(-)

Comments

Andrew Cooper March 25, 2021, 2:29 p.m. UTC | #1 
On 25/03/2021 11:42, Julien Grall wrote:
> From: Julien Grall <jgrall@amazon.com>
>
> The last use of the compat foreign API was dropped in commit
> 38eeb3864de4 "tools/xenstored: Drop mapping of the ring via foreign
> map".
>
> Therefore, we don't need to define XC_WANT_COMPAT_MAP_FOREIGN_API.
>
> Signed-off-by: Julien Grall <jgrall@amazon.com>

Oops sorry.

Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>
Jürgen Groß March 26, 2021, 7:56 a.m. UTC | #2 
On 25.03.21 12:42, Julien Grall wrote:
> From: Julien Grall <jgrall@amazon.com>
> 
> The last use of the compat foreign API was dropped in commit
> 38eeb3864de4 "tools/xenstored: Drop mapping of the ring via foreign
> map".
> 
> Therefore, we don't need to define XC_WANT_COMPAT_MAP_FOREIGN_API.
> 
> Signed-off-by: Julien Grall <jgrall@amazon.com>

Reviewed-by: Juergen Gross <jgross@suse.com>


Juergen
diff mbox series

Patch

diff --git a/tools/xenstore/xenstored_core.h b/tools/xenstore/xenstored_core.h
index 0c9a0961b57e..1cdbc3dcb5f7 100644
--- a/tools/xenstore/xenstored_core.h
+++ b/tools/xenstore/xenstored_core.h
@@ -19,7 +19,6 @@ 
 #ifndef _XENSTORED_CORE_H
 #define _XENSTORED_CORE_H
 
-#define XC_WANT_COMPAT_MAP_FOREIGN_API
 #include <xenctrl.h>
 #include <xengnttab.h>