Re: [Qemu-devel] [PATCH v4 00/16] Introduce Intel 82574 GbE Controller Emulation (e1000e)
On Sun, 10 Apr 2016 19:22:17 +0300 Dmitry Fleytman wrote: > > On 10 Apr 2016, at 18:35, Michael S. Tsirkin wrote: > > > >> On Sun, Apr 10, 2016 at 06:14:49PM +0300, Dmitry Fleytman wrote: > >> From: Dmitry Fleytman > >> > >> Hello All, > >> > >> This is v4 of e1000e series. > >> > >> For convenience, the same patches are available at: > >> https://github.com/daynix/qemu-e1000e/tree/e1000e-submit-v4 > >> > >> Best regards, > >> Dmitry. > >> > >> Changes since v3: > >> > >> 1. Various code fixes as suggested by Jason and Michael > >> 2. Rebased to the latest master > > > > Pls remember to repost when 2.6 is out. > > Sure. When do you think 2.6 will be out? Always subject to change, but http://wiki.qemu.org/Planning/2.6 says April 26th. Thanks, Alex > > > > >> Changes since v2: > >> > >> 1. Interrupt storm on latest Linux kernels fixed > >> 2. Device unit test added > >> 3. Introduced code sharing between e1000 and e1000e > >> 4. Various code fixes as suggested by Jason > >> 5. Rebased to the latest master > >> > >> Changes since v1: > >> > >> 1. PCI_PM_CAP_VER_1_1 is defined now in include/hw/pci/pci_regs.h and > >> not in include/standard-headers/linux/pci_regs.h. > >> 2. Changes in naming and extra comments in hw/pci/pcie.c and in > >> include/hw/pci/pcie.h. > >> 3. Defining pci_dsn_ver and pci_dsn_cap static const variables in > >> hw/pci/pcie.c, instead of PCI_DSN_VER and PCI_DSN_CAP symbolic > >> constants in include/hw/pci/pcie_regs.h. > >> 4. Changing the vmxnet3_device_serial_num function in hw/net/vmxnet3.c > >> to avoid the cast when it is called. > >> 5. Avoiding a preceding underscore in all the e1000e-related names. > >> 6. Minor style changes. > >> > >> === > >> > >> Hello All, > >> > >> This series is the final code of the e1000e device emulation, that we > >> have developed. Please review, and consider acceptance of these patches > >> to the upstream QEMU repository. > >> > >> The code stability was verified by various traffic tests using Fedora 22 > >> Linux, and Windows Server 2012R2 guests. Also, Microsoft Hardware > >> Certification Kit (HCK) tests were run on a Windows Server 2012R2 guest. > >> > >> There was a discussion on the possibility of code sharing between the > >> e1000e, and the existing e1000 devices. We have reviewed the final code > >> for parts that may be shared between this device and the currently > >> available e1000 emulation. The device specifications are very different, > >> and there are almost no registers, nor functions, that were left as is > >> from e1000. The ring descriptor structures were changed as well, by the > >> introduction of extended and PS descriptors, as well as additional bits. > >> > >> Additional differences stem from the fact that the e1000e device re-uses > >> network packet abstractions introduced by the vmxnet3 device, while the > >> e1000 has its own code for packet handling. BTW, it may be worth reusing > >> those abstractions in e1000 as well. (Following these changes the > >> vmxnet3 device was successfully tested for possible regressions.) > >> > >> There are a few minor parts that may be shared, e.g. the default > >> register handlers, and the ring management functions. The total amount > >> of shared lines will be about 100--150, so we're not sure if it makes > >> sense bothering, and taking a risk of breaking e1000, which is a good, > >> old, and stable device. > >> > >> Currently, the e1000e code is stand alone w.r.t. e1000. > >> > >> Please share your thoughts. > >> > >> Thanks in advance, > >> Dmitry. > >> > >> Changes since RFCv2: > >> > >> 1. Device functionality verified using Microsoft Hardware Certification > >> Test Kit (HCK) > >> 2. Introduced a number of performance improvements > >> 3. The code was cleaned, and rebased to the latest master > >> 4. Patches verified with checkpatch.pl > >> > >> === > >> > >> Changes since RFCv1: > >> > >> 1. Added support for all the device features: > >> - Interrupt moderation. > >> - RSS. > >> - Multiqueue. > >> 2. Simulated exact PCI/PCIe configuration space layout. > >> 3. Made fixes needed to pass Microsoft's HW certification tests (HCK). > >> > >> This series is still an RFC, because the following tasks are not done yet: > >> > >> 1. See which code can be shared between this device and the existing e1000 > >> device. > >> 2. Rebase patches to the latest master (current base is v2.3.0). > >> > >> Please share your thoughts, > >> Thanks, Dmitry. > >> > >> === > >> > >> Hello qemu-devel, > >> > >> This patch series is an RFC for the new networking device emulation > >> we're developing for QEMU. > >> > >> This new device emulates the Intel 82574 GbE Controller and works > >> with unmodified Intel e1000e drivers from the Linux/Windows kernels. > >> > >> The status of the current series is "Functional Device Ready, work > >> on Extended Features in Progress". > >> > >> More p
Re: [Qemu-devel] [PATCH v4 00/16] Introduce Intel 82574 GbE Controller Emulation (e1000e)
> On 10 Apr 2016, at 18:35, Michael S. Tsirkin wrote: > >> On Sun, Apr 10, 2016 at 06:14:49PM +0300, Dmitry Fleytman wrote: >> From: Dmitry Fleytman >> >> Hello All, >> >> This is v4 of e1000e series. >> >> For convenience, the same patches are available at: >> https://github.com/daynix/qemu-e1000e/tree/e1000e-submit-v4 >> >> Best regards, >> Dmitry. >> >> Changes since v3: >> >> 1. Various code fixes as suggested by Jason and Michael >> 2. Rebased to the latest master > > Pls remember to repost when 2.6 is out. Sure. When do you think 2.6 will be out? > >> Changes since v2: >> >> 1. Interrupt storm on latest Linux kernels fixed >> 2. Device unit test added >> 3. Introduced code sharing between e1000 and e1000e >> 4. Various code fixes as suggested by Jason >> 5. Rebased to the latest master >> >> Changes since v1: >> >> 1. PCI_PM_CAP_VER_1_1 is defined now in include/hw/pci/pci_regs.h and >> not in include/standard-headers/linux/pci_regs.h. >> 2. Changes in naming and extra comments in hw/pci/pcie.c and in >> include/hw/pci/pcie.h. >> 3. Defining pci_dsn_ver and pci_dsn_cap static const variables in >> hw/pci/pcie.c, instead of PCI_DSN_VER and PCI_DSN_CAP symbolic >> constants in include/hw/pci/pcie_regs.h. >> 4. Changing the vmxnet3_device_serial_num function in hw/net/vmxnet3.c >> to avoid the cast when it is called. >> 5. Avoiding a preceding underscore in all the e1000e-related names. >> 6. Minor style changes. >> >> === >> >> Hello All, >> >> This series is the final code of the e1000e device emulation, that we >> have developed. Please review, and consider acceptance of these patches >> to the upstream QEMU repository. >> >> The code stability was verified by various traffic tests using Fedora 22 >> Linux, and Windows Server 2012R2 guests. Also, Microsoft Hardware >> Certification Kit (HCK) tests were run on a Windows Server 2012R2 guest. >> >> There was a discussion on the possibility of code sharing between the >> e1000e, and the existing e1000 devices. We have reviewed the final code >> for parts that may be shared between this device and the currently >> available e1000 emulation. The device specifications are very different, >> and there are almost no registers, nor functions, that were left as is >> from e1000. The ring descriptor structures were changed as well, by the >> introduction of extended and PS descriptors, as well as additional bits. >> >> Additional differences stem from the fact that the e1000e device re-uses >> network packet abstractions introduced by the vmxnet3 device, while the >> e1000 has its own code for packet handling. BTW, it may be worth reusing >> those abstractions in e1000 as well. (Following these changes the >> vmxnet3 device was successfully tested for possible regressions.) >> >> There are a few minor parts that may be shared, e.g. the default >> register handlers, and the ring management functions. The total amount >> of shared lines will be about 100--150, so we're not sure if it makes >> sense bothering, and taking a risk of breaking e1000, which is a good, >> old, and stable device. >> >> Currently, the e1000e code is stand alone w.r.t. e1000. >> >> Please share your thoughts. >> >> Thanks in advance, >> Dmitry. >> >> Changes since RFCv2: >> >> 1. Device functionality verified using Microsoft Hardware Certification Test >> Kit (HCK) >> 2. Introduced a number of performance improvements >> 3. The code was cleaned, and rebased to the latest master >> 4. Patches verified with checkpatch.pl >> >> === >> >> Changes since RFCv1: >> >> 1. Added support for all the device features: >> - Interrupt moderation. >> - RSS. >> - Multiqueue. >> 2. Simulated exact PCI/PCIe configuration space layout. >> 3. Made fixes needed to pass Microsoft's HW certification tests (HCK). >> >> This series is still an RFC, because the following tasks are not done yet: >> >> 1. See which code can be shared between this device and the existing e1000 >> device. >> 2. Rebase patches to the latest master (current base is v2.3.0). >> >> Please share your thoughts, >> Thanks, Dmitry. >> >> === >> >> Hello qemu-devel, >> >> This patch series is an RFC for the new networking device emulation >> we're developing for QEMU. >> >> This new device emulates the Intel 82574 GbE Controller and works >> with unmodified Intel e1000e drivers from the Linux/Windows kernels. >> >> The status of the current series is "Functional Device Ready, work >> on Extended Features in Progress". >> >> More precisely, these patches represent a functional device, which >> is recognized by the standard Intel drivers, and is able to transfer >> TX/RX packets with CSO/TSO offloads, according to the spec. >> >> Extended features not supported yet (work in progress): >> 1. TX/RX Interrupt moderation mechanisms >> 2. RSS >> 3. Full-featured multi-queue (use of multiqueued network backend) >> >> Also, there will be some cod
Re: [Qemu-devel] [PATCH v4 00/16] Introduce Intel 82574 GbE Controller Emulation (e1000e)
On Sun, Apr 10, 2016 at 06:14:49PM +0300, Dmitry Fleytman wrote: > From: Dmitry Fleytman > > Hello All, > > This is v4 of e1000e series. > > For convenience, the same patches are available at: > https://github.com/daynix/qemu-e1000e/tree/e1000e-submit-v4 > > Best regards, > Dmitry. > > Changes since v3: > > 1. Various code fixes as suggested by Jason and Michael > 2. Rebased to the latest master Pls remember to repost when 2.6 is out. > Changes since v2: > > 1. Interrupt storm on latest Linux kernels fixed > 2. Device unit test added > 3. Introduced code sharing between e1000 and e1000e > 4. Various code fixes as suggested by Jason > 5. Rebased to the latest master > > Changes since v1: > > 1. PCI_PM_CAP_VER_1_1 is defined now in include/hw/pci/pci_regs.h and >not in include/standard-headers/linux/pci_regs.h. > 2. Changes in naming and extra comments in hw/pci/pcie.c and in >include/hw/pci/pcie.h. > 3. Defining pci_dsn_ver and pci_dsn_cap static const variables in >hw/pci/pcie.c, instead of PCI_DSN_VER and PCI_DSN_CAP symbolic >constants in include/hw/pci/pcie_regs.h. > 4. Changing the vmxnet3_device_serial_num function in hw/net/vmxnet3.c >to avoid the cast when it is called. > 5. Avoiding a preceding underscore in all the e1000e-related names. > 6. Minor style changes. > > === > > Hello All, > > This series is the final code of the e1000e device emulation, that we > have developed. Please review, and consider acceptance of these patches > to the upstream QEMU repository. > > The code stability was verified by various traffic tests using Fedora 22 > Linux, and Windows Server 2012R2 guests. Also, Microsoft Hardware > Certification Kit (HCK) tests were run on a Windows Server 2012R2 guest. > > There was a discussion on the possibility of code sharing between the > e1000e, and the existing e1000 devices. We have reviewed the final code > for parts that may be shared between this device and the currently > available e1000 emulation. The device specifications are very different, > and there are almost no registers, nor functions, that were left as is > from e1000. The ring descriptor structures were changed as well, by the > introduction of extended and PS descriptors, as well as additional bits. > > Additional differences stem from the fact that the e1000e device re-uses > network packet abstractions introduced by the vmxnet3 device, while the > e1000 has its own code for packet handling. BTW, it may be worth reusing > those abstractions in e1000 as well. (Following these changes the > vmxnet3 device was successfully tested for possible regressions.) > > There are a few minor parts that may be shared, e.g. the default > register handlers, and the ring management functions. The total amount > of shared lines will be about 100--150, so we're not sure if it makes > sense bothering, and taking a risk of breaking e1000, which is a good, > old, and stable device. > > Currently, the e1000e code is stand alone w.r.t. e1000. > > Please share your thoughts. > > Thanks in advance, > Dmitry. > > Changes since RFCv2: > > 1. Device functionality verified using Microsoft Hardware Certification Test > Kit (HCK) > 2. Introduced a number of performance improvements > 3. The code was cleaned, and rebased to the latest master > 4. Patches verified with checkpatch.pl > > === > > Changes since RFCv1: > > 1. Added support for all the device features: > - Interrupt moderation. > - RSS. > - Multiqueue. > 2. Simulated exact PCI/PCIe configuration space layout. > 3. Made fixes needed to pass Microsoft's HW certification tests (HCK). > > This series is still an RFC, because the following tasks are not done yet: > > 1. See which code can be shared between this device and the existing e1000 > device. > 2. Rebase patches to the latest master (current base is v2.3.0). > > Please share your thoughts, > Thanks, Dmitry. > > === > > Hello qemu-devel, > > This patch series is an RFC for the new networking device emulation > we're developing for QEMU. > > This new device emulates the Intel 82574 GbE Controller and works > with unmodified Intel e1000e drivers from the Linux/Windows kernels. > > The status of the current series is "Functional Device Ready, work > on Extended Features in Progress". > > More precisely, these patches represent a functional device, which > is recognized by the standard Intel drivers, and is able to transfer > TX/RX packets with CSO/TSO offloads, according to the spec. > > Extended features not supported yet (work in progress): > 1. TX/RX Interrupt moderation mechanisms > 2. RSS > 3. Full-featured multi-queue (use of multiqueued network backend) > > Also, there will be some code refactoring and performance > optimization efforts. > > This series was tested on Linux (Fedora 22) and Windows (2012R2) > guests, using Iperf, with TX/RX and TCP/UDP streams, and various > packet sizes. > > More thorough tes