OvmfPkg/Csm/LegacyBiosDxe: Fix Legacy16GetTableAddress call for E820 data master
authorDavid Woodhouse <dwmw2@infradead.org>
Thu, 13 Jun 2019 08:40:05 +0000 (16:40 +0800)
committerHao A Wu <hao.a.wu@intel.com>
Fri, 14 Jun 2019 05:05:49 +0000 (13:05 +0800)
commitb0663641c977f97bef785c86978603c3a31a3d1c
tree6691407c45c94113f4dfa3171edb4555c99e3756
parent3207a872a405ea60417853cfcdb73b4fa06b795f
OvmfPkg/Csm/LegacyBiosDxe: Fix Legacy16GetTableAddress call for E820 data

The DX register is supposed to contain the required alignment for the
allocation. It was zero, and SeaBIOS doesn't (well, didn't) cope well
with that. Set it appropriately.

Also set BX to indicate the regions it's OK to allocate in too. That
wasn't being initialised and was just using whatever the previous user
of the structure had left there.

Finally, actually return an error if the allocation fails. Instead of
going all the way through into the CSM and just letting it have a bogus
pointer to the E820 data.

Signed-off-by: David Woodhouse <dwmw2@infradead.org>
Reviewed-by: Hao A Wu <hao.a.wu@intel.com>
Acked-by: Laszlo Ersek <lersek@redhat.com>
OvmfPkg/Csm/LegacyBiosDxe/LegacyBootSupport.c