diff options
| author | Matt Fleming <matt.fleming@intel.com> | 2012-02-16 12:12:50 +0000 |
|---|---|---|
| committer | Matt Fleming <matt.fleming@intel.com> | 2012-03-23 16:34:41 +0000 |
| commit | 51529126e71d1a2b451c6e809567e1ebd3788aa6 (patch) | |
| tree | 319e50d828ff053e7e769787516544573149474b /core/comboot.inc | |
| parent | 43499c9dc347ee858e9e395eba0954ec52a410ef (diff) | |
| download | syslinux-51529126e71d1a2b451c6e809567e1ebd3788aa6.tar.gz | |
core: Delete the aux segment
We don't need to use a special aux segment because we can represent
'fontbuf' with OFFS() and SEG(). We're guaranteed to be able to break
the 32-bit pointer up with these macros because we control where in
the address space the core is loaded. (Not all 32-bit pointers can be
represented with OFFS() and SEG(), only addresses in the range
0..0xFFFFF.)
This fixes the breakage that was introduced in commit 14531c47bc95
("core: Delete code that is duplicated in ldlinux").
This allows the default font to be displayed. Previously junk was
being returned in the COMBOOT API call to query the userfont, leading
the caller to believe that a user font was installed even when it
wasn't.
Signed-off-by: Matt Fleming <matt.fleming@intel.com>
Diffstat (limited to 'core/comboot.inc')
| -rw-r--r-- | core/comboot.inc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/core/comboot.inc b/core/comboot.inc index e0ad0681..38d65dd9 100644 --- a/core/comboot.inc +++ b/core/comboot.inc @@ -669,10 +669,12 @@ comapi_userfont: and al,al jz .done mov al,[VGAFontSize] - mov P_ES,aux_seg - mov P_BX,aux.fontbuf + pm_call pm_userfont + mov P_ES,es + mov P_BX,bx .done: ; CF=0 here + mov P_AL,al ret ; |