Merge branch 'upstream-1.7' into tizen_qemu_1.7

Signed-off-by: SeokYeon Hwang <syeon.hwang@samsung.com>
Conflicts:
VERSION
block/cow.c
block/raw-win32.c
block/stream.c
block/vmdk.c
blockdev.c
exec.c
hw/i386/pc_piix.c
hw/scsi/scsi-bus.c
include/qom/cpu.h
include/sysemu/kvm.h
qemu-img.c
tcg/tcg.c
tcg/tcg.h
vl.c

Change-Id: Ib8de93ad2c05150934e17e63d7f8e90ffdfccc62

package: version up

1.7.3

Change-Id: Iea9f7d0f748fc6108d12a19c8766f3c749eb3ccf
Signed-off-by: GiWoong Kim <giwoong.kim@samsung.com>

menu: remove temporary code

no need to check a sensord state on Host Keyboard menu.

Change-Id: I28a807adb55d2bb4b70d4b301428a02965a1d82f
Signed-off-by: GiWoong Kim <giwoong.kim@samsung.com>

screenshot: code refactoring

1. restructuring
2. delete unnecessary code
3. fix indenting

Change-Id: Ia50fc0454a1a205f7b9a7ae671337303fd2dd0f1
Signed-off-by: GiWoong Kim <giwoong.kim@samsung.com>

tethering: change a way to convert light sensor value from app to ecs.

In case of light sensor, it needs to set two value to pass to sensor device.

Change-Id: Ib8e0034b61e6774cbd0ccc591bc2b07af41de771
Signed-off-by: Kitae Kim <kt920.kim@samsung.com>

evdi: fix build error

Change-Id: I97c95b3587baad992381fd5ebf58a8c1fbaf56b3

Merge changes Ib3b818ce,I2aa61a73 into tizen

* changes:
  nfc: arrange source codes
  evdi: check if virtqueue is ready

Merge "nfc: check if virtqueue is ready" into tizen

nfc: arrange source codes

Removed unused variables.
Added checking data size.

Change-Id: Ib3b818ce35733345ccdb281a5292a76a1dd160e6
Signed-off-by: munkyu.im <munkyu.im@samsung.com>

evdi: check if virtqueue is ready

Before sending event,
virtio device needs to check if virtqueue is ready.
Otherwise when emulator reset,
device operation may have problem to send message.

Change-Id: I2aa61a73dc1b3e4956545c82293eb35e7c8a4759
Signed-off-by: munkyu.im <munkyu.im@samsung.com>

nfc: check if virtqueue is ready

Before sending event,
virtio device needs to check if virtqueue is ready.
Otherwise when emulator reset,
device operation may have problem to send message.

Change-Id: Ia715e314d0c1f65098eb7f1c612dc79c039bcdbe
Signed-off-by: munkyu.im <munkyu.im@samsung.com>

Merge changes I6a7026f6,I4a3871b5 into tizen

* changes:
  maru_brightness : Modified exit & init functions
  maru_overlay : Added the reset functions

suspend/resume: changed option name

Option name "ignore-suspend-lock" is un-clear the meaning of sentence.
So, it is changed to "enable-suspend" as a client expected action.

Change-Id: I204b175e20f07100bb90f4a481c726e502d68c82
Signed-off-by: Jinhyung Choi <jinhyung2.choi@samsung.com>

maru_brightness : Modified exit & init functions

Added the info log in the functions.
Added a missing statement for releasing memory in the exit function.
Modified the position that initialize the level variable.

Change-Id: I6a7026f66c9da683e36de3622724e83676ea21b4
Signed-off-by: Jinhyung Jo <jinhyung.jo@samsung.com>

maru_overlay : Added the reset functions

Initialize variables related to the overlay device state
when the device is rebooting.

Change-Id: I4a3871b56d551210c1adee94a099ee35fe8c825d
Signed-off-by: Jinhyung Jo <jinhyung.jo@samsung.com>

Merge changes I728423f7,I765e8feb into tizen

* changes:
  nfc: remove global count
  menu: remove preferences and about menu

Merge "host-keyboard: added reset function" into tizen

nfc: remove global count

This value is not meaningful.
nfc do not control count.

Change-Id: I728423f7dcadddf2993c12ef36821ca95c59e398
Signed-off-by: munkyu.im <munkyu.im@samsung.com>

suspend/resume: setting suspend lock state

- qemu option "ignore-suspend-lock" is created, that skips
  suspend lock as default.
- qemu manages suspend state.
- sdb server can request suspend lock and unlock state.
- emulator daemon request to qumu the suspend state, and
  it sets default suspend state through power manager

Change-Id: I4d6e4898b41fca419501ce098eb8d1e67e843cb5
Signed-off-by: Jinhyung Choi <jinhyung2.choi@samsung.com>

menu: remove preferences and about menu

They are default swt menu and do not have handler.
They are unnecessary. so remove them.

Change-Id: I765e8feb3c235bb898a5f48b59090995a5b8560d
Signed-off-by: munkyu.im <munkyu.im@samsung.com>

