[PATCH v10 0/7] igb: packet-split descriptors support

Tomasz Dzieciol posted 7 patches 11 months ago
Failed in applying to current master (apply log)
Maintainers: Dmitry Fleytman <dmitry.fleytman@gmail.com>, Akihiko Odaki <akihiko.odaki@daynix.com>, Jason Wang <jasowang@redhat.com>, Sriram Yagnaraman <sriram.yagnaraman@est.tech>, Thomas Huth <thuth@redhat.com>, Laurent Vivier <lvivier@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>
hw/net/e1000e_core.c     |  78 +++--
hw/net/igb_core.c        | 730 ++++++++++++++++++++++++++++-----------
hw/net/igb_regs.h        |  20 +-
hw/net/trace-events      |   6 +-
tests/qtest/libqos/igb.c |   5 +
5 files changed, 592 insertions(+), 247 deletions(-)
[PATCH v10 0/7] igb: packet-split descriptors support
Posted by Tomasz Dzieciol 11 months ago
Purposes of this series of patches:
* introduce packet-split RX descriptors support. This feature is used by Linux
  VF driver for MTU values from 2048.
* refactor RX descriptor handling for introduction of packet-split RX
  descriptors support
* fix descriptors flags handling

Tomasz Dzieciol (7):
  igb: remove TCP ACK detection
  igb: rename E1000E_RingInfo_st
  igb: RX descriptors guest writting refactoring
  igb: RX payload guest writting refactoring
  igb: add IPv6 extended headers traffic detection
  igb: packet-split descriptors support
  e1000e: rename e1000e_ba_state and e1000e_write_hdr_to_rx_buffers

 hw/net/e1000e_core.c     |  78 +++--
 hw/net/igb_core.c        | 730 ++++++++++++++++++++++++++++-----------
 hw/net/igb_regs.h        |  20 +-
 hw/net/trace-events      |   6 +-
 tests/qtest/libqos/igb.c |   5 +
 5 files changed, 592 insertions(+), 247 deletions(-)

-- 
2.25.1
Re: [PATCH v10 0/7] igb: packet-split descriptors support
Posted by Akihiko Odaki 11 months ago
On 2023/05/29 23:01, Tomasz Dzieciol wrote:
> Purposes of this series of patches:
> * introduce packet-split RX descriptors support. This feature is used by Linux
>    VF driver for MTU values from 2048.
> * refactor RX descriptor handling for introduction of packet-split RX
>    descriptors support
> * fix descriptors flags handling
> 
> Tomasz Dzieciol (7):
>    igb: remove TCP ACK detection
>    igb: rename E1000E_RingInfo_st
>    igb: RX descriptors guest writting refactoring
>    igb: RX payload guest writting refactoring
>    igb: add IPv6 extended headers traffic detection
>    igb: packet-split descriptors support
>    e1000e: rename e1000e_ba_state and e1000e_write_hdr_to_rx_buffers
> 
>   hw/net/e1000e_core.c     |  78 +++--
>   hw/net/igb_core.c        | 730 ++++++++++++++++++++++++++++-----------
>   hw/net/igb_regs.h        |  20 +-
>   hw/net/trace-events      |   6 +-
>   tests/qtest/libqos/igb.c |   5 +
>   5 files changed, 592 insertions(+), 247 deletions(-)
> 

Thanks for keeping working on this. For the entire series:

Reviewed-by: Akihiko Odaki <akihiko.odaki@daynix.com>
Tested-by: Akihiko Odaki <akihiko.odaki@daynix.com>
RE: [PATCH v10 0/7] igb: packet-split descriptors support
Posted by Tomasz Dzieciol/VIM Integration (NC) /SRPOL/Engineer/Samsung Electronics 8 months, 3 weeks ago
Hi,

It's been a while since review was done and nothing happened with those patches since then.

As I understand from guide: https://www.qemu.org/docs/master/devel/submitting-a-patch.html#is-my-patch-in I should wait. Is that correct?

-----Original Message-----
From: Akihiko Odaki <akihiko.odaki@daynix.com> 
Sent: wtorek, 30 maja 2023 04:49
To: Tomasz Dzieciol <t.dzieciol@partner.samsung.com>; qemu-devel@nongnu.org
Cc: sriram.yagnaraman@est.tech; jasowang@redhat.com; k.kwiecien@samsung.com; m.sochacki@samsung.com
Subject: Re: [PATCH v10 0/7] igb: packet-split descriptors support

