[LinuxBIOS] [PATCH] Change payload buildrule to 'cp -f'

Richard Smith smithbone at gmail.com
Mon Jul 10 20:45:40 CEST 2006


> A common use of mkelfImage has been to put a bootable ELF image
> in /tftpboot.  In that scenario someone may be downloading the
> image when you are updating it.  Therefore to correctly update
> the file, you must first write to a new filename and then move
> it to the new filename.  Which gives you atomic update permissions.

Ah... Thanks for the clarification.

> The goal was to catch issues where people did not do that.

> That is exactly what was caught here.

Not quite.  This was just for a normal fallback build for the in-flash
kernel using a payload directive in the config file.

> If the problem was one where what you are updating needs to
> be writable I would agree that it is not a build script issue.
> However since you don't need write permissions and are just stomping
> on the file we are doing the wrong thing in the build.

Stefan.  I think Eric's right and the build script needs the tweak
regardless of the output of mkelfimage.  If the users umask is set
restrictive then the same problem would occur.
I noticed in the makefile that there are a few other cp's that
probably need to be cp -f as well.

Ok to commit?

-- 
Richard A. Smith




More information about the coreboot mailing list