VirtGL: N_SE-48185, Refactor glXCreate[New]Context error handling

There is an error in glXCreate*Context error handling:
It is possible that Open GL function failure are not reported in guest.
Thus guest would have incorrect function result and believe all is ok.
This can lead later in emulator crash sometime after error message
"Couldn't create the PBuffer" on Windows.

Note that this message appears on Windows, but the error in not handled on
any platform.

VirtGL: Fix TDIS-7153 Emulator crashed with WebGL Conformance test

This fix allows some of WebGL test to pass without crashing the emulator.

VirtGL: Fix BUG N_SE-47982

Opening http://www.ro.me/film in browser then press "Begin"
makes Tizen emulator crash.
The crash is due to SwapBuffers being called with non window
parameter and non current pixmap. This parameter is provided
in client application.

Fix calling convention mismatch on Windows.

Conflicts:

tizen/src/hw/opengl_exec.c

Change-Id: I7f8e980b105ba11a83279d5aa2bfb28b92143f24

VirtGL: Fix Bug N_SE-43071.

Reproduction scenario is :
CreatePixmap
CreateContext 1
CreateContext 2
MakeCurrent 1, Pixmap
MakeCurrent NULL, NULL
MakeCurrent 2, Pixmap => Segmentation fault in unbind_qsurface.
Pixmap has a dangling pointer to context 1 that has been destroyed.

Improve GlXPixmap management using reference counter.

This patch add a reference counter to Pixmap surfaces. Other surfaces (Window,
PBuffer) are not modified.

VirtGL: Fix memory leak in eglDestroyPixmap

The code in qemu for glXDestroyPixmap make some verifications that fails :
if ( qsurface &&
     qsurface != process->current_state->current_qsurface &&
     qsurface->glstate == NULL &&
     qsurface->type == SURFACE_PIXMAP)
    /* free  image data */

We can see that before freeing surface : qemu checks that it is pixmap, it is
not current and it is not bound.

I added some traces and this is a sample from output :
[trace:qemu:opengl][2225]> glXDestroyPixmap
[trace:qemu:opengl]glXDestroyPixmap: 0x7f6898fff150
[trace:qemu:opengl]process->current_state->current_qsurface: (nil)
[trace:qemu:opengl]qsurface->glstate: 0x7f689801eef0
[trace:qemu:opengl]qsurface->type: 1, SURFACE_PIXMAP: 1
[trace:qemu:opengl]process->current_state: 0x1627680, qsurface->glstate:
0x7f689801eef0

We can interpret this as : the application requested to free the surface, but
it was still bound in the context.

The two following reasons suggest to resolve the leak by forcing unbinding the
image from qemu.
* Having a destroyed image in a context doesn't really make sense,
* In this use case it is likely that the context will be destroyed soon after
the pixmap.

Reuse stored FB configs if possible instead of allocating new ones

remove unwanted debug traces

On windows, wgl backend, remove usage of wglBindTexImageARB.
This function is provided by extension WGL_ARB_render_texture. The extension is
not supported by Intel hardware. So instead, use glReadPixels/glTexImage2D

Additionnally do not flip pbuffer vertically when using them as textures.

y-axis flipped issue when use evas gl backend got fixed.

Merge "YaGL : modified the variable tpye" into tizen_qemu_1.6

YaGL : modified the variable tpye

Modified the variable type for CPUState used by yagl_thread, etc.
(CPUArchState * -> CPUState *)
It has satisfied for the changes of passing argument to QEMU 1.6,
i.e., compilation warnings disappear.

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

qemu: gathering ecs modules into a ecs folder

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

Fix compilation errors on VIGS/YaGL sources

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

Fix compilation errors

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

Fix compilation errors on maru specific sources

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

Merge tag 'v1.6.0' into tizen_qemu_1.6

