Re: [PATCH v4 20/28] net: Strip virtio-net header when dumping

2023-01-30 Thread Akihiko Odaki
On 2023/01/31 0:47, Michael S. Tsirkin wrote: On Tue, Jan 31, 2023 at 12:36:38AM +0900, Akihiko Odaki wrote: On 2023/01/31 0:12, Michael S. Tsirkin wrote: On Mon, Jan 30, 2023 at 10:47:07PM +0900, Akihiko Odaki wrote: filter-dump specifiees Ethernet as PCAP LinkType, which does not expect

[PATCH v2] hw/timer/hpet: Fix expiration time overflow

2023-01-30 Thread Akihiko Odaki
...@bu.edu/ The fixed test case is: fuzz/crash_2d7036941dcda1ad4380bb8a9174ed0c949bcefd Fixes: 16b29ae180 ("Add HPET emulation to qemu (Beth Kon)") Signed-off-by: Akihiko Odaki Acked-by: Michael S. Tsirkin --- V1 -> V2: Renamed arm() to hpet_arm() hw/timer/hpet.c | 19 +

[PATCH v2] softmmu: Use memmove in flatview_write_continue

2023-01-30 Thread Akihiko Odaki
/crash_47dfe62d9f911bf523ff48cd441b61c0013ed805 Signed-off-by: Akihiko Odaki Acked-by: Alexander Bulekov --- V1 -> V2: Correct spellings in the message softmmu/physmem.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/softmmu/physmem.c b/softmmu/physmem.c index cb998cdf23..3cd27b1c9d 100644 --- a/softmmu/physme

Re: Display update issue on M1 Macs

2023-01-30 Thread Akihiko Odaki
On 2023/01/31 8:58, BALATON Zoltan wrote: On Sat, 28 Jan 2023, Akihiko Odaki wrote: On 2023/01/23 8:28, BALATON Zoltan wrote: On Thu, 19 Jan 2023, Akihiko Odaki wrote: On 2023/01/15 3:11, BALATON Zoltan wrote: On Sat, 14 Jan 2023, Akihiko Odaki wrote: On 2023/01/13 22:43, BALATON Zoltan

Re: [PATCH v5 7/9] igb: Introduce qtest for igb device

2023-01-31 Thread Akihiko Odaki
On 2023/01/31 22:04, Thomas Huth wrote: On 30/01/2023 15.08, Akihiko Odaki wrote: This change is derived from qtest for e1000e device. Signed-off-by: Akihiko Odaki ---   MAINTAINERS |   2 +   tests/qtest/fuzz/generic_fuzz_configs.h |   5 +   tests/qtest/igb-test.c

[PATCH v5 00/29] e1000x cleanups (preliminary for IGB)

2023-01-31 Thread Akihiko Odaki
Added "hw/net/net_tx_pkt: Automatically determine if virtio-net header is used". - Added "hw/net/net_rx_pkt: Remove net_rx_pkt_has_virt_hdr". - Added "e1000e: Perform software segmentation for loopback". - Added "hw/net/net_tx_pkt: Implement TCP segmentation" - Added &q

[PATCH v5 02/29] hw/net: Add more MII definitions

2023-01-31 Thread Akihiko Odaki
The definitions will be used by igb. Signed-off-by: Akihiko Odaki --- include/hw/net/mii.h | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/include/hw/net/mii.h b/include/hw/net/mii.h index 4ae4dcce7e..c6a767a49a 100644 --- a/include/hw/net/mii.h +++ b/include

[PATCH v5 09/29] e1000: Use memcpy to intialize registers

2023-01-31 Thread Akihiko Odaki
Use memcpy instead of memmove to initialize registers. The initial register templates and register table instances will never overlap. Signed-off-by: Akihiko Odaki --- hw/net/e1000.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/net/e1000.c b/hw/net/e1000.c index

[PATCH v5 03/29] fsl_etsec: Use hw/net/mii.h

2023-01-31 Thread Akihiko Odaki
hw/net/mii.h provides common definitions for MII. Signed-off-by: Akihiko Odaki Reviewed-by: Philippe Mathieu-Daudé --- hw/net/fsl_etsec/etsec.c | 11 ++- hw/net/fsl_etsec/etsec.h | 17 - hw/net/fsl_etsec/miim.c | 5 +++-- include/hw/net/mii.h | 1 + 4 files

[PATCH v5 01/29] e1000e: Fix the code style

2023-01-31 Thread Akihiko Odaki
igb implementation first starts off by copying e1000e code. Correct the code style before that. Signed-off-by: Akihiko Odaki Reviewed-by: Philippe Mathieu-Daudé --- hw/net/e1000.c | 41 hw/net/e1000e.c| 72 ++-- hw/net/e1000e_core.c

[PATCH v5 26/29] e1000e: Do not assert when MSI-X is disabled later

2023-01-31 Thread Akihiko Odaki
ot;) Signed-off-by: Akihiko Odaki --- hw/net/e1000e_core.c | 4 1 file changed, 4 deletions(-) diff --git a/hw/net/e1000e_core.c b/hw/net/e1000e_core.c index ff93547f88..76c7814cb8 100644 --- a/hw/net/e1000e_core.c +++ b/hw/net/e1000e_core.c @@ -162,8 +162,6 @@ e1000e_intrmgr_on_thrott

[PATCH v5 07/29] e1000: Use more constant definitions

