On Fri, Jun 26, 2020 at 11:49 AM Peter Maydell wrote:
>
> On Fri, 26 Jun 2020 at 11:25, Stefan Hajnoczi wrote:
> >
> > On Thu, Jun 25, 2020 at 02:31:14PM +0100, Peter Maydell wrote:
> > > On Wed, 24 Jun 2020 at 11:02, Stefan Hajnoczi wrote:
> > > >
> > > > The following changes since commit
> >
On 16/06/2020 14:47, BALATON Zoltan wrote:
> Add a reset function that maps macio to the address expected by the
> firmware of the board at startup.
>
> Signed-off-by: BALATON Zoltan
> ---
> hw/ppc/mac.h | 12
> hw/ppc/mac_oldworld.c | 15 ++-
> 2 files changed
>From 338a25285d945c891eb4fa34a18cb45b8b05dbb1 Mon Sep 17 00:00:00 2001
From: David Carlier
Date: Fri, 26 Jun 2020 13:57:08 +
Subject: [PATCH 2/3] drm build ignored for Haiku.
Signed-off-by: David Carlier
---
util/Makefile.objs | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --g
>From 95ef79ddff73eebd1f1bec6673c2c68209fab107 Mon Sep 17 00:00:00 2001
From: David Carlier
Date: Fri, 26 Jun 2020 13:56:14 +
Subject: [PATCH 1/3] Include endian.h for Haiku to solve bswap* macros build
failure.
Signed-off-by: David Carlier
---
include/qemu/bswap.h | 2 ++
1 file changed,
On Fri, 26 Jun 2020 at 10:01, Andrew Jones wrote:
> nit: How about using '= {0}' when declaring the variable, rather than this
> memset?
We prefer "= {}" -- although "= {0}" is the C standard approved
version, some compiler versions produce spurious warnings for
it in some situations. (cf commit
On Thu, Jun 25, 2020 at 08:48:36PM +0200, Philippe Mathieu-Daudé wrote:
> To be able to use multiple queues on the same hardware,
> we need to have each queue able to receive IRQ notifications
> in the correct AIO context.
> The context has to be proper to each queue, not to the block
> driver. Mov
We should add support of semtimedop syscall as new version of glibc
2.31 uses semop based on semtimedop
(commit:
https://gitlab.com/freedesktop-sdk/mirrors/sourceware/glibc/-/commit/765cdd0bffd77960ae852104fc4ea5edcdb8aed3
).
Signed-off-by: Matus Kysel
---
linux-user/syscall.c | 58 +++
Refactoring ipc syscall for s390x and SPARC, so it matches glibc implementation
Signed-off-by: Matus Kysel
---
linux-user/syscall.c | 26 +-
1 file changed, 25 insertions(+), 1 deletion(-)
diff --git a/linux-user/syscall.c b/linux-user/syscall.c
index 97de9fb5c9..9904127
Updated version according Laurent's comments.
v3: split patch and add some arch specific modification
On 16/06/2020 14:47, BALATON Zoltan wrote:
> This function resets a CPU not the whole machine so reflect that in
> its name.
>
> Signed-off-by: BALATON Zoltan
> Reviewed-by: Philippe Mathieu-Daudé
> ---
> hw/ppc/mac_oldworld.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> di
>From d36aba6190152c626f668fef3704ee6b61bfc323 Mon Sep 17 00:00:00 2001
From: David Carlier
Date: Fri, 26 Jun 2020 13:45:04 +
Subject: [PATCH 2/3] define SIGIO for haiku
Signed-off-by: David Carlier
---
include/qemu/osdep.h | 4
1 file changed, 4 insertions(+)
diff --git a/include/qem
>From 6fa7a4108236f513201192654e07c7044a3d7e58 Mon Sep 17 00:00:00 2001
From: David Carlier
Date: Fri, 26 Jun 2020 13:51:37 +
Subject: [PATCH 3/3] qemu_init_exec_dir Haiku implementation
Signed-off-by: David Carlier
---
util/oslib-posix.c | 19 +++
1 file changed, 19 inserti
Igor Mammedov writes:
> On Wed, 24 Jun 2020 10:37:32 +0200
> Markus Armbruster wrote:
>
>> The Error ** argument must be NULL, &error_abort, &error_fatal, or a
>> pointer to a variable containing NULL. Passing an argument of the
>> latter kind twice without clearing it in between is wrong: if t
>From a4dfa918e6eea7a5ccc4375d83d1f0162e3bb122 Mon Sep 17 00:00:00 2001
From: David Carlier
Date: Fri, 26 Jun 2020 13:43:59 +
Subject: [PATCH 1/3] check sys/signal.h presence
Signed-off-by: David Carlier
---
configure | 8
hw/xen/xen-legacy-backend.c | 1 -
includ
On 16/06/2020 14:47, BALATON Zoltan wrote:
> Signed-off-by: BALATON Zoltan
> ---
> hw/pci-host/grackle.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/pci-host/grackle.c b/hw/pci-host/grackle.c
> index 4b3af0c704..48d11f13ab 100644
> --- a/hw/pci-host/grackle.c
> +
On 6/26/20 12:58 PM, Daniel P. Berrangé wrote:
> On Fri, Jun 26, 2020 at 11:29:03AM +0100, Dr. David Alan Gilbert wrote:
>> * Janosch Frank (fran...@linux.ibm.com) wrote:
>>> On 6/26/20 11:32 AM, Daniel P. Berrangé wrote:
On Fri, Jun 26, 2020 at 11:01:58AM +0200, Janosch Frank wrote:
> On
On 6/26/20 2:24 PM, Stefan Hajnoczi wrote:
> On Thu, Jun 25, 2020 at 08:48:27PM +0200, Philippe Mathieu-Daudé wrote:
>> qemu_try_blockalign() is a generic API that call back to the
>> block driver to return its page alignment. As we call from
>> within the very same driver, we already know to page
ilable in the Git repository at:
>
> git://github.com/mcayland/qemu.git tags/qemu-macppc-20200626
>
> for you to fetch changes up to e590e7f01479a1d4544aac062fe9fdb986502294:
>
> adb: add ADB bus trace events (2020-06-26 10:13:52 +0100)
>
> ---
On Thu, Jun 25, 2020 at 08:48:38PM +0200, Philippe Mathieu-Daudé wrote:
> @@ -529,10 +529,13 @@ static bool nvme_poll_queues(BDRVNVMeState *s)
> bool progress = false;
> int i;
>
> +if (s->plugged) {
> +return false;
> +}
> for (i = 0; i < s->nr_queues; i++) {
>
On 16/06/2020 14:47, BALATON Zoltan wrote:
> Values not used frequently enough may not worth putting in a local
> variable, especially with names almost as long as the original value
> because that does not improve readability, to the contrary it makes it
> harder to see what value is used. Drop a
On 6/26/20 1:19 PM, Stefan Hajnoczi wrote:
> On Thu, Jun 25, 2020 at 08:48:25PM +0200, Philippe Mathieu-Daudé wrote:
>> We allocate an unique chunk of memory then use it for two
>> different structures. Introduce the 'idsz_max' variable to
>> hold the maximum size, to make it clearer the size is e
On 6/26/20 12:37 PM, no-re...@patchew.org wrote:
> Patchew URL: https://patchew.org/QEMU/20200626102744.15053-1-f4...@amsat.org/
>
>
>
> Hi,
>
> This series failed the docker-quick@centos7 build test. Please find the
> testing commands and
> their output below. If you have Docker installed, yo
On Thu, Jun 25, 2020 at 08:48:37PM +0200, Philippe Mathieu-Daudé wrote:
> The queues are tied to the hardware, not to the block driver.
> As this function doesn't need to know about the BDRVNVMeState,
> move the 'plugged' check to the caller.
> Since in nvme_aio_unplug() we know that s->plugged is
On 16/06/2020 14:47, BALATON Zoltan wrote:
> Half of the occurances already use get_system_memory() directly
> instead of sysmem variable, convert the two other uses to
> get_system_memory() tii which seems to be more common and drop the
> variable.
>
> Signed-off-by: BALATON Zoltan
> ---
> hw/
On Thu, Jun 25, 2020 at 08:48:36PM +0200, Philippe Mathieu-Daudé wrote:
> To be able to use multiple queues on the same hardware,
> we need to have each queue able to receive IRQ notifications
> in the correct AIO context.
> The context has to be proper to each queue, not to the block
> driver. Mov
On 16/06/2020 14:47, BALATON Zoltan wrote:
> Fall back to load binary ROM image if loading ELF fails. This also
> moves PROM_BASE and PROM_SIZE defines to board as these are matching
> the ROM size and address on this board.
Probably worth a quick mention here that you are removing BIOS_SIZE and
+ Eduardo / Mark / Edgard / Alistair / Fred for QOM design.
On 6/26/20 12:54 PM, BALATON Zoltan wrote:
> On Fri, 26 Jun 2020, BALATON Zoltan wrote:
>> On Fri, 26 Jun 2020, Philippe Mathieu-Daudé wrote:
>>> Suggested-by: Markus Armbruster
>>> Signed-off-by: Philippe Mathieu-Daudé
>>> ---
>>> Aspe
On 16/06/2020 14:47, BALATON Zoltan wrote:
> The beige G3 Power Macintosh has a 4MB firmware ROM. Fix the size of
> the rom region and fall back to loading a binary image with -bios if
> loading ELF image failed. This allows testing emulation with a ROM
> image from real hardware as well as using
On Thu, Jun 25, 2020 at 08:48:35PM +0200, Philippe Mathieu-Daudé wrote:
> BDRV_POLL_WHILE() is defined as:
>
> #define BDRV_POLL_WHILE(bs, cond) ({ \
> BlockDriverState *bs_ = (bs); \
> AIO_WAIT_WHILE(bdrv_get_aio_context(bs_), \
> cond); })
On Thu, Jun 25, 2020 at 08:48:32PM +0200, Philippe Mathieu-Daudé wrote:
> nvme_create_queue_pair() doesn't require BlockDriverState anymore.
> Replace it by BDRVNVMeState to simplify.
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> block/nvme.c | 7 +++
> 1 file changed, 3 insertions(+), 4
On Thu, Jun 25, 2020 at 08:48:34PM +0200, Philippe Mathieu-Daudé wrote:
> The queues are tied to the hardware, logging the block
> driver using them is irrelevant.
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> block/nvme.c | 6 +++---
> block/trace-events | 6 +++---
> 2 files changed,
On Thu, Jun 25, 2020 at 08:48:33PM +0200, Philippe Mathieu-Daudé wrote:
> The queues are tied to the hardware, logging the block
> driver using them is irrelevant.
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> block/nvme.c | 2 +-
> block/trace-events | 2 +-
> 2 files changed, 2 inser
On Thu, Jun 25, 2020 at 08:48:31PM +0200, Philippe Mathieu-Daudé wrote:
> nvme_init_queue() doesn't require BlockDriverState anymore.
> Replace it by BDRVNVMeState to simplify.
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> block/nvme.c | 7 +++
> 1 file changed, 3 insertions(+), 4 deleti
On Thu, Jun 25, 2020 at 08:48:30PM +0200, Philippe Mathieu-Daudé wrote:
> nvme_free_queue_pair() doesn't use BlockDriverState, remove it.
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> block/nvme.c | 10 +-
> 1 file changed, 5 insertions(+), 5 deletions(-)
Reviewed-by: Stefan Hajnocz
On Thu, Jun 25, 2020 at 08:48:29PM +0200, Philippe Mathieu-Daudé wrote:
> qemu_try_memalign() returns a void*, qemu_vfio_dma_map() consumes
> a void*. Drop the confusing uint8_t* type.
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> block/nvme.c | 2 +-
> 1 file changed, 1 insertion(+), 1 dele
Your pipeline has failed.
Project: QEMU ( https://gitlab.com/qemu-project/qemu )
Branch: master ( https://gitlab.com/qemu-project/qemu/-/commits/master )
Commit: 611ac633 (
https://gitlab.com/qemu-project/qemu/-/commit/611ac63305ff577604e2a7bacf4f204568e08bef
)
Commit Message: Merge remote-tr
On Thu, Jun 25, 2020 at 08:48:28PM +0200, Philippe Mathieu-Daudé wrote:
> Move assignments previous to where the assigned variable is used,
> to make the nvme_identify() body easier to review. No logical change.
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> block/nvme.c | 10 --
> 1
On Thu, Jun 25, 2020 at 08:48:27PM +0200, Philippe Mathieu-Daudé wrote:
> qemu_try_blockalign() is a generic API that call back to the
> block driver to return its page alignment. As we call from
> within the very same driver, we already know to page alignment
> stored in our state. Remove indirect
On 26/06/2020 11:21, BALATON Zoltan wrote:
> What about these patches? At least those that are finished (up to patch 9)
> could be
> merged. I've seen you sent a pull request but not including any of these.
> Will this
> need another rebase after your patches? If I rebase them will you consider
On Thu, Jun 25, 2020 at 08:48:26PM +0200, Philippe Mathieu-Daudé wrote:
> In the next commit we'll get ride of qemu_try_blockalign().
s/ride/rid/
Reviewed-by: Stefan Hajnoczi
signature.asc
Description: PGP signature
Hi Atish,
On Fri, Jun 26, 2020 at 8:33 AM Atish Patra wrote:
>
> OpenSBI is the default firmware in Qemu and has various firmware loading
> options. Currently, qemu loader uses fw_jump which has a compile time
> pre-defined address where fdt & kernel image must reside. This puts a
> constraint on
* Alex Williamson (alex.william...@redhat.com) wrote:
> On Wed, 24 Jun 2020 19:59:39 +0530
> Kirti Wankhede wrote:
>
> > On 6/23/2020 1:58 AM, Alex Williamson wrote:
> > > On Sun, 21 Jun 2020 01:51:12 +0530
> > > Kirti Wankhede wrote:
> > >
> > >> These functions save and restore PCI device s
Hi Atish,
On Fri, Jun 26, 2020 at 8:33 AM Atish Patra wrote:
>
> Currently, the fdt is copied to the ROM after the reset vector. The firmware
> has to copy it to DRAM. Instead of this, directly copy the device tree to a
> pre-computed dram address. The device tree load address should be as far as
On Wed, 22 Apr 2020, BALATON Zoltan wrote:
On Wed, 22 Apr 2020, Philippe Mathieu-Daudé wrote:
On 4/22/20 4:27 PM, BALATON Zoltan wrote:
On Wed, 22 Apr 2020, Markus Armbruster wrote:
The Error ** argument must be NULL, &error_abort, &error_fatal, or a
pointer to a variable containing NULL. Pas
Hi, Aleksandar,
On Wed, Jun 24, 2020 at 7:50 PM Aleksandar Markovic
wrote:
>
> сре, 24. јун 2020. у 12:45 Huacai Chen је написао/ла:
> >
> > Add Loongson-3 based machine support, it use liointc as the interrupt
> > controler and use GPEX as the pci controller. Currently it can only work
> > with
* Vivek Goyal (vgo...@redhat.com) wrote:
> On Fri, Jun 19, 2020 at 12:39:14PM +0100, Daniel P. Berrangé wrote:
> [..]
> > The CAP_SYS_ADMIN requirement for 'trusted.' xattrs is simply a useful
> > mechanism for applications to control access. The host kernel doesn'
> > tuse this namespace itself. L
пет, 26. јун 2020. у 12:54 Jon Doron је написао/ла:
>
> Is there a container I can download which has your build environment?
>
Hello, Jon.
I don't know about the container, but, as Peter noted, the following
(non-beautiful) diff fixes the build problem on the real host:
diff --git a/hw/hyperv/
On Thu, Jun 25, 2020 at 08:48:25PM +0200, Philippe Mathieu-Daudé wrote:
> We allocate an unique chunk of memory then use it for two
> different structures. Introduce the 'idsz_max' variable to
> hold the maximum size, to make it clearer the size is enough
> to hold the two structures.
>
> Signed-o
he Git repository at:
>
> git://github.com/dgibson/qemu.git tags/ppc-for-5.1-20200626
>
> for you to fetch changes up to 737ef968d442cb287b1fcc7da94b53284b0ad1e9:
>
> target/ppc: Remove TIDR from POWER10 processor (2020-06-26 09:22:30 +1000)
>
> ---
On Thu, Jun 25, 2020 at 08:48:24PM +0200, Philippe Mathieu-Daudé wrote:
> Use definitions instead of '0' or '1' indexes. Also this will
> be useful when using multi-queues later.
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> block/nvme.c | 33 +++--
> 1 file chang
On Fri, 26 Jun 2020 at 11:10, David CARLIER wrote:
>
> From 68d4d4312eccd212b4d2484e09425816ebd2346a Mon Sep 17 00:00:00 2001
> From: David Carlier
> Date: Fri, 26 Jun 2020 11:01:54 +
> Subject: [PATCH 5/5] Last chunk of build fix
>
> Signed-off-by: David Carlier
> ---
> include/qemu/bswap.
On Fri, 26 Jun 2020 at 11:09, David CARLIER wrote:
>
> From 775173ded5657de4d4b467f2f68e747f6a9c0750 Mon Sep 17 00:00:00 2001
> From: David Carlier
> Date: Fri, 26 Jun 2020 10:44:36 +
> Subject: [PATCH 4/5] Platform specific changes qemu_exec_dir implementation
If you could provide more deta
On Thu, Jun 25, 2020 at 08:48:23PM +0200, Philippe Mathieu-Daudé wrote:
> As nvme_create_queue_pair() is allowed to fail, replace the
> alloc() calls by try_alloc() to avoid aborting QEMU.
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> block/nvme.c | 12 ++--
> 1 file changed, 10 inse
* David Gibson (da...@gibson.dropbear.id.au) wrote:
> Several architectures have mechanisms which are designed to protect guest
> memory from interference or eavesdropping by a compromised hypervisor. AMD
> SEV does this with in-chip memory encryption and Intel has a similar
> mechanism. POWER's
On 26/06/20 01:57, Joseph Myers wrote:
> Fix some issues relating to SSE floating-point emulation. The first
> patch fixes a problem with the handling of the FTZ bit that was found
> through the testcase written for the second patch. Rather than
> writing a separate standalone test for that bug,
On Fri, 26 Jun 2020, Philippe Mathieu-Daudé wrote:
Suggested-by: Markus Armbruster
Signed-off-by: Philippe Mathieu-Daudé
---
hw/i2c/smbus_eeprom.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/hw/i2c/smbus_eeprom.c b/hw/i2c/smbus_eeprom.c
index 879fd7c416..22ba7b20d4 100644
--- a/hw/i2c
On Fri, Jun 26, 2020 at 11:29:03AM +0100, Dr. David Alan Gilbert wrote:
> * Janosch Frank (fran...@linux.ibm.com) wrote:
> > On 6/26/20 11:32 AM, Daniel P. Berrangé wrote:
> > > On Fri, Jun 26, 2020 at 11:01:58AM +0200, Janosch Frank wrote:
> > >> On 6/26/20 8:53 AM, David Hildenbrand wrote:
> > >
On Fri, 26 Jun 2020 at 11:10, David CARLIER wrote:
>
> From 19a41d406eda976001827d248398d7fb172d140b Mon Sep 17 00:00:00 2001
> From: David Carlier
> Date: Fri, 26 Jun 2020 10:38:17 +
> Subject: [PATCH 2/5] Enable *pty API.
>
> Signed-off-by: David Carlier
> ---
> configure | 9 ++
On Fri, 26 Jun 2020 at 11:11, David CARLIER wrote:
>
> From 56890fa5728d11c72232348b3f3273e2df31c197 Mon Sep 17 00:00:00 2001
> From: David Carlier
> Date: Fri, 26 Jun 2020 10:40:58 +
> Subject: [PATCH 3/5] Checking mlockall symbol presence
>
> Signed-off-by: David Carlier
Reviewed-by: Pete
On Fri, 26 Jun 2020 at 11:08, David CARLIER wrote:
>
> From 4d0933384d2bfcd0fc8c4c06eed2d07f3f1b7f8b Mon Sep 17 00:00:00 2001
> From: David Carlier
> Date: Fri, 26 Jun 2020 10:35:40 +
> Subject: [PATCH 1/5] Haiku build fix enabling BSD symbols.
>
> Signed-off-by: David Carlier
> ---
> confi
Is there a container I can download which has your build environment?
On Fri, Jun 26, 2020 at 12:27 PM Philippe Mathieu-Daudé
wrote:
>
> On 6/26/20 11:20 AM, Thomas Huth wrote:
> > On 26/06/2020 11.13, Philippe Mathieu-Daudé wrote:
> >> On 6/26/20 9:37 AM, Aleksandar Markovic wrote:
> >>> пет, 26
On Fri, 26 Jun 2020, BALATON Zoltan wrote:
On Fri, 26 Jun 2020, Philippe Mathieu-Daudé wrote:
Suggested-by: Markus Armbruster
Signed-off-by: Philippe Mathieu-Daudé
---
Aspeed change pending latest ARM pull-request, so meanwhile sending
as RFC.
---
include/hw/i2c/smbus_eeprom.h | 9 ++---
h
On Fri, 26 Jun 2020 at 11:25, Stefan Hajnoczi wrote:
>
> On Thu, Jun 25, 2020 at 02:31:14PM +0100, Peter Maydell wrote:
> > On Wed, 24 Jun 2020 at 11:02, Stefan Hajnoczi wrote:
> > >
> > > The following changes since commit
> > > 171199f56f5f9bdf1e5d670d09ef1351d8f01bae:
> > >
> > > Merge remo
On Fri, 26 Jun 2020, Philippe Mathieu-Daudé wrote:
Suggested-by: Markus Armbruster
Signed-off-by: Philippe Mathieu-Daudé
---
Aspeed change pending latest ARM pull-request, so meanwhile sending
as RFC.
---
include/hw/i2c/smbus_eeprom.h | 9 ++---
hw/i2c/smbus_eeprom.c | 13 ++
In 32-bit mode, the higher 16 bits of the destination
register are undefined. In practice CR0[31:0] is stored,
just like in 64-bit mode, so just remove the "if" that
currently differentiates the behavior.
Signed-off-by: Paolo Bonzini
---
target/i386/translate.c | 13 +++--
1 file change
Force the end of a translation block after an I/O instruction in
icount mode. For consistency, all CF_USE_ICOUNT code is kept in
disas_insn instead of having it in gen_ins and gen_outs.
Signed-off-by: Paolo Bonzini
---
target/i386/translate.c | 22 +-
1 file changed, 9 inser
* David Gibson (da...@gibson.dropbear.id.au) wrote:
> We haven't yet implemented the fairly involved handshaking that will be
> needed to migrate PEF protected guests. For now, just use a migration
> blocker so we get a meaningful error if someone attempts this (this is the
> same approach used by
Patchew URL: https://patchew.org/QEMU/20200626102744.15053-1-f4...@amsat.org/
Hi,
This series failed the docker-mingw@fedora build test. Please find the testing
commands and
their output below. If you have Docker installed, you can probably reproduce it
locally.
=== TEST SCRIPT BEGIN ===
#! /
The machine may need to pass reserved regions to the
virtio-iommu-pci device (such as the MSI window on x86
or the MSI doorbells on ARM).
So let's add an array of Interval properties.
Note: if some reserved regions are already set by the
machine code - which should be the case in general -,
the l
Patchew URL: https://patchew.org/QEMU/20200626102744.15053-1-f4...@amsat.org/
Hi,
This series failed the docker-quick@centos7 build test. Please find the testing
commands and
their output below. If you have Docker installed, you can probably reproduce it
locally.
=== TEST SCRIPT BEGIN ===
#!/
On Thu, Jun 25, 2020 at 05:29:28PM +0100, Dr. David Alan Gilbert (git) wrote:
> From: "Dr. David Alan Gilbert"
>
> Check the capability calls worked.
>
> Signed-off-by: Dr. David Alan Gilbert
> ---
> tools/virtiofsd/passthrough_ll.c | 16 +---
> 1 file changed, 13 insertions(+), 3
On Thu, Jun 25, 2020 at 08:48:22PM +0200, Philippe Mathieu-Daudé wrote:
> Avoid further processing if TRACE_NVME_SUBMIT_COMMAND_RAW is
> not enabled.
Why?
This saves 8 trace events, each with 8 arguments. I guess the intent is
to improve performance. Did you measure an improvement?
signature.as
On Thu, Jun 25, 2020 at 05:29:27PM +0100, Dr. David Alan Gilbert (git) wrote:
> From: "Dr. David Alan Gilbert"
>
> capng_updatev is a varargs function that needs a -1 to terminate it,
> but it was missing.
>
> In practice what seems to have been happening is that it's added the
> capabilities we
Introduce a new property defining a reserved region:
::.
This will be used to encode reserved IOVA regions.
For instance, in virtio-iommu use case, reserved IOVA regions
will be passed by the machine code to the virtio-iommu-pci
device (an array of those). The type of the reserved region
will mat
At the moment the virtio-iommu translates MSI transactions.
This behavior is inherited from ARM SMMU. The virt machine
code knows where the guest MSI doorbells are so we can easily
declare those regions as VIRTIO_IOMMU_RESV_MEM_T_MSI. With that
setting the guest will not map MSIs through the IOMMU
When translating an address we need to check if it belongs to
a reserved virtual address range. If it does, there are 2 cases:
- it belongs to a RESERVED region: the guest should neither use
this address in a MAP not instruct the end-point to DMA on
them. We report an error
- It belongs to an
By default the virtio-iommu translates MSI transactions. This
behavior is inherited from ARM SMMU. However the virt machine
code knows where the MSI doorbells are, so we can easily
declare those regions as VIRTIO_IOMMU_RESV_MEM_T_MSI. With that
setting the guest iommu subsystem will not need to map
On Thu, Jun 25, 2020 at 05:29:29PM +0100, Dr. David Alan Gilbert (git) wrote:
> +/*
> + * The modcaps option is a colon separated list of caps,
> + * each preceded by either + or -.
> + */
> +while (lo->modcaps) {
> +capng_act_t action;
> +int cap;
> +
> +
This patch implements the PROBE request. At the moment,
only THE RESV_MEM property is handled. The first goal is
to report iommu wide reserved regions such as the MSI regions
set by the machine code. On x86 this will be the IOAPIC MSI
region, [0xFEE0 - 0xFEEF], on ARM this may be the ITS
do
* Janosch Frank (fran...@linux.ibm.com) wrote:
> On 6/26/20 11:32 AM, Daniel P. Berrangé wrote:
> > On Fri, Jun 26, 2020 at 11:01:58AM +0200, Janosch Frank wrote:
> >> On 6/26/20 8:53 AM, David Hildenbrand wrote:
> >>> Does this have any implications when probing with the 'none' machine?
>
Hi, Alexandar,
On Wed, Jun 24, 2020 at 7:27 PM Aleksandar Markovic
wrote:
>
> >
> > What exactly is missing in tcg support? Would it work if Loongson EXT is
> > supported in QEMU?
> >
>
> Huacai, hi.
>
> I couldn't find the answer to this question in v6.
>
> Could you please clarify this aspect?
Suggested-by: Markus Armbruster
Signed-off-by: Philippe Mathieu-Daudé
---
Aspeed change pending latest ARM pull-request, so meanwhile sending
as RFC.
---
include/hw/i2c/smbus_eeprom.h | 9 ++---
hw/i2c/smbus_eeprom.c | 13 ++---
hw/mips/fuloong2e.c | 2 +-
hw/ppc/
Signed-off-by: Philippe Mathieu-Daudé
---
hw/i2c/smbus_eeprom.c | 2 ++
hw/i2c/trace-events | 3 +++
2 files changed, 5 insertions(+)
diff --git a/hw/i2c/smbus_eeprom.c b/hw/i2c/smbus_eeprom.c
index 22ba7b20d4..7a0e1e7455 100644
--- a/hw/i2c/smbus_eeprom.c
+++ b/hw/i2c/smbus_eeprom.c
@@ -31,6
Suggested-by: Markus Armbruster
Signed-off-by: Philippe Mathieu-Daudé
---
hw/i2c/smbus_eeprom.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/hw/i2c/smbus_eeprom.c b/hw/i2c/smbus_eeprom.c
index 879fd7c416..22ba7b20d4 100644
--- a/hw/i2c/smbus_eeprom.c
+++ b/hw/i2c/smbus_eeprom.c
@@ -47,
This RFC is simply a proof-of-concept to see if I correctly
understood Markus' suggestion, see the thread around:
https://lists.gnu.org/archive/html/qemu-devel/2020-06/msg08652.html
Philippe Mathieu-Daudé (3):
hw/i2c/smbus_eeprom: Set QOM parent
hw/i2c/smbus_eeprom: Add description based on ch
On Fri, Jun 26, 2020 at 8:33 AM Atish Patra wrote:
>
> Currently, all riscv machines except sifive_u have identical reset vector
> code implementations with memory addresses being different for all machines.
> They can be easily combined into a single function in common code.
>
> Move it to common
On Thu, Jun 25, 2020 at 02:31:14PM +0100, Peter Maydell wrote:
> On Wed, 24 Jun 2020 at 11:02, Stefan Hajnoczi wrote:
> >
> > The following changes since commit 171199f56f5f9bdf1e5d670d09ef1351d8f01bae:
> >
> > Merge remote-tracking branch
> > 'remotes/alistair/tags/pull-riscv-to-apply-20200619
Hello Mark,
On Tue, 16 Jun 2020, BALATON Zoltan wrote:
v5: Rebased on master, added some more clean ups, CUDA i2c is still to
be sorted out, help with that is welcome.
What about these patches? At least those that are finished (up to patch 9)
could be merged. I've seen you sent a pull request
Hi Corey,
Le 6/22/20 à 11:32 PM, Corey Minyard a écrit :
On Sun, Jun 21, 2020 at 04:43:38PM +0200, BALATON Zoltan wrote:
These functions have a parameter that decides the direction of
transfer but totally confusingly they don't match but inverted sense.
To avoid frequent mistakes when using t
HI Markus,
> -Original Message-
> From: Markus Armbruster
> Sent: Friday, June 26, 2020 11:42 AM
> To: Sai Pavan Boddu
> Cc: Peter Maydell ; Thomas Huth
> ; Eduardo Habkost ; qemu-
> de...@nongnu.org; Alistair Francis ; Gerd
> Hoffmann ; Paolo Bonzini ;
> Ying Fang ; 'Marc-André Lureau'
>From 56890fa5728d11c72232348b3f3273e2df31c197 Mon Sep 17 00:00:00 2001
From: David Carlier
Date: Fri, 26 Jun 2020 10:40:58 +
Subject: [PATCH 3/5] Checking mlockall symbol presence
Signed-off-by: David Carlier
---
configure | 15 +++
os-posix.c | 4
2 files changed, 19 in
>From 19a41d406eda976001827d248398d7fb172d140b Mon Sep 17 00:00:00 2001
From: David Carlier
Date: Fri, 26 Jun 2020 10:38:17 +
Subject: [PATCH 2/5] Enable *pty API.
Signed-off-by: David Carlier
---
configure | 9 +
util/qemu-openpty.c | 2 +-
2 files changed, 10 insertions(
>From 68d4d4312eccd212b4d2484e09425816ebd2346a Mon Sep 17 00:00:00 2001
From: David Carlier
Date: Fri, 26 Jun 2020 11:01:54 +
Subject: [PATCH 5/5] Last chunk of build fix
Signed-off-by: David Carlier
---
include/qemu/bswap.h | 2 ++
util/Makefile.objs | 2 +-
util/compatfd.c | 2 ++
>From 775173ded5657de4d4b467f2f68e747f6a9c0750 Mon Sep 17 00:00:00 2001
From: David Carlier
Date: Fri, 26 Jun 2020 10:44:36 +
Subject: [PATCH 4/5] Platform specific changes qemu_exec_dir implementation
Signed-off-by: David Carlier
---
include/qemu/osdep.h | 4
util/oslib-posix.c | 2
>From 4d0933384d2bfcd0fc8c4c06eed2d07f3f1b7f8b Mon Sep 17 00:00:00 2001
From: David Carlier
Date: Fri, 26 Jun 2020 10:35:40 +
Subject: [PATCH 1/5] Haiku build fix enabling BSD symbols.
Signed-off-by: David Carlier
---
configure | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff
Le 6/16/20 à 4:52 PM, Alex Bennée a écrit :
kon...@adacore.com writes:
From: KONRAD Frederic
With that we can just use chardev=serial0.
I don't quite follow what this means.
./aarch64-softmmu/qemu-system-aarch64 -cpu max -monitor none -chardev=serial0
-M virt -display none -semihostin
On Wed, 24 Jun 2020 16:23:12 -0400
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
On Wed, 24 Jun 2020 16:23:10 -0400
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 store CPU
> entries. This means
Your pipeline has failed.
Project: QEMU ( https://gitlab.com/qemu-project/qemu )
Branch: master ( https://gitlab.com/qemu-project/qemu/-/commits/master )
Commit: 5acc270a (
https://gitlab.com/qemu-project/qemu/-/commit/5acc270a355120ce967ca1f1eeca0abbdb9303c8
)
Commit Message: Merge remote-tr
If something goes wrong during precopy, before stopping the VM, we will
never send a S_DONE indication to the VM, resulting in the hinted pages
not getting released to be used by the guest OS (e.g., Linux).
Easy to reproduce:
1. Start migration (e.g., HMP "migrate -d 'exec:gzip -c > STATEFILE.gz'"
401 - 500 of 573 matches
Mail list logo