On 08/01/2021 20.56, Peter Maydell wrote:
Adding the people to the CC list who were on the previous discussion thread...
-- PMM
On Fri, 8 Jan 2021 at 19:43, Warner Losh wrote:
[...]
So, my new plan is to rebase what changes I can to the tip of master and submit
those for review. I'll work
- Original Message -
> From: "Thomas Huth"
> To: "Miroslav Rezanina"
> Cc: "qemu-s390x" , "Philippe Mathieu-Daudé"
> , qemu-devel@nongnu.org
> Sent: Monday, January 11, 2021 1:54:06 PM
> Subject: Re: [RHEL7 qemu-kvm PATCH 2/3] s390x: Fix vm name copy length
>
> On 11/01/2021 13.42,
- Original Message -
> From: "Philippe Mathieu-Daudé"
> To: mreza...@redhat.com, qemu-devel@nongnu.org
> Sent: Monday, January 11, 2021 1:10:07 PM
> Subject: Re: [RHEL7 qemu-kvm PATCH 2/3] s390x: Fix vm name copy length
>
> Hi Miroslav,
>
> On 1/11/21 12:30 PM, mreza...@redhat.com
On Mon, 4 Jan 2021 21:20:56 +0100
Eric Farman wrote:
> Signed-off-by: Eric Farman
> ---
> .../infiniband/hw/vmw_pvrdma/pvrdma_ring.h| 14 +-
> .../infiniband/hw/vmw_pvrdma/pvrdma_verbs.h | 2 +-
> include/standard-headers/drm/drm_fourcc.h | 175 +-
>
On 11/01/2021 13.42, Miroslav Rezanina wrote:
- Original Message -
From: "Thomas Huth"
To: "Philippe Mathieu-Daudé" , mreza...@redhat.com,
qemu-devel@nongnu.org, "qemu-s390x"
Sent: Monday, January 11, 2021 1:24:57 PM
Subject: Re: [RHEL7 qemu-kvm PATCH 2/3] s390x: Fix vm name copy
- Original Message -
> From: "Philippe Mathieu-Daudé"
> To: mreza...@redhat.com, qemu-devel@nongnu.org
> Cc: "Richard Henderson" , "Eric Blake"
>
> Sent: Monday, January 11, 2021 1:15:04 PM
> Subject: Re: [RHEL7 qemu-kvm PATCH 3/3] Fix tcg_out_op argument mismatch
> warning
>
> On
From: Klaus Jensen
The zoned command set specification states that "All logical blocks in a
zone *shall* be marked as deallocated when [the zone is reset]". Since
the device guarantees 0x00 to be read from deallocated blocks we have to
issue a pwrite_zeroes since we cannot be sure that a discard
From: Klaus Jensen
Add missing string representations for a couple of new commands.
Signed-off-by: Klaus Jensen
---
hw/block/nvme.h | 4
1 file changed, 4 insertions(+)
diff --git a/hw/block/nvme.h b/hw/block/nvme.h
index b7fbcca39d9f..65540b650e1d 100644
--- a/hw/block/nvme.h
+++
- Original Message -
> From: "Thomas Huth"
> To: "Philippe Mathieu-Daudé" , mreza...@redhat.com,
> qemu-devel@nongnu.org, "qemu-s390x"
>
> Sent: Monday, January 11, 2021 1:24:57 PM
> Subject: Re: [RHEL7 qemu-kvm PATCH 2/3] s390x: Fix vm name copy length
>
> On 11/01/2021 13.10,
From: Klaus Jensen
A shutdown is only about flushing stuff. It is the host that should
delete any queues, so do not perform a reset here.
Also, on shutdown, make sure that the PMR is flushed if in use.
Fixes: 368f4e752cf9 ("hw/block/nvme: Process controller reset and shutdown
differently")
From: Klaus Jensen
Align with existing style and use a typedef for header-file enums.
Signed-off-by: Klaus Jensen
---
hw/block/nvme-ns.h | 4 ++--
include/block/nvme.h | 4 ++--
hw/block/nvme.c | 19 +--
3 files changed, 13 insertions(+), 14 deletions(-)
diff --git
From: Klaus Jensen
nvme_io_cmd already checks if the namespace supports the Zone Append
command, so the removed check is dead code.
Signed-off-by: Klaus Jensen
---
hw/block/nvme.c | 4
1 file changed, 4 deletions(-)
diff --git a/hw/block/nvme.c b/hw/block/nvme.c
index
From: Klaus Jensen
These are some follow-up patches to the just merged zoned series.
The biggest addition here is asynchronous zeroing of zones on reset.
Klaus Jensen (6):
hw/block/nvme: fix shutdown/reset logic
hw/block/nvme: merge implicitly/explicitly opened processing masks
From: Klaus Jensen
Implicitly and explicitly opended zones are always bulk processed
together, so merge the two processing masks.
Signed-off-by: Klaus Jensen
---
hw/block/nvme.c | 27 +++
1 file changed, 11 insertions(+), 16 deletions(-)
diff --git a/hw/block/nvme.c
On 11/01/2021 13.10, Philippe Mathieu-Daudé wrote:
Hi Miroslav,
On 1/11/21 12:30 PM, mreza...@redhat.com wrote:
From: Miroslav Rezanina
There are two cases when vm name is copied but closing \0 can be lost
in case name is too long (>=256 characters).
Updating length to copy so there is
On 1/11/21 12:50 PM, Stefan Hajnoczi wrote:
> qemu.org is running out of bandwidth and the QEMU project is moving
> towards a gating CI on GitLab. Use the GitLab repos instead of qemu.org
> (they will become mirrors).
>
> Signed-off-by: Stefan Hajnoczi
> ---
> scripts/get_maintainer.pl | 2 +-
>
On 1/11/21 12:50 PM, Stefan Hajnoczi wrote:
> It is no longer necessary to point .gitmodules at GitLab repos when
> running in GitLab CI since they are now used all the time.
>
> Signed-off-by: Stefan Hajnoczi
> ---
> .gitlab-ci.yml | 1 -
> 1 file changed, 1 deletion(-)
Reviewed-by: Philippe
On 1/11/21 12:50 PM, Stefan Hajnoczi wrote:
> qemu.org is running out of bandwidth and the QEMU project is moving
> towards a gating CI on GitLab. Use the GitLab repos instead of qemu.org
> (they will become mirrors).
>
> Signed-off-by: Stefan Hajnoczi
> ---
> .gitmodules | 44
On 1/11/21 12:30 PM, mreza...@redhat.com wrote:
> From: Miroslav Rezanina
>
> There's prototype mismatch between tcg/tcg.c and tcg/aarch/tcg-target.c.inc:
>
> tcg.c:
>
> static void tcg_out_op(TCGContext *s, TCGOpcode opc, const TCGArg *args,
>const int
Hi Miroslav,
On 1/11/21 12:30 PM, mreza...@redhat.com wrote:
> From: Miroslav Rezanina
>
> There are two cases when vm name is copied but closing \0 can be lost
> in case name is too long (>=256 characters).
>
> Updating length to copy so there is space for closing \0.
>
> Signed-off-by:
Shreyansh Chouhan writes:
> -- Forwarded message -
> From: Shreyansh Chouhan
> Date: Mon, 11 Jan 2021 at 11:59
> Subject: Re: VirtioSound device emulation implementation
> To: Gerd Hoffmann
>
>
>
>
> On Sun, 10 Jan 2021 at 13:55, Shreyansh Chouhan <
>
On Mon, Jan 11, 2021 at 11:50:12AM +, Stefan Hajnoczi wrote:
> Use the GitLab repo URL as the main repo location in order to reduce
> load on qemu.org.
>
> Signed-off-by: Stefan Hajnoczi
> ---
> .github/lockdown.yml | 8
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff
qemu.org is running out of bandwidth and the QEMU project is moving
towards a gating CI on GitLab. Use the GitLab repos instead of qemu.org
(they will become mirrors).
Signed-off-by: Stefan Hajnoczi
---
scripts/get_maintainer.pl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
qemu.org is running out of bandwidth and the QEMU project is moving
towards a gating CI on GitLab. Use the GitLab repos instead of qemu.org
(they will become mirrors).
Signed-off-by: Stefan Hajnoczi
---
README.rst | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
It is no longer necessary to point .gitmodules at GitLab repos when
running in GitLab CI since they are now used all the time.
Signed-off-by: Stefan Hajnoczi
---
.gitlab-ci.yml | 1 -
1 file changed, 1 deletion(-)
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 01c9e46410..4c2a71b934 100644
qemu.org is running out of bandwidth and the QEMU project is moving
towards a gating CI on GitLab. Use the GitLab repos instead of qemu.org
(they will become mirrors).
Signed-off-by: Stefan Hajnoczi
---
pc-bios/README | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
qemu.org is running out of bandwidth and the QEMU project is moving
towards a gating CI on GitLab. Use the GitLab repos instead of qemu.org
(they will become mirrors).
Signed-off-by: Stefan Hajnoczi
---
.gitmodules | 44 ++--
1 file changed, 22
Use the GitLab repo URL as the main repo location in order to reduce
load on qemu.org.
Signed-off-by: Stefan Hajnoczi
---
.github/lockdown.yml | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/.github/lockdown.yml b/.github/lockdown.yml
index 9acc393f1c..07fc2f31ee
Several places in qemu.git reference qemu.org git repo URLs. Let's switch to
GitLab repo URLs in order to enable GitLab as a gating CI and in order to
reduce qemu.org bandwidth consumption.
Paolo has already set up GitLab mirror repos. sgabios was missing and I added
that today. It is now
Dear QEMU, KVM, and rust-vmm community,
QEMU will apply for Google Summer of Code
(https://summerofcode.withgoogle.com/) again this year. This internship
program offers paid, 10-week, remote work internships for
contributing to open source. QEMU can act as an umbrella organization
for KVM kernel
Patchew URL: https://patchew.org/QEMU/cover.1610364304.git.mreza...@redhat.com/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: cover.1610364304.git.mreza...@redhat.com
Subject: [RHEL7 qemu-kvm PATCH 0/3] Fixing several
From: Miroslav Rezanina
When building qemu with GCC 11, compiling eth.c file produce following warning:
warning: array subscript 'struct ip6_ext_hdr_routing[0]' is partly outside
array bounds of 'struct ip6_ext_hdr[1]' [-Warray-bounds]
This caused by retyping from ip6_ext_hdr to
From: Miroslav Rezanina
There's prototype mismatch between tcg/tcg.c and tcg/aarch/tcg-target.c.inc:
tcg.c:
static void tcg_out_op(TCGContext *s, TCGOpcode opc, const TCGArg *args,
const int *const_args);
tcg-target.c.inc:
static void tcg_out_op(TCGContext
From: Miroslav Rezanina
There are two cases when vm name is copied but closing \0 can be lost
in case name is too long (>=256 characters).
Updating length to copy so there is space for closing \0.
Signed-off-by: Miroslav Rezanina
---
target/s390x/kvm.c | 2 +-
From: Miroslav Rezanina
Compiling qemu using GCC 11 we got several new warnings. To allow
build with --enable-werror, we need to solve issues generating these
warnings.
Signed-of-by: Miroslav Rezanina
Miroslav Rezanina (3):
Fix net.c warning on GCC 11
s390x: Fix vm name copy length
Fix
On Jan 11 17:49, zhenwei pi wrote:
> On 1/11/21 5:21 PM, Klaus Jensen wrote:
> > On Jan 11 10:14, Philippe Mathieu-Daudé wrote:
> > > On 1/11/21 8:50 AM, zhenwei pi wrote:
> > > > There is a very low probability that hitting physical NVMe disk
> > > > hardware critical warning case, it's hard to
On Mon, 2021-01-11 at 11:26 +0100, Thomas Huth wrote:
> On 11/01/2021 10.39, Qiuhao Li wrote:
> > On Mon, 2021-01-11 at 10:01 +0100, Philippe Mathieu-Daudé wrote:
> > > On 1/11/21 7:11 AM, Qiuhao Li wrote:
> > > > Simplifying the crash cases by opportunistically setting bits
> > > > in
> > > >
On Fri, Jan 08, 2021 at 05:12:08PM +, Peter Maydell wrote:
> This patchseries converts the PPC UIC "Universal Interrupt
> Controller" to a QOM device. My main reason for doing it is that
> this fixes a couple of long-standing trivial Coverity issues -- the
> current ppcuic_init() function
On 11/01/21 10:57, Stefan Hajnoczi wrote:
I will send a patch to point .gitmodules at GitLab. In the process of
doing this I noticed sgabios.git is not yet mirrored on GitLab. After I
create that repo (and any other missing repos) I will send a qemu.git
patch series that references the GitLab
On Mon, 11 Jan 2021, Peter Maydell wrote:
On Mon, 11 Jan 2021 at 10:20, BALATON Zoltan wrote:
On Mon, 11 Jan 2021, Jiaxun Yang wrote:
On Mon, Jan 11, 2021, at 8:36 AM, Huacai Chen wrote:
I think R_END should be 0x60, Jiaxun, what do you think?
U r right.
The manual is misleading.
The
On Mon, 11 Jan 2021 at 10:20, BALATON Zoltan wrote:
>
> On Mon, 11 Jan 2021, Jiaxun Yang wrote:
> > On Mon, Jan 11, 2021, at 8:36 AM, Huacai Chen wrote:
> >> I think R_END should be 0x60, Jiaxun, what do you think?
> >
> > U r right.
> > The manual is misleading.
>
> The R_END constant is also
The mingw-w64-x86_64-python-sphinx-2.3.1 package has been removed from
the server, so the windows_msys2_task is currently failing. Replace the
old version with the current generic one to fix the issue (the current
version seems to work fine now, too).
Signed-off-by: Thomas Huth
---
Successful
On Fri, Jan 8, 2021 at 7:51 PM Leif Lindholm wrote:
>
> When FEAT_MTE is implemented, the AArch64 view of CTR_EL0 adds the
> TminLine field in bits [37:32].
> Extend the ctr field to be able to hold this context.
>
> Signed-off-by: Leif Lindholm
Reviewed-by: Laurent Desnogues
Thanks,
Laurent
On Fri, Jan 8, 2021 at 7:51 PM Leif Lindholm wrote:
>
> Signed-off-by: Leif Lindholm
Reviewed-by: Laurent Desnogues
Thanks,
Laurent
> ---
> target/arm/cpu.h | 31 +++
> 1 file changed, 31 insertions(+)
>
> diff --git a/target/arm/cpu.h b/target/arm/cpu.h
>
On Mon, 11 Jan 2021, Jiaxun Yang wrote:
On Mon, Jan 11, 2021, at 3:25 AM, BALATON Zoltan wrote:
On Sun, 10 Jan 2021, Philippe Mathieu-Daudé wrote:
+PCI experts
On 1/10/21 1:43 AM, BALATON Zoltan wrote:
On Sun, 10 Jan 2021, Philippe Mathieu-Daudé wrote:
[...]
I'm not a PCI expert but my
On 11/01/2021 10.39, Qiuhao Li wrote:
On Mon, 2021-01-11 at 10:01 +0100, Philippe Mathieu-Daudé wrote:
On 1/11/21 7:11 AM, Qiuhao Li wrote:
Simplifying the crash cases by opportunistically setting bits in
operands of
out/write to zero may help to debug, since usually bit one means
turn on or
On Mon, 11 Jan 2021, Jiaxun Yang wrote:
On Mon, Jan 11, 2021, at 8:36 AM, Huacai Chen wrote:
I think R_END should be 0x60, Jiaxun, what do you think?
U r right.
The manual is misleading.
The R_END constant is also used in loongson_liointc_init() for the length
of the memory region so you
On 11.01.21 11:05, Laszlo Ersek wrote:
> On 01/11/21 10:46, Christian Borntraeger wrote:
>>
>>
>> On 04.01.21 14:24, Laszlo Ersek wrote:
>>> virtio-fs qualifies as a bootable device minimally under OVMF, but
>>> currently the necessary "bootindex" property is missing (fw_cfg kernel
>>> boot
On 01/11/21 10:46, Christian Borntraeger wrote:
>
>
> On 04.01.21 14:24, Laszlo Ersek wrote:
>> virtio-fs qualifies as a bootable device minimally under OVMF, but
>> currently the necessary "bootindex" property is missing (fw_cfg kernel
>> boot notwithstanding).
>>
>> Add the property. For
+Peter
On 1/11/21 2:11 AM, Nathan Chancellor wrote:
> On Tue, Sep 01, 2020 at 12:40:42PM +0200, Philippe Mathieu-Daudé wrote:
>> We call pci_register_root_bus() to register 4 IRQs with the
>> ppc4xx_pci_set_irq() handler. As it can only be called with
>> values in the [0-4[ range, replace the
On Tue, Jan 05, 2021 at 02:12:59PM +, Peter Maydell wrote:
> On Thu, 22 Oct 2020 at 17:48, Paolo Bonzini wrote:
> > now that Gitlab is the primary CI infrastructure for QEMU, and that all
> > QEMU git repositories (including mirrors) are available on Gitlab, I
> > would like to propose that
This should always successfully write exactly two 32-bit integers.
Make it clear with an assert(), like v9fs_receive_status() and
v9fs_receive_response() already do when unmarshalling the same
header.
Fixes: Coverity CID 1438968
Signed-off-by: Greg Kurz
---
hw/9pfs/9p-proxy.c |3 ++-
1 file
On 1/11/21 5:21 PM, Klaus Jensen wrote:
On Jan 11 10:14, Philippe Mathieu-Daudé wrote:
On 1/11/21 8:50 AM, zhenwei pi wrote:
There is a very low probability that hitting physical NVMe disk
hardware critical warning case, it's hard to write & test a monitor
agent service.
For debugging
On 04.01.21 14:24, Laszlo Ersek wrote:
> virtio-fs qualifies as a bootable device minimally under OVMF, but
> currently the necessary "bootindex" property is missing (fw_cfg kernel
> boot notwithstanding).
>
> Add the property. For completeness, add it to the CCW device as well;
> other
On Mon, 2021-01-11 at 10:01 +0100, Philippe Mathieu-Daudé wrote:
> On 1/11/21 7:11 AM, Qiuhao Li wrote:
> > Simplifying the crash cases by opportunistically setting bits in
> > operands of
> > out/write to zero may help to debug, since usually bit one means
> > turn on or
> > trigger a function
On Jan 11 10:14, Philippe Mathieu-Daudé wrote:
> On 1/11/21 8:50 AM, zhenwei pi wrote:
> > There is a very low probability that hitting physical NVMe disk
> > hardware critical warning case, it's hard to write & test a monitor
> > agent service.
> >
> > For debugging purposes, add a new
On 10/01/21 00:34, Philippe Mathieu-Daudé wrote:
Use the dedicated X86Seg enum type for segment registers.
Signed-off-by: Philippe Mathieu-Daudé
---
target/i386/cpu.h| 4 ++--
target/i386/gdbstub.c| 2 +-
target/i386/tcg/seg_helper.c | 8
On 08.01.21 20:21, Guenter Roeck wrote:
> On 1/8/21 5:20 AM, David Hildenbrand wrote:
>> This series fixes booting current upstream Linux kernel compiled by
>> clang-11 and clang-12 under TCG.
>>
>> Decided to pull in already separatly sent patches. The last patch is
>> not required to fix the
On 1/11/21 8:50 AM, zhenwei pi wrote:
> There is a very low probability that hitting physical NVMe disk
> hardware critical warning case, it's hard to write & test a monitor
> agent service.
>
> For debugging purposes, add a new 'smart_critical_warning' property
> to emulate this situation.
>
>
On 1/11/21 7:11 AM, Qiuhao Li wrote:
> Simplifying the crash cases by opportunistically setting bits in operands of
> out/write to zero may help to debug, since usually bit one means turn on or
> trigger a function while zero is the default turn-off setting.
>
> Tested Bug 1908062.
Please use
Ping
On 01/04/21 14:24, Laszlo Ersek wrote:
> virtio-fs qualifies as a bootable device minimally under OVMF, but
> currently the necessary "bootindex" property is missing (fw_cfg kernel
> boot notwithstanding).
>
> Add the property. For completeness, add it to the CCW device as well;
> other
> >> +{ 'command': 'change-vnc-authz',
> >> + 'data': { 'type' : 'str',
> >> +'index': 'str'},
> >> + 'if': 'defined(CONFIG_VNC_SASL)' }
> >
> > type should be an enum I think.
> It is a good idea to set type to enum, I will fix it in the next version.
>
> > Also index should be an
301 - 362 of 362 matches
Mail list logo