2023-01-31 Thread Akihiko Odaki
, E1000_MC_TBL_SIZE, and E1000_VLAN_FILTER_TBL_SIZE were copied from: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/tree/drivers/net/ethernet/intel/e1000/e1000_hw.h?h=v6.0.9#n707 Signed-off-by: Akihiko Odaki Reviewed-by: Philippe Mathieu-Daudé --- hw/net/e1000.c | 50

[PATCH v5 16/29] e1000e: Set MII_ANER_NWAY

2023-01-31 Thread Akihiko Odaki
mismatch. > ACTION: Configure the link partner for auto-negotiation. Signed-off-by: Akihiko Odaki Reviewed-by: Philippe Mathieu-Daudé --- hw/net/e1000e_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/net/e1000e_core.c b/hw/net/e1000e_core.c index d8c17baf8f

[PATCH v5 15/29] e1000e: Introduce e1000_rx_desc_union

2023-01-31 Thread Akihiko Odaki
Before this change, e1000e_write_packet_to_guest() allocated the receive descriptor buffer as an array of uint8_t. This does not ensure the buffer is sufficiently aligned. Introduce e1000_rx_desc_union type, a union type of all receive descriptor types to correct this. Signed-off-by: Akihiko

[PATCH v5 12/29] e1000e: Improve software reset

2023-01-31 Thread Akihiko Odaki
This change makes e1000e reset more things when software reset was triggered. Some registers are exempted from software reset in the datasheet and this change also implements the behavior accordingly. Signed-off-by: Akihiko Odaki --- hw/net/e1000e_core.c | 24 +++- 1 file

[PATCH v5 27/29] MAINTAINERS: Add Akihiko Odaki as a e1000e reviewer

2023-01-31 Thread Akihiko Odaki
I want to know to be notified when there is a new change for e1000e as e1000e is similar to igb and such a change may also be applicable for igb. Signed-off-by: Akihiko Odaki --- MAINTAINERS | 2 ++ 1 file changed, 2 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 08ad1e5341

[PATCH v5 22/29] hw/net/net_rx_pkt: Remove net_rx_pkt_has_virt_hdr

2023-01-31 Thread Akihiko Odaki
NetTxPkt has a valid virtio-net header. Signed-off-by: Akihiko Odaki --- hw/net/e1000e_core.c | 16 hw/net/net_rx_pkt.c | 11 +-- hw/net/net_rx_pkt.h | 12 +--- hw/net/trace-events | 1 - hw/net/virtio-net.c | 2 +- hw/net/vmxnet3.c | 12 ++-- 6

[PATCH v5 25/29] hw/net/net_tx_pkt: Check the payload length

2023-01-31 Thread Akihiko Odaki
/crash_6aeaa33e7211ecd603726c53e834df4c6d1e08bc Fixes: e263cd49c7 ("Packet abstraction for VMWARE network devices") Signed-off-by: Akihiko Odaki --- hw/net/net_tx_pkt.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/hw/net/net_tx_pkt.c b/hw/net/net_tx_pkt.c index 4a35e8429d..986a3adfe9 100644 --- a/hw/net/net_tx_p

[PATCH v5 04/29] e1000: Use hw/net/mii.h

2023-01-31 Thread Akihiko Odaki
hw/net/mii.h provides common definitions for MII. Signed-off-by: Akihiko Odaki Reviewed-by: Philippe Mathieu-Daudé --- hw/net/e1000.c | 86 ++-- hw/net/e1000_regs.h| 46 hw/net/e1000e.c| 1 + hw/net/e1000e_core.c | 99

[PATCH v5 19/29] e1000x: Alter the signature of e1000x_is_vlan_packet

2023-01-31 Thread Akihiko Odaki
e1000x_is_vlan_packet() had a pointer to uint8_t as a parameter, but it does not have to be uint8_t. Change the type to void *. Signed-off-by: Akihiko Odaki --- hw/net/e1000x_common.c | 2 +- hw/net/e1000x_common.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/net

[PATCH v5 20/29] net: Strip virtio-net header when dumping

2023-01-31 Thread Akihiko Odaki
-off-by: Akihiko Odaki --- include/net/net.h | 6 ++ net/dump.c| 11 +++ net/net.c | 18 ++ net/tap.c | 16 4 files changed, 47 insertions(+), 4 deletions(-) diff --git a/include/net/net.h b/include/net/net.h index dc20b31e9f

[PATCH v5 11/29] e1000e: Remove pending interrupt flags

2023-01-31 Thread Akihiko Odaki
They are duplicate of running throttling timer flags and incomplete as the flags are not cleared when the interrupts are fired or the device is reset. Signed-off-by: Akihiko Odaki --- hw/net/e1000e.c | 5 ++--- hw/net/e1000e_core.c | 19 +++ hw/net/e1000e_core.h | 2

[PATCH v5 18/29] net: Check L4 header size

2023-01-31 Thread Akihiko Odaki
net_tx_pkt_build_vheader() inspects TCP header but had no check for the header size, resulting in an undefined behavior. Check the header size and drop the packet if the header is too small. Signed-off-by: Akihiko Odaki --- hw/net/e1000e_core.c | 19 ++- hw/net/net_tx_pkt.c

[PATCH v5 10/29] e1000e: Use memcpy to intialize registers

2023-01-31 Thread Akihiko Odaki
Use memcpy instead of memmove to initialize registers. The initial register templates and register table instances will never overlap. Signed-off-by: Akihiko Odaki --- hw/net/e1000e_core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/net/e1000e_core.c b/hw/net

[PATCH v5 13/29] e1000: Configure ResettableClass

