[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [PATCH 29/40] igb: Implement MSI-X single vector mode
From: |
Sriram Yagnaraman |
Subject: |
RE: [PATCH 29/40] igb: Implement MSI-X single vector mode |
Date: |
Sat, 15 Apr 2023 19:12:04 +0000 |
> -----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 29/40] igb: Implement MSI-X single vector mode
>
> Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
> ---
> hw/net/igb_core.c | 9 +++++++--
> 1 file changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/hw/net/igb_core.c b/hw/net/igb_core.c index
> 429b0ebc03..2013a9a53d 100644
> --- a/hw/net/igb_core.c
> +++ b/hw/net/igb_core.c
> @@ -1870,7 +1870,7 @@ igb_update_interrupt_state(IGBCore *core)
>
> icr = core->mac[ICR] & core->mac[IMS];
>
> - if (msix_enabled(core->owner)) {
> + if (core->mac[GPIE] & E1000_GPIE_MSIX_MODE) {
> if (icr) {
> causes = 0;
> if (icr & E1000_ICR_DRSTA) { @@ -1905,7 +1905,12 @@
> igb_update_interrupt_state(IGBCore *core)
> trace_e1000e_irq_pending_interrupts(core->mac[ICR] & core->mac[IMS],
> core->mac[ICR], core->mac[IMS]);
>
> - if (msi_enabled(core->owner)) {
> + if (msix_enabled(core->owner)) {
> + if (icr) {
> + trace_e1000e_irq_msix_notify_vec(0);
> + msix_notify(core->owner, 0);
> + }
> + } else if (msi_enabled(core->owner)) {
> if (icr) {
> msi_notify(core->owner, 0);
> }
> --
> 2.40.0
Reviewed-by: Sriram Yagnaraman <sriram.yagnaraman@est.tech>
- RE: [PATCH 23/40] igb: Share common VF constants, (continued)
- [PATCH 21/40] vmxnet3: Reset packet state after emptying Tx queue, Akihiko Odaki, 2023/04/14
- [PATCH 24/40] igb: Fix igb_mac_reg_init alignment, Akihiko Odaki, 2023/04/14
- [PATCH 25/40] net/eth: Use void pointers, Akihiko Odaki, 2023/04/14
- [PATCH 26/40] net/eth: Always add VLAN tag, Akihiko Odaki, 2023/04/14
- [PATCH 28/40] tests/qtest/libqos/igb: Set GPIE.Multiple_MSIX, Akihiko Odaki, 2023/04/14
- [PATCH 27/40] hw/net/net_rx_pkt: Enforce alignment for eth_header, Akihiko Odaki, 2023/04/14
- [PATCH 29/40] igb: Implement MSI-X single vector mode, Akihiko Odaki, 2023/04/14
- RE: [PATCH 29/40] igb: Implement MSI-X single vector mode,
Sriram Yagnaraman <=
- [PATCH 30/40] igb: Implement igb-specific oversize check, Akihiko Odaki, 2023/04/14
- [PATCH 31/40] igb: Use UDP for RSS hash, Akihiko Odaki, 2023/04/14
- [PATCH 38/40] vmxnet3: Do not depend on PC, Akihiko Odaki, 2023/04/14
- [PATCH 37/40] igb: Implement Tx timestamp, Akihiko Odaki, 2023/04/14
- [PATCH 33/40] igb: Implement Tx SCTP CSO, Akihiko Odaki, 2023/04/14