Replace a large number of the fprintf(stderr, "*\n" calls with
error_report(). The functions were renamed with these commands and then
compiler issues where manually fixed.
find ./* -type f -exec sed -i \
'N;N;N;N;N;N;N;N;N;N;N;N; {s|fprintf(stderr,
"\(.*\)\\n"\(.*\));|error_report("\1"\2);|I
Replace a large number of the fprintf(stderr, "*\n" calls with
error_report(). The functions were renamed with these commands and then
compiler issues where manually fixed.
find ./* -type f -exec sed -i \
'N;N;N;N;N;N;N;N;N;N;N;N; {s|fprintf(stderr,
"\(.*\)\\n"\(.*\));|error_report("\1"\2);|I
Replace a large number of the fprintf(stderr, "*\n" calls with
error_report(). The functions were renamed with these commands and then
compiler issues where manually fixed.
find ./* -type f -exec sed -i \
'N;N;N;N;N;N;N;N;N;N;N;N; {s|fprintf(stderr,
"\(.*\)\\n"\(.*\));|error_report("\1"\2);|I
Replace a large number of the fprintf(stderr, "*\n" calls with
error_report(). The functions were renamed with these commands and then
compiler issues where manually fixed.
find ./* -type f -exec sed -i \
'N;N;N;N;N;N;N;N;N;N;N;N; {s|fprintf(stderr,
"\(.*\)\\n"\(.*\));|error_report("\1"\2);|I
On Thu, Oct 19, 2017 at 05:04:19PM +0100, Ross Lagerwall wrote:
> Add an option to allow calling unshare() just before starting guest
> execution. The option allows unsharing one or more of the mount
> namespace, the network namespace, and the IPC namespace. This is useful
> to restrict the ability
Some legacy firmwares access unimplemented addresses on the Aspeed SoC
(old U-Boot code using variables in the bss when it shouldn't do).
Let's use a ROM memory region to catch the invalid writes and support
new boards without using the 'ignore_memory_transaction_failures'
flag.
This change is bre
Now that we have a ROM memory region catching the invalid writes, we
can remove this flag and work on fixing the current firwmares still
accessing unimplemented addresses.
Signed-off-by: Cédric Le Goater
Reviewed-by: Peter Maydell
---
hw/arm/aspeed.c | 3 ---
1 file changed, 3 deletions(-)
dif
The Witherspoon boards are OpenPOWER system hosting POWER9 Processors.
Let's add support for their BMC including a couple of I2C devices as
found on real HW.
Signed-off-by: Cédric Le Goater
Reviewed-by: Andrew Jeffery
---
Changes since v2:
- removed 'ignore_memory_transaction_failures' flag
This is an helper routine to add a single EEPROM on an I2C bus. It can
be directly used by smbus_eeprom_init() which adds a certain number of
EEPROMs on mips and x86 machines.
Signed-off-by: Cédric Le Goater
Reviewed-by: Philippe Mathieu-Daudé
---
hw/i2c/smbus_eeprom.c | 16 +++-
i
> From: Peter Maydell [mailto:peter.mayd...@linaro.org]
> Sent: Thursday, 19 October 2017 09:51
>
> On 18 October 2017 at 01:16, Andrew Baumann
> wrote:
> Hi; thanks for this patch. Looks like you forgot to add your
> signed-off-by line.
Thanks for the review!
I didn’t sign the patch, because
The Aspeed boards have at least one EEPROM to hold the Vital Product
Data (VPD).
Signed-off-by: Cédric Le Goater
Reviewed-by: Andrew Jeffery
---
hw/arm/aspeed.c | 13 +
1 file changed, 13 insertions(+)
diff --git a/hw/arm/aspeed.c b/hw/arm/aspeed.c
index 124a0e5240b5..df7f266800ba
The AST2500 EVB does not have an RTC but we can pretend that one is
plugged on the I2C bus header.
The romulus and witherspoon boards expects an Epson RX8900 I2C RTC but
a ds1338 is good enough for the basic features we need.
Signed-off-by: Cédric Le Goater
Reviewed-by: Andrew Jeffery
---
hw/a
On 10/19/2017 10:44 AM, Stefan Berger wrote:
On 10/19/2017 10:33 AM, Marc-André Lureau wrote:
Hi Stefan
- Original Message -
On 10/09/2017 06:55 PM, Marc-André Lureau wrote:
Hi,
I accumulated a series of patch doing some TPM code cleanup while
doing review. I removed some dead code,
This makes it much easier to find a particular thing in config.log.
The information may be lacking in other shells, resulting in harmless
empty output. (This is why we don't use the proper ${FUNCNAME[*]}
array syntax - other shells will choke on that.)
The extra output is only printed if configu
The pca9552 LED blinkers on the Witherspoon machine are used for leds
but also as GPIOs to control fans and GPUs.
Signed-off-by: Cédric Le Goater
Reviewed-by: Andrew Jeffery
---
hw/arm/aspeed.c | 4
1 file changed, 4 insertions(+)
diff --git a/hw/arm/aspeed.c b/hw/arm/aspeed.c
index df7f2
On 09/15/2017 09:27 AM, Cornelia Huck wrote:
>>>
+DEFINE_PROP_UINT8("chpid_type", CcwTesterDevice, chpid_type,
+ 0x98),
> This might also need re-evaluation - we should not really need a new
> chpid type.
>
I'm back at ccw-tester again (for v2). I've reali
I have been working on trying to get qemu, when running as a Xen
device model, to _actually_ not have power equivalent to root.
I think I have achieved this, with some limitations (which are
discussed in my series against xen.git.
However, there are changes to qemu needed. In particular
* The
On Thu, Oct 19, 2017 at 05:38:10PM +0100, Ian Jackson wrote:
> Anthony PERARD writes ("Re: [PATCH 1/8] xen: link against xentoolcore"):
> > I don't think it is necessary to do anything in qemu. The linker should
> > find on its own the new libxentoolcore as long as an option
> > -Wl,-rpath-link= pr
On 19 October 2017 at 13:50, wrote:
> From: Gabriel Augusto Costa
>
> I add a new arm machine with some peripherals. The machine is mk64fn1m0, a
> cortex-m4 microcontroller from NXP Kinetis family. The machine can run a
> simple arm binary file using UART0 in polling mode.
> I prepared two patch
On 18 October 2017 at 01:16, Andrew Baumann
wrote:
> On a successful address translation instruction, PAR is supposed to
> contain cacheability and shareability attributes determined by the
> translation. Previous versions of QEMU returned 0 for these bits (in
> line with the general strategy of i
We are going to want to reuse this.
No functional change.
Signed-off-by: Ian Jackson
Reviewed-by: Anthony PERARD
---
hw/i386/xen/xen-hvm.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/hw/i386/xen/xen-hvm.c b/hw/i386/xen/xen-hvm.c
index 7b60ec6..83420cd 100644
--- a/
xc_interface_open etc. is not going to work if we have dropped
privilege, but xendevicemodel_shutdown will if everything is new
enough.
xendevicemodel_shutdown is only availabe in Xen 4.10 and later, so
provide a stub for earlier versions.
Signed-off-by: Ian Jackson
---
v2: Add compatibility stu
We need to restrict *all* the control fds that qemu opens. Looking in
/proc/PID/fd shows there are many; their allocation seems scattered
throughout Xen support code in qemu.
We must postpone the restrict call until roughly the same time as qemu
changes its uid, chroots (if applicable), and so on
On 19 October 2017 at 18:04, Andrew Baumann
wrote:
>> From: Peter Maydell [mailto:peter.mayd...@linaro.org]
>> Sent: Thursday, 19 October 2017 09:51
>>
>> On 18 October 2017 at 01:16, Andrew Baumann
>> wrote:
>> Hi; thanks for this patch. Looks like you forgot to add your
>> signed-off-by line.
>
From: Anthony PERARD
Xen libraries 4.10 will include a new xentoolcore library, without
which xendevicemodel et al will not work.
Signed-off-by: Ian Jackson
---
configure | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/configure b/configure
index fd7e3a5..6f691df 10
From: Gabriel Augusto Costa
I add a new arm machine with some peripherals. The machine is mk64fn1m0, a
cortex-m4 microcontroller from NXP Kinetis family. The machine can run a
simple arm binary file using UART0 in polling mode.
I prepared two patchs to include this machine:
PATCH v1: Include th
This allows the caller to specify a uid and gid to use, even if there
is no corresponding password entry. This will be useful in certain
Xen configurations.
We don't support just -runas because: (i) deprivileging without
calling setgroups would be ineffective (ii) given only a uid we don't
know
From: Gabriel Augusto Costa
I add a new arm machine with some peripherals. The machine is mk64fn1m0, a
cortex-m4 microcontroller from NXP Kinetis family. The machine can run a
simple arm binary file using UART0 in polling mode.
I prepared two patchs to include this machine:
PATCH v1: Include th
On 19.10.2017 18:17, Alistair Francis wrote:
> Replace a large number of the fprintf(stderr, "*\n" calls with
> error_report(). The functions were renamed with these commands and then
> compiler issues where manually fixed.
[...]
> Some lines where then manually tweaked to pass checkpatch.
>
> One
Hi
- Original Message -
> On 10/19/2017 10:44 AM, Stefan Berger wrote:
> > On 10/19/2017 10:33 AM, Marc-André Lureau wrote:
> >> Hi Stefan
> >>
> >> - Original Message -
> >>> On 10/09/2017 06:55 PM, Marc-André Lureau wrote:
> Hi,
>
> I accumulated a series of patch
On 19 October 2017 at 10:15, Daniel P. Berrange wrote:
> On Wed, Oct 18, 2017 at 02:40:23PM -0400, Aaron Lindsay wrote:
>> This was introduced by:
>> commit aef45d51d1204f3335fb99de6658e0c5612c2b67
>> Author: Daniel P. Berrange
>> Date: Fri Sep 29 11:11:56 2017 +0100
>>
>> bui
ble in the git repository at:
>
> git://git.kraxel.org/qemu tags/seabios-1.11-prerelease-20171019-pull-request
>
> for you to fetch changes up to 083fab0290f2c40d3d04f7f22eed9c8f2d5b6787:
>
> seabios: update
On Thu, Oct 19, 2017 at 05:43:18PM +0200, Paolo Bonzini wrote:
> On 19/10/2017 17:39, Michael S. Tsirkin wrote:
> >> Add VHOST_USER_GET_CONFIG/VHOST_USER_SET_CONFIG messages which can be
> >> used for live migration of vhost user devices, also vhost user devices
> >> can benefit from the messages t
On 19.10.2017 18:18, Alistair Francis wrote:
> Replace a large number of the fprintf(stderr, "*\n" calls with
> error_report(). The functions were renamed with these commands and then
> compiler issues where manually fixed.
[...]
> diff --git a/util/aio-posix.c b/util/aio-posix.c
> index 5946ac09f0
From: Marc-André Lureau
This argument is always false, simplify the code.
Signed-off-by: Marc-André Lureau
Reviewed-by: Stefan Berger
Signed-off-by: Stefan Berger
---
hw/tpm/tpm_tis.c | 13 +++--
1 file changed, 3 insertions(+), 10 deletions(-)
diff --git a/hw/tpm/tpm_tis.c b/hw/tpm
From: Marc-André Lureau
No backend use it.
Signed-off-by: Marc-André Lureau
Reviewed-by: Stefan Berger
Signed-off-by: Stefan Berger
---
backends/tpm.c | 4 +---
include/sysemu/tpm_backend.h | 2 --
2 files changed, 1 insertion(+), 5 deletions(-)
diff --git a/backends/tpm.c b/b
From: Marc-André Lureau
Keep it internal to tpm-tis instead.
Signed-off-by: Marc-André Lureau
Reviewed-by: Stefan Berger
Signed-off-by: Stefan Berger
---
hw/tpm/tpm_emulator.c | 3 +--
hw/tpm/tpm_int.h | 2 --
hw/tpm/tpm_tis.c | 12 +---
3 files changed, 6 insertions(+),
The following changes since commit f2a48d696c12aaac12993364371daae9f6233c37:
Merge remote-tracking branch 'remotes/riku/tags/pull-linux-user-20171018'
into staging (2017-10-19 14:39:30 +0100)
are available in the git repository at:
git://github.com/stefanberger/qemu-tpm.git tags/pull-tpm-20
From: Marc-André Lureau
One step towards removing TPMDriverOps and driver registration.
Signed-off-by: Marc-André Lureau
Reviewed-by: Stefan Berger
Signed-off-by: Stefan Berger
---
tpm.c | 29 +
1 file changed, 21 insertions(+), 8 deletions(-)
diff --git a/tpm.c
From: Marc-André Lureau
Simplify the TPM backend setup, move callback to TPM interface.
Signed-off-by: Marc-André Lureau
Reviewed-by: Stefan Berger
Signed-off-by: Stefan Berger
---
backends/tpm.c | 4 +---
hw/tpm/tpm_emulator.c| 3 ++-
hw/tpm/tpm_int.h |
From: Marc-André Lureau
Signed-off-by: Marc-André Lureau
Reviewed-by: Stefan Berger
Signed-off-by: Stefan Berger
---
hw/tpm/tpm_int.h | 23 ++-
hw/tpm/tpm_tis.h | 19 +++
hw/tpm/tpm_util.c | 1 +
3 files changed, 22 insertions(+), 21 deletions(-)
diff -
From: Marc-André Lureau
This will simplify backend / interface objects relationship, so the
frontend interface will simply have to implement the TPM QOM interface.
Signed-off-by: Marc-André Lureau
Reviewed-by: Stefan Berger
Signed-off-by: Stefan Berger
---
backends/tpm.c | 8
hw/
From: Marc-André Lureau
Close to where it's being used.
Signed-off-by: Marc-André Lureau
Reviewed-by: Stefan Berger
Signed-off-by: Stefan Berger
---
hw/tpm/tpm_tis.h | 5 +
include/sysemu/tpm_backend.h | 5 -
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/
From: Marc-André Lureau
There is only handling of request so far in both backends.
Signed-off-by: Marc-André Lureau
Reviewed-by: Stefan Berger
Signed-off-by: Stefan Berger
---
backends/tpm.c | 7 ++-
hw/tpm/tpm_emulator.c| 42 ---
From: Marc-André Lureau
The tpm_state is passed as argument, the assert() is pointless since
we give it the value of tpm_state->locty_number already.
Signed-off-by: Marc-André Lureau
Reviewed-by: Stefan Berger
Signed-off-by: Stefan Berger
---
hw/tpm/tpm_emulator.c| 3 +--
hw/tpm/tpm_
From: Marc-André Lureau
No need to export the function.
Signed-off-by: Marc-André Lureau
Reviewed-by: Stefan Berger
Signed-off-by: Stefan Berger
---
include/sysemu/tpm_backend.h | 1 -
tpm.c| 2 +-
2 files changed, 1 insertion(+), 2 deletions(-)
diff --git a/include/
From: Marc-André Lureau
Signed-off-by: Marc-André Lureau
Reviewed-by: Stefan Berger
Signed-off-by: Stefan Berger
---
tpm.c | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/tpm.c b/tpm.c
index 4882501..45520f5 100644
--- a/tpm.c
+++ b/tpm.c
@@ -86,7 +86,7 @@ TPMBackend
From: Marc-André Lureau
Use tpm_driver_find_by_type() instead.
Signed-off-by: Marc-André Lureau
Reviewed-by: Stefan Berger
Signed-off-by: Stefan Berger
---
tpm.c | 11 +++
1 file changed, 3 insertions(+), 8 deletions(-)
diff --git a/tpm.c b/tpm.c
index 04de8da..ad2ed6a 100644
--- a/
From: Marc-André Lureau
The previous patch cleaned up a bit error handling, and exposed an
existing bug: error_report_err() could be called with a NULL error.
Instead, make tpm_emulator_set_locality() set the error.
Signed-off-by: Marc-André Lureau
Reviewed-by: Stefan Berger
Signed-off-by: Ste
From: Marc-André Lureau
The definitions are now private to TIS implementation.
Signed-off-by: Marc-André Lureau
Reviewed-by: Stefan Berger
Signed-off-by: Stefan Berger
---
hw/tpm/tpm_passthrough.c | 1 -
hw/tpm/tpm_tis.c | 72 -
hw/tpm/tpm_tis.h
From: Marc-André Lureau
No more users of be_drivers[], drop that too.
Signed-off-by: Marc-André Lureau
Reviewed-by: Stefan Berger
Signed-off-by: Stefan Berger
---
hw/tpm/tpm_emulator.c| 1 -
hw/tpm/tpm_passthrough.c | 1 -
include/sysemu/tpm_backend.h | 1 -
tpm.c
From: Marc-André Lureau
Use TPMBackendClass to hold class methods/fields.
Signed-off-by: Marc-André Lureau
Reviewed-by: Stefan Berger
Signed-off-by: Stefan Berger
---
backends/tpm.c | 31 ---
hw/tpm/tpm_emulator.c| 29
From: Marc-André Lureau
Signed-off-by: Marc-André Lureau
Reviewed-by: Stefan Berger
Signed-off-by: Stefan Berger
---
hw/tpm/tpm_passthrough.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/tpm/tpm_passthrough.c b/hw/tpm/tpm_passthrough.c
index 5cd988e..fed3d69 100644
-
From: Marc-André Lureau
Signed-off-by: Marc-André Lureau
Reviewed-by: Stefan Berger
Signed-off-by: Stefan Berger
---
hw/tpm/tpm_tis.c | 336 ++-
1 file changed, 157 insertions(+), 179 deletions(-)
diff --git a/hw/tpm/tpm_tis.c b/hw/tpm/tpm_
From: Marc-André Lureau
This simplifies a bit locality handling, and argument passing, and
could pave the way to queuing requests (if that makes sense).
Signed-off-by: Marc-André Lureau
Reviewed-by: Stefan Berger
Signed-off-by: Stefan Berger
---
backends/tpm.c | 6 +++---
hw/t
From: Marc-André Lureau
This look like temporary hacking code. It shouldn't be necessary in
release code, or there should be a runtime option for it.
Signed-off-by: Marc-André Lureau
Reviewed-by: Stefan Berger
Signed-off-by: Stefan Berger
---
hw/tpm/tpm_tis.c | 24 +---
1
On 19 October 2017 at 07:58, Mark Cave-Ayland
wrote:
> Hi Peter,
>
> Here are my sun4u patches for 2.11. Note that because of the machine changes
> there is an openbios-sparc64 binary update included in the first patch to
> maintain bisectability. There will be a follow-up patch for OpenBIOS which
On 19 October 2017 at 08:10, Mark Cave-Ayland
wrote:
> Hi Peter,
>
> Here are the latest set of updates for OpenBIOS which should be pulled after
> the sun4u
> patches have been applied due to breaking changes in the sun4u machine.
>
>
> ATB,
>
> Mark.
>
>
> The following changes since commit 861
On Thu, Oct 19, 2017 at 1:01 AM, Christoph Hellwig wrote:
> On Wed, Oct 18, 2017 at 08:51:37AM -0700, Dan Williams wrote:
>> This use case is not "Persistent Memory". Persistent Memory is
>> something you can map and make persistent with CPU instructions.
>> Anything that requires a driver call is
On 10/17/2017 07:57 AM, Vladimir Sementsov-Ogievskiy wrote:
> Minimal implementation: for structured error only error_report error
> message.
>
> Signed-off-by: Vladimir Sementsov-Ogievskiy
> ---
and I replied:
>
> But in the client, I then perform 'w 0 0' (a zero-byte write, which
> should fai
* Jiri Denemark (jdene...@redhat.com) wrote:
> The libvirt changes which will make use of this new migration capability
> can be found in migration-pause branch of my gitlab repository:
>
> git fetch https://gitlab.com/jirkade/libvirt.git migration-pause
>
> It's not properly split into patch
On 19/10/17 19:18, Peter Maydell wrote:
> On 19 October 2017 at 08:10, Mark Cave-Ayland
> wrote:
>> Hi Peter,
>>
>> Here are the latest set of updates for OpenBIOS which should be pulled after
>> the sun4u
>> patches have been applied due to breaking changes in the sun4u machine.
>>
>>
>> ATB,
>
Hi,
This series failed automatic build test. Please find the testing commands and
their output below. If you have docker installed, you can probably reproduce it
locally.
Type: series
Message-id: 20171016222315.407-1-mdr...@linux.vnet.ibm.com
Subject: [Qemu-devel] [PATCH v3 0/3] qdev/vfio: defer
will test these and report back shortly.
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1719196
Title:
[arm64 ocata] newly created instances are unable to raise network
interfaces
Status in Ubunt
I've tested with the packages from the ppa:
https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/2995
qemu:
Installed: 1:2.8+dfsg-3ubuntu2.7~ppa5cloud
qemu-system-arm:
Installed: 1:2.8+dfsg-3ubuntu2.7~ppa5cloud
qemu-system-aarch64:
Installed: 1:2.8+dfsg-3ubuntu2.7~ppa5cloud
Reboot
We don't need qemu-keymap when we build only linux-user qemu.
When we compile in static mode, the libxkbcommon is detected
by configure if the shared one is available, but cannot
be linked if the static version is not available.
As we don't need it for qemu-linux-user, and we generally need
a sta
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: 1508417443-31849-1-git-send-email-gabriel291...@gmail.com
Subject: [Qemu-devel] [PATCH v1] arm: Adding new arm machine, Kinetis K64
MK64FN1M0
=== TEST SCRIPT BEGIN ===
#!/bi
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: 1508417485-31911-1-git-send-email-gabriel291...@gmail.com
Subject: [Qemu-devel] [PATCH v2] arm: Adding makefile for Kinetis K64 MK64FN1M0
=== TEST SCRIPT BEGIN ===
#!/bin/bas
On 10/14/2017 08:01 PM, Eric Blake wrote:
> From: Vladimir Sementsov-Ogievskiy
>
> Split out nbd_request_simple_option to be reused for structured reply
> option.
>
> Signed-off-by: Vladimir Sementsov-Ogievskiy
> Signed-off-by: Eric Blake
>
> +static int nbd_request_simple_option(QIOChannel
Hi,
This series failed automatic build test. Please find the testing commands and
their output below. If you have docker installed, you can probably reproduce it
locally.
Type: series
Message-id: 1508417485-31911-1-git-send-email-gabriel291...@gmail.com
Subject: [Qemu-devel] [PATCH v2] arm: Addin
Am 19.10.2017 um 19:53 schrieb Thomas Huth:
> On 19.10.2017 18:18, Alistair Francis wrote:
>> Replace a large number of the fprintf(stderr, "*\n" calls with
>> error_report(). The functions were renamed with these commands and then
>> compiler issues where manually fixed.
> [...]
>> diff --git a/ut
On 10/17/2017 07:57 AM, Vladimir Sementsov-Ogievskiy wrote:
> Minimal implementation: for structured error only error_report error
> message.
>
> Signed-off-by: Vladimir Sementsov-Ogievskiy
> ---
>
> +static int nbd_co_request(BlockDriverState *bs, NBDRequest *request,
> +
On Thu, Oct 19, 2017 at 04:28:59PM +0100, Daniel P. Berrange wrote:
> On Thu, Oct 19, 2017 at 11:21:22AM -0400, Igor Mammedov wrote:
> > - Original Message -
> > > From: "Daniel P. Berrange"
> > > To: "Igor Mammedov"
> > > Cc: "peter maydell" , pkre...@redhat.com,
> > > ehabk...@redhat.c
I get the following error when building on an NFSv3 filesystem:
% make -j8
GEN aarch64-softmmu/config-devices.mak.tmp
GEN config-host.h
[snip]
GEN qmp-marshal.c
GEN aarch64-softmmu/config-devices.mak
cp: preserving permissions for ‘aarch64-softmmu/config-devices.mak.old’:
On Thu, Oct 19, 2017 at 15:05:17 +0200, Paolo Bonzini wrote:
> On 19/10/2017 00:45, Emilio G. Cota wrote:
> > I have just pushed a branch on top of this series that includes
> > 10 patches that further pave the way for the removal of tb_lock:
> >
> > https://github.com/cota/qemu/tree/multi-tcg-v
Two or more threads might race while invalidating the same TB. We currently
do not check for this at all despite taking tb_lock, which means we would
wrongly invalidate the same TB more than once. This bug has actually been
hit by users: I recently saw a report on IRC, although I have yet to see
th
Hi,
This series failed automatic build test. Please find the testing commands and
their output below. If you have docker installed, you can probably reproduce it
locally.
Type: series
Message-id: 20171019155146.30434-1-richard.hender...@linaro.org
Subject: [Qemu-devel] [PATCH v6 0/9] Support the
On 10/17/2017 04:17 PM, Eric Blake wrote:
> On 10/17/2017 07:57 AM, Vladimir Sementsov-Ogievskiy wrote:
>> Minimal implementation: for structured error only error_report error
>> message.
>>
>> Signed-off-by: Vladimir Sementsov-Ogievskiy
>> ---
> But in the client, I then perform 'w 0 0' (a zero-
On Thu, Oct 19, 2017 at 14:14:12 +0900, Matt wrote:
(snip)
> - VMSimint does nearly that, it runs Qemu in discrete time but
> interface it with a JAVA simulator
> http://www.ikr.uni-stuttgart.de/Content/Publications/Archive/We_SIMUTools_2014_40209.pdf
> (with the code http://www.ikr.uni-stuttgart.d
On Thu, 19 Oct 2017, Peter Maydell wrote:
> On 18 October 2017 at 23:03, Stefano Stabellini
> wrote:
> > WFI/E are 4 bytes long: set ARM_EL_IL_SHIFT in the syndrome.
> >
> > Signed-off-by: Stefano Stabellini
> >
> > diff --git a/target/arm/internals.h b/target/arm/internals.h
> > index 1f6efef..
unknown keycodes `(unnamed)', please report to qemu-devel@nongnu.org
uname -a returns:
Linux spaceboyross-Latitude-D630 4.13.0-16-generic #19-Ubuntu SMP Wed
Oct 11 18:35:14 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
On 19/10/2017 16:52, Jeff Cody wrote:
> On Thu, Oct 19, 2017 at 12:23:39PM +0200, Paolo Bonzini wrote:
>> On 18/10/2017 19:27, Jeff Cody wrote:
>>> On final exit, yes, a test needs not remember to remove all of its mouse
>>> droppings. But as far as not needing to remove images in intermediate
>>>
On 19/10/2017 19:43, Michael S. Tsirkin wrote:
> On Thu, Oct 19, 2017 at 05:43:18PM +0200, Paolo Bonzini wrote:
>> On 19/10/2017 17:39, Michael S. Tsirkin wrote:
Add VHOST_USER_GET_CONFIG/VHOST_USER_SET_CONFIG messages which can be
used for live migration of vhost user devices, also vhost
On Thu, 19 Oct 2017, Peter Maydell wrote:
> On 18 October 2017 at 23:41, Stefano Stabellini
> wrote:
> > extended_addresses_enabled calls arm_el_is_aa64, hardcoding exception
> > level 1. Instead, retrieve the current el calling arm_current_el.
> >
> > Signed-off-by: Stefano Stabellini
> >
> > d
This matches translator behaviour in arm_lduw_code.
Fixes: https://bugs.launchpad.net/qemu/+bug/1724485
Signed-off-by: Richard Henderson
---
target/arm/cpu.c | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/target/arm/cpu.c b/target/arm/cpu.c
index bc9d70df04..a0ed11c
Reported in
https://bugs.launchpad.net/qemu/+bug/1724485
There's one existing bug here, wrt setting INSN_ARM_BE32, and
another when it comes to the capstone disassembler patch set.
r~
Richard Henderson (2):
target/arm: Move BE32 disassembler fixup
target/arm: Don't set INSN_ARM_BE32 for
The Capstone disassembler has its own big-endian fixup.
Doing this twice does not work, of course. Move our current
fixup from target/arm/cpu.c to disas/arm.c.
This makes read_memory_inner_func unused and can be removed.
Signed-off-by: Richard Henderson
---
include/disas/bfd.h | 7 ---
di
On 10/14/2017 08:01 PM, Eric Blake wrote:
> The NBD spec permits including a human-readable error string if
> structured replies are in force, so we might as well send the
> client the message that we logged on any error.
>
> Signed-off-by: Eric Blake
> ---
> nbd/server.c | 22 ++
On 10/09/2017 06:56 PM, Marc-André Lureau wrote:
Lift from the backend implementation the responsability to call the
request_completed() callback outside of thread context. This also
I don't think this is what you are doing here. It's still in thread context.
Something is breaking the TIS inte
On 10/16/2017 08:16 AM, Gerd Hoffmann wrote:
> qemu-keymap generates qemu reverse keymaps from xkb keymaps,
> which can be used with the qemu "-k" command line switch.
>
> Signed-off-by: Gerd Hoffmann
> Message-id: 20171005153330.19210-2-kra...@redhat.com
> ---
> configure | 23 ++
> Ma
On 10/19/2017 05:09 PM, Eric Blake wrote:
> On 10/16/2017 08:16 AM, Gerd Hoffmann wrote:
>> qemu-keymap generates qemu reverse keymaps from xkb keymaps,
>> which can be used with the qemu "-k" command line switch.
>>
>> Signed-off-by: Gerd Hoffmann
>> Message-id: 20171005153330.19210-2-kra...@redh
This is needed in preparation for structured reply handling,
as we will be performing the translation from NBD error to
system errno value higher in the stack at block/nbd-client.c.
Signed-off-by: Eric Blake
Reviewed-by: Vladimir Sementsov-Ogievskiy
---
include/block/nbd.h | 13 +
n
I've incorporated a few more tweaks since v4:
https://lists.gnu.org/archive/html/qemu-devel/2017-10/msg03305.html
I tried to call out the biggest changes in the various commits, but
it includes bug fixes from my testing, and fixing places spotted
by Vladimir. I still haven't validated the behavio
When the server is read-only, we were already reporting an error
message for NBD_CMD_WRITE_ZEROES, but failed to set errp for a
similar NBD_CMD_WRITE. This will matter more once structured
replies allow the server to propagate the errp information back
to the client. While at it, use an error mes
Upcoming patches will implement the NBD structured reply
extension [1] for both client and server roles. Declare the
constants, structs, and lookup routines that will be valuable
whether the server or client code is backported in isolation.
This includes moving one constant from an internal heade
The NBD spec permits including a human-readable error string if
structured replies are in force, so we might as well send the
client the message that we logged on any error.
Signed-off-by: Eric Blake
---
v5: allow NULL msg, cast away const for ease of use [Vladimir]; send
correct length over the
An upcoming change to block/nbd-client.c will want to read the
tail of a structured reply chunk directly from the wire. Move
this function to make it easier.
Based on a patch from Vladimir Sementsov-Ogievskiy.
Signed-off-by: Eric Blake
Reviewed-by: Vladimir Sementsov-Ogievskiy
---
include/blo
NBD errors were originally sent over the wire based on Linux errno
values; but not all the world is Linux, and not all platforms share
the same values. Since a number isn't very easy to decipher on all
platforms, update the trace messages to include the name of NBD
errors being sent/received over
Consolidate the check for a zero-length payload to an option
into a new function, nbd_check_zero_length(); this check will
also be used when introducing support for structured replies.
By sticking a catch-all check at the end of the loop for
processing options, we can simplify several of the inter
From: Vladimir Sementsov-Ogievskiy
Split out nbd_request_simple_option to be reused for structured reply
option.
Signed-off-by: Vladimir Sementsov-Ogievskiy
Signed-off-by: Eric Blake
---
v5: only check length for ACK responses
v4: reduce redundant traces, typo fix in commit message
---
nbd/c
201 - 300 of 353 matches
Mail list logo