2023-01-31 Thread Akihiko Odaki
This is part of recent efforts of refactoring e1000 and e1000e. DeviceClass's reset member is deprecated so migrate to ResettableClass. There is no behavioral difference. Signed-off-by: Akihiko Odaki Reviewed-by: Peter Maydell Reviewed-by: Philippe Mathieu-Daudé --- hw/net/e1000.c

[PATCH v5 17/29] e1000e: Remove extra pointer indirection

2023-01-31 Thread Akihiko Odaki
pointer indirection. Signed-off-by: Akihiko Odaki Reviewed-by: Philippe Mathieu-Daudé --- hw/net/e1000e_core.c | 38 +++--- 1 file changed, 19 insertions(+), 19 deletions(-) diff --git a/hw/net/e1000e_core.c b/hw/net/e1000e_core.c index 736708407c..d143f2ae6f

[PATCH v5 21/29] hw/net/net_tx_pkt: Automatically determine if virtio-net header is used

2023-01-31 Thread Akihiko Odaki
The new function qemu_get_using_vnet_hdr() allows to automatically determine if virtio-net header is used. Signed-off-by: Akihiko Odaki --- hw/net/e1000e_core.c | 3 +-- hw/net/net_tx_pkt.c | 19 ++- hw/net/net_tx_pkt.h | 3 +-- hw/net/vmxnet3.c | 6 ++ 4 files

[PATCH v5 23/29] e1000e: Perform software segmentation for loopback

2023-01-31 Thread Akihiko Odaki
ation if VIRTIO_NET_HDR_F_DATA_VALID is set. Signed-off-by: Akihiko Odaki --- hw/net/e1000e_core.c | 27 ++-- hw/net/net_rx_pkt.c | 7 hw/net/net_rx_pkt.h | 8 + hw/net/net_tx_pkt.c | 76 +--- hw/net/net_tx_pkt.h | 21 ++

[PATCH v5 24/29] hw/net/net_tx_pkt: Implement TCP segmentation

2023-01-31 Thread Akihiko Odaki
implementation; igb provides loopback feature for VMDq and the feature relies on software segmentation. Implement proper TCP segmentation in net_tx_pkt to fix such a scenario. Signed-off-by: Akihiko Odaki --- hw/net/net_tx_pkt.c | 248 include/net

[PATCH v5 05/29] e1000: Mask registers when writing

2023-01-31 Thread Akihiko Odaki
When a register has effective bits fewer than their width, the old code inconsistently masked when writing or reading. Make the code consistent by always masking when writing, and remove some code duplication. Signed-off-by: Akihiko Odaki --- hw/net/e1000.c | 84

[PATCH v5 08/29] e1000e: Use more constant definitions

2023-01-31 Thread Akihiko Odaki
The definitions of SW Semaphore Register were copied from: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/tree/drivers/net/ethernet/intel/e1000e/defines.h?h=v6.0.9#n374 Signed-off-by: Akihiko Odaki --- hw/net/e1000_regs.h | 7 +++ hw/net/e1000e_core.c | 49

[PATCH v5 06/29] e1000e: Mask registers when writing

2023-01-31 Thread Akihiko Odaki
When a register has effective bits fewer than their width, the old code inconsistently masked when writing or reading. Make the code consistent by always masking when writing, and remove some code duplication. Signed-off-by: Akihiko Odaki --- hw/net/e1000e_core.c | 94

[PATCH v5 28/29] MAINTAINERS: Add e1000e test files

2023-01-31 Thread Akihiko Odaki
Signed-off-by: Akihiko Odaki Acked-by: Thomas Huth --- MAINTAINERS | 2 ++ 1 file changed, 2 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 958915f227..e920d0061e 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -2218,6 +2218,8 @@ R: Akihiko Odaki S: Maintained F: hw/net/e1000e* F

[PATCH v5 29/29] e1000e: Combine rx traces

2023-01-31 Thread Akihiko Odaki
e1000e_rx_rss_dispatched_to_queue, which logs the queue index. Signed-off-by: Akihiko Odaki --- hw/net/e1000e_core.c | 6 ++ hw/net/trace-events | 5 ++--- 2 files changed, 4 insertions(+), 7 deletions(-) diff --git a/hw/net/e1000e_core.c b/hw/net/e1000e_core.c index 76c7814cb8..4fec6dfe7e 100644 --- a/hw/net

[PATCH v5 14/29] e1000e: Configure ResettableClass

2023-01-31 Thread Akihiko Odaki
This is part of recent efforts of refactoring e1000 and e1000e. DeviceClass's reset member is deprecated so migrate to ResettableClass. There is no behavioral difference. Signed-off-by: Akihiko Odaki Reviewed-by: Peter Maydell Reviewed-by: Philippe Mathieu-Daudé --- hw/net/e1000e.c

[PATCH v6 0/9] Introduce igb

2023-01-31 Thread Akihiko Odaki
ries. - Restored vnet_hdr offload as there seems nothing preventing from that. Akihiko Odaki (9): hw/net/net_tx_pkt: Introduce net_tx_pkt_get_eth_hdr pcie: Introduce pcie_sriov_num_vfs e1000: Split header files Intrdocue igb device emulation tests/qtest/e1000e-test: Fabricate ethernet hea

[PATCH v6 2/9] pcie: Introduce pcie_sriov_num_vfs

2023-01-31 Thread Akihiko Odaki
igb can use this function to change its behavior depending on the number of virtual functions currently enabled. Signed-off-by: Gal Hammer Signed-off-by: Marcel Apfelbaum Signed-off-by: Akihiko Odaki Reviewed-by: Philippe Mathieu-Daudé --- hw/pci/pcie_sriov.c | 5 + include/hw

