Directly reading sc->regs.psw.addr misses the bswap
that may be performed by __get_user.
Signed-off-by: Richard Henderson
---
linux-user/s390x/signal.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/linux-user/s390x/signal.c b/linux-user/s390x/signal.c
index e455a9818d.
On 4/28/21 12:33 PM, Richard Henderson wrote:
Version 2 splits lazy do-it-all patch.
Patch 1 has an additional fix, so I dropped the r-b.
... and I realized as I hit send that this depends on the
target_restore_altstack cleanup that's part of
https://patchew.org/QEMU/20210426025334.1168495-1
On 4/22/21 7:02 PM, Kevin Wolf wrote:
> This is a partial revert of commits 77542d43149 and bc79c87bcde.
>
> Usually, an error during initialisation means that the configuration was
> wrong. Reconnecting won't make the error go away, but just turn the
> error condition into an endless loop. Avoid
The "save" routines copied from the kernel, which are currently
commented out, are unnecessary in qemu. We can copy from env
where the kernel needs special instructions. Fix comment style.
Signed-off-by: Richard Henderson
---
linux-user/s390x/signal.c | 10 ++
1 file changed, 6 inserti
Signed-off-by: Richard Henderson
---
linux-user/s390x/signal.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/linux-user/s390x/signal.c b/linux-user/s390x/signal.c
index f8515dd332..4dde55d4d5 100644
--- a/linux-user/s390x/signal.c
+++ b/linux-user/s390x/signal.c
@@ -182
On Wed, 28 Apr 2021 20:17:23 +0100
"Dr. David Alan Gilbert" wrote:
> * Dev Audsin (dev.devaq...@gmail.com) wrote:
> > Thanks Dave for your explanation.
> > Any suggestions on how to make VFIO not attempt to map into the
> > unaccessible and unallocated RAM.
>
> I'm not sure;:
>
> static bool
Note that PSW_ADDR_{64,32} are called PSW_MASK_{EA,BA}
in the kernel source.
Signed-off-by: Richard Henderson
---
linux-user/s390x/signal.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/linux-user/s390x/signal.c b/linux-user/s390x/signal.c
index 64a9eab097..17f617c655 100644
--- a/li
Signed-off-by: Richard Henderson
---
linux-user/s390x/signal.c | 29 -
1 file changed, 8 insertions(+), 21 deletions(-)
diff --git a/linux-user/s390x/signal.c b/linux-user/s390x/signal.c
index eabfe4293f..64a9eab097 100644
--- a/linux-user/s390x/signal.c
+++ b/linux-u
28.04.2021 20:03, Kevin Wolf wrote:
Am 28.04.2021 um 17:17 hat Vladimir Sementsov-Ogievskiy geschrieben:
Hi all!
And here is v4.
Thanks, applied to the block branch.
Thanks! And thanks a lot for reviewing!
Though the error message shown by the test in patch 18 does need some
improvement
Bruno Piazera Larsen writes:
>> > > +/*/
>> > > +/* SPR definitions and registration */
>> > > +
>> > > +#ifdef CONFIG_TCG
>> > > +#ifdef CONFIG_USER_ONLY
>> > > +#define spr_register_kvm(env, num, name, uea_read, uea_wri
In order to properly present these arguments, we need to add
code to target/s390x to record LowCore parameters for user-only.
But in the meantime, at least zero the missing last_break
argument, and fixup the comment style in the vicinity.
Signed-off-by: Richard Henderson
---
linux-user/s390x/si
Signed-off-by: Richard Henderson
---
linux-user/s390x/signal.c | 62 +--
1 file changed, 60 insertions(+), 2 deletions(-)
diff --git a/linux-user/s390x/signal.c b/linux-user/s390x/signal.c
index 9d470e4ca0..b537646e60 100644
--- a/linux-user/s390x/signal.c
+++
At point of usage, it's not immediately obvious that
we don't need a loop to copy these arrays.
Signed-off-by: Richard Henderson
---
linux-user/s390x/signal.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/linux-user/s390x/signal.c b/linux-user/s390x/signal.c
index 81ba59b46a..839a7ae4
On 4/28/21 9:24 PM, Richard Henderson wrote:
> On 4/28/21 10:56 AM, Philippe Mathieu-Daudé wrote:
>> Are you suggesting to remove the 'if have_block' check? This makes build
>> a the files pointlessly for user-mode-only builds...
>
> But since the objects are not included in the binary, I don't ca
Meson defaults builds to 'debugoptimized' which adds '-g -O2'
to CFLAGS. If the user specifies '--disable-debug-info' we
should instead build with 'release' which does not emit any
debug info.
Signed-off-by: Joelle van Dyne
---
configure | 1 +
1 file changed, 1 insertion(+)
diff --git a/config
I think you must use a more recent kernel because some bugs have been
fixed in QEMU and kernel that need both of them in sync.
Could you extract the kernel from your m68k disk image to use it with
QEMU "-kernel" and "-initrd" parameters?
--
You received this bug notification because you are a me
On 4/28/21 10:03 AM, Philippe Mathieu-Daudé wrote:
Rename set_pc() as mips_env_set_pc(), declare it inlined
and use it in cpu.c and op_helper.c.
Reported-by: Richard Henderson
Signed-off-by: Philippe Mathieu-Daudé
---
v4: mips_cpu_set_error_pc -> mips_env_set_pc (rth)
---
target/mips/internal.
Reviewed-by: Raphael Norwitz
On Thu, Apr 22, 2021 at 07:02:21PM +0200, Kevin Wolf wrote:
> Creating a device with a number of queues that isn't supported by the
> backend is pointless, the device won't work properly and the error
> messages are rather confusing.
>
> Just fail to create the devic
The kernel in my m68k disk image is vmlinux-4.16.0-1-m68k which is
presumably what comes from
https://cdimage.debian.org/cdimage/ports/10.0/m68k/iso-cd/debian-10.0
-m68k-NETINST-1.iso. Is there a debian image that uses a newer kernel?
--
You received this bug notification because you are a member
On 4/8/21 6:07 PM, Taylor Simpson wrote:
This patch series is a significant update for the Hexagon target
The first 16 patches address feedback from Richard Henderson
and Philippe Mathieu-Daud�
The next 10 patches add the remaining instructions for the Hexagon
scalar core
T
On Tue, Apr 27, 2021 at 04:09:48PM +0800, Like Xu wrote:
> The last branch recording (LBR) is a performance monitor unit (PMU)
> feature on Intel processors that records a running trace of the most
> recent branches taken by the processor in the LBR stack. The QEMU
> could configure whether it's en
It looks like using
https://cdimage.debian.org/cdimage/ports/snapshots/2021-04-17/debian-10.0.0
-m68k-NETINST-1.iso instead fixes the issue. Perhaps the instruction on
https://wiki.qemu.org/Documentation/Platforms/m68k should be updated.
--
You received this bug notification because you are a mem
Viresh,
For rust-vmm, you need to create the new issue in the right project. You can
probably pick up vmm-reference project at rust-vmm and ask for the new crate.
You can also send email to rust-vmm mailing list but github "issues" feature is
used heavily in the rust-vmm project. There is also
Hi, I didn't realize that you had already CCed the rust-vmm mailing list, so
anyways it is a good start.
-Original Message-
From: Trilok Soni
Sent: Wednesday, April 28, 2021 10:14 AM
To: Viresh Kumar ; stratos-...@op-lists.linaro.org;
rust-...@lists.opendev.org
Cc: Vincent Guittot ; Mi
Public bug reported:
There is comment int the code "This is a hint, so ignoring and returning
success is ok"
https://github.com/qemu/qemu/blob/b1cffefa1b163bce9aebc3416f562c1d3886eeaa/linux-user/syscall.c#L11941
But it seems incorrect with the current state of Linux
"man madvise" or https://man
> -Original Message-
> From: Richard Henderson
> Sent: Wednesday, April 28, 2021 4:13 PM
> To: Taylor Simpson ; qemu-devel@nongnu.org
> Cc: phi...@redhat.com; a...@rev.ng; Brian Cain
> Subject: Re: [PATCH v4 00/26] Hexagon (target/hexagon) update
>
> On 4/8/21 6:07 PM, Taylor Simpson w
On 4/28/21 4:20 PM, Taylor Simpson wrote:
I get -Wno-unused-function added to the compiler command line, so I don't see
the error.
Ah, looks like it's the version of glib on your system. The flag gets added in
configure beneath:
# Silence clang warnings triggered by glib < 2.57.2
Both w
On Wed, 28 Apr 2021, Dr. David Alan Gilbert wrote:
* BALATON Zoltan (bala...@eik.bme.hu) wrote:
On Wed, 28 Apr 2021, Andrew Randrianasulu wrote:
On Wednesday, April 28, 2021, Andrew Randrianasulu
wrote:
On Monday, April 26, 2021, BALATON Zoltan wrote:
On Mon, 26 Apr 2021, Dr. David Alan Gil
On Wed, 2021-04-28 at 11:24 -0400, Eduardo Habkost wrote:
> On Wed, Apr 28, 2021 at 10:41:13AM +0800, Robert Hoo wrote:
> > As it's been marked deprecated since v5.2, now I think it's time
> > remove it
> > from code.
> >
> > Signed-off-by: Robert Hoo
> > ---
> > (Sorry, forgot to append changelo
On 2021/4/28 18:23, Andrew Jones wrote:
On Tue, Apr 13, 2021 at 04:31:44PM +0800, Yanan Wang wrote:
A cluster means a group of cores that share some resources (e.g. cache)
among them under the LLC. For example, ARM64 server chip Kunpeng 920 has
6 or 8 clusters in each NUMA, and each cluster ha
As it's been marked deprecated since v5.2, now I think it's time remove it
from code.
Signed-off-by: Robert Hoo
---
Changelog:
v3:
Update deprecated.rst. (Sorry for my carelessness in last search. I
sware I did search.)
v2:
Update removed-features.rst.
---
docs/system/deprecated.rst
Hi Drew,
On 2021/4/28 18:31, Andrew Jones wrote:
On Tue, Apr 13, 2021 at 04:31:45PM +0800, Yanan Wang wrote:
There is a separate function virt_smp_parse() in hw/virt/arm.c used
to parse cpu topology for the ARM machines. So add parsing of -smp
cluster parameter in it, then total number of logic
Hi Drew,
On 2021/4/28 18:13, Andrew Jones wrote:
On Wed, Apr 28, 2021 at 05:36:43PM +0800, wangyanan (Y) wrote:
On 2021/4/27 22:58, Andrew Jones wrote:
On Tue, Apr 13, 2021 at 04:07:45PM +0800, Yanan Wang wrote:
From: Andrew Jones
The virt machine type has never used the CPU topology parame
Without this, libvixl cannot be compiled with macOS 11.3 SDK due to
include file name conflict (usr/include/c++/v1/version conflicts with
VERSION).
Signed-off-by: Katsuhiro Ueno
---
meson.build | 1 +
1 file changed, 1 insertion(+)
diff --git a/meson.build b/meson.build
index c6f4b0cf5e..d007bf
For some hexagon use cases, we would prefer to have finer grained scheduling
among multiple guest cores/threads. We haven't been able to determine exactly
what kind of scheduling algorithm is operating in the baseline case. If the
current hw thread is ready-to-run and is spinning over a tight
On Tue, Apr 27, 2021 at 3:12 PM Jason Wang wrote:
>
>
> 在 2021/4/27 上午11:39, Cindy Lu 写道:
> > Add support for configure interrupt, use kvm_irqfd_assign and set the
> > gsi to kernel. When the configure notifier was eventfd_signal by host
> > kernel, this will finally inject an msix interrupt to gu
On Tue, Apr 27, 2021 at 1:39 PM Jason Wang wrote:
>
>
> 在 2021/4/27 上午11:39, Cindy Lu 写道:
> > In order to support configure interrupt. we will use queue number -1
> > as configure interrupt
> > since all these device are not support the configure interrupt
> > So we will add an check here, if the
> -Original Message-
> From: Brian Cain
> Sent: Wednesday, April 28, 2021 10:06 PM
> To: qemu-devel@nongnu.org
> Cc: Richard Henderson ; Taylor Simpson
> ; Michael Lambert ;
> Manning, Sid
> Subject: sysemu SMP scheduling
>
> For some hexagon use cases, we would prefer to have finer grain
Remove gen_read_reg and gen_set_byte
Reported-by: Richard Henderson
Signed-off-by: Taylor Simpson
---
target/hexagon/genptr.c | 11 ---
1 file changed, 11 deletions(-)
diff --git a/target/hexagon/genptr.c b/target/hexagon/genptr.c
index 55c7cd8..f93f895 100644
--- a/target/hexagon/genp
Ping.
On Wed, Apr 14, 2021 at 8:39 PM Doug Evans wrote:
> This patchset takes the original patch from Maxim,
> https://www.mail-archive.com/qemu-devel@nongnu.org/msg569573.html
> and updates it.
>
> Option hostfwd is extended to support ipv6 addresses.
> Commands hostfwd_add, hostfwd_remove are
The nvdimm devices are expected to ensure write persistence during power
failure kind of scenarios.
The libpmem has architecture specific instructions like dcbf on POWER
to flush the cache data to backend nvdimm device during normal writes
followed by explicit flushes if the backend devices are no
The subsequent patches add definitions which tend to
get the compilation to cyclic dependency. So, prepare
with forward declarations, move the defitions and clean up.
Signed-off-by: Shivaprasad G Bhat
---
hw/ppc/spapr_nvdimm.c | 12
include/hw/ppc/spapr_nvdimm.h | 14 ++-
The patch adds support for the SCM flush hcall for the nvdimm devices.
To be available for exploitation by guest through the next patch.
The hcall expects the semantics such that the flush to return
with H_BUSY when the operation is expected to take longer time along
with a continue_token. The hca
The patch adds the 'sync-dax' property to the nvdimm device.
When the sync-dax is 'direct' indicates the backend is synchronous DAX
capable and no explicit flush requests are required. When the mode is
set to 'writeback' it indicates the backend is not synhronous DAX
capable and explicit flushes t
On Wed, Apr 28, 2021 at 02:47:37PM +, Bruno Piazera Larsen wrote:
> > > > This move is required to enable building without TCG.
> > > > All the logic related to registering SPRs specific to
> > > > some architectures or machines has been hidden in this
> > > > new file.
> > >
> > > Hm... I thou
Hi Taylor,
On 4/29/21 5:32 AM, Taylor Simpson wrote:
> Remove gen_read_reg and gen_set_byte
>
> Reported-by: Richard Henderson
> Signed-off-by: Taylor Simpson
> ---
To help git-tools (and reviewers), please use the 'Based-on' tag
the next time you send a patch depending on another one:
Based-o
On 4/28/21 9:55 PM, Joelle van Dyne wrote:
> Meson defaults builds to 'debugoptimized' which adds '-g -O2'
> to CFLAGS. If the user specifies '--disable-debug-info' we
> should instead build with 'release' which does not emit any
> debug info.
>
> Signed-off-by: Joelle van Dyne
> ---
> configure
On Thu, Apr 29, 2021 at 6:49 AM Philippe Mathieu-Daudé wrote:
>
> Hi Taylor,
>
> On 4/29/21 5:32 AM, Taylor Simpson wrote:
> > Remove gen_read_reg and gen_set_byte
> >
> > Reported-by: Richard Henderson
> > Signed-off-by: Taylor Simpson
> > ---
>
> To help git-tools (and reviewers), please use t
> -Original Message-
> From: Philippe Mathieu-Daudé On
> Behalf Of Philippe Mathieu-Daudé
> Sent: Wednesday, April 28, 2021 11:49 PM
> To: Taylor Simpson ; qemu-devel@nongnu.org
> Cc: richard.hender...@linaro.org; a...@rev.ng; Brian Cain
>
> Subject: Re: [PATCH] Hexagon (target/hexagon)
Philippe Mathieu-Daudé writes:
> Now than we can probe if the TCG accelerator is available
> at runtime with a QMP command, do it once at the beginning
> and only register the tests we can run.
> We can then replace the #ifdef'ry by a runtime check.
>
> Suggested-by: Paolo Bonzini
> Signed-off-b
Stefan Hajnoczi writes:
> On Wed, Apr 28, 2021 at 04:18:17PM +0200, Markus Armbruster wrote:
>> Stefan Hajnoczi writes:
[...]
>> > The approach in this patch is okay but we should keep in mind it only
>> > solves piix3-ide. ISA provides a non-qdev backdoor API and there may be
>> > more instan
On Wed, Apr 28, 2021 at 11:31 PM Jeff <1926...@bugs.launchpad.net> wrote:
>
> It looks like using
> https://cdimage.debian.org/cdimage/ports/snapshots/2021-04-17/debian-10.0.0
> -m68k-NETINST-1.iso instead fixes the issue. Perhaps the instruction on
> https://wiki.qemu.org/Documentation/Platforms/m
301 - 352 of 352 matches
Mail list logo
