qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH] gtk: Replace bogus term "VGA" with "display"


From: Jan Kiszka
Subject: Re: [Qemu-devel] [PATCH] gtk: Replace bogus term "VGA" with "display"
Date: Fri, 22 Feb 2013 15:29:24 +0100
User-agent: Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666

On 2013-02-22 15:17, Anthony Liguori wrote:
> Jan Kiszka <address@hidden> writes:
> 
>> On 2013-02-22 14:21, Anthony Liguori wrote:
>>> Hi Jan,
>>>
>>> Jan Kiszka <address@hidden> writes:
>>>
>>>> VGA is a misnomer, only some of our targets have it. Use the more
>>>> generic term "display" instead.
>>>>
>>>> Signed-off-by: Jan Kiszka <address@hidden>
>>>
>>> Sorry, I responded to your previous note in my head and forgot to also
>>> respond on the mailing list.
>>>
>>> We expose this "misnomer" a lot.  For instance, qemu -vga cirrus.
>>
>> That's because qemu started for x86 only. No excuse to spread this
>> further in new code.
>>
>>>
>>> OTOH, the term "display" is used to indicate the backend via qemu
>>> -display sdl.
>>>
>>> So calling the VGA tab "Display" after selecting '-display gtk' seems a
>>> bit confusing to me.
>>>
>>> So I'm open to picking better naming but I think we should try to make
>>> it consistent throughout QEMU.
>>
>> We can also call it "screen" or "guest display" - just not VGA,
>> please.
> 
> I don't care what we call it, I'm asking that if you want to change it
> in one place, change it every where else too.

For obvious reasons, -vga has to stay. Also, I can leave the code as is.
My primary point is the user-visible misnomer "VGA". That should be
fixed before rolling it out. I'll trim down my patch to this.

Jan

-- 
Siemens AG, Corporate Technology, CT RTC ITP SDP-DE
Corporate Competence Center Embedded Linux



reply via email to

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