[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[xen staging] Revert "tools/firmware/ovmf: Use OvmfXen platform file is exist"



commit c7691f5e340f3b579d40c77548f63133cdf5aac7
Author:     Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
AuthorDate: Tue Jun 22 16:14:53 2021 +0100
Commit:     Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
CommitDate: Tue Jun 22 18:00:08 2021 +0100

    Revert "tools/firmware/ovmf: Use OvmfXen platform file is exist"
    
    This reverts commit aad7b5c11d51d57659978e04702ac970906894e8.
    
    The change from OvmfX64 to OvmfXen causes a change in behaviour, whereby
    OvmfXen maps its shared info page at the top of address space.  When trying 
to
    migrate such a domain, XENMEM_maximum_gpfn returns a very large value.  This
    has uncovered multiple issues:
    
     1) The userspace hypercall wrappers truncate all return values to int on
        Linux and Solaris, even on 64bit.  This needs fixing in libxenctrl.
     2) 32bit toolstacks can't migrate any domain with RAM above the 2^40 mark,
        because of virtual address constraints.  This needs fixing in OVMF.
    
    Fixes for both of these aren't completely trivial.  Revert the change to
    unblock staging in the meantime.
    
    Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
    Acked-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>
---
 tools/firmware/ovmf-makefile | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/tools/firmware/ovmf-makefile b/tools/firmware/ovmf-makefile
index 637ee509c3..55f9992145 100644
--- a/tools/firmware/ovmf-makefile
+++ b/tools/firmware/ovmf-makefile
@@ -17,14 +17,8 @@ all: build
 .PHONY: build
 build:
        if test -e .git ; then $(GIT) submodule update --init --recursive ; fi
-       set -ex; \
-       if test -e OvmfPkg/OvmfXen.dsc; then \
-         OvmfPkg/build.sh -a X64 -b $(TARGET) -n 4 -p OvmfPkg/OvmfXen.dsc; \
-         cp Build/OvmfXen/$(TARGET)_GCC*/FV/OVMF.fd ovmf.bin; \
-       else \
-         OvmfPkg/build.sh -a X64 -b $(TARGET) -n 4; \
-         cp Build/OvmfX64/$(TARGET)_GCC*/FV/OVMF.fd ovmf.bin; \
-       fi
+       OvmfPkg/build.sh -a X64 -b $(TARGET) -n 4
+       cp Build/OvmfX64/$(TARGET)_GCC*/FV/OVMF.fd ovmf.bin
 
 .PHONY: clean
 clean:
--
generated by git-patchbot for /home/xen/git/xen.git#staging



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.