All of lore.kernel.org
 help / color / mirror / Atom feed
From: Saul Wold <saul.wold@intel.com>
To: Dongxiao Xu <dongxiao.xu@intel.com>
Cc: "poky@yoctoproject.org" <poky@yoctoproject.org>
Subject: Re: [PATCH 0/3][RFC] Poky disk space size optimization
Date: Mon, 06 Dec 2010 18:47:16 -0800	[thread overview]
Message-ID: <4CFDA034.9050703@intel.com> (raw)
In-Reply-To: <cover.1291683028.git.dongxiao.xu@intel.com>

On 12/06/2010 04:50 PM, Dongxiao Xu wrote:
> This RFC aims to reduce the poky disk space size.
>
> The optimization point is the duplicated file copies in WORKDIR and
> sysroot directories. When installing, files will be copied from source
> code package into the image folder; Then some files will be copied
> from image folder to sysroot-destdir, and some other files will be
> copied from image folder to package folder. While doing package split,
> some files will be copied from package folder to packages-split folder,
> etc.
>
There are a couple of concerns with this method, which may lead to 
problems, first is that using links and modifying permissions, it's 
possible that depending on the usage the permissions might be set 
differently.  This will could lead to a second problem: psuedo. It's 
unclear at this point how psuedo will handle this.

I am sure you have been doing testing, just not sure of corner cases.

I know we accepted the first change for the package split link to the 
deploy area, that one made sense as it was package level, this is file 
level and may have hidden problems.

I am sure that Richard will correct me and add his comments as he 
recovers from his travels.

Sau!

> The thought is to use hard link to replace the direct copy.
> The three commits do the work of:
>
> a) package.bbclass: hard link "image-->sysroot-destdir"
> b) sstate.bbclass:  hard link "package-->packages-split"
> c) staging.bbclass: hard link "sysroot-destdir-->sysroots".
>
> After testing with minimal/sato/sdk build, this approach could save
> about 20% disk space.
> (minimal: 23G ->  19G, sato: 44G ->  35G, sdk: 54G ->  44G).
>
> Pull URL: git://git.pokylinux.org/poky-contrib.git
>    Branch: dxu4/perf
>    Browse: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=dxu4/perf
>
> Thanks,
>      Dongxiao Xu<dongxiao.xu@intel.com>
> ---
>
>
> Dongxiao Xu (3):
>    package.bbclass: Use hard link for package split instead of copy
>    sstate.bbclass: Use hard link when staging files into sysroots
>    staging.bbclass: use hardlink when staging files to sysroot-destdir
>
>   meta/classes/package.bbclass |    8 +++++---
>   meta/classes/sstate.bbclass  |    2 +-
>   meta/classes/staging.bbclass |   27 ++++++++++++++++++++++++++-
>   meta/lib/oe/path.py          |   28 ++++++++++++++++++++++++++++
>   4 files changed, 60 insertions(+), 5 deletions(-)
>
> _______________________________________________
> poky mailing list
> poky@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/poky
>



  parent reply	other threads:[~2010-12-07  2:47 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-12-07  0:50 [PATCH 0/3][RFC] Poky disk space size optimization Dongxiao Xu
2010-12-06 12:26 ` [PATCH 1/3] package.bbclass: Use hard link for package split instead of copy Dongxiao Xu
2010-12-06 12:36 ` [PATCH 2/3] sstate.bbclass: Use hard link when staging files into sysroots Dongxiao Xu
2010-12-06 12:37 ` [PATCH 3/3] staging.bbclass: use hardlink when staging files to sysroot-destdir Dongxiao Xu
2010-12-07  2:47 ` Saul Wold [this message]
2010-12-07 11:15   ` [PATCH 0/3][RFC] Poky disk space size optimization Tian, Kevin
2010-12-08  6:19     ` Xu, Dongxiao

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=4CFDA034.9050703@intel.com \
    --to=saul.wold@intel.com \
    --cc=dongxiao.xu@intel.com \
    --cc=poky@yoctoproject.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.