[PATCH v6 8/9] tests/avocado: Add igb test

2023-01-31 Thread Akihiko Odaki
This automates ethtool tests for igb registers, interrupts, etc. Signed-off-by: Akihiko Odaki --- MAINTAINERS | 1 + .../org.centos/stream/8/x86_64/test-avocado | 1 + tests/avocado/igb.py | 38 +++ 3 files changed

[PATCH v6 3/9] e1000: Split header files

2023-01-31 Thread Akihiko Odaki
Some definitions in the header files are invalid for igb so extract them to new header files to keep igb from referring to them. Signed-off-by: Gal Hammer Signed-off-by: Marcel Apfelbaum Signed-off-by: Akihiko Odaki Reviewed-by: Philippe Mathieu-Daudé --- hw/net/e1000.c | 1 + hw

[PATCH v6 6/9] tests/qtest/libqos/e1000e: Export macreg functions

2023-01-31 Thread Akihiko Odaki
They will be useful for igb testing. Signed-off-by: Akihiko Odaki Reviewed-by: Thomas Huth --- tests/qtest/libqos/e1000e.c | 12 tests/qtest/libqos/e1000e.h | 12 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/tests/qtest/libqos/e1000e.c b/tests

[PATCH v6 7/9] igb: Introduce qtest for igb device

2023-01-31 Thread Akihiko Odaki
This change is derived from qtest for e1000e device. Signed-off-by: Akihiko Odaki Acked-by: Thomas Huth --- MAINTAINERS | 2 + tests/qtest/fuzz/generic_fuzz_configs.h | 5 + tests/qtest/igb-test.c | 243 tests/qtest

[PATCH v6 5/9] tests/qtest/e1000e-test: Fabricate ethernet header

2023-01-31 Thread Akihiko Odaki
e1000e understands ethernet header so fabricate something convincing. Signed-off-by: Akihiko Odaki Reviewed-by: Thomas Huth --- tests/qtest/e1000e-test.c | 25 +++-- tests/qtest/libqos/e1000e.h | 2 ++ 2 files changed, 17 insertions(+), 10 deletions(-) diff --git a

[PATCH v6 1/9] hw/net/net_tx_pkt: Introduce net_tx_pkt_get_eth_hdr

2023-01-31 Thread Akihiko Odaki
Expose the ethernet header so that igb can utilize it to perform the internal routing among its SR-IOV functions. Signed-off-by: Gal Hammer Signed-off-by: Marcel Apfelbaum Signed-off-by: Akihiko Odaki --- hw/net/net_tx_pkt.c | 6 ++ hw/net/net_tx_pkt.h | 8 2 files changed, 14

[PATCH v6 9/9] docs/system/devices/igb: Add igb documentation

2023-01-31 Thread Akihiko Odaki
Signed-off-by: Akihiko Odaki --- MAINTAINERS | 1 + docs/system/device-emulation.rst | 1 + docs/system/devices/igb.rst | 71 3 files changed, 73 insertions(+) create mode 100644 docs/system/devices/igb.rst diff --git a/MAINTAINERS b

[PATCH v7 0/9] Introduce igb

2023-01-31 Thread Akihiko Odaki
TL.Def_PL. - Implemented the combination of VMDq and RSS. - Noted that igb is tested with Windows HLK. V1 -> V2: - Spun off e1000e general improvements to a distinct series. - Restored vnet_hdr offload as there seems nothing preventing from that. Akihiko Odaki (9): hw/net/net_tx_pkt: Introduce

[PATCH v7 5/9] tests/qtest/e1000e-test: Fabricate ethernet header

2023-01-31 Thread Akihiko Odaki
e1000e understands ethernet header so fabricate something convincing. Signed-off-by: Akihiko Odaki Reviewed-by: Thomas Huth --- tests/qtest/e1000e-test.c | 25 +++-- tests/qtest/libqos/e1000e.h | 2 ++ 2 files changed, 17 insertions(+), 10 deletions(-) diff --git a

[PATCH v7 2/9] pcie: Introduce pcie_sriov_num_vfs

2023-01-31 Thread Akihiko Odaki
igb can use this function to change its behavior depending on the number of virtual functions currently enabled. Signed-off-by: Gal Hammer Signed-off-by: Marcel Apfelbaum Signed-off-by: Akihiko Odaki Reviewed-by: Philippe Mathieu-Daudé --- hw/pci/pcie_sriov.c | 5 + include/hw

[PATCH v7 1/9] hw/net/net_tx_pkt: Introduce net_tx_pkt_get_eth_hdr

2023-01-31 Thread Akihiko Odaki
Expose the ethernet header so that igb can utilize it to perform the internal routing among its SR-IOV functions. Signed-off-by: Gal Hammer Signed-off-by: Marcel Apfelbaum Signed-off-by: Akihiko Odaki --- hw/net/net_tx_pkt.c | 6 ++ hw/net/net_tx_pkt.h | 8 2 files changed, 14

[PATCH v7 8/9] tests/avocado: Add igb test

2023-01-31 Thread Akihiko Odaki
This automates ethtool tests for igb registers, interrupts, etc. Signed-off-by: Akihiko Odaki --- MAINTAINERS | 1 + .../org.centos/stream/8/x86_64/test-avocado | 1 + tests/avocado/igb.py | 38 +++ 3 files changed

[PATCH v7 6/9] tests/qtest/libqos/e1000e: Export macreg functions

