* [PATCH V2] xen/arm64: Use virtual address when setting up early_printk fixmap
@ 2015-04-07 11:24 Chen Baozi
2015-04-08 11:33 ` Julien Grall
2015-05-05 15:06 ` Julien Grall
0 siblings, 2 replies; 6+ messages in thread
From: Chen Baozi @ 2015-04-07 11:24 UTC (permalink / raw)
To: xen-devel; +Cc: Chen Baozi, julien.grall, ian.campbell
We have already had the boot pagetable when reaching the point
of early_printk fixmap setup. Thus there is no longer necessary
to calculate physical address of xen_fixmap.
Signed-off-by: Chen Baozi <baozich@gmail.com>
---
xen/arch/arm/arm64/head.S | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/xen/arch/arm/arm64/head.S b/xen/arch/arm/arm64/head.S
index 75b7ee1..9ed9a93 100644
--- a/xen/arch/arm/arm64/head.S
+++ b/xen/arch/arm/arm64/head.S
@@ -519,8 +519,7 @@ paging:
cbnz x22, 1f
/* Add UART to the fixmap table */
- ldr x1, =xen_fixmap
- add x1, x1, x20 /* x1 := paddr (xen_fixmap) */
+ ldr x1, =xen_fixmap /* x1 := vaddr (xen_fixmap) */
lsr x2, x23, #THIRD_SHIFT
lsl x2, x2, #THIRD_SHIFT /* 4K aligned paddr of UART */
mov x3, #PT_DEV_L3
--
2.1.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH V2] xen/arm64: Use virtual address when setting up early_printk fixmap
2015-04-07 11:24 [PATCH V2] xen/arm64: Use virtual address when setting up early_printk fixmap Chen Baozi
@ 2015-04-08 11:33 ` Julien Grall
2015-04-15 14:41 ` Ian Campbell
2015-05-05 15:06 ` Julien Grall
1 sibling, 1 reply; 6+ messages in thread
From: Julien Grall @ 2015-04-08 11:33 UTC (permalink / raw)
To: Chen Baozi, xen-devel; +Cc: julien.grall, ian.campbell
Hi Chen,
On 07/04/15 12:24, Chen Baozi wrote:
> We have already had the boot pagetable when reaching the point
s/had/added/ ?
> of early_printk fixmap setup. Thus there is no longer necessary
s/there is/it is/ ?
> to calculate physical address of xen_fixmap.
>
> Signed-off-by: Chen Baozi <baozich@gmail.com>
Other than the possible typoes above:
Reviewed-by: Julien Grall <julien.grall@citrix.com>
Regards,
--
Julien Grall
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH V2] xen/arm64: Use virtual address when setting up early_printk fixmap
2015-04-08 11:33 ` Julien Grall
@ 2015-04-15 14:41 ` Ian Campbell
2015-04-17 11:39 ` Chen Baozi
0 siblings, 1 reply; 6+ messages in thread
From: Ian Campbell @ 2015-04-15 14:41 UTC (permalink / raw)
To: Julien Grall; +Cc: Chen Baozi, julien.grall, xen-devel
On Wed, 2015-04-08 at 12:33 +0100, Julien Grall wrote:
> Hi Chen,
>
> On 07/04/15 12:24, Chen Baozi wrote:
> > We have already had the boot pagetable when reaching the point
>
> s/had/added/ ?
I think "switched too" is what was meant and more correct.
> > of early_printk fixmap setup. Thus there is no longer necessary
>
> s/there is/it is/ ?
Yes.
> > to calculate physical address of xen_fixmap.
> >
> > Signed-off-by: Chen Baozi <baozich@gmail.com>
>
> Other than the possible typoes above:
>
> Reviewed-by: Julien Grall <julien.grall@citrix.com>
Acked-by: Ian Campbell <ian.campbell@citrix.com>
If it is confirmed that I got the correct meaning in the first one I can
fix both on commit.
Ian.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH V2] xen/arm64: Use virtual address when setting up early_printk fixmap
2015-04-15 14:41 ` Ian Campbell
@ 2015-04-17 11:39 ` Chen Baozi
2015-05-05 15:36 ` Ian Campbell
0 siblings, 1 reply; 6+ messages in thread
From: Chen Baozi @ 2015-04-17 11:39 UTC (permalink / raw)
To: Ian Campbell; +Cc: Julien Grall, julien.grall, xen-devel
On Wed, Apr 15, 2015 at 03:41:44PM +0100, Ian Campbell wrote:
> On Wed, 2015-04-08 at 12:33 +0100, Julien Grall wrote:
> > Hi Chen,
> >
> > On 07/04/15 12:24, Chen Baozi wrote:
> > > We have already had the boot pagetable when reaching the point
> >
> > s/had/added/ ?
>
> I think "switched too" is what was meant and more correct.
>
So do I ;-)
> > > of early_printk fixmap setup. Thus there is no longer necessary
> >
> > s/there is/it is/ ?
>
> Yes.
>
> > > to calculate physical address of xen_fixmap.
> > >
> > > Signed-off-by: Chen Baozi <baozich@gmail.com>
> >
> > Other than the possible typoes above:
> >
> > Reviewed-by: Julien Grall <julien.grall@citrix.com>
>
> Acked-by: Ian Campbell <ian.campbell@citrix.com>
>
> If it is confirmed that I got the correct meaning in the first one I can
> fix both on commit.
Yes, please. Thanks.
Baozi
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH V2] xen/arm64: Use virtual address when setting up early_printk fixmap
2015-04-07 11:24 [PATCH V2] xen/arm64: Use virtual address when setting up early_printk fixmap Chen Baozi
2015-04-08 11:33 ` Julien Grall
@ 2015-05-05 15:06 ` Julien Grall
1 sibling, 0 replies; 6+ messages in thread
From: Julien Grall @ 2015-05-05 15:06 UTC (permalink / raw)
To: ian.campbell; +Cc: Chen Baozi, julien.grall, xen-devel
Hi Ian,
On 07/04/15 12:24, Chen Baozi wrote:
> We have already had the boot pagetable when reaching the point
> of early_printk fixmap setup. Thus there is no longer necessary
> to calculate physical address of xen_fixmap.
>
> Signed-off-by: Chen Baozi <baozich@gmail.com>
This patch has been fully ack and Chen agreed that you fix the typoes.
Can you commit it?
Cheers,
> ---
> xen/arch/arm/arm64/head.S | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/xen/arch/arm/arm64/head.S b/xen/arch/arm/arm64/head.S
> index 75b7ee1..9ed9a93 100644
> --- a/xen/arch/arm/arm64/head.S
> +++ b/xen/arch/arm/arm64/head.S
> @@ -519,8 +519,7 @@ paging:
> cbnz x22, 1f
>
> /* Add UART to the fixmap table */
> - ldr x1, =xen_fixmap
> - add x1, x1, x20 /* x1 := paddr (xen_fixmap) */
> + ldr x1, =xen_fixmap /* x1 := vaddr (xen_fixmap) */
> lsr x2, x23, #THIRD_SHIFT
> lsl x2, x2, #THIRD_SHIFT /* 4K aligned paddr of UART */
> mov x3, #PT_DEV_L3
>
--
Julien Grall
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH V2] xen/arm64: Use virtual address when setting up early_printk fixmap
2015-04-17 11:39 ` Chen Baozi
@ 2015-05-05 15:36 ` Ian Campbell
0 siblings, 0 replies; 6+ messages in thread
From: Ian Campbell @ 2015-05-05 15:36 UTC (permalink / raw)
To: Chen Baozi; +Cc: Julien Grall, julien.grall, xen-devel
On Fri, 2015-04-17 at 19:39 +0800, Chen Baozi wrote:
> >
> > Acked-by: Ian Campbell <ian.campbell@citrix.com>
> >
> > If it is confirmed that I got the correct meaning in the first one I can
> > fix both on commit.
>
> Yes, please. Thanks.
Done and pushed, thanks.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-05-05 15:36 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-04-07 11:24 [PATCH V2] xen/arm64: Use virtual address when setting up early_printk fixmap Chen Baozi
2015-04-08 11:33 ` Julien Grall
2015-04-15 14:41 ` Ian Campbell
2015-04-17 11:39 ` Chen Baozi
2015-05-05 15:36 ` Ian Campbell
2015-05-05 15:06 ` Julien Grall
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.