host-keyboard: added reset function

initialize index in the virtioqueue when reset state

Change-Id: I4c5a312ea0ec90c1fa7e1764b491af28db96a73f
Signed-off-by: sungmin ha <sungmin82.ha@samsung.com>

Merge changes I2d9a49c3,I249e3fe8,Id9e0c868,Ifdd8895e,I3f603a58,I32010ea2,I10663bec into tizen

* changes:
  communication: added protocol for emulator reset
  menu: modified interpolation menu name
  touchscreen: added reset feature
  menu: source code clean-up
  config: modified emulator skin properties
  menu: added interpolation menu
  display: changed the blank-guide image size

communication: added protocol for emulator reset

The Qemu-Skin protocol has been extended to share the
emulator reset state.

Change-Id: I2d9a49c3ddee0ecf7f9006efddc5e710e88fbb51
Signed-off-by: GiWoong Kim <giwoong.kim@samsung.com>

menu: modified interpolation menu name

modified menu name (Interpolation -> Quality).
and do not scaling when Scale menu is disabled.

Change-Id: I249e3fe86c21757f3187f1aa907d349c10a910ef
Signed-off-by: GiWoong Kim <giwoong.kim@samsung.com>

touchscreen: added reset feature

initialize some counters and numbers of buffers
in the virtqueue when reset state.

Change-Id: Id9e0c868eb5591e9f49c1c95a03ef01c5d092580
Signed-off-by: GiWoong Kim <giwoong.kim@samsung.com>

menu: source code clean-up

Added some null check.
And define DEFAULT_DOCK_POSITION value for Key Window.

Change-Id: Ifdd8895e5a41508e842eab0c8e9515320436adb4
Signed-off-by: GiWoong Kim <giwoong.kim@samsung.com>

config: modified emulator skin properties

1. extended skin properties for interpolation
(save the most recently used interpolation value)
2. enhanced skin properties file loading

Change-Id: I3f603a58ec5868ebbf4de7b541616e20c9839b32
Signed-off-by: GiWoong Kim <giwoong.kim@samsung.com>

Update version for 1.7.0 release

Signed-off-by: Anthony Liguori <aliguori@amazon.com>

qemu-iotests: Fix test 041

Performing multiple drive-mirror blockjobs on the same qemu instance
results in the image file used for the block device being replaced by
the newly mirrored file, which is not what we want.

Fix this by performing one dedicated test per sync mode.

Signed-off-by: Max Reitz <mreitz@redhat.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Kevin Wolf <kwolf@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-id: 1385407736-13941-3-git-send-email-mreitz@redhat.com
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

block/drive-mirror: Reuse backing HD for sync=none

For "none" sync mode in "absolute-paths" mode, the current image should
be used as the backing file for the newly created image.

The current behavior is:
a) If the image to be mirrored has a backing file, use that (which is
   wrong, since the operations recorded by "none" are applied to the
   image itself, not to its backing file).
b) If the image to be mirrored lacks a backing file, the target doesn't
   have one either (which is not really wrong, but not really right,
   either; "none" records a set of operations executed on the image
   file, therefore having no backing file to apply these operations on
   seems rather pointless).

For a, this is clearly a bugfix. For b, it is still a bugfix, although
it might break existing API - but since that case crashed qemu just
three weeks ago (before 1452686495922b81d6cf43edf025c1aef15965c0), we
can safely assume there is no such API relying on that case yet.

Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Max Reitz <mreitz@redhat.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Kevin Wolf <kwolf@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-id: 1385407736-13941-2-git-send-email-mreitz@redhat.com
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

menu: added interpolation menu

1. Now, user can choose the interpolation filter type
for emulator window scaling by popup menu.
(Scale>Interpolation)
2. delete unused resource image file (usb_keyboard.png)

Change-Id: I32010ea2577cabf58abd4c289fe8fd22c400cfb7
Signed-off-by: GiWoong Kim <giwoong.kim@samsung.com>

tethering: fixed compilation error on Windows.

EISCONN is not defined on MinGW and others things.

Change-Id: Ic775f355ffcb75481f89de1baa58dbe36f213cce
Signed-off-by: Kitae Kim <kt920.kim@samsung.com>

display: changed the blank-guide image size

450x450 -> 650x650

Change-Id: I10663bec2b33f3445569357dc09334282cb1692f
Signed-off-by: GiWoong Kim <giwoong.kim@samsung.com>

config: modified getValidResolution function

filter out the negative value from start-up arguments

Change-Id: I4bdbff17fd4c7554ed465fb890d8dcfa686721d3
Signed-off-by: GiWoong Kim <giwoong.kim@samsung.com>

skin: remove dubious method used

1. To specify an explicit charset, I give encoding
argument for String construct.
2. define terminateImmediately function

Change-Id: Ibfd5b8e4184fb492fe9b22e3ceb123f35ed9b185
Signed-off-by: GiWoong Kim <giwoong.kim@samsung.com>

Update version for v1.7.0-rc2 release

curses: fixup SIGWINCH handler mess

Don't run code in the signal handler, only set a flag.
Use sigaction(2) to avoid non-portable signal(2) semantics.
Make #ifdefs less messy.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Message-id: 1385130903-20531-1-git-send-email-kraxel@redhat.com
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

qga: Fix two format strings for MinGW

Both code locations cause a compiler warning. Using "%s" instead of "%lu"
would result in a program crash if the wrong code were executed.

Signed-off-by: Stefan Weil <sw@weilnetz.de>
Message-id: 1385409257-2522-1-git-send-email-sw@weilnetz.de
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

PPC: BookE: Make FIT/WDT timers at best millisecond grained

The default granularity for the FIT timer on 440 is on every 0x1000th
transition of TB from 0 to 1. Translated that means 48828 times a second.

Since interrupts are quite expensive for 440 and we don't really care
about the accuracy of the FIT to that significance, let's force FIT and
WDT to at best millisecond granularity.

This basically restores behavior as it was in QEMU 1.6, where timers
could only deal with millisecond granularities at all.

This patch greatly improves performance with the 440 target and restores
roughly the same performance level that QEMU 1.6 had for me.

Signed-off-by: Alexander Graf <agraf@suse.de>
Message-id: 1385416015-22775-3-git-send-email-agraf@suse.de
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

PPC: Make BookE FIT/WDT timers more lazy

Today we fire FIT and WDT timer events every time the respective bit
position in TB flips from 0 -> 1.

However, there is no need to do this if the end result would be that
we're changing a TSR bit that is set to 1 to 1 again. No guest visible
change would have occured.

So whenever we see that the TSR bit to our timer is already set, don't
even bother to update the timer that would potentially fire it off.

However, we do need to make sure that we update our timer that notifies
us of the TB flip when the respective TSR bit gets unset. In that case
we do care about the flip and need to notify the guest again. So add
a callback into our timer handlers when TSR bits get unset.

This improves performance for me when the guest is busy processing things.

Signed-off-by: Alexander Graf <agraf@suse.de>
Message-id: 1385416015-22775-2-git-send-email-agraf@suse.de
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

acpi-build: fix support for glib < 2.22

glib < 2.22 does not have g_array_get_element_size,
limit it's use (to check all elements are 1 byte
in size) to newer glib.

This fixes build on RHEL 5.3.

Reported-by: Richard Henderson <rth@redhat.com>
Reported-by: Erik Rull <erik.rull@rdsoftware.de>
Tested-by: Richard Henderson <rth@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Message-id: 20131125220039.GA16386@redhat.com
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

package: version up

1.7.1

Change-Id: I1c58e366fdc9eae4c59ccba52f5cc460e130e6f8
Signed-off-by: GiWoong Kim <giwoong.kim@samsung.com>

menu: method for initial scale factor value determining

Emulator does not use fixed value(1/2x) for initial scale
factor anymore. It depend on arrange of Scale menu.
The first sub menu item of Scale menu will be choosen as
initial scale for emulator window. (except when there is
already 1/2x factor as sub menu item)

Change-Id: I1750269035d4324139adf343a76246597e2066a1
Signed-off-by: GiWoong Kim <giwoong.kim@samsung.com>

skin: enhance the function call of skin initialization

1. declare initSkin function to express the inheritance clearly.
2. moved initialization source code to proper class.
3. modified variable name (windowHandleId -> displayHandleId)

Change-Id: Ibafa8763433fc1738365897ab179794d78981da5
Signed-off-by: GiWoong Kim <giwoong.kim@samsung.com>

package: modify tizen-kvm script

Add kvm group if it does not exist,
and then activate udev to apply tizen-kvm.rules

Change-Id: I763f4e31b778dfe845e2874b543d5732ec050769
Signed-off-by: Kitae Kim <kt920.kim@samsung.com>

maru_codec: remove codec device.

maru_codec does not need to build any more,
because the device is not replaced to maru_brillcodec.

Change-Id: I9f1d2b0a990d940348247367a560f58444009b47
Signed-off-by: Kitae Kim <kt920.kim@samsung.com>

Merge "tethering: modified connection error routine." into tizen

Merge changes I8dfa6936,Id967ab1e,I82ab13a9 into tizen

* changes:
  tethering: added resolution message.
  tethering: convert sensor or touch data.
  tethering: changed a way to set sensor or touch data.

Merge "tethering: added tethering feature." into tizen

Merge "YaGL: Replace AVL trees with hashes" into tizen

Merge remote-tracking branch 'mst/tags/for_anthony' into staging

pc very last minute fixes for 1.7

This has a fix for a crasher bug with pci bridges,
boot failure fix for s390 on 32 bit hosts,
and fixes build for hosts with old glib.

There's also a fix for --iasl configure flag - it can be used
to work around broken iasl on some systems either
by using a non-standard iasl or by disabling it.

I've also reverted a e1000/rtl mac programming change
that seems slightly wrong and too risky for 1.8.

Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
# gpg: Signature made Mon 25 Nov 2013 03:40:07 AM PST using RSA key ID D28D5469
# gpg: Can't check signature: public key not found

# By Michael S. Tsirkin (5) and Bandan Das (1)
# Via Michael S. Tsirkin
* mst/tags/for_anthony:
  configure: make --iasl option actually work
  Revert "e1000/rtl8139: update HMP NIC when every bit is written"
  acpi-build: fix build on glib < 2.14
  acpi-build: fix build on glib < 2.22
  pci: unregister vmstate_pcibus on unplug
  s390x: fix flat file load on 32 bit systems

Message-id: 1385379990-32093-1-git-send-email-mst@redhat.com
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

Merge remote-tracking branch 'bonzini/tags/for-anthony' into staging

Here are a bunch of 1.7-tagged patches that I was afraid
were getting forgotten or that did not have a clear maintainer responsible
for making a pull request.

# gpg: Signature made Thu 21 Nov 2013 08:40:59 AM PST using RSA key ID 9B4D86F2
# gpg: Can't check signature: public key not found

# By Peter Maydell (3) and others
# Via Paolo Bonzini
* bonzini/tags/for-anthony:
  qga: Fix compiler warnings (missing format attribute, wrong format strings)
  mips jazz: do not raise data bus exception when accessing invalid addresses
  target-i386: yield to another VCPU on PAUSE
  rng-egd: offset the point when repeatedly read from the buffer
  rng-egd: remove redundant free
  target-i386: Fix build by providing stub kvm_arch_get_supported_cpuid()
  vfio-pci: Fix multifunction=on
  atomic.h: Fix build with clang
  pc: get rid of builtin pvpanic for "-M pc-1.5"
  configure: Explicitly set ARFLAGS so we can build with GNU Make 4.0
  sun4m: Add FCode ROM for TCX framebuffer

Message-id: 1385052578-32352-1-git-send-email-pbonzini@redhat.com
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

Merge remote-tracking branch 'mdroth/qga-pull-2013-11-22' into staging

# By Tomoki Sekiyama
# Via Michael Roth
* mdroth/qga-pull-2013-11-22:
  qemu-ga: vss-win32: Install VSS provider COM+ application service

Message-id: 1385154505-15145-1-git-send-email-mdroth@linux.vnet.ibm.com
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

Merge remote-tracking branch 'stefanha/net' into staging

# By Vlad Yasevich
# Via Stefan Hajnoczi
* stefanha/net:
  qdev-properties-system.c: Allow vlan or netdev for -device, not both

Message-id: 1385118544-28482-1-git-send-email-stefanha@redhat.com
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

YaGL: Replace AVL trees with hashes

The number of OpenGL objects and/or shader
program locations can be really large in
case of heavy graphics applications, so object
lookup time matters. Using hash tables is more
appropriate since they have much less lookup
time. The difference can be seen when running
Doom3, AVL lookup function was hot in perf report

Change-Id: I46121ecd8d8c0d392fe6f3aa4aba0c96a90280ae

tethering: modified connection error routine.

Added routine to handle connection refused case.
In addition to this, port number needs to be sent to ecp
when ecp is launched again after being connected.

Change-Id: Ia5a4104452024f7558febf1410b366e94ed3d8a1
Signed-off-by: Kitae Kim <kt920.kim@samsung.com>

tethering: added resolution message.

Touch point might be different, depending on emulator resolution.
That's why resolution message is added.
In addition to this, resolution and max count for touch have to be sent
on start_ans message.

Change-Id: I8dfa69361f7ed69832d05f3c8ef26d89d6ef522c
Signed-off-by: Kitae Kim <kt920.kim@samsung.com>

tethering: convert sensor or touch data.

Sensor data from app should be changed into ecs_sensor format.
The format is defined as sensor type, number of sensor values and each parameter.
In addition to this, touch state such as pressed or released has to changed,
because its value is different to touch device.

Change-Id: Id967ab1eb083d165656a13f39c544c5514918c3c
Signed-off-by: Kitae Kim <kt920.kim@samsung.com>

tethering: changed a way to set sensor or touch data.

app_tethering module does not need to call set data fuction of each device.
It would be better to use wrap up functions for setting data,
because the purpose of ecs_tethering module is for communicating with ecp and each device.

Change-Id: I82ab13a96810418908aed6f5a2a6e5751b9e5df8
Signed-off-by: Kitae Kim <kt920.kim@samsung.com>

tethering: added tethering feature.

app_tethering is for communicating with App on target.
This module is a client side to connect App that creates sensor or touch point data
and it deliveries received data to each device.

ecs_tethering is for communicating with ECP. This module handles messages from ECP.
The messages consist of connect, disconnect, connection status and etc..

Change-Id: I2c31f785fe824fd578fafb08e9f9efe423f26ea6
Signed-off-by: Kitae Kim <kt920.kim@samsung.com>

Merge changes I275a5231,I4eae90b7 into tizen

* changes:
  YaGL: Add glFlush before makeCurrent
  YaGL/VIGS: Return glFinish

Merge "hwkey: added reset feature." into tizen