2023-01-31 Thread Akihiko Odaki
They will be useful for igb testing. Signed-off-by: Akihiko Odaki Reviewed-by: Thomas Huth --- tests/qtest/libqos/e1000e.c | 12 tests/qtest/libqos/e1000e.h | 12 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/tests/qtest/libqos/e1000e.c b/tests

[PATCH v7 7/9] igb: Introduce qtest for igb device

2023-01-31 Thread Akihiko Odaki
This change is derived from qtest for e1000e device. Signed-off-by: Akihiko Odaki Acked-by: Thomas Huth --- MAINTAINERS | 2 + hw/net/igb_core.c | 8 +- tests/qtest/fuzz/generic_fuzz_configs.h | 5 + tests/qtest/igb-test.c

[PATCH v7 3/9] e1000: Split header files

2023-01-31 Thread Akihiko Odaki
Some definitions in the header files are invalid for igb so extract them to new header files to keep igb from referring to them. Signed-off-by: Gal Hammer Signed-off-by: Marcel Apfelbaum Signed-off-by: Akihiko Odaki Reviewed-by: Philippe Mathieu-Daudé --- hw/net/e1000.c | 1 + hw

[PATCH v7 9/9] docs/system/devices/igb: Add igb documentation

2023-01-31 Thread Akihiko Odaki
Signed-off-by: Akihiko Odaki --- MAINTAINERS | 1 + docs/system/device-emulation.rst | 1 + docs/system/devices/igb.rst | 71 3 files changed, 73 insertions(+) create mode 100644 docs/system/devices/igb.rst diff --git a/MAINTAINERS b

Re: [PATCH v3 3/9] igb: add ICR_RXDW

2023-01-31 Thread Akihiko Odaki
On 2023/01/31 18:42, Sriram Yagnaraman wrote: IGB uses RXDW ICR bit to indicate that rx descriptor has been written back. This is the same as RXT0 bit in older HW. Signed-off-by: Sriram Yagnaraman --- hw/net/e1000x_regs.h | 4 hw/net/igb_core.c| 46 +

Re: [PATCH v3 5/9] igb: check oversized packets for VMDq

2023-01-31 Thread Akihiko Odaki
On 2023/01/31 18:42, Sriram Yagnaraman wrote: Signed-off-by: Sriram Yagnaraman --- hw/net/igb_core.c | 48 +++ 1 file changed, 40 insertions(+), 8 deletions(-) diff --git a/hw/net/igb_core.c b/hw/net/igb_core.c index 4a1b98bf0e..2f6f30341f 100644 -

Re: [PATCH v3 4/9] igb: implement VFRE and VFTE registers

2023-01-31 Thread Akihiko Odaki
On 2023/01/31 18:42, Sriram Yagnaraman wrote: Also add checks for RXDCTL/TXDCTL queue enable bits Signed-off-by: Sriram Yagnaraman --- hw/net/igb_core.c | 30 +- hw/net/igb_core.h | 1 + hw/net/igb_regs.h | 3 +++ 3 files changed, 29 insertions(+), 5 deletions

Re: [PATCH v3 9/9] igb: respect VMVIR and VMOLR for VLAN

2023-01-31 Thread Akihiko Odaki
On 2023/01/31 18:42, Sriram Yagnaraman wrote: Add support for stripping/inserting VLAN for VFs. Signed-off-by: Sriram Yagnaraman --- hw/net/igb_core.c | 51 ++- 1 file changed, 42 insertions(+), 9 deletions(-) diff --git a/hw/net/igb_core.c b/hw/n

Re: [PATCH v3 8/9] igb: respect VT_CTL ignore MAC field

2023-01-31 Thread Akihiko Odaki
On 2023/01/31 18:42, Sriram Yagnaraman wrote: Also trace out a warning if replication mode is disabled, since we only support replication mode enabled. Signed-off-by: Sriram Yagnaraman --- hw/net/igb_core.c | 9 + hw/net/trace-events | 2 ++ 2 files changed, 11 insertions(+) diff

Re: [PATCH v3 8/9] igb: respect VT_CTL ignore MAC field

2023-02-01 Thread Akihiko Odaki
On 2023/02/01 19:29, Sriram Yagnaraman wrote: -Original Message- From: Akihiko Odaki Sent: Wednesday, 1 February 2023 05:58 To: Sriram Yagnaraman Cc: qemu-devel@nongnu.org; Jason Wang ; Dmitry Fleytman ; Michael S . Tsirkin ; Marcel Apfelbaum Subject: Re: [PATCH v3 8/9] igb

Re: [PATCH v4 4/9] igb: implement VFRE and VFTE registers

2023-02-01 Thread Akihiko Odaki
On 2023/02/01 20:17, Sriram Yagnaraman wrote: Also introduce: - Checks for RXDCTL/TXDCTL queue enable bits - IGB_NUM_VM_POOLS enum (Sec 1.5: Table 1-7) It may be better to have RXDCTL.ENABLE check in igb_can_receive(). Signed-off-by: Sriram Yagnaraman --- hw/net/igb_core.c | 36 ++

[PATCH v2] MAINTAINERS: Update Akihiko Odaki's email address

2023-05-06 Thread Akihiko Odaki
From: Akihiko Odaki I am now employed by Daynix. Although my role as a reviewer of macOS-related change is not very relevant to the employment, I decided to use the company email address to avoid confusions from different addresses. Signed-off-by: Akihiko Odaki Reviewed-by: Marc-André Lureau

