Message ID | 20200730181827.1670-4-julien@xen.org (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | xen: Consolidate asm-*/guest_access.h in xen/guest_access.h | expand |
On Thu, 30 Jul 2020, Julien Grall wrote: > From: Julien Grall <jgrall@amazon.com> > > We usually have xen/ includes first and then asm/. They are also ordered > alphabetically among themselves. > > Signed-off-by: Julien Grall <jgrall@amazon.com> Might wanna mention the change from asm/guest_access.h to xen/guest_access.h. Anyway: Acked-by: Stefano Stabellini <sstabellini@kernel.org> > --- > xen/arch/arm/decode.c | 5 +++-- > xen/arch/arm/kernel.c | 2 +- > 2 files changed, 4 insertions(+), 3 deletions(-) > > diff --git a/xen/arch/arm/decode.c b/xen/arch/arm/decode.c > index 8b1e15d11892..144793c8cea0 100644 > --- a/xen/arch/arm/decode.c > +++ b/xen/arch/arm/decode.c > @@ -17,11 +17,12 @@ > * GNU General Public License for more details. > */ > > -#include <xen/types.h> > +#include <xen/lib.h> > #include <xen/sched.h> > +#include <xen/types.h> > + > #include <asm/current.h> > #include <asm/guest_access.h> > -#include <xen/lib.h> > > #include "decode.h" > > diff --git a/xen/arch/arm/kernel.c b/xen/arch/arm/kernel.c > index f95fa392af44..032923853f2c 100644 > --- a/xen/arch/arm/kernel.c > +++ b/xen/arch/arm/kernel.c > @@ -5,6 +5,7 @@ > */ > #include <xen/domain_page.h> > #include <xen/errno.h> > +#include <xen/guest_access.h> > #include <xen/gunzip.h> > #include <xen/init.h> > #include <xen/lib.h> > @@ -14,7 +15,6 @@ > #include <xen/vmap.h> > > #include <asm/byteorder.h> > -#include <asm/guest_access.h> > #include <asm/kernel.h> > #include <asm/setup.h>
Hi Stefano, On 30/07/2020 20:37, Stefano Stabellini wrote: > On Thu, 30 Jul 2020, Julien Grall wrote: >> From: Julien Grall <jgrall@amazon.com> >> >> We usually have xen/ includes first and then asm/. They are also ordered >> alphabetically among themselves. >> >> Signed-off-by: Julien Grall <jgrall@amazon.com> > > Might wanna mention the change from asm/guest_access.h to > xen/guest_access.h. Actually, the change in kernel.c belongs to patch #5. I will move it there. > Anyway: > > Acked-by: Stefano Stabellini <sstabellini@kernel.org> I will keep the Acked-by for the first change. Thank you,
diff --git a/xen/arch/arm/decode.c b/xen/arch/arm/decode.c index 8b1e15d11892..144793c8cea0 100644 --- a/xen/arch/arm/decode.c +++ b/xen/arch/arm/decode.c @@ -17,11 +17,12 @@ * GNU General Public License for more details. */ -#include <xen/types.h> +#include <xen/lib.h> #include <xen/sched.h> +#include <xen/types.h> + #include <asm/current.h> #include <asm/guest_access.h> -#include <xen/lib.h> #include "decode.h" diff --git a/xen/arch/arm/kernel.c b/xen/arch/arm/kernel.c index f95fa392af44..032923853f2c 100644 --- a/xen/arch/arm/kernel.c +++ b/xen/arch/arm/kernel.c @@ -5,6 +5,7 @@ */ #include <xen/domain_page.h> #include <xen/errno.h> +#include <xen/guest_access.h> #include <xen/gunzip.h> #include <xen/init.h> #include <xen/lib.h> @@ -14,7 +15,6 @@ #include <xen/vmap.h> #include <asm/byteorder.h> -#include <asm/guest_access.h> #include <asm/kernel.h> #include <asm/setup.h>