]> code.ossystems Code Review - openembedded-core.git/commit
bootimg: Use mcopy to construct the hddimg
authorDarren Hart <dvhart@linux.intel.com>
Fri, 16 Dec 2011 06:14:19 +0000 (22:14 -0800)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Fri, 16 Dec 2011 15:57:52 +0000 (15:57 +0000)
commit92d2ea1a306354c6565a1b05b51b5719e481840f
tree99d9b9fa9f4682796144d06114905acd2f4de198
parent5d2554ec5f1d3b15e02b1d76cc9bd2d4861969f8
bootimg: Use mcopy to construct the hddimg

The initial directory support (-d) added to mkdosfs has proven to be incomplete
and non-compliant with FAT. Rather than continue to maintain this feature and
work around the various issues, we can use mcopy to construct the image.

bootimg.bbclass already depends on mtools-native (although it may not have
needed to previously). No new dependencies are introduced. The image created
passes dosfsck cleanly. Remove the call to dosfsck.

mcopy reported an error with the image we were creating:
Total number of sectors (107574) not a multiple of sectors per track (32)!

Add some logic to ensure the total sector count is an integral number of sectors
per track, including forcing the logical sector size to 512 in the mkdosfs
command.

The du -bks arguments are contradictory, -b is equivalent to "--apparent-size
--block-size=1" and -k is --block-size=1K. If reordered, -kbs will report the
disk usage in bytes insteadk of 1k blocks. Eliminate the ambiguity by using:
du --apparent-size -ks

Signed-off-by: Darren Hart <dvhart@linux.intel.com>
CC: Nitin A. Kamble <nitin.a.kamble@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/classes/bootimg.bbclass