RE: [PATCH 14/40] e1000x: Share more Rx filtering logic

[Sriram Yagnaraman]

> -----Original Message-----
> From: Akihiko Odaki <akihiko.odaki@daynix.com>
> Sent: Friday, 14 April 2023 13:37
> Cc: Sriram Yagnaraman <sriram.yagnaraman@est.tech>; Jason Wang
> <jasowang@redhat.com>; Dmitry Fleytman <dmitry.fleytman@gmail.com>;
> Michael S. Tsirkin <mst@redhat.com>; Alex Bennée <alex.bennee@linaro.org>;
> Philippe Mathieu-Daudé <philmd@linaro.org>; Thomas Huth
> <thuth@redhat.com>; Wainer dos Santos Moschetta
> <wainersm@redhat.com>; Beraldo Leal <bleal@redhat.com>; Cleber Rosa
> <crosa@redhat.com>; Laurent Vivier <lvivier@redhat.com>; Paolo Bonzini
> <pbonzini@redhat.com>; qemu-devel@nongnu.org; Akihiko Odaki
> <akihiko.odaki@daynix.com>
> Subject: [PATCH 14/40] e1000x: Share more Rx filtering logic
> 
> This saves some code and enables tracepoint for e1000's VLAN filtering.
> 
> Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
> ---
>  hw/net/e1000.c         | 35 +++++--------------------------
>  hw/net/e1000e_core.c   | 47 +++++-------------------------------------
>  hw/net/e1000x_common.c | 44 +++++++++++++++++++++++++++++++++----
> --
>  hw/net/e1000x_common.h |  4 +++-
>  hw/net/igb_core.c      | 41 +++---------------------------------
>  hw/net/trace-events    |  4 ++--
>  6 files changed, 56 insertions(+), 119 deletions(-)

Reviewed-by: Sriram Yagnaraman <sriram.yagnaraman@est.tech>