Re: [Xen-devel] [PATCH 0/6] xen/arm: Move in/out code to/from init section

2015-02-02 Thread Julien Grall
On 02/02/15 13:12, Jan Beulich wrote: > This example in particular makes clear that looking at just the > declaration is insufficient in certain cases.Whoever is looking > for specific properties of a functions needs to know which ones > (s)he cares about, and then look in the appropriate place. _

Re: [Xen-devel] [PATCH 0/6] xen/arm: Move in/out code to/from init section

2015-02-02 Thread Jan Beulich
>>> On 02.02.15 at 13:52, wrote: > On 02/02/15 11:15, Jan Beulich wrote: > On 02.02.15 at 11:58, wrote: >>> On Fri, 2015-01-30 at 11:33 +, Julien Grall wrote: On 30/01/15 11:30, Ian Campbell wrote: > On Thu, 2015-01-29 at 18:32 +, Julien Grall wrote: >> Hello, >>

Re: [Xen-devel] [PATCH 0/6] xen/arm: Move in/out code to/from init section

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 12:48 +, Julien Grall wrote: > On 02/02/15 10:58, Ian Campbell wrote: > > On Fri, 2015-01-30 at 11:33 +, Julien Grall wrote: > >> Hi Ian, > >> > >> On 30/01/15 11:30, Ian Campbell wrote: > >>> On Thu, 2015-01-29 at 18:32 +, Julien Grall wrote: > Hello, >

Re: [Xen-devel] [PATCH 0/6] xen/arm: Move in/out code to/from init section

2015-02-02 Thread Julien Grall
On 02/02/15 11:15, Jan Beulich wrote: On 02.02.15 at 11:58, wrote: >> On Fri, 2015-01-30 at 11:33 +, Julien Grall wrote: >>> On 30/01/15 11:30, Ian Campbell wrote: On Thu, 2015-01-29 at 18:32 +, Julien Grall wrote: > Hello, > > Ping? Any more review for this version o

Re: [Xen-devel] [PATCH 0/6] xen/arm: Move in/out code to/from init section

2015-02-02 Thread Julien Grall
On 02/02/15 10:58, Ian Campbell wrote: > On Fri, 2015-01-30 at 11:33 +, Julien Grall wrote: >> Hi Ian, >> >> On 30/01/15 11:30, Ian Campbell wrote: >>> On Thu, 2015-01-29 at 18:32 +, Julien Grall wrote: Hello, Ping? Any more review for this version of this series? >>> >>> I w

Re: [Xen-devel] [PATCH 0/6] xen/arm: Move in/out code to/from init section

2015-02-02 Thread Jan Beulich
>>> On 02.02.15 at 11:58, wrote: > On Fri, 2015-01-30 at 11:33 +, Julien Grall wrote: >> On 30/01/15 11:30, Ian Campbell wrote: >> > On Thu, 2015-01-29 at 18:32 +, Julien Grall wrote: >> >> Hello, >> >> >> >> Ping? Any more review for this version of this series? >> > >> > I was awaiting

Re: [Xen-devel] [PATCH 0/6] xen/arm: Move in/out code to/from init section

2015-02-02 Thread Ian Campbell
On Fri, 2015-01-30 at 11:33 +, Julien Grall wrote: > Hi Ian, > > On 30/01/15 11:30, Ian Campbell wrote: > > On Thu, 2015-01-29 at 18:32 +, Julien Grall wrote: > >> Hello, > >> > >> Ping? Any more review for this version of this series? > > > > I was awaiting a version with the declaration

Re: [Xen-devel] [PATCH 0/6] xen/arm: Move in/out code to/from init section

2015-01-30 Thread Julien Grall
Hi Ian, On 30/01/15 11:30, Ian Campbell wrote: > On Thu, 2015-01-29 at 18:32 +, Julien Grall wrote: >> Hello, >> >> Ping? Any more review for this version of this series? > > I was awaiting a version with the declarations in the right places as > pointed out by Andy (including his point about

Re: [Xen-devel] [PATCH 0/6] xen/arm: Move in/out code to/from init section

2015-01-30 Thread Ian Campbell
On Thu, 2015-01-29 at 18:32 +, Julien Grall wrote: > Hello, > > Ping? Any more review for this version of this series? I was awaiting a version with the declarations in the right places as pointed out by Andy (including his point about definition vs. prototype I'm afraid, which is the style w

Re: [Xen-devel] [PATCH 0/6] xen/arm: Move in/out code to/from init section

2015-01-29 Thread Julien Grall
Hello, Ping? Any more review for this version of this series? Regards, On 16/01/15 16:20, Julien Grall wrote: > Hello, > > This small series add/remove __init on different functions. This allow Xen to > free around 8Kb more of memory after it has finished to boot. > > Xen size in memory before

[Xen-devel] [PATCH 0/6] xen/arm: Move in/out code to/from init section

2015-01-16 Thread Julien Grall
Hello, This small series add/remove __init on different functions. This allow Xen to free around 8Kb more of memory after it has finished to boot. Xen size in memory before to free init section: 1029Kb Freed memory init: 288Kb Regards, Julien Grall (6): arm/setup: Add missing __init to add_bo