All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ian Campbell <Ian.Campbell@citrix.com>
To: Julien Grall <julien.grall@linaro.org>
Cc: xen-devel@lists.xenproject.org, tim@xen.org,
	stefano.stabellini@citrix.com
Subject: Re: [PATCH 0/6] xen/arm: Move in/out code to/from init section
Date: Fri, 30 Jan 2015 11:30:30 +0000	[thread overview]
Message-ID: <1422617430.12551.9.camel@citrix.com> (raw)
In-Reply-To: <54CA7CBB.7050308@linaro.org>

On Thu, 2015-01-29 at 18:32 +0000, 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 we use).

> 
> 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 to free init section: 1029Kb
> > Freed memory init: 288Kb
> > 
> > Regards,
> > 
> > Julien Grall (6):
> >   arm/setup: Add missing __init to add_boot_module
> >   xen/arm: domain_build: Move all DOM0 building code in init section
> >   xen/arm: kernel: Move kernel loading code in init section
> >   xen/arm: device: Move device_type in init section
> >   xen/arm: platforms: Move init_time and specific_mapping in init
> >     section
> >   xen/arm: SMP: Move out of the init section the code to bring up a CPU
> > 
> >  xen/arch/arm/arm32/smpboot.c         |  4 +--
> >  xen/arch/arm/device.c                |  2 +-
> >  xen/arch/arm/domain_build.c          | 66 ++++++++++++++++++------------------
> >  xen/arch/arm/kernel.c                | 30 ++++++++--------
> >  xen/arch/arm/kernel.h                |  4 +--
> >  xen/arch/arm/platform.c              |  2 +-
> >  xen/arch/arm/platforms/exynos5.c     |  4 +--
> >  xen/arch/arm/platforms/omap5.c       |  4 +--
> >  xen/arch/arm/platforms/xgene-storm.c | 16 ++++-----
> >  xen/arch/arm/setup.c                 |  6 ++--
> >  xen/arch/arm/smpboot.c               |  2 +-
> >  xen/include/asm-arm/device.h         |  2 +-
> >  xen/include/asm-arm/setup.h          | 10 +++---
> >  xen/include/asm-arm/smp.h            |  2 +-
> >  14 files changed, 77 insertions(+), 77 deletions(-)
> > 
> 
> 

  reply	other threads:[~2015-01-30 11:30 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-16 16:20 [PATCH 0/6] xen/arm: Move in/out code to/from init section Julien Grall
2015-01-16 16:20 ` [PATCH 1/6] arm/setup: Add missing __init to add_boot_module Julien Grall
2015-01-16 16:28   ` Andrew Cooper
2015-01-16 16:33     ` Julien Grall
2015-01-16 16:20 ` [PATCH 2/6] xen/arm: domain_build: Move all DOM0 building code in init section Julien Grall
2015-01-16 16:20 ` [PATCH 3/6] xen/arm: kernel: Move kernel loading " Julien Grall
2015-01-16 17:33   ` Vitaly Kuznetsov
2015-01-16 17:49     ` Julien Grall
2015-01-19 10:31       ` Ian Campbell
2015-01-16 16:20 ` [PATCH 4/6] xen/arm: device: Move device_type " Julien Grall
2015-01-16 16:20 ` [PATCH 5/6] xen/arm: platforms: Move init_time and specific_mapping " Julien Grall
2015-01-16 16:20 ` [PATCH 6/6] xen/arm: SMP: Move out of the init section the code to bring up a CPU Julien Grall
2015-01-29 18:32 ` [PATCH 0/6] xen/arm: Move in/out code to/from init section Julien Grall
2015-01-30 11:30   ` Ian Campbell [this message]
2015-01-30 11:33     ` Julien Grall
2015-02-02 10:58       ` Ian Campbell
2015-02-02 11:15         ` Jan Beulich
2015-02-02 12:52           ` Julien Grall
2015-02-02 13:12             ` Jan Beulich
2015-02-02 13:34               ` Julien Grall
2015-02-02 12:48         ` Julien Grall
2015-02-02 13:03           ` Ian Campbell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1422617430.12551.9.camel@citrix.com \
    --to=ian.campbell@citrix.com \
    --cc=julien.grall@linaro.org \
    --cc=stefano.stabellini@citrix.com \
    --cc=tim@xen.org \
    --cc=xen-devel@lists.xenproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.