On Fri, 26 Oct 2018 03:53:11 PDT (-0700), kbast...@mail.uni-paderborn.de wrote:
On 10/25/18 12:21 AM, Palmer Dabbelt wrote:
On Sat, 20 Oct 2018 00:14:22 PDT (-0700),
kbast...@mail.uni-paderborn.de wrote:
Hi,
this patchset converts the RISC-V decoder to decodetree in three
major steps:
1) Con
[Expired for QEMU because there has been no activity for 60 days.]
** Changed in: qemu
Status: Incomplete => Expired
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1485180
Title:
Ctrl Alt G
[Expired for QEMU because there has been no activity for 60 days.]
** Changed in: qemu
Status: Incomplete => Expired
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1493033
Title:
memory leak
On Tue, Oct 23, 2018 at 08:02:47 +0100, Richard Henderson wrote:
> +static void tlb_flush_page_locked(CPUArchState *env, int midx,
> + target_ulong addr)
> +{
> +target_ulong lp_addr = env->tlb_d[midx].large_page_addr;
> +target_ulong lp_mask = env->tlb_d[mi
On Wed, Oct 17, 2018 at 12:10:15 +0200, Paolo Bonzini wrote:
> On 16/10/2018 13:10, guangrong.x...@gmail.com wrote:
> An idea: the total number of requests is going to be very small, and a
> PtrRing is not the nicest data structure for multiple producer/single
> consumer. So you could instead:
(s
On 25 October 2018 at 21:19, Aleksandar Markovic
wrote:
> From: Aleksandar Markovic
>
> The following changes since commit 808ebd66e467f77c0d1f8c6346235f81e9c99cf2:
>
> Merge remote-tracking branch 'remotes/riscv/tags/riscv-for-master-3.1-sf0'
> into staging (2018-10-25 17:41:03 +0100)
>
> are
In a conversation this week I mentioned the scripts I use for running
ad-hoc QEMU guest images that I have. The idea here is just that
by making sure that whenever I get a test image I set it up to be
run in the same way, I don't have the problem of trying to remember
how to run the guest if I come
Hi Aleksandar,
> This series renames MMI-related code elements so that they do not
> contain TX79 substring. Tx79 is one of CPUs that support MMI ASE.
> Opcodes and other code elements should be as generic as possible,
> and should not contain CPU name if they are supported by multiple
> CPUs. In
On 26 October 2018 at 20:43, P J P wrote:
> From: Prasad J Pandit
>
> While writing a message in 'lsi_do_msgin', message length value
> in 'msg_len' could be invalid. Add check to avoid OOB access issue.
>
> Signed-off-by: Prasad J Pandit
> ---
> hw/scsi/lsi53c895a.c | 15 ++-
> 1 f
From: Aleksandar Markovic
Rename MMI-related functions.
Signed-off-by: Aleksandar Markovic
---
target/mips/translate.c | 32
1 file changed, 16 insertions(+), 16 deletions(-)
diff --git a/target/mips/translate.c b/target/mips/translate.c
index 4b008d8..155331f
From: Aleksandar Markovic
This series renames MMI-related code elements so that they do not
contain TX79 substring. Tx79 is one of CPUs that support MMI ASE.
Opcodes and other code elements should be as generic as possible,
and should not contain CPU name if they are supported by multiple
CPUs. I
From: Aleksandar Markovic
Rename MMI-related masks.
Signed-off-by: Aleksandar Markovic
---
target/mips/translate.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/target/mips/translate.c b/target/mips/translate.c
index 51a5488..e38d50d 100644
--- a/ta
From: Aleksandar Markovic
Rename MMI-related opcodes.
Signed-off-by: Aleksandar Markovic
---
target/mips/translate.c | 470
1 file changed, 235 insertions(+), 235 deletions(-)
diff --git a/target/mips/translate.c b/target/mips/translate.c
index
+-- On Sun, 21 Oct 2018, P J P wrote --+
| The length parameter values are not negative, thus use an unsigned
| type 'size_t' for them. Many routines pass 'len' values to memcpy(3)
| calls. If it was negative, it could lead to memory corruption issues.
| Add check to avoid it.
|
| Reported-by: Ara
From: Prasad J Pandit
While writing a message in 'lsi_do_msgin', message length value
in 'msg_len' could be invalid. Add check to avoid OOB access issue.
Signed-off-by: Prasad J Pandit
---
hw/scsi/lsi53c895a.c | 15 ++-
1 file changed, 14 insertions(+), 1 deletion(-)
Update v1: ad
On 26 October 2018 at 14:26, Singh, Brijesh wrote:
>
>
> On 10/25/2018 07:59 PM, Michael S. Tsirkin wrote:
>> On Thu, Oct 25, 2018 at 08:16:44PM +0100, Peter Maydell wrote:
>>> These should presumably all be "ULL". (The "UL" suffix is
>>> usually a bug, as it's either unnecessary or should be ULL.
On 25 October 2018 at 14:32, Eduardo Habkost wrote:
> Changes v1 -> v2:
> * Fix 'make check' warnings (Igor)
>
>
> The following changes since commit 13399aad4fa87b2878c49d02a5d3bafa6c966ba3:
>
> Merge remote-tracking branch 'remotes/armbru/tags/pull-error-2018-10-22'
> into staging (2018-10-23
> From: Peter Maydell
> Subject: Re: [Qemu-devel] [PATCH 5/6] Determine the desired FPU mode
>
> On 26 October 2018 at 15:21, Stefan Markovic
> wrote:
> > From: Stefan Markovic
> >
> > Floating-point mode is calculated from MIPS.abiflags FP ABI value
> > (based on kernel implementation). Illeg
+-- On Fri, 26 Oct 2018, Mark Kanda wrote --+
| Yes, please use that acknowledgement text in lieu of a 'Reported-by' line.
Okay, thank you.
--
Prasad J Pandit / Red Hat Product Security Team
47AF CE69 3A90 54AA 9045 1053 DD13 3D32 FE5B 041F
+-- On Fri, 26 Oct 2018, Daniel P. Berrangé wrote --+
| ...
| One thing we should do, however, is to make it clear which of the
| device models we consider secure, and which we consider only usable
| in a friendly guest environment, as we have very different code
| maintainership & quality standar
* Jia Lina (jialin...@baidu.com) wrote:
> During an active background migration, snapshot will trigger a
> segmentfault. As snapshot clears the "current_migration" struct
> and updates "to_dst_file" before it finds out that there is a
> migration task, Migration accesses the null pointer in
> "curr
On 10/26/2018 1:37 PM, P J P wrote:
+-- On Fri, 26 Oct 2018, Mark Kanda wrote --+
| Deja vu requested that we include the following text in the commit message:
|
| Discovered by Deja vu Security. Reported by Oracle.
|
| Would that be acceptable?
Generally an email-id is used/preferred in t
+-- On Fri, 26 Oct 2018, Mark Kanda wrote --+
| Deja vu requested that we include the following text in the commit message:
|
| Discovered by Deja vu Security. Reported by Oracle.
|
| Would that be acceptable?
Generally an email-id is used/preferred in the commit log message. We could
use
While it would be possible to concatenate input files with make,
passing the original input files to decodetree.py allows us to
generate error messages which allows compilation environments
(read: emacs) to next-error to the correct input file.
Signed-off-by: Richard Henderson
---
Bastian, I thi
Hi,
I submitted the patch, but just found this has been already fixed by
Michael Clark
and pushed to riscv/riscv-qemu https://github.com/riscv/riscv-qemu/pull/166
but not in the upstream.
Do we still need this patch?
Thanks,
Dayeol
On Fri, Oct 26, 2018 at 11:04 AM Dayeol Lee wrote:
> pmp_rea
On 26 October 2018 at 15:21, Stefan Markovic wrote:
> From: Stefan Markovic
>
> Floating-point mode is calculated from MIPS.abiflags FP ABI value
> (based on kernel implementation). Illegal combinations are rejected.
>
> Signed-off-by: Stefan Markovic
> ---
> linux-user/mips/cpu_loop.c | 75
>
pmp_read_cfg() returns 8-bit value, which is combined together to form a single
pmpcfg CSR.
The default promotion rules will result in an integer here ("i*8" is integer,
which
flows through) resulting in a 32-bit signed value on most hosts.
That's bogus on RV64I, with the high bits of the CSR be
> Subject: [PATCH 0/6] target/mips: Add support for prctl() PR_GET_FP_MODE and
> PR_SET_FP_MODE
>
> From: Stefan Markovic
>
> This series includes support for prctl() PR_GET_FP_MODE and PR_SET_FP_MODE.
> This requires extracting MIPS.abiflags section from ELF file and fp_abi value
> handling.
On 10/26/18 2:33 PM, P J P wrote:
> From: Prasad J Pandit
>
> While performing PowerNV memory r/w operations, the access length
> 'sz' could exceed the data[4] buffer size. Add check to avoid OOB
> access.
>
> Reported-by: Moguofang
> Signed-off-by: Prasad J Pandit
Reviewed-by: Cédric Le Goat
> Subject: [PATCH 1/6] Define MIPS_ABI_FP_UNKNOWN macro
>
> From: Stefan Markovic
>
> Signed-off-by: Stefan Markovic
> ---
A brief commit message is needed. From what kernel or glibc header is this
constant copied? Other than that:
Reviewed-by: Aleksandar Markovic
> Subject: [PATCH 2/6] Extend image_info struct with MIPS specific fp_abi and
> interp_fp_abi fields
>
> From: Stefan Markovic
>
> Signed-off-by: Stefan Markovic
> ---
A brief commit message is needed. Perhaps with the description of the
role/purpose of two fields introduced in this patch. O
> Subject: [PATCH 3/6] Extract MIPS abiflags from ELF file
>
> From: Stefan Markovic
>
> Signed-off-by: Stefan Markovic
> ---
A brief commit message is needed. Other than that:
Reviewed-by: Aleksandar Markovic
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> This will simplify the following commit's diff.
>
> Signed-off-by: Emilio G. Cota
> ---
> cpus-common.c | 18 +-
> 1 file changed, 9 insertions(+), 9 deletions(-)
Reviewed-by: Richard Henderson
r~
On Fri, Oct 26, 2018 at 14:39:21 +0100, Alex Bennée wrote:
>
> Emilio G. Cota writes:
>
> > Cc: Peter Maydell
>
> This will need to catch-up in the next re-base as there is a merge conflict.
Yep, this series is so long that I decided to keep the same
baseline as in v3, so that I could just gi
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> Some async jobs do not need the BQL.
>
> Signed-off-by: Emilio G. Cota
> ---
> include/qom/cpu.h | 14 ++
> cpus-common.c | 39 ++-
> 2 files changed, 48 insertions(+), 5 deletions(-)
Reviewed-by: R
> Subject: [PATCH 4/6] Read and set FP ABI value from MIPS abiflags
>
> From: Stefan Markovic
>
> Signed-off-by: Stefan Markovic
> ---
A short commit message is needed. Other than that:
Reviewed-by: Aleksandar Markovic
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> Soon we will call cpu_has_work without the BQL.
>
> Cc: Max Filippov
> Signed-off-by: Emilio G. Cota
> ---
> target/xtensa/cpu.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
Reviewed-by: Richard Henderson
r~
> Subject: [PATCH 5/6] Determine the desired FPU mode
>
> From: Stefan Markovic
>
> Floating-point mode is calculated from MIPS.abiflags FP ABI value
> (based on kernel implementation). Illegal combinations are rejected.
>
> Signed-off-by: Stefan Markovic
> ---
Reviewed-by: Aleksandar Markovi
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> After completing the conversion to per-CPU locks, there is no need
> to release the BQL after having called cpu_kick.
>
> Signed-off-by: Emilio G. Cota
> ---
> cpus-common.c | 20 +---
> 1 file changed, 5 insertions(+), 15 deletions(-)
> From: Stefan Markovic
> Subject: [PATCH 6/6] Add prctl() PR_SET_FP_MODE and PR_GET_FP_MODE
> implementations
>
> From: Stefan Markovic
>
> Signed-off-by: Stefan Markovic
> #define TARGET_PR_SET_FP_MODE 45
> #define TARGET_PR_GET_FP_MODE 46
> +#define TARGET_PR_FP_MODE_FR (1 << 0)
> +#
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> Signed-off-by: Emilio G. Cota
> ---
> include/qom/cpu.h | 12 ++--
> 1 file changed, 10 insertions(+), 2 deletions(-)
Reviewed-by: Richard Henderson
r~
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> Soon we will call cpu_has_work without the BQL.
>
> Cc: Mark Cave-Ayland
> Cc: Artyom Tarasenko
> Signed-off-by: Emilio G. Cota
> ---
> target/sparc/cpu.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
Reviewed-by: Richard Henderson
r
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> Soon we will call cpu_has_work without the BQL.
>
> Cc: Michael Clark
> Cc: Palmer Dabbelt
> Cc: Sagar Karandikar
> Cc: Bastian Koppelmann
> Reviewed-by: Palmer Dabbelt
> Signed-off-by: Emilio G. Cota
> ---
> target/riscv/cpu.c | 5 -
> 1 fil
From: Li Qiang
Found by reading the code.
Signed-off-by: Li Qiang
Message-Id: <1536150548-2797-1-git-send-email-liq...@gmail.com>
Signed-off-by: Laurent Vivier
---
include/qom/cpu.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/qom/cpu.h b/include/qom/cpu.h
index
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> Soon we will call cpu_has_work without the BQL.
>
> Cc: Aurelien Jarno
> Cc: Aleksandar Markovic
> Signed-off-by: Emilio G. Cota
> ---
> target/mips/cpu.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
Reviewed-by: Richard Henderson
r
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> Soon we will call cpu_has_work without the BQL.
>
> Cc: Cornelia Huck
> Cc: Alexander Graf
> Cc: David Hildenbrand
> Cc: qemu-s3...@nongnu.org
> Signed-off-by: Emilio G. Cota
> ---
> target/s390x/cpu.c | 4 +++-
> 1 file changed, 3 insertions(+), 1
From: Cleber Rosa
Signed-off-by: Cleber Rosa
Reviewed-by: Philippe Mathieu-Daudé
Message-Id: <20181004161852.11673-8-cr...@redhat.com>
Signed-off-by: Laurent Vivier
---
scripts/decodetree.py | 2 --
1 file changed, 2 deletions(-)
diff --git a/scripts/decodetree.py b/scripts/decodetree.py
ind
From: Cleber Rosa
Commit cce293a2945 moved some functions from common.config to
common.rc, but the error messages still reference the old file
location.
Signed-off-by: Cleber Rosa
Reviewed-by: Max Reitz
Reviewed-by: Philippe Mathieu-Daudé
Message-Id: <20181004161852.11673-5-cr...@redhat.com>
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> Soon we will call cpu_has_work without the BQL.
>
> Cc: David Gibson
> Cc: Alexander Graf
> Cc: qemu-...@nongnu.org
> Signed-off-by: Emilio G. Cota
> ---
> target/ppc/translate_init.inc.c | 16
> 1 file changed, 12 insertions(+), 4
From: Cleber Rosa
The line immediate following a ".. code::" block is considered
to contains arguments to the "code directive". The lack of a
new line gives me during at parse time:
testing.rst:63: (ERROR/3) Error in "code" directive:
maximum 1 argument(s) allowed, 3 supplied.
.. code
From: Philippe Mathieu-Daudé
qemu_log_mask(GUEST_ERROR) is more appropriate:
$ qemu -d help
Log items (comma separated):
guest_errorslog when the guest OS does something invalid (eg accessing a
non-existent register)
Signed-off-by: Philippe Mathieu-Daudé
Acked-by: Michael Walle
Mes
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> It will gain some users soon.
>
> Suggested-by: Paolo Bonzini
> Signed-off-by: Emilio G. Cota
> ---
> include/qom/cpu.h | 36 +---
> 1 file changed, 33 insertions(+), 3 deletions(-)
Reviewed-by: Richard Henderson
r~
The following changes since commit 808ebd66e467f77c0d1f8c6346235f81e9c99cf2:
Merge remote-tracking branch 'remotes/riscv/tags/riscv-for-master-3.1-sf0'
into staging (2018-10-25 17:41:03 +0100)
are available in the Git repository at:
git://github.com/vivier/qemu.git tags/trivial-patches-pull
From: Paolo Bonzini
AT24c EEPROM is currently gated by CONFIG_I2C, and as such it is
being included in all emulators that use I2C, even if they do not
really need it. Separate it and, since it was added for the e500
machines, add it to qemu-system-ppc and qemu-system-ppc64.
Signed-off-by: Paolo
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> Signed-off-by: Emilio G. Cota
> ---
> accel/tcg/cpu-exec.c | 15 ---
> accel/tcg/tcg-all.c | 12 +---
> accel/tcg/translate-all.c | 2 +-
> 3 files changed, 18 insertions(+), 11 deletions(-)
Reviewed-by: Richard Henders
From: Cleber Rosa
Point to the right and obvious location for lm32 tests.
Signed-off-by: Cleber Rosa
Reviewed-by: Thomas Huth
Acked-by: Alex Bennée
Message-Id: <20181004161852.11673-3-cr...@redhat.com>
Signed-off-by: Laurent Vivier
---
tests/tcg/README | 2 +-
1 file changed, 1 insertion(+)
From: Li Qiang
Signed-off-by: Li Qiang
Reviewed-by: Peter Maydell
Message-Id: <1539080467-2976-1-git-send-email-liq...@gmail.com>
[lv: s/types/typos/]
Signed-off-by: Laurent Vivier
---
include/exec/memory.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/include/exec
From: Cleber Rosa
Signed-off-by: Cleber Rosa
Reviewed-by: Philippe Mathieu-Daudé
Message-Id: <20181004161852.11673-11-cr...@redhat.com>
Signed-off-by: Laurent Vivier
---
scripts/qemu.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/scripts/qemu.py b/scripts/qemu.py
index
From: Philippe Mathieu-Daudé
Patch created mechanically by rerunning:
$ spatch --sp-file scripts/coccinelle/qobject.cocci \
--macro-file scripts/cocci-macro-file.h \
--dir . --in-place
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Markus Armbruster
Acked-by: Mi
From: Thomas Huth
We don't use CONFIG_PARALLEL_ISA in any of our Makefiles, so this
is just a dead config option which can be removed.
Fixes: a4cb773928e047b137c6998209cf2eec857fac6b
Signed-off-by: Thomas Huth
Acked-by: Richard Henderson
Reviewed-by: Philippe Mathieu-Daudé
Message-Id: <154037
From: "Emilio G. Cota"
Signed-off-by: Emilio G. Cota
Message-Id: <20180811211011.6277-1-c...@braap.org>
Signed-off-by: Laurent Vivier
---
linux-user/qemu.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/linux-user/qemu.h b/linux-user/qemu.h
index 1beb6a2cfc..dde3f26f5a 100
From: Cleber Rosa
Commit 990dc39c made all tests executable at the time, but 218 came in
later, and missing those permissions.
Signed-off-by: Cleber Rosa
Message-Id: <20181004161852.11673-4-cr...@redhat.com>
Signed-off-by: Laurent Vivier
---
tests/qemu-iotests/218 | 0
1 file changed, 0 inser
From: Philippe Mathieu-Daudé
TYPE_XEN_PT_DEVICE is a subclass of TYPE_PCI_DEVICE, the clean way
to access the PCIDevice pointer is using the PCI_DEVICE() macro.
Suggested-by: Peter Maydell
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Markus Armbruster
Acked-by: Anthony PERARD
Acked-by:
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> Cc: Aurelien Jarno
> Cc: Aleksandar Markovic
> Cc: James Hogan
> Signed-off-by: Emilio G. Cota
> ---
> target/mips/cpu.c | 7 ---
> target/mips/kvm.c | 2 +-
> 2 files changed, 5 insertions(+), 4 deletions(-)
Reviewed-by: Richard Henderson
r~
From: Philippe Mathieu-Daudé
Patch created mechanically by rerunning:
$ spatch --sp-file scripts/coccinelle/round.cocci \
--macro-file scripts/cocci-macro-file.h \
--dir . --in-place
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Markus Armbruster
Acked-by: Mich
From: Stefan Weil
This is needed for builds with the mingw64-* packages from Cygwin,
but also works for Linux.
Move the zlib test also more to the end because users should
get information on the really important missing packages
(which also require zlib) first.
Signed-off-by: Stefan Weil
Revie
From: Philippe Mathieu-Daudé
Patch created mechanically by rerunning:
$ spatch --sp-file scripts/coccinelle/round.cocci \
--macro-file scripts/cocci-macro-file.h \
--dir . --in-place
Signed-off-by: Philippe Mathieu-Daudé
Acked-by: Michael S. Tsirkin
Message-Id: <201
From: yuchenlin
Signed-off-by: yuchenlin
Reviewed-by: Philippe Mathieu-Daudé
Message-Id: <20181022080053.9379-1-yuchen...@synology.com>
Signed-off-by: Laurent Vivier
---
hw/display/vga_int.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/hw/display/vga_int.h b/hw/display/vga_int.h
index 6
From: Philippe Mathieu-Daudé
Patch created mechanically by rerunning:
$ spatch --sp-file scripts/coccinelle/typecast.cocci \
--macro-file scripts/cocci-macro-file.h \
--dir . --in-place
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Markus Armbruster
Acked-by: M
On Wed, Oct 24, 2018 at 10:14:56PM +0800, Zhang Yi wrote:
> We will never get the canonical path from the object
> before object_property_add_child.
>
> Signed-off-by: Zhang Yi
Thanks. I'm queueing it on machine-next, after rewriting the
subject line as:
hostmem-file: remove object id from p
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> Cc: David Gibson
> Cc: Alexander Graf
> Cc: qemu-...@nongnu.org
> Signed-off-by: Emilio G. Cota
> ---
> hw/ppc/ppc.c| 2 +-
> target/ppc/excp_helper.c| 2 +-
> target/ppc/kvm.c| 4 ++--
> target/ppc/tran
From: Stefan Weil
Fix also a grammar issue.
Signed-off-by: Stefan Weil
Reviewed-by: Alex Bennée
Message-Id: <20180713054755.23323-1...@weilnetz.de>
Signed-off-by: Laurent Vivier
---
tests/bios-tables-test.c | 2 +-
tests/docker/Makefile.include | 2 +-
tests/docker/docke
On Fri, 26 Oct 2018 07:53:17 PDT (-0700), Bastian Koppelmann wrote:
On 10/26/18 3:58 PM, Richard Henderson wrote:
On 10/26/18 11:49 AM, Bastian Koppelmann wrote:
I think you can pick up everything up to the RVC conversion which still needs
the work suggested by Richard. Thanks, for picking it
On Thu, Oct 04, 2018 at 12:18:42PM -0400, Cleber Rosa wrote:
> Just a collection of trivial fixes and clean ups that have been lying
> around here for some time.
I'm queueing patches 07-10 on python-next. I'm aware that
Laurent already queued some of them (thanks!), but I want to
avoid conflicts
* Peter Xu (pet...@redhat.com) wrote:
> On Fri, Oct 26, 2018 at 09:10:19PM +0800, Fei Li wrote:
> >
> >
> > On 10/25/2018 08:58 PM, Peter Xu wrote:
> > > On Thu, Oct 25, 2018 at 05:04:00PM +0800, Fei Li wrote:
> > >
> > > [...]
> > >
> > > > @@ -1325,22 +1325,24 @@ bool multifd_recv_all_channel
On Mon, Oct 08, 2018 at 03:44:14PM -0400, John Snow wrote:
>
>
> On 10/04/2018 12:18 PM, Cleber Rosa wrote:
> > Signed-off-by: Cleber Rosa
> > ---
> > dtc | 2 +-
> > scripts/qemu.py | 65 +++--
> > 2 files changed, 42 insertions(+), 25 d
On Thu, Oct 18, 2018 at 11:31:31AM -0400, Cleber Rosa wrote:
> TL;DR
> =
>
> Allow acceptance tests to be run with `make check-acceptance`.
Queued on python-next, thanks!
--
Eduardo
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> Signed-off-by: Emilio G. Cota
> ---
> target/i386/hvf/hvf.c| 8 +---
> target/i386/hvf/x86hvf.c | 26 +++---
> 2 files changed, 20 insertions(+), 14 deletions(-)
Reviewed-by: Richard Henderson
r~
On Mon, Oct 22, 2018 at 03:33:30PM +0100, Igor Mammedov wrote:
> On Wed, 12 Sep 2018 16:55:23 +0400
> Marc-André Lureau wrote:
>
> > Improve a bit code readability.
> >
> > Signed-off-by: Marc-André Lureau
> > ---
> > include/qom/object_interfaces.h | 4
> > qom/object.c
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> Signed-off-by: Emilio G. Cota
> ---
> target/i386/whpx-all.c | 41 -
> 1 file changed, 24 insertions(+), 17 deletions(-)
Reviewed-by: Richard Henderson
r~
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> Signed-off-by: Emilio G. Cota
> ---
> target/i386/hax-all.c | 30 +-
> 1 file changed, 17 insertions(+), 13 deletions(-)
Reviewed-by: Richard Henderson
r~
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> Signed-off-by: Emilio G. Cota
> ---
> target/i386/kvm.c | 54 +++
> 1 file changed, 31 insertions(+), 23 deletions(-)
Reviewed-by: Richard Henderson
r~
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> Signed-off-by: Emilio G. Cota
> ---
> exec.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Richard Henderson
r~
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> Signed-off-by: Emilio G. Cota
> ---
> target/i386/cpu.c| 2 +-
> target/i386/helper.c | 4 ++--
> target/i386/svm_helper.c | 4 ++--
> 3 files changed, 5 insertions(+), 5 deletions(-)
Reviewed-by: Richard Henderson
r~
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> Add a comment about how atomic_read works here. The comment refers to
> a "BQL-less CPU loop", which will materialize toward the end
> of this series.
>
> Note that the modifications to cpu_reset_interrupt are there to
> avoid deadlock during the CPU lo
On 10/25/18 3:45 PM, Emilio G. Cota wrote:
> Cc: Michael Clark
> Cc: Palmer Dabbelt
> Cc: Sagar Karandikar
> Cc: Bastian Koppelmann
> Cc: Alistair Francis
> Reviewed-by: Palmer Dabbelt
> Signed-off-by: Emilio G. Cota
> ---
> target/riscv/op_helper.c | 2 +-
> 1 file changed, 1 insertion(+),
On 10/25/18 3:46 PM, Emilio G. Cota wrote:
> Signed-off-by: Emilio G. Cota
> ---
> accel/tcg/cpu-exec.c | 24
> 1 file changed, 20 insertions(+), 4 deletions(-)
Reviewed-by: Richard Henderson
r~
On 10/25/18 3:45 PM, Emilio G. Cota wrote:
> Cc: "Edgar E. Iglesias"
> Signed-off-by: Emilio G. Cota
> ---
> target/microblaze/translate.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
Reviewed-by: Richard Henderson
r~
On 10/25/18 3:45 PM, Emilio G. Cota wrote:
> Signed-off-by: Emilio G. Cota
> ---
> accel/tcg/tcg-runtime.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Richard Henderson
r~
On 10/25/18 3:45 PM, Emilio G. Cota wrote:
> In ppce500_spin.c, acquire the lock just once to update
> both cpu->halted and cpu->stopped.
>
> In hw/ppc/spapr_hcall.c, acquire the lock just once to
> update cpu->halted and call cpu_has_work, since later
> in the series we'll acquire the BQL (if not
On 10/25/18 3:45 PM, Emilio G. Cota wrote:
> Cc: Laurent Vivier
> Signed-off-by: Emilio G. Cota
> ---
> target/m68k/translate.c | 9 -
> 1 file changed, 4 insertions(+), 5 deletions(-)
Reviewed-by: Richard Henderson
r~
On 10/25/18 3:45 PM, Emilio G. Cota wrote:
> Signed-off-by: Emilio G. Cota
> ---
> target/alpha/translate.c | 6 ++
> 1 file changed, 2 insertions(+), 4 deletions(-)
Reviewed-by: Richard Henderson
r~
On 10/25/18 3:45 PM, Emilio G. Cota wrote:
> Signed-off-by: Emilio G. Cota
> ---
> target/hppa/translate.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
Reviewed-by: Richard Henderson
r~
On 10/25/18 3:45 PM, Emilio G. Cota wrote:
> Cc: David Gibson
> Cc: Alexander Graf
> Cc: qemu-...@nongnu.org
> Signed-off-by: Emilio G. Cota
> ---
> target/ppc/translate.c | 6 ++
> 1 file changed, 2 insertions(+), 4 deletions(-)
Reviewed-by: Richard Henderson
r~
On 10/25/18 3:45 PM, Emilio G. Cota wrote:
> And fix the temp leak along the way.
>
> Cc: "Edgar E. Iglesias"
> Signed-off-by: Emilio G. Cota
> ---
> target/cris/translate.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
Reviewed-by: Richard Henderson
r~
On 10/25/18 3:45 PM, Emilio G. Cota wrote:
> +void HELPER(cpu_halted_set)(CPUArchState *env, uint32_t val)
> +{
> +CPUState *cpu = ENV_GET_CPU(env);
> +
> +cpu->halted = val;
> +}
I suppose this is fine as a first step. I'd like to clean up all of the
targets to also use a helper like thi
On 10/26/18 3:58 PM, Richard Henderson wrote:
On 10/26/18 11:49 AM, Bastian Koppelmann wrote:
I think you can pick up everything up to the RVC conversion which still needs
the work suggested by Richard. Thanks, for picking it up :)
Even then I thought we were talking about splitting the RV64
On 10/25/18 3:45 PM, Emilio G. Cota wrote:
> This eliminates the need to use the BQL to queue CPU work.
>
> While at it, give the per-cpu field a generic name ("cond") since
> it will soon be used for more than just queueing CPU work.
>
> Signed-off-by: Emilio G. Cota
> ---
> include/qom/cpu.h
On 10/25/18 3:45 PM, Emilio G. Cota wrote:
> The few direct users of &cpu->lock will be converted soon.
>
> Signed-off-by: Emilio G. Cota
> ---
> include/qom/cpu.h | 33 +++
> cpus.c | 48 +++--
> stubs/cpu-lock.c
From: Stefan Markovic
Signed-off-by: Stefan Markovic
---
include/elf.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/include/elf.h b/include/elf.h
index 5f45f9b..c151164 100644
--- a/include/elf.h
+++ b/include/elf.h
@@ -87,6 +87,8 @@ typedef int64_t Elf64_Sxword;
#define EF_MIPS_MACH
1 - 100 of 203 matches
Mail list logo