qemu-arm
[Top][All Lists]
Advanced

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

Re: [PATCH 1/2] docs: sbsa: correct graphics card name


From: Mark Cave-Ayland
Subject: Re: [PATCH 1/2] docs: sbsa: correct graphics card name
Date: Thu, 25 May 2023 12:06:48 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.0

On 25/05/2023 11:44, Peter Maydell wrote:

On Thu, 25 May 2023 at 11:32, Thomas Huth <thuth@redhat.com> wrote:

On 25/05/2023 12.05, Peter Maydell wrote:
On Tue, 23 May 2023 at 19:41, Thomas Huth <thuth@redhat.com> wrote:

On 23/05/2023 19.30, Marcin Juszkiewicz wrote:
...
(is there a reason why it can't be disabled with "-vga none" or
"-nodefaults"?)

That's something I need to check how it should be done.

Other boards set mc->default_display in their ...class_init
function and then use pci_vga_init() (or vga_interface_type)
to instantiate their default display adapter ... however, that
does not seem to support the bochs adapter yet (see
vga_interfaces[] in softmmu/vl.c).

Not sure whether it's worth the effort to extend vga_interfaces[]
in vl.c

Isn't that a legacy-command-line-option thing? I feel like
the recommended way to select a different graphics card
these days would be to use -device my-pci-vga-card ...

"-vga" is kind of legacy, indeed, but currently the sbsa-ref hard-codes the
graphics card to be always available, so if you add a "-device
something-vga-card" on the command line, you'd get two graphic cards on your
machine, even if you use -nodefaults.

At least some boards do "only create the default graphics
type if vga_interface_type != VGA_NONE".

Mostly what I would like here is consistency. But also, we
haven't added a new item to the '-vga' option list since
2014, which makes me strongly suspect it's legacy and we
should only be keeping it for backwards compatibility.

So there needs to be at least some logic dealing with vga_interface_type if
we want to be able to select a different graphics card for this machine.
Then why not go the full way and use pci_vga_init() here, too? ... that's
certainly the least confusing way for the users.

Is it? From an Arm perspective, having "-vga" do anything
at all is pretty confusing: it's a rather PC-centric option name.
(Also pretty noticeable for the Sparc TCX/CG3 framebuffers,
which are not VGA in any way.)

Right. From the SPARC perspective it was added to allow the user to select either the TCX (default) or CG3 framebuffers from the command line.

However I guess that shouldn't be needed anymore now that mc->default_display exists. Presumably there is now some kind of -global sun4m.default_display=cg3 command line option that could set the machine default_display property value instead?


ATB,

Mark.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]