On Sun, Mar 14, 2021 at 01:23:14PM +0100, vit...@cyberhaven.com wrote:
> From: Vitaly Chipounov
>
> This enables higher resolutions.
No. virtio-vga supports higher resolutions just fine once the guest
driver is loaded. The video memory is used at boot only, before the
guest driver is loaded, a
On 15/03/2021 07.43, Mahmoud Mandour wrote:
If it's unrelated, then maybe better do it in a separate patch.
I thought so but I didn't know whether it was a so-small change
that it didn't require its own patch or not. I will amend that.
Since this is only a very small allocation, I thin
On Sun, Mar 14, 2021 at 04:01:47PM +0900, Akihiko Odaki wrote:
> ui/cocoa used to raise all keys before it resigns active to prevent a
> stuck key problem caused by key up events it does not see while it is
> inactive. The problem is solved by checking -[NSEvent modifierFlags] in
> commit 6d73bb643
> +typedef struct Timings {
> +static void generate_timings(Timings *timings, uint32_t refresh_rate,
> + uint32_t xres, uint32_t yres)
Adding these should be splitted to a separate patch.
> -static void edid_checksum(uint8_t *edid)
> +static void edid_checksum(uint8_t
Working well now. Thank you.
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1916112
Title:
Illegal instruction crash of QEMU on Jetson Nano
Status in QEMU:
Fix Committed
Bug description:
I hav
The minimum Ethernet frame length is 60 bytes. For short frames with
smaller length like ARP packets (only 42 bytes), on a real world NIC
it can choose either padding its length to the minimum required 60
bytes, or sending it out directly to the wire. Such behavior can be
hardcoded or controled by
Add a new macro ETH_ZLEN which represents the minimum size of an
Ethernet frame without FCS.
Signed-off-by: Bin Meng
---
include/net/eth.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/net/eth.h b/include/net/eth.h
index 0671be6916..7c825ecb2f 100644
--- a/include/net/eth.h
+++ b/
Now that we have implemented unified short frames padding in the
QEMU networking codes, remove the same logic in the NIC codes.
Signed-off-by: Bin Meng
---
hw/net/ne2000.c | 12
1 file changed, 12 deletions(-)
diff --git a/hw/net/ne2000.c b/hw/net/ne2000.c
index 6c17ee1ae2..b0a120
Do the same for tap backend as what we did for slirp.
Signed-off-by: Bin Meng
---
net/tap-win32.c | 12
net/tap.c | 12
2 files changed, 24 insertions(+)
diff --git a/net/tap-win32.c b/net/tap-win32.c
index 2b5dcda36e..ec35ab8ce7 100644
--- a/net/tap-win32.c
+++
This adds a flag in NetClientState, so that a net client can tell
its peer that the packets do not need to be padded to the minimum
size of an Ethernet frame (60 bytes) before sending to it.
Signed-off-by: Bin Meng
---
include/net/net.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/includ
For virtio-net, there is no need to pad the Ethernet frame size to
60 bytes before sending to it.
Signed-off-by: Bin Meng
---
hw/net/virtio-net.c | 4
1 file changed, 4 insertions(+)
diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
index 96a3cc8357..66b9ff4511 100644
--- a/hw/net/vi
Now that we have implemented unified short frames padding in the
QEMU networking codes, remove the same logic in the NIC codes.
Signed-off-by: Bin Meng
---
hw/net/pcnet.c | 9 -
1 file changed, 9 deletions(-)
diff --git a/hw/net/pcnet.c b/hw/net/pcnet.c
index f3f18d8598..16330335cd 100
The minimum Ethernet frame length is 60 bytes. For short frames with
smaller length like ARP packets (only 42 bytes), on a real world NIC
it can choose either padding its length to the minimum required 60
bytes, or sending it out directly to the wire. Such behavior can be
hardcoded or controled by
Now that we have implemented unified short frames padding in the
QEMU networking codes, remove the same logic in the NIC codes.
This actually reverts commit 78aeb23eded2d0b765bf9145c71f80025b568acd.
Signed-off-by: Bin Meng
---
hw/net/e1000.c | 11 +--
1 file changed, 1 insertion(+), 10
Now that we have implemented unified short frames padding in the
QEMU networking codes, remove the same logic in the NIC codes.
This actually reverts commit 40a87c6c9b11ef9c14e0301f76abf0eb2582f08e.
Signed-off-by: Bin Meng
---
hw/net/vmxnet3.c | 10 --
1 file changed, 10 deletions(-)
Now that we have implemented unified short frames padding in the
QEMU networking codes, remove the same logic in the NIC codes.
Signed-off-by: Bin Meng
---
hw/net/rtl8139.c | 12
1 file changed, 12 deletions(-)
diff --git a/hw/net/rtl8139.c b/hw/net/rtl8139.c
index 4675ac878e..cbf
From: Miroslav Rezanina
When declaring g_autofree variable without inicialization, compiler
will raise "may be used uninitialized in this function" warning due
to automatic free handling.
This is mentioned in docs/devel/style.rst (quote from section
"Automatic memory deallocation"):
* Variabl
Now that we have implemented unified short frames padding in the
QEMU networking codes, remove the same logic in the NIC codes.
Signed-off-by: Bin Meng
---
hw/net/i82596.c | 18 --
1 file changed, 18 deletions(-)
diff --git a/hw/net/i82596.c b/hw/net/i82596.c
index 055c3a1470..
Now that we have implemented unified short frames padding in the
QEMU networking codes, remove the same logic in the NIC codes.
Signed-off-by: Bin Meng
---
hw/net/sungem.c | 14 --
1 file changed, 14 deletions(-)
diff --git a/hw/net/sungem.c b/hw/net/sungem.c
index 33c3722df6..3fa8
在 2021/3/15 下午1:39, Thomas Huth 写道:
On 14/03/2021 12.37, Peter Maydell wrote:
On Fri, 12 Mar 2021 at 06:16, Jason Wang wrote:
The following changes since commit
f4abdf32714d1845b7c01ec136dd2b04c2f7db47:
Merge remote-tracking branch
'remotes/stsquad/tags/pull-testing-docs-xen-updates-
Now that we have implemented unified short frames padding in the
QEMU networking codes, remove the same logic in the NIC codes.
Signed-off-by: Bin Meng
---
hw/net/sunhme.c | 11 ---
1 file changed, 11 deletions(-)
diff --git a/hw/net/sunhme.c b/hw/net/sunhme.c
index fc34905f87..697179
On 15/03/2021 09.00, mreza...@redhat.com wrote:
From: Miroslav Rezanina
When declaring g_autofree variable without inicialization, compiler
will raise "may be used uninitialized in this function" warning due
to automatic free handling.
This is mentioned in docs/devel/style.rst (quote from sect
在 2021/3/12 下午6:29, Alexey Kirillov 写道:
ping
Patchew page:
https://patchew.org/QEMU/20210303095910.78277-1-lekir...@yandex-team.ru
Applied.
Thanks
03.03.2021, 13:01, "Alexey Kirillov" :
This patch series introduces a new QMP command "query-netdev" to get
information about currently a
On Mon, Mar 15, 2021 at 09:08:01AM +0100, Thomas Huth wrote:
> On 15/03/2021 09.00, mreza...@redhat.com wrote:
> > From: Miroslav Rezanina
> >
> > When declaring g_autofree variable without inicialization, compiler
> > will raise "may be used uninitialized in this function" warning due
> > to aut
Based on work from Richard and Thomas.
v6 -> v7:
- "target/s390x: Implement the MVPG condition-code-option bit"
-- Fix PGM_PROTECTION for QEMU_USER_ONLY (overridden by patch #2)
v5 -> v6:
- "target/s390x: Implement the MVPG condition-code-option bit"
-- Better handle CONFIG_USER_ONLY
-- Factor ha
From: Richard Henderson
If the CCO bit is set, MVPG should not generate an exception but
report page translation faults via a CC code.
Create a new helper, access_prepare_nf, which can use probe_access_flags
in non-faulting mode, and then handle watchpoints.
Reviewed-by: Richard Henderson
Sign
The PoP states:
When EDAT-1 does not apply, and a program interruption due to a
page-translation exception is recognized by the MOVE PAGE
instruction, the contents of the R1 field of the instruction are
stored in bit positions 0-3 of location 162, and the contents of
the R2 fie
On 04.03.21 11:55, David Hildenbrand wrote:
Fix and cleanup initializing the maximum size of mutable ACPI blobs.
v1/v2 -> v3:
- 'acpi: Set proper maximum size for "etc/table-loader" blob'
-- Move "etc/table-loader" change to separate patch
-- Extend description
-- Fixup maximum size (now really
Hi Philippe,
It seems that vmstate_ecc_state and vmstate_x86_cpu can also be constified.
Found by .
Thanks,
Keqian
On 2021/3/14 1:11, Philippe Mathieu-Daudé wrote:
> VMStateDescription isn't supposed to be modified.
>
>
>
> Philippe Mathieu-Daudé (3):
>
> hw/arm: Constify VMStateDescriptio
On 3/15/21 12:48 AM, Richard Henderson wrote:
> Use fma to simulatneously scale and round up fraction.
"simultaneously"
> The libm function will always return a properly rounded double precision
> value, which will eliminate any extra precision the x87 co-processor may
> give us, which will keep
On 3/15/21 8:57 AM, Bin Meng wrote:
> Add a new macro ETH_ZLEN which represents the minimum size of an
> Ethernet frame without FCS.
>
> Signed-off-by: Bin Meng
> ---
>
> include/net/eth.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/include/net/eth.h b/include/net/eth.h
> index 06
Currently, the default msix vectors for virtio-net-pci is 3 which is
obvious not suitable for multiqueue guest, so we depends on the user
or management tools to pass a correct vectors parameter. In fact, we
can simplifying this by calculating the number of vectors on realize.
Consider we have N qu
The following changes since commit 6157b0e19721aadb4c7fdcfe57b2924af6144b14:
Merge remote-tracking branch
'remotes/vivier2/tags/linux-user-for-6.0-pull-request' into staging (2021-03-14
17:47:49 +)
are available in the git repository at:
https://github.com/jasowang/qemu.git tags/net-pu
From: Bin Meng
"qemu-common.h" should be included to provide the forward declaration
of qemu_hexdump() when DEBUG_NET is on.
Signed-off-by: Bin Meng
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Jason Wang
---
net/net.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/net/net.c b/net
This patch switches to use qemu_receive_receive_iov() which can detect
reentrancy and return early.
This is intended to address CVE-2021-3416.
Cc: Prasad J Pandit
Cc: qemu-sta...@nongnu.org
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Jason Wang
---
hw/net/net_tx_pkt.c | 2 +-
1 file ch
Some NIC supports loopback mode and this is done by calling
nc->info->receive() directly which in fact suppresses the effort of
reentrancy check that is done in qemu_net_queue_send().
Unfortunately we can't use qemu_net_queue_send() here since for
loopback there's no sender as peer, so this patch
From: Paolo Bonzini
When a network or network device is created from the command line or HMP,
QemuOpts ensures that the id passes the id_wellformed check. However,
QMP skips this:
$ qemu-system-x86_64 -qmp stdio -S -nic user,id=123/456
qemu-system-x86_64: -nic user,id=123/456: Parameter i
During procss_tx_desc(), driver can try to chain data descriptor with
legacy descriptor, when will lead underflow for the following
calculation in process_tx_desc() for bytes:
if (tp->size + bytes > msh)
bytes = msh - tp->size;
This will lead a infinite loop. So check
From: Alexander Bulekov
This patch switches to use qemu_receive_packet() which can detect
reentrancy and return early.
This is intended to address CVE-2021-3416.
Cc: Prasad J Pandit
Cc: qemu-sta...@nongnu.org
Buglink: https://bugs.launchpad.net/qemu/+bug/1917085
Reviewed-by: Philippe Mathieu-D
This patch switches to use qemu_receive_packet() which can detect
reentrancy and return early.
This is intended to address CVE-2021-3416.
Cc: Prasad J Pandit
Cc: qemu-sta...@nongnu.org
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Jason Wang
---
hw/net/msf2-emac.c | 2 +-
1 file changed,
This patch switches to use qemu_receive_packet() which can detect
reentrancy and return early.
This is intended to address CVE-2021-3416.
Cc: Prasad J Pandit
Cc: qemu-sta...@nongnu.org
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Jason Wang
---
hw/net/e1000.c | 2 +-
1 file changed, 1 i
From: Alexey Kirillov
The info_str field of the NetClientState structure is static and has a size
of 256 bytes. This amount is often unclaimed, and the field itself is used
exclusively for HMP "info network".
The patch translates info_str to dynamic memory allocation.
This action is also allows
This patch switches to use qemu_receive_packet() which can detect
reentrancy and return early.
This is intended to address CVE-2021-3416.
Cc: Prasad J Pandit
Cc: qemu-sta...@nongnu.org
Reviewed-by: Philippe Mathieu-Daudé
---
hw/net/dp8393x.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-
This patch switches to use qemu_receive_packet() which can detect
reentrancy and return early.
This is intended to address CVE-2021-3416.
Cc: Prasad J Pandit
Cc: qemu-sta...@nongnu.org
Reviewed-by: Mark Cave-Ayland
Reviewed-by: Philippe Mathieu-Daudé
Reviewed-by: Alistair Francis
Signed-off-b
From: Alexander Bulekov
This patch switches to use qemu_receive_packet() which can detect
reentrancy and return early.
This is intended to address CVE-2021-3416.
Cc: Prasad J Pandit
Cc: qemu-sta...@nongnu.org
Buglink: https://bugs.launchpad.net/qemu/+bug/1910826
Reviewed-by: Philippe Mathieu-D
On 3/15/21 10:17 AM, Philippe Mathieu-Daudé wrote:
> On 3/15/21 8:57 AM, Bin Meng wrote:
>> This adds a flag in NetClientState, so that a net client can tell
>> its peer that the packets do not need to be padded to the minimum
>> size of an Ethernet frame (60 bytes) before sending to it.
>>
>> Sign
From: Alexey Kirillov
The query-netdev command is used to get the configuration of the current
network device backends (netdevs).
This is the QMP analog of the HMP command "info network" but only for
netdevs (i.e. excluding NIC and hubports).
The query-netdev command returns an array of objects
From: Alexander Bulekov
This patch switches to use qemu_receive_packet() which can detect
reentrancy and return early.
This is intended to address CVE-2021-3416.
Cc: Prasad J Pandit
Cc: qemu-sta...@nongnu.org
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Alexander Bulekov
Signed-off-by:
On 3/15/21 8:57 AM, Bin Meng wrote:
> This adds a flag in NetClientState, so that a net client can tell
> its peer that the packets do not need to be padded to the minimum
> size of an Ethernet frame (60 bytes) before sending to it.
>
> Signed-off-by: Bin Meng
> ---
>
> include/net/net.h | 1 +
From: Alexey Kirillov
Replace usage of legacy field info_str of NetClientState for backend
network devices with QAPI NetdevInfo stored_config that already used
in QMP query-netdev.
This change increases the detail of the "info network" output and takes
a more general approach to composing the ou
On 3/15/21 5:55 AM, Taylor Simpson wrote:
> Signed-of-by: Taylor Simpson
'git-commit -s' flag add the S-o-b line automatically ;)
Otherwise:
Reviewed-by: Philippe Mathieu-Daudé
> ---
> target/hexagon/op_helper.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/target/
From: Alexander Bulekov
This patch switches to use qemu_receive_packet() which can detect
reentrancy and return early.
This is intended to address CVE-2021-3416.
Cc: Prasad J Pandit
Cc: qemu-sta...@nongnu.org
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Jason Wang
---
hw/net/lan9118.c
From: Alexey Kirillov
As we use QAPI NetClientState->stored_config to store and get information
about backend network devices, we can drop fill of legacy field info_str
for them.
We still use info_str field for NIC and hubports, so we can not completely
remove it.
Signed-off-by: Alexey Kirillov
From: Cornelia Huck
The pvrdma code relies on the pvrdma_ring.h kernel header for some
basic ring buffer handling. The content of that header isn't very
exciting, but contains some (q)atomic_*() invocations that (a)
cause manual massaging when doing a headers update, and (b) are
an indication tha
The change to edid_checksum is needed because the DisplayID section has another
checksum inside for the actually used part of its 128-byte extension block. The
checksum in this case uses the same algorithm, but for a shorter block. Thus I
added a parameter to specify the size of the block.
I'll
From: Alexey Kirillov
A simply qtest that checks for correct number of netdevs in the response
of the query-netdev.
Signed-off-by: Alexey Kirillov
Acked-by: Thomas Huth
Signed-off-by: Jason Wang
---
tests/qtest/meson.build | 3 +
tests/qtest/test-query-netdev.c | 120 ++
On 3/15/21 10:18 AM, Philippe Mathieu-Daudé wrote:
> On 3/15/21 10:17 AM, Philippe Mathieu-Daudé wrote:
>> On 3/15/21 8:57 AM, Bin Meng wrote:
>>> This adds a flag in NetClientState, so that a net client can tell
>>> its peer that the packets do not need to be padded to the minimum
>>> size of an E
On 15/03/2021 09.54, David Hildenbrand wrote:
Based on work from Richard and Thomas.
v6 -> v7:
- "target/s390x: Implement the MVPG condition-code-option bit"
-- Fix PGM_PROTECTION for QEMU_USER_ONLY (overridden by patch #2)
I just double-checked my version of the MVPG kvm-unit-test and some Li
Paolo Bonzini writes:
> On 13/03/21 14:28, Markus Armbruster wrote:
>> Kevin Wolf writes:
>>
>>> This switches the HMP command object_add from a QemuOpts-based parser to
>>> user_creatable_add_from_str() which uses a keyval parser and enforces
>>> the QAPI schema.
>>>
>>> Apart from being a cle
On 3/15/21 5:55 AM, Taylor Simpson wrote:
> Address feedback from Richard Henderson <
>
Reported-by: Richard Henderson
Reviewed-by: Philippe Mathieu-Daudé
> Signed-off-by: Taylor Simpson
> ---
> target/hexagon/gen_tcg.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --
Hi Philippe,
On 3/14/21 1:00 AM, Philippe Mathieu-Daudé wrote:
> Hi Claudio,
>
> On 2/4/21 5:39 PM, Claudio Fontana wrote:
>> v15 -> v16:
>>
>> * cpu: Move synchronize_from_tb() to tcg_ops:
>> - adjusted comments (Alex)
>>
>> * cpu: tcg_ops: move to tcg-cpu-ops.h, keep a pointer in CPUClass:
>>
The following changes since commit 6157b0e19721aadb4c7fdcfe57b2924af6144b14:
Merge remote-tracking branch 'remotes/vivier2/tags/linux-user-for-6.0-pull-=
request' into staging (2021-03-14 17:47:49 +)
are available in the Git repository at:
https://gitlab.com/stefanha/qemu.git tags/block-
On Sun, Mar 14, 2021 at 05:23:22AM +0200, Mahmoud Mandour wrote:
> Replaced a call to calloc() and its respective free() call
> with GLib's g_try_new0() and g_free() calls.
>
> Signed-off-by: Mahmoud Mandour
> ---
> tools/virtiofsd/buffer.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletion
From: Akihiko Odaki
Report the configured granularity for discard operation to the
guest. If this is not set use the block size.
Since until now we have ignored the configured discard granularity
and always reported the block size, let's add
'report-discard-granularity' property and disable it f
On Sun, Mar 14, 2021 at 05:23:23AM +0200, Mahmoud Mandour wrote:
> Replaced a malloc() call and its respective free() with
> GLib's g_try_malloc() and g_free() calls.
>
> Signed-off-by: Mahmoud Mandour
> ---
> tools/virtiofsd/fuse_opt.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
On Mon, Mar 15, 2021 at 9:10 AM Jason Wang wrote:
> 在 2021/3/15 下午1:39, Thomas Huth 写道:
> > On 14/03/2021 12.37, Peter Maydell wrote:
> >> On Fri, 12 Mar 2021 at 06:16, Jason Wang wrote:
> >>>
> >>> The following changes since commit
> >>> f4abdf32714d1845b7c01ec136dd2b04c2f7db47:
> >>>
> >>>
On 3/15/21 10:44 AM, Claudio Fontana wrote:
> Hi Philippe,
>
> On 3/14/21 1:00 AM, Philippe Mathieu-Daudé wrote:
>> Hi Claudio,
>>
>> I am looking at this code:
>>
>> $ git grep tcg_ softmmu/physmem.c
>> softmmu/physmem.c:153:static void
>> tcg_log_global_after_sync(MemoryListener *listener);
>>
On 12.03.21 19:43, Vladimir Sementsov-Ogievskiy wrote:
12.03.2021 21:15, Max Reitz wrote:
On 05.03.21 18:35, Vladimir Sementsov-Ogievskiy wrote:
Compressed writes are unaligned to 512, which works very slow in
O_DIRECT mode. Let's use the cache.
Signed-off-by: Vladimir Sementsov-Ogievskiy
---
On Sun, Mar 14, 2021 at 05:23:24AM +0200, Mahmoud Mandour wrote:
> @@ -130,7 +130,7 @@ static struct fuse_req *fuse_ll_alloc_req(struct
> fuse_session *se)
> {
> struct fuse_req *req;
>
> -req = (struct fuse_req *)calloc(1, sizeof(struct fuse_req));
> +req = g_try_new(struct fuse_r
On Sun, 14 Mar 2021 19:36:04 -0400
Vivek Goyal wrote:
> On Fri, Mar 12, 2021 at 03:10:03PM +0100, Greg Kurz wrote:
> > POSIX.1-2017 clearly stipulates that empty filenames aren't
> > allowed ([1] and [2]). Since virtiofsd is supposed to mirror
> > the host file system hierarchy and the host can b
On 3/15/21 6:14 AM, Thomas Huth wrote:
> On 15/03/2021 00.29, Philippe Mathieu-Daudé wrote:
>> Restrict CPU I/O instructions to architectures providing
>> I/O bus.
>>
>> Signed-off-by: Philippe Mathieu-Daudé
>> ---
>> tests/qtest/fuzz/generic_fuzz.c | 16 ++--
>> tests/qtest/fuzz/
From: Miroslav Rezanina
When declaring g_autofree variable without inicialization, compiler
will raise "may be used uninitialized in this function" warning due
to automatic free handling.
This is mentioned in docs/devel/style.rst (quote from section
"Automatic memory deallocation"):
* Variabl
2021年3月15日(月) 16:32 Gerd Hoffmann :
>
> On Sun, Mar 14, 2021 at 04:01:47PM +0900, Akihiko Odaki wrote:
> > ui/cocoa used to raise all keys before it resigns active to prevent a
> > stuck key problem caused by key up events it does not see while it is
> > inactive. The problem is solved by checking
Hi Philippe,
On Mon, Mar 15, 2021 at 5:13 PM Philippe Mathieu-Daudé
wrote:
>
> On 3/15/21 8:57 AM, Bin Meng wrote:
> > Add a new macro ETH_ZLEN which represents the minimum size of an
> > Ethernet frame without FCS.
> >
> > Signed-off-by: Bin Meng
> > ---
> >
> > include/net/eth.h | 1 +
> > 1
Hi Philippe,
On Mon, Mar 15, 2021 at 5:22 PM Philippe Mathieu-Daudé
wrote:
>
> On 3/15/21 10:18 AM, Philippe Mathieu-Daudé wrote:
> > On 3/15/21 10:17 AM, Philippe Mathieu-Daudé wrote:
> >> On 3/15/21 8:57 AM, Bin Meng wrote:
> >>> This adds a flag in NetClientState, so that a net client can tell
On 3/15/21 11:13 AM, mreza...@redhat.com wrote:
> From: Miroslav Rezanina
>
> When declaring g_autofree variable without inicialization, compiler
Typo "initialization"
> will raise "may be used uninitialized in this function" warning due
> to automatic free handling.
>
> This is mentioned in d
On Mon, 15 Mar 2021 at 10:17, Bin Meng wrote:
>
> Hi Philippe,
>
> On Mon, Mar 15, 2021 at 5:22 PM Philippe Mathieu-Daudé
> wrote:
> >
> > On 3/15/21 10:18 AM, Philippe Mathieu-Daudé wrote:
> > > On 3/15/21 10:17 AM, Philippe Mathieu-Daudé wrote:
> > >> On 3/15/21 8:57 AM, Bin Meng wrote:
> > >>>
On 3/14/21 6:59 PM, Richard Henderson wrote:
> Only one of the three places in hw/ppc that modify msr updated
> hflags. Even in that case, use the official interface instead
> of a direct call to hreg_compute_hflags.
ppc_store_msr() is the interface to use.
> Signed-off-by: Richard Henderson
>
On 3/15/21 11:15 AM, Bin Meng wrote:
> Hi Philippe,
>
> On Mon, Mar 15, 2021 at 5:13 PM Philippe Mathieu-Daudé
> wrote:
>>
>> On 3/15/21 8:57 AM, Bin Meng wrote:
>>> Add a new macro ETH_ZLEN which represents the minimum size of an
>>> Ethernet frame without FCS.
>>>
>>> Signed-off-by: Bin Meng
>
From: Phillip Tennen
This patch implements a new netdev device, reachable via -netdev
vmnet-macos, that’s backed by macOS’s vmnet framework.
The vmnet framework provides native bridging support, and its usage in
this patch is intended as a replacement for attempts to use a tap device
via the tun
On Fri, Mar 12, 2021 at 10:22:06AM +0100, Greg Kurz wrote:
> slave_read() checks EAGAIN when reading or writing to the socket
> fails. This gives the impression that the slave channel is in
> non-blocking mode, which is certainly not the case with the current
> code base. And the rest of the code i
On Fri, Mar 12, 2021 at 10:22:07AM +0100, Greg Kurz wrote:
> Some message types, e.g. VHOST_USER_SLAVE_VRING_HOST_NOTIFIER_MSG,
> can convey file descriptors. These must be closed before returning
> from slave_read() to avoid being leaked. This can currently be done
> in two different places:
>
>
On Fri, Mar 12, 2021 at 10:22:08AM +0100, Greg Kurz wrote:
> Signed-off-by: Greg Kurz
> ---
> hw/virtio/vhost-user.c | 19 ++-
> 1 file changed, 10 insertions(+), 9 deletions(-)
Reviewed-by: Stefan Hajnoczi
signature.asc
Description: PGP signature
Patchew URL:
https://patchew.org/QEMU/20210315103209.20870-1-akihiko.od...@gmail.com/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: 20210315103209.20870-1-akihiko.od...@gmail.com
Subject: [PATCH] net/macos: implement
On Fri, Mar 12, 2021 at 10:22:09AM +0100, Greg Kurz wrote:
> The slave channel is implemented with socketpair() : QEMU creates
> the pair, passes one of the socket to virtiofsd and monitors the
> other one with the main event loop using qemu_set_fd_handler().
>
> In order to fix a potential deadlo
On Fri, Mar 12, 2021 at 10:22:10AM +0100, Greg Kurz wrote:
> A deadlock condition potentially exists if a vhost-user process needs
> to request something to QEMU on the slave channel while processing a
> vhost-user message.
>
> This doesn't seem to affect any vhost-user implementation so far, but
On Mon, 15 Mar 2021 11:13:52 +0100
mreza...@redhat.com wrote:
> From: Miroslav Rezanina
>
> When declaring g_autofree variable without inicialization, compiler
> will raise "may be used uninitialized in this function" warning due
> to automatic free handling.
>
> This is mentioned in docs/devel
On Mon, Mar 15, 2021 at 12:01 PM Stefan Hajnoczi
wrote:
> On Sun, Mar 14, 2021 at 05:23:24AM +0200, Mahmoud Mandour wrote:
> > @@ -130,7 +130,7 @@ static struct fuse_req *fuse_ll_alloc_req(struct
> fuse_session *se)
> > {
> > struct fuse_req *req;
> >
> > -req = (struct fuse_req *)callo
Replaced a call to malloc() and its respective call to free()
with g_malloc() and g_free().
g_malloc() is preferred more than g_try_* functions, which
return NULL on error, when the size of the requested
allocation is small. This is because allocating few
bytes should not be a problem in a health
In this small series, I fix two styling issues that were causing
errors when the checkpatch.pl scritp was run against the
file util/compatfd.c.
Also, I replace a call to malloc() and its respective call
to free() with GLib's allocation and deallocation.
Used g_malloc() because the allocation is
Fixed two styling issues that caused checkpatch.pl errors.
Signed-off-by: Mahmoud Mandour
---
util/compatfd.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/util/compatfd.c b/util/compatfd.c
index ee47dd8089..174f394533 100644
--- a/util/compatfd.c
+++ b/util/compatfd.
** Tags added: semihosting
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1918302
Title:
qemu-system-arm segfaults while servicing SYS_HEAPINFO
Status in QEMU:
New
Bug description:
I compiled
From: Klaus Jensen
Fix three issues reported by coverity (CID 1450756, CID 1450757 and CID
1450758).
Klaus Jensen (2):
hw/block/nvme: fix potential overflow
hw/block/nvme: assert namespaces array indices
hw/block/nvme-subsys.h | 2 ++
hw/block/nvme.h| 10 --
hw/block/nvme-
From: Klaus Jensen
page_size is a uint32_t, and zasl is a uint8_t, so the expression
`page_size << zasl` is done using 32-bit arithmetic and might overflow.
Since we then compare this against a 64 bit data_size value, Coverity
complains that we might overflow unintentionally. An MDTS/ZASL value i
From: Klaus Jensen
Coverity complains about a possible memory corruption in the
nvme_ns_attach and _detach functions. While we should not (famous last
words) be able to reach this function without nsid having previously
been validated, this is still an open door for future misuse.
Make Coverity
Cc'ing audio team
On 3/15/21 11:57 AM, Izak van Langevelde wrote:
> It's been a while since I used qemu, but when I try to run my virtual
> machine I get the following errors:
>
> ALSA lib pcm_dsnoop.c:641:(snd_pcm_dsnoop_open) unable to open slave
> alsa: Could not initialize ADC
> alsa: Failed
Am 15.03.2021 um 10:39 hat Markus Armbruster geschrieben:
> Paolo Bonzini writes:
>
> > On 13/03/21 14:28, Markus Armbruster wrote:
> >> Kevin Wolf writes:
> >>
> >>> This switches the HMP command object_add from a QemuOpts-based parser to
> >>> user_creatable_add_from_str() which uses a keyval
Hi Mahmoud,
On 3/15/21 11:58 AM, Mahmoud Mandour wrote:
> Replaced a call to malloc() and its respective call to free()
> with g_malloc() and g_free().
>
> g_malloc() is preferred more than g_try_* functions, which
> return NULL on error, when the size of the requested
> allocation is small. Thi
On 3/15/21 12:03 PM, Klaus Jensen wrote:
> From: Klaus Jensen
>
> Coverity complains about a possible memory corruption in the
> nvme_ns_attach and _detach functions. While we should not (famous last
> words) be able to reach this function without nsid having previously
> been validated, this is
On Mar 15 12:19, Philippe Mathieu-Daudé wrote:
> On 3/15/21 12:03 PM, Klaus Jensen wrote:
> > From: Klaus Jensen
> >
> > Coverity complains about a possible memory corruption in the
> > nvme_ns_attach and _detach functions. While we should not (famous last
> > words) be able to reach this functio
1 - 100 of 495 matches
Mail list logo
