On Fri, Aug 23, 2019 at 05:48:37PM +0400, Marc-André Lureau wrote:
> Hi
>
> On Fri, Aug 23, 2019 at 5:00 PM Dr. David Alan Gilbert
> wrote:
> >
> > * Daniel P. Berrangé (berra...@redhat.com) wrote:
> >
> >
> >
> > > This means QEMU still has to iterate over every single client
> > > on the bus t
* Marc-André Lureau (marcandre.lur...@gmail.com) wrote:
> Hi
>
> On Fri, Aug 23, 2019 at 5:00 PM Dr. David Alan Gilbert
> wrote:
> >
> > * Daniel P. Berrangé (berra...@redhat.com) wrote:
> >
> >
> >
> > > This means QEMU still has to iterate over every single client
> > > on the bus to identify
On Thu, 22 Aug 2019 at 18:42, Paolo Bonzini wrote:
>
> $(call land, $(CONFIG_SPICE), $(CONFIG_GIO)) will never return "m" so
> ui-spice-app is always linked into QEMU.
>
> Signed-off-by: Paolo Bonzini
> ---
> tests/modules-test.c | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/tests/m
On Thu, 22 Aug 2019 at 18:42, Paolo Bonzini wrote:
>
> Signed-off-by: Paolo Bonzini
> ---
> tests/modules-test.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/modules-test.c b/tests/modules-test.c
> index 3aef0e5..a8118e9 100644
> --- a/tests/modules-test.c
> +++
Hi Guys - I'm currently trying to use IPv6 with QEMU on ARM / AARCH64 Models
(Sabre-Lite / Zync SOCs) and I see this problem. The system gets an IPv6
address from the QEMU in-built router which can be used to ping the default
gateway (the dhcp router itself). All this is good but not really usef
On Thu, 22 Aug 2019 at 14:47, Marc-André Lureau
wrote:
>
> The following changes since commit 810923480863c43ecb22ae124156298385439339:
>
> Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into
> staging (2019-08-22 13:13:35 +0100)
>
> are available in the Git repository at:
>
>
On 8/23/19 4:49 AM, Peter Maydell wrote:
> On Mon, 19 Aug 2019 at 22:38, Richard Henderson
> wrote:
>>
>> Signed-off-by: Richard Henderson
>> ---
>> target/arm/translate.c | 78 --
>> target/arm/a32.decode | 7
>> target/arm/t32.decode | 2 ++
>>
On Fri, Aug 23, 2019 at 03:09:48PM +0100, Dr. David Alan Gilbert wrote:
> * Marc-André Lureau (marcandre.lur...@gmail.com) wrote:
> > Hi
> >
> > On Fri, Aug 23, 2019 at 5:00 PM Dr. David Alan Gilbert
> > wrote:
> > >
> > > * Daniel P. Berrangé (berra...@redhat.com) wrote:
> > >
> > >
> > >
> > >
Most tracing backends are implemented within QEMU, except the
DTrace/SystemTap backends.
One side effect is when running 'qemu -trace help', an incomplete
list of trace events is displayed when using the DTrace/SystemTap
backends.
This is partly due to trace events registered as modules with
trac
* Daniel P. Berrangé (berra...@redhat.com) wrote:
> On Fri, Aug 23, 2019 at 03:09:48PM +0100, Dr. David Alan Gilbert wrote:
> > * Marc-André Lureau (marcandre.lur...@gmail.com) wrote:
> > > Hi
> > >
> > > On Fri, Aug 23, 2019 at 5:00 PM Dr. David Alan Gilbert
> > > wrote:
> > > >
> > > > * Daniel
On 8/23/19 3:58 PM, Philippe Mathieu-Daudé wrote:
> A KVM-only build won't be able to run pre-ARMv7 cpus, disable them.
>
> If KVM is not enabled, they are enabled by default.
[...]
> config CHEETAH
> bool
> +select ARM_V4
> select OMAP
> select TSC210X
Are you sure about the
This is a cover letter to a series of patches being proposed in tandem
to four different projects:
- nbd: Document a new NBD_CMD_FLAG_FAST_ZERO command flag
- qemu: Implement the flag for both clients and server
- libnbd: Implement the flag for clients
- nbdkit: Implement the flag for servers, incl
In job_finish_sync job_enter should be enough for a job to make some
progress and draining is a wrong tool for it. So use job_enter directly
here and drop job_drain with all related staff not used more.
Suggested-by: Kevin Wolf
Signed-off-by: Vladimir Sementsov-Ogievskiy
---
v2: drop drain from
First we use ARM_CPU_TYPE_NAME() when we should.
Then is follow up of [1]:
This series looks at Eduardo suggestions from [2]
and Thomas commit aff39be0ed97 to replace various
object_initialize + qdev_set_parent_bus calls by
sysbus_init_child_obj().
Finally, some devices are declared orph
On 8/23/19 5:03 AM, Peter Maydell wrote:
> On Mon, 19 Aug 2019 at 22:38, Richard Henderson
> wrote:
>
> In subject, typo: "Miscellaneous".
>
>> This fixes an exiting bug with the T5 encoding of SUBS PC, LR, #IMM,
>
> "existing"
>
>> in that it may be executed from user mode as with any other e
Child properties form the composition tree. All objects need to be
a child of another object. Objects can only be a child of one object.
Respect this with the i.MX SoC, to get a cleaner composition tree.
Signed-off-by: Philippe Mathieu-Daudé
---
v2: reworded commit description with Markus sugges
Commit ba1ba5cca introduce the ARM_CPU_TYPE_NAME() macro.
Unify the code base by use it in all places.
Reviewed-by: Alistair Francis
Signed-off-by: Philippe Mathieu-Daudé
---
v2: rebased, use in imx25/31 (pm215)
---
hw/arm/allwinner-a10.c | 3 ++-
hw/arm/cubieboard.c| 3 ++-
hw/arm/digic.c
See the cross-post cover letter for details:
https://www.redhat.com/archives/libguestfs/2019-August/msg00322.html
Eric Blake (1):
api: Add support for FAST_ZERO flag
lib/nbd-protocol.h | 2 ++
generator/generator| 29 +++--
lib/flags.c| 12 +
As explained in commit aff39be0ed97:
Both functions, object_initialize() and object_property_add_child()
increase the reference counter of the new object, so one of the
references has to be dropped afterwards to get the reference
counting right. Otherwise the child object will not be prope
As explained in commit aff39be0ed97:
Both functions, object_initialize() and object_property_add_child()
increase the reference counter of the new object, so one of the
references has to be dropped afterwards to get the reference
counting right. Otherwise the child object will not be prope
As explained in commit aff39be0ed97:
Both functions, object_initialize() and object_property_add_child()
increase the reference counter of the new object, so one of the
references has to be dropped afterwards to get the reference
counting right. Otherwise the child object will not be prope
See the cross-post cover letter for details:
https://www.redhat.com/archives/libguestfs/2019-August/msg00322.html
Eric Blake (1):
protocol: Add NBD_CMD_FLAG_FAST_ZERO
doc/proto.md | 50 +-
1 file changed, 49 insertions(+), 1 deletion(-)
--
2.21
While it may be counterintuitive at first, the introduction of
NBD_CMD_WRITE_ZEROES and NBD_CMD_BLOCK_STATUS has caused a performance
regression in qemu [1], when copying a sparse file. When the
destination file must contain the same contents as the source, but it
is not known in advance whether th
As explained in commit aff39be0ed97:
Both functions, object_initialize() and qdev_set_parent_bus()
increase the reference counter of the new object, so one of the
references has to be dropped afterwards to get the reference
counting right. Otherwise the child object will not be properly
Allow plugins to affect the handling of the new NBD_CMD_FLAG_FAST_ZERO
flag, then update affected plugins. In particular, in-memory plugins
are always fast; the full plugin is better served by omitting .zero
and relying on .pwrite fallback; nbd takes advantage of libnbd
extensions proposed in para
The server side is fairly straightforward: we can always advertise
support for detection of fast zero, and implement it by mapping the
request to the block layer BDRV_REQ_NO_FALLBACK.
Signed-off-by: Eric Blake
---
Again, this may be worth merging with the previous patch.
---
nbd/server.c | 8 +
The client side is fairly straightforward: if the server advertised
fast zero support, then we can map that to BDRV_REQ_NO_FALLBACK
support. A server that advertises FAST_ZERO but not WRITE_ZEROES
is technically broken, but we can ignore that situation as it does
not change our behavior.
Signed-o
See the cross-post cover letter for more details:
https://www.redhat.com/archives/libguestfs/2019-August/msg00322.html
Based-on: https://lists.gnu.org/archive/html/qemu-devel/2019-08/msg04805.html
[Andrey Shinkevich block: workaround for unaligned byte range in fallocate()]
Eric Blake (5):
nbd:
A server may have a reason to reject a request for structured replies,
beyond just not recognizing them as a valid request. It doesn't hurt
us to continue talking to such a server; otherwise 'qemu-nbd --list'
of such a server fails to display all possible details about the
export.
Encountered whe
Commit fe0480d6 and friends added BDRV_REQ_NO_FALLBACK as a way to
avoid wasting time on a preliminary write-zero request that will later
be rewritten by actual data, if it is known that the write-zero
request will use a slow fallback; but in doing so, could not optimize
for NBD. The NBD specifica
When creating a read-only image, we are still advertising support for
TRIM and WRITE_ZEROES to the client, even though the client should not
be issuing those commands. But seeing this requires looking across
multiple functions:
All callers to nbd_export_new() passed a single flag based solely on
Qemu was able to demonstrate that knowing whether a zero operation is
fast is useful when copying from one image to another: there is a
choice between bulk pre-zeroing and then revisiting the data sections
(fewer transactions, but depends on the zeroing to be fast),
vs. visiting every portion of th
On Fri, Aug 23, 2019 at 03:26:02PM +0100, Dr. David Alan Gilbert wrote:
> * Daniel P. Berrangé (berra...@redhat.com) wrote:
> > On Fri, Aug 23, 2019 at 03:09:48PM +0100, Dr. David Alan Gilbert wrote:
> > > * Marc-André Lureau (marcandre.lur...@gmail.com) wrote:
> > > > Hi
> > > >
> > > > On Fri, A
Qemu was able to demonstrate that knowing whether a zero operation is
fast is useful when copying from one image to another: there is a
choice between bulk pre-zeroing and then revisiting the data sections
(fewer transactions, but depends on the zeroing to be fast),
vs. visiting every portion of th
Allow filters to affect the handling of the new NBD_CMD_FLAG_FAST_ZERO
flag, then update affected filters. In particular, the log filter
logs the state of the new flag (requires a tweak to expected output in
test-fua.sh), the delay filter gains a bool parameter delay-fast-zero,
several filters rej
On Fri, 23 Aug 2019 at 15:45, Richard Henderson
wrote:
>
> On 8/23/19 6:04 AM, Peter Maydell wrote:
> >> +&ri rd imm
> >> &r rm
> >> &i imm
> >> &msr_reg rn r mask
> >
> > Should this change be in some other patch ?
>
> No, it's used by ADR.
>
>
See the cross-post cover letter for details:
https://www.redhat.com/archives/libguestfs/2019-August/msg00322.html
Notably, this series did NOT add fast zero support to the file plugin
yet; there, I probably need to do more testing and/or kernel source
code reading to learn whether to mark fallocat
* Daniel P. Berrangé (berra...@redhat.com) wrote:
> On Fri, Aug 23, 2019 at 03:26:02PM +0100, Dr. David Alan Gilbert wrote:
> > * Daniel P. Berrangé (berra...@redhat.com) wrote:
> > > On Fri, Aug 23, 2019 at 03:09:48PM +0100, Dr. David Alan Gilbert wrote:
> > > > * Marc-André Lureau (marcandre.lur.
On 8/23/19 3:07 AM, David Hildenbrand wrote:
> Hm,, how did that "-" slip in (-TAGRET_PAGE_SIZE would be correct). This
> currently makes us exceed one page in a single probe_write() call,
> essentially leaving some memory unchecked.
>
> Fixes: c5a7392cfb96 ("s390x/tcg: Provide probe_write_access
On 8/23/19 6:04 AM, Peter Maydell wrote:
>> +static bool trans_ADR(DisasContext *s, arg_ri *a)
>> +{
>> +store_reg_bx(s, a->rd, add_reg_for_lit(s, 15, a->imm));
>> +return true;
>> +}
...
>> -if (rn == 13 && rd == 13) {
>> -/* ADD SP, SP,
On 8/23/19 7:47 AM, Peter Maydell wrote:
> On Fri, 23 Aug 2019 at 15:45, Richard Henderson
> wrote:
>>
>> On 8/23/19 6:04 AM, Peter Maydell wrote:
+&ri rd imm
&r rm
&i imm
&msr_reg rn r mask
>>>
>>> Should this change be in so
Le 22/08/2019 à 08:49, Chen Zhang via Qemu-trivial a écrit :
> Signed-off-by: Chen Zhang
> ---
> hw/vfio/pci.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/vfio/pci.c b/hw/vfio/pci.c
> index dc3479c..c5e6fe6 100644
> --- a/hw/vfio/pci.c
> +++ b/hw/vfio/pci.c
>
On 8/23/19 3:07 AM, David Hildenbrand wrote:
> If I'm not completely wrong, we are dealing with guest addresses here
> and not with host addresses. Use the right check.
>
> Fixes: c5a7392cfb96 ("s390x/tcg: Provide probe_write_access helper")
> Signed-off-by: David Hildenbrand
> ---
> target/s390
On Fri, Aug 23, 2019 at 09:40:54AM -0400, Cleber Rosa wrote:
> On Thu, Aug 22, 2019 at 06:54:20PM -0300, Eduardo Habkost wrote:
> > On Thu, Aug 22, 2019 at 05:19:26PM -0400, Cleber Rosa wrote:
> > > On Thu, Aug 22, 2019 at 05:48:46PM +0100, Peter Maydell wrote:
> > > > On Fri, 9 Nov 2018 at 15:09,
On Fri, Aug 23, 2019 at 03:56:34PM +0100, Dr. David Alan Gilbert wrote:
> * Daniel P. Berrangé (berra...@redhat.com) wrote:
> > On Fri, Aug 23, 2019 at 03:26:02PM +0100, Dr. David Alan Gilbert wrote:
> > > * Daniel P. Berrangé (berra...@redhat.com) wrote:
> > > > On Fri, Aug 23, 2019 at 03:09:48PM
On 08/22/19 20:51, Paolo Bonzini wrote:
> On 22/08/19 20:29, Laszlo Ersek wrote:
>> On 08/22/19 08:18, Paolo Bonzini wrote:
>>> On 21/08/19 22:17, Kinney, Michael D wrote:
DMA protection of memory ranges is a chipset feature. For the current
QEMU implementation, what ranges of memory are
On Mon, 19 Aug 2019 at 22:38, Richard Henderson
wrote:
>
> Signed-off-by: Richard Henderson
> ---
> target/arm/translate.c | 797 ++---
> target/arm/a32.decode | 120 +++
> target/arm/t32.decode | 141
> 3 files changed, 615 insertions(+), 443 d
23.08.2019 17:30, Eric Blake wrote:
> This is a cover letter to a series of patches being proposed in tandem
> to four different projects:
I always knew you were great, but that breaks all the records
> - nbd: Document a new NBD_CMD_FLAG_FAST_ZERO command flag
> - qemu: Implement the flag for bot
On 2019/8/23 17:24, Stefan Hajnoczi wrote:
> Do not use warn(3) and warnx(3) since they print to stderr. When
> --syslog is used these messages must go to syslog(3) instead.
>
> Signed-off-by: Stefan Hajnoczi
> ---
> contrib/virtiofsd/passthrough_ll.c | 36 +++---
> 1
On Thu, 22 Aug 2019 at 14:58, Cornelia Huck wrote:
>
> The following changes since commit f3b8f18ebf344ab359e8f79f6ed777e740dae77c:
>
> Merge remote-tracking branch 'remotes/armbru/tags/pull-monitor-2019-08-21'
> into staging (2019-08-22 10:31:21 +0100)
>
> are available in the Git repository a
* Daniel P. Berrangé (berra...@redhat.com) wrote:
> On Fri, Aug 23, 2019 at 03:56:34PM +0100, Dr. David Alan Gilbert wrote:
> > * Daniel P. Berrangé (berra...@redhat.com) wrote:
> > > On Fri, Aug 23, 2019 at 03:26:02PM +0100, Dr. David Alan Gilbert wrote:
> > > > * Daniel P. Berrangé (berra...@redh
On 8/23/19 4:32 PM, Philippe Mathieu-Daudé wrote:
> As explained in commit aff39be0ed97:
>
> Both functions, object_initialize() and object_property_add_child()
> increase the reference counter of the new object, so one of the
> references has to be dropped afterwards to get the reference
>
On 8/22/19 8:33 PM, Samuel Thibault wrote:
> Philippe Mathieu-Daudé, le jeu. 22 août 2019 16:41:33 +0200, a ecrit:
>> Later the newly calculated pointer q is converted into ip structure
>> and values are modified, Due to the wrong calculation of the delta,
>> ip will be pointing to incorrect
On 8/23/19 3:07 AM, David Hildenbrand wrote:
> Factor it out, we'll do some further changes/extensions to both
> probe_write() implementations soon. Make sure to allow "size = 0".
>
> Signed-off-by: David Hildenbrand
> ---
> accel/tcg/user-exec.c | 16
> include/exec/exec-al
On Fri, Aug 23, 2019 at 04:14:48PM +0100, Dr. David Alan Gilbert wrote:
> * Daniel P. Berrangé (berra...@redhat.com) wrote:
> > On Fri, Aug 23, 2019 at 03:56:34PM +0100, Dr. David Alan Gilbert wrote:
> > > * Daniel P. Berrangé (berra...@redhat.com) wrote:
> > > > If two helpers are running as the s
On 8/23/19 3:07 AM, David Hildenbrand wrote:
> +void probe_write(CPUArchState *env, target_ulong addr, int size, int mmu_idx,
> + uintptr_t retaddr)
> +{
> +CPUState *cpu = env_cpu(env);
> +CPUClass *cc;
> +
> +if (!guest_addr_valid(addr) ||
> +(size > 0 && !gues
On Thu, Aug 15, 2019 at 2:38 PM Alistair Francis
wrote:
>
> The mip register is an MXLEN-bit long register. Convert it to a
> target_ulong type instead of uint32_t.
>
> Signed-off-by: Alistair Francis
MIP is 32-bit to allow atomic access on 32-bit hosts, so I am going to
send a v4 with this patc
Update the Hypervisor CSR addresses to match the v0.4 spec.
Signed-off-by: Alistair Francis
Reviewed-by: Chih-Min Chao
Reviewed-by: Bin Meng
---
target/riscv/cpu_bits.h | 35 ++-
1 file changed, 18 insertions(+), 17 deletions(-)
diff --git a/target/riscv/cpu_bi
The first three patches are ones that I have pulled out of my original
Hypervisor series at an attempt to reduce the number of patches in the
series.
These three patches all make sense without the Hypervisor series so can
be merged seperatley and will reduce the review burden of the next
version
From: Atish Patra
Use both the generic register name and ABI name for the general purpose
registers and floating point registers.
Signed-off-by: Atish Patra
Signed-off-by: Alistair Francis
Reviewed-by: Bin Meng
---
target/riscv/cpu.c | 19 +++
1 file changed, 11 insertions(+)
Setting write permission on dirty PTEs results in userspace inside a
Hypervisor guest (VU) becoming corrupted. This appears to be because it
ends up with write permission in the second stage translation in cases
where we aren't doing a store.
Signed-off-by: Alistair Francis
Reviewed-by: Bin Meng
Signed-off-by: Alistair Francis
Reviewed-by: Jonathan Behrens
Reviewed-by: Philippe Mathieu-Daudé
Reviewed-by: Chih-Min Chao
Reviewed-by: Bin Meng
---
hw/riscv/sifive_plic.c | 12
include/hw/riscv/sifive_plic.h | 3 ---
2 files changed, 15 deletions(-)
diff --git a/hw/r
* Daniel P. Berrangé (berra...@redhat.com) wrote:
> On Fri, Aug 23, 2019 at 04:14:48PM +0100, Dr. David Alan Gilbert wrote:
> > * Daniel P. Berrangé (berra...@redhat.com) wrote:
> > > On Fri, Aug 23, 2019 at 03:56:34PM +0100, Dr. David Alan Gilbert wrote:
> > > > * Daniel P. Berrangé (berra...@redh
Signed-off-by: Alistair Francis
Reviewed-by: Bin Meng
---
target/riscv/csr.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/target/riscv/csr.c b/target/riscv/csr.c
index 2789215b5e..f767ad24be 100644
--- a/target/riscv/csr.c
+++ b/target/riscv/csr.c
@@ -335,7 +335,7 @@ stati
On Mon, 19 Aug 2019 at 22:38, Richard Henderson
wrote:
>
> Signed-off-by: Richard Henderson
> ---
> target/arm/translate.c | 560 ++---
> target/arm/a32.decode | 48
> target/arm/t32.decode | 46
> 3 files changed, 396 insertions(+), 258 deletion
On 8/23/19 4:32 PM, Philippe Mathieu-Daudé wrote:
> As explained in commit aff39be0ed97:
>
> Both functions, object_initialize() and qdev_set_parent_bus()
Commit aff39be0ed97 was not about qdev_set_parent_bus(), so the first
sentence sounds somewhat misleading here. Maybe rephrase the commit
me
Let's create a function that tests if floating point support is
enabled. We can then protect all floating point operations based on if
they are enabled.
This patch so far doesn't change anything, it's just preparing for the
Hypervisor support for floating point operations.
Signed-off-by: Alistair
On 8/23/19 3:07 AM, David Hildenbrand wrote:
> We now have a variant for CONFIG_USER_ONLY as well.
>
> Signed-off-by: David Hildenbrand
> ---
> target/hppa/op_helper.c | 2 --
> 1 file changed, 2 deletions(-)
Reviewed-by: Richard Henderson
r~
On 8/23/19 3:07 AM, David Hildenbrand wrote:
> While the CONFIG_USER_ONLY variant can handle multiple pages (no MMU), the
> !CONFIG_USER_ONLY variant can't and won't. We'll want to convert
> probe_write() to return a host address (similar to tlb_vaddr_to_host())
> soon. This only works on page gran
Use the TB_FLAGS_MSTATUS_FS macro when enabling floating point in the tb
flags.
Signed-off-by: Alistair Francis
---
target/riscv/cpu.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/target/riscv/cpu.h b/target/riscv/cpu.h
index eb7b5b0af3..0347be453b 100644
--- a/target/risc
Le 09/08/2019 à 19:11, Richard Henderson a écrit :
> Add the HWCAP2_* bits from kernel version v5.3-rc3.
> Enable the bits corresponding to ARMv8.5-CondM and ARMv8.5-FRINT.
>
> Signed-off-by: Richard Henderson
> ---
> linux-user/elfload.c | 31 +++
> 1 file changed, 2
Hi Jiewen,
If a hot add CPU needs to run any code before the
first SMI, I would recommend is only executes code
from a write protected FLASH range without a stack
and then wait for the first SMI.
For this OVMF use case, is any CPU init required
before the first SMI?
From Paolo's list of steps ar
16.08.2019 4:01, John Snow wrote:
>
>
> On 7/19/19 12:30 PM, Andrey Shinkevich wrote:
>> To synchronize the time when QEMU is running longer under the Valgrind,
>> increase the sleeping time in the test 247.
>>
>> Signed-off-by: Andrey Shinkevich
>> Reviewed-by: Vladimir Sementsov-Ogievskiy
>>
* Wei Yang (richard.weiy...@gmail.com) wrote:
> On Mon, Aug 19, 2019 at 12:26:32PM +0100, Dr. David Alan Gilbert wrote:
> >* Wei Yang (richardw.y...@linux.intel.com) wrote:
> >> No functional change. Add default case to fix warning.
> >
> >I think the problem with this is that migrate_set_state use
On 8/23/19 8:28 AM, Richard Henderson wrote:
> On 8/23/19 3:07 AM, David Hildenbrand wrote:
>> While the CONFIG_USER_ONLY variant can handle multiple pages (no MMU), the
>> !CONFIG_USER_ONLY variant can't and won't. We'll want to convert
>> probe_write() to return a host address (similar to tlb_vad
Le 14/07/2019 à 15:40, Laurent Vivier a écrit :
> filename is only used to open the file if AT_EXECFD is not provided.
> But exec_path already contains the path of the file to open.
> Remove filename as it is only used in main.c whereas exec_path is
> also used in syscall.c.
>
> Fixes: d088d664f20
On Fri, 23 Aug 2019 at 16:37, Alistair Francis wrote:
>
> Use the TB_FLAGS_MSTATUS_FS macro when enabling floating point in the tb
> flags.
>
> Signed-off-by: Alistair Francis
> ---
> target/riscv/cpu.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/target/riscv/cpu.h b
On 8/23/19 3:07 AM, David Hildenbrand wrote:
> ... and also call it for CONFIG_USER_ONLY. This function probably will
> also need some refactoring in regards to probing, however, we'll have to
> come back to that later, once cleaning up the other mem helpers.
>
> The alignment check always makes s
On Mon, 19 Aug 2019 at 22:38, Richard Henderson
wrote:
>
> Signed-off-by: Richard Henderson
> ---
> target/arm/translate.c | 200 +
> target/arm/a32.decode | 20 +
> target/arm/t32.decode | 19
> 3 files changed, 143 insertions(+), 96 deletion
Patchew URL: https://patchew.org/QEMU/20190822231231.1306-1-phi...@redhat.com/
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 ===
#!
On Fri, Aug 23, 2019 at 8:44 AM Peter Maydell wrote:
>
> On Fri, 23 Aug 2019 at 16:37, Alistair Francis
> wrote:
> >
> > Use the TB_FLAGS_MSTATUS_FS macro when enabling floating point in the tb
> > flags.
> >
> > Signed-off-by: Alistair Francis
> > ---
> > target/riscv/cpu.h | 2 +-
> > 1 file
On 23.08.19 17:22, Richard Henderson wrote:
> On 8/23/19 3:07 AM, David Hildenbrand wrote:
>> +void probe_write(CPUArchState *env, target_ulong addr, int size, int
>> mmu_idx,
>> + uintptr_t retaddr)
>> +{
>> +CPUState *cpu = env_cpu(env);
>> +CPUClass *cc;
>> +
>> +if
Le 17/08/2019 à 02:07, Shu-Chun Weng via Qemu-devel a écrit :
> The ioctl numeric values are platform-independent and determined by
> the file include/uapi/linux/sockios.h in Linux kernel source code:
>
> #define SIOCETHTOOL 0x8946
>
> These ioctls get (or set) the field ifr_data of type char
Hi,
We (mostly Cheng Xiang's work) have been trying to use the raspi2
machine to boot an image through u-boot.
It seems, however, that there are some missing pieces:
The first problem is that it trips very early if we keep the cpus count
at 4 but will continue if we hard-patch it to be a single
On Mon, 19 Aug 2019 at 22:38, Richard Henderson
wrote:
>
> Signed-off-by: Richard Henderson
> ---
> target/arm/translate.c | 229 -
> target/arm/a32.decode | 44
> target/arm/t32.decode | 44
> 3 files changed, 200 insertions(+), 117
* Wei Yang (richardw.y...@linux.intel.com) wrote:
> This patch enable compress with postcopy.
>
> This is a RFC and based on some unmerged patch
>
> "migration: extract ram_load_precopy"
> "migration/postcopy: skip compression when postcopy is active"
>
> Signed-off-by: Wei Yang
> ---
> mi
On Fri, 23 Aug 2019 at 16:54, Bonnans, Laurent wrote:
>
> Hi,
>
> We (mostly Cheng Xiang's work) have been trying to use the raspi2
> machine to boot an image through u-boot.
> It seems, however, that there are some missing pieces:
>
> The first problem is that it trips very early if we keep the c
On 8/23/19 4:27 AM, David Hildenbrand wrote:
> On 23.08.19 12:07, David Hildenbrand wrote:
>> We want to perform the same checks in probe_write() to trigger a cpu
>> exit before doing any modifications. We'll have to pass a PC.
>>
>> Signed-off-by: David Hildenbrand
>> ---
>> exec.c
On 8/23/19 3:07 AM, David Hildenbrand wrote:
> Let's check for write watchpoints. We'll want to do something similar
> for probe_read() in the future (once we introduce that).
>
> Suggested-by: Richard Henderson
> Signed-off-by: David Hildenbrand
> ---
> accel/tcg/cputlb.c | 16
* Dr. David Alan Gilbert (dgilb...@redhat.com) wrote:
> * Wei Yang (richard.weiy...@gmail.com) wrote:
> > On Mon, Aug 19, 2019 at 12:26:32PM +0100, Dr. David Alan Gilbert wrote:
> > >* Wei Yang (richardw.y...@linux.intel.com) wrote:
> > >> No functional change. Add default case to fix warning.
> >
On Thu, Aug 22, 2019 at 05:59:43PM +, Vladimir Sementsov-Ogievskiy wrote:
> 22.08.2019 20:39, Vladimir Sementsov-Ogievskiy wrote:
> > 22.08.2019 20:24, Vladimir Sementsov-Ogievskiy wrote:
> >> 22.08.2019 18:50, Stefan Hajnoczi wrote:
> >>> On Tue, Jun 04, 2019 at 07:15:02PM +0300, Vladimir Seme
On 8/19/19 9:08 AM, Wei Yang wrote:
> On Mon, Aug 19, 2019 at 12:26:32PM +0100, Dr. David Alan Gilbert wrote:
>> * Wei Yang (richardw.y...@linux.intel.com) wrote:
Typo in the subject line: migrtion should be migration
>>> No functional change. Add default case to fix warning.
>>
>> I think the pr
On 8/23/19 7:54 AM, Peter Maydell wrote:
> On Mon, 19 Aug 2019 at 22:38, Richard Henderson
> wrote:
>>
>> Signed-off-by: Richard Henderson
>> ---
>> target/arm/translate.c | 797 ++---
>> target/arm/a32.decode | 120 +++
>> target/arm/t32.decode | 141 ++
* Wei Yang (richardw.y...@linux.intel.com) wrote:
> Now postcopy is not compatible with compression. And we disable setting
> these two capability at the same time. While we can still leverage
> compress before postcopy is active, for example at the bulk stage.
>
> This patch skips compression whe
Le 19/08/2019 à 20:53, Shu-Chun Weng via Qemu-devel a écrit :
> timer_getoverrun returns the "overrun count" for the timer, which is not
> a file descriptor and thus should not call fd_trans_unregister on it.
>
> Signed-off-by: Shu-Chun Weng
> ---
> linux-user/syscall.c | 1 -
> 1 file changed,
On 8/23/19 8:28 AM, Peter Maydell wrote:
>> +gen_aa32_st_i32(s, tmp, addr, get_mem_index(s), mop | s->be_data);
>> +disas_set_da_iss(s, mop, a->rt | ISSIsAcqRel | ISSIsWrite);
...
>> -switch (op1) {
>> -case 0: /* stl */
>> -
On 8/23/19 8:39 AM, Peter Maydell wrote:
>> +static bool op_bfx(DisasContext *s, arg_UBFX *a, bool u)
>> +{
>> +TCGv_i32 tmp;
>> +int width = a->widthm1 + 1;
>> +int shift = a->lsb;
>> +
>> +if (!ENABLE_ARCH_6T2) {
>> +return false;
>> +}
>> +
>> +tmp = load_reg(s, a
On Fri, Aug 23, 2019 at 4:58 PM Max Reitz wrote:
> On 22.08.19 21:01, Nir Soffer wrote:
>
...
> > > > @@ -1794,6 +1815,8 @@ static int handle_aiocb_truncate(void
> > *opaque)
> > > > /* posix_fallocate() doesn't set errno. */
> > > >
* Boeuf, Sebastien (sebastien.bo...@intel.com) wrote:
> From 734625fe0c031d26e612800cd9331235f58ae2e0 Mon Sep 17 00:00:00 2001
> From: Sebastien Boeuf
> Date: Wed, 7 Aug 2019 07:15:32 -0700
> Subject: [PATCH] libvhost-user: Fix the VHOST_USER_PROTOCOL_F_SLAVE_SEND_FD
> check
>
> Vhost user proto
This is ostensibly about adding docs for the g_autofree/g_autoptr
macros. As part of doing that, however, the existing HACKING doc
is merged into the CODING_STYLE doc and the text is converted to
markdown with a table of contents.
Daniel P. Berrangé (4):
docs: convert CODING_STYLE and HACKING to
101 - 200 of 363 matches
Mail list logo