Public bug reported:
Hi all,
I recently tried to upgrade from Qemu 2.4.1 to 2.6.0, but found some
problems with VNC reverse connections.
1) In "-vnc 172.16.1.3:5902,reverse" used to mean "connect to port 5902"
That seems to have changed changed since 2.4.1, the thing after the IP
address is
The script is not prefixed with test- so it won't run with "make docker-test",
because it can take too long.
Run it with "make docker-travis@ubuntu".
Reviewed-by: Alex Bennée
Signed-off-by: Fam Zheng
Message-id: 1463986466-764-12-git-send-email-f...@redhat.com
---
tests/docker/travis| 21 +
The (currently partially commented out) configure options are suggested
by John Snow .
Signed-off-by: Fam Zheng
Reviewed-by: Alex Bennée
Message-id: 1463986466-764-10-git-send-email-f...@redhat.com
---
tests/docker/test-clang | 26 ++
1 file changed, 26 insertions(+)
cr
Reviewed-by: Alex Bennée
Signed-off-by: Fam Zheng
Message-id: 1463986466-764-15-git-send-email-f...@redhat.com
---
MAINTAINERS | 7 +++
1 file changed, 7 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 3700d51..e8ff270 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1614,3 +1614,1
This builds all available targets.
Signed-off-by: Fam Zheng
Reviewed-by: Alex Bennée
Message-id: 1463986466-764-9-git-send-email-f...@redhat.com
---
tests/docker/test-full | 17 +
1 file changed, 17 insertions(+)
create mode 100755 tests/docker/test-full
diff --git a/tests/doc
Reviewed-by: Alex Bennée
Signed-off-by: Fam Zheng
Message-id: 1463986466-764-8-git-send-email-f...@redhat.com
---
tests/docker/test-quick | 19 +++
1 file changed, 19 insertions(+)
create mode 100755 tests/docker/test-quick
diff --git a/tests/docker/test-quick b/tests/docker/te
Reviewed-by: Alex Bennée
Signed-off-by: Fam Zheng
Message-id: 1463986466-764-11-git-send-email-f...@redhat.com
---
tests/docker/test-mingw | 34 ++
1 file changed, 34 insertions(+)
create mode 100755 tests/docker/test-mingw
diff --git a/tests/docker/test-mingw b
Whatever passed in this variable will be appended to all
configure commands.
Signed-off-by: Fam Zheng
Reviewed-by: Alex Bennée
Message-id: 1463986466-764-14-git-send-email-f...@redhat.com
---
tests/docker/Makefile.include | 3 +++
tests/docker/common.rc| 1 +
2 files changed, 4 insertio
Using "," literal in $(call quiet-command, ...) arguments is awkward.
Add this constant to make it at least doable.
Signed-off-by: Fam Zheng
Message-id: 1463986466-764-3-git-send-email-f...@redhat.com
---
rules.mak | 2 ++
1 file changed, 2 insertions(+)
diff --git a/rules.mak b/rules.mak
index
Reviewed-by: Alex Bennée
Signed-off-by: Fam Zheng
Message-id: 1463986466-764-13-git-send-email-f...@redhat.com
---
docs/build-system.txt | 5 +
1 file changed, 5 insertions(+)
diff --git a/docs/build-system.txt b/docs/build-system.txt
index 5ea..2af1e66 100644
--- a/docs/build-system.tx
"requires" checks the "FEATURE" environment for specified prerequisits,
and skip the execution of test if not found.
"build_qemu" is the central routine to compile QEMU for tests to call.
Reviewed-by: Alex Bennée
Signed-off-by: Fam Zheng
Message-id: 1463986466-764-7-git-send-email-f...@redhat.c
The following changes since commit 84cfc756d158a061bd462473d42b0a9f072218de:
Merge remote-tracking branch 'remotes/awilliam/tags/vfio-update-20160526.1'
into staging (2016-05-26 19:18:08 +0100)
are available in the git repository at:
git://github.com/famz/qemu tags/pull-docker-20160527-1
f
Reviewed-by: Alex Bennée
Signed-off-by: Fam Zheng
Message-id: 1463986466-764-5-git-send-email-f...@redhat.com
---
tests/docker/dockerfiles/centos6.docker | 6 ++
tests/docker/dockerfiles/fedora.docker | 7 +++
tests/docker/dockerfiles/ubuntu.docker | 11 +++
3 files changed,
docker.py is added with a number of useful subcommands to manager docker
images and instances for QEMU docker testing. Subcommands are:
run: A wrapper of "docker run" (or "sudo -n docker run" if necessary),
which takes care of killing and removing the running container at
SIGINT.
clean: Tear down
It's better to have a launcher for all tests, to make it easier to
initialize and manage the environment.
If "DEBUG=1" a shell prompt will show up before the test runs.
Reviewed-by: Alex Bennée
Signed-off-by: Fam Zheng
Message-id: 1463986466-764-6-git-send-email-f...@redhat.com
---
tests/dock
This adds a group of make targets to run docker tests, all are available
in source tree without running ./configure.
The usage is shown with "make docker".
Besides the fixed ones, dynamic targets for building each image and
running each test in each image are generated automatically by make,
scan
Hi Richard,
On Friday, 27 May 2016, Richard Henderson wrote:
> Cc: Claudio Fontana >
> Signed-off-by: Richard Henderson >
> ---
> tcg/aarch64/tcg-target.inc.c | 7 +++
> 1 file changed, 7 insertions(+)
>
> diff --git a/tcg/aarch64/tcg-target.inc.c b/tcg/aarch64/tcg-target.inc.c
> index 144
On Fri, May 27, 2016 at 10:14 AM, David Gibson
wrote:
> On Tue, May 17, 2016 at 11:02:48AM +0530, Bharata B Rao wrote:
>> On Mon, May 16, 2016 at 11:55 AM, Alexey Kardashevskiy
>> wrote:
>> > On 05/13/2016 06:41 PM, Bharata B Rao wrote:
>> >>
>> >> On Wed, May 4, 2016 at 12:22 PM, Alexey Kardash
Functions of type FindSysbusDeviceFunc currently return an integer.
However, this return value is always ignored by the caller in
find_sysbus_device().
This changes the function type to return void, to avoid confusion over
the function semantics.
Signed-off-by: David Gibson
---
hw/arm/sysbus-fd
Value matching allows Linux to boot with CONFIG_NO_HZ_IDLE=y on the
palmetto-bmc machine. Two match registers are provided for each timer.
Signed-off-by: Andrew Jeffery
---
The change pulls out ptimer in favour of the regular timer infrastructure. As a
consequence it implements the conversions b
From: Steven Luo
On my Debian jessie system, including nettle/pbkdf2.h does not cause
NULL to be defined, which causes the test to fail to compile. Include
stddef.h to bring in a definition of NULL.
Cc: qemu-triv...@nongnu.org
Cc: qemu-sta...@nongnu.org
Signed-off-by: Steven Luo
---
I suppose
On Sat, Apr 30, 2016 at 10:48:00AM +1000, Anton Blanchard wrote:
> We don't support transactional memory in PR KVM, so don't tell
> the OS that we do.
>
> Signed-off-by: Anton Blanchard
Relying on CAP_PPC_GET_PVINFO is a hack we need in some cases, but
it's not something to be encouraged. I'd p
On Fri, May 13, 2016 at 02:11:48PM +0530, Bharata B Rao wrote:
> On Wed, May 4, 2016 at 12:22 PM, Alexey Kardashevskiy wrote:
> > This adds support for Dynamic DMA Windows (DDW) option defined by
> > the SPAPR specification which allows to have additional DMA window(s)
> >
> > The "ddw" property i
On Tue, May 17, 2016 at 11:02:48AM +0530, Bharata B Rao wrote:
> On Mon, May 16, 2016 at 11:55 AM, Alexey Kardashevskiy wrote:
> > On 05/13/2016 06:41 PM, Bharata B Rao wrote:
> >>
> >> On Wed, May 4, 2016 at 12:22 PM, Alexey Kardashevskiy
> >> wrote:
> >
> >
> >>
> >>> +
> >>> +avail = SPAPR
On 27.05.2016 05:38, David Gibson wrote:
> On Tue, May 03, 2016 at 06:03:29PM +0200, Cédric Le Goater wrote:
>> From: Benjamin Herrenschmidt
>>
>> We use an env. flag which is set to the initial value of MSR_HVB in
>> the msr_mask. We also adjust the POWER8 mask to set SHV.
>>
>> Also use this to
On Thu, May 26, 2016 at 9:00 PM, Richard Henderson wrote:
> This is a reorg of Pranith's first patch set, correcting a few
> mistakes and adding backend support for all of the other hosts.
>
> In addition, I added front-end support for Alpha, since I didn't
> actually have any armv7 images handy t
On Fri, May 27, 2016 at 01:49:19PM +1000, Alexey Kardashevskiy wrote:
> On 17/05/16 06:20, Alex Williamson wrote:
> > On Mon, 16 May 2016 14:52:41 +1000
> > Alexey Kardashevskiy wrote:
> >
> >> On 05/14/2016 08:26 AM, Alex Williamson wrote:
> >>> On Wed, 4 May 2016 16:52:30 +1000
> >>> Alexey Ka
On Thu, May 26, 2016 at 10:02:10AM +0200, Greg Kurz wrote:
> The KVM API restricts vcpu ids to be < KVM_CAP_MAX_VCPUS. On PowerPC
> targets, depending on the number of threads per core in the host and
> in the guest, some topologies do generate higher vcpu ids actually.
> When this happens, QEMU ba
On Thu, May 26, 2016 at 10:02:23AM +0200, Greg Kurz wrote:
> As stated in linux/Documentation/virtual/kvm/api.txt:
>
> The maximum possible value for max_vcpu_id can be retrieved using the
> KVM_CAP_MAX_VCPU_ID of the KVM_CHECK_EXTENSION ioctl() at run-time.
>
> If the KVM_CAP_MAX_VCPU_ID does no
On Tue, May 03, 2016 at 06:03:22PM +0200, Cédric Le Goater wrote:
> Hello,
>
> Here is a new set of fixes extracted from Ben's PowerNV tree :
>
> https://github.com/ozbenh/qemu/commits/powernv
>
> It was quickly tested with a pseries guest using KVM and TCG.
Ok, back now.
I've merged thes
On Tue, May 03, 2016 at 06:03:29PM +0200, Cédric Le Goater wrote:
> From: Benjamin Herrenschmidt
>
> We use an env. flag which is set to the initial value of MSR_HVB in
> the msr_mask. We also adjust the POWER8 mask to set SHV.
>
> Also use this to adjust ctx.hv so that it is *set* when the proc
On 17/05/16 06:20, Alex Williamson wrote:
> On Mon, 16 May 2016 14:52:41 +1000
> Alexey Kardashevskiy wrote:
>
>> On 05/14/2016 08:26 AM, Alex Williamson wrote:
>>> On Wed, 4 May 2016 16:52:30 +1000
>>> Alexey Kardashevskiy wrote:
>>>
New VFIO_SPAPR_TCE_v2_IOMMU type supports dynamic DMA
On 2016年05月26日 23:08, Peter Maydell wrote:
On 26 May 2016 at 03:16, Jason Wang wrote:
The following changes since commit 287db79df8af8e31f18e262feb5e05103a09e4d4:
Merge remote-tracking branch 'remotes/ehabkost/tags/x86-pull-request' into
staging (2016-05-24 13:06:33 +0100)
are available
From: Alexey Kardashevskiy
6a81dd17 "spapr_iommu: Rename vfio_accel parameter" renamed vfio_accel
flag everywhere but one spot was missed.
Signed-off-by: Alexey Kardashevskiy
Reviewed-by: David Gibson
Signed-off-by: David Gibson
---
target-ppc/kvm_ppc.h | 2 +-
1 file changed, 1 insertion(+)
On Thu, May 26, 2016 at 12:12:41PM +0200, Paolo Bonzini wrote:
>
>
> On 12/05/2016 05:48, Bharata B Rao wrote:
> > CPUState *cpu gets added to the cpus list during cpu_exec_init(). It
> > should be removed from cpu_exec_exit().
> >
> > cpu_exec_exit() is called from generic CPU::instance_finaliz
From: Alexey Kardashevskiy
The user could have picked LIOBN via the CLI but the device tree
rendering code would still use the value derived from the PHB index
(which is the default fallback if LIOBN is not set in the CLI).
This replaces SPAPR_PCI_LIOBN() with the actual DMA LIOBN value.
Signed
From: Richard Henderson
A 32-bit rotate insn is more common on hosts than a deposit insn,
and if the host has neither the result is truely horrific.
At the same time, tidy up the temporaries within these functions,
drop the over-use of "likely", drop some checks for identity that
will also be ch
From: Alexey Kardashevskiy
At the moment presence of vfio-pci devices on a bus affect the way
the guest view table is allocated. If there is no vfio-pci on a PHB
and the host kernel supports KVM acceleration of H_PUT_TCE, a table
is allocated in KVM. However, if there is vfio-pci and we do yet no
I've been de facto co-maintainer of all ppc target related code for some
time. Alex Graf isworking on other things and doesn't have a whole lot of
time for qemu ppc maintainership. So, update the MAINTAINERS file to
reflect this.
Signed-off-by: David Gibson
Reviewed-by: Alexander Graf
Acked-by
From: Jianjun Duan
There are possible racing situations involving hotplug events and
guest migration. For cases where a hotplug event is migrated, or
the guest is in the process of fetching device tree at the time of
migration, we need to ensure the device tree is created and
associated with the
From: Thomas Huth
Currently, the spapr-vlan device is trying to flush the RX queue
after each RX buffer that has been added by the guest via the
H_ADD_LOGICAL_LAN_BUFFER hypercall. In case the receive buffer pool
was empty before, we only pass single packets to the guest this
way. This can cause
From: Thomas Huth
The last 8 bytes of the receive buffer list page (that has been supplied
by the guest with the H_REGISTER_LOGICAL_LAN call) contain a counter
for frames that have been dropped because there was no suitable receive
buffer available. This patch introduces code to use this field to
From: Greg Kurz
The KVM API restricts vcpu ids to be < KVM_CAP_MAX_VCPUS. On PowerPC
targets, depending on the number of threads per core in the host and
in the guest, some topologies do generate higher vcpu ids actually.
When this happens, QEMU bails out with the following error:
kvm_init_vcpu
From: Richard Henderson
Signed-off-by: Richard Henderson
Signed-off-by: David Gibson
---
target-ppc/translate.c | 29 +++--
1 file changed, 11 insertions(+), 18 deletions(-)
diff --git a/target-ppc/translate.c b/target-ppc/translate.c
index 745f4de..3ea6625 100644
---
ppc_hash64_set_external_hpt() was added in e5c0d3c "target-ppc: Add helpers
for updating a CPU's SDR1 and external HPT". This helper contains a
cpu_synchronize_state() since it may need to push state back to KVM
afterwards.
This turns out to break things when it is used in the reset path, which i
From: Zhou Jie
This patch adds check for negative return value from get_image_size(),
where it is missing. It avoids unnecessary two function calls.
Signed-off-by: Zhou Jie
Signed-off-by: David Gibson
---
hw/ppc/spapr.c | 4
1 file changed, 4 insertions(+)
diff --git a/hw/ppc/spapr.c b/
From: Richard Henderson
Mirror the cleanups just done to rlwinm, rlwnm and rlwimi.
This adds use of deposit to rldimi.
Signed-off-by: Richard Henderson
Signed-off-by: David Gibson
---
target-ppc/translate.c | 91 +-
1 file changed, 46 insertions
The following changes since commit 84cfc756d158a061bd462473d42b0a9f072218de:
Merge remote-tracking branch 'remotes/awilliam/tags/vfio-update-20160526.1'
into staging (2016-05-26 19:18:08 +0100)
are available in the git repository at:
git://github.com/dgibson/qemu.git tags/ppc-for-2.7-201605
Public bug reported:
getting weird annoying seabios hell and not sure how to fix it.
ok.
there IS a SEA-BIOS. There IS a way in.
-I found it by mistake.(and yall need to move the BIOS key...its in the
wrong place)
I was tryng to boot Yosemite to re-install. I mashed the key too early
and it wa
From: Chen Fan
For supporting aer recovery, host and guest would run the same aer
recovery code, that would do the secondary bus reset if the error
is fatal, the aer recovery process:
1. error_detected
2. reset_link (if fatal)
3. slot_reset/mmio_enabled
4. resume
It indicates that host w
在 2016年05月25日 21:15, Paolo Bonzini 写道:
@@ -310,8 +312,7 @@ static void pl011_class_init(ObjectClass *oc, void *data)
>> >
>> > dc->realize = pl011_realize;
>> > dc->vmsd = &vmstate_pl011;
>> >-/* Reason: realize() method uses qemu_char_get_next_serial() */
>> >-dc->cannot_inst
在 2016年05月25日 21:08, Peter Maydell 写道:
-
> /* AUX / UART1 */
>-/* TODO: don't call qemu_char_get_next_serial() here, instead set
>- * chardev properties for each uart at the board level, once pl011
>- * (uart0) has been updated to avoid qemu_char_get_next_serial()
>- */
Thi
Ping here : )
Hi fam, do you have time to help reviewing this patchset? Consider of we
are in the same time zone what will speed up code reviewing process,
any feedback will be appreciated.
Thanks
-Xie
On 05/20/2016 03:36 PM, Changlong Xie wrote:
Block replication is a very important
On 05/20/2016 03:36 PM, Changlong Xie wrote:
+static void io_write(BlockDriverState *bs, long pattern, int64_t pattern_count,
+ int64_t offset, int64_t count, bool expect_failed)
+{
+void *pattern_buf;
Should initialize as NULL to avoid below warnning:
tests/test-replic
On Thu, May 19, 2016 at 05:11:35PM +0500, Talha Imran wrote:
> With specification at hand from the reference manual from Freescale
> http://cache.nxp.com/files/32bit/doc/ref_manual/SPEPEM.pdf , I have found a
> fix
> to efscmp* instructions handling in QEMU.
>
> efscmp* instructions in QEMU set c
Cc: Blue Swirl
Signed-off-by: Richard Henderson
---
tcg/sparc/tcg-target.inc.c | 8
1 file changed, 8 insertions(+)
diff --git a/tcg/sparc/tcg-target.inc.c b/tcg/sparc/tcg-target.inc.c
index 9938a50..16d8d8f 100644
--- a/tcg/sparc/tcg-target.inc.c
+++ b/tcg/sparc/tcg-target.inc.c
@@ -2
Signed-off-by: Pranith Kumar
Message-Id: <20160524171856.1000-4-bobby.pr...@gmail.com>
Signed-off-by: Richard Henderson
---
target-arm/translate.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/target-arm/translate.c b/target-arm/translate.c
index e525f1e..0f71800 100
Signed-off-by: Richard Henderson
---
tcg/ppc/tcg-target.inc.c | 8
1 file changed, 8 insertions(+)
diff --git a/tcg/ppc/tcg-target.inc.c b/tcg/ppc/tcg-target.inc.c
index da10052..ea576f9 100644
--- a/tcg/ppc/tcg-target.inc.c
+++ b/tcg/ppc/tcg-target.inc.c
@@ -469,6 +469,9 @@ static int
Signed-off-by: Richard Henderson
---
target-alpha/translate.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/target-alpha/translate.c b/target-alpha/translate.c
index 76dab15..2dbcb82 100644
--- a/target-alpha/translate.c
+++ b/target-alpha/translate.c
@@ -2334,11 +2334,1
Signed-off-by: Richard Henderson
---
tcg/mips/tcg-target.inc.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/tcg/mips/tcg-target.inc.c b/tcg/mips/tcg-target.inc.c
index 50e98ea..cad1d4d 100644
--- a/tcg/mips/tcg-target.inc.c
+++ b/tcg/mips/tcg-target.inc.c
@@ -292,6 +292,7 @@ typedef
Cc: Stefan Weil
Signed-off-by: Richard Henderson
---
tcg/tci/tcg-target.inc.c | 3 +++
tci.c| 3 +++
2 files changed, 6 insertions(+)
diff --git a/tcg/tci/tcg-target.inc.c b/tcg/tci/tcg-target.inc.c
index fa74d52..bf65416 100644
--- a/tcg/tci/tcg-target.inc.c
+++ b/tcg/tci/t
Cc: Alexander Graf
Signed-off-by: Richard Henderson
---
tcg/s390/tcg-target.inc.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/tcg/s390/tcg-target.inc.c b/tcg/s390/tcg-target.inc.c
index e0a60e6..4c63621 100644
--- a/tcg/s390/tcg-target.inc.c
+++ b/tcg/s390/tcg-target.inc.c
@@ -3
Cc: Aurelien Jarno
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.inc.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/tcg/ia64/tcg-target.inc.c b/tcg/ia64/tcg-target.inc.c
index 395223e..6bbc6dc5 100644
--- a/tcg/ia64/tcg-target.inc.c
+++ b/tcg/ia64/tcg-target.inc.c
@@ -247,
Cc: Claudio Fontana
Signed-off-by: Richard Henderson
---
tcg/aarch64/tcg-target.inc.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/tcg/aarch64/tcg-target.inc.c b/tcg/aarch64/tcg-target.inc.c
index 1447f7c..839569d 100644
--- a/tcg/aarch64/tcg-target.inc.c
+++ b/tcg/aarch64/tcg-targ
This commit introduces the TCGOpcode for fence instruction.
Signed-off-by: Pranith Kumar
Message-Id: <20160524171856.1000-2-bobby.pr...@gmail.com>
Signed-off-by: Richard Henderson
---
tcg/tcg-op.c | 6 ++
tcg/tcg-op.h | 2 ++
tcg/tcg-opc.h | 2 ++
3 files changed, 10 insertions(+)
diff -
Signed-off-by: Pranith Kumar
Message-Id: <20160524171856.1000-3-bobby.pr...@gmail.com>
[rth: Check for sse2, fallback to locked memory op otherwise.]
Signed-off-by: Richard Henderson
---
tcg/i386/tcg-target.inc.c | 35 +++
1 file changed, 35 insertions(+)
diff --
This is a reorg of Pranith's first patch set, correcting a few
mistakes and adding backend support for all of the other hosts.
In addition, I added front-end support for Alpha, since I didn't
actually have any armv7 images handy to test the backends.
Be warned: Only x86, ppc, aarch64, and tci hav
Cc: Andrzej Zaborowski
Cc: Peter Maydell
Signed-off-by: Richard Henderson
---
tcg/arm/tcg-target.inc.c | 12
1 file changed, 12 insertions(+)
diff --git a/tcg/arm/tcg-target.inc.c b/tcg/arm/tcg-target.inc.c
index f9f54c6..951d110 100644
--- a/tcg/arm/tcg-target.inc.c
+++ b/tcg/arm
On Mon, May 16, 2016 at 02:20:02PM -0600, Alex Williamson wrote:
> On Mon, 16 May 2016 14:52:41 +1000
> Alexey Kardashevskiy wrote:
>
> > On 05/14/2016 08:26 AM, Alex Williamson wrote:
> > > On Wed, 4 May 2016 16:52:30 +1000
> > > Alexey Kardashevskiy wrote:
> > >
> > >> New VFIO_SPAPR_TCE_v2
On Thu, 05/26 17:26, Alex Bennée wrote:
>
> Fam Zheng writes:
>
> > v5: Use docker run's selinux option "z" on passed volume, drop --privileged.
> > Allow overriding "TARGET_LIST" in clang-test.
> > Add Alex's r-b lines in patches 1, 3, 9, 11 and 13.
> >
> > This series adds a new "docke
On Fri, May 20, 2016 at 06:04:42PM +1000, Alexey Kardashevskiy wrote:
> The sPAPR TCE tables manage 2 copies when VFIO is using an IOMMU -
> a guest view of the table and a hardware TCE table. If there is no VFIO
> presense in the address space, then just the guest view is used, if
Nit: s/presense
On Fri, May 13, 2016 at 04:25:59PM -0600, Alex Williamson wrote:
> On Wed, 4 May 2016 16:52:28 +1000
> Alexey Kardashevskiy wrote:
>
> > There are going to be multiple IOMMUs per a container. This moves
> > the single host IOMMU parameter set to a list of VFIOHostDMAWindow.
> >
> > This should
On Thu, 05/26 11:20, Paolo Bonzini wrote:
>
>
> On 26/05/2016 10:30, Fam Zheng wrote:
> >> >
> >> > This doesn't look too wrong... Should the right sequence of events be
> >> > head/after_head or head/after_tail? It's probably simplest to just emit
> >> > all four events.
> > I've no idea. (Th
On Mon, 23 May 2016 11:53:42 -0600
Alex Williamson wrote:
> On Sat, 21 May 2016 19:19:50 +0300
> "Aviv B.D" wrote:
>
> > From: "Aviv Ben-David"
> >
>
> Some commentary about the changes necessary to achieve $SUBJECT would
> be nice here.
>
> > Signed-off-by: Aviv Ben-David
> > ---
> > h
On Tue, May 17, 2016 at 04:57:28PM +0200, Paolo Bonzini wrote:
> That said, there is at least another case where it will be used. In the
> dataplane branch, where AIO callbacks take the AioContext mutex
> themselves, we have:
>
> static void bdrv_co_io_em_complete(void *opaque, int ret)
> {
>
Seem to be a RHEL/Fedora on the same issue:
https://bugzilla.redhat.com/show_bug.cgi?id=1339267
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1581936
Title:
Frozen Windows 7 VMs with VGA CVE-2016-
I experience the same behavior on RHEL 7.2 since I installed the lastest
patch.
** Bug watch added: Red Hat Bugzilla #1339267
https://bugzilla.redhat.com/show_bug.cgi?id=1339267
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
http
On 26 May 2016 at 19:00, Alex Williamson wrote:
> The following changes since commit 2c56d06bafd8933d2a9c6e0aeb5d45f7c1fb5616:
>
> Merge remote-tracking branch 'remotes/kevin/tags/for-upstream' into staging
> (2016-05-26 14:29:30 +0100)
>
> are available in the git repository at:
>
>
> git://
> On May 16, 2016, at 11:33 AM, Stefan Hajnoczi wrote:
>
> The way it's done in the "null" block driver is:
>
> static coroutine_fn int null_co_common(BlockDriverState *bs)
> {
>BDRVNullState *s = bs->opaque;
>
>if (s->latency_ns) {
>co_aio_sleep_ns(bdrv_get_aio_context(bs), QE
From: Alexey Kardashevskiy
7532d3cbf "vfio: Fix 128 bit handling" added support for 64bit IOMMU
memory regions when those are added to VFIO address space; however
removing code cannot cope with these as int128_get64() will fail on
1<<64.
This copies 128bit handling from region_add() to region_de
From: Alexey Kardashevskiy
Since a788f227 "memory: Allow replay of IOMMU mapping notifications"
when new VFIO listener is added, all existing IOMMU mappings are
replayed. However there is a problem that the base address of
an IOMMU memory region (IOMMU MR) is ignored which is not a problem
for th
From: Alexey Kardashevskiy
At the moment IOMMU MR only translate to the system memory.
However if some new code changes this, we will need clear indication why
it is not working so here is the check.
Signed-off-by: Alexey Kardashevskiy
Reviewed-by: David Gibson
Signed-off-by: Alex Williamson
Document the usage modes, host primary graphics considerations, usage,
and fw_cfg ABI required for IGD assignment with vfio.
Signed-off-by: Alex Williamson
Reviewed-by: Gerd Hoffmann
Tested-by: Gerd Hoffmann
---
docs/igd-assign.txt | 133 +++
1
Combine VGA discovery and registration. Quirks can have dependencies
on BARs, so the quirks push out until after we've scanned the BARs.
Signed-off-by: Alex Williamson
Reviewed-by: Gerd Hoffmann
Tested-by: Gerd Hoffmann
---
hw/vfio/pci.c | 49 ++--
The IGD OpRegion is enabled automatically when running in legacy mode,
but it can sometimes be useful in universal passthrough mode as well.
Without an OpRegion, output spigots don't work, and even though Intel
doesn't officially support physical outputs in UPT mode, it's a
useful feature. Note th
Enable quirks to support SandyBridge and newer IGD devices as primary
VM graphics. This requires new vfio-pci device specific regions added
in kernel v4.6 to expose the IGD OpRegion, the shadow ROM, and config
space access to the PCI host bridge and LPC/ISA bridge. VM firmware
support, SeaBIOS on
This function returns success if either we setup the VGA region or
the host vfio doesn't return enough regions to support the VGA index.
This latter case doesn't make any sense. If we're asked to populate
VGA, fail if it doesn't exist and let the caller decide if that's
important.
Signed-off-by:
The sparse mmap capability in a vfio region info allows vfio to tell
us which sub-areas of a region may be mmap'd. Thus rather than
assuming a single mmap covers the entire region and later frobbing it
ourselves for things like the PCI MSI-X vector table, we can read that
directly from vfio.
Sign
Given a device specific region type and sub-type, find it. Also
cleanup return point on error in vfio_get_region_info() so that we
always return 0 with a valid pointer or -errno and NULL.
Signed-off-by: Alex Williamson
Reviewed-by: Gerd Hoffmann
Tested-by: Gerd Hoffmann
---
hw/vfio/common.c
Capability probing modifies wmask, which quirks may be interested in
changing themselves. Apply our BAR quirks after the capability scan
to make this possible.
Signed-off-by: Alex Williamson
Reviewed-by: Gerd Hoffmann
Tested-by: Gerd Hoffmann
---
hw/vfio/pci.c |8 +---
1 file changed,
The following changes since commit 2c56d06bafd8933d2a9c6e0aeb5d45f7c1fb5616:
Merge remote-tracking branch 'remotes/kevin/tags/for-upstream' into staging
(2016-05-26 14:29:30 +0100)
are available in the git repository at:
git://github.com/awilliam/qemu-vfio.git tags/vfio-update-20160526.1
On 26/05/2016 18:43, Jianjun Duan wrote:
>>> The user may only care the position of head and entry. But to
>>> implement QTAILQ_RAW_***, we need more offset information than that.
>>> If we don't query the offsets using something like offset() and store
>>> it in a metadata, we have to make the a
Add a simple helper function to flush the TLB at the indexes specified
by a bitmap. The function will be more useful in the following patches,
when it will be possible to query tlb_flush_by_mmuidx() to VCPUs.
Signed-off-by: Alvise Rigo
---
cputlb.c | 30 +++---
1 file cha
If a VCPU returns EXCP_HALTED from the guest code execution and in the
mean time receives a work item, it will go to sleep without processing
the job.
Before sleeping, check if any work has been added.
Signed-off-by: Alvise Rigo
---
cpus.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
On 05/26/2016 12:11 AM, Paolo Bonzini wrote:
>
>
> On 25/05/2016 22:17, Jianjun Duan wrote:
>>
>>
>> On 05/25/2016 12:22 PM, Paolo Bonzini wrote:
1 QTAILQ should only be accessed using the interfaces defined in
queue.h. Its structs should not be directly used. So I created
interf
Secure tlb_flush_page_all() by waiting the queried flushes to be
actually completed using async_wait_run_on_cpu();
Signed-off-by: Alvise Rigo
---
cputlb.c| 15 ++-
include/exec/exec-all.h | 4 ++--
target-arm/helper.c | 4 ++--
3 files changed, 14 insertions(+),
Some architectures need to flush the TLB by MMU index. As per
tlb_flush(), also these flushes have to be properly queried to the
target VCPU. For the time being, this type of flush is used only in the
ARM/aarch64 target architecture and is the result of guest instructions
emulation. As a result, we
Using tcg_exclusive_{lock,unlock}(), make the emulation of
LoadLink/StoreConditional thread safe.
During an LL access, this lock protects the load access itself, the
update of the exclusive history and the update of the VCPU's protected
range. In a SC access, the lock protects the store access it
Introduce a new function that allows the calling VCPU to add a work item
to another VCPU (aka target VCPU). This new function differs from
async_run_on_cpu() since it makes the calling VCPU waiting for the target
VCPU to finish the work item. The mechanism makes use of the halt_cond
to wait and in
Similarly to the previous commit, make tlb_flush_page_by_mmuidx query the
flushes when targeting different VCPUs.
Signed-off-by: Alvise Rigo
---
cputlb.c| 90 ++---
include/exec/exec-all.h | 5 +--
target-arm/helper.c | 35
1 - 100 of 210 matches
Mail list logo