Signed-off-by: SeokYeon Hwang <syeon.hwang@samsung.com>
Conflicts:
VERSION
arch_init.c
block/vmdk.c
configure
cpu-exec.c
cpus.c
exec.c
hw/9pfs/virtio-9p-device.c
hw/display/vga-pci.c
hw/pci-host/q35.c
hw/usb/dev-storage.c
hw/virtio/virtio-mmio.c
include/exec/cpu-defs.h
include/qemu/error-report.h
include/qemu/log.h
include/sysemu/kvm.h
include/sysemu/sysemu.h
pc-bios/bios.bin
qapi-schema.json
qemu-char.c
ui/input.c
util/oslib-win32.c
vl.c

menu: change a parent of screenshot shell

change a parent of screenshot shell
and modified the init position

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

Add Input Device for mouse

- modify ps2 mouse point, ABS to REL
- modify typo
- add input.mouse parameter

Change-Id: I15d01cc6e49dc0a9b36e80a090da70ed80824b1f
Signed-off-by: hyunjin816.lee <hyunjin816.lee@samsung.com>

skin: modified test.hb.ignore to hb.ignore

remote: add conditional sentence for remote excution

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

display: added mutex lock to SDL_Quit

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

package: version up

1.5.87

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

display: modified SDL logic

SDL_SetVideoMode should be called in the main thread

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

display: removed sdl_opengl source code

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

build: change space to tab

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

build: modify build script for macosx

add libraries and header files
add file modification by sed

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

qemu: removed warnings & unnecessary logs, and changed log location.
replaced tab to 4 white space

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

Merge "build-script: apply emulator-lib-dev package for windows-32" into develop

build-script: apply emulator-lib-dev package for windows-32

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

keyboard: seperate a key event on Windows

seperate a key event if the release events
were superimposed

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

package: apply emulator-lib-dev package

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

VIGS: Make a single memcpy call inside vigs_server_update_display when possible

If display stride == root surface stride one memcpy will suffice

YaGL: Cache minmax index query result in yagl_gles_buffer

This seemed to be a bottleneck, according to perf. With
this fix minmax index query is no longer "red" in perf report

Merge "qemu: set 100ms timeout to epoll_wait function" into develop

qemu: set 100ms timeout to epoll_wait function

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

vga: added CONFIG_MARU

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

display: blank-guide image is replaced

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

menu: modified about image loading & etc

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

display: enable blank guide on maru_shm

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

Merge "semaphore: fix a hangup problem under load on NetBSD hosts." into develop

Merge "package: version up" into develop

Merge "sdb: remove calling sdb notification" into develop

Merge "mac: change directory for temp file" into develop

package: version up

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

sdb: remove calling sdb notification

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

mac: change directory for temp file

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

semaphore: fix a hangup problem under load on NetBSD hosts.

Fix following bugs in "fallback implementation of counting semaphores
with mutex+condvar" added in c166cb72f1676855816340666c3b618beef4b976:
 - waiting threads are not restarted properly if more than one threads
   are waiting unblock signals in qemu_sem_timedwait()
 - possible missing pthread_cond_signal(3) calls when waiting threads
   are returned by ETIMEDOUT
 - fix an uninitialized variable
The problem is analyzed by and fix is provided by Noriyuki Soda.

Also put additional cleanup suggested by Laszlo Ersek:
 - make QemuSemaphore.count unsigned (it won't be negative)
 - check a return value of in pthread_cond_wait() in qemu_sem_wait()

Change-Id: Ic785784de211427cd3d0db746dbc1dbac3c8bc33
Signed-off-by: Izumi Tsutsui <tsutsui@ceres.dti.ne.jp>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Message-id: 1372841894-10634-1-git-send-email-tsutsui@ceres.dti.ne.jp
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>

skin: ImageRegistry refactoring

and added blank-guide image loading

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

communication: added DRAW_BLANK_GUIDE protocol

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

Merge "YaGL: Fixed framebuffers and renderbuffers leak" into develop

Merge "YaGL: Support EGL_ALPHA_MASK_SIZE" into develop

Merge "YaGL: GL_EXT_texture_format_BGRA8888 implementation bug fixed" into develop

Merge "YaGL: EGL_KHR_surfaceless_context implemented" into develop

Merge "YaGL: Don't check 'target' in eglCreateImageKHR" into develop

Merge "YaGL: GL_EXT_texture_format_BGRA8888 implemented" into develop

Merge "YaGL: glFinish removed, using single render finish command winsys: added 'set_dirty' to winsys_surface" into develop

Merge "YaGL: Remove remaining X11 specific stuff from protocol" into develop

Merge "sdb: add device_serial_number" into develop

sdb: add device_serial_number

device_serial_number = base_port + 1

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

emulator: remove double quotation marks from the build info

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

guest_server: Changed sdcard path.

Since emulator-vms path under the tizen-sdk-data has been changed as emulator,
qemu is required to modify the path.

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

skin: optimize the hover drawing

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

YaGL: Fixed framebuffers and renderbuffers leak

package: version up

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

sdb: call sdb notify when get sensor server notification

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

skin: modified qHD Hwkey region

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

skin: added special keyWindow

Change-Id: Id8a5f048d26be3a80f9b58917e1412ba1c49353e
Signed-off-by: hyunjin816.lee <hyunjin816.lee@samsung.com>

Merge "tizen: Makefile clean-up." into develop

tizen: Makefile clean-up.

There were duplicated commands in the Makefile because of host OS.

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

skin: remove previous hwkey hover always

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

display: blank guide image processing & align

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

package: version up

1.5.80

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

skin: modified skin info.ini

added skin.profile property

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

skin: modified skin folder name

emul-720x1280-3btn -> mobile-720x1280-3btn

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

guest_server: unified string utility functions as glib.

c and glib string functions were mixed.

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

Fix bug when detect sdk data path in sdk.info

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

skin: set states before menu creating

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

skin: modified qHD skin name

Phone 3btn 540x960 -> Phone 540x960

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

Update version for 1.6.0

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

mips_malta: do not raise exceptions when accessing invalid memory

Since commit c658b94f6e8c206c59d02aa6fbac285b86b53d2c, MIPS raises
exceptions when accessing invalid memory. This is not the correct
behaviour for MIPS Malta Core LV, as the GT-64120A system controller
just ignore undecoded access. This feature is used by the Linux kernel
to probe for some devices.

Emulate the correct behaviour in QEMU by adding an empty slot covering
the entire memory space decoded by the GT-64120A.

Tested-by: Stefan Weil <sw@weilnetz.de>
Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>

block: Dont ignore previously set bdrv_flags

bdrv_flags is set by bdrv_parse_discard_flags(), but later it is reset
to zero.

Signed-off-by: M. Mohan Kumar <mohan@in.ibm.com>
Message-id: 1376483201-13466-1-git-send-email-mohan@in.ibm.com
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>

qemu-char: fix infinite recursion connecting to monitor pty

Since commit bd5c51e (qemu-char: don't issue CHR_EVENT_OPEN in a BH), an
infinite recursion occurs when putting the monitor on a pty (-monitor
pty) and connecting a terminal to the slave port.

This is because of the qemu_chr_be_event(s, CHR_EVENT_OPENED) added to
qemu_chr_be_generic_open(). This event is captured by monitor_event()
which prints a welcome message to the character device. The flush of
that welcome message retriggers another open event in pty_chr_state()
because it checks s->connected, but only sets it to 1 after calling
qemu_chr_be_generic_open().

I've fixed this by setting s->connected = 1 before the call to
qemu_chr_be_generic_open() instead of after, so that the recursive
pty_chr_state() doesn't call it again.

An example snippet of repeating backtrace:
 ...
 #107486 0x007aec58 in monitor_flush (mon=0xf418b0) at qemu/monitor.c:288
 #107487 0x007aee7c in monitor_puts (mon=0xf418b0, str=0x1176d07 "") at qemu/monitor.c:322
 #107488 0x007aef20 in monitor_vprintf (mon=0xf418b0, fmt=0x8d4820 "QEMU %s monitor - type 'help' for more information\n",
     ap=0x7f432be0) at qemu/monitor.c:339
 #107489 0x007aefac in monitor_printf (mon=0xf418b0, fmt=0x8d4820 "QEMU %s monitor - type 'help' for more information\n")
     at qemu/monitor.c:347
 #107490 0x007ba4bc in monitor_event (opaque=0xf418b0, event=2) at qemu/monitor.c:4699
 #107491 0x00684c28 in qemu_chr_be_event (s=0xf37788, event=2) at qemu/qemu-char.c:108
 #107492 0x00684c70 in qemu_chr_be_generic_open (s=0xf37788) at qemu/qemu-char.c:113
 #107493 0x006880a4 in pty_chr_state (chr=0xf37788, connected=1) at qemu/qemu-char.c:1145
 #107494 0x00687fa4 in pty_chr_update_read_handler (chr=0xf37788) at qemu/qemu-char.c:1121
 #107495 0x00687c9c in pty_chr_write (chr=0xf37788, buf=0x70b3c008 <Address 0x70b3c008 out of bounds>, len=538720)
     at qemu/qemu-char.c:1063
 #107496 0x00684cc4 in qemu_chr_fe_write (s=0xf37788, buf=0x70b3c008 <Address 0x70b3c008 out of bounds>, len=538720)
     at qemu/qemu-char.c:118
 ...

Signed-off-by: James Hogan <james.hogan@imgtec.com>
Tested-by: Michael Roth <mdroth@linux.vnet.ibm.com>
Message-id: 1375960178-10882-1-git-send-email-james.hogan@imgtec.com
Cc: Michael Roth <mdroth@linux.vnet.ibm.com>
Cc: Anthony Liguori <aliguori@us.ibm.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>

pvpanic: fix bad merge

Context matching caused the 'has_pvpanic = true' to be applied to
the 1.6 machine type instead of the 1.5 machine type.

Reported-by: Markus Armbruster <armbru@redhat.com>
Reported-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>

ecs: fix select procedure

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

skin: added qHD skin

emul-540x960-3btn

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

package: version up

1.5.77

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

emulator: modified lib linking on Windows

lpng -> lpng14

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

Update version for 1.6.0-rc3

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

Merge remote-tracking branch 'afaerber/tags/qom-cpu-for-anthony' into staging

QOM CPUState refactorings

* Fix X86CPU Westmere CPUID for pc-*-1.4 and older

* afaerber/tags/qom-cpu-for-anthony:
  pc: Remove PCLMULQDQ from Westmere on pc-*-1.4 and older

Conflicts:
hw/i386/pc_piix.c
hw/i386/pc_q35.c

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

pc: drop external DSDT loading

This breaks migration and is unneeded with modern SeaBIOS.

Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
Message-id: 1376316104-11269-1-git-send-email-aliguori@us.ibm.com

hw/misc: make pvpanic known to user

This patch is based on Hu Tao's:
http://lists.nongnu.org/archive/html/qemu-devel/2013-08/msg00125.html

The pvpanic device may be enabled now with "-device pvpanic"
from command line.

Signed-off-by: Marcel Apfelbaum <marcel.a@redhat.com>
Reviewed-by: Hu Tao <hutao@cn.fujitsu.com>
Message-id: 1376233843-19410-3-git-send-email-marcel.a@redhat.com
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>

hw/misc: don't create pvpanic device by default

This patch is based on Hu Tao's:
http://lists.nongnu.org/archive/html/qemu-devel/2013-08/msg00124.html

No need to hard-code pvpanic as part of the machine.
It can be added with "-device pvpanic" from command line (The next patch).
Anyway, for backport compatibility it is still part of 1.5
machine.

Signed-off-by: Marcel Apfelbaum <marcel.a@redhat.com>
Reviewed-by: Hu Tao <hutao@cn.fujitsu.com>
Message-id: 1376233843-19410-2-git-send-email-marcel.a@redhat.com
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>

pc: Remove PCLMULQDQ from Westmere on pc-*-1.4 and older

Commit 41cb383f42d0cb51d8e3e25e3ecebc954dd4196f made a guest-visible
change by adding the PCLMULQDQ bit to Westmere without adding
compatibility code to keep the ABI for older machine-types.
Fix it by adding the missing compat code.

Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>

YaGL: Support EGL_ALPHA_MASK_SIZE

glmark2 requires it

rdma: remaining documentation fixes

Was missing 'setup-time' in some of the QMP documentation...

Signed-off-by: Michael R. Hines <mrhines@us.ibm.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-id: 1376078746-24948-7-git-send-email-mrhines@linux.vnet.ibm.com
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>

rdma: IPv6 over Ethernet (RoCE) is broken in linux - workaround

We've gotten reports from multiple testers (including Frank Yangjie
and myself) that RDMA IPv6 support over RocE (Ethernet) is broken
in linux.

