Re: [PATCH v2] virtio-net: Add check for mac address while peer is vdpa

Hi jason On Mon, Oct 26, 2020 at 11:20 AM Jason Wang wrote: > > On 2020/10/26 上午10:43, Jason Wang wrote: > > > > On 2020/10/23 下午5:15, Cindy Lu wrote: > >> Sometime vdpa get an all 0 mac address from the hardware, this will > >> cause the traffic down > >> So we add the check for this part. >

Re: Enable MSI-X support in PCIe device.

I have tried to use msi only, but failed again. Is there and documentation details this? From: Qemu-devel on behalf of Douglas Su Sent: Thursday, October 22, 2020 20:32 To: QEMU Developers Subject: Enable MSI-X support in PCIe device. To use MSI-X interrupt

Re: [PATCH v3] net: remove an assert call in eth_get_gso_type

On 2020/10/21 下午5:23, P J P wrote: +-- On Wed, 21 Oct 2020, Jason Wang wrote --+ | It should not be a guest error, since guest is allowed to send a packet | other than IPV4(6). * Ah...sigh! :( * I very hesitantly used guest_error mask, since it was g_assert-ing before. To me both

Re: [PATCH v2] virtio-net: Add check for mac address while peer is vdpa

On 2020/10/26 上午10:43, Jason Wang wrote: On 2020/10/23 下午5:15, Cindy Lu wrote: Sometime vdpa get an all 0 mac address from the hardware, this will cause the traffic down So we add the check for this part. if we get an 0 mac address we will use the default mac address instead Signed-off-by:

Re: [PATCH v2] virtio-net: Add check for mac address while peer is vdpa

On 2020/10/23 下午5:15, Cindy Lu wrote: Sometime vdpa get an all 0 mac address from the hardware, this will cause the traffic down So we add the check for this part. if we get an 0 mac address we will use the default mac address instead Signed-off-by: Cindy Lu --- hw/net/virtio-net.c | 7

[PATCH] i386/kvm: fix setting up nested_state for SVM

nested_state->format needs to ne set appropriately for VMX/SVM, otherwise KVM_SET_NESTED_STATE ioctl will fail. Signed-off-by: Marek Marczykowski-Górecki --- target/i386/kvm.c | 18 -- 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/target/i386/kvm.c

Re: [PATCH] hw/sd: Fix 2 GiB card CSD register values

Hi Philippe, On Mon, Oct 26, 2020 at 2:56 AM Philippe Mathieu-Daudé wrote: > > On 10/25/20 4:23 PM, Bin Meng wrote: > > From: Bin Meng > > > > Per the SD spec, to indicate a 2 GiB card, BLOCK_LEN shall be 1024 > > bytes, hence the READ_BL_LEN field in the CSD register shall be 10 > > instead of

Re: [PATCH v2 6/9] tcg: implement mirror mapped JIT for iOS

On 201025 1351, Joelle van Dyne wrote: > > Finally, I'd like to have this implemented on Linux as well, or I'm afraid > > the > > feature will bit-rot. This can be trivially done by either (1) > > MREMAP_DONTUNMAP or (2) mapping from posix shared memory instead of > > MAP_ANON so > > that you

Re: [PATCH 0/5] spapr: Error handling fixes and cleanups (round 3)

On Sun, 25 Oct 2020 11:13:40 +0100 Greg Kurz wrote: > On Thu, 22 Oct 2020 15:11:42 +1100 > David Gibson wrote: > > > On Mon, Oct 19, 2020 at 10:47:52AM +0200, Greg Kurz wrote: > > > Hi, > > > > > > This is a followup to a previous cleanup for the sPAPR code: > > > > > >

Re: [PATCH 1/5] pc-dimm: Drop @errp argument of pc_dimm_plug()

On Fri, 23 Oct 2020 21:19:19 +0200 Igor Mammedov wrote: > On Mon, 19 Oct 2020 10:48:04 +0200 > Greg Kurz wrote: > > > pc_dimm_plug() doesn't use it. It only aborts on error. > > > > Drop @errp and adapt the callers accordingly. > > > > Signed-off-by: Greg Kurz > Reviewed-by: Igor Mammedov

Re: [PATCH v3 1/2] hw/misc: add an EMC141{3,4} device model

On 10/25/20 2:14 PM, John Wang wrote: Largely inspired by the TMP421 temperature sensor, here is a model for the EMC1413/EMC1414 temperature sensors. Specs can be found here : http://ww1.microchip.com/downloads/en/DeviceDoc/20005274A.pdf Signed-off-by: John Wang --- v3: - update the

Re: [PATCH v2 6/9] tcg: implement mirror mapped JIT for iOS

> Finally, I'd like to have this implemented on Linux as well, or I'm afraid the > feature will bit-rot. This can be trivially done by either (1) > MREMAP_DONTUNMAP or (2) mapping from posix shared memory instead of MAP_ANON > so > that you can map the same memory twice. Thus virtually all of

[Bug 1901440] [NEW] Instability in IVSHMEM after updating QEMU 4.2 -> 5.0

Public bug reported: Updating Ubuntu from 20.08 to 20.10 which updates QEMU from 4.2 to 5.0 results in the virtual machines freezing when the IVSHMEM interface is active. This workstation typically runs several windows 10 virtual machines that are accessed locally: two using the spice viewer

Re: [PATCH] hw/sd: Fix 2 GiB card CSD register values

On 10/25/20 7:56 PM, Philippe Mathieu-Daudé wrote: On 10/25/20 4:23 PM, Bin Meng wrote: From: Bin Meng Per the SD spec, to indicate a 2 GiB card, BLOCK_LEN shall be 1024 bytes, hence the READ_BL_LEN field in the CSD register shall be 10 instead of 9. This fixes the acceptance test error for

Re: [RFC PATCH 4/4] tests/acceptance: Test U-Boot/Linux from Armbian 20.08 on Orange Pi PC

Cc'ing avocado-devel@ On 10/26/20 12:35 AM, Niek Linnenbank wrote: On Sun, Oct 25, 2020 at 3:17 AM Bin Meng > wrote: On Sun, Oct 25, 2020 at 5:01 AM Niek Linnenbank mailto:nieklinnenb...@gmail.com>> wrote: ... > After applying that change, all Orange Pi

Re: [PATCH v2 6/9] tcg: implement mirror mapped JIT for iOS

On Mon, Oct 19, 2020 at 5:20 PM Richard Henderson wrote: > > 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

Re: [PATCH v2 2/9] configure: cross-compiling without cross_prefix

As the patch stands --cross-prefix="" works but I like having the --enable-cross-compile option because it's more clear of the intention (in logs and such). I've been struggling to come up with a concise modification to the help text for --cross-prefix that explains that empty PREFIX is allowed

Re: [PATCH] hw/sd: Fix 2 GiB card CSD register values

On 10/25/20 4:23 PM, Bin Meng wrote: From: Bin Meng Per the SD spec, to indicate a 2 GiB card, BLOCK_LEN shall be 1024 bytes, hence the READ_BL_LEN field in the CSD register shall be 10 instead of 9. This fixes the acceptance test error for the NetBSD 9.0 test of the Orange Pi PC that has an

Re: [PATCH 0/6] qemu-storage-daemon: QAPIfy --chardev

Paolo Bonzini writes: > On 23/10/20 15:40, Markus Armbruster wrote: >>> >>> The benefit of the user creatable object approach is that we dont >>> have to add custom CLI args for different types of object, nor write >>> code to populate QOM from QAPI. The downside is that we're divorced >>> from

Re: [PATCH] tests/acceptance: expand Orange Pi PC SD images for NetBSD, Armbian above nearest power of two

Hi Philippe, I've tested the patch by Bin on the 2G & Sd blocksize issue, see my response there. https://lists.gnu.org/archive/html/qemu-devel/2020-10/msg07318.html So probably we do not need the patch I send here. Regards, Niek On Mon, Oct 26, 2020 at 12:48 AM Niek Linnenbank wrote: >

Re: [PATCH] hw/sd: Fix 2 GiB card CSD register values

Hello Bin, On Sun, Oct 25, 2020 at 4:25 PM Bin Meng wrote: > From: Bin Meng > > Per the SD spec, to indicate a 2 GiB card, BLOCK_LEN shall be 1024 > bytes, hence the READ_BL_LEN field in the CSD register shall be 10 > instead of 9. > > This fixes the acceptance test error for the NetBSD 9.0

Re: [PULL 25/33] tests/acceptance: Add a test for the N800 and N810 arm machines

On Fri, 23 Oct 2020 at 20:04, Igor Mammedov wrote: > > On Fri, 23 Oct 2020 19:39:16 +0200 > Philippe Mathieu-Daudé wrote: > > Are you saying arm_boot_info should hold a pointer to MachineState* > > instead of duplicating? > > yep, some parts of it (fdt related) already use MachineState* so it's

Re: [PATCH] tests/acceptance: expand Orange Pi PC SD images for NetBSD, Armbian above nearest power of two

Hello Philippe, On Sun, Oct 25, 2020 at 1:11 PM Philippe Mathieu-Daudé wrote: > On 10/25/20 2:31 AM, Philippe Mathieu-Daudé wrote: > > Hi Niek, > > > > On 10/24/20 11:52 PM, Niek Linnenbank wrote: > >> The acceptance tests for the Orange Pi PC need to expand the SD card > >> images > >> to a

Re: [RFC PATCH 4/4] tests/acceptance: Test U-Boot/Linux from Armbian 20.08 on Orange Pi PC

Hi Bin, On Sun, Oct 25, 2020 at 3:17 AM Bin Meng wrote: > Hi Niek, > > On Sun, Oct 25, 2020 at 5:01 AM Niek Linnenbank > wrote: > > > > Hi Philippe, > > > > I have ran this series with the new Armbian 20.08 test and noticed this > failure: > > > > -console: /dev/mmcblk0p1: The filesystem size

Re: [PATCH] tests/acceptance: expand Orange Pi PC SD images for NetBSD, Armbian above nearest power of two

On Sun, Oct 25, 2020 at 8:13 PM Philippe Mathieu-Daudé wrote: > > On 10/25/20 2:31 AM, Philippe Mathieu-Daudé wrote: > > Hi Niek, > > > > On 10/24/20 11:52 PM, Niek Linnenbank wrote: > >> The acceptance tests for the Orange Pi PC need to expand the SD card > >> images > >> to a size which is a

[PATCH] hw/sd: Fix 2 GiB card CSD register values

From: Bin Meng Per the SD spec, to indicate a 2 GiB card, BLOCK_LEN shall be 1024 bytes, hence the READ_BL_LEN field in the CSD register shall be 10 instead of 9. This fixes the acceptance test error for the NetBSD 9.0 test of the Orange Pi PC that has an expanded SD card image of 2 GiB size.

Re: [PATCH 4/5] spapr: Pass _abort when getting some PC DIMM properties

On Fri, 23 Oct 2020 21:15:09 +0200 Igor Mammedov wrote: > On Mon, 19 Oct 2020 10:48:41 +0200 > Greg Kurz wrote: > > > Both PC_DIMM_SLOT_PROP and PC_DIMM_ADDR_PROP are defined in the > > default property list of the PC DIMM device class: > > > > DEFINE_PROP_UINT64(PC_DIMM_ADDR_PROP,

Re: [PATCH 0/6] tests/acceptance: Test the Fuloong 2E machine

On Sun, 25 Oct 2020, Philippe Mathieu-Daudé wrote: On 10/25/20 12:18 PM, Mark Cave-Ayland wrote: On 21/10/2020 11:50, Philippe Mathieu-Daudé wrote: This series add tests for the MIPS Fuloong 2E machine. I have been using these tests for over 1 year now. After recent discussions with Huacai

[PATCH v3 2/2] aspeed: Add support for the g220a-bmc board

G220A is a 2 socket x86 motherboard supported by OpenBMC. Strapping configuration was obtained from hardware. Signed-off-by: John Wang Reviewed-by: Cédric Le Goater Reviewed-by: Joel Stanley --- v3: - No changes v2: - No changes --- hw/arm/aspeed.c | 60

[PATCH v3 1/2] hw/misc: add an EMC141{3,4} device model

Largely inspired by the TMP421 temperature sensor, here is a model for the EMC1413/EMC1414 temperature sensors. Specs can be found here : http://ww1.microchip.com/downloads/en/DeviceDoc/20005274A.pdf Signed-off-by: John Wang --- v3: - update the link to the spec - Rename emc1413.c to

Re: [PATCH 0/6] tests/acceptance: Test the Fuloong 2E machine

On 10/25/20 12:18 PM, Mark Cave-Ayland wrote: On 21/10/2020 11:50, Philippe Mathieu-Daudé wrote: This series add tests for the MIPS Fuloong 2E machine. I have been using these tests for over 1 year now. After recent discussions with Huacai and Mark regarding how to contribute tests for

Re: [PATCH] tests/acceptance: expand Orange Pi PC SD images for NetBSD, Armbian above nearest power of two

On 10/25/20 2:31 AM, Philippe Mathieu-Daudé wrote: Hi Niek, On 10/24/20 11:52 PM, Niek Linnenbank wrote: The acceptance tests for the Orange Pi PC need to expand the SD card images to a size which is a power of two. As Qemu uses the size of the SD image file as well for the size of the

Re: [PATCH v3] hw/display/tcx: Allow 64-bit accesses to framebuffer stippler and blitter

On 10/25/20 11:55 AM, Mark Cave-Ayland wrote: On 24/10/2020 21:51, Philippe Mathieu-Daudé wrote: The S24/TCX datasheet is listed as "Unable to locate" on [1]. However the NetBSD revision 1.32 of the driver introduced 64-bit accesses to the stippler and blitter [2]. It is safe to assume these

Re: [PATCH] sabre: use object_initialize_child() for iommu child object

On 10/25/20 12:11 PM, Mark Cave-Ayland wrote: On 21/10/2020 12:43, Mark Cave-Ayland wrote: Store the child object directly within the sabre object rather than using link properties. Signed-off-by: Mark Cave-Ayland ---   hw/pci-host/sabre.c | 10 --   hw/sparc64/sun4u.c 

Re: [PATCH 0/6] tests/acceptance: Test the Fuloong 2E machine

On 21/10/2020 11:50, Philippe Mathieu-Daudé wrote: This series add tests for the MIPS Fuloong 2E machine. I have been using these tests for over 1 year now. After recent discussions with Huacai and Mark regarding how to contribute tests for machines using private blob, I think it is time to

Re: [PATCH] sabre: use object_initialize_child() for iommu child object

On 21/10/2020 12:43, Mark Cave-Ayland wrote: Store the child object directly within the sabre object rather than using link properties. Signed-off-by: Mark Cave-Ayland --- hw/pci-host/sabre.c | 10 -- hw/sparc64/sun4u.c | 8 +--- include/hw/pci-host/sabre.h |

Re: [PATCH v2 0/2] pci: assert that irqnum is between 0 and bus->nirqs in pci_change_irq_level()

On 24/10/2020 21:38, Philippe Mathieu-Daudé wrote: Respin of Mark's patch: https://www.mail-archive.com/qemu-devel@nongnu.org/msg749459.html Supersedes: <20201011082022.3016-1-mark.cave-ayl...@ilande.co.uk> Mark Cave-Ayland (1): pci: Assert irqnum is between 0 and bus->nirqs in

Re: [PATCH] sabre: increase number of PCI bus IRQs from 32 to 64

On 24/10/2020 21:41, Philippe Mathieu-Daudé wrote: On 10/21/20 11:27 AM, Mark Cave-Ayland wrote: On 11/10/2020 09:13, Mark Cave-Ayland wrote: The rework of the sabre IRQs in commit 6864fa3897 "sun4u: update PCI topology to include simba PCI bridges" changed the IRQ routing so that both PCI

Re: [PATCH v3] hw/display/tcx: Allow 64-bit accesses to framebuffer stippler and blitter

On 24/10/2020 21:51, Philippe Mathieu-Daudé wrote: The S24/TCX datasheet is listed as "Unable to locate" on [1]. However the NetBSD revision 1.32 of the driver introduced 64-bit accesses to the stippler and blitter [2]. It is safe to assume these memory regions are 64-bit accessible. QEMU

Re: [PATCH 0/5] spapr: Error handling fixes and cleanups (round 3)

On Thu, 22 Oct 2020 15:11:42 +1100 David Gibson wrote: > On Mon, Oct 19, 2020 at 10:47:52AM +0200, Greg Kurz wrote: > > Hi, > > > > This is a followup to a previous cleanup for the sPAPR code: > > > > https://lists.gnu.org/archive/html/qemu-devel/2020-09/msg04860.html > > > > The last two

Re: [PATCH v2 3/6] qapi: Remove wrapper struct for simple unions

I apologize for writing so much. It's the only way I know to figure out how things work. This may feel like I'm trying to shoot down your patch. I'm not! I'm merely trying to understand and maybe improve it. I'd love to get rid of implicit wrapper objects, actually. They complicate things in