[PATCH v8 0/7] igb: Implement Transmit Rate Limiting (TRL)

Josh Hilke posted 7 patches 1 month ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260623-igb-trl-v8-0-c9b1fc6c3304@google.com
Maintainers: Akihiko Odaki <odaki@rsg.ci.i.u-tokyo.ac.jp>, Sriram Yagnaraman <sriram.yagnaraman@ericsson.com>, Jason Wang <jasowang@redhat.com>, Fabiano Rosas <farosas@suse.de>, Laurent Vivier <lvivier@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>
hw/net/igb.c           |  43 +++++++++++
hw/net/igb_common.h    |   1 +
hw/net/igb_core.c      | 142 ++++++++++++++++++++++++++++++++--
hw/net/igb_core.h      |  12 +++
hw/net/igb_regs.h      |   5 ++
tests/qtest/igb-test.c | 206 +++++++++++++++++++++++++++++++++++++++++++++++++
6 files changed, 404 insertions(+), 5 deletions(-)
[PATCH v8 0/7] igb: Implement Transmit Rate Limiting (TRL)
Posted by Josh Hilke 1 month ago
This is v8 of a patch series implementing the Transmit Rate Limiter (TRL) feature for
the Intel 82576 (igb) network device emulation in QEMU.

v7: https://lists.gnu.org/archive/html/qemu-devel/2026-06/msg05440.html

Changelog:

v8:
  - Fix extraneous whitespace modifications.
  - Use muldiv64() in igb_trl_calculate_delay().
  - Fix comment for rate factor in igb_trl_calculate_delay().

Signed-off-by: Josh Hilke <jrhilke@google.com>
---
Josh Hilke (7):
      igb: Define TRL registers and core state
      igb: Add VMState support for TRL state
      igb: Implement TRL register write and configuration
      igb: Refactor TX path to return processed byte count
      igb: Implement TRL throttling and timer resumption
      igb: Add QTest for Transmit Rate Limiter register multiplexing
      igb: Add QTest for Transmit Rate Limiter traffic throttling

 hw/net/igb.c           |  43 +++++++++++
 hw/net/igb_common.h    |   1 +
 hw/net/igb_core.c      | 142 ++++++++++++++++++++++++++++++++--
 hw/net/igb_core.h      |  12 +++
 hw/net/igb_regs.h      |   5 ++
 tests/qtest/igb-test.c | 206 +++++++++++++++++++++++++++++++++++++++++++++++++
 6 files changed, 404 insertions(+), 5 deletions(-)
---
base-commit: 98b060da3a4f92b2a994ead5b16a87e783baf77c
change-id: 20260608-igb-trl-19be198da6f2

Best regards,
-- 
Josh Hilke <jrhilke@google.com>
Re: [PATCH v8 0/7] igb: Implement Transmit Rate Limiting (TRL)
Posted by Akihiko Odaki 1 month ago
On 2026/06/24 0:05, Josh Hilke wrote:
> This is v8 of a patch series implementing the Transmit Rate Limiter (TRL) feature for
> the Intel 82576 (igb) network device emulation in QEMU.
> 
> v7: https://lists.gnu.org/archive/html/qemu-devel/2026-06/msg05440.html
> 
> Changelog:
> 
> v8:
>    - Fix extraneous whitespace modifications.
>    - Use muldiv64() in igb_trl_calculate_delay().
>    - Fix comment for rate factor in igb_trl_calculate_delay().>
> Signed-off-by: Josh Hilke <jrhilke@google.com>

For the whole series:

Reviewed-by: Akihiko Odaki <odaki@rsg.ci.i.u-tokyo.ac.jp>

Regards,
Akihiko Odaki

> ---
> Josh Hilke (7):
>        igb: Define TRL registers and core state
>        igb: Add VMState support for TRL state
>        igb: Implement TRL register write and configuration
>        igb: Refactor TX path to return processed byte count
>        igb: Implement TRL throttling and timer resumption
>        igb: Add QTest for Transmit Rate Limiter register multiplexing
>        igb: Add QTest for Transmit Rate Limiter traffic throttling
> 
>   hw/net/igb.c           |  43 +++++++++++
>   hw/net/igb_common.h    |   1 +
>   hw/net/igb_core.c      | 142 ++++++++++++++++++++++++++++++++--
>   hw/net/igb_core.h      |  12 +++
>   hw/net/igb_regs.h      |   5 ++
>   tests/qtest/igb-test.c | 206 +++++++++++++++++++++++++++++++++++++++++++++++++
>   6 files changed, 404 insertions(+), 5 deletions(-)
> ---
> base-commit: 98b060da3a4f92b2a994ead5b16a87e783baf77c
> change-id: 20260608-igb-trl-19be198da6f2
> 
> Best regards,
Re: [PATCH v8 0/7] igb: Implement Transmit Rate Limiting (TRL)
Posted by Josh Hilke 3 weeks, 5 days ago
Do I need to push a v9 to include your "Reviewed-by" tag?

