On Mon, Oct 19, 2020 at 04:41:38PM -0400, Cleber Rosa wrote:
> On Mon, Oct 19, 2020 at 12:26:10PM +0200, Erik Skultety wrote:
> > On Sun, Oct 18, 2020 at 09:50:02PM -0400, Cleber Rosa wrote:
> > > To have the jobs dispatched to custom runners, gitlab-runner must
> > > be installed, active as a serv
On 10/20/20 8:42 AM, Bihong Yu wrote:
Signed-off-by: Bihong Yu
---
migration/block.c | 36 ++--
migration/page_cache.c | 13 +++--
migration/trace-events | 13 +
3 files changed, 34 insertions(+), 28 deletions(-)
diff --git a/migrati
On 10/12/20 3:09 PM, Bihong Yu wrote:
Signed-off-by: Bihong Yu
Reviewed-by: Chuan Zheng
---
migration/rdma.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Philippe Mathieu-Daudé
On 10/14/20 9:21 PM, Dr. David Alan Gilbert wrote:
* Bihong Yu (yubih...@huawei.com) wrote:
Signed-off-by: Bihong Yu
Reviewed-by: Chuan Zheng
OK, sometimes I find initialisation clearer
Reviewed-by: Dr. David Alan Gilbert
---
migration/ram.c| 2 +-
migration/savevm.c | 2 +-
2
On 10/12/20 3:08 PM, Bihong Yu wrote:
Signed-off-by: Bihong Yu
Reviewed-by: Chuan Zheng
---
migration/ram.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
Reviewed-by: Philippe Mathieu-Daudé
Hi
On Mon, Oct 19, 2020 at 11:09 PM Eric Blake wrote:
> On 10/13/20 3:25 PM, marcandre.lur...@redhat.com wrote:
> > From: Marc-André Lureau
> >
> > Add new commands to add and remove SSH public keys from
> > ~/.ssh/authorized_keys.
> >
>
> > +++ b/qga/qapi-schema.json
> > @@ -1306,3 +1306,35 @@
On 10/12/20 3:08 PM, Bihong Yu wrote:
Signed-off-by: Bihong Yu
Reviewed-by: Chuan Zheng
---
migration/block.c | 2 +-
migration/ram.c | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
Reviewed-by: Philippe Mathieu-Daudé
John Snow writes:
> This script has not seen a patch that was specifically for this script
> since it was moved to this location in 2013, and I doubt it is used. It
> uses "man qmp" for its help message, which does not exist. It also
> presumes there is a manual page for qmp-XXX, for each defined
David Edmondson writes:
> If the user selects pretty-printing (-p) the contents of any
> dictionaries in the output are sorted by key.
>
> Signed-off-by: David Edmondson
> ---
> scripts/qmp/qmp-shell | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/scripts/qmp/qmp-shell
Signed-off-by: Bihong Yu
---
migration/block.c | 36 ++--
migration/page_cache.c | 13 +++--
migration/trace-events | 13 +
3 files changed, 34 insertions(+), 28 deletions(-)
diff --git a/migration/block.c b/migration/block.c
index 273392b
You forgot to Cc qemu-devel@.
(See https://wiki.qemu.org/Contribute/SubmitAPatch#CC_the_relevant_maintainer)
On Tue, Oct 20, 2020 at 4:46 AM Lichang Zhao wrote:
Now your name is properly displayed, so I assume you finally ran:
$ git config user.name "Lichang Zhao".
To fix how your name is dis
On Mon, Oct 19, 2020 at 04:25:31PM -0400, Cleber Rosa wrote:
> On Mon, Oct 19, 2020 at 12:27:41PM +0200, Erik Skultety wrote:
> > ...
> >
> > > diff --git a/scripts/ci/setup/inventory b/scripts/ci/setup/inventory
> > > new file mode 100644
> > > index 00..8bb7ba6b33
> > > --- /dev/null
> >
So is this now working for everybody with the correct ssh config (maybe
also check your firewall settings)? Could we close this ticket nowadays?
Or is somebody still having trouble?
** Changed in: qemu
Status: Confirmed => Incomplete
--
You received this bug notification because you are a
On 19/10/2020 23.04, John Snow wrote:
> This script has not seen a patch that was specifically for this script
> since it was moved to this location in 2013, and I doubt it is used. It
> uses "man qmp" for its help message, which does not exist. It also
> presumes there is a manual page for qmp-XXX
On 2020/10/20 上午1:39, Peter Xu wrote:
Dave magically found this. Fix them with "0x%x".
Reported-by: Dr. David Alan Gilbert
Signed-off-by: Peter Xu
---
hw/i386/intel_iommu.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/i386/intel_iommu.c b/hw/i386/intel_iommu
Philippe Mathieu-Daudé writes:
> On 10/19/20 6:48 PM, Markus Armbruster wrote:
>> Eduardo Habkost writes:
>>
>>> On Mon, Oct 19, 2020 at 09:55:20AM +0200, Markus Armbruster wrote:
Eduardo Habkost writes:
> On Mon, Oct 12, 2020 at 02:15:31PM +0200, Philippe Mathieu-Daudé wrote:
>>
Mark Cave-Ayland writes:
> One thing I have thought about is being able to mark a link property
> as mandatory so if a value hasn't been set before realize then you
A non-null value, I presume.
> receive a fatal error. This would be for cases like this where 2
> internal devices are connected t
On 20/10/2020 00.20, Joelle van Dyne wrote:
> On Mon, Oct 19, 2020 at 1:27 AM Thomas Huth wrote:
>>
>> On 19/10/2020 03.39, Joelle van Dyne wrote:
>>> From: osy
>>
>> That "From:" line looks wrong ... could you please fix the "Author" of your
>> patches / your git config?
> osy wrote the original
Eric Blake writes:
> On 10/8/20 10:49 AM, Daniel P. Berrangé wrote:
>> Currently the vmstate will be stored in the first block device that
>> supports snapshots. Historically this would have usually been the
>> root device, but with UEFI it might be the variable store. There
>> needs to be a way
On 20/10/2020 00.24, Joelle van Dyne wrote:
> Correct me if I'm wrong but wouldn't the following test still fail
> with --cross-prefix=""
>
> if test -n "$cross_prefix"; then
> ...
>
> That was my main reason for making this change.
That's why I wrote "still introduce the cross_compile=yes varia
Eric Blake writes:
> On 10/8/20 10:49 AM, Daniel P. Berrangé wrote:
>> None of the callers care about the errno value since there is a full
>> Error object populated. This gives consistency with save_snapshot()
>> which already just returns -1.
>> Reviewed-by: Dr. David Alan Gilbert
>> Signed-of
[Expired for QEMU because there has been no activity for 60 days.]
** Changed in: qemu
Status: Incomplete => Expired
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1889411
Title:
RISC-V: Una
patch [1/2] - add write function and wrire-once feature
patch [2/2] - add file backend support
Test Steps: (should work even only 1/2 is applied)
1) Follow instructions to prepare fw_payload -
https://github.com/riscv/opensbi/blob/master/docs/platform/sifive_fu540.md
a) build 1-round opensbi
Add '-drive' support to OTP device. Allow users to assign a raw file
as OTP image.
test commands for 16k otp.img filled with zero:
$ dd if=/dev/zero of=./otp.img bs=1k count=16
$ ./qemu-system-riscv64 -M sifive_u -m 256M -nographic -bios none \
-kernel ../opensbi/build/platform/sifive/fu540/firmw
- Add write operation to update fuse data bit when PWE bit is on.
- Add array, fuse_wo, to store the 'written' status for all bits
of OTP to block the write operation.
Signed-off-by: Green Wan
Reviewed-by: Alistair Francis
Reviewed-by: Bin Meng
Tested-by: Bin Meng
---
hw/misc/sifive_u_ot
Patchew URL:
https://patchew.org/QEMU/1603163448-27122-1-git-send-email-yubih...@huawei.com/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: 1603163448-27122-1-git-send-email-yubih...@huawei.com
Subject: [PATCH v3 0/8]
On Wed, Oct 14, 2020 at 8:28 AM Alistair Francis
wrote:
>
> Instead of loading the kernel at a hardcoded start address, let's load
> the kernel at the next alligned address after the end of the firmware.
typo of "aligned"
>
> This should have no impact for current users of OpenSBI, but will
> al
On Wed, Oct 14, 2020 at 8:28 AM Alistair Francis
wrote:
>
> Instead of returning the unused entry address from riscv_load_firmware()
> instead return the end address. Also return the end address from
> riscv_find_and_load_firmware().
>
> This tells the caller if a firmware was loaded and how big i
On Wed, Oct 14, 2020 at 8:28 AM Alistair Francis
wrote:
>
> Signed-off-by: Alistair Francis
> ---
> include/hw/riscv/boot.h | 2 ++
> hw/riscv/boot.c | 9 +
> 2 files changed, 11 insertions(+)
>
Reviewed-by: Bin Meng
Tested-by: Bin Meng
Signed-off-by: Bihong Yu
Reviewed-by: Chuan Zheng
Reviewed-by: Dr. David Alan Gilbert
---
migration/rdma.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/migration/rdma.c b/migration/rdma.c
index ca4d315..00eac34 100644
--- a/migration/rdma.c
+++ b/migration/rdma.c
@@ -855,
Signed-off-by: Bihong Yu
Reviewed-by: Chuan Zheng
Reviewed-by: Dr. David Alan Gilbert
---
migration/block.c | 2 +-
migration/ram.c | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/migration/block.c b/migration/block.c
index 4b8576b..273392b 100644
--- a/migration/bloc
On Wed, Oct 14, 2020 at 8:28 AM Alistair Francis
wrote:
>
> Allow the user to specify the main application CPU for the sifive_u
> machine.
>
> Signed-off-by: Alistair Francis
> Reviewed-by: Bin Meng
> ---
> include/hw/riscv/sifive_u.h | 1 +
> hw/riscv/sifive_u.c | 18 +
Signed-off-by: Bihong Yu
Reviewed-by: Chuan Zheng
Reviewed-by: Dr. David Alan Gilbert
---
migration/ram.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index 0aea78f..09178cc 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -101
Signed-off-by: Bihong Yu
Reviewed-by: Chuan Zheng
Reviewed-by: Dr. David Alan Gilbert
---
migration/rdma.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/migration/rdma.c b/migration/rdma.c
index 0eb42b7..ca4d315 100644
--- a/migration/rdma.c
+++ b/migration/rdma.c
@@ -27
Signed-off-by: Bihong Yu
Reviewed-by: Chuan Zheng
Reviewed-by: Dr. David Alan Gilbert
---
migration/migration.c| 4 ++--
migration/postcopy-ram.c | 2 +-
migration/ram.c | 2 +-
migration/savevm.c | 2 +-
migration/vmstate.c | 10 +-
5 files changed, 10 inser
Signed-off-by: Bihong Yu
Reviewed-by: Chuan Zheng
Reviewed-by: Dr. David Alan Gilbert
---
migration/migration.h | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/migration/migration.h b/migration/migration.h
index deb411a..99784b4 100644
--- a/migration/migration.h
+++ b/mig
Signed-off-by: Bihong Yu
Reviewed-by: Chuan Zheng
Reviewed-by: Dr. David Alan Gilbert
---
migration/ram.c| 2 +-
migration/savevm.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index 09178cc..2da2b62 100644
--- a/migration/ram.c
++
Recently I am reading migration related code, find some style problems in
migration directory while using checkpatch.pl to check migration code. Fix the
error style problems.
v2:
- fix Signed-off-by error
- fix printf format error: "%0x" -> "0x%x"
v3:
- change "VMStateDescription * []" to "VMStat
Signed-off-by: Bihong Yu
Reviewed-by: Chuan Zheng
Reviewed-by: Dr. David Alan Gilbert
---
migration/block.c | 2 +-
migration/rdma.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/migration/block.c b/migration/block.c
index 737b649..4b8576b 100644
--- a/migration/block.
more ping ...
On Thu, Oct 15, 2020 at 8:22 PM Ani Sinha wrote:
> Ping ...
>
> On Mon, Oct 12, 2020 at 8:27 PM Ani Sinha wrote:
> >
> > Request to queue this patch for the next pull.
> >
> > On Wed, Oct 7, 2020 at 23:25 Philippe Mathieu-Daudé
> wrote:
> >>
> >> On 10/7/20 6:19 PM, Ani Sinha wro
On 10/16/2020 6:07 PM, Andrew Jones wrote:
On Fri, Oct 16, 2020 at 05:40:02PM +0800, Ying Fang wrote:
On 10/15/2020 3:59 PM, Andrew Jones wrote:
On Thu, Oct 15, 2020 at 10:07:16AM +0800, Ying Fang wrote:
On 10/14/2020 2:08 AM, Andrew Jones wrote:
On Tue, Oct 13, 2020 at 12:11:20PM +000
OK, I will change it to "VMStateDescription *[]". Thank you for your review.
On 2020/10/19 19:59, Markus Armbruster wrote:
> Bihong Yu writes:
>
>> Yes, I used to think "const VMStateDescription *[]" was right, but when I
>> search
>> similar expressions, most of all are "xxx * []". Such as:
>>
Sorry for replying late. I missed this email. I will revise the patch
today. Thanks,
- Green
On Thu, Oct 15, 2020 at 4:01 PM Bin Meng wrote:
>
> On Thu, Oct 15, 2020 at 12:15 PM Green Wan wrote:
> >
> > Add '-drive' support to OTP device. Allow users to assign a raw file
> > as OTP image.
> >
>
The type of input variable is unsigned int
while the printer type is int. So fix incorrect print type.
Signed-off-by: Zhengui li
---
hw/display/virtio-gpu.c | 32
1 file changed, 16 insertions(+), 16 deletions(-)
diff --git a/hw/display/virtio-gpu.c b/hw/display
The type of input variable is unsigned int
while the printer type is int. So fix incorrect print type.
Signed-off-by: Zhengui li
---
hw/virtio/virtio-iommu.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/hw/virtio/virtio-iommu.c b/hw/virtio/virtio-iommu.c
index 21ec63
The type of input variable is unsigned int
while the printer type is int. So fix incorrect print type.
Signed-off-by: Zhengui li
---
hw/vfio/common.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/vfio/common.c b/hw/vfio/common.c
index 13471ae..acc3356 100644
--- a/hw
The type of input variable is unsigned int
while the printer type is int. So fix incorrect print type.
Signed-off-by: Zhengui li
---
hw/virtio/vhost-user.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/hw/virtio/vhost-user.c b/hw/virtio/vhost-user.c
index 9c5b4f7
The type of input variable is unsigned int
while the printer type is int. So fix incorrect print type.
Signed-off-by: Zhengui li
---
contrib/vhost-user-scsi/vhost-user-scsi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/contrib/vhost-user-scsi/vhost-user-scsi.c
b/contrib/
The type of input variable is unsigned int
while the printer type is int. So fix incorrect print type.
Signed-off-by: Zhengui li
---
contrib/vhost-user-gpu/vhost-user-gpu.c | 34 -
1 file changed, 17 insertions(+), 17 deletions(-)
diff --git a/contrib/vhost-user-
Hi, Philippe,
On Fri, Oct 16, 2020 at 10:24 PM Philippe Mathieu-Daudé wrote:
>
> Hi Huacai,
>
> On 10/16/20 8:51 AM, Huacai Chen wrote:
> > Preparing to add Loongson-3 machine support, add Loongson-3's LEFI (a
> > UEFI-like interface for BIOS-Kernel boot parameters) helpers first.
> >
> > Signed-
On 2020/10/19 下午6:42, Eugenio Pérez wrote:
I am able to hit this assertion when a Red Hat 7 guest virtio_net device
raises an "Invalidation" of all the TLB entries. This happens in the
guest's startup if 'intel_iommu=on' argument is passed to the guest
kernel and right IOMMU/ATS devices are dec
Hi, Philippe,
On Fri, Oct 16, 2020 at 11:15 PM Philippe Mathieu-Daudé wrote:
>
> On 10/16/20 8:51 AM, Huacai Chen wrote:
> > From: Jiaxun Yang
> >
> > Our current code assumed the target page size is always 4k
> > when handling PageMask and VPN2, however, variable page size
> > was just added to
On 2020/10/19 21:25, Paolo Bonzini wrote:
> On 19/10/20 14:40, Zhenyu Ye wrote:
>> The kernel backtrace for io_submit in GUEST is:
>>
>> guest# ./offcputime -K -p `pgrep -nx fio`
>> b'finish_task_switch'
>> b'__schedule'
>> b'schedule'
>> b'io_schedule'
>>
Hi,
Does anyone know the reason why raw-format.c doesn't have compression support
(but qcow has the supported added)?
For example, raw image backup with compression, "qemu-img convert -c -O raw
origin.img dist.img", doesn't work.
Thanks,
Wei
On 10/18/20 6:39 PM, Joelle van Dyne wrote:
> From: osy
>
> This allows jailbroken devices with entitlements to switch the option off.
>
> Signed-off-by: Joelle van Dyne
> ---
I can guess why this performs better: half the page table entries and thus half
the tlb entries required. Which for a
On 10/19/20 3:39 PM, Joelle van Dyne wrote:
>> Explicit cast may not be needed here so this could be a macro if caling it
>> differently helps or why don't you just use tcg_mirror_prr_rw directly
>> everywhere?
>
> There are quite a bit of code that depends on tcg_insn_unit * type such as
>
> *tc
On Dienstag, 20. Oktober 2020 01:13:24 CEST Christian Schoenebeck wrote:
> This test case uses a Tlink request to create a hard link to a regular
> file using the 9pfs 'local' fs driver.
>
> Signed-off-by: Christian Schoenebeck
> ---
> tests/qtest/virtio-9p-test.c | 61 ++
On Mon, 19 Oct 2020, Joelle van Dyne wrote:
Explicit cast may not be needed here so this could be a macro if caling it
differently helps or why don't you just use tcg_mirror_prr_rw directly
everywhere?
There are quite a bit of code that depends on tcg_insn_unit * type such as
*tcg_code_ptr_rw(
This test case uses a Tunlinkat request to remove a previously hard
linked file by using the 9pfs 'local' fs driver.
Signed-off-by: Christian Schoenebeck
---
tests/qtest/virtio-9p-test.c | 34 ++
1 file changed, 34 insertions(+)
diff --git a/tests/qtest/virtio-9p
This test case uses a Tlink request to create a hard link to a regular
file using the 9pfs 'local' fs driver.
Signed-off-by: Christian Schoenebeck
---
tests/qtest/virtio-9p-test.c | 61
1 file changed, 61 insertions(+)
diff --git a/tests/qtest/virtio-9p-test
This test case uses a Tunlinkat request to remove a regular file using
the 9pfs 'local' fs driver.
Signed-off-by: Christian Schoenebeck
---
tests/qtest/virtio-9p-test.c | 23 +++
1 file changed, 23 insertions(+)
diff --git a/tests/qtest/virtio-9p-test.c b/tests/qtest/virtio-
This test case uses a Tunlinkat request to remove a symlink using
the 9pfs 'local' fs driver.
Signed-off-by: Christian Schoenebeck
---
tests/qtest/virtio-9p-test.c | 26 ++
1 file changed, 26 insertions(+)
diff --git a/tests/qtest/virtio-9p-test.c b/tests/qtest/virtio-9p
On 10/19/20 4:36 PM, Joelle van Dyne wrote:
> Seems like I missed a few. Sorry about that. Will fix.
I'll keep looking, Just In Case. ;-)
>> s->code_gen_epilogue = tb_ret_addr = s->code_ptr;
In this case, just splitting the chained assignment to two statements is
sufficient to fix.
r~
This test case uses a Tlcreate 9p request to create a regular file inside
host's test directory.
Signed-off-by: Christian Schoenebeck
---
tests/qtest/virtio-9p-test.c | 78
1 file changed, 78 insertions(+)
diff --git a/tests/qtest/virtio-9p-test.c b/tests/qt
Just a bunch of more test case using the 9pfs 'local' fs driver backend,
namely for these 9p requests:
* Tunlinkat, Tlcreate, Tsymlink and Tlink.
Christian Schoenebeck (8):
tests/9pfs: simplify fs_mkdir()
tests/9pfs: add local unlinkat directory test
tests/9pfs: add local Tlcreate test
te
This test case uses a Tsymlink 9p request to create a symbolic link using
the 9pfs 'local' fs driver.
Signed-off-by: Christian Schoenebeck
---
tests/qtest/virtio-9p-test.c | 78
1 file changed, 78 insertions(+)
diff --git a/tests/qtest/virtio-9p-test.c b/tes
Split out walking a directory path to a separate new utility function
fs_walk_fid() and use that function in fs_mkdir().
The code difference saved this way is not much, but we'll use that new
fs_walk_fid() function in the upcoming patches, so it will avoid quite
some code duplication after all.
S
This test case uses a Tunlinkat 9p request with flag AT_REMOVEDIR
(see 'man 2 unlink') to remove a directory from host's test directory.
Signed-off-by: Christian Schoenebeck
---
tests/qtest/virtio-9p-test.c | 72
1 file changed, 72 insertions(+)
diff --git a
Seems like I missed a few. Sorry about that. Will fix.
-j
On Mon, Oct 19, 2020 at 4:27 PM Richard Henderson
wrote:
>
> On 10/18/20 6:39 PM, Joelle van Dyne wrote:
> > From: osy
> >
> > We will introduce mirror mapping for JIT segment with separate RX and RW
> > access. Adding 'const' hints will
On 10/18/20 6:39 PM, Joelle van Dyne wrote:
> From: osy
>
> We will introduce mirror mapping for JIT segment with separate RX and RW
> access. Adding 'const' hints will make it easier to identify read-only
> accesses and allow us to easier catch bugs at compile time in the future.
>
> Signed-off
You can --author "Joelle van Dyne "
-j
On Mon, Oct 19, 2020 at 4:19 PM Richard Henderson
wrote:
>
> On 10/18/20 6:39 PM, Joelle van Dyne wrote:
> > From: osy
> >
> > We will introduce mirror mapping for JIT segment with separate RX and RW
> > access. Adding 'const' hints will make it easier to
On 10/18/20 6:39 PM, Joelle van Dyne wrote:
> From: osy
>
> We will introduce mirror mapping for JIT segment with separate RX and RW
> access. Adding 'const' hints will make it easier to identify read-only
> accesses and allow us to easier catch bugs at compile time in the future.
>
> Signed-off
On Tue, 13 Oct 2020 17:17:30 PDT (-0700), Alistair Francis wrote:
Signed-off-by: Alistair Francis
---
include/hw/riscv/boot.h | 2 ++
hw/riscv/boot.c | 9 +
2 files changed, 11 insertions(+)
diff --git a/include/hw/riscv/boot.h b/include/hw/riscv/boot.h
index 0acbd8aa6e..2975ed
On Tue, 13 Oct 2020 17:17:25 PDT (-0700), Alistair Francis wrote:
Allow the user to specify the main application CPU for the sifive_u
machine.
Signed-off-by: Alistair Francis
Reviewed-by: Bin Meng
---
include/hw/riscv/sifive_u.h | 1 +
hw/riscv/sifive_u.c | 18 +-
2 f
On Tue, 13 Oct 2020 17:17:28 PDT (-0700), Alistair Francis wrote:
Instead of returning the unused entry address from riscv_load_firmware()
instead return the end address. Also return the end address from
riscv_find_and_load_firmware().
This tells the caller if a firmware was loaded and how big i
On Tue, 13 Oct 2020 17:17:33 PDT (-0700), Alistair Francis wrote:
Instead of loading the kernel at a hardcoded start address, let's load
the kernel at the next alligned address after the end of the firmware.
This should have no impact for current users of OpenSBI, but will
allow loading a noMMU
Ping.
On 10/13/20 3:23 PM, Richard Henderson wrote:
> In several cases, it's easy to optimize across a non-taken branch
> simply by *not* flushing the relevant tables. This is true both
> for value propagation and register allocation.
>
> This comes up in quite a number of cases with arm, most s
On 10/19/20 3:44 PM, Owen Anderson wrote:
> My use case is currently using QEMU 4.0, but we will be moving to QEMU
> 4.2 soon. I do not have --enable-tcg-debug enabled.
> e979972a6a1 does look promising, and like it might deliver increased
> performance for our use case. It looks like the code in
This is another layer wrapper for sending a page request to the source VM. The
new migrate_send_rp_message_req_pages() will be used elsewhere in coming
patches.
Reviewed-by: Dr. David Alan Gilbert
Signed-off-by: Peter Xu
---
migration/migration.c | 10 --
migration/migration.h | 2 ++
We synchronize the requested pages right after a postcopy recovery happens.
This helps to synchronize the prioritized pages on source so that the faulted
threads can be served faster.
Reported-by: Xiaohui Li
Signed-off-by: Peter Xu
---
migration/savevm.c | 57 +++
The errors are very useful when debugging qtest failures, especially when
QTEST_LOG=1 is set. Let's allow override MigrateStart.hide_stderr when
QTEST_LOG=1 is specified, because that means the user wants to be verbose.
Not very nice to introduce the first QTEST_LOG env access in migration-test.c
Maintain a list of faulted addresses on the destination host for which we're
waiting on. This is implemented using a GTree rather than a real list to make
sure even there're plenty of vCPUs/threads that are faulting, the lookup will
still be fast with O(log(N)) (because we'll do that after placing
Logically below race could trigger with the old code:
test programmigration thread
wait_until('postcopy-pause')
postcopy_pause()
It'll be used in follow up patches to access more fields out of it. Meanwhile
fetch the userfaultfd inside the function.
Reviewed-by: Dr. David Alan Gilbert
Signed-off-by: Peter Xu
---
migration/postcopy-ram.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/migratio
This is v5 of the series. Probably my first series that got queued/unqueued
twice.
I found a bug in v4 that was about page sizes, however that didn't match with
PeterM's report on big endian hosts. My manual reproduce on s390x also didn't
reproduce. However after I ran the tree (with the fix) o
On Mon, Oct 19, 2020 at 11:22 AM Richard Henderson
wrote:
>
> (1) What qemu version are you looking at and,
> (2) Do you have --enable-tcg-debug enabled?
My use case is a large automated testing environment for large C++
binaries with heavy use of virtual dispatch. The binaries are
generally not
> Explicit cast may not be needed here so this could be a macro if caling it
> differently helps or why don't you just use tcg_mirror_prr_rw directly
> everywhere?
There are quite a bit of code that depends on tcg_insn_unit * type such as
*tcg_code_ptr_rw(s, code_ptr) = insn;
and
(tcg_code_ptr_
Correct me if I'm wrong but wouldn't the following test still fail
with --cross-prefix=""
if test -n "$cross_prefix"; then
...
That was my main reason for making this change.
-j
On Mon, Oct 19, 2020 at 4:24 AM BALATON Zoltan wrote:
>
> On Mon, 19 Oct 2020, Thomas Huth wrote:
> > On 19/10/2020
On Mon, Oct 19, 2020 at 1:27 AM Thomas Huth wrote:
>
> On 19/10/2020 03.39, Joelle van Dyne wrote:
> > From: osy
>
> That "From:" line looks wrong ... could you please fix the "Author" of your
> patches / your git config?
osy wrote the original changes. I joined the UTM project to help bring
the
On 12/10/2020 18:09, Philippe Mathieu-Daudé wrote:
Notes while trying to understand Mark's patch from yesterday:
"sabre: increase number of PCI bus IRQs from 32 to 64"
https://www.mail-archive.com/qemu-devel@nongnu.org/msg749458.html
Philippe Mathieu-Daudé (6):
hw/pci-host/sabre: Update docu
On 12/10/2020 18:09, Philippe Mathieu-Daudé wrote:
Report accesses to reserved registers using qemu_log_mask(GUEST_ERROR).
Signed-off-by: Philippe Mathieu-Daudé
---
hw/pci-host/sabre.c | 8
1 file changed, 8 insertions(+)
diff --git a/hw/pci-host/sabre.c b/hw/pci-host/sabre.c
inde
On 10/12/20 7:33 PM, LIU Zhiwei wrote:
>> +if (a->rw) {
>> +/* WHILERW */
>> +/* diff = abs(op1 - op0), noting that op0/1 are unsigned. */
>> +t1 = tcg_temp_new_i64();
>> +tcg_gen_sub_i64(diff, op0, op1);
>> +tcg_gen_sub_i64(t1, op1, op0);
>> +tcg
On 12/10/2020 18:09, Philippe Mathieu-Daudé wrote:
Signed-off-by: Philippe Mathieu-Daudé
---
hw/pci-host/sabre.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/hw/pci-host/sabre.c b/hw/pci-host/sabre.c
index 4412e23131c..67699ac9058 100644
--- a/hw/pci-host/sabre.c
+++ b/hw/pci-host/s
On 12/10/2020 18:09, Philippe Mathieu-Daudé wrote:
Report unimplemented register accesses using qemu_log_mask(UNIMP).
Signed-off-by: Philippe Mathieu-Daudé
---
hw/pci-host/sabre.c | 12
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/hw/pci-host/sabre.c b/hw/pci-h
On 10/14/20 7:15 AM, Vladimir Sementsov-Ogievskiy wrote:
10.10.2020 00:55, Eric Blake wrote:
Since 'nbd-server-add' is deprecated, and 'block-export-add' is new to
5.2, we can still tweak the interface. Allowing 'bitmaps':['str'] is
nicer than 'bitmap':'str'. This wires up the qapi and qemu-nb
On Thu, Oct 08, 2020 at 07:15:55PM +0200, Jean-Philippe Brucker wrote:
> From: Bharat Bhushan
>
> Allow to set the page size mask supported by an iommu memory region.
> This enables a vIOMMU to communicate the page size granule supported by
> an assigned device, on hosts that use page sizes great
On 10/8/20 10:49 AM, Daniel P. Berrangé wrote:
The traditional HMP "savevm" command will overwrite an existing snapshot
if it already exists with the requested name. This new flag allows this
to be controlled allowing for safer behaviour with a future QMP command.
Signed-off-by: Daniel P. Berran
On Thu, Oct 08, 2020 at 07:15:49PM +0200, Jean-Philippe Brucker wrote:
> Due to an invalid mask, virtio_iommu_mr() may return the wrong memory
> region. It hasn't been too problematic so far because the function was
> only used to test existence of an endpoint, but that is about to change.
>
> Fix
On Thu, Oct 08, 2020 at 07:15:57PM +0200, Jean-Philippe Brucker wrote:
> From: Bharat Bhushan
>
> The virtio-iommu device can deal with arbitrary page sizes for virtual
> endpoints, but for endpoints assigned with VFIO it must follow the page
> granule used by the host IOMMU driver.
>
> Implemen
On 10/8/20 10:49 AM, Daniel P. Berrangé wrote:
Currently bdrv_all_find_snapshot() will return 0 if it finds
a snapshot, -1 if an error occurs, or if it fails to find a
snapshot. New callers to be added want to distinguish between
the error scenario and failing to find a snapshot.
Rename it to bd
1 - 100 of 416 matches
Mail list logo
