Hi Juergen,
On 13/09/17 16:46, Juergen Gross wrote:
diff --git a/xen/include/xen/grant_table.h b/xen/include/xen/grant_table.h
index df11b31264..f3f2fb9ebc 100644
--- a/xen/include/xen/grant_table.h
+++ b/xen/include/xen/grant_table.h
@@ -27,7 +27,8 @@
#include <xen/rwlock.h>
#include <public/grant_table.h>
#include <asm/page.h>
-#include <asm/grant_table.h>
This change looks a bit strange to me. This is the only place where
asm/grant_table.h is pulled. Because you remove it, it now means that
the prototype will not be defined first and may result to mismatch in
the future.
Ideally we should enforce, although it would require some work as we
didn't really follow that rule in a few places.
Cheers,
+
+struct grant_table;
/* The maximum size of a grant table. */
extern unsigned int max_grant_frames;
--
Julien Grall
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
