Re: [PATCH] PoC: Rust binding for QAPI (qemu-ga only, for now)

2020-09-11 Thread Marc-André Lureau
Hi On Fri, Sep 11, 2020 at 2:47 PM Daniel P. Berrangé wrote: > On Thu, Sep 10, 2020 at 09:48:50PM +0400, marcandre.lur...@redhat.com > wrote: > > From: Marc-André Lureau > > > > Hi, > > > > Among the QEMU developers, there is a desire to use Rust. (see previous > > thread from Stefan "Why QEMU

Re: [PULL v2 01/46] qemu-iotests: move check-block back to Makefiles

2020-09-11 Thread Paolo Bonzini
On 11/09/20 13:05, Max Reitz wrote: > On closer inspection it seems like it’s because of the > “build_by_default: false”, which seems like a rather conscious decision. > Was I only lucky that the socket_scm_helper was built by default so > far? Should I have explicitly built it all this time?

Re: [PULL v2 01/46] qemu-iotests: move check-block back to Makefiles

2020-09-11 Thread Paolo Bonzini
On 11/09/20 12:58, Max Reitz wrote: > It seems like the socket_scm_helper is now only built as a dependency of > check-block, instead of all the time. That’s a bit of a shame. > (The obvious workaround of course is to specifically build the > socket_scm_helper, but that doesn’t seem right.) Or

Re: [PATCH 11/14] block/qcow2-bitmap: return startus from qcow2_store_persistent_dirty_bitmaps

2020-09-11 Thread Vladimir Sementsov-Ogievskiy
11.09.2020 14:21, Greg Kurz wrote: On Fri, 11 Sep 2020 13:18:32 +0300 Vladimir Sementsov-Ogievskiy wrote: 11.09.2020 12:38, Greg Kurz wrote: s/startus/status On Wed, 9 Sep 2020 21:59:27 +0300 Vladimir Sementsov-Ogievskiy wrote: It's better to return status together with setting errp. It

Re: [Bug 1895080] [NEW] pgb_reserved_va: Assertion `addr == test' failed

2020-09-11 Thread Hansni Bu
> > No, it's not set by CentOS-7.5. > > Does it mean that we just cannot run the ELF in such a case? I've tried > > many times, the assert always fails. Maybe, we can blame CentOS-7.5. > > The trouble is without MAP_FIXED_NOREPLACE we are at the mercy of the > host kernel to allow the address

Re: [PATCH] cphp: remove deprecated cpu-add command(s)

2020-09-11 Thread Thomas Huth
What does cphp in the subject mean? On 11/09/2020 14.33, Igor Mammedov wrote: > theses were deprecatedince since 4.0, remove both HMP and QMP variants. deprecated since > Users should use device_add commnad instead. To get list of command > possible CPUs and options, use 'info

Re: [PATCH] hw/arm/pxa2xx: Add missing fallthrough comment

2020-09-11 Thread Laurent Vivier
Le 11/09/2020 à 14:41, Philippe Mathieu-Daudé a écrit : > On 9/11/20 2:18 PM, Thomas Huth wrote: >> Let's make this file compilable with -Werror=implicit-fallthrough : >> Looking at the code, it seems like the fallthrough is intended here, >> so we should add the corresponding "/* fallthrough */"

Re: [PULL v2 0/8] Tracing patches

2020-09-11 Thread Peter Maydell
On Thu, 10 Sep 2020 at 09:11, Stefan Hajnoczi wrote: > > The following changes since commit 9435a8b3dd35f1f926f1b9127e8a906217a5518a: > > Merge remote-tracking branch > 'remotes/kraxel/tags/sirius/ipxe-20200908-pull-request' into staging > (2020-09-08 21:21:13 +0100) > > are available in the

Re: [PATCH] target/arm: Count PMU events when MDCR.SPME is set

2020-09-11 Thread Peter Maydell
On Tue, 25 Aug 2020 at 15:49, Aaron Lindsay wrote: > > This check was backwards when introduced in commit > 033614c47de78409ad3fb39bb7bd1483b71c6789: > > target/arm: Filter cycle counter based on PMCCFILTR_EL0 > > Signed-off-by: Aaron Lindsay > --- > target/arm/helper.c | 2 +- > 1 file

Re: [PATCH] PoC: Rust binding for QAPI (qemu-ga only, for now)

2020-09-11 Thread John Snow
On 9/11/20 7:28 AM, Paolo Bonzini wrote: On 11/09/20 12:46, Daniel P. Berrangé wrote: Do we actually need/want it to be in the same monolithic repo as qemu, as opposed to a qemu-qapi-rust repo ? I think QAPI and qemu-ga should be separate repos altogether. QAPI should be included as a

Re: [PATCH] PoC: Rust binding for QAPI (qemu-ga only, for now)

2020-09-11 Thread Marc-André Lureau
Hi On Thu, Sep 10, 2020 at 9:49 PM wrote: > > Shameful pain point: meson & cargo don't play nicely together. > I realize you need to configure with --enable-debug if you try this patch, or it will fail to find the library location. (eh, it's really a PoC!). Obviously, tested on a Linux/Fedora

[PATCH] seccomp: fix killing of whole process instead of thread

2020-09-11 Thread Daniel P . Berrangé
Back in 2018 we introduced support for killing the whole QEMU process instead of just one thread, when a seccomp rule is violated: commit bda08a5764d470f101fa38635d30b41179a313e1 Author: Marc-André Lureau Date: Wed Aug 22 19:02:48 2018 +0200 seccomp: prefer SCMP_ACT_KILL_PROCESS if

[RFC PATCH] linux-user: test, don't assert addr != test in pgb_reserved_va

2020-09-11 Thread Alex Bennée
On older kernels which don't implement MAP_FIXED_NOREPLACE the kernel may still fail to give us the address we asked for despite having already probed the map for a valid hole. Asserting isn't particularly useful to the user so let us move the check up and expand the error_report a little to give

Re: PATCH: Increase System Firmware Max Size

2020-09-11 Thread Dr. David Alan Gilbert
* Laszlo Ersek (ler...@redhat.com) wrote: > On 09/11/20 10:34, Dr. David Alan Gilbert wrote: > > * Laszlo Ersek (ler...@redhat.com) wrote: > >> +Markus, Dave, Phil > >> > >> On 09/11/20 03:45, McMillan, Erich wrote: > >>> Hi all, > >>> > >>> (this is my first Qemu patch submission, please let me

Re: [PATCH v5 8/8] s390: guest support for diagnose 0x318

2020-09-11 Thread Thomas Huth
On 10/09/2020 11.36, Collin Walling wrote: > DIAGNOSE 0x318 (diag318) is an s390 instruction that allows the storage > of diagnostic information that is collected by the firmware in the case > of hardware/firmware service events. > > QEMU handles the instruction by storing the info in the CPU

Re: About 'qemu-security' mailing list

2020-09-11 Thread Li Qiang
P J P 于2020年9月11日周五 下午10:21写道: > >Hello all, > > Recently while conversing with DanPB this point came up > > -> https://www.qemu.org/contribute/security-process/ > > * Currently QEMU security team is a handful of individual contacts which >restricts community participation in dealing

Re: About 'qemu-security' mailing list

2020-09-11 Thread Peter Maydell
On Fri, 11 Sep 2020 at 15:22, P J P wrote: > Proposal: (to address above limitations) > = > > * We set up a new 'qemu-security' mailing list. > > * QEMU security issues are reported to this new list only. > > * Representatives from various communities subscribe to this list. (List maybe >

Re: [PATCH] target/i386/kvm: Add missing fallthrough comment

2020-09-11 Thread Paolo Bonzini
On 11/09/20 14:53, Thomas Huth wrote: > Let's make this file compilable with -Werror=implicit-fallthrough : > Looking at the code, it seems like the fallthrough is intended here, > so we should add the corresponding "/* fallthrough */" comment here. > > Signed-off-by: Thomas Huth > --- >

[PATCH v1 2/2] colo-compare: Record packet creation time by QEMU_CLOCK_REALTIME

2020-09-11 Thread Derek Su
Record packet creation time by QEMU_CLOCK_REALTIME instead of QEMU_CLOCK_HOST. The time difference between `now` and packet `creation_ms` has the possibility of an unexpected negative value and results in wrong comparison after changing the host clock. Signed-off-by: Derek Su ---

[PATCH v1 1/2] colo-compare: Fix incorrect data type conversion

2020-09-11 Thread Derek Su
Fix data type conversion of compare_timeout. The incorrect conversion results in a random compare_timeout value and unexpected stalls in packet comparison. Signed-off-by: Derek Su --- net/colo-compare.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/net/colo-compare.c

RE: [PATCH v5 7/7] Versal: Connect DWC3 controller with virt-versal

2020-09-11 Thread Sai Pavan Boddu
Hi Edgar, Thanks, I would try to clean this up as per your comments. Regards, Sai Pavan > -Original Message- > From: Edgar E. Iglesias > Sent: Friday, September 11, 2020 1:33 PM > To: Sai Pavan Boddu > Cc: Peter Maydell ; Markus Armbruster > ; 'Marc-André Lureau' > ; Paolo Bonzini ; >

Re: [PATCH 07/14] block/blklogwrites: drop error propagation

2020-09-11 Thread Markus Armbruster
Greg Kurz writes: > On Fri, 11 Sep 2020 07:30:37 +0200 > Markus Armbruster wrote: [...] >> No, the patch is okay as is. >> >> Dumbing it down to keep it simple would also be okay. >> > > What about Ari's proposal to add ERRP_GUARD() and check errors > with "if (*errp)" like we do for void

Re: [PATCH v5 2/8] s390/sclp: rework sclp boundary checks

2020-09-11 Thread Collin Walling
On 9/11/20 6:24 AM, Cornelia Huck wrote: > On Thu, 10 Sep 2020 19:45:01 +0200 > Thomas Huth wrote: > >> On 10/09/2020 11.36, Collin Walling wrote: >>> Rework the SCLP boundary check to account for different SCLP commands >>> (eventually) allowing different boundary sizes. >>> >>> Signed-off-by:

Re: [PATCH] hw: usb: hcd-ohci: check len and frame_number variables

2020-09-11 Thread Li Qiang
P J P 于2020年9月11日周五 下午8:30写道: > > From: Prasad J Pandit > > While servicing the OHCI transfer descriptors(TD), OHCI host > controller derives variables 'start_addr', 'end_addr', 'len' > etc. from values supplied by the host controller driver. > Host controller driver may supply values such that

Re: PATCH: Increase System Firmware Max Size

2020-09-11 Thread Daniel P . Berrangé
On Fri, Sep 11, 2020 at 04:06:02PM +0100, Dr. David Alan Gilbert wrote: > * Laszlo Ersek (ler...@redhat.com) wrote: > > On 09/11/20 10:34, Dr. David Alan Gilbert wrote: > > > it doesn't have any pretty graphics > > > or snazzy stuff, > > > > Which is arguably completely superfluous on every

Re: About 'qemu-security' mailing list

2020-09-11 Thread Alexander Bulekov
Hi Prasad, A couple questions: * I'm guessing this will be a closed list with some application/vetting procedure for the participants? (Maybe this is what you mean by "moderated" ?) * How will the communication be encrypted? * Will secalert still be subscribed (for managing CVE ID

Re: [PATCH v3] i440fx/acpi: do not add hotplug related amls for cold plugged bridges

2020-09-11 Thread Michael S. Tsirkin
On Thu, Sep 10, 2020 at 12:19:19AM +0530, Ani Sinha wrote: > On Sep 10, 2020, 00:00 +0530, Ani Sinha , wrote: > > On Sep 9, 2020, 23:20 +0530, Julia Suvorova , wrote: > > On Fri, Sep 4, 2020 at 6:10 PM Ani Sinha wrote: > > > > Cold plugged bridges are not hot

Re: [PATCH v2 0/2] util/hexdump: Cleanup qemu_hexdump()

2020-09-11 Thread Laurent Vivier
Le 11/09/2020 à 16:35, Philippe Mathieu-Daudé a écrit : > Hi Laurent, > > On 8/22/20 8:09 PM, Philippe Mathieu-Daudé wrote: >> - Pass const void* buffer >> - Reorder arguments >> >> Supersedes: <20200822150457.1322519-1-f4...@amsat.org> >> >> Philippe Mathieu-Daudé (2): >> util/hexdump: Convert

Re: [PATCH v2 2/2] util/hexdump: Reorder qemu_hexdump() arguments

2020-09-11 Thread Laurent Vivier
Le 22/08/2020 à 20:09, Philippe Mathieu-Daudé a écrit : > qemu_hexdump()'s pointer to the buffer and length of the > buffer are closely related arguments but are widely separated > in the argument list order (also, the format of > function prototypes is usually to have the FILE* argument > coming

Re: About 'qemu-security' mailing list

2020-09-11 Thread Alexander Bulekov
And I forgot to mention that I think it is a great idea :) . Over the past couple months, I reported a few dozen bugs on launchpad. Even though many of them are memory-corruptions and might fall under the "security-bug" label, they could be fixed faster simply because the reports can reach the

Re: [PATCH v3] i440fx/acpi: do not add hotplug related amls for cold plugged bridges

2020-09-11 Thread Ani Sinha
On Fri, Sep 11, 2020 at 9:22 PM Michael S. Tsirkin wrote: > > On Thu, Sep 10, 2020 at 12:19:19AM +0530, Ani Sinha wrote: > > On Sep 10, 2020, 00:00 +0530, Ani Sinha , wrote: > > > > On Sep 9, 2020, 23:20 +0530, Julia Suvorova , wrote: > > > > On Fri, Sep 4, 2020 at 6:10 PM Ani Sinha

Re: PATCH: Increase System Firmware Max Size

2020-09-11 Thread Laszlo Ersek
On 09/11/20 17:23, Daniel P. Berrangé wrote: > I don't see why we should have this as a hard coded > limit that is not runtime configurable. > > IOW, why can't we keep our current default and provide a machine type > property "firmware_max_size" which users can opt-in to setting if > their

Re: [PATCH v3] i440fx/acpi: do not add hotplug related amls for cold plugged bridges

2020-09-11 Thread Ani Sinha
On Fri, Sep 11, 2020 at 9:39 PM Michael S. Tsirkin wrote: > > On Fri, Sep 11, 2020 at 09:35:42PM +0530, Ani Sinha wrote: > > On Fri, Sep 11, 2020 at 9:22 PM Michael S. Tsirkin wrote: > > > > > > On Thu, Sep 10, 2020 at 12:19:19AM +0530, Ani Sinha wrote: > > > > On Sep 10, 2020, 00:00 +0530, Ani

Re: [PATCH V1 01/32] savevm: add vmstate handler iterators

2020-09-11 Thread Dr. David Alan Gilbert
Apologies for taking a while to get around to this, * Steve Sistare (steven.sist...@oracle.com) wrote: > Provide the SAVEVM_FOREACH and SAVEVM_FORALL macros to loop over all save > VM state handlers. The former will filter handlers based on the operation > in the later patch "savevm: VM

Re: [PATCH] tests: add missing genh dependency

2020-09-11 Thread Claudio Fontana
On 9/11/20 3:42 PM, Paolo Bonzini wrote: > Fix high-parallelism builds by forcing all generated headers > to be created before tests are compiled. > > Reported-by: Claudio Fontana > Signed-off-by: Paolo Bonzini > --- > tests/meson.build | 4 ++-- > 1 file changed, 2 insertions(+), 2

[PATCH 0/2] Make preallocate_co() resize the image to the correct size

2020-09-11 Thread Alberto Garcia
preallocate_co() does not resize the image correctly if the original size was not cluster-aligned. This series should be applied on top of Max's block branch (I tested it with commit 8e66c829eda997dad661d49d73668b1fd3e6043d). https://git.xanclic.moe/XanClic/qemu/commits/branch/block Alberto

About 'qemu-security' mailing list

2020-09-11 Thread P J P
Hello all, Recently while conversing with DanPB this point came up -> https://www.qemu.org/contribute/security-process/ * Currently QEMU security team is a handful of individual contacts which restricts community participation in dealing with these issues. * The Onus also lies with the

Re: [PATCH] tests: add missing genh dependency

2020-09-11 Thread Philippe Mathieu-Daudé
On 9/11/20 3:42 PM, Paolo Bonzini wrote: > Fix high-parallelism builds by forcing all generated headers > to be created before tests are compiled. FYI I had this problem too (simply building with -j8) simply adding a new qapi/file.json. Reviewed-by: Philippe Mathieu-Daudé > > Reported-by:

Re: [PATCH] spapr: Handle HPT allocation failure in nested guest

2020-09-11 Thread Fabiano Rosas
Fabiano Rosas writes: > The nested KVM code does not yet support HPT guests. Calling the > KVM_CAP_PPC_ALLOC_HTAB ioctl currently leads to KVM setting the guest > as HPT and erroneously executing code in L1 that should only run in > hypervisor mode, leading to an exception in the L1 vcpu thread

Re: [PATCH] smp: drop support for deprecated (invalid topologies)

2020-09-11 Thread Michael S. Tsirkin
On Fri, Sep 11, 2020 at 09:32:02AM -0400, Igor Mammedov wrote: > it's was deprecated since 3.1 > > Support for invalid topologies is removed, the user must ensure > that topologies described with -smp include all possible cpus, > i.e. (sockets * cores * threads) == maxcpus or QEMU will > exit

Re: [PATCH v1 2/3] tests/acpi: unit test for 'acpi-pci-hotplug-with-bridge-support' bridge flag

2020-09-11 Thread Ani Sinha
On Fri, Sep 11, 2020 at 8:29 PM Michael S. Tsirkin wrote: > > On Thu, Sep 10, 2020 at 10:34:20PM +0530, Ani Sinha wrote: > > On Sep 5, 2020, 16:05 +0530, Ani Sinha , wrote: > > > > This change adds a new unit test for the global flag > > > > 'acpi-pci-hotplug-with-bridge-support' which is

Re: [PATCH v4 9/9] scsi/scsi_bus: fix races in REPORT LUNS

2020-09-11 Thread Stefan Hajnoczi
On Wed, Sep 09, 2020 at 11:20:24AM +0300, Maxim Levitsky wrote: > On Tue, 2020-09-08 at 16:27 +0100, Stefan Hajnoczi wrote: > > On Mon, Aug 31, 2020 at 06:01:24PM +0300, Maxim Levitsky wrote: > > > @@ -460,46 +466,36 @@ static bool > > > scsi_target_emulate_report_luns(SCSITargetReq *r) > > >

[Bug 1895310] [NEW] Heap-overflow (read) in sd_erase

2020-09-11 Thread Alexander Bulekov
Public bug reported: Hello, One more bug in sd ... cat << EOF | ./qemu-system-i386 -nodefaults \ -device sdhci-pci,sd-spec-version=3 \ -device sd-card,drive=mydrive \ -drive if=sd,index=0,file=null-co://,format=raw,id=mydrive \ -nographic -qtest stdio -m 64m -trace 'sd*' outl 0xcf8 0x80001003

Re: [PATCH] WHPX: vmware cpuid leaf for tsc and apic frequency

2020-09-11 Thread Paolo Bonzini
On 31/07/20 00:11, Sunil Muthuswamy wrote: > Newer versions of WHPX provide the capability to query the tsc > and apic frequency. Expose these through the vmware cpuid leaf. > This patch doesnt support setting the tsc frequency; that will > come as a separate fix. > > Signed-off-by: Sunil

Re: PATCH: Increase System Firmware Max Size

2020-09-11 Thread Laszlo Ersek
On 09/11/20 17:06, Dr. David Alan Gilbert wrote: > * Laszlo Ersek (ler...@redhat.com) wrote: >> On 09/11/20 10:34, Dr. David Alan Gilbert wrote: >>> I'm missing what this constant exists for - why is the >>> check there at all Is there something else that lives below this >>> address that we

Re: [PATCH v2 1/2] util/hexdump: Convert to take a void pointer argument

2020-09-11 Thread Laurent Vivier
Le 22/08/2020 à 20:09, Philippe Mathieu-Daudé a écrit : > Most uses of qemu_hexdump() do not take an array of char > as input, forcing use of cast. Since we can use this > helper to dump any kind of buffer, use a pointer to void > argument instead. > > Signed-off-by: Philippe Mathieu-Daudé > ---

Re: [PATCH v3] i440fx/acpi: do not add hotplug related amls for cold plugged bridges

2020-09-11 Thread Michael S. Tsirkin
On Fri, Sep 11, 2020 at 09:35:42PM +0530, Ani Sinha wrote: > On Fri, Sep 11, 2020 at 9:22 PM Michael S. Tsirkin wrote: > > > > On Thu, Sep 10, 2020 at 12:19:19AM +0530, Ani Sinha wrote: > > > On Sep 10, 2020, 00:00 +0530, Ani Sinha , wrote: > > > > > > On Sep 9, 2020, 23:20 +0530, Julia

Re: PATCH: Increase System Firmware Max Size

2020-09-11 Thread Dr. David Alan Gilbert
* Laszlo Ersek (ler...@redhat.com) wrote: > On 09/11/20 17:06, Dr. David Alan Gilbert wrote: > > * Laszlo Ersek (ler...@redhat.com) wrote: > >> On 09/11/20 10:34, Dr. David Alan Gilbert wrote: > > >>> I'm missing what this constant exists for - why is the > >>> check there at all Is there

Re: PATCH: Increase System Firmware Max Size

2020-09-11 Thread Daniel P . Berrangé
On Fri, Sep 11, 2020 at 06:06:27PM +0200, Laszlo Ersek wrote: > On 09/11/20 17:23, Daniel P. Berrangé wrote: > > > I don't see why we should have this as a hard coded > > limit that is not runtime configurable. > > > > IOW, why can't we keep our current default and provide a machine type > >

Re: [for-5.2 v4 09/10] host trust limitation: Alter virtio default properties for protected guests

2020-09-11 Thread Halil Pasic
On Fri, 11 Sep 2020 12:04:42 +1000 David Gibson wrote: > On Mon, Sep 07, 2020 at 05:10:46PM +0200, Halil Pasic wrote: > > On Fri, 24 Jul 2020 12:57:43 +1000 > > David Gibson wrote: > > > > > The default behaviour for virtio devices is not to use the platforms > > > normal > > > DMA paths, but

Re: [PATCH v5 6/8] s390/sclp: add extended-length sccb support for kvm guest

2020-09-11 Thread Thomas Huth
On 11/09/2020 15.41, Thomas Huth wrote: > On 10/09/2020 11.36, Collin Walling wrote: >> As more features and facilities are added to the Read SCP Info (RSCPI) >> response, more space is required to store them. The space used to store >> these new features intrudes on the space originally used to

Re: [PATCH 00/18] chardev: QOM cleanups

2020-09-11 Thread Eduardo Habkost
On Fri, Sep 11, 2020 at 09:10:18AM +0100, Daniel P. Berrangé wrote: > On Fri, Sep 11, 2020 at 12:07:27PM +0400, Marc-André Lureau wrote: > > Hi > > > > On Thu, Sep 10, 2020 at 11:50 PM Eduardo Habkost > > wrote: > > > > > Some chardev QOM cleanup patches had to be dropped from my queue > > >

Re: [PATCH v2 1/2] Introduce (x86) CPU model deprecation API

2020-09-11 Thread Eduardo Habkost
On Fri, Sep 11, 2020 at 02:22:51PM +0800, Robert Hoo wrote: > On Wed, 2020-09-09 at 14:15 -0400, Eduardo Habkost wrote: > > Hi, > > > > Thanks for the patch, and sorry for taking so long to review > > this. I'm finally getting to the patches that were postponed to > > 5.2. > > > > Comments and

Re: [PATCH] PoC: Rust binding for QAPI (qemu-ga only, for now)

2020-09-11 Thread Marc-André Lureau
Hi On Fri, Sep 11, 2020 at 5:08 PM Paolo Bonzini wrote: > On 11/09/20 12:24, Paolo Bonzini wrote: > >> > >> - from_qemu_none(ptr: *const sys::P) -> T > >> Return a Rust type T for a const ffi pointer P. > >> > >> - from_qemu_full(ptr: *mut sys::P) -> T > >> Return a Rust type T for a ffi

[Bug 1895305] [NEW] pthread_cancel fails with "RT33" with musl libc

2020-09-11 Thread Luca Weiss
Public bug reported: >From my testing it seems that QEMU built against musl libc crashes on pthread_cancel cancel calls - if the binary is also built with musl libc. Minimal sample: #include #include #include void* threadfunc(void* ignored) { while (1) { pause();

Re: [PATCH] cphp: remove deprecated cpu-add command(s)

2020-09-11 Thread Dr. David Alan Gilbert
* Igor Mammedov (imamm...@redhat.com) wrote: > theses were deprecatedince since 4.0, remove both HMP and QMP variants. > > Users should use device_add commnad instead. To get list of > possible CPUs and options, use 'info hotpluggable-cpus' HMP > or query-hotpluggable-cpus QMP command. > >

Re: [Bug 1895080] [NEW] pgb_reserved_va: Assertion `addr == test' failed

2020-09-11 Thread Alex Bennée
Hansni Bu <1895...@bugs.launchpad.net> writes: >> > No, it's not set by CentOS-7.5. >> > Does it mean that we just cannot run the ELF in such a case? I've tried >> > many times, the assert always fails. Maybe, we can blame CentOS-7.5. >> >> The trouble is without MAP_FIXED_NOREPLACE we are at

Re: PATCH: Increase System Firmware Max Size

2020-09-11 Thread Laszlo Ersek
On 09/11/20 10:34, Dr. David Alan Gilbert wrote: > * Laszlo Ersek (ler...@redhat.com) wrote: >> +Markus, Dave, Phil >> >> On 09/11/20 03:45, McMillan, Erich wrote: >>> Hi all, >>> >>> (this is my first Qemu patch submission, please let me know if my >>> formatting/content needs to be fixed). >>>

[Bug 1895305] Re: pthread_cancel fails with "RT33" with musl libc

2020-09-11 Thread Laurent Vivier
** Tags added: linux-user -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1895305 Title: pthread_cancel fails with "RT33" with musl libc Status in QEMU: New Bug description: From my testing it

Re: [PATCH v2 0/2] Enable virtio-fs on s390x

2020-09-11 Thread Stefan Hajnoczi
On Tue, Sep 01, 2020 at 05:00:17PM +0200, Marc Hartmayer wrote: > This patch series is about enabling virtio-fs on s390x. For that we need > + some shim code (first patch), and we need > + libvhost-user to deal with virtio endiannes for non-legacy virtio >devices as mandated by the spec.

Re: [PATCH] PoC: Rust binding for QAPI (qemu-ga only, for now)

2020-09-11 Thread Paolo Bonzini
On 11/09/20 16:00, Marc-André Lureau wrote: > - from_qemu_none should be a "to_*" or "constructor" conversion (I used > new_from_foreign) > > new_ prefix is not very rusty. Right, I have changed it to with_foreign so now there is {as,with,to,into}_foreign, plus unsafe_{from,into}. These

Re: [PATCH 11/14] block/qcow2-bitmap: return startus from qcow2_store_persistent_dirty_bitmaps

2020-09-11 Thread Markus Armbruster
Vladimir Sementsov-Ogievskiy writes: > 11.09.2020 14:21, Greg Kurz wrote: >> On Fri, 11 Sep 2020 13:18:32 +0300 >> Vladimir Sementsov-Ogievskiy wrote: >> >>> 11.09.2020 12:38, Greg Kurz wrote: s/startus/status On Wed, 9 Sep 2020 21:59:27 +0300 Vladimir Sementsov-Ogievskiy

Re: PATCH: Increase System Firmware Max Size

2020-09-11 Thread McMillan, Erich
Hi Dave, and Laszlo, I'm not exactly following your email '>' formatting on this discussion so apologies if I've messed it up. From: Dr. David Alan Gilbert Sent: Friday, September 11, 2020 10:06 AM To: Laszlo Ersek Cc: McMillan, Erich ; qemu-devel@nongnu.org

Re: [PATCH v1 0/3] unit tests for change 'do not add hotplug related amls for cold plugged bridges'

2020-09-11 Thread Michael S. Tsirkin
I am not sure why, but the expected files did not match for me. I dropped these for now: tests/acpi: add a new ACPI table in order to test root pci hotplug on/off tests/acpi: add a new unit test to test hotplug off/on feature on the root pci bus tests/acpi: document addition of table

Re: PATCH: Increase System Firmware Max Size

2020-09-11 Thread Laszlo Ersek
On 09/11/20 17:22, McMillan, Erich wrote: > I agree that fw has become the vendor OS, but at this point there's no > going back. > Utilizing a virtual platform allows us to greatly increase the security > of our code, > could we make this change a Qemu experimental flag, so that fw vendors could

Re: [PATCH] target/i386/kvm: Add missing fallthrough comment

2020-09-11 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200911125301.413081-1-th...@redhat.com/ Hi, This series failed build test on FreeBSD host. Please find the details below. === TEST SCRIPT BEGIN === #!/bin/bash # Testing script will be invoked under the git checkout with # HEAD pointing to a commit that

[PATCH v1 0/2] colo-compare bugfixes

2020-09-11 Thread Derek Su
Hello, The fixes are for the bugs found in colo-compare during our testing and applications. Please help to review, thanks a lot. Regards, Derek Su Derek Su (2): colo-compare: Fix incorrect data type conversion colo-compare: Record packet creation time by QEMU_CLOCK_REALTIME

[PATCH 1/2] qcow2: Make preallocate_co() resize the image to the correct size

2020-09-11 Thread Alberto Garcia
This function preallocates metadata structures and then extends the image to its new size, but that new size calculation is wrong because it doesn't take into account that the host_offset variable is always cluster-aligned. This problem can be reproduced with preallocation=metadata when the

[PATCH 2/2] qcow2: Convert qcow2_alloc_cluster_offset() into qcow2_alloc_host_offset()

2020-09-11 Thread Alberto Garcia
qcow2_alloc_cluster_offset() takes an (unaligned) guest offset and returns the (aligned) offset of the corresponding cluster in the qcow2 image. In practice none of the callers need to know where the cluster starts so this patch makes the function calculate and return the final host offset

Re: [PATCH v2 0/2] util/hexdump: Cleanup qemu_hexdump()

2020-09-11 Thread Philippe Mathieu-Daudé
Hi Laurent, On 8/22/20 8:09 PM, Philippe Mathieu-Daudé wrote: > - Pass const void* buffer > - Reorder arguments > > Supersedes: <20200822150457.1322519-1-f4...@amsat.org> > > Philippe Mathieu-Daudé (2): > util/hexdump: Convert to take a void pointer argument > util/hexdump: Reorder

Re: [PATCH v5 6/8] s390/sclp: add extended-length sccb support for kvm guest

2020-09-11 Thread Collin Walling
On 9/11/20 9:54 AM, Thomas Huth wrote: > On 11/09/2020 15.41, Thomas Huth wrote: >> On 10/09/2020 11.36, Collin Walling wrote: >>> As more features and facilities are added to the Read SCP Info (RSCPI) >>> response, more space is required to store them. The space used to store >>> these new

Re: [PATCH v1 2/3] tests/acpi: unit test for 'acpi-pci-hotplug-with-bridge-support' bridge flag

2020-09-11 Thread Michael S. Tsirkin
On Thu, Sep 10, 2020 at 10:34:20PM +0530, Ani Sinha wrote: > On Sep 5, 2020, 16:05 +0530, Ani Sinha , wrote: > > This change adds a new unit test for the global flag > > 'acpi-pci-hotplug-with-bridge-support' which is available for cold plugged > pci > > bridges in i440fx. The

Re: [PATCH v5 8/8] s390: guest support for diagnose 0x318

2020-09-11 Thread Thomas Huth
On 11/09/2020 17.08, Thomas Huth wrote: > On 10/09/2020 11.36, Collin Walling wrote: >> DIAGNOSE 0x318 (diag318) is an s390 instruction that allows the storage >> of diagnostic information that is collected by the firmware in the case >> of hardware/firmware service events. >> >> QEMU handles the

Re: [PATCH] qemu-img: Support bitmap --merge into backing image

2020-09-11 Thread Eric Blake
On 9/11/20 3:31 AM, Max Reitz wrote: On 09.09.20 14:33, Eric Blake wrote: If you have the chain 'base.qcow2 <- top.qcow2' and want to merge a bitmap from top into base, qemu-img was failing with: qemu-img: Could not open 'top.qcow2': Could not open backing file: Failed to get shared "write"

Re: [PATCH] WHPX: vmware cpuid leaf for tsc and apic frequency

2020-09-11 Thread Paolo Bonzini
On 01/09/20 01:32, Sunil Muthuswamy wrote: > Can I please get some eyes on this? Yes, sorry about the bad timing (5.1 freeze + Meson conversion + parental leave). Paolo >> -Original Message- >> From: Sunil Muthuswamy >> Sent: Thursday, July 30, 2020 3:11 PM >> To: Paolo Bonzini ;

Re: About 'qemu-security' mailing list

2020-09-11 Thread Daniel P . Berrangé
On Fri, Sep 11, 2020 at 07:50:24PM +0530, P J P wrote: > Hello all, > > Recently while conversing with DanPB this point came up > >-> https://www.qemu.org/contribute/security-process/ > > * Currently QEMU security team is a handful of individual contacts which > restricts community

Re: [PATCH v1 0/3] unit tests for change 'do not add hotplug related amls for cold plugged bridges'

2020-09-11 Thread Ani Sinha
On Fri, Sep 11, 2020 at 9:38 PM Michael S. Tsirkin wrote: > > > I am not sure why, but the expected files did not match for me. > > I dropped these for now: > > tests/acpi: add a new ACPI table in order to test root pci hotplug on/off > tests/acpi: add a new unit test to test hotplug off/on

Re: [PATCH v3] i440fx/acpi: do not add hotplug related amls for cold plugged bridges

2020-09-11 Thread Ani Sinha
On Fri, Sep 11, 2020 at 9:45 PM Ani Sinha wrote: > > On Fri, Sep 11, 2020 at 9:39 PM Michael S. Tsirkin wrote: > > > > On Fri, Sep 11, 2020 at 09:35:42PM +0530, Ani Sinha wrote: > > > On Fri, Sep 11, 2020 at 9:22 PM Michael S. Tsirkin > > > wrote: > > > > > > > > On Thu, Sep 10, 2020 at

Re: [PATCH v1 0/3] unit tests for change 'do not add hotplug related amls for cold plugged bridges'

2020-09-11 Thread Ani Sinha
I can't repro the breakage. What test command line are you running with? I am using " make check-qtest-x86_64 V=1" On Fri, Sep 11, 2020 at 9:41 PM Ani Sinha wrote: > > On Fri, Sep 11, 2020 at 9:38 PM Michael S. Tsirkin wrote: > > > > > > I am not sure why, but the expected files did not match

Re: [PATCH] hw: usb: hcd-ohci: check len and frame_number variables

2020-09-11 Thread Alexander Bulekov
On 200911 2257, Li Qiang wrote: > P J P 于2020年9月11日周五 下午8:30写道: > > > > From: Prasad J Pandit > > > > While servicing the OHCI transfer descriptors(TD), OHCI host > > controller derives variables 'start_addr', 'end_addr', 'len' > > etc. from values supplied by the host controller driver. > >

Re: [PATCH 1/5] target/arm: Replace ARM_FEATURE_PXN with ID_MMFR0.VMSA check

2020-09-11 Thread Richard Henderson
On 9/10/20 10:38 AM, Peter Maydell wrote: > The ARM_FEATURE_PXN bit indicates whether the CPU supports the PXN > bit in short-descriptor translation table format descriptors. This > is indicated by ID_MMFR0.VMSA being at least 0b0100. Replace the > feature bit with an ID register check, in line

Re: [PATCH 3/5] hw/intc/armv7m_nvic: Only show ID register values for Main Extension CPUs

2020-09-11 Thread Richard Henderson
On 9/10/20 10:38 AM, Peter Maydell wrote: > M-profile CPUs only implement the ID registers as guest-visible if > the CPU implements the Main Extension (all our current CPUs except > the Cortex-M0 do). > > Currently we handle this by having the Cortex-M0 leave the ID > register values in the

[PULL 01/18] hw/mips/fuloong2e: Convert pointless error message to an assert()

2020-09-11 Thread Laurent Vivier
From: Philippe Mathieu-Daudé Displaying "vt82c686b_init error" doesn't give any hint about why this call failed. As this message targets developers and not users, replace the pointless error message by a call to assert() which will provide more useful information. Signed-off-by: Philippe

[PULL 05/18] hw/gpio/max7310: Replace disabled printf() by qemu_log_mask(UNIMP)

2020-09-11 Thread Laurent Vivier
From: Philippe Mathieu-Daudé Replace disabled printf() by qemu_log_mask(UNIMP). Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Laurent Vivier Message-Id: <20200901104234.92159-3-f4...@amsat.org> Signed-off-by: Laurent Vivier --- hw/gpio/max7310.c | 11 +-- 1 file changed, 5

[PULL 12/18] hw: hyperv: vmbus: Fix 32bit compilation

2020-09-11 Thread Laurent Vivier
From: Jon Doron Fix 32-bit build error for vmbus: hw/hyperv/vmbus.c: In function ‘gpadl_iter_io’: hw/hyperv/vmbus.c:383:13: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast] 383 | p = (void *)(((uintptr_t)iter->map & TARGET_PAGE_MASK) |

[PULL 08/18] hw/net/e1000e: Remove overwritten read handler for STATUS register

2020-09-11 Thread Laurent Vivier
From: Philippe Mathieu-Daudé The STATUS register readop handler is initialized first with the generic e1000e_mac_readreg() handler: 2861 #define e1000e_getreg(x)[x] = e1000e_mac_readreg 2862 typedef uint32_t (*readops)(E1000ECore *, int); 2863 static const readops

[PULL 15/18] hw/arm/pxa2xx: Add missing fallthrough comment

2020-09-11 Thread Laurent Vivier
From: Thomas Huth Let's make this file compilable with -Werror=implicit-fallthrough : Looking at the code, it seems like the fallthrough is intended here, so we should add the corresponding "/* fallthrough */" comment here. Signed-off-by: Thomas Huth Reviewed-by: Philippe Mathieu-Daudé

[PULL 16/18] util/hexdump: Convert to take a void pointer argument

2020-09-11 Thread Laurent Vivier
From: Philippe Mathieu-Daudé Most uses of qemu_hexdump() do not take an array of char as input, forcing use of cast. Since we can use this helper to dump any kind of buffer, use a pointer to void argument instead. Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Alistair Francis

[PULL 14/18] target/i386/kvm: Rename host_tsx_blacklisted() as host_tsx_broken()

2020-09-11 Thread Laurent Vivier
From: Philippe Mathieu-Daudé In order to use inclusive terminology, rename host_tsx_blacklisted() as host_tsx_broken(). Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Thomas Huth Acked-by: Paolo Bonzini Message-Id: <20200910070131.435543-7-phi...@redhat.com> Signed-off-by: Laurent Vivier

[PULL 17/18] util/hexdump: Reorder qemu_hexdump() arguments

2020-09-11 Thread Laurent Vivier
From: Philippe Mathieu-Daudé qemu_hexdump()'s pointer to the buffer and length of the buffer are closely related arguments but are widely separated in the argument list order (also, the format of function prototypes is usually to have the FILE* argument coming first). Reorder the arguments as

Re: [PATCH v5 2/7] hw/misc/led: Allow connecting from GPIO output

2020-09-11 Thread Luc Michel
Hi Phil, On 9/10/20 10:54 PM, Philippe Mathieu-Daudé wrote: Some devices expose GPIO lines. Add a GPIO qdev input to our LED device, so we can connect a GPIO output using qdev_connect_gpio_out(). When used with GPIOs, the intensity can only be either minium or maximum. This depends of the

Re: Moving to C11? (was Re: Redefinition of typedefs (C11 feature))

2020-09-11 Thread Warner Losh
On Fri, Sep 11, 2020 at 2:07 PM Eduardo Habkost wrote: > On Fri, Sep 11, 2020 at 08:06:10PM +0100, Peter Maydell wrote: > > On Fri, 11 Sep 2020 at 19:49, Eduardo Habkost > wrote: > > > > > > I'm wondering: do our supported build host platforms all include > > > compilers that are new enough to

Re: [PATCH v4 1/2] sifive_e: Rename memmap enum constants

2020-09-11 Thread Alistair Francis
On Fri, Sep 11, 2020 at 10:35 AM Eduardo Habkost wrote: > > Some of the enum constant names conflict with a QOM type check > macro (SIFIVE_E_PRCI). This needs to be addressed to allow us to > transform the QOM type check macros into functions generated by > OBJECT_DECLARE_TYPE(). > > Rename all

Re: [PATCH v4 2/2] sifive_u: Rename memmap enum constants

2020-09-11 Thread Alistair Francis
On Fri, Sep 11, 2020 at 10:35 AM Eduardo Habkost wrote: > > Some of the enum constant names conflict with the QOM type check > macros (SIFIVE_U_OTP, SIFIVE_U_PRCI). This needs to be addressed > to allow us to transform the QOM type check macros into functions > generated by

[PATCH 1/3] vfio: Get DMA limit information

2020-09-11 Thread Matthew Rosato
The underlying host may be limiting the number of outstanding DMA requests for type 1 IOMMU. Determine if this is the case and if so retrieve the number of outstanding mappings allowed. Signed-off-by: Matthew Rosato --- hw/vfio/common.c | 76

[PATCH 0/3] s390x/pci: Accomodate vfio DMA limiting

2020-09-11 Thread Matthew Rosato
Kernel commit 492855939bdb added a limit to the number of outstanding DMA requests for a type1 vfio container. However, lazy unmapping in s390 can in quite a large number of outstanding DMA requests to build up prior to being purged, potentially the entire guest DMA space. This results in

Re: [PATCH] configure: Do not intent to build WHPX on 32-bit host

2020-09-11 Thread Paolo Bonzini
On 09/09/20 20:24, Philippe Mathieu-Daudé wrote: > Hyper-V is available on 64-bit versions of Windows, > do not try to build its support on 32-bit versions. > > Signed-off-by: Philippe Mathieu-Daudé > --- > configure | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/configure

Re: [PATCH v2] configure: Do not intent to build WHPX on 32-bit host

2020-09-11 Thread Paolo Bonzini
On 10/09/20 07:45, Philippe Mathieu-Daudé wrote: > Hyper-V is available on 64-bit versions of Windows, > do not try to build its support on 32-bit versions. > > Signed-off-by: Philippe Mathieu-Daudé > --- > Since v1: Handle MinGW automatic detection (Thomas, Stefan) > --- > configure | 5 -

Re: [PATCH V1 05/32] savevm: QMP command for cprload

2020-09-11 Thread Dr. David Alan Gilbert
* Steven Sistare (steven.sist...@oracle.com) wrote: > On 7/30/2020 12:14 PM, Eric Blake wrote: > > On 7/30/20 10:14 AM, Steve Sistare wrote: > >> Provide the cprload QMP command.  The VM is created from the file produced > >> by the cprsave command.  Guest RAM is restored in-place from the

Re: [PATCH v2] configure: Do not intent to build WHPX on 32-bit host

2020-09-11 Thread Eric Blake
On 9/10/20 12:45 AM, Philippe Mathieu-Daudé wrote: In the subject, s/intent/attempt/ Hyper-V is available on 64-bit versions of Windows, do not try to build its support on 32-bit versions. Signed-off-by: Philippe Mathieu-Daudé --- Since v1: Handle MinGW automatic detection (Thomas, Stefan)

<    1   2   3   >