Re: [PATCH v5 6/6] igb: packet-split descriptors support

2023-05-11 Thread Akihiko Odaki
Hi, Thank you for continuously working on this series. I have some comments, but I guess this series will be ready after one or two more rounds. On 2023/05/10 17:22, Tomasz Dzieciol wrote: Packet-split descriptors are used by Linux VF driver for MTU values from 2048 Signed-off-by: Tomasz Dzi

Re: [PATCH v6 6/7] igb: packet-split descriptors support

2023-05-13 Thread Akihiko Odaki
On 2023/05/13 0:43, Tomasz Dzieciol wrote: Packet-split descriptors are used by Linux VF driver for MTU values from 2048 Signed-off-by: Tomasz Dzieciol --- hw/net/igb_core.c | 360 ++-- hw/net/igb_regs.h | 9 ++ hw/net/trace-events | 2 +- 3

Re: [PATCH v7 3/7] igb: RX descriptors guest writting refactoring

2023-05-17 Thread Akihiko Odaki
On 2023/05/18 0:46, Tomasz Dzieciol wrote: Refactoring is done in preparation for support of multiple advanced descriptors RX modes, especially packet-split modes. Signed-off-by: Tomasz Dzieciol --- hw/net/igb_core.c | 176 ++-- hw/net/igb_regs.h |

[PATCH v8 4/4] module: Use bundle mechanism

2022-06-24 Thread Akihiko Odaki
Before this change, the directory of the executable was being added to resolve modules in the build tree. However, get_relocated_path() can now resolve them with the new bundle mechanism. Signed-off-by: Akihiko Odaki --- util/module.c | 1 - 1 file changed, 1 deletion(-) diff --git a/util

[PATCH v8 1/4] tests/vm: do not specify -bios option

2022-06-24 Thread Akihiko Odaki
installation. Just remove the -bios option, since it is unnecessary and in fact there are other x86 VM tests that do not bother specifying it. Signed-off-by: Paolo Bonzini Reviewed-by: Daniel P. Berrangé Reviewed-by: Thomas Huth Signed-off-by: Akihiko Odaki Message-Id: <20220616083025.116902-1-pb

[PATCH v8 2/4] cutils: Introduce bundle mechanism

2022-06-24 Thread Akihiko Odaki
system. Signed-off-by: Akihiko Odaki Suggested-by: Paolo Bonzini --- docs/about/build-platforms.rst | 2 +- include/qemu/cutils.h | 18 +++-- meson.build | 4 ++ scripts/symlink-install-tree.py | 37 ++ util/cutils.c | 68

[PATCH v8 0/4] cutils: Introduce bundle mechanism

2022-06-24 Thread Akihiko Odaki
luding but not limited to datadir. (Peter Maydell) * Fix "bridge" typo (Philippe Mathieu-Daudé) v2: Rebased to the latest QEMU. Akihiko Odaki (3): cutils: Introduce bundle mechanism datadir: Use bundle mechanism module: Use bundle mechanism Paolo Bonzini (1): tests/vm: do not

[PATCH] qga: Relocate a path emitted in the help text