A patch to Linux is still in review:

http://comments.gmane.org/gmane.linux.drivers.rdma/16448

If the user is listening on '[::]', then we will not have a opened a device
yet and have no way of verifying if the device is RoCE or not.

In this case, the source VM will throw an error for ALL types of
connections (both IPv4 and IPv6) if the destination machine does not have
a regular infiniband network available for use.

The only way to gaurantee that an error is thrown for broken kernels is
for the management software to choose a *specific* interface at bind time
and validate what time of hardware it is.

Unfortunately, this puts the user in a fix:

 If the source VM connects with an IPv4 address without knowing that the
 destination has bound to '[::]' the migration will unconditionally fail
 unless the management software is not explicitly listening on the the IPv4
 address while using a RoCE-based device.

 If the source VM connects with an IPv6 address, then we're OK because we can
 throw an error on the source (and similarly on the destination).

 But in mixed environments, this will be broken for a while until it is fixed
 inside linux.

We do provide a *tiny* bit of help in mixed environments, though in this patch:

We can list all of the devices in the system and check to see if all the
devices are RoCE or Infiniband.

If we detect that we have a *pure* RoCE environment, then we can safely
thrown an error even if the management sofware has specified '[::]' as the
bind address.

However, if there is are multiple hetergeneous devices, then we cannot make
this assumption and the user just has to be sure they know what they are doing.

