On Wed, Aug 4, 2021 at 3:43 PM Thomas Huth wrote:
>
> On 04/08/2021 08.51, Qiang Liu wrote:
> > xlnx_dp_read allows an out-of-bounds read at its default branch because
> > of an improper index.
> >
> > According to
> > https://www.xilinx.com/html_docs/registers/ug1087/ug1087-zynq-ultrascale-regist
On 06/08/2021 09.00, Qiang Liu wrote:
On Wed, Aug 4, 2021 at 3:43 PM Thomas Huth wrote:
On 04/08/2021 08.51, Qiang Liu wrote:
xlnx_dp_read allows an out-of-bounds read at its default branch because
of an improper index.
According to
https://www.xilinx.com/html_docs/registers/ug1087/ug1087-zy
On 06/08/2021 08.52, David Hildenbrand wrote:
According to the PoP:
"When the enhanced-DAT facility 1 is not installed, or
when the facility is installed but the multiple-block
control is zero, general register R 2 contains a real
address. When the enhanced-DAT facility 1 is
install
On 06.08.21 09:11, Thomas Huth wrote:
On 06/08/2021 08.52, David Hildenbrand wrote:
According to the PoP:
"When the enhanced-DAT facility 1 is not installed, or
when the facility is installed but the multiple-block
control is zero, general register R 2 contains a real
address. When
Hi
On Fri, Aug 6, 2021 at 2:28 AM Eugene Huang wrote:
> This patch enables running generic EGL devices such as Nvidia’s in
> headless mode. It assumes single device. More work is needed to support
> multiple devices.
>
>
>
> Signed-off-by: Eugene Huang
>
Thanks for the patch. It isn't correctl
On 06/08/2021 08:25, Jason Wang wrote:
>
> 在 2021/8/2 下午4:42, Laurent Vivier 写道:
>> On 02/08/2021 06:50, Jason Wang wrote:
>>> 在 2021/7/30 上午3:19, Laurent Vivier 写道:
Add virtio_queue_disable()/virtio_queue_enable() to disable/enable a queue
by setting vring.num to 0 (or num_default).
>>>
On 05/08/2021 17.27, David Hildenbrand wrote:
The access type is unused.
Signed-off-by: David Hildenbrand
---
target/s390x/mmu_helper.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/target/s390x/mmu_helper.c b/target/s390x/mmu_helper.c
index 0620b1803e..167f1b1455 1
On 05/08/2021 17.27, David Hildenbrand wrote:
Looks like we forgot to adjust documentation of one parameter.
Signed-off-by: David Hildenbrand
---
target/s390x/mmu_helper.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/target/s390x/mmu_helper.c b/target/s390x/mmu_helpe
On 06.08.21 09:30, Thomas Huth wrote:
Fixes: 81d7e3bc45 ("s390x/mmu: Inject DAT exceptions from a single place")
Reviewed-by: Thomas Huth
I'm usually a bit careful with Fixes tags if we're not fixing real BUGs.
At least in the kernel people will really complain if you do that
(because it mig
On 06/08/2021 09.34, David Hildenbrand wrote:
On 06.08.21 09:30, Thomas Huth wrote:
Fixes: 81d7e3bc45 ("s390x/mmu: Inject DAT exceptions from a single place")
Reviewed-by: Thomas Huth
I'm usually a bit careful with Fixes tags if we're not fixing real BUGs. At
least in the kernel people will
On 06.08.21 09:36, Thomas Huth wrote:
On 06/08/2021 09.34, David Hildenbrand wrote:
On 06.08.21 09:30, Thomas Huth wrote:
Fixes: 81d7e3bc45 ("s390x/mmu: Inject DAT exceptions from a single place")
Reviewed-by: Thomas Huth
I'm usually a bit careful with Fixes tags if we're not fixing real BUG
Hi Eric,
在 2021/8/5 16:10, Eric Auger 写道:
Hi Chenxiang,
On 8/5/21 9:48 AM, chenxiang (M) wrote:
Hi Eric,
在 2021/8/5 0:26, Eric Auger 写道:
Hi Chenxiang,
On 8/4/21 10:49 AM, chenxiang wrote:
From: Xiang Chen
It splits invalidations into ^2 range invalidations in the patch
6d9cd115b(" hw/ar
On 2021/8/6 12:41, Pankaj Gupta wrote:
In the real SMP hardware topology world, it's much more likely that
we have high cores-per-socket counts and few sockets totally. While
the current preference of sockets over cores in smp parsing results
in a virtual cpu topology with low cores-per-sockets c
On 05/08/2021 17.27, David Hildenbrand wrote:
Let's move address validation into mmu_translate() and
mmu_translate_real(). This allows for checking whether an absolute
address is valid before looking up the storage key. We can now get rid of
the ram_size check.
Interestingly, we're already handl
On 06.08.21 10:18, Thomas Huth wrote:
On 05/08/2021 17.27, David Hildenbrand wrote:
Let's move address validation into mmu_translate() and
mmu_translate_real(). This allows for checking whether an absolute
address is valid before looking up the storage key. We can now get rid of
the ram_size che
On 05/08/2021 17.27, David Hildenbrand wrote:
Let's move address validation into mmu_translate() and
mmu_translate_real(). This allows for checking whether an absolute
address is valid before looking up the storage key. We can now get rid of
the ram_size check.
Interestingly, we're already handl
On 06/08/2021 10.20, David Hildenbrand wrote:
On 06.08.21 10:18, Thomas Huth wrote:
On 05/08/2021 17.27, David Hildenbrand wrote:
Let's move address validation into mmu_translate() and
mmu_translate_real(). This allows for checking whether an absolute
address is valid before looking up the stor
On 06.08.21 10:22, Thomas Huth wrote:
On 06/08/2021 10.20, David Hildenbrand wrote:
On 06.08.21 10:18, Thomas Huth wrote:
On 05/08/2021 17.27, David Hildenbrand wrote:
Let's move address validation into mmu_translate() and
mmu_translate_real(). This allows for checking whether an absolute
addr
On 05/08/2021 17.28, David Hildenbrand wrote:
Avoid setting the key if nothing changed.
Signed-off-by: David Hildenbrand
---
target/s390x/mmu_helper.c | 11 +++
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git a/target/s390x/mmu_helper.c b/target/s390x/mmu_helper.c
index 3
On 06/08/2021 10.23, David Hildenbrand wrote:
On 06.08.21 10:22, Thomas Huth wrote:
On 06/08/2021 10.20, David Hildenbrand wrote:
On 06.08.21 10:18, Thomas Huth wrote:
On 05/08/2021 17.27, David Hildenbrand wrote:
Let's move address validation into mmu_translate() and
mmu_translate_real(). Th
A gentle ping ...
Will this patch be picked up soon for 6.1 or is there anything else to
tweak?
Thanks,
Yanan
On 2021/7/23 17:57, Yanan Wang wrote:
In the SMP configuration, we should either specify a topology
parameter with a reasonable value (equal to or greater than 1)
or just leave it om
On 05/08/2021 17.28, David Hildenbrand wrote:
Let's use the guest_phys_blocks API to get physical memory regions
that are well defined inside our physical address space and migrate the
storage keys of these.
This is a prepearation for having memory besides initial ram defined in
s/prepearation
On 05/08/2021 17.28, David Hildenbrand wrote:
Handle it similar to migration. Assert that we're holding the BQL, to
make sure we don't see concurrent modifications.
Signed-off-by: David Hildenbrand
---
hw/s390x/s390-skeys.c | 50 ++-
1 file changed, 30
On 05/08/2021 17.28, David Hildenbrand wrote:
Let's validate the given address and report a proper error in case it's
not. All call paths now properly check the validity of the given GFN.
Remove the TODO.
The errors inside the getter and setter should only trigger if something
really goes wrong
On 06.08.21 10:53, Thomas Huth wrote:
On 05/08/2021 17.28, David Hildenbrand wrote:
Let's validate the given address and report a proper error in case it's
not. All call paths now properly check the validity of the given GFN.
Remove the TODO.
The errors inside the getter and setter should only
On Thu, Aug 05, 2021 at 11:00:24PM -0400, Eduardo Habkost wrote:
> Hello,
>
> I'm looking for help dealing with a naming conflict when building
> QEMU for Windows hosts.
>
> The summary is: I'm trying to replace the ACCEL() macro in
> include/qemu/accel.h with an inline function, but the ACCEL na
030912e01c0385b6b09e76549c1a8a04b624f49a:
linux-user/elfload: byteswap i386 registers when dumping core (2021-08-05
12:17:30 +0200)
linux-user pull request 20210806
Fix endianness and addresses in i386 linux-user target
From: Ilya Leoshkevich
setup_rt_frame() passes siginfo and ucontext host addresses to guest
signal handlers, causing problems when e.g. emulating x86_64 on s390x.
Signed-off-by: Ilya Leoshkevich
Reviewed-by: Richard Henderson
Message-Id: <20210803171858.148394-1-...@linux.ibm.com>
Signed-off-b
From: Ilya Leoshkevich
Core dumps from emulating x86_64 on big-endian hosts contain incorrect
register values.
Signed-off-by: Ilya Leoshkevich
Reviewed-by: Peter Maydell
Reviewed-by: Philippe Mathieu-Daudé
Message-Id: <20210803172013.148446-1-...@linux.ibm.com>
Signed-off-by: Laurent Vivier
> >> In the real SMP hardware topology world, it's much more likely that
> >> we have high cores-per-socket counts and few sockets totally. While
> >> the current preference of sockets over cores in smp parsing results
> >> in a virtual cpu topology with low cores-per-sockets counts and a
> >> larg
Finalizing the job may cause its AioContext to change. This is noted by
job_exit(), which points at job_txn_apply() to take this fact into
account.
However, job_completed() does not necessarily invoke job_txn_apply()
(through job_completed_txn_success()), but potentially also
job_completed_txn_ab
Hi,
v1 cover letter:
https://lists.nongnu.org/archive/html/qemu-block/2021-07/msg00705.html
v2 cover letter:
https://lists.nongnu.org/archive/html/qemu-block/2021-07/msg00747.html
Changes in v3:
- Patch 1: After adding patch 11, I got a failed assertion in
tests/unit/test-block-iothread (faili
We largely have two cancel modes for jobs:
First, there is actual cancelling. The job is terminated as soon as
possible, without trying to reach a consistent result.
Second, we have mirror in the READY state. Technically, the job is not
really cancelled, but it just is a different completion mo
An error does not take us out of the READY phase, which is what
s->synced signifies. It does of course mean that source and target are
no longer in sync, but that is what s->actively_sync is for -- s->synced
never meant that source and target are in sync, only that they were at
some point (and at
As of HEAD^, there is no meaning to s->synced other than whether the job
is READY or not. job_is_ready() gives us that information, too.
Suggested-by: Vladimir Sementsov-Ogievskiy
Signed-off-by: Max Reitz
Reviewed-by: Eric Blake
Reviewed-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: Kevin Wol
Most callers of job_is_cancelled() actually want to know whether the job
is on its way to immediate termination. For example, we refuse to pause
jobs that are cancelled; but this only makes sense for jobs that are
really actually cancelled.
A mirror job that is cancelled during READY with force=f
Callers should be able to specify whether they want job_cancel_sync() to
force-cancel the job or not.
In fact, almost all invocations do not care about consistency of the
result and just want the job to terminate as soon as possible, so they
should pass force=true. The replication block driver is
When a transaction is aborted, no result matters, and so all jobs within
should be force-cancelled.
Signed-off-by: Max Reitz
---
job.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/job.c b/job.c
index 3fe23bb77e..24e7c4fcb7 100644
--- a/job.c
+++ b/job.c
@@ -766,7 +76
On 05/08/2021 17.28, David Hildenbrand wrote:
Let's enable storage keys lazily under TCG, just as we do under KVM.
Only fairly old Linux versions actually make use of storage keys, so it
can be kind of wasteful to allocate quite some memory and track
changes and references if nobody cares.
We ha
We must check whether the job is force-cancelled early in our main loop,
most importantly before any `continue` statement. For example, we used
to have `continue`s before our current checking location that are
triggered by `mirror_flush()` failing. So, if `mirror_flush()` kept
failing, force-canc
On Fri, 2021-08-06 at 07:25 +0200, Thomas Huth wrote:
> On 05/08/2021 00.51, Ilya Leoshkevich wrote:
> > Verify that s390x-specific uc_mcontext.psw.addr is reported
> > correctly
> > and that signal handling interacts properly with debugging.
> >
> > Signed-off-by: Ilya Leoshkevich
> > ---
> >
>
mirror_drained_poll() returns true whenever the job is cancelled,
because "we [can] be sure that it won't issue more requests". However,
this is only true for force-cancelled jobs, so use job_is_cancelled().
Signed-off-by: Max Reitz
---
block/mirror.c | 2 +-
1 file changed, 1 insertion(+), 1 d
Once the mirror job is force-cancelled (job_is_cancelled() is true), we
should not generate new I/O requests. This applies to active mirroring,
too, so stop it once the job is cancelled.
(We must still forward all I/O requests to the source, though, of
course, but those are not really I/O request
Clearing .cancelled before leaving the main loop when the job has been
soft-cancelled is no longer necessary since job_is_cancelled() only
returns true for jobs that have been force-cancelled.
Therefore, this only makes a differences in places that call
job_cancel_requested(). In block/mirror.c,
Test what happens when there is an I/O error after a mirror job in the
READY phase has been cancelled.
Signed-off-by: Max Reitz
Reviewed-by: Vladimir Sementsov-Ogievskiy
Tested-by: Vladimir Sementsov-Ogievskiy
---
.../tests/mirror-ready-cancel-error | 143 ++
.../test
On Fri, Aug 6, 2021 at 2:12 PM Ruinland Chuan-Tzu Tsa(蔡傳資)
wrote:
>
> Hi Bin and Alistair,
>
> >> Adding Andes AX25 and A25 CPU model into cpu.h and cpu.c without
>
> > The latest RISC-V core from Andes is AX45 and A45. Should we just
> > support the latest one?
>
> Maybe we can have them all ?
>
On Thu, Aug 5, 2021 at 5:15 PM LIU Zhiwei wrote:
>
>
> On 2021/8/5 下午2:01, Alistair Francis wrote:
> > On Thu, Aug 5, 2021 at 12:55 PM LIU Zhiwei wrote:
> >> This patch set implements UXL field in mstatus register. Programmer can
> >> change
> >> UXLEN by writting to this field. So that you can
On Fri, 6 Aug 2021 at 07:24, Duo jia wrote:
> I am simulating a device. When an interrupt occurs, another interrupt
> comes, and the second interrupt will not be triggered because the
> first interrupt has not yet finished.
>
> I want to know whether qemu can detect whether the interrupt has been
On Thu, 5 Aug 2021 18:13:21 -0400
"Michael S. Tsirkin" wrote:
> On Thu, Aug 05, 2021 at 07:42:35PM +0530, Ani Sinha wrote:
> > Also to be noted that there is a stub for acpi_get_i386_pci_host() which
> > simply returns NULL. This activates when CONFIG_PC is disabled. It is this
> > stub that gets
On Thu, 5 Aug 2021 at 18:39, Peter Maydell wrote:
>
> On Wed, 4 Aug 2021 at 15:33, Peter Maydell wrote:
> >
> > On Wed, 4 Aug 2021 at 14:43, Michael S. Tsirkin wrote:
> > >
> > > On Wed, Aug 04, 2021 at 01:40:37PM +0100, Peter Maydell wrote:
> > > > Saw this intermittent as part of my ad-hoc CI
On Thu, 5 Aug 2021 19:42:35 +0530 (IST)
Ani Sinha wrote:
> On Thu, 5 Aug 2021, Ani Sinha wrote:
>
> >
> >
> > On Thu, 5 Aug 2021, Ani Sinha wrote:
> >
> > >
> > >
> > > On Thu, 5 Aug 2021, Igor Mammedov wrote:
> > >
> > > > On Mon, 26 Jul 2021 22:27:43 +0530
> > > > Ani Sinha wrote:
> > > >
[sorry for the thread necromancy -- got preempted in other stuff
and am slowly coming back to this]
On 6/28/21 3:32 PM, Igor Mammedov wrote:
> On Wed, 23 Jun 2021 14:04:19 +0100
> Joao Martins wrote:
>
>> On 6/23/21 12:39 PM, Igor Mammedov wrote:
>>> On Wed, 23 Jun 2021 10:37:38 +0100
>>> Joao
On Thu, 5 Aug 2021 15:34:26 -0400
Eduardo Habkost wrote:
> The macro never worked and never will, because the
> AcpiGedX86State type never existed.
>
> Signed-off-by: Eduardo Habkost
Reviewed-by: Igor Mammedov
> ---
> Cc: "Michael S. Tsirkin"
> Cc: Igor Mammedov
> Cc: qemu-devel@nongnu.or
On Fri, 6 Aug 2021, Igor Mammedov wrote:
> On Thu, 5 Aug 2021 19:42:35 +0530 (IST)
> Ani Sinha wrote:
>
> > On Thu, 5 Aug 2021, Ani Sinha wrote:
> >
> > >
> > >
> > > On Thu, 5 Aug 2021, Ani Sinha wrote:
> > >
> > > >
> > > >
> > > > On Thu, 5 Aug 2021, Igor Mammedov wrote:
> > > >
> > > > > O
On Thu, 5 Aug 2021 15:34:31 -0400
Eduardo Habkost wrote:
> OBJECT_CHECK(PciHostState, ..., TYPE_PCI_HOST_BRIDGE) is exactly
> what the PCI_HOST_BRIDGE macro does. We can just use the macro
> instead of using OBJECT_CHECK manually.
>
> Signed-off-by: Eduardo Habkost
Reviewed-by: Igor Mammedov
On Thu, 5 Aug 2021 18:30:01 -0400
Shashi Mallela wrote:
> Included creation of ITS as part of virt platform GIC
> initialization. This Emulated ITS model now co-exists with kvm
> ITS and is enabled in absence of kvm irq kernel support in a
> platform.
>
> Signed-off-by: Shashi Mallela
> Review
On Thu, 5 Aug 2021 18:30:00 -0400
Shashi Mallela wrote:
> Added expected IORT files applicable with latest GICv3
> ITS changes.Temporarily differences in these files are
> okay.
>
> Signed-off-by: Shashi Mallela
Acked-by: Igor Mammedov
> ---
> tests/data/acpi/virt/IORT |
On Thu, 5 Aug 2021 18:30:02 -0400
Shashi Mallela wrote:
> Updated expected IORT files applicable with latest GICv3
> ITS changes.
>
> Full diff of new file disassembly:
>
> /*
> * Intel ACPI Component Architecture
> * AML/ASL+ Disassembler version 20180629 (64-bit version)
> * Copyright (c)
On Thu, Aug 05 2021, David Hildenbrand wrote:
> We not only invalidate the translation of the range 0x0-0x2000, we also
> invalidate the translation of the new prefix range and the translation
> of the old prefix range -- because real2abs would return different
> results for all of these ranges w
On Thu, Aug 05 2021, David Hildenbrand wrote:
> schib->pmcw.chars is 32bit, not 16bit. This fixes the kvm-unit-tests
> "css" test, which fails with:
>
> FAIL: Channel Subsystem: measurement block format1: Unaligned MB origin:
> Program interrupt: expected(21) == received(0)
>
> Because we end
Marc-André Lureau writes:
> Hi
>
> On Mon, Aug 2, 2021 at 1:21 PM Markus Armbruster wrote:
>
>> marcandre.lur...@redhat.com writes:
>>
>> > From: Marc-André Lureau
>> >
>> > Except for the special casing assert in _make_implicit_object_type(),
>> > which needs to handle schema objects, it's a m
On Fri, Aug 06 2021, David Hildenbrand wrote:
> On 06.08.21 09:11, Thomas Huth wrote:
>> On 06/08/2021 08.52, David Hildenbrand wrote:
According to the PoP:
"When the enhanced-DAT facility 1 is not installed, or
when the facility is installed but the multiple-block
>>
On 06.08.21 13:17, Cornelia Huck wrote:
On Thu, Aug 05 2021, David Hildenbrand wrote:
We not only invalidate the translation of the range 0x0-0x2000, we also
invalidate the translation of the new prefix range and the translation
of the old prefix range -- because real2abs would return differen
On 06.08.21 13:19, Cornelia Huck wrote:
On Thu, Aug 05 2021, David Hildenbrand wrote:
schib->pmcw.chars is 32bit, not 16bit. This fixes the kvm-unit-tests
"css" test, which fails with:
FAIL: Channel Subsystem: measurement block format1: Unaligned MB origin:
Program interrupt: expected(2
On 06.08.21 13:25, Cornelia Huck wrote:
On Fri, Aug 06 2021, David Hildenbrand wrote:
On 06.08.21 09:11, Thomas Huth wrote:
On 06/08/2021 08.52, David Hildenbrand wrote:
According to the PoP:
"When the enhanced-DAT facility 1 is not installed, or
when the facility is installed but the
On Thu, Aug 05 2021, Eduardo Habkost wrote:
> We have a SCLPEventsBus struct defined, but the struct is not
> used at the TypeInfo definition. This works today but will break
> silently if anybody adds a new field to SCLPEventsBus.
>
> Set instance_size properly to avoid problems in the future.
On Thu, Aug 05 2021, Eduardo Habkost wrote:
> We have a SCLPEventsBus struct type defined, but no QOM type
> checkers are declared for the type.
>
> Use OBJECT_DECLARE_SIMPLE_TYPE to declare the struct type and
> have a SCLP_EVENT_BUS typecast wrapper defined.
>
> Signed-off-by: Eduardo Habkost
On Fri, Aug 06 2021, David Hildenbrand wrote:
> On 06.08.21 06:15, Thomas Huth wrote:
>> On 05/08/2021 21.34, Eduardo Habkost wrote:
>>> Use the SCLP_EVENT_BUS macro instead of manually calling
>>> OBJECT_CHECK.
>>>
>>> Signed-off-by: Eduardo Habkost
>>> ---
>>> Cc: Cornelia Huck
>>> Cc: Thomas
---
> linux-user pull request 20210806
>
> Fix endianness and addresses in i386 linux-user target
>
Applied, thanks.
Please update the changelog at https://wiki.qemu.org/ChangeLog/6.1
for any user-visible changes.
-- PMM
QAPISchema._make_implicit_object_type() asserts that when an implicit
object type is used multiple times, @ifcond is the same for all uses.
It will be for legitimate uses, i.e. simple union branch wrapper
types. A comment explains this.
The assertion fails when a command or event is redefined wit
On Thu, Aug 05, 2021 at 05:47:39PM -0400, Eduardo Habkost wrote:
> commit 58b350280e97 ("hw/arm/bcm2836: Restrict BCM283XInfo
> declaration to C source") didn't just move the struct
> BCM283XClass definition to bcm2836.c. It also introduced a
> typedef (BCM283XClass) and two type checking macros
>
> On Fri, Aug 6, 2021 at 10:39 AM Bin Meng wrote:
> >
> > On Fri, Aug 6, 2021 at 1:57 AM Ruinland Chuan-Tzu Tsai
> > wrote:
> > >
> > > From: Ruinland ChuanTzu Tsai
> > >
> > > Adding option `riscv_custom` to configure script, meson.build and
> > > meson_options.txt so as to toggle custom CSR an
> From: Ruinalnd ChuanTzu Tsai
>
> In this patch we enabled custom CSR logic for Andes AX25 and A25 logic.
> Hence csr_andes.inc.c and andes_cpu_bits.h is added.
>
> Signed-off-by: Dylan Jhong
> ---
> target/riscv/andes_cpu_bits.h | 124 +
> target/riscv/cpu.c
Thanks for sharing the observations Neil.
Will take care of it in the next patch-set.
On Fri, 2021-08-06 at 10:58 +0200, Neil Armstrong wrote:
> Hi,
>
> On 06/08/2021 00:29, Shashi Mallela wrote:
> > Added ITS command queue handling for MAPTI,MAPI commands,handled
> > ITS
> > translation which tr
Hi,
On 06/08/2021 00:29, Shashi Mallela wrote:
> Added ITS command queue handling for MAPTI,MAPI commands,handled ITS
> translation which triggers an LPI via INT command as well as write
> to GITS_TRANSLATER register,defined enum to differentiate between ITS
> command interrupt trigger and GITS_TR
On 6 Aug 2021, at 14:06, Jessica Clarke wrote:
>
>> From: Ruinalnd ChuanTzu Tsai
>>
>> In this patch we enabled custom CSR logic for Andes AX25 and A25 logic.
>> Hence csr_andes.inc.c and andes_cpu_bits.h is added.
>>
>> Signed-off-by: Dylan Jhong
>> ---
>> target/riscv/andes_cpu_bits.h | 12
On 06.08.21 11:42, Thomas Huth wrote:
On 05/08/2021 17.28, David Hildenbrand wrote:
Let's enable storage keys lazily under TCG, just as we do under KVM.
Only fairly old Linux versions actually make use of storage keys, so it
can be kind of wasteful to allocate quite some memory and track
changes
On 06/08/2021 15.18, David Hildenbrand wrote:
On 06.08.21 11:42, Thomas Huth wrote:
On 05/08/2021 17.28, David Hildenbrand wrote:
Let's enable storage keys lazily under TCG, just as we do under KVM.
Only fairly old Linux versions actually make use of storage keys, so it
can be kind of wasteful
On Fri, Aug 6, 2021 at 8:58 PM Jessica Clarke wrote:
>
> > On Fri, Aug 6, 2021 at 10:39 AM Bin Meng wrote:
> > >
> > > On Fri, Aug 6, 2021 at 1:57 AM Ruinland Chuan-Tzu Tsai
> > > wrote:
> > > >
> > > > From: Ruinland ChuanTzu Tsai
> > > >
> > > > Adding option `riscv_custom` to configure scrip
On 8/6/21 12:52 PM, Ani Sinha wrote:
> On Fri, 6 Aug 2021, Igor Mammedov wrote:
>> On Thu, 5 Aug 2021 19:42:35 +0530 (IST)
>> Ani Sinha wrote:
>>> On Thu, 5 Aug 2021, Ani Sinha wrote:
On Thu, 5 Aug 2021, Ani Sinha wrote:
> On Thu, 5 Aug 2021, Igor Mammedov wrote:
>> On Mon, 26 Jul 202
On 6 Aug 2021, at 14:54, Bin Meng wrote:
> On Fri, Aug 6, 2021 at 8:58 PM Jessica Clarke wrote:
>>
>>> On Fri, Aug 6, 2021 at 10:39 AM Bin Meng wrote:
On Fri, Aug 6, 2021 at 1:57 AM Ruinland Chuan-Tzu Tsai
wrote:
>
> From: Ruinland ChuanTzu Tsai
>
> Adding op
On 04/08/21 13:30, Lara Lazier wrote:
APM2 requires that VMRUN and VMLOAD canonicalize (sign extend to 63
from 48/57) all base addresses in the segment registers that have been
respectively loaded.
Signed-off-by: Lara Lazier
---
target/i386/cpu.c | 19 +++
t
From: Thomas Huth
The container already features meson and ninja, so there is no need
to try to install it with dnf again.
Signed-off-by: Thomas Huth
Reviewed-by: Willian Rampazzo
Message-Id: <20210730143809.717079-3-th...@redhat.com>
Signed-off-by: Alex Bennée
---
.gitlab-ci.d/buildtest.yml
Hi,
There isn't much in my queue but I figured I best post it now so I can
cut a PR from it next week. I'll be off most of August so this will
likely be the last PR for 6.1 from me.
Mahmoud Mandour (1):
plugins/execlog: removed unintended "s" at the end of log lines.
Thomas Huth (3):
gitlab-
From: Thomas Huth
The patch that recently introduced the S390X_RUNNER_AVAILABLE variable
in custom-runners.yml missed that the bottom half of the file is rather
about aarch64 than s390x. Thus rename the S390X_RUNNER_AVAILABLE to
AARCH64_RUNNER_AVAILABLE in those jobs.
Finally mention both variab
From: Mahmoud Mandour
Signed-off-by: Mahmoud Mandour
Signed-off-by: Alex Bennée
Message-Id: <20210803151428.125323-1-ma.mando...@gmail.com>
---
contrib/plugins/execlog.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/contrib/plugins/execlog.c b/contrib/plugins/execlog.c
in
From: Thomas Huth
Both jobs are testing more or less the same thing (building QEMU with
features disabled), so we are wasting precious CI cycles here by doing
this twice. Merge the jobs by using --without-default-features by default
and just adding some additional --disable-... switches which are
On Fri, Aug 06 2021, David Hildenbrand wrote:
> On 06.08.21 11:42, Thomas Huth wrote:
>> On 05/08/2021 17.28, David Hildenbrand wrote:
>>> -static void qemu_s390_skeys_init(Object *obj)
>>> +static int qemu_s390_skeys_enabled(S390SKeysState *ss)
>>>{
>>> -QEMUS390SKeysState *skeys = QEMU_
On Fri, Jul 30, 2021 at 05:01:25PM +0200, Max Reitz wrote:
> We only need to drop/switch our credentials for the (f)setxattr() call
> alone, not for the openat() or fchdir() around it.
>
> (Right now, this may not be that big of a problem, but with inodes being
> identified by file handles instead
On 8/6/21 1:30 PM, David Hildenbrand wrote:
> On 06.08.21 13:19, Cornelia Huck wrote:
>> On Thu, Aug 05 2021, David Hildenbrand wrote:
>>
>>> schib->pmcw.chars is 32bit, not 16bit. This fixes the kvm-unit-tests
>>> "css" test, which fails with:
>>>
>>> FAIL: Channel Subsystem: measurement block
On 06/08/2021 16.13, Cornelia Huck wrote:
On Fri, Aug 06 2021, David Hildenbrand wrote:
On 06.08.21 11:42, Thomas Huth wrote:
On 05/08/2021 17.28, David Hildenbrand wrote:
-static void qemu_s390_skeys_init(Object *obj)
+static int qemu_s390_skeys_enabled(S390SKeysState *ss)
{
-QEMUS3
Ilya Leoshkevich writes:
> Verify that s390x-specific uc_mcontext.psw.addr is reported correctly
> and that signal handling interacts properly with debugging.
>
> Signed-off-by: Ilya Leoshkevich
Acked-by: Alex Bennée
> ---
>
> v7: https://lists.nongnu.org/archive/html/qemu-devel/2021-08/msg
On 06.08.21 16:17, Philippe Mathieu-Daudé wrote:
On 8/6/21 1:30 PM, David Hildenbrand wrote:
On 06.08.21 13:19, Cornelia Huck wrote:
On Thu, Aug 05 2021, David Hildenbrand wrote:
schib->pmcw.chars is 32bit, not 16bit. This fixes the kvm-unit-tests
"css" test, which fails with:
FAIL: Cha
On Fri, Jul 30, 2021 at 05:01:26PM +0200, Max Reitz wrote:
> We are planning to add file handles to lo_inode objects as an
> alternative to lo_inode.fd. That means that everywhere where we
> currently reference lo_inode.fd, we will have to open a temporary file
> descriptor that needs to be closed
On 210804 1451, Qiang Liu wrote:
> xlnx_dp_read allows an out-of-bounds read at its default branch because
> of an improper index.
>
> According to
> https://www.xilinx.com/html_docs/registers/ug1087/ug1087-zynq-ultrascale-registers.html
> (DP Module), registers 0x3A4/0x3A4/0x3AC are allowed.
>
>
This would previously give error messages like
> Received unexpected msg type.Expected 0 received 1
Signed-off-by: Alyssa Ross
---
hw/virtio/vhost-user.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/virtio/vhost-user.c b/hw/virtio/vhost-user.c
index 29ea2b4fce..ad9fc40
Break events are currently only handled by chardev/char-serial.c, so we
just ignore errors, which results in no behaviour change for other
chardevs.
Signed-off-by: Jan Luebbe
---
hw/char/pl011.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/hw/char/pl011.c b/hw/char/pl011.c
index dc8
On Fri, Aug 06 2021, David Hildenbrand wrote:
> On 06.08.21 16:17, Philippe Mathieu-Daudé wrote:
>> On 8/6/21 1:30 PM, David Hildenbrand wrote:
>>> On 06.08.21 13:19, Cornelia Huck wrote:
On Thu, Aug 05 2021, David Hildenbrand wrote:
> schib->pmcw.chars is 32bit, not 16bit. This fi
The logic before was
if not get_option('gnutls').auto() or have_system
Which is equivalent to
if get_option('gnutls').enabled() or get_option('gnutls').disabled() or
have_system
This means that the check for gnutls is performed even if gnutls is
disabled, which means that the b
These CLI options had been removed/replaced in QEMU v2.12. Still, some
people might want to update from older versions to the recent QEMU version,
so we should give some recommendations for the replacements in our
documentation.
Signed-off-by: Thomas Huth
---
docs/about/removed-features.rst | 37
1 - 100 of 202 matches
Mail list logo
