qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [RFC PATCH 0/2] qga: add guest-get-os-version for windo


From: zhanghailiang
Subject: Re: [Qemu-devel] [RFC PATCH 0/2] qga: add guest-get-os-version for windows
Date: Tue, 16 Dec 2014 19:06:43 +0800
User-agent: Mozilla/5.0 (Windows NT 6.1; rv:31.0) Gecko/20100101 Thunderbird/31.1.1

On 2014/12/16 17:38, Yan Vugenfirer wrote:
Hi,

My suggestion is to handle the case when the newer OS will be installed in the 
guest as well.


Yes, we can do this when we install guest OS, save the version info into some 
places,
and get the info when we need, But it is not so flexible, especially when we 
manage the OS image directly,
we have to record the relationship between image name and its OS version, 
besides, we need to apply an API to
return image's OS version.

So IMHO, get the OS version info from guest and use qga command to get the info 
are more flexible and reliable.


Please look at version helper API - 
http://msdn.microsoft.com/en-us/library/windows/desktop/dn424972(v=vs.85).aspx 
<http://msdn.microsoft.com/en-us/library/windows/desktop/dn424972(v=vs.85).aspx>
and keep in mind that the usual GetVersion and GetVersionEx (that returned 
major, minor versions and build number) will be deprecated or changed after 
Windows 8.1.


Hmm, i noticed this ;), i have handle it specially for windows 8.1 in this 
patch.
We can still get its real version by other method.

And here i just add a framework for get-os-version, we can improve it when new 
OS comes out.

Thanks,
zhanghailiang



On Dec 16, 2014, at 9:30 AM, zhanghailiang <address@hidden> wrote:

Hi,

This patch series add a new guest command 'guest-get-os-version'.
It is now only available for windows guest.

It will return guest's OS version name and type, like bellow:
'{"return":{"name":"Microsoft Windows Server 2012 R2","type":64}}'

Sometimes we need to know guest's OS version info.
(Actually, we need this info when we update guest's applications and drivers
in our project.)

This patch is only RFC, it is another try to add more commands for qemu-ga.
Though qemu-ga has supported some commands now, but it is not enough and some
of them only support for linux ...

I would like to enrich it, if someone think it is acceptable.

So, any comments will be welcome! Thanks ;)

zhanghailiang (2):
  qga: Introduce guest-get-os-version command with stubs
  qga: implement qmp_guest_get_os_version for windows

qga/commands-posix.c |   7 +++
qga/commands-win32.c | 125 +++++++++++++++++++++++++++++++++++++++++++++++++++
qga/qapi-schema.json |  26 +++++++++++
3 files changed, 158 insertions(+)

--
1.7.12.4










reply via email to

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