Bug 3700 - When inherit boot-directdisk class, the first build fail with error "disk full"
Summary: When inherit boot-directdisk class, the first build fail with error "disk full"
Status: RESOLVED NOTABUG
Alias: None
Product: BSPs
Classification: Build System, Metadata & Runtime
Component: bsps-oe-core (show other bugs)
Version: 1.3
Hardware: x86 Multiple
: Medium normal
Target Milestone: 1.3 M4
Assignee: Darren Hart
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2013-01-10 12:42 UTC by Julien Ciesla
Modified: 2013-01-11 22:45 UTC (History)
0 users

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Julien Ciesla 2013-01-10 12:42:00 UTC 
in the boot-directdisk.bbclass, the call to the mkdosfs fonction is specified with an incorrect calculaled value of the image size.

The size should be calculed with the HDDIMG variable and not the HDDDIR variable. This is the correction:

...
	install -m 0644 ${S}/syslinux.cfg ${HDDDIR}/syslinux.cfg
	install -m 444 ${STAGING_LIBDIR}/syslinux/ldlinux.sys ${HDDDIR}/ldlinux.sys

---BLOCKS=`du -bks ${HDDDIR} | cut -f 1`
+++BLOCKS=`du -bks ${HDDIMG} | cut -f 1`
	BLOCKS=`expr $BLOCKS + ${BOOTDD_EXTRA_SPACE}`

	# Ensure total sectors is an integral number of sectors per
	# track or mcopy will complain. Sectors are 512 bytes, and we
...

Tested working.

Regards
Comment 1 Darren Hart 2013-01-11 22:45:57 UTC 
Closing at the request of the reporter. I hadn't yet commented, but the fix noted here was definitely incorrect. This code could use a revamping along the lines of that in bootimg.bbclass, but this particular bug was opened in error.