xen/xsm/flask/ss/avtab.c | 3 +-- xen/xsm/flask/ss/conditional.c | 9 ++++----- xen/xsm/flask/ss/ebitmap.c | 8 ++++---- xen/xsm/flask/ss/policydb.c | 9 ++++----- 4 files changed, 13 insertions(+), 16 deletions(-)
This is to complement patch 'common: move Linux-inherited fixed width type
decls to common header' by removing the unnecessary include of
'asm/byteorder.h'. In the process of removing these includes, the ordering was
corrected to comply with current coding style.
Signed-off-by: Daniel P. Smith <dpsmith@apertussolutions.com>
---
xen/xsm/flask/ss/avtab.c | 3 +--
xen/xsm/flask/ss/conditional.c | 9 ++++-----
xen/xsm/flask/ss/ebitmap.c | 8 ++++----
xen/xsm/flask/ss/policydb.c | 9 ++++-----
4 files changed, 13 insertions(+), 16 deletions(-)
diff --git a/xen/xsm/flask/ss/avtab.c b/xen/xsm/flask/ss/avtab.c
index 9761d028d8..732004b010 100644
--- a/xen/xsm/flask/ss/avtab.c
+++ b/xen/xsm/flask/ss/avtab.c
@@ -19,11 +19,10 @@
/* Ported to Xen 3.0, George Coker, <gscoker@alpha.ncsc.mil> */
+#include <xen/errno.h>
#include <xen/lib.h>
-#include <asm/byteorder.h>
#include <xen/types.h>
#include <xen/xmalloc.h>
-#include <xen/errno.h>
#include "avtab.h"
#include "policydb.h"
diff --git a/xen/xsm/flask/ss/conditional.c b/xen/xsm/flask/ss/conditional.c
index e74fc01746..f948c6172d 100644
--- a/xen/xsm/flask/ss/conditional.c
+++ b/xen/xsm/flask/ss/conditional.c
@@ -9,16 +9,15 @@
/* Ported to Xen 3.0, George Coker, <gscoker@alpha.ncsc.mil> */
-#include <asm/byteorder.h>
-#include <xen/lib.h>
-#include <xen/types.h>
#include <xen/errno.h>
-#include <xen/string.h>
+#include <xen/lib.h>
#include <xen/spinlock.h>
+#include <xen/string.h>
+#include <xen/types.h>
#include <xen/xmalloc.h>
-#include "security.h"
#include "conditional.h"
+#include "security.h"
/*
* cond_evaluate_expr evaluates a conditional expr
diff --git a/xen/xsm/flask/ss/ebitmap.c b/xen/xsm/flask/ss/ebitmap.c
index e1d0a586a7..2cca25b775 100644
--- a/xen/xsm/flask/ss/ebitmap.c
+++ b/xen/xsm/flask/ss/ebitmap.c
@@ -10,12 +10,12 @@
/* Ported to Xen 3.0, George Coker, <gscoker@alpha.ncsc.mil> */
-#include <asm/byteorder.h>
-#include <xen/lib.h>
-#include <xen/xmalloc.h>
+#include <xen/bitmap.h>
#include <xen/errno.h>
+#include <xen/lib.h>
#include <xen/spinlock.h>
-#include <xen/bitmap.h>
+#include <xen/xmalloc.h>
+
#include "ebitmap.h"
#include "policydb.h"
diff --git a/xen/xsm/flask/ss/policydb.c b/xen/xsm/flask/ss/policydb.c
index 7f232f92fa..fd51a5a35d 100644
--- a/xen/xsm/flask/ss/policydb.c
+++ b/xen/xsm/flask/ss/policydb.c
@@ -22,18 +22,17 @@
/* Ported to Xen 3.0, George Coker, <gscoker@alpha.ncsc.mil> */
-#include <asm/byteorder.h>
+#include <xen/errno.h>
#include <xen/lib.h>
+#include <xen/string.h>
#include <xen/types.h>
#include <xen/xmalloc.h>
-#include <xen/string.h>
-#include <xen/errno.h>
#include <conditional.h>
-#include "security.h"
-#include "policydb.h"
#include "conditional.h"
#include "mls.h"
+#include "policydb.h"
+#include "security.h"
#define _DEBUG_HASHES
--
2.20.1
On Wed, Aug 16, 2023 at 3:05 PM Daniel P. Smith <dpsmith@apertussolutions.com> wrote: > > This is to complement patch 'common: move Linux-inherited fixed width type > decls to common header' by removing the unnecessary include of > 'asm/byteorder.h'. In the process of removing these includes, the ordering was > corrected to comply with current coding style. > > Signed-off-by: Daniel P. Smith <dpsmith@apertussolutions.com> > --- > xen/xsm/flask/ss/avtab.c | 3 +-- > xen/xsm/flask/ss/conditional.c | 9 ++++----- > xen/xsm/flask/ss/ebitmap.c | 8 ++++---- > xen/xsm/flask/ss/policydb.c | 9 ++++----- > 4 files changed, 13 insertions(+), 16 deletions(-) These four files all call le32_to_cpu(), so AFAICT, you are trading an explicit dependency for an implicit one. Is that what you want? Regards, Jason
On 8/17/23 08:17, Jason Andryuk wrote: > On Wed, Aug 16, 2023 at 3:05 PM Daniel P. Smith > <dpsmith@apertussolutions.com> wrote: >> >> This is to complement patch 'common: move Linux-inherited fixed width type >> decls to common header' by removing the unnecessary include of >> 'asm/byteorder.h'. In the process of removing these includes, the ordering was >> corrected to comply with current coding style. >> >> Signed-off-by: Daniel P. Smith <dpsmith@apertussolutions.com> >> --- >> xen/xsm/flask/ss/avtab.c | 3 +-- >> xen/xsm/flask/ss/conditional.c | 9 ++++----- >> xen/xsm/flask/ss/ebitmap.c | 8 ++++---- >> xen/xsm/flask/ss/policydb.c | 9 ++++----- >> 4 files changed, 13 insertions(+), 16 deletions(-) > > These four files all call le32_to_cpu(), so AFAICT, you are trading an > explicit dependency for an implicit one. Is that what you want? In fact, no. You comment here caused me to go back and try to see why Jan said they needed to move. I realized there was a misunderstanding about his commit message. He was not saying later, at a point in the future, they need to be moved, but that as a result of his change the inclusion ordering had to be changed. The former is what I understood and thus attempted a build without them to see if it would work. I was not expecting that conversions would have been able to be implicitly resolved. Since I already rebased to latest stable, I can resubmit a v2 with correcting the inclusion order but with byteoder.h inclusion remaining. Though that would render this down to simply a style change patch. Would that still be acceptable Jan? v/r, dps
© 2016 - 2024 Red Hat, Inc.