| Message ID | 20200730181827.1670-5-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> Acked-by: Stefano Stabellini <sstabellini@kernel.org> > --- > xen/arch/arm/guestcopy.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/xen/arch/arm/guestcopy.c b/xen/arch/arm/guestcopy.c > index 7a0f3e9d5fc6..c8023e2bca5d 100644 > --- a/xen/arch/arm/guestcopy.c > +++ b/xen/arch/arm/guestcopy.c > @@ -1,7 +1,8 @@ > -#include <xen/lib.h> > #include <xen/domain_page.h> > +#include <xen/lib.h> > #include <xen/mm.h> > #include <xen/sched.h> > + > #include <asm/current.h> > #include <asm/guest_access.h> > > -- > 2.17.1 >
> On 30 Jul 2020, at 20:18, Julien Grall <julien@xen.org> 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> This could have been merged in patch 3. But anyway: Reviewed-by: Bertrand Marquis <bertrand.marquis@arm.com> > --- > xen/arch/arm/guestcopy.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/xen/arch/arm/guestcopy.c b/xen/arch/arm/guestcopy.c > index 7a0f3e9d5fc6..c8023e2bca5d 100644 > --- a/xen/arch/arm/guestcopy.c > +++ b/xen/arch/arm/guestcopy.c > @@ -1,7 +1,8 @@ > -#include <xen/lib.h> > #include <xen/domain_page.h> > +#include <xen/lib.h> > #include <xen/mm.h> > #include <xen/sched.h> > + > #include <asm/current.h> > #include <asm/guest_access.h> > > -- > 2.17.1 > > IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
> On 31 Jul 2020, at 14:53, Bertrand Marquis <Bertrand.Marquis@arm.com> wrote: > > > >> On 30 Jul 2020, at 20:18, Julien Grall <julien@xen.org> 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> > > This could have been merged in patch 3. > > But anyway: > Reviewed-by: Bertrand Marquis <bertrand.marquis@arm.com> > > >> --- >> xen/arch/arm/guestcopy.c | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/xen/arch/arm/guestcopy.c b/xen/arch/arm/guestcopy.c >> index 7a0f3e9d5fc6..c8023e2bca5d 100644 >> --- a/xen/arch/arm/guestcopy.c >> +++ b/xen/arch/arm/guestcopy.c >> @@ -1,7 +1,8 @@ >> -#include <xen/lib.h> >> #include <xen/domain_page.h> >> +#include <xen/lib.h> >> #include <xen/mm.h> >> #include <xen/sched.h> >> + >> #include <asm/current.h> >> #include <asm/guest_access.h> >> >> -- >> 2.17.1 >> >> > > IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. sorry for the notice, i need to find a way to turn it off automatically :-)
Hi, On 31/07/2020 13:53, Bertrand Marquis wrote: > > >> On 30 Jul 2020, at 20:18, Julien Grall <julien@xen.org> 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> > > This could have been merged in patch 3. This is not a tree-wide conversion (there are more instances in arch/arm), so I would prefer to keep one patch per file. > But anyway: > Reviewed-by: Bertrand Marquis <bertrand.marquis@arm.com> Thanks! Cheers,
diff --git a/xen/arch/arm/guestcopy.c b/xen/arch/arm/guestcopy.c index 7a0f3e9d5fc6..c8023e2bca5d 100644 --- a/xen/arch/arm/guestcopy.c +++ b/xen/arch/arm/guestcopy.c @@ -1,7 +1,8 @@ -#include <xen/lib.h> #include <xen/domain_page.h> +#include <xen/lib.h> #include <xen/mm.h> #include <xen/sched.h> + #include <asm/current.h> #include <asm/guest_access.h>