[prefix] Remove impossible progress message
authorMichael Brown <mcb30@ipxe.org>
Tue, 6 Dec 2016 07:36:33 +0000 (07:36 +0000)
committerMichael Brown <mcb30@ipxe.org>
Tue, 6 Dec 2016 07:44:10 +0000 (07:44 +0000)
commitce81601181f741e0ab63a77405d2eddec98576e9
tree4be6fa57eeaf9d72b997be4d83a5acc4aa840e28
parent6997d3c2fab72732d8af0955c9c760f7f5c4562b
[prefix] Remove impossible progress message

The "progress" macro can be used only from within the .prefix section.
At the point of calling relocate(), we are running in .text16 and so
the near call to print_message() will end up calling a random function
somewhere in .text16.

Interestingly, this problem has remained unnoticed for some time.  It
is rare to build with DEBUG=libprefix.  In the few cases that it has
been used during development, the randomly selected function in
.text16 seems to have been a harmless no-op with no visible
side-effects (beyond the unnoticed failure to print the "relocate"
progress message).

Fix by removing the futile attempt to print a progress message before
calling relocate().

Reported-by: Raed Salem <raeds@mellanox.com>
Signed-off-by: Michael Brown <mcb30@ipxe.org>
src/arch/x86/prefix/libprefix.S