Signed-off-by: Michael R. Hines <mrhines@us.ibm.com>
Message-id: 1376078746-24948-6-git-send-email-mrhines@linux.vnet.ibm.com
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>

rdma: proper getaddrinfo() handling

getaddrinfo() already knows what it's doing,
but it can potentially return multiple addresses.
We need to handle that...

Reviewed-by: Orit Wasserman <owasserm@redhat.com>
Signed-off-by: Michael R. Hines <mrhines@us.ibm.com>
Message-id: 1376078746-24948-5-git-send-email-mrhines@linux.vnet.ibm.com
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>

rdma: check if RDMAControlHeader::len match transferred byte

RDMAControlHeader::len is provided from remote, so check if the value
match the actual transferred byte_len.

Reviewed-by: Orit Wasserman <owasserm@redhat.com>
Reviewed-by: Michael R. Hines <mrhines@us.ibm.com>
Signed-off-by: Isaku Yamahata <yamahata@private.email.ne.jp>
Signed-off-by: Michael R. Hines <mrhines@us.ibm.com>
Message-id: 1376078746-24948-4-git-send-email-mrhines@linux.vnet.ibm.com
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>

rdma: validate RDMAControlHeader::len

RMDAControlHeader::len is provided from remote, so validate it.

Reviewed-by: Orit Wasserman <owasserm@redhat.com>
Reviewed-by: Michael R. Hines <mrhines@us.ibm.com>
Signed-off-by: Isaku Yamahata <yamahata@private.email.ne.jp>
Signed-off-by: Michael R. Hines <mrhines@us.ibm.com>
Message-id: 1376078746-24948-3-git-send-email-mrhines@linux.vnet.ibm.com
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>