Thanks,
Josh

On Wed, Jun 24, 2026 at 1:18 AM Akihiko Odaki
<odaki@rsg.ci.i.u-tokyo.ac.jp> wrote:
>
> On 2026/06/24 0:05, Josh Hilke wrote:
> > This is v8 of a patch series implementing the Transmit Rate Limiter (TRL) feature for
> > the Intel 82576 (igb) network device emulation in QEMU.
> >
> > v7: https://lists.gnu.org/archive/html/qemu-devel/2026-06/msg05440.html
> >
> > Changelog:
> >
> > v8:
> >    - Fix extraneous whitespace modifications.
> >    - Use muldiv64() in igb_trl_calculate_delay().
> >    - Fix comment for rate factor in igb_trl_calculate_delay().>
> > Signed-off-by: Josh Hilke <jrhilke@google.com>
>
> For the whole series:
>
> Reviewed-by: Akihiko Odaki <odaki@rsg.ci.i.u-tokyo.ac.jp>
>
> Regards,
> Akihiko Odaki
>
> > ---
> > Josh Hilke (7):
> >        igb: Define TRL registers and core state
> >        igb: Add VMState support for TRL state
> >        igb: Implement TRL register write and configuration
> >        igb: Refactor TX path to return processed byte count
> >        igb: Implement TRL throttling and timer resumption
> >        igb: Add QTest for Transmit Rate Limiter register multiplexing
> >        igb: Add QTest for Transmit Rate Limiter traffic throttling
> >
> >   hw/net/igb.c           |  43 +++++++++++
> >   hw/net/igb_common.h    |   1 +
> >   hw/net/igb_core.c      | 142 ++++++++++++++++++++++++++++++++--
> >   hw/net/igb_core.h      |  12 +++
> >   hw/net/igb_regs.h      |   5 ++
> >   tests/qtest/igb-test.c | 206 +++++++++++++++++++++++++++++++++++++++++++++++++
> >   6 files changed, 404 insertions(+), 5 deletions(-)
> > ---
> > base-commit: 98b060da3a4f92b2a994ead5b16a87e783baf77c
> > change-id: 20260608-igb-trl-19be198da6f2
> >
> > Best regards,
>
Re: [PATCH v8 0/7] igb: Implement Transmit Rate Limiting (TRL)
Posted by Akihiko Odaki 3 weeks, 5 days ago
On 2026/06/30 3:11, Josh Hilke wrote:
> Do I need to push a v9 to include your "Reviewed-by" tag?

No. I think this series is ready to pull along with other network patches.

Regards,
Akihiko Odaki

> 
> Thanks,
> Josh
> 
> On Wed, Jun 24, 2026 at 1:18 AM Akihiko Odaki
> <odaki@rsg.ci.i.u-tokyo.ac.jp> wrote:
>>
>> On 2026/06/24 0:05, Josh Hilke wrote:
>>> This is v8 of a patch series implementing the Transmit Rate Limiter (TRL) feature for
>>> the Intel 82576 (igb) network device emulation in QEMU.
>>>
>>> v7: https://lists.gnu.org/archive/html/qemu-devel/2026-06/msg05440.html
>>>
>>> Changelog:
>>>
>>> v8:
>>>     - Fix extraneous whitespace modifications.
>>>     - Use muldiv64() in igb_trl_calculate_delay().
>>>     - Fix comment for rate factor in igb_trl_calculate_delay().>
>>> Signed-off-by: Josh Hilke <jrhilke@google.com>
>>
>> For the whole series:
>>
>> Reviewed-by: Akihiko Odaki <odaki@rsg.ci.i.u-tokyo.ac.jp>
>>
>> Regards,
>> Akihiko Odaki
>>
>>> ---
>>> Josh Hilke (7):
>>>         igb: Define TRL registers and core state
>>>         igb: Add VMState support for TRL state
>>>         igb: Implement TRL register write and configuration
>>>         igb: Refactor TX path to return processed byte count
>>>         igb: Implement TRL throttling and timer resumption
>>>         igb: Add QTest for Transmit Rate Limiter register multiplexing
>>>         igb: Add QTest for Transmit Rate Limiter traffic throttling
>>>
>>>    hw/net/igb.c           |  43 +++++++++++
>>>    hw/net/igb_common.h    |   1 +
>>>    hw/net/igb_core.c      | 142 ++++++++++++++++++++++++++++++++--
>>>    hw/net/igb_core.h      |  12 +++
>>>    hw/net/igb_regs.h      |   5 ++
>>>    tests/qtest/igb-test.c | 206 +++++++++++++++++++++++++++++++++++++++++++++++++
>>>    6 files changed, 404 insertions(+), 5 deletions(-)
>>> ---
>>> base-commit: 98b060da3a4f92b2a994ead5b16a87e783baf77c
>>> change-id: 20260608-igb-trl-19be198da6f2
>>>
>>> Best regards,
>>