From: Robert Yang <liezhi.yang@windriver.com>
To: <openembedded-core@lists.openembedded.org>
Cc: dvhart@linux.intel.com
Subject: [PATCH 0/6] replace genext2fs with populate-extfs.sh
Date: Thu, 22 Aug 2013 09:13:08 -0400 [thread overview]
Message-ID: <cover.1377175027.git.liezhi.yang@windriver.com> (raw)
* The benefits:
- Really support ext4
- Support the sparse file (we lost the sparse file in the image in the
past, the sparse file became into the common file)
- Fix the error reported by fsck: (ext2/ext3)
"Inode 1025, i_size is 16384, should be 17408."
- Have a uniform code for ext2/3/4 generation
* Impact
- Build time:
a) If we build fresh core-image-sato, there is nearly no impact.
b) If we do the image generation, which means:
$ bitbake core-image-sato
$ bitbake core-image-sato -ccleansstate
$ bitbake core-image-sato
About 40 extra seconds are needed, here is my test result:
Before the patches: 4m53s
After the patches: 5m35s
This is because the genext2fs is much faster than the
populate-extfs.sh, we will replace this script by the mke2fs when it
supports create the filesystem from a initial directory.
- Disk space (take core-image-sato as an example)
a) The image file size is the same as before (519M)
b) The disk usage is a little different: (du -sh)
before now
ext2: 356M 379M
ext3: 372M 395M
ext4: 372M 380M
I have done some simple runtime testing on core-image-sato based on
ext2/3/4, they worked well.
* The "fsck.extX -fn <image>.extX" shows it is OK.
// Robert
The following changes since commit b2ff1add530b1fec2fb7f385227a03db47015c37:
poky.conf: Don't force the addition of extra DISTRO_FEATURES (2013-08-20 22:58:04 +0100)
are available in the git repository at:
git://git.pokylinux.org/poky-contrib robert/extX
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=robert/extX
Robert Yang (6):
e2fsprogs: the max length of debugfs argument is too short
e2fsprogs: let debugfs do sparse copy
e2fsprogs: only update the icache for ext2_inode
e2fsprogs: properly set up extent header in do_write
e2fsprogs: add populate-extfs.sh
image_types.bbclass: replace genext2fs with populate-extfs.sh
meta/classes/image_types.bbclass | 46 +++---
.../e2fsprogs-1.42.8/debugfs-extent-header.patch | 47 +++++++
.../e2fsprogs-1.42.8/debugfs-sparse-copy.patch | 147 ++++++++++++++++++++
.../e2fsprogs-1.42.8/debugfs-too-short.patch | 41 ++++++
.../e2fsprogs/e2fsprogs-1.42.8/fix-icache.patch | 69 +++++++++
.../e2fsprogs/e2fsprogs-1.42.8/populate-extfs.sh | 96 +++++++++++++
.../recipes-devtools/e2fsprogs/e2fsprogs_1.42.8.bb | 6 +
7 files changed, 424 insertions(+), 28 deletions(-)
create mode 100644 meta/recipes-devtools/e2fsprogs/e2fsprogs-1.42.8/debugfs-extent-header.patch
create mode 100644 meta/recipes-devtools/e2fsprogs/e2fsprogs-1.42.8/debugfs-sparse-copy.patch
create mode 100644 meta/recipes-devtools/e2fsprogs/e2fsprogs-1.42.8/debugfs-too-short.patch
create mode 100644 meta/recipes-devtools/e2fsprogs/e2fsprogs-1.42.8/fix-icache.patch
create mode 100644 meta/recipes-devtools/e2fsprogs/e2fsprogs-1.42.8/populate-extfs.sh
--
1.7.10.4
next reply other threads:[~2013-08-22 13:13 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-22 13:13 Robert Yang [this message]
2013-08-22 13:13 ` [PATCH 1/6] e2fsprogs: the max length of debugfs argument is too short Robert Yang
2013-08-22 14:48 ` Darren Hart
2013-08-22 13:13 ` [PATCH 2/6] e2fsprogs: let debugfs do sparse copy Robert Yang
2013-08-22 14:52 ` Darren Hart
2013-08-23 2:06 ` Robert Yang
2013-08-23 2:33 ` Rongqing Li
2013-08-23 6:45 ` Robert Yang
2013-08-23 17:06 ` Darren Hart
2013-08-26 1:39 ` Robert Yang
2013-08-22 13:13 ` [PATCH 3/6] e2fsprogs: only update the icache for ext2_inode Robert Yang
2013-08-22 17:23 ` Darren Hart
2013-08-23 1:58 ` Robert Yang
2013-08-23 17:03 ` Darren Hart
2013-08-22 13:13 ` [PATCH 4/6] e2fsprogs: properly set up extent header in do_write Robert Yang
2013-08-22 17:24 ` Darren Hart
2013-08-23 2:02 ` Robert Yang
2013-08-23 17:04 ` Darren Hart
2013-08-22 13:13 ` [PATCH 5/6] e2fsprogs: add populate-extfs.sh Robert Yang
2013-08-22 17:29 ` Darren Hart
2013-08-23 2:05 ` Robert Yang
2013-08-22 13:13 ` [PATCH 6/6] image_types.bbclass: replace genext2fs with populate-extfs.sh Robert Yang
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=cover.1377175027.git.liezhi.yang@windriver.com \
--to=liezhi.yang@windriver.com \
--cc=dvhart@linux.intel.com \
--cc=openembedded-core@lists.openembedded.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox