The bugs this was working around were fixed with commits
022d6378c7fd target/unicore32: remove tlb_flush from uc32_init_fn
6e11beecfde0 target/alpha: remove tlb_flush from alpha_cpu_initfn
Signed-off-by: Richard Henderson
---
accel/tcg/cputlb.c | 7 ---
1 file changed, 7 deletions(-)
diff
Protect it with the tlb_lock instead of using atomics.
The move puts it in or near the same cacheline as the lock;
using the lock means we don't need a second atomic operation
in order to perform the update. Which makes it cheap to also
update pending_flush in tlb_flush_by_mmuidx_async_work.
Sign
The difference between the two sets of APIs is now miniscule.
This allows tlb_flush, tlb_flush_all_cpus, and tlb_flush_all_cpus_synced
to be merged with their corresponding by_mmuidx functions as well. For
accounting, consider mmu_idx_bitmask = ALL_MMUIDX_BITS to be a full flush.
Signed-off-by:
The rest of the tlb victim cache is per-tlb,
the next use index should be as well.
Signed-off-by: Richard Henderson
---
include/exec/cpu-defs.h | 5 +++--
accel/tcg/cputlb.c | 5 ++---
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/include/exec/cpu-defs.h b/include/exec/cpu-
The difference between the two sets of APIs is now miniscule.
Signed-off-by: Richard Henderson
---
accel/tcg/cputlb.c | 58 ++
1 file changed, 12 insertions(+), 46 deletions(-)
diff --git a/accel/tcg/cputlb.c b/accel/tcg/cputlb.c
index 6b0f93ec01..444
Our only statistic so far was "full" tlb flushes, where all mmu_idx
are flushed at the same time.
Now count "partial" tlb flushes where sets of mmu_idx are flushed,
but the set is not maximal. Account one per mmu_idx flushed, as
that is the unit of work performed.
We don't actually count elided
In several places we use assert(FEATURE), and assume that if FEATURE
is disabled, all following code is removed as unreachable. Which allows
us to compile-out functions that are only present with FEATURE, and
have a link-time failure if the functions remain used.
MinGW does not mark its internal
This is essentially redundant with tlb_c.dirty.
[??? Collect data to back up this supposition]
Signed-off-by: Richard Henderson
---
include/exec/cpu-defs.h | 6 --
accel/tcg/cputlb.c | 16 ++--
2 files changed, 2 insertions(+), 20 deletions(-)
diff --git a/include/exec/cpu
Especially for guests with large numbers of tlbs, like ARM or PPC,
we may well not use all of them in between flush operations.
Remember which tlbs have been used since the last flush, and
avoid any useless flushing.
Signed-off-by: Richard Henderson
---
include/exec/cpu-defs.h | 7 ++-
acce
On 2018-10-18 19:28, Mark Cave-Ayland wrote:
> From: Laurent Vivier
>
> Co-developed-by: Mark Cave-Ayland
> Signed-off-by: Mark Cave-Ayland
> Signed-off-by: Laurent Vivier
> ---
> arch_init.c | 4 +
> hw/display/Makefile.objs| 1 +
> hw/display/macfb-template.h | 158 +
This is the first of several moves to reduce the size of the
CPU_COMMON_TLB macro and improve some locality of refernce.
Signed-off-by: Richard Henderson
---
include/exec/cpu-defs.h | 17 ---
accel/tcg/cputlb.c | 48 -
2 files changed, 38
The motivation here is reducing the total overhead.
Before a few patches went into target-arm.next, I measured total
tlb flush overhead for aarch64 at 25%. This appears to reduce the
total overhead to about 5% (I do need to re-run the control tests,
not just watch perf top as I'm doing now).
The
The set of large pages in the kernel is probably not the same
as the set of large pages in the application. Forcing one
range to cover both will flush more often than necessary.
This allows tlb_flush_page_async_work to flush just the one
mmu_idx implicated, which in turn allows us to remove
tlb_c
On 2018-10-18 19:28, Mark Cave-Ayland wrote:
> From: Laurent Vivier
>
> This is broken as the linux driver seems broken too...
So is this required for the initial version, or does Linux also boot
without this device in the emulated machine? In the first case, please
mention it in the cover lette
On 2018-10-18 19:28, Mark Cave-Ayland wrote:
> From: Laurent Vivier
>
> Co-developed-by: Mark Cave-Ayland
> Signed-off-by: Mark Cave-Ayland
> Signed-off-by: Laurent Vivier
> ---
[...]
> +static void nubus_register_format_block(NubusDevice *dev)
> +{
> +char fblock_name[27];
> +
> +spri
在 2018/10/22 下午8:17, Thomas Huth 写道:
On 2018-10-22 10:02, Yi Min Zhao wrote:
Common function measurement block is used to report counters of
successfully issued pcilg/stg/stb and rpcit instructions. This patch
introduces a new struct ZpciFmb and schedules a timer callback to
copy fmb to the g
On 10/17/18 4:43 PM, Paolo Bonzini wrote:
On 17/10/2018 15:54, Eric Blake wrote:
So the problem with shutdown_caused_by_guest is that you get the same
value for both guest reset and guest shutdown. Could we instead just
pass the ShutdownCause in the event (similar to what was proposed even
whe
On 18/10/18 17:31, Cleber Rosa wrote:
A number of QEMU tests are written in Python, and may benefit
from an untainted Python venv.
By using make rules, tests that depend on specific Python libs
can set that rule as a requirement, along with rules that require
the presence or installation of spec
* Zhang Chen (zhangc...@gmail.com) wrote:
> This structure and command have missed qemu version 3.0, so fix it to since
> version 3.1.
>
> Signed-off-by: Zhang Chen
Reviewed-by: Dr. David Alan Gilbert
> ---
> qapi/migration.json | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
On 10/20/18 8:14 AM, Bastian Koppelmann wrote:
> Signed-off-by: Bastian Koppelmann
> Signed-off-by: Peer Adelt
> ---
> v1 -> v2:
> - Add REQUIRE_FPU to arithm helpers
Reviewed-by: Richard Henderson
r~
On 10/20/18 8:14 AM, Bastian Koppelmann wrote:
> v1 -> v2:
> - Stack allocate arg_c_* structs
> - ex_rvc_register returns int
> - special case of trans_c_addi4spn() returns false
...
> +static bool trans_c_addi4spn(DisasContext *ctx, arg_c_addi4spn *a,
> +uint16_t insn)
> +{
> +
On 10/20/18 8:14 AM, Bastian Koppelmann wrote:
> Signed-off-by: Bastian Koppelmann
> Signed-off-by: Peer Adelt
> ---
> v1 -> v2:
> - simplified gen_amo() with function pointers
> - rs2 @atom_ld is now decimal
>
> target/riscv/insn32.decode | 17 +++
> target/riscv/insn_tra
On 10/20/18 8:14 AM, Bastian Koppelmann wrote:
> --- a/target/riscv/insn_trans/trans_rvc.inc.c
> +++ b/target/riscv/insn_trans/trans_rvc.inc.c
> @@ -23,8 +23,7 @@ static bool trans_c_addi4spn(DisasContext *ctx,
> arg_c_addi4spn *a,
> {
> if (a->nzuimm == 0) {
> /* Reserved in ISA */
On 10/20/18 8:14 AM, Bastian Koppelmann wrote:
> This also removes all functions that now became obsolete.
>
> Signed-off-by: Bastian Koppelmann
> ---
> target/riscv/insn16.decode | 34 +-
> target/riscv/insn_trans/trans_rvc.inc.c | 103
> target/riscv/translat
On 10/20/18 8:14 AM, Bastian Koppelmann wrote:
> static bool trans_srli(DisasContext *ctx, arg_srli *a, uint32_t insn)
> {
> -gen_arith_imm(ctx, OPC_RISC_SHIFT_RIGHT_I, a->rd, a->rs1, a->shamt);
> +if (a->rd != 0) {
> +TCGv t = tcg_temp_new();
> +gen_get_gpr(t, a->rs1);
>
On 10/20/18 8:14 AM, Bastian Koppelmann wrote:
> +static bool trans_arith(DisasContext *ctx, arg_arith *a,
> +void(*func)(TCGv, TCGv, TCGv))
gen_arith.
r~
On 10/23/18 9:53 AM, Richard Henderson wrote:
> On 10/20/18 8:14 AM, Bastian Koppelmann wrote:
>> +static bool trans_arith(DisasContext *ctx, arg_arith *a,
>> +void(*func)(TCGv, TCGv, TCGv))
>
> gen_arith.
Oh, right, last patch.
Reviewed-by: Richard Henderson
r~
On 19 October 2018 at 13:19, Laurent Vivier wrote:
> The following changes since commit 1b7490446bf41f54130c2d495dd4c8768c8e1ce3:
>
> Merge remote-tracking branch
> 'remotes/vivier2/tags/linux-user-for-3.1-pull-request' into staging
> (2018-10-19 11:20:05 +0100)
>
> are available in the Git re
On Mon, Oct 22, 2018 at 05:28:54PM -0400, Michael S. Tsirkin wrote:
> On Mon, Oct 22, 2018 at 08:36:30PM +0200, Samuel Ortiz wrote:
> > This patch set implements support for the ACPI hardware-reduced
> > specification.
> >
> > The changes are coming from the NEMU [1] project where we're defining
>
On 10/20/18 8:14 AM, Bastian Koppelmann wrote:
> static bool trans_mulh(DisasContext *ctx, arg_mulh *a, uint32_t insn)
> {
> -gen_arith(ctx, OPC_RISC_MULH, a->rd, a->rs1, a->rs2);
> +TCGv source1 = tcg_temp_new();
> +TCGv source2 = tcg_temp_new();
> +gen_get_gpr(source1, a->rs1);
On 10/20/18 8:14 AM, Bastian Koppelmann wrote:
> decodetree handles all instructions now so the fallback is not necessary
> anymore.
>
> Signed-off-by: Bastian Koppelmann
> Signed-off-by: Peer Adelt
> ---
> v1 -> v2:
> - Add missing gen_exception_illegal
Reviewed-by: Richard Henderson
r~
On 10/20/18 8:14 AM, Bastian Koppelmann wrote:
> Signed-off-by: Bastian Koppelmann
> ---
> target/riscv/insn_trans/trans_rvi.inc.c | 14 +++---
> target/riscv/insn_trans/trans_rvm.inc.c | 14 +++---
> target/riscv/translate.c| 4 ++--
> 3 files changed, 16 inserti
From: Manish Jaggi
When KVM_SET_ONE_REG returns KVM_EINVARIANT call KVM_GET_ONE_REG to query
and then replace the particular guest invariant register value with destination
hosts register.
Signed-off-by: Manish Jaggi
diff --git a/target/arm/kvm.c b/target/arm/kvm.c
index 65f867d..8cf4dc9 10064
From: Manish Jaggi
This option is user opt-in. hostinvariant will replace guest's invariant
registers with hosts.
Signed-off-by: Manish Jaggi
diff --git a/qemu-options.hx b/qemu-options.hx
index 654ef48..d8c0da7 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -3896,6 +3896,19 @@ STEXI
p
From: Manish Jaggi
QEMU on arm systems use -machine virt -cpu host option for a VM.
Migration thus is limited between machines with same cpu.
This is a limitation if migration is desired between cpus which are of same
family and have only few difeerences like bug fixes which have no effect on
VM
From: Aleksandar Rikalo
Add support for semtimedop() emulation.
Signed-off-by: Aleksandar Rikalo
Signed-off-by: Aleksandar Markovic
---
linux-user/syscall.c | 27 +++
1 file changed, 27 insertions(+)
diff --git a/linux-user/syscall.c b/linux-user/syscall.c
index b8435
Hi,
I'm going to collect patches for the next pull request for the
qemu-trivial I'd like to do before the softfreeze.
As we didn't have any pull request for more than 3 months and some of
the patches have been pushed through another maintainer branches, it's a
little bit difficult to collect corr
From: Aleksandar Rikalo
Implement support for translation of system call statx(). The
implementation includes invoking other (more mature) syscalls
(from the same 'stat' family) on the host side. This way,
problems of availability of statx() on the host side are
avoided.
Signed-off-by: Aleksanda
From: Yunqiang Su
Add support for SO_REUSEPORT, including strace support. SO_REUSEPORT
was introduced relatively recently, since Linux 3.9, so use
'#if defined SO_REUSEPORT'.
Signed-off-by: Yunqiang Su
Signed-off-by: Aleksandar Markovic
---
linux-user/strace.c | 3 +++
linux-user/syscall.c
From: Aleksandar Markovic
This series add support for two system calls and a socket-related
improvement.
Aleksandar Rikalo (2):
linux-user: Add support for statx() syscall
linux-user: Add support for semtimedop() syscall
Yunqiang Su (1):
linux-user: Add support for SO_REUSEPORT
linux-us
On 19/10/18 15:38, Daniel P. Berrangé wrote:
From: "Daniel P. Berrange"
Add a QAuthZList object type that implements the QAuthZ interface. This
built-in implementation maintains a trivial access control list with a
sequence of match rules and a final default policy. This replicates the
function
Well, after checking manually, the header files
"hw/arm/primecell.h"
"qapi/visitor.h"
"hw/arm/smmuv3.h"
are really not used in virt.c, still can be removed.
On 22 October 2018 at 18:17, Hongbo Zhang wrote:
> On 22 October 2018 at 09:37, Hongbo Zhang wrote:
>> On 19 October 2018 at 19:59, Peter M
From: Manish Jaggi
This patch introduces an error code KVM_EINVARIANT which is returned
by KVM when userland tries to set an invariant register.
The need for this error code is in VM Migration for arm64.
ARM64 systems use mainly -machine virt -cpu host as parameter to qemu.
Migration requires bo
On 23/10/18 9:02, Richard Henderson wrote:
In several places we use assert(FEATURE), and assume that if FEATURE
is disabled, all following code is removed as unreachable. Which allows
us to compile-out functions that are only present with FEATURE, and
have a link-time failure if the functions re
On 23/10/18 9:02, Richard Henderson wrote:
This is the first of several moves to reduce the size of the
CPU_COMMON_TLB macro and improve some locality of refernce.
"reference"
Signed-off-by: Richard Henderson
Reviewed-by: Philippe Mathieu-Daudé
---
include/exec/cpu-defs.h | 17 ++
On 23/10/18 9:02, Richard Henderson wrote:
The rest of the tlb victim cache is per-tlb,
the next use index should be as well.
Signed-off-by: Richard Henderson
Reviewed-by: Philippe Mathieu-Daudé
---
include/exec/cpu-defs.h | 5 +++--
accel/tcg/cputlb.c | 5 ++---
2 files changed,
On 19/10/18 15:38, Daniel P. Berrangé wrote:
From: "Daniel P. Berrange"
The 'qemu_acl' type was a previous non-QOM based attempt to provide an
authorization facility in QEMU. Because it is non-QOM based it cannot be
created via the command line and requires special monitor commands to
manipulat
On 22 October 2018 at 19:16, Richard Henderson
wrote:
> In several places we use assert(FEATURE), and assume that if FEATURE
> is disabled, all following code is removed as unreachable. Which allows
> us to compile-out functions that are only present with FEATURE, and
> have a link-time failure i
On 2018-10-18 19:28, Mark Cave-Ayland wrote:
> From: Laurent Vivier
>
> Co-developed-by: Mark Cave-Ayland
> Signed-off-by: Mark Cave-Ayland
> Signed-off-by: Laurent Vivier
> ---
[...]
> diff --git a/hw/m68k/Makefile.objs b/hw/m68k/Makefile.objs
> index d1f089c08a..18ebf1937a 100644
> --- a/hw/
From: Sameeh Jubran
The fsinfo command is currently implemented for Windows only and it's disk
parameter can be enabled by adding the define "CONFIG_QGA_NTDDSCSI" to the qga
code. When enabled and executed the qemu-ga crashed with the following message:
--
Refactor code that queries bus type to be more generic. The function
get_disk_bus_type() has been renamed to build_guest_disk_info().
Following commit(s) will extend this function.
Signed-off-by: Tomáš Golembiovský
---
qga/commands-win32.c | 44 +++-
1 fil
From: Sameeh Jubran
The call to SetupDiGetDeviceRegistryProperty might fail because the
value doesn't exist in the registry, in this case we shouldn't exit from
the loop but instead continue to look for other available values in the
registry and set this value as unavailable (-1).
Signed-off-by:
Note that PCI controller reporting on Windows was and still is broken.
Unfortunately I don't know how to fix it at the momemnt. See commit message and
code comment. If anyone has environment where the original code works let me
know. CCing author of the code In case I missed something obvious.
v5:
Signed-off-by: Tomáš Golembiovský
Reviewed-by: Marc-André Lureau
---
configure | 22 ++
1 file changed, 22 insertions(+)
diff --git a/configure b/configure
index c3da1a2587..26e8605bbc 100755
--- a/configure
+++ b/configure
@@ -477,6 +477,7 @@ libxml2=""
docker="no"
debug_
Add reporting of disk serial number on Linux guests. The feature depends
on libudev.
Example:
{
"name": "dm-2",
"mountpoint": "/",
...
"disk": [
{
"serial": "SAMSUNG_MZ7LN512HCHP-000L1_S1ZKNXAG822493",
...
}
],
}
Signed-of
From: Michael Roth
(cherry picked from commit 51fe925a7b473c76e06b60ddb8a862c91901b290)
Signed-off-by: Michael Roth
Signed-off-by: Tomáš Golembiovský
---
qga/commands-win32.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/qga/commands-win32.c b/qga/commands-win32.c
Signed-off-by: Tomáš Golembiovský
---
qga/commands-win32.c | 25 +
1 file changed, 25 insertions(+)
diff --git a/qga/commands-win32.c b/qga/commands-win32.c
index 5ebacd241f..e7d9cdea62 100644
--- a/qga/commands-win32.c
+++ b/qga/commands-win32.c
@@ -609,6 +609,31 @@ stat
There was inconsistency between commits:
50cbebb9a3 configure: add configure check for ntdddisk.h
a3ef3b2272 qga: added bus type and disk location path
The first commit added #define CONFIG_QGA_NTDDDISK but the second commit
expected the name to be CONFIG_QGA_NTDDSCSI. As a result the code in
Report device node of the disk on Linux (e.g. "/dev/sda2").
Requirs libudev.
Signed-off-by: Tomáš Golembiovský
---
qga/commands-posix.c | 7 ++-
qga/qapi-schema.json | 3 ++-
2 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/qga/commands-posix.c b/qga/commands-posix.c
index d087
The windows code generaly lacks debug information (compared to posix
code). This patch adds some related to HW info in guest-get-fsinfo
command.
Signed-off-by: Tomáš Golembiovský
Reviewed-by: Marc-André Lureau
---
dtc | 2 +-
qga/commands-win32.c | 15 +++
2 files
Probe the volume for disk extents and return list of all disks.
Originally only first disk of composite volume was returned.
Note that the patch changes get_pci_info() from one state of brokenness
into a different state of brokenness. In other words it still does not do
what it's supposed to do (s
Refactor building of disk info into a function that builds the list and
a function that returns infor for single disk. This will be used in
future commit that will handle multi-disk volumes.
Signed-off-by: Tomáš Golembiovský
---
qga/commands-win32.c | 58 ++---
Report device UNC of the disk. It is reported as "\\.\PhysicalDriveX".
Signed-off-by: Tomáš Golembiovský
---
qga/commands-win32.c | 21 +++--
1 file changed, 11 insertions(+), 10 deletions(-)
diff --git a/qga/commands-win32.c b/qga/commands-win32.c
index 09566b3f47..12aba8cc50 1
It was not obvious what exactly the cryptic string copying does to the
GUID. This change makes the intent clearer.
Signed-off-by: Tomáš Golembiovský
---
qga/commands-win32.c | 10 +-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/qga/commands-win32.c b/qga/commands-win32.c
??? Should be merged back into previous patches.
??? Additional changes required for compact mode.
---
.../riscv/insn_trans/trans_privileged.inc.c | 19 ++-
target/riscv/insn_trans/trans_rva.inc.c | 44 +++---
target/riscv/insn_trans/trans_rvc.inc.c | 148 +-
target
For format x, use &x for the argument set and @x for the extract.
This is less confusing than e.g. "arith" for format R.
---
target/riscv/insn_trans/trans_rvi.inc.c | 2 +-
target/riscv/translate.c| 10 +-
target/riscv/insn32.decode | 12 ++--
3 files
The first patch is the promised change to argument sets to allow
them to be shared between two decoders.
Then I set about trying to use this... and ran into some other
interesting problems. The first of which is that
bool trans_add(DecodeContext *ctx, struct arg_r *a, uint32_t insn);
bool t
This primarily solves the case for RVC that several insns are
completely different, decode and all, between the two. But it
also means that we need less ifdefing for RV{I,M,A,F,D}.
---
target/riscv/insn_trans/trans_rva.inc.c | 46 +---
target/riscv/insn_trans/trans_rvc.inc.c | 71
??? Needs simultaneous corresponding changes to all
translators using decodetree.
---
scripts/decodetree.py | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/scripts/decodetree.py b/scripts/decodetree.py
index a9b10452ef..c0bb447095 100755
--- a/scripts/decodetree.py
+++ b/s
These all expand simply to R format instructions.
---
target/riscv/insn_trans/trans_rvc.inc.c | 36 -
target/riscv/translate.c| 20 +++---
target/riscv/insn16.decode | 17 +++-
3 files changed, 26 insertions(+), 47 deletions(-)
---
target/riscv/insn_trans/trans_rvc.inc.c | 34 +++--
target/riscv/insn16.decode | 18 +++--
target/riscv/insn32.decode | 3 ++-
3 files changed, 16 insertions(+), 39 deletions(-)
diff --git a/target/riscv/insn_trans/trans_rvc.inc.c
b/targ
Allow argument sets to be shared between two decoders by avoiding
a re-declaration error. Make sure that anonymous argument sets
have unique names.
Signed-off-by: Richard Henderson
---
scripts/decodetree.py | 34 +++---
1 file changed, 23 insertions(+), 11 deletions(
On 17/10/2018 14:42, Markus Armbruster wrote:
> Quick peek only for now.
>
> David Hildenbrand writes:
>
>> Right now, we parse uint64_t values just like int64_t values, resulting
>> in negative values getting accepted and certain valid large numbers only
>> being representable as negative numbe
Hi Aleksandar,
On 23/10/18 11:58, Aleksandar Markovic wrote:
From: Aleksandar Rikalo
Add support for semtimedop() emulation.
Signed-off-by: Aleksandar Rikalo
Signed-off-by: Aleksandar Markovic
---
linux-user/syscall.c | 27 +++
1 file changed, 27 insertions(+)
di
On 23/10/18 14:04, Richard Henderson wrote:
This primarily solves the case for RVC that several insns are
completely different, decode and all, between the two. But it
also means that we need less ifdefing for RV{I,M,A,F,D}.
Lovely!
Probably due to the jet lag you forgot your S-o-b :p
With it
On 23/10/18 14:04, Richard Henderson wrote:
For format x, use &x for the argument set and @x for the extract.
This is less confusing than e.g. "arith" for format R.
With your S-o-b:
Reviewed-by: Philippe Mathieu-Daudé
---
target/riscv/insn_trans/trans_rvi.inc.c | 2 +-
target/riscv/trans
On 23/10/18 14:04, Richard Henderson wrote:
These all expand simply to R format instructions.
---
target/riscv/insn_trans/trans_rvc.inc.c | 36 -
target/riscv/translate.c| 20 +++---
target/riscv/insn16.decode | 17 +++-
3
With S-o-b:
Reviewed-by: Philippe Mathieu-Daudé
On 23/10/18 14:04, Richard Henderson wrote:
---
target/riscv/insn_trans/trans_rvc.inc.c | 34 +++--
target/riscv/insn16.decode | 18 +++--
target/riscv/insn32.decode | 3 ++-
3 files chan
On Mon, Oct 22, 2018 at 06:55:06PM +0200, Vitaly Kuznetsov wrote:
> Adds a new CPU flag to enable the Enlightened VMCS KVM feature.
> QEMU enables KVM_CAP_HYPERV_ENLIGHTENED_VMCS and gets back the
> version to be advertised in lower 16 bits of CPUID.0x400A:EAX.
>
> Suggested-by: Ladi Prosek
>
On Tue, Oct 23, 2018 at 11:00:18AM +0200, Samuel Ortiz wrote:
> > I wonder whether the code can be refactored slightly differently:
> > instead of common code calling out to platform specific one,
> > have platform code call out to generic one.
> > That would avoid indicectiin
should have been ind
On 10/23/18 2:04 PM, Richard Henderson wrote:
Allow argument sets to be shared between two decoders by avoiding
a re-declaration error. Make sure that anonymous argument sets
have unique names.
Signed-off-by: Richard Henderson
---
scripts/decodetree.py | 34 +++-
On Tue, Oct 23, 2018 at 01:49:16AM +0200, Philippe Mathieu-Daudé wrote:
> On 22/10/18 4:17, yuchenlin via Qemu-devel wrote:
> > Ping?
> >
> > On 2018-10-12 17:07, yuchen...@synology.com wrote:
> > > From: yuchenlin
> > >
> > > There are 3 virtqueues (ctrl, event and cmd) for virtio scsi device,
On 10/23/18 1:21 PM, Philippe Mathieu-Daudé wrote:
> On 23/10/18 14:04, Richard Henderson wrote:
>> For format x, use &x for the argument set and @x for the extract.
>> This is less confusing than e.g. "arith" for format R.
>
> With your S-o-b:
> Reviewed-by: Philippe Mathieu-Daudé
Oh no. I ful
The following changes since commit b312532fd03413d0e6ae6767ec793a3e30f487b8:
Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into staging
(2018-10-19 19:01:07 +0100)
are available in the Git repository at:
git://repo.or.cz/qemu/armbru.git tags/pull-error-2018-10-22-v2
for
On 10/23/18 1:26 PM, Philippe Mathieu-Daudé wrote:
>> +#pragma GCC diagnostic push
>> +#pragma GCC diagnostic ignored "-Wredundant-decls"
>
> We might move that to a generic Makefile rule, this will be useful for other
> archs.
>
> Something like:
>
> translate*.o: CPPFLAGS += -Wno-redundant-d
On 10/23/18 2:04 PM, Richard Henderson wrote:
??? Needs simultaneous corresponding changes to all
translators using decodetree.
---
scripts/decodetree.py | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
Was the only intend of the insn argument to be used for manual decoding
in a
On 19 October 2018 at 14:43, Daniel P. Berrangé wrote:
> The following changes since commit 2ec24af2379e331d062a6fc1cda65bc262c7c17b:
>
> Merge remote-tracking branch
> 'remotes/amarkovic/tags/mips-queue-october-2018-part1-v2' into staging
> (2018-10-19 10:08:31 +0100)
>
> are available in the
Calling error_report() in a function that takes an Error ** argument
is suspicious. Convert a few that are actually warnings to
warn_report().
While there, improve a rather useless warning in sheepdog.c.
Cc: Kevin Wolf
Cc: Ronnie Sahlberg
Cc: Paolo Bonzini
Cc: Peter Lieven
Cc: Liu Yuan
Sign
From: Aleksandar Rikalo
Add support for semtimedop() emulation.
Signed-off-by: Aleksandar Rikalo
Signed-off-by: Aleksandar Markovic
---
linux-user/syscall.c | 27 +++
1 file changed, 27 insertions(+)
diff --git a/linux-user/syscall.c b/linux-user/syscall.c
index 8b01a
From: Aleksandar Rikalo
Implement support for translation of system call statx(). The
implementation includes invoking other (more mature) syscalls
(from the same 'stat' family) on the host side. This way,
problems of availability of statx() on the host side are
avoided.
Signed-off-by: Aleksanda
From: Aleksandar Rikalo
Add support for initial ramdisk loading for the Mips Boston board.
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Aleksandar Rikalo
Signed-off-by: Aleksandar Markovic
---
v2->v3:
- a comment was reformatted
- rebased to the latest QEMU code
v1->v2:
- 'long in
On 10/23/18 2:04 PM, Bastian Koppelmann wrote:
> On 10/23/18 2:04 PM, Richard Henderson wrote:
>> ??? Needs simultaneous corresponding changes to all
>> translators using decodetree.
>> ---
>> scripts/decodetree.py | 5 ++---
>> 1 file changed, 2 insertions(+), 3 deletions(-)
>>
>
> Was the onl
From: Aleksandar Markovic
This series add support for two system calls and a socket-related
improvement.
v1->v2:
- fixed a typo in the title
- cosmetic comment reformats
- fixed build error related to old label 'efault'
- fixed wrong email addresses
Aleksandar Rikalo (2):
linux-user:
From: Yunqiang Su
Add support for SO_REUSEPORT, including strace support. SO_REUSEPORT
was introduced relatively recently, since Linux 3.9, so use
'#if defined SO_REUSEPORT'.
Signed-off-by: Yunqiang Su
Signed-off-by: Aleksandar Markovic
---
linux-user/strace.c | 3 +++
linux-user/syscall.c
Hi Laurent, Mark,
On 18/10/18 20:28, Mark Cave-Ayland wrote:
From: Laurent Vivier
Co-developed-by: Mark Cave-Ayland
Signed-off-by: Mark Cave-Ayland
Signed-off-by: Laurent Vivier
---
MAINTAINERS | 19 ++
default-configs/m68k-softmmu.mak | 14 ++
hw/intc/Makefile.ob
On 10/23/18 3:08 PM, Richard Henderson wrote:
On 10/23/18 2:04 PM, Bastian Koppelmann wrote:
On 10/23/18 2:04 PM, Richard Henderson wrote:
??? Needs simultaneous corresponding changes to all
translators using decodetree.
---
scripts/decodetree.py | 5 ++---
1 file changed, 2 insertions(+
On Mon, Oct 22, 2018 at 10:43:32AM -0400, Jintack Lim wrote:
> On Mon, Oct 22, 2018 at 5:27 AM Peter Xu wrote:
> >
> > On Mon, Oct 22, 2018 at 12:22:02AM -0400, Jintack Lim wrote:
> > > Hi,
> > >
> > > I wonder if vIOMMU is working for Windows VM?
> > >
> > > I tried it with v2.11.0, but it didn't
On 23/10/18 15:08, Richard Henderson wrote:
On 10/23/18 2:04 PM, Bastian Koppelmann wrote:
On 10/23/18 2:04 PM, Richard Henderson wrote:
??? Needs simultaneous corresponding changes to all
translators using decodetree.
---
scripts/decodetree.py | 5 ++---
1 file changed, 2 insertions(+), 3
On 23/10/18 14:04, Richard Henderson wrote:
Allow argument sets to be shared between two decoders by avoiding
a re-declaration error. Make sure that anonymous argument sets
have unique names.
Signed-off-by: Richard Henderson
---
scripts/decodetree.py | 34 +++---
On 10/23/18 2:04 PM, Richard Henderson wrote:
??? Should be merged back into previous patches.
??? Additional changes required for compact mode.
---
.../riscv/insn_trans/trans_privileged.inc.c | 19 ++-
target/riscv/insn_trans/trans_rva.inc.c | 44 +++---
target/riscv/insn_trans/t
1 - 100 of 239 matches
Mail list logo
