Mailing List Archive: [PATCH 00/15] xen: arm: remove dummy.S

[PATCH 00/15] xen: arm: remove dummy.S

Ian.Campbell at citrix

Dec 4, 2012, 3:56 AM

Post #1 of 8 (686 views)

This was a short term hack to get something linking quickly, but its
usefulness has now passed.

This series replaces everything in here with proper functions. In many
cases these are still just stubs.

It seems to me that at least some of this stuff consists of x86-isms
which should instead be removed from the common code.

This highlights two large missing pieces of functionality: wallclock
time and cleaning up on domain destroy.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH 00/15] xen: arm: remove dummy.S [ In reply to ]

Dec 6, 2012, 4:09 AM

Post #2 of 8 (672 views)

At 11:56 +0000 on 04 Dec (1354622173), Ian Campbell wrote:
> This was a short term hack to get something linking quickly, but its
> usefulness has now passed.
>
> This series replaces everything in here with proper functions. In many
> cases these are still just stubs.

For the arch/arm parts,

Acked-by: Tim Deegan <tim@xen.org>

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH 00/15] xen: arm: remove dummy.S [ In reply to ]

Ian.Campbell at citrix

Dec 18, 2012, 9:14 AM

Post #3 of 8 (668 views)

Ping? Any comments on the ARM parts before I go through a rebase +
refresh cycle?

On Tue, 2012-12-04 at 11:56 +0000, Ian Campbell wrote:
> This was a short term hack to get something linking quickly, but its
> usefulness has now passed.
>
> This series replaces everything in here with proper functions. In many
> cases these are still just stubs.
>
> It seems to me that at least some of this stuff consists of x86-isms
> which should instead be removed from the common code.
>
> This highlights two large missing pieces of functionality: wallclock
> time and cleaning up on domain destroy.
>
> Ian.
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH 00/15] xen: arm: remove dummy.S [ In reply to ]

stefano.stabellini at eu

Dec 18, 2012, 10:31 AM

Post #4 of 8 (667 views)

I don't have any comments, they all look pretty straightforward.

On Tue, 18 Dec 2012, Ian Campbell wrote:
> Ping? Any comments on the ARM parts before I go through a rebase +
> refresh cycle?
>
> On Tue, 2012-12-04 at 11:56 +0000, Ian Campbell wrote:
> > This was a short term hack to get something linking quickly, but its
> > usefulness has now passed.
> >
> > This series replaces everything in here with proper functions. In many
> > cases these are still just stubs.
> >
> > It seems to me that at least some of this stuff consists of x86-isms
> > which should instead be removed from the common code.
> >
> > This highlights two large missing pieces of functionality: wallclock
> > time and cleaning up on domain destroy.
> >
> > Ian.
> >
> >
> > _______________________________________________
> > Xen-devel mailing list
> > Xen-devel@lists.xen.org
> > http://lists.xen.org/xen-devel
>
>
>

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH 00/15] xen: arm: remove dummy.S [ In reply to ]

Ian.Campbell at citrix

Dec 18, 2012, 12:54 PM

Post #5 of 8 (661 views)

On Tue, 2012-12-18 at 18:31 +0000, Stefano Stabellini wrote:
> I don't have any comments, they all look pretty straightforward.

Is that an Acked-by: Stefano on all 15?

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH 00/15] xen: arm: remove dummy.S [ In reply to ]

stefano.stabellini at eu

Dec 19, 2012, 3:21 AM

Post #6 of 8 (663 views)

On Tue, 18 Dec 2012, Ian Campbell wrote:
> On Tue, 2012-12-18 at 18:31 +0000, Stefano Stabellini wrote:
> > I don't have any comments, they all look pretty straightforward.
>
> Is that an Acked-by: Stefano on all 15?
>

Yes

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH 00/15] xen: arm: remove dummy.S [ In reply to ]

Ian.Campbell at citrix

Dec 19, 2012, 5:27 AM

Post #7 of 8 (664 views)

On Thu, 2012-12-06 at 12:09 +0000, Tim Deegan wrote:
> At 11:56 +0000 on 04 Dec (1354622173), Ian Campbell wrote:
> > This was a short term hack to get something linking quickly, but its
> > usefulness has now passed.
> >
> > This series replaces everything in here with proper functions. In many
> > cases these are still just stubs.
>
> For the arch/arm parts,
>
> Acked-by: Tim Deegan <tim@xen.org>

I somehow missed this in my ping yesterday, sorry!

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH 00/15] xen: arm: remove dummy.S [ In reply to ]

Ian.Campbell at citrix

Dec 19, 2012, 6:03 AM

Post #8 of 8 (663 views)

On Tue, 2012-12-04 at 11:56 +0000, Ian Campbell wrote:
> This was a short term hack to get something linking quickly, but its
> usefulness has now passed.
>
> This series replaces everything in here with proper functions. In many
> cases these are still just stubs.
>
> It seems to me that at least some of this stuff consists of x86-isms
> which should instead be removed from the common code.
>
> This highlights two large missing pieces of functionality: wallclock
> time and cleaning up on domain destroy.

Applied 1..4,6..14 with Tim+Stefano's acks. Thanks.

5 was the patch which touched x86 which I need to refresh after Jan's
comment. 15 is the final removal of dummy.S which needs 5.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel