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>

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>

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>

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

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

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>

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>

Merge changes I478cb1a1,I5851bbca,I4b38ab3c into tizen

* changes:
  menu: added XML schema for scale menu
  menu: prepare for dynamic scale factor
  skin: no need to set SWT interpolation on Windows/MacOS

Merge "nfc: remove excutable permission" into tizen

Merge "esm: Implement reset function." into tizen

YaGL/VIGS: Return glFinish

Though we use single host thread for all OpenGL operations
it's still not guaranteed that commands in different contexts will
be processed in order, thus, single glFinish in VIGS will not wait
for GL commands to be processed in YaGL. Because of this contents
of windows on target may appear incomplete, this was observed on
Mac OS X in particular. glFinish will reduce performance, but it'll
ensure correct rendering. Later we'll implement another approach
that'll use GL fences in order not to wait for rendering to be
completed

Change-Id: I4eae90b73f4d66cd8c7fa1939db1c18ad9751051

menu: added XML schema for scale menu

1. To constitute the scale menu's factor array
through XML, I declared "factorItem" element.
2. declare "blankGuide" element directly

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

nfc: remove excutable permission

It was added accidentally.
So removed it.

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

esm: Implement reset function.

Change-Id: I0770e6011e12185b74b23af908dd3c54f719ddce
Signed-off-by: SeokYeon Hwang <syeon.hwang@samsung.com>

menu: prepare for dynamic scale factor

The range of scale factor has to depend on their
emulator resolution. Factors must be dynamic. Therefore,
the scale factor should be treated as a number.

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

skin: no need to set SWT interpolation on Windows/MacOS

On Ubuntu, when drawing the skin image stretched with an
interpolation algorithm, the edges of the image are faded.
This problem is not occured at Windows or MacOS.

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

Merge "suspend/resume: NOT allow to add duplicated client." into tizen

Merge "brillcodec: changed a way to release avcontext." into tizen

brillcodec: changed a way to release avcontext.

release_context does not need to iterate all contexts,
because codec driver manages context_id and give the context_id to device.

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

suspend/resume: NOT allow to add duplicated client.

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

package: version up.

1.7.0

Change-Id: I37435087bafa9d0340deb4271ae7443f1e732099
Signed-off-by: SeokYeon Hwang <syeon.hwang@samsung.com>

maru_pm: Add suspend / wakeup notifier.

Add suspend / wakeup notifier to tell sdb.

Change-Id: I038f70f7a97cc3b60fab357a7c637c754aa035f9
Signed-off-by: SeokYeon Hwang <syeon.hwang@samsung.com>

suspend/resume: changed message format

The message length was set as decimal(ex. 0026).
It is changed to hexadecimal (ex. 001A) for first 4byte.

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

ecs: save a ecs port into a .ecs.property

ECS port is randomly created, and it is saved into
[tizen-sdk-data]/emulator/vms/[VM name]/.ecs.properties

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

skin: added updateSkin interface

The each skin composer can update the skin(like
a ColorTag updating) itself by diffrent way
if it required.

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

package: version up

1.6.4

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

menu: added schema for host keyboard menu

Now, user can configure the host keyboard menu
(Right click popup>Advance>Host Keyboard menu) by
skin layout XML file.

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

skin: modified indenting

1. modified indenting
2. added boilerplate title

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

menu: skin popup menu code clean up

1. To improve the readability, I declare the some new functions.
2. Do not show a seperator from popup menu when it has no items.

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

menu: skin dialog code clean up

1. allow to create a button composite optionally in SkinDialog
2. added boilerplate title (SkinDialog, LicenceDialog,
RamdumpDialog)
3. define static string
4. modified indenting & etc

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

communication: added protocol for the host keyboard state

The Qemu-Skin protocol has been extended to share the
host keyboard state between ECP and right-click popup
menu.

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

Enable "SLEEP" on emulator.