On 2023/05/29 23:01, Tomasz Dzieciol wrote:
> Purposes of this series of patches:
> * introduce packet-split RX descriptors support. This feature is used by Linux
>    VF driver for MTU values from 2048.
> * refactor RX descriptor handling for introduction of packet-split RX
>    descriptors support
> * fix descriptors flags handling
> 
> Tomasz Dzieciol (7):
>    igb: remove TCP ACK detection
>    igb: rename E1000E_RingInfo_st
>    igb: RX descriptors guest writting refactoring
>    igb: RX payload guest writting refactoring
>    igb: add IPv6 extended headers traffic detection
>    igb: packet-split descriptors support
>    e1000e: rename e1000e_ba_state and e1000e_write_hdr_to_rx_buffers
> 
>   hw/net/e1000e_core.c     |  78 +++--
>   hw/net/igb_core.c        | 730 ++++++++++++++++++++++++++++-----------
>   hw/net/igb_regs.h        |  20 +-
>   hw/net/trace-events      |   6 +-
>   tests/qtest/libqos/igb.c |   5 +
>   5 files changed, 592 insertions(+), 247 deletions(-)
> 

Thanks for keeping working on this. For the entire series:

Reviewed-by: Akihiko Odaki <akihiko.odaki@daynix.com>
Tested-by: Akihiko Odaki <akihiko.odaki@daynix.com>
Re: [PATCH v10 0/7] igb: packet-split descriptors support
Posted by Jason Wang 8 months, 3 weeks ago
On Mon, Aug 7, 2023 at 10:52 PM Tomasz Dzieciol/VIM Integration (NC)
/SRPOL/Engineer/Samsung Electronics <t.dzieciol@partner.samsung.com>
wrote:
>
> Hi,
>
> It's been a while since review was done and nothing happened with those patches since then.
>
> As I understand from guide: https://www.qemu.org/docs/master/devel/submitting-a-patch.html#is-my-patch-in I should wait. Is that correct?

I've queued this for 8.2.

Thanks

>
> -----Original Message-----
> From: Akihiko Odaki <akihiko.odaki@daynix.com>
> Sent: wtorek, 30 maja 2023 04:49
> To: Tomasz Dzieciol <t.dzieciol@partner.samsung.com>; qemu-devel@nongnu.org
> Cc: sriram.yagnaraman@est.tech; jasowang@redhat.com; k.kwiecien@samsung.com; m.sochacki@samsung.com
> Subject: Re: [PATCH v10 0/7] igb: packet-split descriptors support
>
> On 2023/05/29 23:01, Tomasz Dzieciol wrote:
> > Purposes of this series of patches:
> > * introduce packet-split RX descriptors support. This feature is used by Linux
> >    VF driver for MTU values from 2048.
> > * refactor RX descriptor handling for introduction of packet-split RX
> >    descriptors support
> > * fix descriptors flags handling
> >
> > Tomasz Dzieciol (7):
> >    igb: remove TCP ACK detection
> >    igb: rename E1000E_RingInfo_st
> >    igb: RX descriptors guest writting refactoring
> >    igb: RX payload guest writting refactoring
> >    igb: add IPv6 extended headers traffic detection
> >    igb: packet-split descriptors support
> >    e1000e: rename e1000e_ba_state and e1000e_write_hdr_to_rx_buffers
> >
> >   hw/net/e1000e_core.c     |  78 +++--
> >   hw/net/igb_core.c        | 730 ++++++++++++++++++++++++++++-----------
> >   hw/net/igb_regs.h        |  20 +-
> >   hw/net/trace-events      |   6 +-
> >   tests/qtest/libqos/igb.c |   5 +
> >   5 files changed, 592 insertions(+), 247 deletions(-)
> >
>
> Thanks for keeping working on this. For the entire series:
>
> Reviewed-by: Akihiko Odaki <akihiko.odaki@daynix.com>
> Tested-by: Akihiko Odaki <akihiko.odaki@daynix.com>
>