Merge "brillcodec: added reset feature." into tizen

configure: make --iasl option actually work

--iasl option was added to CC option parsing section by mistake,
it's not effective there and attempts to use cause
an 'unknown option' error.

Fix this up.

Tested-by: Marcel Apfelbaum <marcel.a@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>

qemu-ga: vss-win32: Install VSS provider COM+ application service

Currently, qemu-ga for Windows fails to execute guset-fsfreeze-freeze when
no user is logging in to Windows, with an error message:
  {"error":{"class":"GenericError",
            "desc":"failed to add C:\\ to snapshotset:  (error: 8004230f)"}}

To enable guest-fsfreeze-freeze/thaw without logging in users, this installs
a service to execute qemu-ga VSS provider COM+ application that has full
access privileges to the local system. The service will automatically be
removed when the COM+ application is deregistered.

This patch replaces ICOMAdminCatalog interface with ICOMAdminCatalog2
interface that contains CreateServiceForApplication() method in addition.

Signed-off-by: Tomoki Sekiyama <tomoki.sekiyama@hds.com>
Reviewed-by: Gal Hammer <ghammer@redhat.com>
Reviewed-by: Yan Vugenfirer <yvugenfi@redhat.com>
Tested-by: Yan Vugenfirer <yvugenfi@redhat.com>
Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>

hwkey: added reset feature.

initialize numbers of buffers in the virtqueue when reset state.

Change-Id: I72c40e156ca3fa16e33f732505cd7ca012139122
Signed-off-by: Kitae Kim <kt920.kim@samsung.com>

brillcodec: added reset feature.

Support reset feature and remove useless codes.

Change-Id: I05ac5cf32c9e2a43e46f699b5f7966dbd09698b6
Signed-off-by: Kitae Kim <kt920.kim@samsung.com>

qdev-properties-system.c: Allow vlan or netdev for -device, not both

It is currently possible to specify things like:
-device e1000,netdev=foo,vlan=1
With this usage, whichever argument was specified last (vlan or netdev)
overwrites what was previousely set and results in a non-working
configuration.  Even worse, when used with multiqueue devices,
it causes a segmentation fault on exit in qemu_free_net_client.

That patch treates the above command line options as invalid and
generates an error at start-up.

Signed-off-by: Vlad Yasevich <vyasevic@redhat.com>
Acked-by: Jason Wang <jasowang@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>

YaGL: Add glFlush before makeCurrent

On Mac OS X aglSetCurrentContext doesn't implicitly
glFlush like on linux and windows. glFlush is required
because some apps, like webkit based apps for example,
rely on this behavior

Change-Id: I275a523152369206daa1602217443f28e2e6479f

display: modified Blank Guide image resizing

Emulator should resize the Blank Guide image
to fit the display, base on its resolution and scale.

Change-Id: If8a89f2e46068ad15fc8d5163b85108f22773093
Signed-off-by: GiWoong Kim <giwoong.kim@samsung.com>

display: modified some variable & function names

In Qemu, to avoid a confusing about screen size,
I changed some variable names and function names.
(set_emul_lcd_size -> set_emul_resolution)

Change-Id: I567291504aab16f04255d16351fb18e40223e1b6
Signed-off-by: GiWoong Kim <giwoong.kim@samsung.com>

qga: Fix compiler warnings (missing format attribute, wrong format strings)

gcc 4.8.2 reports this warning when extra warnings are enabled (-Wextra):

  CC    qga/commands.o
qga/commands.c: In function ‘slog’:
qga/commands.c:28:5: error:
 function might be possible candidate for ‘gnu_printf’ format attribute [-Werror=suggest-attribute=format]
     g_logv("syslog", G_LOG_LEVEL_INFO, fmt, ap);
     ^

gcc 4.8.2 reports this warning when slog is declared with the
gnu_printf format attribute:

qga/commands-posix.c: In function ‘qmp_guest_file_open’:
qga/commands-posix.c:404:5: warning:
 format ‘%d’ expects argument of type ‘int’, but argument 2 has type ‘int64_t’ [-Wformat=]
     slog("guest-file-open, handle: %d", handle);
     ^

On 32 bit hosts there are three more warnings which are also fixed here.

Signed-off-by: Stefan Weil <sw@weilnetz.de>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

mips jazz: do not raise data bus exception when accessing invalid addresses

MIPS Jazz chipset doesn't seem to raise data bus exceptions on invalid accesses.
However, there is no easy way to prevent them. Creating a big memory region
for the whole address space doesn't prevent memory core to directly call
unassigned_mem_read/write which in turn call cpu->do_unassigned_access,
which (for MIPS CPU) raise an data bus exception.

This fixes a MIPS Jazz regression introduced in c658b94f6e8c206c59d02aa6fbac285b86b53d2c.

Signed-off-by: Hervé Poussineau <hpoussin@reactos.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

target-i386: yield to another VCPU on PAUSE

After commit b1bbfe7 (aio / timers: On timer modification, qemu_notify
or aio_notify, 2013-08-21) FreeBSD guests report a huge slowdown.

