瀏覽代碼

[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>
tags/v1.20.1
Michael Brown 8 年之前
父節點
當前提交
ce81601181
共有 1 個文件被更改,包括 0 次插入1 次删除
  1. 0
    1
      src/arch/x86/prefix/libprefix.S

+ 0
- 1
src/arch/x86/prefix/libprefix.S 查看文件

@@ -873,7 +873,6 @@ install_prealloc:
873 873
 	 * relocate() will return with %esi, %edi and %ecx set up
874 874
 	 * ready for the copy to the new location.
875 875
 	 */
876
-	progress "  relocate\n"
877 876
 	virtcall relocate
878 877
 
879 878
 	/* Jump back to .prefix segment */

Loading…
取消
儲存