Rewrite "maru_pm".
Cooperate "SLEEP" with qemu suspend.
Clean-up power management code.

Change-Id: Ieae0513d10fed9110e8dc9a2b67e1d2316f3ff36
Signed-off-by: SeokYeon Hwang <syeon.hwang@samsung.com>

Merge "ecs: apply nfc multi-session" into tizen

Merge changes I14ea9453,I86f46a5f into tizen

* changes:
  skin: move to the proper java package
  touch: remove data race condition

Merge "guest server: changed socket connection & added logs" into tizen

Merge "brillcodec: fixed null pointer deferences." into tizen

ecs: apply nfc multi-session

modified protobuf protocol
added id, type value to identify each connection

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

guest server: changed socket connection & added logs

Socket connection is established first, then send message to sdb server.
Also logs are added for debugging.

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

skin: move to the proper java package

EmulatorSkinState: skin -> skin.info
GeneralKeyWindow, SpecialKeyWindow: custom -> menu

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

touch: remove data race condition

There is no need to reset the ring buffer counter
in the pthread lock section.

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

communication: no need to define the various DataTransfers

1. merge progressDataTransfer and brightnessDataTransfer
2. fix typo at boilerplate title

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

brillcodec: fixed null pointer deferences.

Removed unnecessary null check.

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

Merge "vigs & yagl : Move sources to own directories" into tizen

Merge "vigs & yagl : Insert the boilerplate" into tizen

guest_server: changed message format when a sdb server registers in guest server

Previously it gets ip, port, serial, but now serial only

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

vigs & yagl : Move sources to own directories

Make new directories for each module in the hw directory
, move the sources and add makefiles.

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

vigs & yagl : Insert the boilerplate

Added the GPL2 boilerplate

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

Merge changes Iea9ab886,I457a35cd into tizen

* changes:
  emulator: boot completed logging
  communication: modified event message names

Merge "package: modify install and remove script." into tizen

Fix compilation error on maru_board.c

A xen_hvm_init() function prototype is modified.

Change-Id: I6ec43199ddb65db591bedea8c89db6d46010daf0
Signed-off-by: SeokYeon Hwang <syeon.hwang@samsung.com>

Update VERSION for 1.6.1 release

Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>

scsi: Allocate SCSITargetReq r->buf dynamically

BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1007330
Brew: https://brewweb.devel.redhat.com/taskinfo?taskID=6282465

This is the backport of the following commit. The patch is not
sent public since it is a embargoed bug.

   r->buf is hardcoded to 2056 which is (256 + 1) * 8, allowing 256 luns at
   most. If more than 256 luns are specified by user, we have buffer
   overflow in scsi_target_emulate_report_luns.

   To fix, we allocate the buffer dynamically.

Signed-off-by: Asias He <asias@redhat.com>
Signed-off-by: Asias He <asias@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
*s/&r->buf/r->buf/ due to type change

Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>

qemu: Add qemu xen logic for Xen HVM S3 resume

This patch is qemu patch 2 to fix Xen HVM S3 bug, adding qemu
xen logic. When qemu wakeup, qemu xen logic is notified and
hypercall to xen hypervisor to unpause domain.

Signed-off-by: Liu Jinsong <jinsong.liu@intel.com>
Signed-off-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Reviewed-by: Anthony PERARD <anthony.perard@citrix.com>
(cherry picked from commit 11addd0ab9371af2b6ec028c7fe4e4c4992252fc)

Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>

qemu: Adjust qemu wakeup

Currently Xen hvm s3 has a bug coming from the difference between
qemu-traditioanl and qemu-xen. For qemu-traditional, the way to
resume from hvm s3 is via 'xl trigger' command. However, for
qemu-xen, the way to resume from hvm s3 inherited from standard
qemu, i.e. via QMP, and it doesn't work under Xen.

The root cause is, for qemu-xen, 'xl trigger' command didn't reset
devices, while QMP didn't unpause hvm domain though they did qemu
system reset.

We have two qemu patches and one xl patch to fix Xen hvm s3 bug.
This patch is the qemu patch 1. It adjusts qemu wakeup so that
Xen s3 resume logic (which will be implemented at qemu patch 2)
will be notified after qemu system reset.

Signed-off-by: Liu Jinsong <jinsong.liu@intel.com>
Signed-off-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Anthony PERARD <anthony.perard@citrix.com>
(cherry picked from commit 4bc78a877252d772b983810a7d2c0be00e9be70e)

Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>

coroutine: add ./configure --disable-coroutine-pool

The 'gthread' coroutine backend was written before the freelist (aka
pool) existed in qemu-coroutine.c.

This means that every thread is expected to exit when its coroutine
terminates.  It is not possible to reuse threads from a pool.

This patch automatically disables the pool when 'gthread' is used.  This
allows the 'gthread' backend to work again (for example,
tests/test-coroutine completes successfully instead of hanging).

I considered implementing thread reuse but I don't want quirks like CPU
affinity differences due to coroutine threads being recycled.  The
'gthread' backend is a reference backend and it's therefore okay to skip
the pool optimization.

Note this patch also makes it easy to toggle the pool for benchmarking
purposes:

  ./configure --with-coroutine-backend=ucontext \
              --disable-coroutine-pool

Reported-by: Gabriel Kerneis <gabriel@kerneis.info>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Reviewed-by: Gabriel Kerneis <gabriel@kerneis.info>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
(cherry picked from commit 70c60c089fdc6bf8a79324e492c13e8c08d55942)

Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>

piix4: disable io on reset

io base register at 0x40 is cleared on reset,
but io is not disabled until some other event
happens to call pm_io_space_update.

Invoke pm_io_space_update directly to make this
consistent.

Cc: qemu-stable@nongnu.org
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
(cherry picked from commit c046e8c4a26c902ca1b4f5bdf668a2da6bc75f54)

Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>

vmdk: fix cluster size check for flat extents

