Changes in tools/ew.py [56a3c29e:e9f7778] 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, stderr = subprocess.STDOUT)
+                        subprocess.check_output('which ' + termemu, shell = True)
                         return termemu
                 except:
@@ -144 +144 @@
                         return 'system-arm', '-M raspi1ap'
         elif platform == 'arm64':
-                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
+                # 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)
@@ -173 +172 @@
                         raise Exception
                 if processor == 'us':
-                        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
+                        return 'system-sparc64', '-M sun4u --prom-env boot-args="console=devices/\\hw\\pci0\\01:01.0\\com1\\a"'
 
                 # processor = 'sun4v'
@@ -288 +284 @@
         if (is_override('nographic')):
                 cmdline += ' -nographic'
-                console = True
 
         if (not console and (not is_override('nographic')) and not is_override('noserial')):
@@ -331 +326 @@
 def msim_run(platform, machine, processor):
         hdisk_mk()
-        run_in_console('msim -n -c ' + TOOLS_DIR + '/conf/msim.conf', 'HelenOS/mips32 on msim')
+        run_in_console('msim -c ' + TOOLS_DIR + '/conf/msim.conf', 'HelenOS/mips32 on msim')
 
 def spike_run(platform, machine, processor):
@@ -414 +409 @@
                                 'image' : 'image.iso',
                                 'audio' : False,
-                                'console' : False,
+                                'console' : True,
                                 'net' : False,
                                 'usb' : False,
@@ -422 +417 @@
                         'sun4v' : {
                                 'run' : qemu_run,
-                                # 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',
+                                'image' : '-drive if=pflash,readonly=on,file=image.iso',
                                 'audio' : False,
                                 'console' : True,