The problem shows up as soon as FreeBSD turns out its periodic (~1 ms)
tick, but the timers are only the trigger for a pre-existing problem.

Before the offending patch, setting a timer did a timer_settime system call.

After, setting the timer exits the event loop (which uses poll) and
reenters it with a new deadline.  This does not cause any slowdown; the
difference is between one system call (timer_settime and a signal
delivery (SIGALRM) before the patch, and two system calls afterwards
(write to a pipe or eventfd + calling poll again when re-entering the
event loop).

Unfortunately, the exit/enter causes the main loop to grab the iothread
lock, which in turns kicks the VCPU thread out of execution.  This
causes TCG to execute the next VCPU in its round-robin scheduling of
VCPUS.  When the second VCPU is mostly unused, FreeBSD runs a "pause"
instruction in its idle loop which only burns cycles without any
progress.  As soon as the timer tick expires, the first VCPU runs
the interrupt handler but very soon it sets it again---and QEMU
then goes back doing nothing in the second VCPU.

The fix is to make the pause instruction do "cpu_loop_exit".

Reported-by: Luigi Rizzo <rizzo@iet.unipi.it>
Reviewed-by: Richard Henderson <rth@twiddle.net>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

rng-egd: offset the point when repeatedly read from the buffer

The buffer content might be read out more than once, currently
we just repeatedly read the first data block, buffer offset is
missing.

Cc: qemu-stable@nongnu.org
Signed-off-by: Amos Kong <akong@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

rng-egd: remove redundant free

We didn't set default chr_name, the free is redundant.

Signed-off-by: Amos Kong <akong@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

target-i386: Fix build by providing stub kvm_arch_get_supported_cpuid()

Fix build failures with clang when KVM is not enabled by
providing a stub version of kvm_arch_get_supported_cpuid().
We retain the compile time check that this function isn't
called when CONFIG_KVM is not set by guarding the stub with
ifndef __OPTIMIZE__ (we assume that an optimizing build will
do sufficient constant folding and dead code elimination to
remove the calls before linking).

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

vfio-pci: Fix multifunction=on

When an assigned device is initialized it copies the device config
space into the emulated config space.  Unfortunately multifunction is
setup prior to the device initfn and gets clobbered.  We need to
restore it just like pci-assign does.

Cc: qemu-stable@nongnu.org
Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

atomic.h: Fix build with clang

clang defines __ATOMIC_SEQ_CST but its implementation of the
__atomic_exchange() builtin differs from that of gcc. Move the
__clang__ branch of the ifdef ladder to the top and fix its
implementation (there is no such builtin as __sync_exchange),
so we can compile with clang again.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

pc: get rid of builtin pvpanic for "-M pc-1.5"

This causes two slight backwards-incompatibilities between "-M pc-1.5"
and 1.5's "-M pc":

(1) a fw_cfg file is removed with this patch.  This is only a problem
if migration stops the virtual machine exactly during fw_cfg enumeration.

(2) after migration, a VM created without an explicit "-device pvpanic"
will stop reporting panics to management.

The first problem only occurs if migration is done at a very, very
early point (and I'm not sure it can happen in practice for reasonable-size
VMs, since it will likely take more time to send the RAM to destination,
than it will take for BIOS to scan fw_cfg).

The second problem only occurs if the guest panics _and_ has a guest
driver _and_ management knows to look at the crash event, so it is
mostly theoretical at this point in time.

Thus keep the code simple, and pretend it was never broken.

Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

configure: Explicitly set ARFLAGS so we can build with GNU Make 4.0

Our rules.mak adds '-rR' to MAKEFLAGS to indicate that we will be
explicitly specifying everything and not relying on any default
variables or rules. However we were accidentally relying on the
default ARFLAGS ("rv"). This went unnoticed because of a bug in
GNU Make 3.82 and earlier which meant that adding -rR to MAKEFLAGS
only affected submakes, not the currently running instance.
Explicitly set ARFLAGS in config-host.mak, in the same way we
handle CFLAGS and LDFLAGS; this will allow us to work with
Make 4.0.

Thanks to Paul Smith for analyzing this bug for us.

Cc: qemu-stable@nongnu.org
Reported-by: Ken Moffat <zarniwhoop@ntlworld.com>
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

sun4m: Add FCode ROM for TCX framebuffer

Upstream OpenBIOS now implements SBus probing in order to determine the
contents of a physical bus slot, which is required to allow OpenBIOS to
identify the framebuffer without help from the fw_cfg interface.

SBus probing works by detecting the presence of an FCode program
(effectively tokenised Forth) at the base address of each slot, and if
present executes it so that it creates its own device node in the
OpenBIOS device tree.

The FCode ROM is generated as part of the OpenBIOS build and should
generally be updated at the same time.

Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
CC: Blue Swirl <blauwirbel@gmail.com>
CC: Bob Breuer <breuerr@mc.net>
CC: Artyom Tarasenko <atar4qemu@gmail.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

Update version for 1.7.0-rc1 release

Signed-off-by: Anthony Liguori <aliguori@amazon.com>

vfio-pci: Fix multifunction=on

When an assigned device is initialized it copies the device config
space into the emulated config space.  Unfortunately multifunction is
setup prior to the device initfn and gets clobbered.  We need to
restore it just like pci-assign does.

Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
Reviewed-by: Bandan Das <bsd@redhat.com>
Message-id: 20131112185059.7262.33780.stgit@bling.home
Cc: qemu-stable@nongnu.org
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

target-i386: Fix addr32 prefix in gen_lea_modrm

Fix the following run-test-x86_64 testsuite failures:

-lea (%%eax) = 0000000000000001
-lea (%%ebx) = 0000000000000002
-lea (%%ecx) = 0000000000000004
-lea (%%edx) = 0000000000000008
-lea (%%esi) = 0000000000000010
-lea (%%edi) = 0000000000000020
+lea (%%eax) = 0000abcc00000001
+lea (%%ebx) = 0000abcf00000002
+lea (%%ecx) = 0000abc900000004
+lea (%%edx) = 0000abc500000008
+lea (%%esi) = 0000abdd00000010
+lea (%%edi) = 0000abed00000020

In addition, reduce ifdeffery and minimize the number of TCG ops
produced during address computation.

Signed-off-by: Richard Henderson <rth@twiddle.net>
Message-id: 1384219016-5170-1-git-send-email-rth@twiddle.net
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

atomic.h: Fix build with clang

clang defines __ATOMIC_SEQ_CST but its implementation of the
__atomic_exchange() builtin differs from that of gcc. Move the
__clang__ branch of the ifdef ladder to the top and fix its
implementation (there is no such builtin as __sync_exchange),
so we can compile with clang again.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Message-id: 1382435921-18438-1-git-send-email-peter.maydell@linaro.org
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

target-i386: do not override nr_cores for -cpu host

Commit 787aaf5 (target-i386: forward CPUID cache leaves when -cpu host is
used, 2013-09-02) brings bits 31..26 of CPUID leaf 04h out of sync with
the APIC IDs that QEMU reserves for each package.  This number must come
from "-smp" options rather than from the host CPUID.

It also turns out that this unsyncing makes Windows Server 2012R2 fail
to boot.

Tested-by: Peter Lieven <pl@kamp.de>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Benoit Canet <benoit@irqsave.net>
Reviewed-by: Eduardo Habkost <ehabkost@redhat.com>
Message-id: 1384879786-6721-1-git-send-email-pbonzini@redhat.com
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

mips jazz: do not raise data bus exception when accessing invalid addresses

MIPS Jazz chipset doesn't seem to raise data bus exceptions on invalid accesses.
However, there is no easy way to prevent them. Creating a big memory region
for the whole address space doesn't prevent memory core to directly call
unassigned_mem_read/write which in turn call cpu->do_unassigned_access,
which (for MIPS CPU) raise an data bus exception.

This fixes a MIPS Jazz regression introduced in c658b94f6e8c206c59d02aa6fbac285b86b53d2c.

Signed-off-by: Hervé Poussineau <hpoussin@reactos.org>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Hervé Poussineau <hpoussin@reactos.org>
Message-id: 1383603977-7003-1-git-send-email-hpoussin@reactos.org
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

target-i386: yield to another VCPU on PAUSE

After commit b1bbfe7 (aio / timers: On timer modification, qemu_notify
or aio_notify, 2013-08-21) FreeBSD guests report a huge slowdown.

The problem shows up as soon as FreeBSD turns out its periodic (~1 ms)
tick, but the timers are only the trigger for a pre-existing problem.

Before the offending patch, setting a timer did a timer_settime system call.

After, setting the timer exits the event loop (which uses poll) and
reenters it with a new deadline.  This does not cause any slowdown; the
difference is between one system call (timer_settime and a signal
delivery (SIGALRM) before the patch, and two system calls afterwards
(write to a pipe or eventfd + calling poll again when re-entering the
event loop).

Unfortunately, the exit/enter causes the main loop to grab the iothread
lock, which in turns kicks the VCPU thread out of execution.  This
causes TCG to execute the next VCPU in its round-robin scheduling of
VCPUS.  When the second VCPU is mostly unused, FreeBSD runs a "pause"
instruction in its idle loop which only burns cycles without any
progress.  As soon as the timer tick expires, the first VCPU runs
the interrupt handler but very soon it sets it again---and QEMU
then goes back doing nothing in the second VCPU.

The fix is to make the pause instruction do "cpu_loop_exit".

Cc: Richard Henderson <rth@twiddle.net>
Reported-by: Luigi Rizzo <rizzo@iet.unipi.it>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Richard Henderson <rth@twiddle.net>
Message-id: 1384948442-24217-1-git-send-email-pbonzini@redhat.com
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

rng-egd: offset the point when repeatedly read from the buffer

The buffer content might be read out more than once, currently
we just repeatedly read the first data block, buffer offset is
missing.

Cc: qemu-stable@nongnu.org
Signed-off-by: Amos Kong <akong@redhat.com>
Message-id: 1385023371-8198-3-git-send-email-akong@redhat.com
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

rng-egd: remove redundant free

We didn't set default chr_name, the free is redundant.

Signed-off-by: Amos Kong <akong@redhat.com>
Message-id: 1385023371-8198-2-git-send-email-akong@redhat.com
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

virtio-rng: add check of period

If period is assigned to 0, limit timer will expire immediately.
It causes a qemu warning:

"main-loop: WARNING: I/O thread spun for 1000 iterations"

This limit is meaningless. This patch forbids to assign 0 to period.

Reviewed-by: Amit Shah <amit.shah@redhat.com>
Signed-off-by: Amos Kong <akong@redhat.com>
Message-id: 1385031203-23790-1-git-send-email-akong@redhat.com
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

s390x: fix flat file load on 32 bit systems

pc-bios/s390-zipl.rom is a flat image so it's expected that
loading it as elf will fail.
It should fall back on loading a flat file, but doesn't
on 32 bit systems, instead it fails printing:
    qemu: hardware error: could not load bootloader 's390-zipl.rom'

The result is boot failure.

The reason is that a 64 bit unsigned interger which is set
to -1 on error is compared to -1UL which on a 32 bit system
with gcc is a 32 bit unsigned interger.
Since both are unsigned, no sign extension takes place and
comparison evaluates to non-equal.

There's no reason to do clever tricks: all functions
we call actually return int so just use int.
And then we can use == -1 everywhere, consistently.

Reviewed-by: Alexander Graf <agraf@suse.de>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Message-id: 20131121133426.GA30827@redhat.com
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

acpi-build: fix build on glib < 2.14

g_array_get_element_size was only added in glib 2.14.
Fortunately we don't use it for any arrays where
element size is > 1, so just add an assert.

Reported-by: Richard Henderson <rth@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Message-id: 1385036128-8753-2-git-send-email-mst@redhat.com
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

acpi-build: fix build on glib < 2.22

g_string_vprintf was only introduced in 2.24 so switch to vsnprintf
instead.  A bit uglier but name size is fixed at 4 bytes here so it's
easy.

Reported-by: Richard Henderson <rth@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Message-id: 1385036128-8753-1-git-send-email-mst@redhat.com
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

Merge remote-tracking branch 'qemu-kvm/uq/master' into staging

# By Jan Kiszka (1) and others
# Via Gleb Natapov
* qemu-kvm/uq/master:
  kvm: Fix uninitialized cpuid_data
  pci-assign: Remove dead code for direct I/O region access from userspace
  KVM: x86: fix typo in KVM_GET_XCRS

Message-id: cover.1385040432.git.gleb@redhat.com
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

Merge remote-tracking branch 'jliu/or32' into staging

# By Sebastian Macke
# Via Jia Liu
* jliu/or32:
  target-openrisc: Correct carry flag check of l.addc and l.addic test cases
  target-openrisc: Correct memory bounds checking for the tlb buffers
  openrisc-timer: Reduce overhead, Separate clock update functions
  target-openrisc: Correct wrong epcr register in interrupt handler
  target-openrisc: Remove executable flag for every page
  target-openrisc: Remove unnecessary code generated by jump instructions
  target-openrisc: Speed up move instruction

Message-id: 1384958318-9145-1-git-send-email-proljc@gmail.com
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

Merge remote-tracking branch 'quintela/migration.next' into staging

# By Wangting (Kathy)
# Via Juan Quintela
* quintela/migration.next:
  The calculation of bytes_xfer in qemu_put_buffer() is wrong

Message-id: 1384946787-8190-1-git-send-email-quintela@redhat.com
Signed-off-by: Anthony Liguori <aliguori@amazon.com>

Revert "e1000/rtl8139: update HMP NIC when every bit is written"

This reverts commit cd5be5829c1ce87aa6b3a7806524fac07ac9a757.
Digging into hardware specs shows this does not
actually make QEMU behave more like hardware:

There are valid arguments backed by the spec to indicate why the version
of e1000 prior to cd5be582 was more correct: the high byte actually
includes a valid bit, this is why all guests write it last.

For rtl8139 there's actually a separate undocumented valid bit, but we
don't implement it yet.

To summarize all the drivers we know about behave in one way
that allows us to make an assumption about write order and avoid
spurious, incorrect mac address updates to the monitor.

Let's stick to the tried heuristic for 1.7 and
possibly revisit for 1.8.

Reported-by: Vlad Yasevich <vyasevic@redhat.com>
Reviewed-by: Vlad Yasevich <vyasevic@redhat.com>
Cc: Amos Kong <akong@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>

acpi-build: fix build on glib < 2.14

g_array_get_element_size was only added in glib 2.14,
there's no way to find element size in with an older glib.

Fortunately we only use a single table (linker) where element size > 1.
Switch element size to 1 everywhere, then we can just look at len field
to get table size in bytes.

Add an assert to make sure we catch any violations of this rule.

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Reported-by: Richard Henderson <rth@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>