From: "Philippe Mathieu-Daudé" <f4bug@amsat.org>
To: "Paolo Bonzini" <pbonzini@redhat.com>,
"Philippe Mathieu-Daudé" <philmd@redhat.com>,
"Markus Armbruster" <armbru@redhat.com>,
"BALATON Zoltan" <balaton@eik.bme.hu>,
"Peter Maydell" <peter.maydell@linaro.org>,
"Gerd Hoffmann" <kraxel@redhat.com>
Cc: Kevin Wolf <kwolf@redhat.com>,
Eduardo Habkost <ehabkost@redhat.com>,
qemu-devel@nongnu.org,
Aleksandar Markovic <amarkovic@wavecomp.com>,
Cleber Rosa <crosa@redhat.com>,
Aleksandar Rikalo <aleksandar.rikalo@rt-rk.com>,
John Snow <jsnow@redhat.com>,
Aurelien Jarno <aurelien@aurel32.net>
Subject: Re: [Qemu-devel] What should a virtual board emulate?
Date: Sun, 19 Jan 2020 12:36:20 +0100 [thread overview]
Message-ID: <36d5b922-b134-a5f4-8d9e-1ea3d87daedc@amsat.org> (raw)
In-Reply-To: <6b5459b5-deb2-e70c-deab-fa7246b43e38@redhat.com>
On 1/18/20 9:46 PM, Paolo Bonzini wrote:
> On 04/01/20 22:16, Philippe Mathieu-Daudé wrote:
>> 1/ the Radeon chip is soldered on the motherboard,
>>
>> 2/ the default BIOS expects the Radeon chip to be
>> unconditionally present,
>>
>> I insist this patch is incorrect for the particular case of the
>> Fuloong2e board. I plan to revert it when I post the test.
>>
>> BTW I'm not using --nodefault, I'm running default ./configure:
>>
>> qemu-system-mips64el -M fulong2e -bios pmon_2e.bin \
>> -display none -vga none -serial stdio
>
> But if you're not specifying -nodefaults, why are you specifying a
> configuration that your BIOS does not support? You should just remove
> -vga none and leave in -display none.
OK, with this information I searched what enforcing -vga none, with:
$ git grep -- -vga.*none python/
python/qemu/machine.py:232: args = ['-display', 'none', '-vga',
'none']
Context:
61 class QEMUMachine(object):
...
230
231 def _base_args(self):
232 args = ['-display', 'none', '-vga', 'none']
...
290 def launch(self):
301 try:
302 self._launch()
...
314 def _launch(self):
315 """
316 Launch the VM and establish a QMP connection
317 """
318 devnull = open(os.path.devnull, 'rb')
319 self._pre_launch()
320 self._qemu_full_args = (self._wrapper + [self._binary] +
321 self._base_args() + self._args)
Except the refactors moving this code, the original commit is:
$ git show 0fd05e8dd1e
commit 0fd05e8dd1ee7ae143fba3d6bcc6abe3fbeaeb34
Author: Paolo Bonzini <pbonzini@redhat.com>
Date: Wed Jun 6 16:23:27 2012 +0200
qemu-iotests: start vms in qtest mode
This way, they will not execute any VM code at all. However, right now
the cancellation test is "relying" on being slowed down by TCG executing
BIOS code. So, change the timeouts.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
diff --git a/tests/qemu-iotests/iotests.py b/tests/qemu-iotests/iotests.py
index e27b40e289..e05b1d640b 100644
--- a/tests/qemu-iotests/iotests.py
+++ b/tests/qemu-iotests/iotests.py
@@ -54,7 +54,9 @@ class VM(object):
self._qemu_log_path = os.path.join(test_dir, 'qemu-log.%d' %
os.getpid())
self._args = qemu_args + ['-chardev',
'socket,id=mon,path=' + self._monitor_path,
- '-mon', 'chardev=mon,mode=control', '-nographic']
+ '-mon', 'chardev=mon,mode=control',
+ '-qtest', 'stdio', '-machine', 'accel=qtest',
+ '-display', 'none', '-vga', 'none']
self._num_drives = 0
def add_drive(self, path, opts=''):
diff --git a/tests/qemu-iotests/030 b/tests/qemu-iotests/030
index 4ab7d62961..cc671dd7aa 100755
--- a/tests/qemu-iotests/030
+++ b/tests/qemu-iotests/030
@@ -147,7 +147,7 @@ class TestStreamStop(ImageStreamingTestCase):
result = self.vm.qmp('block-stream', device='drive0')
self.assert_qmp(result, 'return', {})
- time.sleep(1)
+ time.sleep(0.1)
events = self.vm.get_qmp_events(wait=False)
self.assertEqual(events, [], 'unexpected QMP event: %s' % events)
So IIUC we need to rework a bit the defaults of QEMUMachine::base_args.
next prev parent reply other threads:[~2020-01-19 11:37 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20190319144013.26584-1-pbonzini@redhat.com>
[not found] ` <20190319144013.26584-2-pbonzini@redhat.com>
2019-04-03 12:49 ` [Qemu-devel] [PATCH] sun4m: obey -vga none Philippe Mathieu-Daudé
[not found] ` <alpine.BSF.2.21.9999.1903192350550.11467@zero.eik.bme.hu>
[not found] ` <87imwe847x.fsf_-_@dusky.pond.sub.org>
[not found] ` <CAAdtpL5Z1OXqZ_zzA0RLq9soydNtTFt2oCOEfui1rq19bLKUgA@mail.gmail.com>
[not found] ` <d9d5ac96-c0f5-dd83-d305-10e20c745dd1@redhat.com>
2020-01-04 21:16 ` [Qemu-devel] What should a virtual board emulate? Philippe Mathieu-Daudé
2020-01-05 10:59 ` BALATON Zoltan
2020-01-06 6:45 ` Gerd Hoffmann
2020-01-10 19:42 ` Aleksandar Markovic
2020-01-10 19:56 ` Aleksandar Markovic
2020-01-18 20:46 ` Paolo Bonzini
2020-01-19 11:36 ` Philippe Mathieu-Daudé [this message]
2020-01-21 5:32 ` Markus Armbruster
2020-01-21 14:04 ` BALATON Zoltan
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=36d5b922-b134-a5f4-8d9e-1ea3d87daedc@amsat.org \
--to=f4bug@amsat.org \
--cc=aleksandar.rikalo@rt-rk.com \
--cc=amarkovic@wavecomp.com \
--cc=armbru@redhat.com \
--cc=aurelien@aurel32.net \
--cc=balaton@eik.bme.hu \
--cc=crosa@redhat.com \
--cc=ehabkost@redhat.com \
--cc=jsnow@redhat.com \
--cc=kraxel@redhat.com \
--cc=kwolf@redhat.com \
--cc=pbonzini@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=philmd@redhat.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.