qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 1/2] net: Provide MemReentrancyGuard * to qemu_new_nic()


From: Akihiko Odaki
Subject: Re: [PATCH v2 1/2] net: Provide MemReentrancyGuard * to qemu_new_nic()
Date: Mon, 5 Jun 2023 19:50:21 +0900
User-agent: Mozilla/5.0 (X11; Linux aarch64; rv:102.0) Gecko/20100101 Thunderbird/102.11.0

On 2023/06/05 17:06, Alexander Bulekov wrote:
On 230601 1218, Akihiko Odaki wrote:
Recently MemReentrancyGuard was added to DeviceState to record that the
device is engaging in I/O. The network device backend needs to update it
when delivering a packet to a device.

In preparation for such a change, add MemReentrancyGuard * as a
parameter of qemu_new_nic().

Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>


Reviewed-by: Alexander Bulekov <alxndr@bu.edu>

One minor comment below.

---
  include/net/net.h             | 1 +
  hw/net/allwinner-sun8i-emac.c | 3 ++-
  hw/net/allwinner_emac.c       | 3 ++-
  hw/net/cadence_gem.c          | 3 ++-
  hw/net/dp8393x.c              | 3 ++-
  hw/net/e1000.c                | 3 ++-
  hw/net/e1000e.c               | 2 +-
  hw/net/eepro100.c             | 4 +++-
  hw/net/etraxfs_eth.c          | 3 ++-
  hw/net/fsl_etsec/etsec.c      | 3 ++-
  hw/net/ftgmac100.c            | 3 ++-
  hw/net/i82596.c               | 2 +-
  hw/net/igb.c                  | 2 +-
  hw/net/imx_fec.c              | 2 +-
  hw/net/lan9118.c              | 3 ++-
  hw/net/mcf_fec.c              | 3 ++-
  hw/net/mipsnet.c              | 3 ++-
  hw/net/msf2-emac.c            | 3 ++-
  hw/net/mv88w8618_eth.c        | 3 ++-
  hw/net/ne2000-isa.c           | 3 ++-
  hw/net/ne2000-pci.c           | 3 ++-
  hw/net/npcm7xx_emc.c          | 3 ++-
  hw/net/opencores_eth.c        | 3 ++-
  hw/net/pcnet.c                | 3 ++-
  hw/net/rocker/rocker_fp.c     | 4 ++--
  hw/net/rtl8139.c              | 3 ++-
  hw/net/smc91c111.c            | 3 ++-
  hw/net/spapr_llan.c           | 3 ++-
  hw/net/stellaris_enet.c       | 3 ++-
  hw/net/sungem.c               | 2 +-
  hw/net/sunhme.c               | 3 ++-
  hw/net/tulip.c                | 3 ++-
  hw/net/virtio-net.c           | 6 ++++--
  hw/net/vmxnet3.c              | 2 +-
  hw/net/xen_nic.c              | 4 ++--
  hw/net/xgmac.c                | 3 ++-
  hw/net/xilinx_axienet.c       | 3 ++-
  hw/net/xilinx_ethlite.c       | 3 ++-
  hw/usb/dev-network.c          | 3 ++-
  net/net.c                     | 1 +
  40 files changed, 75 insertions(+), 41 deletions(-)

diff --git a/include/net/net.h b/include/net/net.h
index 1448d00afb..a7d8deaccb 100644
--- a/include/net/net.h
+++ b/include/net/net.h
@@ -157,6 +157,7 @@ NICState *qemu_new_nic(NetClientInfo *info,
                         NICConf *conf,
                         const char *model,
                         const char *name,
+                       MemReentrancyGuard *reentrancy_guard,
                         void *opaque);

Does it make sense to roll *reentrancy_guard into NICConf here?

Probably no. All members of NICConf are device properties the user configure, but reentrancy_guard is an internal state. Similarly, the opaque parameter is an internal state and not included in NICConf.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]