2022-06-24 Thread Akihiko Odaki
Signed-off-by: Akihiko Odaki --- qga/main.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/qga/main.c b/qga/main.c index c373fec3ee6..5f1efa23334 100644 --- a/qga/main.c +++ b/qga/main.c @@ -223,6 +223,10 @@ void reopen_fd_to_null(int fd) static void usage(const char

[PATCH v8 3/4] datadir: Use bundle mechanism

2022-06-24 Thread Akihiko Odaki
softmmu/datadir.c had its own implementation to find files in the build tree, but now bundle mechanism provides the unified implementation which works for datadir and the other files. Signed-off-by: Akihiko Odaki --- .travis.yml | 2 +- pc-bios/keymaps/meson.build | 21

[PATCH] build: Do not depend on pc-bios for config-host.mak

2022-06-24 Thread Akihiko Odaki
Commit 45f1eecdd63f9e4fa93fef01dd826e7706ac6d7b removed the dependency from configure to pc-bios Signed-off-by: Akihiko Odaki --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 3c0d89057eb..2e95d473b3e 100644 --- a/Makefile +++ b/Makefile

Re: [PATCH v8 0/4] cutils: Introduce bundle mechanism

2022-06-24 Thread Akihiko Odaki
inions? Actually I prepended qemu- to the directory because all executables of QEMU has the prefix. We can safely say QEMU installation will never collide if no file prefixed with qemu- exists in the bindir. If anyone has an idea what should follow "qemu-", please tell us. Regards, Akihiko Odaki

[PATCH] meson: Prefix each element of firmware path

2022-06-24 Thread Akihiko Odaki
Signed-off-by: Akihiko Odaki --- configure | 23 +++ meson.build | 10 -- meson_options.txt | 2 +- scripts/meson-buildoptions.py | 7 +-- scripts/meson-buildoptions.sh | 6 +++--- softmmu/datadir.c

Re: [PATCH v6 0/7] cutils: Introduce bundle mechanism

2022-06-24 Thread Akihiko Odaki
d d2[1] == ':': +    return d1 + d2[2:] +    return d1 + d2 This is from Meson but buggy so I fixed it and opened a pull request for Meson: https://github.com/mesonbuild/meson/pull/10531 The script included in v8 has the fixed version of destdir_join. Regards, Akihiko Odaki

[PATCH v9 0/4] cutils: Introduce bundle mechanism

2022-06-24 Thread Akihiko Odaki
is for any files located relative to the installation tree including but not limited to datadir. (Peter Maydell) * Fix "bridge" typo (Philippe Mathieu-Daudé) v2: Rebased to the latest QEMU. Akihiko Odaki (3): cutils: Introduce bundle mechanism datadir: Use bundle mechanism module:

[PATCH v9 1/4] tests/vm: do not specify -bios option

2022-06-24 Thread Akihiko Odaki
installation. Just remove the -bios option, since it is unnecessary and in fact there are other x86 VM tests that do not bother specifying it. Signed-off-by: Paolo Bonzini Reviewed-by: Daniel P. Berrangé Reviewed-by: Thomas Huth Signed-off-by: Akihiko Odaki Message-Id: <20220616083025.116902-1-pb

[PATCH v9 3/4] datadir: Use bundle mechanism

2022-06-24 Thread Akihiko Odaki
softmmu/datadir.c had its own implementation to find files in the build tree, but now bundle mechanism provides the unified implementation which works for datadir and the other files. Signed-off-by: Akihiko Odaki --- .travis.yml | 2 +- pc-bios/keymaps/meson.build | 21

[PATCH v9 2/4] cutils: Introduce bundle mechanism

2022-06-24 Thread Akihiko Odaki
system. Signed-off-by: Akihiko Odaki Suggested-by: Paolo Bonzini --- docs/about/build-platforms.rst | 2 +- include/qemu/cutils.h | 18 +++-- include/qemu/osdep.h| 2 +- meson.build | 4 ++ scripts/symlink-install-tree.py | 37

[PATCH v9 4/4] module: Use bundle mechanism

2022-06-24 Thread Akihiko Odaki
Before this change, the directory of the executable was being added to resolve modules in the build tree. However, get_relocated_path() can now resolve them with the new bundle mechanism. Signed-off-by: Akihiko Odaki --- util/module.c | 1 - 1 file changed, 1 deletion(-) diff --git a/util

[PATCH v10 0/4] cutils: Introduce bundle mechanism

2022-06-26 Thread Akihiko Odaki
po (Philippe Mathieu-Daudé) v2: Rebased to the latest QEMU. Akihiko Odaki (3): cutils: Introduce bundle mechanism datadir: Use bundle mechanism module: Use bundle mechanism Paolo Bonzini (1): tests/vm: do not specify -bios option .travis.yml | 2 +- docs/about/build

[PATCH v10 1/4] tests/vm: do not specify -bios option

2022-06-26 Thread Akihiko Odaki
installation. Just remove the -bios option, since it is unnecessary and in fact there are other x86 VM tests that do not bother specifying it. Signed-off-by: Paolo Bonzini Reviewed-by: Daniel P. Berrangé Reviewed-by: Thomas Huth Signed-off-by: Akihiko Odaki Message-Id: <20220616083025.116902-1-pb

[PATCH v10 4/4] module: Use bundle mechanism

2022-06-26 Thread Akihiko Odaki
Before this change, the directory of the executable was being added to resolve modules in the build tree. However, get_relocated_path() can now resolve them with the new bundle mechanism. Signed-off-by: Akihiko Odaki --- util/module.c | 1 - 1 file changed, 1 deletion(-) diff --git a/util

[PATCH v10 2/4] cutils: Introduce bundle mechanism

2022-06-26 Thread Akihiko Odaki
system. Signed-off-by: Akihiko Odaki Suggested-by: Paolo Bonzini --- docs/about/build-platforms.rst | 2 +- include/qemu/cutils.h | 18 +++-- include/qemu/osdep.h| 2 +- meson.build | 4 ++ scripts/symlink-install-tree.py | 34

[PATCH v10 3/4] datadir: Use bundle mechanism

2022-06-26 Thread Akihiko Odaki
softmmu/datadir.c had its own implementation to find files in the build tree, but now bundle mechanism provides the unified implementation which works for datadir and the other files. Signed-off-by: Akihiko Odaki --- .travis.yml | 2 +- pc-bios/keymaps/meson.build | 21

[PATCH v4] ui/cocoa: Take refresh rate into account

2022-07-02 Thread Akihiko Odaki
Hz physical display updates it is pointless for the guest to update the screen at 60Hz frequency, the guest can spare some work instead. Signed-off-by: Akihiko Odaki --- meson.build | 3 ++- ui/cocoa.m | 12 2 files changed, 14 insertions(+), 1 deletion(-) diff --git a/meson.build

Re: [PATCH] ui/cocoa: Fix switched_to_fullscreen warning

2022-07-02 Thread Akihiko Odaki
Reviewed-by: Akihiko Odaki On 2022/07/02 13:43, Peter Delevoryas wrote: I noticed this error while building QEMU on Mac OS X: [1040/1660] Compiling Objective-C object libcommon.fa.p/ui_cocoa.m.o ../ui/cocoa.m:803:17: warning: variable 'switched_to_fullscreen' set but

[PATCH 0/3] ui/cocoa: Run qemu_init in the main thread

2022-07-05 Thread Akihiko Odaki
implementation of main() for builds with ui/cocoa and ones without ui/cocoa. Akihiko Odaki (3): ui/cocoa: Run qemu_init in the main thread Revert "main-loop: Disable block backend global state assertion on Cocoa" meson: Allow to enable gtk and sdl while cocoa is enabled

[PATCH 1/3] ui/cocoa: Run qemu_init in the main thread

2022-07-05 Thread Akihiko Odaki
implementation of main() for builds with ui/cocoa and ones without ui/cocoa. Signed-off-by: Akihiko Odaki --- docs/devel/fuzzing.rst | 4 +- include/qemu-main.h | 3 +- include/sysemu/sysemu.h | 2 +- softmmu/main.c | 14 +-- softmmu/vl.c| 2 +- tests/qtest/fuzz

[PATCH 2/3] Revert "main-loop: Disable block backend global state assertion on Cocoa"

2022-07-05 Thread Akihiko Odaki
This reverts commit 47281859f66bdab1974fb122cab2cbb4a1c9af7f. Signed-off-by: Akihiko Odaki --- include/qemu/main-loop.h | 13 - 1 file changed, 13 deletions(-) diff --git a/include/qemu/main-loop.h b/include/qemu/main-loop.h index 5518845299d..0aa36a4f17e 100644 --- a/include/qemu

[PATCH 3/3] meson: Allow to enable gtk and sdl while cocoa is enabled

2022-07-05 Thread Akihiko Odaki
As ui/cocoa does no longer override main(), ui/gtk and ui/sdl can be enabled even ui/cocoa is enabled. Signed-off-by: Akihiko Odaki --- meson.build | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/meson.build b/meson.build index 6e1c3eb2bc5..4714a0d5cf8 100644

virtio-iommu hotplug issue

2023-04-12 Thread Akihiko Odaki
t on QEMU-side. It is not completely clear for me which solution is more appropriate as the virtio-iommu specification is written in a way independent of the endpoint mechanism and does not say what should be done when a PCI device is unplugged. Regards, Akihiko Odaki

Re: virtio-iommu hotplug issue

2023-04-13 Thread Akihiko Odaki
On 2023/04/13 19:40, Jean-Philippe Brucker wrote: Hello, On Thu, Apr 13, 2023 at 01:49:43PM +0900, Akihiko Odaki wrote: Hi, Recently I encountered a problem with the combination of Linux's virtio-iommu driver and QEMU when a SR-IOV virtual function gets disabled. I'd like to ask you

Re: virtio-iommu hotplug issue

2023-04-13 Thread Akihiko Odaki
On 2023/04/13 22:39, Eric Auger wrote: Hi, On 4/13/23 13:01, Akihiko Odaki wrote: On 2023/04/13 19:40, Jean-Philippe Brucker wrote: Hello, On Thu, Apr 13, 2023 at 01:49:43PM +0900, Akihiko Odaki wrote: Hi, Recently I encountered a problem with the combination of Linux's virtio-iommu d

[PATCH] docs: Remove obsolete descriptions of SR-IOV support

2023-04-14 Thread Akihiko Odaki
The documentation used to say there is no device implemented with SR-IOV, but igb and nvme support SR-IOV today. Signed-off-by: Akihiko Odaki --- docs/pcie_sriov.txt | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/docs/pcie_sriov.txt b/docs/pcie_sriov.txt index

[PATCH 00/40] igb: Fix for DPDK

2023-04-14 Thread Akihiko Odaki
] implement new features. Patch [39, 40] update documentations. While this includes so many patches, it is not necessary to land them at once. Only bug fix patches may be applied first, for example. Akihiko Odaki (40): hw/net/net_tx_pkt: Decouple from PCI e1000x: Fix BPRC and MPRC igb: Fix Rx