We use the extent size as cluster size for flat extents (where no L1/L2
table is allocated so it's safe) reuse sector calculating code with
sparse extents.

Don't pass in the cluster size for adding flat extent, just set it to
sectors later, then the cluster size checking will not fail.

The cluster_sectors is changed to int64_t to allow big flat extent.

Without this, flat extent opening is broken:

    # qemu-img create -f vmdk -o subformat=monolithicFlat /tmp/a.vmdk 100G
    Formatting '/tmp/a.vmdk', fmt=vmdk size=107374182400 compat6=off subformat='monolithicFlat' zeroed_grain=off
    # qemu-img info /tmp/a.vmdk
    image: /tmp/a.vmdk
    file format: raw
    virtual size: 0 (0 bytes)
    disk size: 4.0K

Signed-off-by: Fam Zheng <famz@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
(cherry picked from commit 301c7d38a0c359b91526391d13617386f3d9bb29)

Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>

rbd: avoid qemu_rbd_snap_list() memory leaks

When there are no snapshots qemu_rbd_snap_list() returns 0 and the
snapshot table pointer is NULL.  Don't forget to free the snaps buffer
we allocated for librbd rbd_snap_list().

When the function succeeds don't forget to free the snaps buffer after
calling rbd_snap_list_end().

Cc: qemu-stable@nongnu.org
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
(cherry picked from commit 9e6337d0818650362149b734d53edf9489f3acaa)

Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>

tap: Use numbered tap/tun devices on all *BSD OS's

The following patch simplifies the *BSD tap/tun code and makes use of numbered
tap/tun interfaces on all *BSD OS's. NetBSD has a patch in their pkgsrc tree
to make use of this feature and DragonFly also supports this as well.

Signed-off-by: Brad Smith <brad@comstyle.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
(cherry picked from commit aa4f082f7526d39dac8e2ca64d192d858014ee10)

Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>

iov: avoid "orig_len may be used unitialized" warning

Signed-off-by: Wenchao Xia <xiawenc@linux.vnet.ibm.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
(cherry picked from commit 2be178a475289286db80de5ddd7830e67e112bdd)

Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>

xhci: emulate intr endpoint intervals correctly

Respect the interval for interrupt endpoints, so we don't finish
transfers as fast as possible but at the rate configured by the guest.

Fixes guest deadlocks triggered by interrupt storms.

Cc:
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
(cherry picked from commit 4d7a81c06f5f17e019a2d3a18300500bd64f6f40)

Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>

virtio-blk: do not relay a previous driver's WCE configuration to the current

The following sequence happens:
- the SeaBIOS virtio-blk driver does not support the WCE feature, which
causes QEMU to disable writeback caching

- the Linux virtio-blk driver resets the device, finds WCE is available
but writeback caching is disabled; tells block layer to not send cache
flush commands

- the Linux virtio-blk driver sets the DRIVER_OK bit, which causes
writeback caching to be re-enabled, but the Linux virtio-blk driver does
not know of this side effect and cache flushes remain disabled

The bug is at the third step.  If the guest does know about CONFIG_WCE,
QEMU should ignore the WCE feature's state.  The guest will control the
cache mode solely using configuration space.  This change makes Linux
do flushes correctly, but Linux will keep SeaBIOS's writethrough mode.

Hence, whenever the guest is reset, the cache mode of the disk should
be reset to whatever was specified in the "-drive" option.  With this
change, the Linux virtio-blk driver finds that writeback caching is
enabled, and tells the block layer to send cache flush commands
appropriately.

Reported-by: Rusty Russell <rusty@au1.ibm.com
Cc: qemu-stable@nongnu.org
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
(cherry picked from commit ef5bc96268ceec64769617dc53b0ac3a20ff351c)

Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>

blockdev: do not default cache.no-flush to true

That's why all my VMs were so fast lately. :)

This changed in 1.6.0 by mistake in patch 29c4e2b (blockdev: Split up
'cache' option, 2013-07-18).

Cc: qemu-stable@nongnu.org
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
(cherry picked from commit 1df6fa4bc6754a170cf511a78e2e6fef84eb5228)

Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>

tci: Fix qemu-alpha on 32 bit hosts (wrong assertions)

Debian busybox-static for alpha has a load address of 0x0000000120000000
which is mapped to 0x0000000020000000 for 32 bit hosts.

qemu-alpha uses the TCG opcodes qemu_ld32, qemu_ld64, qemu_st32 and
qemu_st64 which all raise the assertion (taddr == host_addr).

Remove all assertions of this type because they are either wrong or
unnecessary (when sizeof(tcg_target_ulong) >= sizeof(target_ulong)).

Cc: qemu-stable <qemu-stable@nongnu.org>
Signed-off-by: Stefan Weil <sw@weilnetz.de>
Reviewed-by: Richard Henderson <rth@twiddle.net>
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
(cherry picked from commit 07ac4dc5db22a31e47b149abdbc5ea99013cf4de)

Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>

kvmvapic: Clear also physical ROM address when entering INACTIVE state

To avoid misinterpreting INACTIVE after migration as old qemu-kvm's
STANDBY, also clear rom_state_paddr when going back to this state.

CC: qemu-stable@nongnu.org
Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
(cherry picked from commit 4357930b8a7d2fcff2d8121ec518117428a781e7)

Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>

kvmvapic: Enter inactive state on hardware reset

ROM layout may change after reset of devices are hotplugged, so we have
to pick up the physical address again when the ROM is initialized. This
is best achieved by resetting the state to INACTIVE.

CC: qemu-stable@nongnu.org
Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
(cherry picked from commit c056bc3f3464cfae1c94b7dd633d3ec13b13b655)

Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>