The tests/acceptance/pvh.py test builds the Linux kernel
at runtime so it needs dependencies installed in the
container. Current used container image misses only
libelf-dev, which is then added with this change.
Signed-off-by: Wainer dos Santos Moschetta
---
.travis.yml | 2 ++
1 file changed, 2
Because serial backend readiness isn't checked, the socket frontend (in
websocket mode) would send new characters before previous characters
were consumed. This lead to skipped characters, or worse, SysRq keys
being triggered.
This patch ensures the readable size is refreshed before consuming any
06.12.2019 10:46, Markus Armbruster wrote:
> Vladimir Sementsov-Ogievskiy writes:
>
>> 30.11.2019 22:42, Markus Armbruster wrote:
>>> fit_load_fdt() recovers from fit_image_addr() failing with ENOENT.
>>> Except it doesn't when its @errp argument is &error_fatal or
>>> &error_abort, because it bl
On Sun, 1 Dec 2019 at 12:20, Marc Zyngier wrote:
>
> HCR_EL2.TID3 requires that AArch32 reads of MVFR[012] are trapped to
> EL2, and HCR_EL2.TID0 does the same for reads of FPSID.
> In order to handle this, introduce a new TCG helper function that
> checks for these control bits before executing t
On Fri, 2019-12-06 at 10:17 +0800, Pan Nengyuan wrote:
> On 2019/12/6 0:45, Amit Shah wrote:
> > On Wed, 2019-12-04 at 15:31 +0800, pannengy...@huawei.com wrote:
> > > From: Pan Nengyuan
> >
> > Shouldn't this be From: mst?
> >
> > I didn't find a ref to the original patch to confirm if you had
I have two mail address, add entries for
showing author and email correctly.
Signed-off-by: Yu-Chen Lin
---
.mailmap | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/.mailmap b/.mailmap
index 3816e4effe..3fbf3902a3 100644
--- a/.mailmap
+++ b/.mailmap
@@ -151,7 +151,8 @@ Xia
On Fri, 6 Dec 2019 09:45:41 +0100
Janosch Frank wrote:
> On 12/6/19 9:29 AM, Cornelia Huck wrote:
> > On Fri, 6 Dec 2019 08:44:52 +0100
> > Janosch Frank wrote:
> >
> >> On 12/5/19 6:46 PM, Cornelia Huck wrote:
> >>> On Thu, 5 Dec 2019 18:34:32 +0100
> >>> Janosch Frank wrote:
> >>>
>
We neglected to clean up pending interrupts and emergency signals;
fix that.
Signed-off-by: Cornelia Huck
---
v1->v2:
- rebased on top of my s390-next branch; we can now move the fields
to be reset instead of clearing them manually
Further cleanup possible in a follow-up patch.
---
target/s
On Thu, 5 Dec 2019 13:17:22 +0100
Laszlo Ersek wrote:
> On 12/04/19 18:05, Igor Mammedov wrote:
> > Correct returned value description in case 'Command field' == 0x0,
> > it's in not PXM but CPU selector value with pending event
>
> (1) s/in not/not/
>
> >
> > In addition describe 0 blanket
On 12/6/19 10:08 AM, Cornelia Huck wrote:
> On Fri, 6 Dec 2019 09:45:41 +0100
> Janosch Frank wrote:
>
>> On 12/6/19 9:29 AM, Cornelia Huck wrote:
>>> On Fri, 6 Dec 2019 08:44:52 +0100
>>> Janosch Frank wrote:
>>>
On 12/5/19 6:46 PM, Cornelia Huck wrote:
> On Thu, 5 Dec 2019 18:34:
On 12/6/19 3:46 AM, Peter Maydell wrote:
>> +case ARMMMUIdx_TagS:
>> +case ARMMMUIdx_TagNS:
>> +/* Indicate tag memory to arm_asidx_from_attrs. */
>> +attrs->target_tlb_bit2 = true;
>> +break;
>
> So here we fall through to the "handle a stage 1 lookup" code, which
06.12.2019 11:26, Markus Armbruster wrote:
> Eric Blake writes:
>
>> On 12/5/19 8:48 AM, Vladimir Sementsov-Ogievskiy wrote:
>>
> @@ -918,27 +917,26 @@ static void device_set_realized(Object *obj, bool
> value, Error **errp)
>}
>} else if (!value && dev->r
06.12.2019 11:54, Markus Armbruster wrote:
> Vladimir Sementsov-Ogievskiy writes:
>
>> 05.12.2019 20:14, Eric Blake wrote:
>>> On 12/5/19 9:20 AM, Vladimir Sementsov-Ogievskiy wrote:
The local_err parameter is not here to return information about
nbd_iter_channel_error failure. Instead
Jan Kiszka writes:
> On 05.12.19 12:14, Markus Armbruster wrote:
>> This has been on the list for more than three weeks already. I
>> apologize for the delay.
>
> No problem! Your feedback is highly appreciated.
>
>>
>> Jan Kiszka writes:
>>
>>> From: Jan Kiszka
>>>
>>> This imports the ivsh
On 12/5/19 6:46 PM, Cornelia Huck wrote:
> On Thu, 5 Dec 2019 18:34:32 +0100
> Janosch Frank wrote:
>
>> On 12/5/19 6:15 PM, Cornelia Huck wrote:
>>> On Fri, 29 Nov 2019 04:48:02 -0500
>>> Janosch Frank wrote:
>>>
Secure guests no longer intercept with code 4 for an instruction
inte
From: Yang Zhong
If kvm does not support VMX feature by nested=0, the kvm_vmx_basic
can't get the right value from MSR_IA32_VMX_BASIC register, which
make qemu coredump when qemu do KVM_SET_MSRS.
The coredump info:
error: failed to set MSR 0x480 to 0x0
kvm_put_msrs: Assertion `ret == cpu->kvm_ms
05.12.2019 23:16, John Snow wrote:
>
>
> On 12/5/19 3:09 PM, Eric Blake wrote:
>> On 12/5/19 1:30 PM, Vladimir Sementsov-Ogievskiy wrote:
>>> Here is double bug:
>>>
>>> First, return error but not set errp. This may lead to:
>>> qmp block-dirty-bitmap-remove may report success when actually fail
06.12.2019 1:37, John Snow wrote:
> This has come up in the past, and I believe we discussed this at KVM
> Forum, too:
>
> There have been requests from oVirt (via Nir Soffer) to add some offline
> bitmap manipulation functionality. In the past, our stance has generally
> been "Use QEMU without an
b822e06fb2b13666fd9278e:
> > >
> > > Update version for v4.2.0-rc4 release (2019-12-03 17:56:30 +)
> > >
> > > are available in the Git repository at:
> > >
> > > git://github.com/dgibson/qemu.git tags/ppc-for-4.2-20191206
> > >
>
The script checkpatch.pl located in scripts folder was
used to detect all errors and warrnings in files:
hw/mips/mips_mipssim.c
hw/net/mipsnet.c
All these mips mipssim machine files were edited and
all the errors and warrings generated by the checkpatch.pl
script were corrected and then th
06.12.2019 3:02, David Gibson wrote:
> On Thu, Dec 05, 2019 at 06:20:05PM +0300, Vladimir Sementsov-Ogievskiy wrote:
>> Make kvmppc_hint_smt_possible hint append helper well formed:
>> rename errp to errp_in, as it is IN-parameter here (which is unusual
>> for errp), rename function to be kvmppc_er
On Sun, 1 Dec 2019 at 12:20, Marc Zyngier wrote:
>
> Hi all,
>
> This series is a follow-up on [1], which tried to address the
> remaining missing HCR_EL2.TIDx traps. I've hopefully now adressed the
> comments that Peter and Edgar raised.
>
> I've also tried to tackle missing traps generated by HS
On 12/6/19 5:02 AM, Peter Maydell wrote:
>> +/*
>> + * Find the physical address for the virtual access.
>> + *
>> + * TODO: It should be possible to have the tag mmu_idx map
>> + * from main memory ram_addr to tag memory host address.
>> + * that would allow this lookup ste
On Fri, 6 Dec 2019 at 01:26, Alexey Kardashevskiy wrote:
> *** Note: this is not for master, this is for pseries
The way to say this in a way that works with my filters for
handling pullrequests is to say "not for master" and
have the subject tag be "[PULL SUBSYSTEM name-of-subsystem]"
(see https
Laszlo Ersek writes:
> On 12/05/19 14:05, Philippe Mathieu-Daudé wrote:
>> Hi Cornelia,
>>
>> On 12/5/19 12:53 PM, Cornelia Huck wrote:
>>> The Posix implementation of guest-set-time invokes hwclock to
>>> set/retrieve the time to/from the hardware clock. If hwclock
>>> is not available, the user
This series add a new acceptance test: boot an uncompressed
Linux kernel built with CONFIG_PVH, so checking the PVH
capability introduced in QEMU 4.0 works.
The test implementation depends on [1] which is likely released
on next Avocado. So that will need a version 2 of this
series to bump Avocado
The script checkpatch.pl located in scripts folder was
used to detect all errors and warrnings in file:
hw/mips/mips_r4k.c
This mips r4000 machine file was edited and
all the errors and warrings generated by the checkpatch.pl
script were corrected and then the script was
ran again to make sure
On 2019-12-06 14:08, Peter Maydell wrote:
On Sun, 1 Dec 2019 at 12:20, Marc Zyngier wrote:
HCR_EL2.TID3 requires that AArch32 reads of MVFR[012] are trapped to
EL2, and HCR_EL2.TID0 does the same for reads of FPSID.
In order to handle this, introduce a new TCG helper function that
checks for t
On Thu, 5 Dec 2019 at 19:24, Richard Henderson
wrote:
>
> On 12/5/19 10:40 AM, Peter Maydell wrote:
> > Are there really separate S and NS tag RAMs?
>
> Implementation defined, I believe. As with everything about tag storage, it
> would seem. But since there are separate S and NS normal RAMS, I
Vladimir Sementsov-Ogievskiy writes:
> 05.12.2019 17:58, Vladimir Sementsov-Ogievskiy wrote:
>> 05.12.2019 15:36, Markus Armbruster wrote:
>>> Vladimir Sementsov-Ogievskiy writes:
>>>
04.12.2019 17:59, Markus Armbruster wrote:
> Vladimir Sementsov-Ogievskiy writes:
>
>> Here is
The scripts checkpatch.pl located in scripts folder
was used to check for errors and warnings in fllowing
mips machines:
Jazz
Malta
Mipssim
R4000
Fulong 2E
Boston
All generated errors and warnings were corrected
and the script was ran again to make sure there
are no more e
Cc'ing qemu-trivial@
On 12/6/19 3:09 PM, Yu-Chen Lin wrote:
I have two mail address, add entries for
showing author and email correctly.
Signed-off-by: Yu-Chen Lin
---
.mailmap | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/.mailmap b/.mailmap
index 3816e4effe..3fbf39
Cornelia Huck writes:
> On Thu, 5 Dec 2019 14:05:19 +0100
> Philippe Mathieu-Daudé wrote:
>
>> Hi Cornelia,
>>
>> On 12/5/19 12:53 PM, Cornelia Huck wrote:
>> > The Posix implementation of guest-set-time invokes hwclock to
>> > set/retrieve the time to/from the hardware clock. If hwclock
>> > i
The script checkpatch.pl located in scripts folder was
used to detect all errors and warrnings in files:
hw/mips/mips_jazz.c
hw/display/jazz_led.c
hw/dma/rc4030.c
All these mips jazz machine files were edited and
all the errors and warrings generated by the checkpatch.pl
script were co
QEMU 4.0 onward is able to boot an uncompressed kernel
image by using the x86/HVM direct boot ABI. It needs
Linux >= 4.21 built with CONFIG_PVH=y.
This introduces an acceptance test which checks an
uncompressed Linux kernel image boots properly.
Signed-off-by: Wainer dos Santos Moschetta
---
te
06.12.2019 10:38, Philippe Mathieu-Daudé wrote:
> On 12/5/19 6:46 PM, Vladimir Sementsov-Ogievskiy wrote:
>> Signed-off-by: Vladimir Sementsov-Ogievskiy
>> ---
>> hw/usb/dev-network.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/hw/usb/dev-network.c b/hw/usb/dev-n
On Tue, 3 Dec 2019 at 19:32, Niek Linnenbank wrote:
> Indeed that sounds like an interesting combination. Are there plans to build
> a multi-arch/single-binary QEMU?
This is in the category of "it would be nice in theory to
support multi-arch guest machines, and we've made some
small steps in th
The script checkpatch.pl located in scripts folder was
used to detect all errors and warrnings in files:
hw/mips/mips_malta.c
hw/mips/gt64xxx_pci.c
tests/acceptance/linux_ssh_mips_malta.py
All these mips malta machine files were edited and
all the errors and warrings generated by the c
Remove the packet size upper limit by using a GByteArray
instead of a statically allocated array for last_packet.
Thus we can now send big packets.
Also remove the last_packet_len field and use last_packet->len
instead.
Signed-off-by: Damien Hedde
---
This patch is a follow-up of Alex's series
The script checkpatch.pl located in scripts folder was
used to detect all errors and warrnings in files:
hw/mips/mips_fulong2e.c
hw/isa/vt82c686.c
hw/pci-host/bonito.c
include/hw/isa/vt82c686.h
These mips Fulong 2E machine files were edited and
all the errors and warrings generated
On Tue, 15 Oct 2019 at 17:47, Richard Henderson
wrote:
>
> This is a refresh of the user-only patch set from March.
>
> I do not attempt to implement any part of a kernel abi wrt mmap
> and/or mprotect. Instead, it uses a x-tagged-pages property to
> assume that all anonymous pages have tags.
>
>
On Thu, 5 Dec 2019 15:07:53 +0100
Laszlo Ersek wrote:
> On 12/04/19 18:05, Igor Mammedov wrote:
> > Describe how to enable and detect modern CPU hotplug interface.
> > Detection part is based on new CPHP_GET_CPU_ID_CMD command,
> > introduced by "acpi: cpuhp: add CPHP_GET_CPU_ID_CMD command" patc
On Fri, Dec 06, 2019 at 05:40:02PM +0800, Jason Wang wrote:
>
> On 2019/12/6 下午4:22, Yan Zhao wrote:
> > On Thu, Dec 05, 2019 at 09:05:54PM +0800, Jason Wang wrote:
> >> On 2019/12/5 下午4:51, Yan Zhao wrote:
> >>> On Thu, Dec 05, 2019 at 02:33:19PM +0800, Jason Wang wrote:
> Hi:
>
>
* Cleber Rosa (cr...@redhat.com) wrote:
> On Thu, Dec 05, 2019 at 12:01:56PM +0100, Philippe Mathieu-Daudé wrote:
> >
> > Understood, thanks for clearing this out!
> >
> > Side note, we don't do cross-arch migration testing, but we talked about
> > having a 'different QEMU version' migration test
On 05.12.19 18:46, Vladimir Sementsov-Ogievskiy wrote:
> Signed-off-by: Vladimir Sementsov-Ogievskiy
> Reviewed-by: Cornelia Huck
> ---
> hw/s390x/event-facility.c | 2 +-
> hw/s390x/s390-stattrib.c | 3 ++-
> 2 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/hw/s390x/event-fac
On 05/12/19 17:06, Thomas Huth wrote:
> diff --git a/include/hw/pci/pci.h b/include/hw/pci/pci.h
> index db75c6dfd0..5b6ebd15c6 100644
> --- a/include/hw/pci/pci.h
> +++ b/include/hw/pci/pci.h
> @@ -174,9 +174,6 @@ enum {
> #define QEMU_PCI_CAP_MULTIFUNCTION_BITNR3
> QEMU_PCI_CAP_MULT
On Fri, Dec 06, 2019 at 07:55:15AM +0800, Alex Williamson wrote:
> On Wed, 4 Dec 2019 22:26:38 -0500
> Yan Zhao wrote:
>
> > Vendor driver specifies when to support a migration region through cap
> > VFIO_PCI_DEVICE_CAP_MIGRATION in vfio_pci_mediate_ops->open().
> >
> > If vfio-pci detects this
On 12/6/19 2:42 PM, Damien Hedde wrote:
Remove the packet size upper limit by using a GByteArray
instead of a statically allocated array for last_packet.
Thus we can now send big packets.
Also remove the last_packet_len field and use last_packet->len
instead.
Signed-off-by: Damien Hedde
---
T
On 12/6/19 6:52 AM, David Gibson wrote:
On Thu, Dec 05, 2019 at 10:39:29AM +0530, Ganesh wrote:
On 11/19/19 8:15 AM, David Gibson wrote:
On Thu, Oct 24, 2019 at 01:13:06PM +0530, Ganesh Goudar wrote:
From: Aravinda Prasad
This patch includes migration support for machine check
handling. Es
On 11/29/19 11:30 AM, David Hildenbrand wrote:
> On 29.11.19 10:48, Janosch Frank wrote:
>> We need to know if we run in pv state or not when emulating
>> instructions.
>>
>> Signed-off-by: Janosch Frank
>> ---
>> hw/s390x/s390-virtio-ccw.c | 2 ++
>> target/s390x/cpu.h | 1 +
>> 2 files
Hi Paolo and Eduardo
I digged into the put msr assertion bug a little more, and seems I
found the root cause.
Please have a review.
Best regards.
Catherine
On Fri, 6 Dec 2019 at 18:25, Catherine Ho wrote:
>
> Commit 1389309c811b ("KVM: nVMX: expose VMX capabilities for nested
> hypervisors to us
The following changes since commit 1bdc319ab5d289ce6b822e06fb2b13666fd9278e:
Update version for v4.2.0-rc4 release (2019-12-03 17:56:30 +)
are available in the Git repository at:
git://github.com/bonzini/qemu.git tags/for-upstream
for you to fetch changes up to 2605188240f939fa9ae9353f5
On 2019-12-06 14:13, Peter Maydell wrote:
On Sun, 1 Dec 2019 at 12:20, Marc Zyngier wrote:
Hi all,
This series is a follow-up on [1], which tried to address the
remaining missing HCR_EL2.TIDx traps. I've hopefully now adressed
the
comments that Peter and Edgar raised.
I've also tried to ta
On Mon, 2 Dec 2019 at 21:10, Niek Linnenbank wrote:
>
> This change ensures that the FPU can be accessed in Non-Secure mode
> when the CPU core is reset using the arm_set_cpu_on() function call.
> The NSACR.{CP11,CP10} bits define the exception level required to
> access the FPU in Non-Secure mode
On Fri, Dec 06, 2019 at 07:55:19AM +0800, Alex Williamson wrote:
> On Wed, 4 Dec 2019 22:25:36 -0500
> Yan Zhao wrote:
>
> > when vfio-pci is bound to a physical device, almost all the hardware
> > resources are passthroughed.
> > Sometimes, vendor driver of this physcial device may want to medi
On Mon, 2 Dec 2019 at 15:58, Richard Henderson
wrote:
>
> On 12/1/19 12:20 PM, Marc Zyngier wrote:
> > +if (cpu_isar_feature(jazelle, cpu)) {
> > +ARMCPRegInfo jazelle_regs[] = {
>
> static const.
If this can be static const we should just declare it
at file scope. The only arrays we
On 12/6/19 9:29 AM, Cornelia Huck wrote:
> On Fri, 6 Dec 2019 08:44:52 +0100
> Janosch Frank wrote:
>
>> On 12/5/19 6:46 PM, Cornelia Huck wrote:
>>> On Thu, 5 Dec 2019 18:34:32 +0100
>>> Janosch Frank wrote:
>>>
On 12/5/19 6:15 PM, Cornelia Huck wrote:
> On Fri, 29 Nov 2019 04:48:
On Sat, 30 Nov 2019 at 03:47, Guoheyi wrote:
>
> Hi Peter, Igor,
>
> I couldn't find these 2 patches in the latest tree. Could you help to
> merge them?
In future I recommend pinging unapplied patches with a shorter
delay than nine months :-) In QEMU's process, unless somebody
has specifically s
>
> > > > The following changes since commit
> > > > 1bdc319ab5d289ce6b822e06fb2b13666fd9278e:
> > > >
> > > > Update version for v4.2.0-rc4 release (2019-12-03 17:56:30 +)
> > > >
> > > > are available in the Git repos
On 12/6/19 2:54 PM, Cornelia Huck wrote:
We neglected to clean up pending interrupts and emergency signals;
fix that.
Signed-off-by: Cornelia Huck
---
v1->v2:
- rebased on top of my s390-next branch; we can now move the fields
to be reset instead of clearing them manually
Yep, much cleane
On Mon, 2 Dec 2019 at 21:10, Niek Linnenbank wrote:
>
> The Security Identifier device in Allwinner H3 System on Chip
> gives applications a per-board unique identifier. This commit
> adds support for the Allwinner H3 Security Identifier using
> randomized data as input.
If this is a fixed value
On 12/5/19 6:46 PM, Vladimir Sementsov-Ogievskiy wrote:
We don't need Error **, as all callers pass local Error object, which
isn't used after the call. Use Error * instead.
Good idea.
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Vladimir Sementsov-Ogievskiy
Acked-by: Dr. David Alan
On Wed, Nov 27, 2019 at 02:10:38PM +0400, Marc-André Lureau wrote:
> Use int.from_bytes() from python 3.2 instead.
>
> Signed-off-by: Marc-André Lureau
> ---
> scripts/analyze-migration.py | 35 +++
> 1 file changed, 19 insertions(+), 16 deletions(-)
>
> diff --g
)
> >
> > are available in the Git repository at:
> >
> > git://github.com/dgibson/qemu.git tags/ppc-for-4.2-20191206
> >
> > for you to fetch changes up to d887a8cfc083bcf3856f9f42352989f3624e0cae:
> >
On Fri, 6 Dec 2019 08:44:52 +0100
Janosch Frank wrote:
> On 12/5/19 6:46 PM, Cornelia Huck wrote:
> > On Thu, 5 Dec 2019 18:34:32 +0100
> > Janosch Frank wrote:
> >
> >> On 12/5/19 6:15 PM, Cornelia Huck wrote:
> >>> On Fri, 29 Nov 2019 04:48:02 -0500
> >>> Janosch Frank wrote:
> >>>
On 06.12.19 14:54, Cornelia Huck wrote:
> We neglected to clean up pending interrupts and emergency signals;
> fix that.
>
> Signed-off-by: Cornelia Huck
> ---
>
> v1->v2:
> - rebased on top of my s390-next branch; we can now move the fields
> to be reset instead of clearing them manually
>
>
If kvm does not support VMX feature by nested=0, the kvm_vmx_basic
can't get the right value from MSR_IA32_VMX_BASIC register, which
make qemu coredump when qemu do KVM_SET_MSRS.
The coredump info:
error: failed to set MSR 0x480 to 0x0
kvm_put_msrs: Assertion `ret == cpu->kvm_msr_buf->nmsrs' faile
Vladimir Sementsov-Ogievskiy writes:
> 30.11.2019 22:42, Markus Armbruster wrote:
>> Cc: "Daniel P. Berrangé"
>> Signed-off-by: Markus Armbruster
>> ---
>> include/io/task.h | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/include/io/task.h b/include/io/task.h
>>
On Monday, November 25, 2019, Filip Bozuta wrote:
> The script checkpatch.pl located in scripts folder was
> used to detect all errors and warrnings in files:
> hw/mips/mips_mipssim.c
> hw/net/mipsnet.c
>
> All these mips mipssim machine files were edited and
> all the errors and warrings
05.12.2019 23:09, Eric Blake wrote:
> On 12/5/19 1:30 PM, Vladimir Sementsov-Ogievskiy wrote:
>> Here is double bug:
>>
>> First, return error but not set errp. This may lead to:
>> qmp block-dirty-bitmap-remove may report success when actually failed
>>
>> block-dirty-bitmap-remove used in a trans
On 12/5/19 6:46 PM, Vladimir Sementsov-Ogievskiy wrote:
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: Philippe Mathieu-Daudé
---
hw/i386/amd_iommu.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/hw/i386/amd_iommu.c b/hw/i386/amd_iommu.c
ind
Mention that this is a PCI device address & give the format it is
expected it. Also mention that it must be first unbound from any
host kernel driver.
Signed-off-by: Daniel P. Berrangé
---
qapi/block-core.json | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/qapi/block-co
Instead of filling an array of qemu_irq and passing it around,
directly call qdev_get_gpio_in() on the GIC.
Signed-off-by: Philippe Mathieu-Daudé
---
I accept better patch subject suggestions :)
---
hw/arm/sbsa-ref.c | 58 +++
1 file changed, 29 insert
On 12/5/19 6:46 PM, Vladimir Sementsov-Ogievskiy wrote:
Mostly, Error ** is for returning error from the function, so the
callee sets it. However these three functions get already filled errp
parameter. They dont change the pointer itself, only change the
internal state of referenced Error object
On 12/5/19 6:46 PM, Vladimir Sementsov-Ogievskiy wrote:
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: Stefan Berger
Reviewed-by: Philippe Mathieu-Daudé
---
hw/tpm/tpm_emulator.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/hw/tpm/tpm_emulator.c
Eric Blake writes:
> On 12/5/19 8:48 AM, Vladimir Sementsov-Ogievskiy wrote:
>
@@ -918,27 +917,26 @@ static void device_set_realized(Object *obj, bool
value, Error **errp)
}
} else if (!value && dev->realized) {
-Error **local_errp = NULL;
>>>
On 12/5/19 6:46 PM, Vladimir Sementsov-Ogievskiy wrote:
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: Philippe Mathieu-Daudé
---
monitor/qmp-cmds.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/monitor/qmp-cmds.c b/monitor/qmp-cmds.c
index 0880341a2d..c6f
On Fri, Dec 06, 2019 at 02:38:11PM +, Daniel P. Berrangé wrote:
> Mention that this is a PCI device address & give the format it is
> expected it. Also mention that it must be first unbound from any
> host kernel driver.
>
> Signed-off-by: Daniel P. Berrangé
> ---
> qapi/block-core.json | 6
On Fri, 6 Dec 2019 10:36:40 +0100
David Hildenbrand wrote:
> On 05.12.19 11:38, Cornelia Huck wrote:
> > We neglected to clean up pending interrupts and emergency signals;
> > fix that.
> >
> > Signed-off-by: Cornelia Huck
> > ---
> >
> > Noted while looking at the fixes for the kvm reset hand
On Thursday, December 5, 2019, Cleber Rosa wrote:
> On Mon, Dec 02, 2019 at 05:49:58PM -0300, Eduardo Habkost wrote:
> > On Sun, Dec 01, 2019 at 12:46:12AM +0100, Aleksandar Markovic wrote:
> > > On Monday, November 25, 2019, Filip Bozuta
> wrote:
> > >
> > > > The script checkpatch.pl located i
On 06/12/2019 07.49, Markus Armbruster wrote:
> Thomas Huth writes:
>
>> These have been on the deprecation list since a year now, so it's
>> time to finally remove the pc-0.x machine types.
>>
>> We then can also remove some compatibility hacks in the devices, i.e.
>> the "use_broken_id" in ac97
On Fri, 11 Oct 2019 at 14:50, Richard Henderson
wrote:
>
> The process by which one goes from an address space plus physical
> address to a host pointer is complex. It is easiest to reuse the
> mechanism already present within cputlb, and letting that cache
> the results.
>
> Signed-off-by: Richa
On Fri, Dec 06, 2019 at 07:55:30AM +0800, Alex Williamson wrote:
> On Wed, 4 Dec 2019 22:26:50 -0500
> Yan Zhao wrote:
>
> > Dynamic trap bar info region is a channel for QEMU and vendor driver to
> > communicate dynamic trap info. It is of type
> > VFIO_REGION_TYPE_DYNAMIC_TRAP_BAR_INFO and sub
object_dynamic_cast() should return machine (or GPIO) state instad of NMI
state in do_nmi(). So it's wrong to convert it to NMI state unconditionally.
This changes the prototype of NMIClass::nmi_monitor_handler() to accept
the parent object of NMI state instead of itself. With this, he parent obje
On Tue, Nov 26, 2019 at 11:38:09PM +0100, Philippe Mathieu-Daudé wrote:
> The kernel packaged was fetched from an unstable repository.
> Use the stable snapshot archive instead.
>
> Reviewed-by: Alex Bennée
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> v2:
> - Rebased
> - Dropped Alex's Tested
On 06/12/19 11:23, Catherine Ho wrote:
> Commit 1389309c811b ("KVM: nVMX: expose VMX capabilities for nested
> hypervisors to userspace") expands the msr_based_features with
> MSR_IA32_VMX_BASIC and others. Then together with an old kernel before
> 1389309c811b, the qemu call KVM_GET_MSR_FEATURE_IN
On 05.12.19 11:38, Cornelia Huck wrote:
> We neglected to clean up pending interrupts and emergency signals;
> fix that.
>
> Signed-off-by: Cornelia Huck
> ---
>
> Noted while looking at the fixes for the kvm reset handling.
>
> We now clear some fields twice in the paths for clear or initial r
On 2019/12/6 下午4:22, Yan Zhao wrote:
On Thu, Dec 05, 2019 at 09:05:54PM +0800, Jason Wang wrote:
On 2019/12/5 下午4:51, Yan Zhao wrote:
On Thu, Dec 05, 2019 at 02:33:19PM +0800, Jason Wang wrote:
Hi:
On 2019/12/5 上午11:24, Yan Zhao wrote:
For SRIOV devices, VFs are passthroughed into guest di
On 12/5/19 6:46 PM, Vladimir Sementsov-Ogievskiy wrote:
Make qbus_list_bus and qbus_list_dev hint append helpers well formed:
switch errp paramter to Error *const * type, as it has uncommon
"parameter"
behavior: not change the pointer to return error, but operate on
already existent error obj
Vladimir Sementsov-Ogievskiy writes:
> 30.11.2019 22:42, Markus Armbruster wrote:
>> fit_load_fdt() recovers from fit_image_addr() failing with ENOENT.
>> Except it doesn't when its @errp argument is &error_fatal or
>> &error_abort, because it blindly passes @errp to fit_image_addr().
>> Messed u
On 12/3/19 3:48 AM, Peter Maydell wrote:
>> +{ .name = "GMID_EL1", .state = ARM_CP_STATE_AA64,
>> + .opc0 = 3, .opc1 = 1, .crn = 0, .crm = 0, .opc2 = 4,
>> + .access = PL1_R, .type = ARM_CP_CONST, .resetvalue = GMID_EL1_BS },
>
> This should trap if HCR_EL2.TID5 is 1 (since we're add
On 12/5/19 6:46 PM, Vladimir Sementsov-Ogievskiy wrote:
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: Cornelia Huck
Reviewed-by: Philippe Mathieu-Daudé
---
hw/s390x/event-facility.c | 2 +-
hw/s390x/s390-stattrib.c | 3 ++-
2 files changed, 3 insertions(+), 2 deletions(-)
On 12/6/19 8:38 AM, Daniel P. Berrangé wrote:
Mention that this is a PCI device address & give the format it is
expected it. Also mention that it must be first unbound from any
maybe s/expected it/expecting/
host kernel driver.
Signed-off-by: Daniel P. Berrangé
---
qapi/block-core.json |
[adding in Peter Maydell, as there is now potential talk of other
4.2-worthy patches]
On 12/6/19 4:18 AM, Vladimir Sementsov-Ogievskiy wrote:
05.12.2019 23:16, John Snow wrote:
On 12/5/19 3:09 PM, Eric Blake wrote:
On 12/5/19 1:30 PM, Vladimir Sementsov-Ogievskiy wrote:
Here is double bug:
On 12/5/19 6:46 PM, Vladimir Sementsov-Ogievskiy wrote:
Signed-off-by: Vladimir Sementsov-Ogievskiy
---
hw/usb/dev-network.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/usb/dev-network.c b/hw/usb/dev-network.c
index 5de6213cc4..b81a8abe83 100644
--- a/hw/usb/dev-net
On Thu, Dec 05, 2019 at 12:01:56PM +0100, Philippe Mathieu-Daudé wrote:
>
> Understood, thanks for clearing this out!
>
> Side note, we don't do cross-arch migration testing, but we talked about
> having a 'different QEMU version' migration test. When we get a such test
> setup, it shouldn't be t
The @cpu_option shouldn't be NULL, otherwise assertion from g_strsplit()
should be raised as below message indicates. So it's meaningless to validate
@model_pices[0] in parse_cpu_option() as it shouldn't be NULL either.
qemu-system-aarch64: GLib: g_strsplit: assertion 'string != NULL' failed
T
On 12/5/19 5:06 PM, Thomas Huth wrote:
They can't be used reliable for live-migration anymore (see
https://lists.gnu.org/archive/html/qemu-devel/2018-12/msg04516.html
"reliably"?
I'd keep the reference, but also paste Paolo's 7 lines here.
Reviewed-by: Philippe Mathieu-Daudé
for details) a
Commit 1389309c811b ("KVM: nVMX: expose VMX capabilities for nested
hypervisors to userspace") expands the msr_based_features with
MSR_IA32_VMX_BASIC and others. Then together with an old kernel before
1389309c811b, the qemu call KVM_GET_MSR_FEATURE_INDEX_LIST and got the
smaller kvm_feature_msrs.
Vladimir Sementsov-Ogievskiy writes:
> 05.12.2019 18:26, Cornelia Huck wrote:
>> On Thu, 5 Dec 2019 18:19:58 +0300
>> Vladimir Sementsov-Ogievskiy wrote:
>>
>>> Hi all!
>>>
>>> This is the first part of the bit series, which contains mostly simple
>>> cleanups.
>>
>> What's the plan? Should s
1 - 100 of 271 matches
Mail list logo
