On 06/04/2018 02:03 PM, Daniel P. Berrangé wrote:
> When using --daemonize, the initial lead process will fork a child and
> then wait to be notified that setup is complete via a pipe, before it
> exits. When using --preconfig there is an extra call to main_loop()
> before the notification is done
On 4 June 2018 at 08:10, Fam Zheng wrote:
> The following changes since commit 392fba9f583223786f844dce9b2e7f9a0ce0147a:
>
> Merge remote-tracking branch
> 'remotes/stsquad/tags/pull-travis-updates-010618-1' into staging (2018-06-01
> 17:32:30 +0100)
>
> are available in the Git repository at:
This tests for a working docker installation without sudo and sets up
config-host.mak accordingly. This will be useful from cross compiling
things in the future.
Signed-off-by: Alex Bennée
Reviewed-by: Richard Henderson
---
configure | 17 +
1 file changed, 17 insertions(+)
dif
This is a helper function for the configure script. It replies yes,
sudo or no to inform the user if non-interactive docker support is
available. We trap the Exception to fail gracefully.
Signed-off-by: Alex Bennée
Reviewed-by: Fam Zheng
---
tests/docker/docker.py | 18 ++
1 fil
From: Fam Zheng
Signed-off-by: Fam Zheng
Signed-off-by: Alex Bennée
---
tests/docker/dockerfiles/fedora.docker | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/docker/dockerfiles/fedora.docker
b/tests/docker/dockerfiles/fedora.docker
index b706f42405..65d7761cf5 10064
From: Fam Zheng
To be more accurate on its purpose and make code that looks for a certain
target out of this variable more readable.
Signed-off-by: Fam Zheng
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
Reviewed-by: Richard Henderson
Signed-off-by: Alex Bennée
---
From: Paolo Bonzini
Do not test the deprecated API versions. debian-win32-cross and
debian-win64-cross
are already using SDL2 (they do not cover GTK+ at all).
Signed-off-by: Paolo Bonzini
Reviewed-by: Daniel P. Berrangé
[AJB: fix merge conflicts]
Signed-off-by: Alex Bennée
---
tests/docker
Hi,
I was prompted on IRC so I've gathered together some of the
outstanding docker related patches from the list. It also includes a
few of the docker related tweaks from my mega TCG testing series whcih
should hopefully bring the delta down. I plan to send the pull request
in tomorrow if nothing
From: Fam Zheng
It has some basic *-devel.i686 packages to be used with "gcc -m32" as a
32 bit cross build environment.
Signed-off-by: Fam Zheng
[AJB: add glibc-static]
Signed-off-by: Alex Bennée
Tested-by: Philippe Mathieu-Daudé
---
tests/docker/dockerfiles/fedora-i386-cross.docker | 14 +++
From: Philippe Mathieu-Daudé
TriCore binutils is built from Bastian Koppelmann repository.
Note: There is no TriCore compiler in this image (only assembler/linker).
Signed-off-by: Philippe Mathieu-Daudé
[AJB: base of Debian9, add to Makefile.include]
Signed-off-by: Alex Bennée
---
tests/dock
On Mon, 4 Jun 2018 16:21:48 +0200
Michal Privoznik wrote:
> On 06/04/2018 02:03 PM, Daniel P. Berrangé wrote:
> > When using --daemonize, the initial lead process will fork a child and
> > then wait to be notified that setup is complete via a pipe, before it
> > exits. When using --preconfig the
Hi
On Mon, Jun 4, 2018 at 10:06 AM, Peter Xu wrote:
> Previously we cleanup the queues when we got CLOSED event. It was used
> to make sure we won't leftover replies/events of a old client to a new
> client. Now this patch postpones that until OPENED.
>
> In most cases, it does not really matte
Peter Maydell writes:
> On 23 May 2018 at 10:08, Alex Bennée wrote:
>>
>> Peter Maydell writes:
>>
>>> Add support for multiple IOMMU indexes to the IOMMU notifier APIs.
>>> When initializing a notifier with iommu_notifier_init(), the caller
>>> must pass the IOMMU index that it is interested
Since v1 [1]:
- zero-initialize c->argv[] as suggested by Gerd Hoffmann
- drop 4th patch, bug already fixed in 53735bef108
[1] http://lists.nongnu.org/archive/html/qemu-devel/2017-07/msg07453.html
Philippe Mathieu-Daudé (3):
usb: correctly handle Zero Length Packets
usb/dev-mtp: Fix use of un
USB Specification Revision 2.0, §5.5.3:
The Data stage of a control transfer from an endpoint to the host is complete
when the endpoint does one of the following:
• Has transferred exactly the amount of data specified during the Setup stage
• Transfers a packet with a payload size less than
This fixes:
hw/usb/dev-mtp.c:971:5: warning: 4th function call argument is an
uninitialized value
trace_usb_mtp_op_get_partial_object(s->dev.addr, o->handle, o->path,
c->argv[1], c->argv[2]);
On Mon, Jun 04, 2018 at 05:01:11PM +0200, Igor Mammedov wrote:
> On Mon, 4 Jun 2018 16:21:48 +0200
> Michal Privoznik wrote:
>
> > On 06/04/2018 02:03 PM, Daniel P. Berrangé wrote:
> > > When using --daemonize, the initial lead process will fork a child and
> > > then wait to be notified that set
This fixes:
hw/usb/dev-mtp.c:1212:13: warning: 2nd function call argument is an
uninitialized value
o = usb_mtp_object_lookup(s, c->argv[0]);
^~~~
Reported-by: Clang Static Analyzer
Suggested-by: Gerd Hoffmann
Signed-off-by: Philippe Mathieu-Dau
Le 04/06/2018 à 16:20, Philippe Mathieu-Daudé a écrit :
> On 06/03/2018 08:33 PM, Laurent Vivier wrote:
>> Le 29/05/2018 à 17:19, Laurent Vivier a écrit :
>>> Le 29/05/2018 à 16:25, Philippe Mathieu-Daudé a écrit :
Hi Laurent,
>>>
>>> Hi Philippe,
>>>
On 07/24/2017 04:16 PM, Laurent Vivie
The following changes since commit 392fba9f583223786f844dce9b2e7f9a0ce0147a:
Merge remote-tracking branch
'remotes/stsquad/tags/pull-travis-updates-010618-1' into staging (2018-06-01
17:32:30 +0100)
are available in the Git repository at:
git://github.com/vivier/qemu.git tags/linux-user-fo
No code change.
Signed-off-by: Laurent Vivier
Acked-by: Richard Henderson
Message-Id: <20180529194207.31503-5-laur...@vivier.eu>
---
linux-user/aarch64/target_fcntl.h | 5 +
linux-user/arm/target_fcntl.h | 6 ++
linux-user/m68k/target_fcntl.h| 6 ++
linux-user/syscall_defs.h
Remove sparc64/target_signal.h, use sparc/target_signal.h instead.
Signed-off-by: Laurent Vivier
Message-Id: <20180529194207.31503-11-laur...@vivier.eu>
---
linux-user/sparc/target_signal.h | 48 +++
linux-user/sparc64/signal.c| 1 -
linux-user/sparc64/
No code change.
Signed-off-by: Laurent Vivier
Message-Id: <20180529194207.31503-6-laur...@vivier.eu>
---
linux-user/mips/target_fcntl.h | 27 +++
linux-user/syscall_defs.h | 31 ++-
2 files changed, 29 insertions(+), 29 deletions(-)
diff
From: Philippe Mathieu-Daudé
linux-user/syscall.c:9860:17: warning: Call to function 'strcpy' is insecure as
it does not provide bounding of the memory buffer. Replace unbounded copy
functions with analogous functions that support length arguments such as
'strlcpy'. CWE-119
str
No code change.
Signed-off-by: Laurent Vivier
Message-Id: <20180529194207.31503-8-laur...@vivier.eu>
---
linux-user/sparc/target_fcntl.h | 34 ++
linux-user/syscall_defs.h | 39 ---
2 files changed, 34 insertions(+), 39 de
No code change.
Signed-off-by: Laurent Vivier
Acked-by: Richard Henderson
Message-Id: <20180529194207.31503-4-laur...@vivier.eu>
---
linux-user/hppa/target_fcntl.h | 31 +++
linux-user/syscall_defs.h | 34 +-
2 files changed, 32 i
No code change.
Signed-off-by: Laurent Vivier
Acked-by: Richard Henderson
Message-Id: <20180529194207.31503-3-laur...@vivier.eu>
---
linux-user/alpha/target_fcntl.h | 29 +
linux-user/syscall_defs.h | 32 ++--
2 files changed, 31 ins
No code change.
Signed-off-by: Laurent Vivier
Message-Id: <20180529194207.31503-10-laur...@vivier.eu>
---
linux-user/aarch64/target_signal.h| 2 ++
linux-user/arm/target_signal.h| 2 ++
linux-user/cris/target_signal.h | 2 ++
linux-user/generic/signal.h | 57 ++
we have the same problem decribed in 7d6b1daedd
("linux-user, ppc: mftbl can be used by user application")
for ppc in the case of sparc.
When we use an application trying to resolve a name, it hangs in
0xff5dd40c: rd %tick, %o5
0xff5dd410: srlx %o5, 0x20, %o4
0x000
No code change.
Signed-off-by: Laurent Vivier
Message-Id: <20180529194207.31503-15-laur...@vivier.eu>
---
linux-user/hppa/target_signal.h | 46
linux-user/syscall_defs.h | 52 -
2 files changed, 46 insertions(+),
No code change.
Signed-off-by: Laurent Vivier
Message-Id: <20180529194207.31503-12-laur...@vivier.eu>
---
linux-user/mips/target_signal.h | 50 ++
linux-user/mips64/target_signal.h | 49 ++
linux-user/syscall_defs.h | 56 +
No code change.
Signed-off-by: Laurent Vivier
Message-Id: <20180529194207.31503-7-laur...@vivier.eu>
---
linux-user/ppc/target_fcntl.h | 6 ++
linux-user/syscall_defs.h | 7 +--
2 files changed, 7 insertions(+), 6 deletions(-)
diff --git a/linux-user/ppc/target_fcntl.h b/linux-user/
The MPC is guest-configurable for whether blocked accesses:
* should be RAZ/WI or cause a bus error
* should generate an interrupt or not
Implement this behaviour in the blocked-access handlers.
Signed-off-by: Peter Maydell
---
hw/misc/tz-mpc.c | 50 +++
No code change.
Signed-off-by: Laurent Vivier
Message-Id: <20180529194207.31503-13-laur...@vivier.eu>
---
linux-user/openrisc/target_signal.h | 8
linux-user/syscall_defs.h | 10 +-
2 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/linux-user/openrisc/tar
No code change.
Signed-off-by: Laurent Vivier
Message-Id: <20180529194207.31503-14-laur...@vivier.eu>
---
linux-user/alpha/target_signal.h | 45 +++
linux-user/syscall_defs.h| 51 ++--
2 files changed, 47 insertions(+),
Pavel Dovgalyuk writes:
> Ping?
I started having a look but I ran into this straight away. First I
recorded a boot of the kernel:
./aarch64-softmmu/qemu-system-aarch64 -machine
virt,graphics=on,gic-version=3,virtualization=on -cpu cortex-a53 --serial
mon:stdio -display none -kernel
../im
The final part of the Memory Protection Controller we need to
implement is actually using the BLK_LUT data programmed by the
guest to determine whether to block the transaction or not.
Since this means we now change transaction mappings when
the guest writes to BLK_LUT, we must also call the IOMMU
add a per target target_fcntl.h and include the generic one from them
No code change.
Signed-off-by: Laurent Vivier
Message-Id: <20180529194207.31503-2-laur...@vivier.eu>
---
linux-user/aarch64/target_fcntl.h| 11 +++
linux-user/alpha/target_fcntl.h | 11 +++
linux-user/arm/target_fc
Remove useless includes
Fix HPPA include guard.
Signed-off-by: Laurent Vivier
Acked-by: Richard Henderson
Message-Id: <20180529194207.31503-9-laur...@vivier.eu>
---
linux-user/aarch64/signal.c | 1 -
linux-user/aarch64/target_cpu.h | 4
linux-user/aarch64/target_signal.h
Remove a "#if defined(XX) || defined(YY) || ..." with all available
targets
Signed-off-by: Laurent Vivier
Message-Id: <20180529194207.31503-16-laur...@vivier.eu>
---
linux-user/syscall_defs.h | 10 --
1 file changed, 10 deletions(-)
diff --git a/linux-user/syscall_defs.h b/linux-user/sy
Implement the missing registers for the TZ MPC.
Signed-off-by: Peter Maydell
---
include/hw/misc/tz-mpc.h | 10 +++
hw/misc/tz-mpc.c | 137 ++-
2 files changed, 144 insertions(+), 3 deletions(-)
diff --git a/include/hw/misc/tz-mpc.h b/include/hw/misc
This fixes:
linux-user/flatload.c:740:9: warning: Loss of sign in implicit conversion
if (res > (unsigned long)-4096)
^~~
Reported-by: Clang Static Analyzer
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Laurent Vivier
---
linux-user/qemu.h| 5 +
linux-user/syscal
For the IoTKit MPC support, we need to wire together the
interrupt outputs of 17 MPCs; this exceeds the current
value of MAX_OR_LINES. Increase MAX_OR_LINES to 32 (which
should be enough for anyone).
The tricky part is retaining the migration compatibility for
existing OR gates; we add a subsectio
Hi; this is v2 of my iommu patchset, which does:
* support IOMMUs that are aware of memory transaction attributes and
may generate different translations for different attributes
* support TCG execution out of memory which is behind an IOMMU
* implement the Arm TrustZone Memory Protection Con
This fixes:
linux-user/flatload.c:740:9: warning: Loss of sign in implicit conversion
if (res > (unsigned long)-4096)
^~~
Reported-by: Clang Static Analyzer
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Laurent Vivier
---
linux-user/flatload.c | 15 +--
1 fil
On 02/06/2018 10:52, Stefan Hajnoczi wrote:
> Signed-off-by: Stefan Hajnoczi
> ---
> include/qemu/main-loop.h | 14 ++
> 1 file changed, 14 insertions(+)
>
> diff --git a/include/qemu/main-loop.h b/include/qemu/main-loop.h
> index 6b4b60bf6d..721aa2416a 100644
> --- a/include/qemu/ma
Implement the Arm TrustZone Memory Protection Controller, which sits
in front of RAM and allows secure software to configure it to either
pass through or reject transactions.
We implement the MPC as a QEMU IOMMU, which will direct transactions
either through to the devices and memory behind it or
On 4 June 2018 at 16:09, Alex Bennée wrote:
>
> Peter Maydell writes:
>
>> On 23 May 2018 at 10:08, Alex Bennée wrote:
>>> Its a minor thing but are we ever expecting iommu_idx to ever be
>>> negative?
>>
>> Coming back to this one -- no, we don't expect negative iommu_idxs.
>> But on the other
Currently we don't support board configurations that put an IOMMU
in the path of the CPU's memory transactions, and instead just
assert() if the memory region fonud in address_space_translate_for_iotlb()
is an IOMMUMemoryRegion.
Remove this limitation by having the function handle IOMMUs.
This is
;
> are available in the git repository at:
>
> git://git.kraxel.org/qemu tags/vga-20180604-pull-request
>
> for you to fetch changes up to 6bc2fd57e1fc2d1957d1ff952793c53764130218:
>
> vga: cleanup surface handling (2018-06-04 09:44:10 +0200)
>
> --
Add an IOMMU index argument to the translate method of
IOMMUs. Since all of our current IOMMU implementations
support only a single IOMMU index, this has no effect
on the behaviour.
Signed-off-by: Peter Maydell
Reviewed-by: Richard Henderson
Reviewed-by: Alex Bennée
---
include/exec/memory.h
Add support for multiple IOMMU indexes to the IOMMU notifier APIs.
When initializing a notifier with iommu_notifier_init(), the caller
must pass the IOMMU index that it is interested in. When a change
happens, the IOMMU implementation must pass
memory_region_notify_iommu() the IOMMU index that has
On 02/06/2018 07:09, Philippe Mathieu-Daudé wrote:
>
> I realized the binary expect CMD8 to not return, to use the SD card
> with Spec v1.x. Which is the very patch Paul Brook attached...
> http://lists.nongnu.org/archive/html/qemu-devel/2012-04/msg03790.html
>
> curl
> http://lists.nongnu.org/ar
Wire up the one MPC that is part of the IoTKit itself. For the
moment we don't wire up its interrupt line.
Signed-off-by: Peter Maydell
Reviewed-by: Alex Bennée
---
include/hw/arm/iotkit.h | 2 ++
hw/arm/iotkit.c | 38 +++---
2 files changed, 29 insertio
If an IOMMU supports mappings that care about the memory
transaction attributes, then it no longer has a unique
address -> output mapping, but more than one. We can
represent these using an IOMMU index, analogous to TCG's
mmu indexes.
Signed-off-by: Peter Maydell
Reviewed-by: Richard Henderson
-
On 02/06/2018 03:22, John Snow wrote:
> - Status: Should be the status register after receiving the H2D Register
> update FIS, but prior to the data transfer, I think. "New value of the
> Status register of the Command Block for initiation of host data
> transfer."
> I think this is being s
Implement the SECMPCINTSTATUS register. This is the only register
in the security controller that deals with Memory Protection
Controllers, and it simply provides a read-only view of the
interrupt lines from the various MPCs in the system.
Signed-off-by: Peter Maydell
---
include/hw/misc/iotkit-
Instantiate and wire up the Memory Protection Controllers
in the MPS2 board itself.
Signed-off-by: Peter Maydell
Reviewed-by: Alex Bennée
---
hw/arm/mps2-tz.c | 71 ++--
1 file changed, 44 insertions(+), 27 deletions(-)
diff --git a/hw/arm/mps2-tz.c
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: 20180604152015.13359-1-laur...@vivier.eu
Subject: [Qemu-devel] [PULL 00/17] Linux user for 3.0 patches
=== TEST SCRIPT BEGIN ===
#!/bin/bash
BASE=base
n=1
total=$(git log --
On Mon, 4 Jun 2018 16:15:23 +0100
Daniel P. Berrangé wrote:
> On Mon, Jun 04, 2018 at 05:01:11PM +0200, Igor Mammedov wrote:
> > On Mon, 4 Jun 2018 16:21:48 +0200
> > Michal Privoznik wrote:
> >
> > > On 06/04/2018 02:03 PM, Daniel P. Berrangé wrote:
> > > > When using --daemonize, the init
This is a resend of v1 resend ... not sure what happened previously, MTA sent:
550 8-bit characters not allowed in subject (see RFC 2822, sections 3.6.5,
2.2.1)
v1 here:
http://lists.nongnu.org/archive/html/qemu-devel/2017-07/msg07453.html
Philippe Mathieu-Daudé (2):
linux-user: Export us
On Mon, 4 Jun 2018 13:03:44 +0100
Daniel P. Berrangé wrote:
> The RUN_STATE_PRECONFIG state is not supposed to be reachable unless the
> --preconfig argument is given to QEMU, but when it was introduced in:
>
> commit 047f7038f586d2150f16c6d9ba9cfd0479f0f6ac
> Author: Igor Mammedov
> Dat
This fixes:
linux-user/flatload.c:740:9: warning: Loss of sign in implicit conversion
if (res > (unsigned long)-4096)
^~~
Reported-by: Clang Static Analyzer
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Laurent Vivier
---
linux-user/qemu.h| 5 +
linux-user/syscal
This fixes:
linux-user/flatload.c:740:9: warning: Loss of sign in implicit conversion
if (res > (unsigned long)-4096)
^~~
Reported-by: Clang Static Analyzer
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Laurent Vivier
---
linux-user/flatload.c | 15 +--
1 fil
On 02/06/2018 03:17, John Snow wrote:
>
>
> On 04/17/2018 11:39 AM, Paolo Bonzini wrote:
>> For the case where the end_transfer_func is also the caller of
>> ide_transfer_start, the mutual recursion can lead to unlimited
>> stack usage. Introduce a new version that can be used to change
>> tail
On 03/06/2018 02:08, Philippe Mathieu-Daudé wrote:
> Paolo, I guess this would be a good use of the
> Based-on: 201204252110.20873.p...@codesourcery.com tag in
> the git history :P
Oh, I see you did send it. Thanks!
"Based-on" means that it applies to a tree with that patch. It doesn't
mean tha
On 02/06/2018 02:24, John Snow wrote:
>> -if (s->bus->dma->ops->start_transfer) {
>> -s->bus->dma->ops->start_transfer(s->bus->dma);
>> +if (!s->bus->dma->ops->pio_transfer) {
>> +s->end_transfer_func = end_transfer_func;
>> +return;
>> }
>> +s->bus->dma->op
20.04.2018 15:12, Vladimir Sementsov-Ogievskiy wrote:
19.04.2018 23:57, John Snow wrote:
On 03/19/2018 04:07 AM, Vladimir Sementsov-Ogievskiy wrote:
Signed-off-by: Vladimir Sementsov-Ogievskiy
---
If it appropriate for 2.12, let's push it. If not - then for 2.13.
I wonder if I can make the
>> # hw/mem/pc-dimm.c
>> mhp_pc_dimm_assigned_slot(int slot) "%d"
>> mhp_pc_dimm_assigned_address(uint64_t addr) "0x%"PRIx64
>> +# hw/mem/memory-device.c
>> +memory_device_unassign_address(uint64_t addr) "0x%"PRIx64
> maybe split out tracing into a separate patch?
Can do, although I think thi
The interrupt outputs from the MPC in the IoTKit and the expansion
MPCs in the board must be wired up to the security controller, and
also all ORed together to produce a single line to the NVIC.
Signed-off-by: Peter Maydell
Reviewed-by: Alex Bennée
---
include/hw/arm/iotkit.h | 6
hw/arm/
On 1 June 2018 at 02:36, Shannon Zhao wrote:
> While we skip the GIC_INTERNAL irqs, we don't change the register offset
> accordingly. This will overlap the GICR registers value and leave the
> last GIC_INTERNAL irq's registers out of update.
>
> Fix this by skipping the registers banked by GICR.
On Mon, Jun 04, 2018 at 05:40:32PM +0200, Igor Mammedov wrote:
> On Mon, 4 Jun 2018 13:03:44 +0100
> Daniel P. Berrangé wrote:
>
> > The RUN_STATE_PRECONFIG state is not supposed to be reachable unless the
> > --preconfig argument is given to QEMU, but when it was introduced in:
> >
> > commi
The codebase has a bit of a mix of
/* multiline comments
* like this
*/
and
/* multiline comments like this
in the GNU Coding Standards style */
State a preference for the former.
Signed-off-by: Peter Maydell
---
I admit that to some extent I'm imposing my aesthetic
preferences here; p
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: 20180604152941.20374-1-peter.mayd...@linaro.org
Subject: [Qemu-devel] [PATCH v2 00/13] iommu: support txattrs, support TCG
execution, implement TZ MPC
=== TEST SCRIPT BEGIN
On 30 May 2018 at 19:01, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
> target/arm/helper-sve.h| 2 +
> target/arm/sve_helper.c| 11 ++
> target/arm/translate-sve.c | 318 +
> target/arm/sve.decode | 20 +++
> 4 files chan
On Mon, Jun 04, 2018 at 02:01:29PM +0100, Daniel P. Berrangé wrote:
> On Mon, Jun 04, 2018 at 09:54:15AM -0300, Eduardo Habkost wrote:
> > On Mon, Jun 04, 2018 at 04:38:22AM +0300, Michael S. Tsirkin wrote:
> > > On Sun, Jun 03, 2018 at 12:27:49PM +0300, Marcel Apfelbaum wrote:
> > > > Moving to QE
On 30 May 2018 at 19:01, Richard Henderson wrote:
> Rearrange the arithmetic so that we are agnostic about the total size
> of the vector and the size of the element. This will allow us to index
> up to the 32nd byte and with 16-byte elements.
>
> Signed-off-by: Richard Henderson
> ---
> target
On 30 May 2018 at 19:01, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
> target/arm/translate-sve.c | 19 +++
> target/arm/sve.decode | 6 ++
> 2 files changed, 25 insertions(+)
Reviewed-by: Peter Maydell
thanks
-- PMM
On Mon, Jun 04, 2018 at 07:48:51PM +0300, Michael S. Tsirkin wrote:
> On Mon, Jun 04, 2018 at 02:01:29PM +0100, Daniel P. Berrangé wrote:
> > On Mon, Jun 04, 2018 at 09:54:15AM -0300, Eduardo Habkost wrote:
> > > On Mon, Jun 04, 2018 at 04:38:22AM +0300, Michael S. Tsirkin wrote:
> > > > On Sun, Ju
On 30 May 2018 at 19:01, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
> target/arm/helper-sve.h| 14 +
> target/arm/sve_helper.c| 41 +++---
> target/arm/translate-sve.c | 38 +++
> target/ar
On 30 May 2018 at 19:01, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
> target/arm/helper-sve.h| 2 ++
> target/arm/sve_helper.c| 37 +
> target/arm/translate-sve.c | 13 +
> target/arm/sve.decode | 3 +++
> 4 fi
On 30 May 2018 at 19:01, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
> target/arm/helper-sve.h| 9 +++
> target/arm/sve_helper.c| 55 ++
> target/arm/translate-sve.c | 2 ++
> target/arm/sve.decode | 6 +
> 4 files
On Mon, Jun 04, 2018 at 10:26:24AM -0300, Eduardo Habkost wrote:
> On Mon, Jun 04, 2018 at 02:01:29PM +0100, Daniel P. Berrangé wrote:
> > On Mon, Jun 04, 2018 at 09:54:15AM -0300, Eduardo Habkost wrote:
> > > On Mon, Jun 04, 2018 at 04:38:22AM +0300, Michael S. Tsirkin wrote:
> > > > On Sun, Jun 0
POWER9 introduced a new variant of the eieio instruction using bit 6
as a hint to tell the CPU it is a store-forwarding barrier.
The usage of this eieio extension was recently added in Linux 4.17
which activated the "support for a store forwarding barrier at kernel
entry/exit".
This loosen the QE
On 06/04/2018 12:11 PM, Vladimir Sementsov-Ogievskiy wrote:
> 20.04.2018 15:12, Vladimir Sementsov-Ogievskiy wrote:
>> 19.04.2018 23:57, John Snow wrote:
>>>
>>> On 03/19/2018 04:07 AM, Vladimir Sementsov-Ogievskiy wrote:
Signed-off-by: Vladimir Sementsov-Ogievskiy
---
If it
On 30 May 2018 at 19:01, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
> target/arm/helper-sve.h| 115 +++
> target/arm/sve_helper.c| 187 +
> target/arm/translate-sve.c | 91 ++
> target/arm/sve
0618-1' into staging (2018-06-01
>> 17:32:30 +0100)
>>
>> are available in the git repository at:
>>
>> git://git.kraxel.org/qemu tags/vga-20180604-pull-request
>>
>> for you to fetch changes up to 6bc2fd57e1fc2d1957d1ff95279
On 30 May 2018 at 19:01, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
> target/arm/helper-sve.h| 44 +++
> target/arm/sve_helper.c| 88 ++
> target/arm/translate-sve.c | 66
> target/arm
On Mon, 4 Jun 2018 17:11:57 +0100
Daniel P. Berrangé wrote:
> On Mon, Jun 04, 2018 at 05:40:32PM +0200, Igor Mammedov wrote:
> > On Mon, 4 Jun 2018 13:03:44 +0100
> > Daniel P. Berrangé wrote:
> >
> > > The RUN_STATE_PRECONFIG state is not supposed to be reachable unless the
> > > --preconfi
On 06/04/2018 11:48 AM, Paolo Bonzini wrote:
> On 02/06/2018 02:24, John Snow wrote:
>>> -if (s->bus->dma->ops->start_transfer) {
>>> -s->bus->dma->ops->start_transfer(s->bus->dma);
>>> +if (!s->bus->dma->ops->pio_transfer) {
>>> +s->end_transfer_func = end_transfer_func;
On 30 May 2018 at 07:40, Cédric Le Goater wrote:
> Hello
>
> This series adds a new Aspeed machine to emulate the BMC of a
> Witherspoon system. It also extends the other Aspeed machines with I2C
> devices and adds a simple model for the pca9552 LED blinker present on
> the Witherspoon board.
>
>
On 06/04/2018 12:21 PM, Peter Maydell wrote:
> The codebase has a bit of a mix of
> /* multiline comments
> * like this
> */
> and
> /* multiline comments like this
> in the GNU Coding Standards style */
>
> State a preference for the former.
>
> Signed-off-by: Peter Maydell
> ---
>
On Sat, Jun 2, 2018 at 3:26 PM, Philippe Mathieu-Daudé wrote:
> On 06/02/2018 05:55 PM, Philippe Mathieu-Daudé wrote:
>> These macros are always used to deposit a field in place.
>> Update them to take the pointer argument.
>>
>> As these macros are constructed using compound statements,
>> it is
On 06/04/2018 08:35 AM, Paolo Bonzini wrote:
> On 04/06/2018 14:24, Igor Mammedov wrote:
>>> Yes it's easy to add -machine pc but there's no documentation
>>> that tells you to do so. Add to that shortcuts like -cdrom
>>> stop working, hotplug needs extra bridges to work, and one
>>> can see tha
On Sat, Jun 2, 2018 at 5:08 PM, Philippe Mathieu-Daudé wrote:
> Hi,
>
> This series adds a 'spec_version' property to the SD Card device,
> to allow to limit some commands to specific spec version range
> (some firmwares use this feature to detect which spec version the
> card implements).
>
> Thi
On Sat, Jun 2, 2018 at 5:08 PM, Philippe Mathieu-Daudé wrote:
> From the "Physical Layer Simplified Specification Version 1.10"
> Chapter 7.3 "SPI Mode Transaction Packets"
> Table 57: "Commands and arguments"
>
> Signed-off-by: Philippe Mathieu-Daudé
Acked-by: Alistair Francis
Alistair
On Sat, Jun 2, 2018 at 5:08 PM, Philippe Mathieu-Daudé wrote:
> Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Alistair Francis
Alistair
> ---
> include/hw/sd/sd.h | 4
> hw/sd/sd.c | 11 +++
> 2 files changed, 15 insertions(+)
>
> diff --git a/include/hw/sd/sd.h b/
On Sat, Jun 2, 2018 at 5:08 PM, Philippe Mathieu-Daudé wrote:
> Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Alistair Francis
Alistair
> ---
> include/hw/sd/sd.h | 1 +
> hw/sd/sd.c | 13 +
> 2 files changed, 10 insertions(+), 4 deletions(-)
>
> diff --git a/includ
On Sat, Jun 2, 2018 at 5:08 PM, Philippe Mathieu-Daudé wrote:
> Signed-off-by: Philippe Mathieu-Daudé
Can you add a justification for this in the commit message?
Alistair
> ---
> hw/sd/ssi-sd.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/hw/sd/ssi-sd.c b/hw/sd/ssi-sd.c
> index
Hi Michael,
On 06/04/2018 04:38 AM, Michael S. Tsirkin wrote:
On Sun, Jun 03, 2018 at 12:27:49PM +0300, Marcel Apfelbaum wrote:
Moving to QEMU 3.0 seems like a good opportunity for such a change.
I440FX is really old and does not support modern features like IOMMU.
Q35's SATA emulation is fast
1 - 100 of 322 matches
Mail list logo