amluto / virtme

An easy way to virtualize the running system
GNU General Public License v2.0
330 stars 66 forks source link

s390x: avoid to set serial console #58

Open alicefr opened 4 years ago

alicefr commented 4 years ago

The serial console and the sclpconsole conflict with this error on s390x: qemu-system-s390x: -device sclpconsole,chardev=console: Multiple VT220 operator consoles are not supported qemu-system-s390x: -device sclpconsole,chardev=console: SCLP event initialization failed.

We can only enable the sclpconsole console

Signed-off-by: Alice Frosi afrosi@de.ibm.com

marcosps commented 4 years ago

LGTM. @amluto do you have a better idea to solve this issue?

amluto commented 4 years ago

I clearly don't understand what's going on here. Without this patch, we get this:

-device sclpconsole,chardev=console
-parallel none
-net none
-echr 1
-serial none
-chardev stdio,id=console,signal=off,mux=on
-serial chardev:console
-mon chardev=console

This does indeed fail, but removing -serial none doesn't help.

Want to make a new patch that wires this up in a somewhat sensible way through architectures.py and makes the correct change to the arguments? It looks like there should be a default function that supplies -serial chardev:console and an s390x override that supplies the -device sclpconsole,chardev=console.

marcosps commented 2 years ago

I tested the solution proposed in #75, and since it's more generic I believe that we can have it merged. What do you think?

arighi commented 1 year ago

This has been merged/fixed in https://github.com/arighi/virtme

NOTE: we are trying to catch up with all the pending PR in this temporary fork. In the future we may merge everything back here if this project become active again, but for now please consider to follow also the development of arighi/virtme.