[PATCH 02/40] e1000x: Fix BPRC and MPRC

2023-04-14 Thread Akihiko Odaki
ned-off-by: Akihiko Odaki --- hw/net/e1000.c | 6 +++--- hw/net/e1000e_core.c | 20 +++- hw/net/e1000x_common.c | 25 +++-- hw/net/e1000x_common.h | 5 +++-- hw/net/igb_core.c | 22 +- 5 files changed, 33 insertions(+), 45

[PATCH 04/40] igb: Include the second VLAN tag in the buffer

2023-04-14 Thread Akihiko Odaki
Signed-off-by: Akihiko Odaki --- hw/net/igb_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/net/igb_core.c b/hw/net/igb_core.c index 55de212447..f725ab97ae 100644 --- a/hw/net/igb_core.c +++ b/hw/net/igb_core.c @@ -1590,7 +1590,7 @@ static ssize_t

[PATCH 03/40] igb: Fix Rx packet type encoding

2023-04-14 Thread Akihiko Odaki
igb's advanced descriptor uses a packet type encoding different from one used in e1000e's extended descriptor. Fix the logic to encode Rx packet type accordingly. Fixes: 3a977deebe ("Intrdocue igb device emulation") Signed-off-by: Akihiko Odaki --- hw

[PATCH 11/40] tests/avocado: Remove test_igb_nomsi_kvm

2023-04-14 Thread Akihiko Odaki
It is unlikely to find more bugs with KVM so remove test_igb_nomsi_kvm to save time to run it. Signed-off-by: Akihiko Odaki --- tests/avocado/netdev-ethtool.py | 12 +--- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/tests/avocado/netdev-ethtool.py b/tests/avocado/netdev

[PATCH 12/40] hw/net/net_tx_pkt: Remove net_rx_pkt_get_l4_info

2023-04-14 Thread Akihiko Odaki
This function is not used. Signed-off-by: Akihiko Odaki --- hw/net/net_rx_pkt.c | 5 - hw/net/net_rx_pkt.h | 9 - 2 files changed, 14 deletions(-) diff --git a/hw/net/net_rx_pkt.c b/hw/net/net_rx_pkt.c index 63be6e05ad..6125a063d7 100644 --- a/hw/net/net_rx_pkt.c +++ b/hw/net

<    4   5   6   7   8   9   10   11   12   13   >