Changes in tools/ew.py [e9f7778:56a3c29e] in mainline

File:

• tools/ew.py (modified) (7 diffs)

tools/ew.py

@@ -79 +79 @@
         for termemu in emus:
                 try:
-                        subprocess.check_output('which ' + termemu, shell = True)
+                        subprocess.check_output('which ' + termemu, shell = True, stderr = subprocess.STDOUT)
                         return termemu
                 except:
@@ -144 +144 @@
                         return 'system-arm', '-M raspi1ap'
         elif platform == 'arm64':
-                # Search for the EDK2 firmware image
-                default_paths = (
-                        '/usr/local/qemu-efi-aarch64/QEMU_EFI.fd', # Custom
-                        '/usr/share/edk2/aarch64/QEMU_EFI.fd',     # Fedora
-                        '/usr/share/qemu-efi-aarch64/QEMU_EFI.fd', # Ubuntu
-                )
-                extra_info = ("Pre-compiled binary can be obtained from "
-                    "http://snapshots.linaro.org/components/kernel/leg-virt-tianocore-edk2-upstream/latest/QEMU-AARCH64/RELEASE_GCC5/QEMU_EFI.fd.\n")
-                efi_path = find_firmware(
-                    "EDK2", 'EW_QEMU_EFI_AARCH64', default_paths, extra_info)
-                if efi_path is None:
-                        raise Exception
-
-                return 'system-aarch64', \
-                    '-M virt -cpu cortex-a57 -m 1024 -bios %s' % efi_path
+                if machine == 'virt':
+                        # Search for the EDK2 firmware image
+                        default_paths = (
+                                '/usr/local/qemu-efi-aarch64/QEMU_EFI.fd', # Custom
+                                '/usr/share/edk2/aarch64/QEMU_EFI.fd',     # Fedora
+                                '/usr/share/qemu-efi-aarch64/QEMU_EFI.fd', # Ubuntu
+                        )
+                        extra_info = ("Pre-compiled binary can be obtained from "
+                            "http://snapshots.linaro.org/components/kernel/leg-virt-tianocore-edk2-upstream/latest/QEMU-AARCH64/RELEASE_GCC5/QEMU_EFI.fd.\n")
+                        efi_path = find_firmware(
+                            "EDK2", 'EW_QEMU_EFI_AARCH64', default_paths, extra_info)
+                        if efi_path is None:
+                                raise Exception
+
+                        return 'system-aarch64', \
+                            '-M virt -cpu cortex-a57 -m 1024 -bios %s' % efi_path
         elif platform == 'ia32':
                 return 'system-i386', pc_options(32)
@@ -172 +173 @@
                         raise Exception
                 if processor == 'us':
-                        return 'system-sparc64', '-M sun4u --prom-env boot-args="console=devices/\\hw\\pci0\\01:01.0\\com1\\a"'
+                        cmdline = '-M sun4u'
+                        if is_override('nographic'):
+                            cmdline += ' --prom-env boot-args="console=devices/\\hw\\pci0\\01:01.0\\com1\\a"'
+                        return 'system-sparc64', cmdline
 
                 # processor = 'sun4v'
@@ -284 +288 @@
         if (is_override('nographic')):
                 cmdline += ' -nographic'
+                console = True
 
         if (not console and (not is_override('nographic')) and not is_override('noserial')):
@@ -326 +331 @@
 def msim_run(platform, machine, processor):
         hdisk_mk()
-        run_in_console('msim -c ' + TOOLS_DIR + '/conf/msim.conf', 'HelenOS/mips32 on msim')
+        run_in_console('msim -n -c ' + TOOLS_DIR + '/conf/msim.conf', 'HelenOS/mips32 on msim')
 
 def spike_run(platform, machine, processor):
@@ -409 +414 @@
                                 'image' : 'image.iso',
                                 'audio' : False,
-                                'console' : True,
+                                'console' : False,
                                 'net' : False,
                                 'usb' : False,
@@ -417 +422 @@
                         'sun4v' : {
                                 'run' : qemu_run,
-                                'image' : '-drive if=pflash,readonly=on,file=image.iso',
+                                # QEMU 8.0.0 ignores the 'file' argument and instead uses 'id',
+                                # which defaults to 'pflash0', but can be changed to the same value
+                                # as 'file'
+                                'image' : '-drive if=pflash,id=image.iso,readonly=on,file=image.iso',
                                 'audio' : False,
                                 'console' : True,