Series comparison

-[PULL 00/23] Net patches
+[PULL V2 0/3] Net patches
-The following changes since commit e0175b71638cf4398903c0d25f93fe62e0606389:
+The following changes since commit d90f154867ec0ec22fd719164b88716e8fd48672:
-  Merge remote-tracking branch 'remotes/pmaydell/tags/pull-target-arm-20200228' into staging (2020-02-28 16:39:27 +0000)
+  Merge remote-tracking branch 'remotes/dg-gitlab/tags/ppc-for-6.1-20210504' into staging (2021-05-05 20:29:14 +0100)
 are available in the git repository at:
   https://github.com/jasowang/qemu.git tags/net-pull-request
-for you to fetch changes up to 41aa2e3f9b27fd259a13711545d933a20f1d2f16:
+for you to fetch changes up to 4f8a39494aded9f2026a26b137378ea2ee3d5338:
-  l2tpv3: fix RFC number typo in qemu-options.hx (2020-03-02 15:30:08 +0800)
+  tap-bsd: Remove special casing for older OpenBSD releases (2021-05-27 11:03:55 +0800)
 ----------------------------------------------------------------
 ----------------------------------------------------------------
-Bin Meng (1):
+Brad Smith (1):
-      hw: net: cadence_gem: Fix build errors in DB_PRINT()
+      tap-bsd: Remove special casing for older OpenBSD releases
-Finn Thain (14):
+Guenter Roeck (1):
-      dp8393x: Mask EOL bit from descriptor addresses
+      hw/net/imx_fec: return 0xffff when accessing non-existing PHY
       dp8393x: Always use 32-bit accesses
       dp8393x: Clean up endianness hacks
       dp8393x: Have dp8393x_receive() return the packet size
       dp8393x: Update LLFA and CRDA registers from rx descriptor
       dp8393x: Clear RRRA command register bit only when appropriate
       dp8393x: Implement packet size limit and RBAE interrupt
       dp8393x: Don't clobber packet checksum
       dp8393x: Use long-word-aligned RRA pointers in 32-bit mode
       dp8393x: Pad frames to word or long word boundary
       dp8393x: Clear descriptor in_use field to release packet
       dp8393x: Always update RRA pointers and sequence numbers
       dp8393x: Don't reset Silicon Revision register
       dp8393x: Don't stop reception upon RBE interrupt assertion
-Lukas Straub (4):
+Laurent Vivier (1):
-      block/replication.c: Ignore requests after failover
+      virtio-net: failover: add missing remove_migration_state_change_notifier()
       tests/test-replication.c: Add test for for secondary node continuing replication
       net/filter.c: Add Options to insert filters anywhere in the filter list
       colo: Update Documentation for continuous replication
-Stefan Hajnoczi (1):
+ hw/net/imx_fec.c    | 8 +++-----
-      l2tpv3: fix RFC number typo in qemu-options.hx
+ hw/net/trace-events | 2 ++
+ hw/net/virtio-net.c | 1 +
-Yuri Benditovich (3):
+ net/tap-bsd.c       | 8 --------
-      e1000e: Avoid hw_error if legacy mode used
+files changed, 6 insertions(+), 13 deletions(-)
       NetRxPkt: Introduce support for additional hash types
       NetRxPkt: fix hash calculation of IPV6 TCP
  block/replication.c        |  35 ++++++-
  docs/COLO-FT.txt           | 224 +++++++++++++++++++++++++++++++++------------
  docs/block-replication.txt |  28 ++++--
  hw/net/cadence_gem.c       |  11 ++-
  hw/net/dp8393x.c           | 200 ++++++++++++++++++++++++++--------------
  hw/net/e1000e_core.c       |  15 +--
  hw/net/net_rx_pkt.c        |  44 ++++++++-
  hw/net/net_rx_pkt.h        |   6 +-
  hw/net/trace-events        |   4 +
  include/net/filter.h       |   2 +
  net/filter.c               |  92 ++++++++++++++++++-
  qemu-options.hx            |  35 +++++--
  tests/test-replication.c   |  52 +++++++++++
 files changed, 591 insertions(+), 157 deletions(-)

-[PULL 01/23] dp8393x: Mask EOL bit from descriptor addresses
+Deleted patch
-From: Finn Thain <fthain@telegraphics.com.au>
-The Least Significant bit of a descriptor address register is used as
-an EOL flag. It has to be masked when the register value is to be used
-as an actual address for copying memory around. But when the registers
-are to be updated the EOL bit should not be masked.
-Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
-Tested-by: Laurent Vivier <laurent@vivier.eu>
----
- hw/net/dp8393x.c | 17 +++++++++++------
-file changed, 11 insertions(+), 6 deletions(-)
-diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
-index XXXXXXX..XXXXXXX 100644
---- a/hw/net/dp8393x.c
-+++ b/hw/net/dp8393x.c
-@@ -XXX,XX +XXX,XX @@ do { printf("sonic ERROR: %s: " fmt, __func__ , ## __VA_ARGS__); } while (0)
- #define SONIC_ISR_PINT   0x0800
- #define SONIC_ISR_LCD    0x1000
-+#define SONIC_DESC_EOL   0x0001
-+#define SONIC_DESC_ADDR  0xFFFE
-+
- #define TYPE_DP8393X "dp8393x"
- #define DP8393X(obj) OBJECT_CHECK(dp8393xState, (obj), TYPE_DP8393X)
-@@ -XXX,XX +XXX,XX @@ static uint32_t dp8393x_crba(dp8393xState *s)
- static uint32_t dp8393x_crda(dp8393xState *s)
- {
--    return (s->regs[SONIC_URDA] << 16) | s->regs[SONIC_CRDA];
-+    return (s->regs[SONIC_URDA] << 16) |
-+           (s->regs[SONIC_CRDA] & SONIC_DESC_ADDR);
- }
- static uint32_t dp8393x_rbwc(dp8393xState *s)
-@@ -XXX,XX +XXX,XX @@ static uint32_t dp8393x_tsa(dp8393xState *s)
- static uint32_t dp8393x_ttda(dp8393xState *s)
- {
--    return (s->regs[SONIC_UTDA] << 16) | s->regs[SONIC_TTDA];
-+    return (s->regs[SONIC_UTDA] << 16) |
-+           (s->regs[SONIC_TTDA] & SONIC_DESC_ADDR);
- }
- static uint32_t dp8393x_wt(dp8393xState *s)
-@@ -XXX,XX +XXX,XX @@ static void dp8393x_do_transmit_packets(dp8393xState *s)
-                                MEMTXATTRS_UNSPECIFIED, s->data,
-                                size);
-             s->regs[SONIC_CTDA] = dp8393x_get(s, width, 0) & ~0x1;
--            if (dp8393x_get(s, width, 0) & 0x1) {
-+            if (dp8393x_get(s, width, 0) & SONIC_DESC_EOL) {
-                 /* EOL detected */
-                 break;
-             }
-@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
-     /* XXX: Check byte ordering */
-     /* Check for EOL */
--    if (s->regs[SONIC_LLFA] & 0x1) {
-+    if (s->regs[SONIC_LLFA] & SONIC_DESC_EOL) {
-         /* Are we still in resource exhaustion? */
-         size = sizeof(uint16_t) * 1 * width;
-         address = dp8393x_crda(s) + sizeof(uint16_t) * 5 * width;
-         address_space_read(&s->as, address, MEMTXATTRS_UNSPECIFIED,
-                            s->data, size);
--        if (dp8393x_get(s, width, 0) & 0x1) {
-+        if (dp8393x_get(s, width, 0) & SONIC_DESC_EOL) {
-             /* Still EOL ; stop reception */
-             return -1;
-         } else {
-@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
-                        dp8393x_crda(s) + sizeof(uint16_t) * 5 * width,
-                        MEMTXATTRS_UNSPECIFIED, s->data, size);
-     s->regs[SONIC_LLFA] = dp8393x_get(s, width, 0);
--    if (s->regs[SONIC_LLFA] & 0x1) {
-+    if (s->regs[SONIC_LLFA] & SONIC_DESC_EOL) {
-         /* EOL detected */
-         s->regs[SONIC_ISR] |= SONIC_ISR_RDE;
-     } else {
---
-.5.0

-[PULL 02/23] dp8393x: Always use 32-bit accesses
+Deleted patch
-From: Finn Thain <fthain@telegraphics.com.au>
-The DP83932 and DP83934 have 32 data lines. The datasheet says,
-    Data Bus: These bidirectional lines are used to transfer data on the
-    system bus. When the SONIC is a bus master, 16-bit data is transferred
-    on D15-D0 and 32-bit data is transferred on D31-D0. When the SONIC is
-    accessed as a slave, register data is driven onto lines D15-D0.
-    D31-D16 are held TRI-STATE if SONIC is in 16-bit mode. If SONIC is in
--bit mode, they are driven, but invalid.
-Always use 32-bit accesses both as bus master and bus slave.
-Force the MSW to zero in bus master mode.
-This gets the Linux 'jazzsonic' driver working, and avoids the need for
-prior hacks to make the NetBSD 'sn' driver work.
-Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
-Tested-by: Laurent Vivier <laurent@vivier.eu>
-Signed-off-by: Jason Wang <jasowang@redhat.com>
----
- hw/net/dp8393x.c | 47 +++++++++++++++++++++++++++++------------------
-file changed, 29 insertions(+), 18 deletions(-)
-diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
-index XXXXXXX..XXXXXXX 100644
---- a/hw/net/dp8393x.c
-+++ b/hw/net/dp8393x.c
-@@ -XXX,XX +XXX,XX @@ static void dp8393x_put(dp8393xState *s, int width, int offset,
-                         uint16_t val)
- {
-     if (s->big_endian) {
--        s->data[offset * width + width - 1] = cpu_to_be16(val);
-+        if (width == 2) {
-+            s->data[offset * 2] = 0;
-+            s->data[offset * 2 + 1] = cpu_to_be16(val);
-+        } else {
-+            s->data[offset] = cpu_to_be16(val);
-+        }
-     } else {
--        s->data[offset * width] = cpu_to_le16(val);
-+        if (width == 2) {
-+            s->data[offset * 2] = cpu_to_le16(val);
-+            s->data[offset * 2 + 1] = 0;
-+        } else {
-+            s->data[offset] = cpu_to_le16(val);
-+        }
-     }
- }
-@@ -XXX,XX +XXX,XX @@ static uint64_t dp8393x_read(void *opaque, hwaddr addr, unsigned int size)
-     DPRINTF("read 0x%04x from reg %s\n", val, reg_names[reg]);
--    return val;
-+    return s->big_endian ? val << 16 : val;
- }
- static void dp8393x_write(void *opaque, hwaddr addr, uint64_t data,
-@@ -XXX,XX +XXX,XX @@ static void dp8393x_write(void *opaque, hwaddr addr, uint64_t data,
- {
-     dp8393xState *s = opaque;
-     int reg = addr >> s->it_shift;
-+    uint32_t val = s->big_endian ? data >> 16 : data;
--    DPRINTF("write 0x%04x to reg %s\n", (uint16_t)data, reg_names[reg]);
-+    DPRINTF("write 0x%04x to reg %s\n", (uint16_t)val, reg_names[reg]);
-     switch (reg) {
-         /* Command register */
-         case SONIC_CR:
--            dp8393x_do_command(s, data);
-+            dp8393x_do_command(s, val);
-             break;
-         /* Prevent write to read-only registers */
-         case SONIC_CAP2:
-@@ -XXX,XX +XXX,XX @@ static void dp8393x_write(void *opaque, hwaddr addr, uint64_t data,
-         /* Accept write to some registers only when in reset mode */
-         case SONIC_DCR:
-             if (s->regs[SONIC_CR] & SONIC_CR_RST) {
--                s->regs[reg] = data & 0xbfff;
-+                s->regs[reg] = val & 0xbfff;
-             } else {
-                 DPRINTF("writing to DCR invalid\n");
-             }
-             break;
-         case SONIC_DCR2:
-             if (s->regs[SONIC_CR] & SONIC_CR_RST) {
--                s->regs[reg] = data & 0xf017;
-+                s->regs[reg] = val & 0xf017;
-             } else {
-                 DPRINTF("writing to DCR2 invalid\n");
-             }
-             break;
-         /* 12 lower bytes are Read Only */
-         case SONIC_TCR:
--            s->regs[reg] = data & 0xf000;
-+            s->regs[reg] = val & 0xf000;
-             break;
-         /* 9 lower bytes are Read Only */
-         case SONIC_RCR:
--            s->regs[reg] = data & 0xffe0;
-+            s->regs[reg] = val & 0xffe0;
-             break;
-         /* Ignore most significant bit */
-         case SONIC_IMR:
--            s->regs[reg] = data & 0x7fff;
-+            s->regs[reg] = val & 0x7fff;
-             dp8393x_update_irq(s);
-             break;
-         /* Clear bits by writing 1 to them */
-         case SONIC_ISR:
--            data &= s->regs[reg];
--            s->regs[reg] &= ~data;
--            if (data & SONIC_ISR_RBE) {
-+            val &= s->regs[reg];
-+            s->regs[reg] &= ~val;
-+            if (val & SONIC_ISR_RBE) {
-                 dp8393x_do_read_rra(s);
-             }
-             dp8393x_update_irq(s);
-@@ -XXX,XX +XXX,XX @@ static void dp8393x_write(void *opaque, hwaddr addr, uint64_t data,
-         case SONIC_REA:
-         case SONIC_RRP:
-         case SONIC_RWP:
--            s->regs[reg] = data & 0xfffe;
-+            s->regs[reg] = val & 0xfffe;
-             break;
-         /* Invert written value for some registers */
-         case SONIC_CRCT:
-         case SONIC_FAET:
-         case SONIC_MPT:
--            s->regs[reg] = data ^ 0xffff;
-+            s->regs[reg] = val ^ 0xffff;
-             break;
-         /* All other registers have no special contrainst */
-         default:
--            s->regs[reg] = data;
-+            s->regs[reg] = val;
-     }
-     if (reg == SONIC_WT0 || reg == SONIC_WT1) {
-@@ -XXX,XX +XXX,XX @@ static void dp8393x_write(void *opaque, hwaddr addr, uint64_t data,
- static const MemoryRegionOps dp8393x_ops = {
-     .read = dp8393x_read,
-     .write = dp8393x_write,
--    .impl.min_access_size = 2,
--    .impl.max_access_size = 2,
-+    .impl.min_access_size = 4,
-+    .impl.max_access_size = 4,
-     .endianness = DEVICE_NATIVE_ENDIAN,
- };
---
-.5.0

-[PULL 16/23] NetRxPkt: Introduce support for additional hash types
+[PULL V2 1/3] hw/net/imx_fec: return 0xffff when accessing non-existing PHY
-From: Yuri Benditovich <yuri.benditovich@daynix.com>
+From: Guenter Roeck <linux@roeck-us.net>
-Add support for following hash types:
+If a PHY does not exist, attempts to read from it should return 0xffff.
-IPV6 TCP with extension headers
+Otherwise the Linux kernel will believe that a PHY is there and select
-IPV4 UDP
+the non-existing PHY. This in turn will result in network errors later
-IPV6 UDP
+on since the real PHY is not selected or configured.
 IPV6 UDP with extension headers
-Signed-off-by: Yuri Benditovich <yuri.benditovich@daynix.com>
+Since reading from or writing to a non-existing PHY is not an emulation
-Acked-by: Dmitry Fleytman <dmitry.fleytman@gmail.com>
+error, replace guest error messages with traces.
 Fixes: 461c51ad4275 ("Add a phy-num property to the i.MX FEC emulator")
 Cc: Jean-Christophe Dubois <jcd@tribudubois.net>
 Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
 Tested-by: Bin Meng <bmeng.cn@gmail.com>
 Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
 Signed-off-by: Guenter Roeck <linux@roeck-us.net>
 Signed-off-by: Jason Wang <jasowang@redhat.com>
 ---
- hw/net/net_rx_pkt.c | 42 ++++++++++++++++++++++++++++++++++++++++++
+ hw/net/imx_fec.c    | 8 +++-----
- hw/net/net_rx_pkt.h |  6 +++++-
+ hw/net/trace-events | 2 ++
- hw/net/trace-events |  4 ++++
+files changed, 5 insertions(+), 5 deletions(-)
 files changed, 51 insertions(+), 1 deletion(-)
-diff --git a/hw/net/net_rx_pkt.c b/hw/net/net_rx_pkt.c
+diff --git a/hw/net/imx_fec.c b/hw/net/imx_fec.c
 index XXXXXXX..XXXXXXX 100644
---- a/hw/net/net_rx_pkt.c
+--- a/hw/net/imx_fec.c
-+++ b/hw/net/net_rx_pkt.c
++++ b/hw/net/imx_fec.c
-@@ -XXX,XX +XXX,XX @@ _net_rx_rss_prepare_tcp(uint8_t *rss_input,
+@@ -XXX,XX +XXX,XX @@ static uint32_t imx_phy_read(IMXFECState *s, int reg)
-                           &tcphdr->th_dport, sizeof(uint16_t));
+     uint32_t phy = reg / 32;
- }
+     if (phy != s->phy_num) {
-+static inline void
+-        qemu_log_mask(LOG_GUEST_ERROR, "[%s.phy]%s: Bad phy num %u\n",
-+_net_rx_rss_prepare_udp(uint8_t *rss_input,
+-                      TYPE_IMX_FEC, __func__, phy);
-+                        struct NetRxPkt *pkt,
+-        return 0;
-+                        size_t *bytes_written)
++        trace_imx_phy_read_num(phy, s->phy_num);
-+{
++        return 0xffff;
-+    struct udp_header *udphdr = &pkt->l4hdr_info.hdr.udp;
+     }
-+
-+    _net_rx_rss_add_chunk(rss_input, bytes_written,
+     reg %= 32;
-+                          &udphdr->uh_sport, sizeof(uint16_t));
+@@ -XXX,XX +XXX,XX @@ static void imx_phy_write(IMXFECState *s, int reg, uint32_t val)
-+
+     uint32_t phy = reg / 32;
-+    _net_rx_rss_add_chunk(rss_input, bytes_written,
-+                          &udphdr->uh_dport, sizeof(uint16_t));
+     if (phy != s->phy_num) {
-+}
+-        qemu_log_mask(LOG_GUEST_ERROR, "[%s.phy]%s: Bad phy num %u\n",
-+
+-                      TYPE_IMX_FEC, __func__, phy);
- uint32_t
++        trace_imx_phy_write_num(phy, s->phy_num);
- net_rx_pkt_calc_rss_hash(struct NetRxPkt *pkt,
+         return;
-                          NetRxPktRssType type,
+     }
-@@ -XXX,XX +XXX,XX @@ net_rx_pkt_calc_rss_hash(struct NetRxPkt *pkt,
          trace_net_rx_pkt_rss_ip6_ex();
          _net_rx_rss_prepare_ip6(&rss_input[0], pkt, true, &rss_length);
          break;
 +    case NetPktRssIpV6TcpEx:
 +        assert(pkt->isip6);
 +        assert(pkt->istcp);
 +        trace_net_rx_pkt_rss_ip6_ex_tcp();
 +        _net_rx_rss_prepare_ip6(&rss_input[0], pkt, true, &rss_length);
 +        _net_rx_rss_prepare_tcp(&rss_input[0], pkt, &rss_length);
 +        break;
 +    case NetPktRssIpV4Udp:
 +        assert(pkt->isip4);
 +        assert(pkt->isudp);
 +        trace_net_rx_pkt_rss_ip4_udp();
 +        _net_rx_rss_prepare_ip4(&rss_input[0], pkt, &rss_length);
 +        _net_rx_rss_prepare_udp(&rss_input[0], pkt, &rss_length);
 +        break;
 +    case NetPktRssIpV6Udp:
 +        assert(pkt->isip6);
 +        assert(pkt->isudp);
 +        trace_net_rx_pkt_rss_ip6_udp();
 +        _net_rx_rss_prepare_ip6(&rss_input[0], pkt, false, &rss_length);
 +        _net_rx_rss_prepare_udp(&rss_input[0], pkt, &rss_length);
 +        break;
 +    case NetPktRssIpV6UdpEx:
 +        assert(pkt->isip6);
 +        assert(pkt->isudp);
 +        trace_net_rx_pkt_rss_ip6_ex_udp();
 +        _net_rx_rss_prepare_ip6(&rss_input[0], pkt, true, &rss_length);
 +        _net_rx_rss_prepare_udp(&rss_input[0], pkt, &rss_length);
 +        break;
      default:
          assert(false);
          break;
 diff --git a/hw/net/net_rx_pkt.h b/hw/net/net_rx_pkt.h
 index XXXXXXX..XXXXXXX 100644
 --- a/hw/net/net_rx_pkt.h
 +++ b/hw/net/net_rx_pkt.h
@@ -XXX,XX +XXX,XX @@ typedef enum {
      NetPktRssIpV4Tcp,
      NetPktRssIpV6Tcp,
      NetPktRssIpV6,
 -    NetPktRssIpV6Ex
 +    NetPktRssIpV6Ex,
 +    NetPktRssIpV6TcpEx,
 +    NetPktRssIpV4Udp,
 +    NetPktRssIpV6Udp,
 +    NetPktRssIpV6UdpEx,
  } NetRxPktRssType;
  /**
 diff --git a/hw/net/trace-events b/hw/net/trace-events
 index XXXXXXX..XXXXXXX 100644
 --- a/hw/net/trace-events
 +++ b/hw/net/trace-events
-@@ -XXX,XX +XXX,XX @@ net_rx_pkt_l3_csum_validate_csum(size_t l3hdr_off, uint32_t csl, uint32_t cntr,
+@@ -XXX,XX +XXX,XX @@ i82596_channel_attention(void *s) "%p: Received CHANNEL ATTENTION"
- net_rx_pkt_rss_ip4(void) "Calculating IPv4 RSS  hash"
+ # imx_fec.c
- net_rx_pkt_rss_ip4_tcp(void) "Calculating IPv4/TCP RSS  hash"
+ imx_phy_read(uint32_t val, int phy, int reg) "0x%04"PRIx32" <= phy[%d].reg[%d]"
-+net_rx_pkt_rss_ip4_udp(void) "Calculating IPv4/UDP RSS  hash"
++imx_phy_read_num(int phy, int configured) "read request from unconfigured phy %d (configured %d)"
- net_rx_pkt_rss_ip6_tcp(void) "Calculating IPv6/TCP RSS  hash"
+ imx_phy_write(uint32_t val, int phy, int reg) "0x%04"PRIx32" => phy[%d].reg[%d]"
-+net_rx_pkt_rss_ip6_udp(void) "Calculating IPv6/UDP RSS  hash"
++imx_phy_write_num(int phy, int configured) "write request to unconfigured phy %d (configured %d)"
- net_rx_pkt_rss_ip6(void) "Calculating IPv6 RSS  hash"
+ imx_phy_update_link(const char *s) "%s"
- net_rx_pkt_rss_ip6_ex(void) "Calculating IPv6/EX RSS  hash"
+ imx_phy_reset(void) ""
-+net_rx_pkt_rss_ip6_ex_tcp(void) "Calculating IPv6/EX/TCP RSS  hash"
+ imx_fec_read_bd(uint64_t addr, int flags, int len, int data) "tx_bd 0x%"PRIx64" flags 0x%04x len %d data 0x%08x"
 +net_rx_pkt_rss_ip6_ex_udp(void) "Calculating IPv6/EX/UDP RSS  hash"
  net_rx_pkt_rss_hash(size_t rss_length, uint32_t rss_hash) "RSS hash for %zu bytes: 0x%X"
  net_rx_pkt_rss_add_chunk(void* ptr, size_t size, size_t input_offset) "Add RSS chunk %p, %zu bytes, RSS input offset %zu bytes"
 --
-.5.0
+.7.4

-[PULL 03/23] dp8393x: Clean up endianness hacks
+[PULL V2 2/3] virtio-net: failover: add missing remove_migration_state_change_notifier()
-From: Finn Thain <fthain@telegraphics.com.au>
+From: Laurent Vivier <lvivier@redhat.com>
-According to the datasheet, section 3.4.4, "in 32-bit mode ... the SONIC
+In the failover case configuration, virtio_net_device_realize() uses an
-always writes long words".
+add_migration_state_change_notifier() to add a state notifier, but this
 notifier is not removed by the unrealize function when the virtio-net
 card is unplugged.
-Therefore, use the same technique for the 'in_use' field that is used
+If the card is unplugged and a migration is started, the notifier is
-everywhere else, and write the full long word.
+called and as it is not valid anymore QEMU crashes.
-Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
+This patch fixes the problem by adding the
-Tested-by: Laurent Vivier <laurent@vivier.eu>
+remove_migration_state_change_notifier() in virtio_net_device_unrealize().
-Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
 The problem can be reproduced with:
   $ qemu-system-x86_64 -enable-kvm -m 1g -M q35 \
     -device pcie-root-port,slot=4,id=root1 \
     -device pcie-root-port,slot=5,id=root2 \
     -device virtio-net-pci,id=net1,mac=52:54:00:6f:55:cc,failover=on,bus=root1 \
     -monitor stdio disk.qcow2
   (qemu) device_del net1
   (qemu) migrate "exec:gzip -c > STATEFILE.gz"
   Thread 1 "qemu-system-x86" received signal SIGSEGV, Segmentation fault.
 x0000000000000000 in ?? ()
   (gdb) bt
   #0  0x0000000000000000 in  ()
   #1  0x0000555555d726d7 in notifier_list_notify (...)
       at .../util/notify.c:39
   #2  0x0000555555842c1a in migrate_fd_connect (...)
       at .../migration/migration.c:3975
   #3  0x0000555555950f7d in migration_channel_connect (...)
       error@entry=0x0) at .../migration/channel.c:107
   #4  0x0000555555910922 in exec_start_outgoing_migration (...)
       at .../migration/exec.c:42
 Reported-by: Igor Mammedov <imammedo@redhat.com>
 Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
 Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
 Signed-off-by: Laurent Vivier <lvivier@redhat.com>
 Signed-off-by: Jason Wang <jasowang@redhat.com>
 ---
- hw/net/dp8393x.c | 17 ++++++-----------
+ hw/net/virtio-net.c | 1 +
-file changed, 6 insertions(+), 11 deletions(-)
+file changed, 1 insertion(+)
-diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
+diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
 index XXXXXXX..XXXXXXX 100644
---- a/hw/net/dp8393x.c
+--- a/hw/net/virtio-net.c
-+++ b/hw/net/dp8393x.c
++++ b/hw/net/virtio-net.c
-@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
+@@ -XXX,XX +XXX,XX @@ static void virtio_net_device_unrealize(DeviceState *dev)
-         return -1;
      if (n->failover) {
          device_listener_unregister(&n->primary_listener);
 +        remove_migration_state_change_notifier(&n->migration_state);
      }
--    /* XXX: Check byte ordering */
+     max_queues = n->multiqueue ? n->max_queues : 1;
 -
      /* Check for EOL */
      if (s->regs[SONIC_LLFA] & SONIC_DESC_EOL) {
          /* Are we still in resource exhaustion? */
@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
          /* EOL detected */
          s->regs[SONIC_ISR] |= SONIC_ISR_RDE;
      } else {
 -        /* Clear in_use, but it is always 16bit wide */
 -        int offset = dp8393x_crda(s) + sizeof(uint16_t) * 6 * width;
 -        if (s->big_endian && width == 2) {
 -            /* we need to adjust the offset of the 16bit field */
 -            offset += sizeof(uint16_t);
 -        }
 -        s->data[0] = 0;
 -        address_space_write(&s->as, offset, MEMTXATTRS_UNSPECIFIED,
 -                            s->data, sizeof(uint16_t));
 +        /* Clear in_use */
 +        size = sizeof(uint16_t) * width;
 +        address = dp8393x_crda(s) + sizeof(uint16_t) * 6 * width;
 +        dp8393x_put(s, width, 0, 0);
 +        address_space_write(&s->as, address, MEMTXATTRS_UNSPECIFIED,
 +                            s->data, size);
          s->regs[SONIC_CRDA] = s->regs[SONIC_LLFA];
          s->regs[SONIC_ISR] |= SONIC_ISR_PKTRX;
          s->regs[SONIC_RSC] = (s->regs[SONIC_RSC] & 0xff00) | (((s->regs[SONIC_RSC] & 0x00ff) + 1) & 0x00ff);
 --
-.5.0
+.7.4

-[PULL 04/23] dp8393x: Have dp8393x_receive() return the packet size
+Deleted patch
-From: Finn Thain <fthain@telegraphics.com.au>
-This function re-uses its 'size' argument as a scratch variable.
-Instead, declare a local 'size' variable for that purpose so that the
-function result doesn't get messed up.
-Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
-Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
-Tested-by: Laurent Vivier <laurent@vivier.eu>
-Signed-off-by: Jason Wang <jasowang@redhat.com>
----
- hw/net/dp8393x.c | 9 +++++----
-file changed, 5 insertions(+), 4 deletions(-)
-diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
-index XXXXXXX..XXXXXXX 100644
---- a/hw/net/dp8393x.c
-+++ b/hw/net/dp8393x.c
-@@ -XXX,XX +XXX,XX @@ static int dp8393x_receive_filter(dp8393xState *s, const uint8_t * buf,
- }
- static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
--                               size_t size)
-+                               size_t pkt_size)
- {
-     dp8393xState *s = qemu_get_nic_opaque(nc);
-     int packet_type;
-     uint32_t available, address;
--    int width, rx_len = size;
-+    int width, rx_len = pkt_size;
-     uint32_t checksum;
-+    int size;
-     width = (s->regs[SONIC_DCR] & SONIC_DCR_DW) ? 2 : 1;
-     s->regs[SONIC_RCR] &= ~(SONIC_RCR_PRX | SONIC_RCR_LBK | SONIC_RCR_FAER |
-         SONIC_RCR_CRCR | SONIC_RCR_LPKT | SONIC_RCR_BC | SONIC_RCR_MC);
--    packet_type = dp8393x_receive_filter(s, buf, size);
-+    packet_type = dp8393x_receive_filter(s, buf, pkt_size);
-     if (packet_type < 0) {
-         DPRINTF("packet not for netcard\n");
-         return -1;
-@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
-     /* Done */
-     dp8393x_update_irq(s);
--    return size;
-+    return pkt_size;
- }
- static void dp8393x_reset(DeviceState *dev)
---
-.5.0

-[PULL 05/23] dp8393x: Update LLFA and CRDA registers from rx descriptor
+Deleted patch
-From: Finn Thain <fthain@telegraphics.com.au>
-Follow the algorithm given in the National Semiconductor DP83932C
-datasheet in section 3.4.7:
-    At the next reception, the SONIC re-reads the last RXpkt.link field,
-    and updates its CRDA register to point to the next descriptor.
-The chip is designed to allow the host to provide a new list of
-descriptors in this way.
-Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
-Tested-by: Laurent Vivier <laurent@vivier.eu>
-Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
-Signed-off-by: Jason Wang <jasowang@redhat.com>
----
- hw/net/dp8393x.c | 11 +++++++----
-file changed, 7 insertions(+), 4 deletions(-)
-diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
-index XXXXXXX..XXXXXXX 100644
---- a/hw/net/dp8393x.c
-+++ b/hw/net/dp8393x.c
-@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
-         address = dp8393x_crda(s) + sizeof(uint16_t) * 5 * width;
-         address_space_read(&s->as, address, MEMTXATTRS_UNSPECIFIED,
-                            s->data, size);
--        if (dp8393x_get(s, width, 0) & SONIC_DESC_EOL) {
-+        s->regs[SONIC_LLFA] = dp8393x_get(s, width, 0);
-+        if (s->regs[SONIC_LLFA] & SONIC_DESC_EOL) {
-             /* Still EOL ; stop reception */
-             return -1;
--        } else {
--            s->regs[SONIC_CRDA] = s->regs[SONIC_LLFA];
-         }
-+        /* Link has been updated by host */
-+        s->regs[SONIC_CRDA] = s->regs[SONIC_LLFA];
-     }
-     /* Save current position */
-@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
-                         MEMTXATTRS_UNSPECIFIED,
-                         s->data, size);
--    /* Move to next descriptor */
-+    /* Check link field */
-     size = sizeof(uint16_t) * width;
-     address_space_read(&s->as,
-                        dp8393x_crda(s) + sizeof(uint16_t) * 5 * width,
-@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
-         dp8393x_put(s, width, 0, 0);
-         address_space_write(&s->as, address, MEMTXATTRS_UNSPECIFIED,
-                             s->data, size);
-+
-+        /* Move to next descriptor */
-         s->regs[SONIC_CRDA] = s->regs[SONIC_LLFA];
-         s->regs[SONIC_ISR] |= SONIC_ISR_PKTRX;
-         s->regs[SONIC_RSC] = (s->regs[SONIC_RSC] & 0xff00) | (((s->regs[SONIC_RSC] & 0x00ff) + 1) & 0x00ff);
---
-.5.0

-[PULL 06/23] dp8393x: Clear RRRA command register bit only when appropriate
+Deleted patch
-From: Finn Thain <fthain@telegraphics.com.au>
-It doesn't make sense to clear the command register bit unless the
-command was actually issued.
-Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
-Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
-Tested-by: Laurent Vivier <laurent@vivier.eu>
-Signed-off-by: Jason Wang <jasowang@redhat.com>
----
- hw/net/dp8393x.c | 7 +++----
-file changed, 3 insertions(+), 4 deletions(-)
-diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
-index XXXXXXX..XXXXXXX 100644
---- a/hw/net/dp8393x.c
-+++ b/hw/net/dp8393x.c
-@@ -XXX,XX +XXX,XX @@ static void dp8393x_do_read_rra(dp8393xState *s)
-         s->regs[SONIC_ISR] |= SONIC_ISR_RBE;
-         dp8393x_update_irq(s);
-     }
--
--    /* Done */
--    s->regs[SONIC_CR] &= ~SONIC_CR_RRRA;
- }
- static void dp8393x_do_software_reset(dp8393xState *s)
-@@ -XXX,XX +XXX,XX @@ static void dp8393x_do_command(dp8393xState *s, uint16_t command)
-         dp8393x_do_start_timer(s);
-     if (command & SONIC_CR_RST)
-         dp8393x_do_software_reset(s);
--    if (command & SONIC_CR_RRRA)
-+    if (command & SONIC_CR_RRRA) {
-         dp8393x_do_read_rra(s);
-+        s->regs[SONIC_CR] &= ~SONIC_CR_RRRA;
-+    }
-     if (command & SONIC_CR_LCAM)
-         dp8393x_do_load_cam(s);
- }
---
-.5.0

-[PULL 07/23] dp8393x: Implement packet size limit and RBAE interrupt
+Deleted patch
-From: Finn Thain <fthain@telegraphics.com.au>
-Add a bounds check to prevent a large packet from causing a buffer
-overflow. This is defensive programming -- I haven't actually tried
-sending an oversized packet or a jumbo ethernet frame.
-The SONIC handles packets that are too big for the buffer by raising
-the RBAE interrupt and dropping them. Linux uses that interrupt to
-count dropped packets.
-Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
-Tested-by: Laurent Vivier <laurent@vivier.eu>
-Signed-off-by: Jason Wang <jasowang@redhat.com>
----
- hw/net/dp8393x.c | 9 +++++++++
-file changed, 9 insertions(+)
-diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
-index XXXXXXX..XXXXXXX 100644
---- a/hw/net/dp8393x.c
-+++ b/hw/net/dp8393x.c
-@@ -XXX,XX +XXX,XX @@ do { printf("sonic ERROR: %s: " fmt, __func__ , ## __VA_ARGS__); } while (0)
- #define SONIC_TCR_CRCI   0x2000
- #define SONIC_TCR_PINT   0x8000
-+#define SONIC_ISR_RBAE   0x0010
- #define SONIC_ISR_RBE    0x0020
- #define SONIC_ISR_RDE    0x0040
- #define SONIC_ISR_TC     0x0080
-@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
-     s->regs[SONIC_RCR] &= ~(SONIC_RCR_PRX | SONIC_RCR_LBK | SONIC_RCR_FAER |
-         SONIC_RCR_CRCR | SONIC_RCR_LPKT | SONIC_RCR_BC | SONIC_RCR_MC);
-+    if (pkt_size + 4 > dp8393x_rbwc(s) * 2) {
-+        DPRINTF("oversize packet, pkt_size is %d\n", pkt_size);
-+        s->regs[SONIC_ISR] |= SONIC_ISR_RBAE;
-+        dp8393x_update_irq(s);
-+        dp8393x_do_read_rra(s);
-+        return pkt_size;
-+    }
-+
-     packet_type = dp8393x_receive_filter(s, buf, pkt_size);
-     if (packet_type < 0) {
-         DPRINTF("packet not for netcard\n");
---
-.5.0

-[PULL 08/23] dp8393x: Don't clobber packet checksum
+Deleted patch
-From: Finn Thain <fthain@telegraphics.com.au>
-A received packet consumes pkt_size bytes in the buffer and the frame
-checksum that's appended to it consumes another 4 bytes. The Receive
-Buffer Address register takes the former quantity into account but
-not the latter. So the next packet written to the buffer overwrites
-the frame checksum. Fix this.
-Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
-Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
-Tested-by: Laurent Vivier <laurent@vivier.eu>
-Signed-off-by: Jason Wang <jasowang@redhat.com>
----
- hw/net/dp8393x.c | 1 +
-file changed, 1 insertion(+)
-diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
-index XXXXXXX..XXXXXXX 100644
---- a/hw/net/dp8393x.c
-+++ b/hw/net/dp8393x.c
-@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
-     address += rx_len;
-     address_space_write(&s->as, address, MEMTXATTRS_UNSPECIFIED,
-                         &checksum, 4);
-+    address += 4;
-     rx_len += 4;
-     s->regs[SONIC_CRBA1] = address >> 16;
-     s->regs[SONIC_CRBA0] = address & 0xffff;
---
-.5.0

-[PULL 09/23] dp8393x: Use long-word-aligned RRA pointers in 32-bit mode
+Deleted patch
-From: Finn Thain <fthain@telegraphics.com.au>
-Section 3.4.1 of the datasheet says,
-    The alignment of the RRA is confined to either word or long word
-    boundaries, depending upon the data width mode. In 16-bit mode,
-    the RRA must be aligned to a word boundary (A0 is always zero)
-    and in 32-bit mode, the RRA is aligned to a long word boundary
-    (A0 and A1 are always zero).
-This constraint has been implemented for 16-bit mode; implement it
-for 32-bit mode too.
-Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
-Tested-by: Laurent Vivier <laurent@vivier.eu>
-Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
-Signed-off-by: Jason Wang <jasowang@redhat.com>
----
- hw/net/dp8393x.c | 8 ++++++--
-file changed, 6 insertions(+), 2 deletions(-)
-diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
-index XXXXXXX..XXXXXXX 100644
---- a/hw/net/dp8393x.c
-+++ b/hw/net/dp8393x.c
-@@ -XXX,XX +XXX,XX @@ static void dp8393x_write(void *opaque, hwaddr addr, uint64_t data,
-                 qemu_flush_queued_packets(qemu_get_queue(s->nic));
-             }
-             break;
--        /* Ignore least significant bit */
-+        /* The guest is required to store aligned pointers here */
-         case SONIC_RSA:
-         case SONIC_REA:
-         case SONIC_RRP:
-         case SONIC_RWP:
--            s->regs[reg] = val & 0xfffe;
-+            if (s->regs[SONIC_DCR] & SONIC_DCR_DW) {
-+                s->regs[reg] = val & 0xfffc;
-+            } else {
-+                s->regs[reg] = val & 0xfffe;
-+            }
-             break;
-         /* Invert written value for some registers */
-         case SONIC_CRCT:
---
-.5.0

-[PULL 10/23] dp8393x: Pad frames to word or long word boundary
+Deleted patch
-From: Finn Thain <fthain@telegraphics.com.au>
-The existing code has a bug where the Remaining Buffer Word Count (RBWC)
-is calculated with a truncating division, which gives the wrong result
-for odd-sized packets.
-Section 1.4.1 of the datasheet says,
-    Once the end of the packet has been reached, the serializer will
-    fill out the last word (16-bit mode) or long word (32-bit mode)
-    if the last byte did not end on a word or long word boundary
-    respectively. The fill byte will be 0FFh.
-Implement buffer padding so that buffer limits are correctly enforced.
-Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
-Tested-by: Laurent Vivier <laurent@vivier.eu>
-Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
-Signed-off-by: Jason Wang <jasowang@redhat.com>
----
- hw/net/dp8393x.c | 39 ++++++++++++++++++++++++++++-----------
-file changed, 28 insertions(+), 11 deletions(-)
-diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
-index XXXXXXX..XXXXXXX 100644
---- a/hw/net/dp8393x.c
-+++ b/hw/net/dp8393x.c
-@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
-     dp8393xState *s = qemu_get_nic_opaque(nc);
-     int packet_type;
-     uint32_t available, address;
--    int width, rx_len = pkt_size;
-+    int width, rx_len, padded_len;
-     uint32_t checksum;
-     int size;
--    width = (s->regs[SONIC_DCR] & SONIC_DCR_DW) ? 2 : 1;
--
-     s->regs[SONIC_RCR] &= ~(SONIC_RCR_PRX | SONIC_RCR_LBK | SONIC_RCR_FAER |
-         SONIC_RCR_CRCR | SONIC_RCR_LPKT | SONIC_RCR_BC | SONIC_RCR_MC);
--    if (pkt_size + 4 > dp8393x_rbwc(s) * 2) {
-+    rx_len = pkt_size + sizeof(checksum);
-+    if (s->regs[SONIC_DCR] & SONIC_DCR_DW) {
-+        width = 2;
-+        padded_len = ((rx_len - 1) | 3) + 1;
-+    } else {
-+        width = 1;
-+        padded_len = ((rx_len - 1) | 1) + 1;
-+    }
-+
-+    if (padded_len > dp8393x_rbwc(s) * 2) {
-         DPRINTF("oversize packet, pkt_size is %d\n", pkt_size);
-         s->regs[SONIC_ISR] |= SONIC_ISR_RBAE;
-         dp8393x_update_irq(s);
-@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
-     s->regs[SONIC_TRBA0] = s->regs[SONIC_CRBA0];
-     /* Calculate the ethernet checksum */
--    checksum = cpu_to_le32(crc32(0, buf, rx_len));
-+    checksum = cpu_to_le32(crc32(0, buf, pkt_size));
-     /* Put packet into RBA */
-     DPRINTF("Receive packet at %08x\n", dp8393x_crba(s));
-     address = dp8393x_crba(s);
-     address_space_write(&s->as, address, MEMTXATTRS_UNSPECIFIED,
--                        buf, rx_len);
--    address += rx_len;
-+                        buf, pkt_size);
-+    address += pkt_size;
-+
-+    /* Put frame checksum into RBA */
-     address_space_write(&s->as, address, MEMTXATTRS_UNSPECIFIED,
--                        &checksum, 4);
--    address += 4;
--    rx_len += 4;
-+                        &checksum, sizeof(checksum));
-+    address += sizeof(checksum);
-+
-+    /* Pad short packets to keep pointers aligned */
-+    if (rx_len < padded_len) {
-+        size = padded_len - rx_len;
-+        address_space_rw(&s->as, address, MEMTXATTRS_UNSPECIFIED,
-+            (uint8_t *)"\xFF\xFF\xFF", size, 1);
-+        address += size;
-+    }
-+
-     s->regs[SONIC_CRBA1] = address >> 16;
-     s->regs[SONIC_CRBA0] = address & 0xffff;
-     available = dp8393x_rbwc(s);
--    available -= rx_len / 2;
-+    available -= padded_len >> 1;
-     s->regs[SONIC_RBWC1] = available >> 16;
-     s->regs[SONIC_RBWC0] = available & 0xffff;
---
-.5.0

-[PULL 11/23] dp8393x: Clear descriptor in_use field to release packet
+Deleted patch
-From: Finn Thain <fthain@telegraphics.com.au>
-When the SONIC receives a packet into the last available descriptor, it
-retains ownership of that descriptor for as long as necessary.
-Section 3.4.7 of the datasheet says,
-    When the system appends more descriptors, the SONIC releases ownership
-    of the descriptor after writing 0000h to the RXpkt.in_use field.
-The packet can now be processed by the host, so raise a PKTRX interrupt,
-just like the normal case.
-Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
-Tested-by: Laurent Vivier <laurent@vivier.eu>
-Signed-off-by: Jason Wang <jasowang@redhat.com>
----
- hw/net/dp8393x.c | 10 ++++++++++
-file changed, 10 insertions(+)
-diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
-index XXXXXXX..XXXXXXX 100644
---- a/hw/net/dp8393x.c
-+++ b/hw/net/dp8393x.c
-@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
-             return -1;
-         }
-         /* Link has been updated by host */
-+
-+        /* Clear in_use */
-+        size = sizeof(uint16_t) * width;
-+        address = dp8393x_crda(s) + sizeof(uint16_t) * 6 * width;
-+        dp8393x_put(s, width, 0, 0);
-+        address_space_rw(&s->as, address, MEMTXATTRS_UNSPECIFIED,
-+                         (uint8_t *)s->data, size, 1);
-+
-+        /* Move to next descriptor */
-         s->regs[SONIC_CRDA] = s->regs[SONIC_LLFA];
-+        s->regs[SONIC_ISR] |= SONIC_ISR_PKTRX;
-     }
-     /* Save current position */
---
-.5.0

-[PULL 12/23] dp8393x: Always update RRA pointers and sequence numbers
+Deleted patch
-From: Finn Thain <fthain@telegraphics.com.au>
-These operations need to take place regardless of whether or not
-rx descriptors have been used up (that is, EOL flag was observed).
-The algorithm is now the same for a packet that was withheld as for
-a packet that was not.
-Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
-Tested-by: Laurent Vivier <laurent@vivier.eu>
-Signed-off-by: Jason Wang <jasowang@redhat.com>
----
- hw/net/dp8393x.c | 12 +++++++-----
-file changed, 7 insertions(+), 5 deletions(-)
-diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
-index XXXXXXX..XXXXXXX 100644
---- a/hw/net/dp8393x.c
-+++ b/hw/net/dp8393x.c
-@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
-         /* Move to next descriptor */
-         s->regs[SONIC_CRDA] = s->regs[SONIC_LLFA];
-         s->regs[SONIC_ISR] |= SONIC_ISR_PKTRX;
--        s->regs[SONIC_RSC] = (s->regs[SONIC_RSC] & 0xff00) | (((s->regs[SONIC_RSC] & 0x00ff) + 1) & 0x00ff);
-+    }
--        if (s->regs[SONIC_RCR] & SONIC_RCR_LPKT) {
--            /* Read next RRA */
--            dp8393x_do_read_rra(s);
--        }
-+    s->regs[SONIC_RSC] = (s->regs[SONIC_RSC] & 0xff00) |
-+                         ((s->regs[SONIC_RSC] + 1) & 0x00ff);
-+
-+    if (s->regs[SONIC_RCR] & SONIC_RCR_LPKT) {
-+        /* Read next RRA */
-+        dp8393x_do_read_rra(s);
-     }
-     /* Done */
---
-.5.0

-[PULL 13/23] dp8393x: Don't reset Silicon Revision register
+Deleted patch
-From: Finn Thain <fthain@telegraphics.com.au>
-The jazzsonic driver in Linux uses the Silicon Revision register value
-to probe the chip. The driver fails unless the SR register contains 4.
-Unfortunately, reading this register in QEMU usually returns 0 because
-the s->regs[] array gets wiped after a software reset.
-Fixes: bd8f1ebce4 ("net/dp8393x: fix hardware reset")
-Suggested-by: Philippe Mathieu-Daudé <philmd@redhat.com>
-Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
-Signed-off-by: Jason Wang <jasowang@redhat.com>
----
- hw/net/dp8393x.c | 2 +-
-file changed, 1 insertion(+), 1 deletion(-)
-diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
-index XXXXXXX..XXXXXXX 100644
---- a/hw/net/dp8393x.c
-+++ b/hw/net/dp8393x.c
-@@ -XXX,XX +XXX,XX @@ static void dp8393x_reset(DeviceState *dev)
-     timer_del(s->watchdog);
-     memset(s->regs, 0, sizeof(s->regs));
-+    s->regs[SONIC_SR] = 0x0004; /* only revision recognized by Linux/mips */
-     s->regs[SONIC_CR] = SONIC_CR_RST | SONIC_CR_STP | SONIC_CR_RXDIS;
-     s->regs[SONIC_DCR] &= ~(SONIC_DCR_EXBUS | SONIC_DCR_LBR);
-     s->regs[SONIC_RCR] &= ~(SONIC_RCR_LB0 | SONIC_RCR_LB1 | SONIC_RCR_BRD | SONIC_RCR_RNT);
-@@ -XXX,XX +XXX,XX @@ static void dp8393x_realize(DeviceState *dev, Error **errp)
-     qemu_format_nic_info_str(qemu_get_queue(s->nic), s->conf.macaddr.a);
-     s->watchdog = timer_new_ns(QEMU_CLOCK_VIRTUAL, dp8393x_watchdog, s);
--    s->regs[SONIC_SR] = 0x0004; /* only revision recognized by Linux */
-     memory_region_init_ram(&s->prom, OBJECT(dev),
-                            "dp8393x-prom", SONIC_PROM_SIZE, &local_err);
---
-.5.0

-[PULL 14/23] dp8393x: Don't stop reception upon RBE interrupt assertion
+Deleted patch
-From: Finn Thain <fthain@telegraphics.com.au>
-Section 3.4.7 of the datasheet explains that,
-    The RBE bit in the Interrupt Status register is set when the
-    SONIC finishes using the second to last receive buffer and reads
-    the last RRA descriptor. Actually, the SONIC is not truly out of
-    resources, but gives the system an early warning of an impending
-    out of resources condition.
-RBE does not mean actual receive buffer exhaustion, and reception should
-not be stopped. This is important because Linux will not check and clear
-the RBE interrupt until it receives another packet. But that won't
-happen if can_receive returns false. This bug causes the SONIC to become
-deaf (until reset).
-Fix this with a new flag to indicate actual receive buffer exhaustion.
-Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
-Tested-by: Laurent Vivier <laurent@vivier.eu>
-Signed-off-by: Jason Wang <jasowang@redhat.com>
----
- hw/net/dp8393x.c | 35 ++++++++++++++++++++++-------------
-file changed, 22 insertions(+), 13 deletions(-)
-diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
-index XXXXXXX..XXXXXXX 100644
---- a/hw/net/dp8393x.c
-+++ b/hw/net/dp8393x.c
-@@ -XXX,XX +XXX,XX @@ typedef struct dp8393xState {
-     /* Hardware */
-     uint8_t it_shift;
-     bool big_endian;
-+    bool last_rba_is_full;
-     qemu_irq irq;
- #ifdef DEBUG_SONIC
-     int irq_level;
-@@ -XXX,XX +XXX,XX @@ static void dp8393x_do_read_rra(dp8393xState *s)
-         s->regs[SONIC_RRP] = s->regs[SONIC_RSA];
-     }
--    /* Check resource exhaustion */
-+    /* Warn the host if CRBA now has the last available resource */
-     if (s->regs[SONIC_RRP] == s->regs[SONIC_RWP])
-     {
-         s->regs[SONIC_ISR] |= SONIC_ISR_RBE;
-         dp8393x_update_irq(s);
-     }
-+
-+    /* Allow packet reception */
-+    s->last_rba_is_full = false;
- }
- static void dp8393x_do_software_reset(dp8393xState *s)
-@@ -XXX,XX +XXX,XX @@ static void dp8393x_write(void *opaque, hwaddr addr, uint64_t data,
-                 dp8393x_do_read_rra(s);
-             }
-             dp8393x_update_irq(s);
--            if (dp8393x_can_receive(s->nic->ncs)) {
--                qemu_flush_queued_packets(qemu_get_queue(s->nic));
--            }
-             break;
-         /* The guest is required to store aligned pointers here */
-         case SONIC_RSA:
-@@ -XXX,XX +XXX,XX @@ static int dp8393x_can_receive(NetClientState *nc)
-     if (!(s->regs[SONIC_CR] & SONIC_CR_RXEN))
-         return 0;
--    if (s->regs[SONIC_ISR] & SONIC_ISR_RBE)
--        return 0;
-     return 1;
- }
-@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
-     s->regs[SONIC_RCR] &= ~(SONIC_RCR_PRX | SONIC_RCR_LBK | SONIC_RCR_FAER |
-         SONIC_RCR_CRCR | SONIC_RCR_LPKT | SONIC_RCR_BC | SONIC_RCR_MC);
-+    if (s->last_rba_is_full) {
-+        return pkt_size;
-+    }
-+
-     rx_len = pkt_size + sizeof(checksum);
-     if (s->regs[SONIC_DCR] & SONIC_DCR_DW) {
-         width = 2;
-@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
-         DPRINTF("oversize packet, pkt_size is %d\n", pkt_size);
-         s->regs[SONIC_ISR] |= SONIC_ISR_RBAE;
-         dp8393x_update_irq(s);
--        dp8393x_do_read_rra(s);
--        return pkt_size;
-+        s->regs[SONIC_RCR] |= SONIC_RCR_LPKT;
-+        goto done;
-     }
-     packet_type = dp8393x_receive_filter(s, buf, pkt_size);
-@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
-         s->regs[SONIC_ISR] |= SONIC_ISR_PKTRX;
-     }
-+    dp8393x_update_irq(s);
-+
-     s->regs[SONIC_RSC] = (s->regs[SONIC_RSC] & 0xff00) |
-                          ((s->regs[SONIC_RSC] + 1) & 0x00ff);
-+done:
-+
-     if (s->regs[SONIC_RCR] & SONIC_RCR_LPKT) {
--        /* Read next RRA */
--        dp8393x_do_read_rra(s);
-+        if (s->regs[SONIC_RRP] == s->regs[SONIC_RWP]) {
-+            /* Stop packet reception */
-+            s->last_rba_is_full = true;
-+        } else {
-+            /* Read next resource */
-+            dp8393x_do_read_rra(s);
-+        }
-     }
--    /* Done */
--    dp8393x_update_irq(s);
--
-     return pkt_size;
- }
---
-.5.0

-[PULL 15/23] e1000e: Avoid hw_error if legacy mode used
+Deleted patch
-From: Yuri Benditovich <yuri.benditovich@daynix.com>
-https://bugzilla.redhat.com/show_bug.cgi?id=1787142
-The emulation issues hw_error if PSRCTL register
-is written, for example, with zero value.
-Such configuration does not present any problem when
-DTYP bits of RCTL register define legacy format of
-transfer descriptors. Current commit discards check
-for BSIZE0 and BSIZE1 when legacy mode used.
-Acked-by: Dmitry Fleytman <dmitry.fleytman@gmail.com>
-Signed-off-by: Yuri Benditovich <yuri.benditovich@daynix.com>
-Signed-off-by: Jason Wang <jasowang@redhat.com>
----
- hw/net/e1000e_core.c | 13 ++++++++-----
-file changed, 8 insertions(+), 5 deletions(-)
-diff --git a/hw/net/e1000e_core.c b/hw/net/e1000e_core.c
-index XXXXXXX..XXXXXXX 100644
---- a/hw/net/e1000e_core.c
-+++ b/hw/net/e1000e_core.c
-@@ -XXX,XX +XXX,XX @@ e1000e_set_eitr(E1000ECore *core, int index, uint32_t val)
- static void
- e1000e_set_psrctl(E1000ECore *core, int index, uint32_t val)
- {
--    if ((val & E1000_PSRCTL_BSIZE0_MASK) == 0) {
--        hw_error("e1000e: PSRCTL.BSIZE0 cannot be zero");
--    }
-+    if (core->mac[RCTL] & E1000_RCTL_DTYP_MASK) {
-+
-+        if ((val & E1000_PSRCTL_BSIZE0_MASK) == 0) {
-+            hw_error("e1000e: PSRCTL.BSIZE0 cannot be zero");
-+        }
--    if ((val & E1000_PSRCTL_BSIZE1_MASK) == 0) {
--        hw_error("e1000e: PSRCTL.BSIZE1 cannot be zero");
-+        if ((val & E1000_PSRCTL_BSIZE1_MASK) == 0) {
-+            hw_error("e1000e: PSRCTL.BSIZE1 cannot be zero");
-+        }
-     }
-     core->mac[PSRCTL] = val;
---
-.5.0

-[PULL 17/23] NetRxPkt: fix hash calculation of IPV6 TCP
+Deleted patch
-From: Yuri Benditovich <yuri.benditovich@daynix.com>
-When requested to calculate the hash for TCPV6 packet,
-ignore overrides of source and destination addresses
-in in extension headers.
-Use these overrides when new hash type NetPktRssIpV6TcpEx
-requested.
-Use this type in e1000e hash calculation for IPv6 TCP, which
-should take in account overrides of the addresses.
-Signed-off-by: Yuri Benditovich <yuri.benditovich@daynix.com>
-Acked-by: Dmitry Fleytman <dmitry.fleytman@gmail.com>
-Signed-off-by: Jason Wang <jasowang@redhat.com>
----
- hw/net/e1000e_core.c | 2 +-
- hw/net/net_rx_pkt.c  | 2 +-
-files changed, 2 insertions(+), 2 deletions(-)
-diff --git a/hw/net/e1000e_core.c b/hw/net/e1000e_core.c
-index XXXXXXX..XXXXXXX 100644
---- a/hw/net/e1000e_core.c
-+++ b/hw/net/e1000e_core.c
-@@ -XXX,XX +XXX,XX @@ e1000e_rss_calc_hash(E1000ECore *core,
-         type = NetPktRssIpV4Tcp;
-         break;
-     case E1000_MRQ_RSS_TYPE_IPV6TCP:
--        type = NetPktRssIpV6Tcp;
-+        type = NetPktRssIpV6TcpEx;
-         break;
-     case E1000_MRQ_RSS_TYPE_IPV6:
-         type = NetPktRssIpV6;
-diff --git a/hw/net/net_rx_pkt.c b/hw/net/net_rx_pkt.c
-index XXXXXXX..XXXXXXX 100644
---- a/hw/net/net_rx_pkt.c
-+++ b/hw/net/net_rx_pkt.c
-@@ -XXX,XX +XXX,XX @@ net_rx_pkt_calc_rss_hash(struct NetRxPkt *pkt,
-         assert(pkt->isip6);
-         assert(pkt->istcp);
-         trace_net_rx_pkt_rss_ip6_tcp();
--        _net_rx_rss_prepare_ip6(&rss_input[0], pkt, true, &rss_length);
-+        _net_rx_rss_prepare_ip6(&rss_input[0], pkt, false, &rss_length);
-         _net_rx_rss_prepare_tcp(&rss_input[0], pkt, &rss_length);
-         break;
-     case NetPktRssIpV6:
---
-.5.0

-[PULL 18/23] hw: net: cadence_gem: Fix build errors in DB_PRINT()
+Deleted patch
-From: Bin Meng <bmeng.cn@gmail.com>
-When CADENCE_GEM_ERR_DEBUG is turned on, there are several
-compilation errors in DB_PRINT(). Fix them.
-While we are here, update to use appropriate modifiers in
-the same DB_PRINT() call.
-Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
-Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
-Signed-off-by: Jason Wang <jasowang@redhat.com>
----
- hw/net/cadence_gem.c | 11 ++++++-----
-file changed, 6 insertions(+), 5 deletions(-)
-diff --git a/hw/net/cadence_gem.c b/hw/net/cadence_gem.c
-index XXXXXXX..XXXXXXX 100644
---- a/hw/net/cadence_gem.c
-+++ b/hw/net/cadence_gem.c
-@@ -XXX,XX +XXX,XX @@ static ssize_t gem_receive(NetClientState *nc, const uint8_t *buf, size_t size)
-             return -1;
-         }
--        DB_PRINT("copy %d bytes to 0x%x\n", MIN(bytes_to_copy, rxbufsize),
--                rx_desc_get_buffer(s->rx_desc[q]));
-+        DB_PRINT("copy %u bytes to 0x%" PRIx64 "\n",
-+                 MIN(bytes_to_copy, rxbufsize),
-+                 rx_desc_get_buffer(s, s->rx_desc[q]));
-         /* Copy packet data to emulated DMA buffer */
-         address_space_write(&s->dma_as, rx_desc_get_buffer(s, s->rx_desc[q]) +
-@@ -XXX,XX +XXX,XX @@ static void gem_transmit(CadenceGEMState *s)
-             if (tx_desc_get_length(desc) > sizeof(tx_packet) -
-                                                (p - tx_packet)) {
--                DB_PRINT("TX descriptor @ 0x%x too large: size 0x%x space " \
--                         "0x%x\n", (unsigned)packet_desc_addr,
--                         (unsigned)tx_desc_get_length(desc),
-+                DB_PRINT("TX descriptor @ 0x%" HWADDR_PRIx \
-+                         " too large: size 0x%x space 0x%zx\n",
-+                         packet_desc_addr, tx_desc_get_length(desc),
-                          sizeof(tx_packet) - (p - tx_packet));
-                 break;
-             }
---
-.5.0

-[PULL 19/23] block/replication.c: Ignore requests after failover
+Deleted patch
-From: Lukas Straub <lukasstraub2@web.de>
-After failover the Secondary side of replication shouldn't change state, because
-it now functions as our primary disk.
-In replication_start, replication_do_checkpoint, replication_stop, ignore
-the request if current state is BLOCK_REPLICATION_DONE (sucessful failover) or
-BLOCK_REPLICATION_FAILOVER (failover in progres i.e. currently merging active
-and hidden images into the base image).
-Signed-off-by: Lukas Straub <lukasstraub2@web.de>
-Reviewed-by: Zhang Chen <chen.zhang@intel.com>
-Acked-by: Max Reitz <mreitz@redhat.com>
-Signed-off-by: Jason Wang <jasowang@redhat.com>
----
- block/replication.c | 35 ++++++++++++++++++++++++++++++++++-
-file changed, 34 insertions(+), 1 deletion(-)
-diff --git a/block/replication.c b/block/replication.c
-index XXXXXXX..XXXXXXX 100644
---- a/block/replication.c
-+++ b/block/replication.c
-@@ -XXX,XX +XXX,XX @@ static void replication_start(ReplicationState *rs, ReplicationMode mode,
-     aio_context_acquire(aio_context);
-     s = bs->opaque;
-+    if (s->stage == BLOCK_REPLICATION_DONE ||
-+        s->stage == BLOCK_REPLICATION_FAILOVER) {
-+        /*
-+         * This case happens when a secondary is promoted to primary.
-+         * Ignore the request because the secondary side of replication
-+         * doesn't have to do anything anymore.
-+         */
-+        aio_context_release(aio_context);
-+        return;
-+    }
-+
-     if (s->stage != BLOCK_REPLICATION_NONE) {
-         error_setg(errp, "Block replication is running or done");
-         aio_context_release(aio_context);
-@@ -XXX,XX +XXX,XX @@ static void replication_do_checkpoint(ReplicationState *rs, Error **errp)
-     aio_context_acquire(aio_context);
-     s = bs->opaque;
-+    if (s->stage == BLOCK_REPLICATION_DONE ||
-+        s->stage == BLOCK_REPLICATION_FAILOVER) {
-+        /*
-+         * This case happens when a secondary was promoted to primary.
-+         * Ignore the request because the secondary side of replication
-+         * doesn't have to do anything anymore.
-+         */
-+        aio_context_release(aio_context);
-+        return;
-+    }
-+
-     if (s->mode == REPLICATION_MODE_SECONDARY) {
-         secondary_do_checkpoint(s, errp);
-     }
-@@ -XXX,XX +XXX,XX @@ static void replication_get_error(ReplicationState *rs, Error **errp)
-     aio_context_acquire(aio_context);
-     s = bs->opaque;
--    if (s->stage != BLOCK_REPLICATION_RUNNING) {
-+    if (s->stage == BLOCK_REPLICATION_NONE) {
-         error_setg(errp, "Block replication is not running");
-         aio_context_release(aio_context);
-         return;
-@@ -XXX,XX +XXX,XX @@ static void replication_stop(ReplicationState *rs, bool failover, Error **errp)
-     aio_context_acquire(aio_context);
-     s = bs->opaque;
-+    if (s->stage == BLOCK_REPLICATION_DONE ||
-+        s->stage == BLOCK_REPLICATION_FAILOVER) {
-+        /*
-+         * This case happens when a secondary was promoted to primary.
-+         * Ignore the request because the secondary side of replication
-+         * doesn't have to do anything anymore.
-+         */
-+        aio_context_release(aio_context);
-+        return;
-+    }
-+
-     if (s->stage != BLOCK_REPLICATION_RUNNING) {
-         error_setg(errp, "Block replication is not running");
-         aio_context_release(aio_context);
---
-.5.0

-[PULL 20/23] tests/test-replication.c: Add test for for secondary node continuing replication
+Deleted patch
-From: Lukas Straub <lukasstraub2@web.de>
-This simulates the case that happens when we resume COLO after failover.
-Signed-off-by: Lukas Straub <lukasstraub2@web.de>
-Signed-off-by: Jason Wang <jasowang@redhat.com>
----
- tests/test-replication.c | 52 ++++++++++++++++++++++++++++++++++++++++++++++++
-file changed, 52 insertions(+)
-diff --git a/tests/test-replication.c b/tests/test-replication.c
-index XXXXXXX..XXXXXXX 100644
---- a/tests/test-replication.c
-+++ b/tests/test-replication.c
-@@ -XXX,XX +XXX,XX @@ static void test_secondary_stop(void)
-     teardown_secondary();
- }
-+static void test_secondary_continuous_replication(void)
-+{
-+    BlockBackend *top_blk, *local_blk;
-+    Error *local_err = NULL;
-+
-+    top_blk = start_secondary();
-+    replication_start_all(REPLICATION_MODE_SECONDARY, &local_err);
-+    g_assert(!local_err);
-+
-+    /* write 0x22 to s_local_disk (IMG_SIZE / 2, IMG_SIZE) */
-+    local_blk = blk_by_name(S_LOCAL_DISK_ID);
-+    test_blk_write(local_blk, 0x22, IMG_SIZE / 2, IMG_SIZE / 2, false);
-+
-+    /* replication will backup s_local_disk to s_hidden_disk */
-+    test_blk_read(top_blk, 0x11, IMG_SIZE / 2,
-+                  IMG_SIZE / 2, 0, IMG_SIZE, false);
-+
-+    /* write 0x33 to s_active_disk (0, IMG_SIZE / 2) */
-+    test_blk_write(top_blk, 0x33, 0, IMG_SIZE / 2, false);
-+
-+    /* do failover (active commit) */
-+    replication_stop_all(true, &local_err);
-+    g_assert(!local_err);
-+
-+    /* it should ignore all requests from now on */
-+
-+    /* start after failover */
-+    replication_start_all(REPLICATION_MODE_PRIMARY, &local_err);
-+    g_assert(!local_err);
-+
-+    /* checkpoint */
-+    replication_do_checkpoint_all(&local_err);
-+    g_assert(!local_err);
-+
-+    /* stop */
-+    replication_stop_all(true, &local_err);
-+    g_assert(!local_err);
-+
-+    /* read from s_local_disk (0, IMG_SIZE / 2) */
-+    test_blk_read(top_blk, 0x33, 0, IMG_SIZE / 2,
-+                  0, IMG_SIZE / 2, false);
-+
-+
-+    /* read from s_local_disk (IMG_SIZE / 2, IMG_SIZE) */
-+    test_blk_read(top_blk, 0x22, IMG_SIZE / 2,
-+                  IMG_SIZE / 2, 0, IMG_SIZE, false);
-+
-+    teardown_secondary();
-+}
-+
- static void test_secondary_do_checkpoint(void)
- {
-     BlockBackend *top_blk, *local_blk;
-@@ -XXX,XX +XXX,XX @@ int main(int argc, char **argv)
-     g_test_add_func("/replication/secondary/write", test_secondary_write);
-     g_test_add_func("/replication/secondary/start", test_secondary_start);
-     g_test_add_func("/replication/secondary/stop",  test_secondary_stop);
-+    g_test_add_func("/replication/secondary/continuous_replication",
-+                    test_secondary_continuous_replication);
-     g_test_add_func("/replication/secondary/do_checkpoint",
-                     test_secondary_do_checkpoint);
-     g_test_add_func("/replication/secondary/get_error_all",
---
-.5.0

-[PULL 21/23] net/filter.c: Add Options to insert filters anywhere in the filter list
+Deleted patch
-From: Lukas Straub <lukasstraub2@web.de>
-To switch the Secondary to Primary, we need to insert new filters
-before the filter-rewriter.
-Add the options insert= and position= to be able to insert filters
-anywhere in the filter list.
-position should be "head" or "tail" to insert at the head or
-tail of the filter list or it should be "id=<id>" to specify
-the id of another filter.
-insert should be either "before" or "behind" to specify where to
-insert the new filter relative to the one specified with position.
-Signed-off-by: Lukas Straub <lukasstraub2@web.de>
-Reviewed-by: Zhang Chen <chen.zhang@intel.com>
-Signed-off-by: Jason Wang <jasowang@redhat.com>
----
- include/net/filter.h |  2 ++
- net/filter.c         | 92 +++++++++++++++++++++++++++++++++++++++++++++++++++-
- qemu-options.hx      | 31 +++++++++++++++---
-files changed, 119 insertions(+), 6 deletions(-)
-diff --git a/include/net/filter.h b/include/net/filter.h
-index XXXXXXX..XXXXXXX 100644
---- a/include/net/filter.h
-+++ b/include/net/filter.h
-@@ -XXX,XX +XXX,XX @@ struct NetFilterState {
-     NetClientState *netdev;
-     NetFilterDirection direction;
-     bool on;
-+    char *position;
-+    bool insert_before_flag;
-     QTAILQ_ENTRY(NetFilterState) next;
- };
-diff --git a/net/filter.c b/net/filter.c
-index XXXXXXX..XXXXXXX 100644
---- a/net/filter.c
-+++ b/net/filter.c
-@@ -XXX,XX +XXX,XX @@ static void netfilter_set_status(Object *obj, const char *str, Error **errp)
-     }
- }
-+static char *netfilter_get_position(Object *obj, Error **errp)
-+{
-+    NetFilterState *nf = NETFILTER(obj);
-+
-+    return g_strdup(nf->position);
-+}
-+
-+static void netfilter_set_position(Object *obj, const char *str, Error **errp)
-+{
-+    NetFilterState *nf = NETFILTER(obj);
-+
-+    nf->position = g_strdup(str);
-+}
-+
-+static char *netfilter_get_insert(Object *obj, Error **errp)
-+{
-+    NetFilterState *nf = NETFILTER(obj);
-+
-+    return nf->insert_before_flag ? g_strdup("before") : g_strdup("behind");
-+}
-+
-+static void netfilter_set_insert(Object *obj, const char *str, Error **errp)
-+{
-+    NetFilterState *nf = NETFILTER(obj);
-+
-+    if (strcmp(str, "before") && strcmp(str, "behind")) {
-+        error_setg(errp, "Invalid value for netfilter insert, "
-+                         "should be 'before' or 'behind'");
-+        return;
-+    }
-+
-+    nf->insert_before_flag = !strcmp(str, "before");
-+}
-+
- static void netfilter_init(Object *obj)
- {
-     NetFilterState *nf = NETFILTER(obj);
-     nf->on = true;
-+    nf->insert_before_flag = false;
-+    nf->position = g_strdup("tail");
-     object_property_add_str(obj, "netdev",
-                             netfilter_get_netdev_id, netfilter_set_netdev_id,
-@@ -XXX,XX +XXX,XX @@ static void netfilter_init(Object *obj)
-     object_property_add_str(obj, "status",
-                             netfilter_get_status, netfilter_set_status,
-                             NULL);
-+    object_property_add_str(obj, "position",
-+                            netfilter_get_position, netfilter_set_position,
-+                            NULL);
-+    object_property_add_str(obj, "insert",
-+                            netfilter_get_insert, netfilter_set_insert,
-+                            NULL);
- }
- static void netfilter_complete(UserCreatable *uc, Error **errp)
- {
-     NetFilterState *nf = NETFILTER(uc);
-+    NetFilterState *position = NULL;
-     NetClientState *ncs[MAX_QUEUE_NUM];
-     NetFilterClass *nfc = NETFILTER_GET_CLASS(uc);
-     int queues;
-@@ -XXX,XX +XXX,XX @@ static void netfilter_complete(UserCreatable *uc, Error **errp)
-         return;
-     }
-+    if (strcmp(nf->position, "head") && strcmp(nf->position, "tail")) {
-+        Object *container;
-+        Object *obj;
-+        char *position_id;
-+
-+        if (!g_str_has_prefix(nf->position, "id=")) {
-+            error_setg(errp, QERR_INVALID_PARAMETER_VALUE, "position",
-+                       "'head', 'tail' or 'id=<id>'");
-+            return;
-+        }
-+
-+        /* get the id from the string */
-+        position_id = g_strndup(nf->position + 3, strlen(nf->position) - 3);
-+
-+        /* Search for the position to insert before/behind */
-+        container = object_get_objects_root();
-+        obj = object_resolve_path_component(container, position_id);
-+        if (!obj) {
-+            error_setg(errp, "filter '%s' not found", position_id);
-+            g_free(position_id);
-+            return;
-+        }
-+
-+        position = NETFILTER(obj);
-+
-+        if (position->netdev != ncs[0]) {
-+            error_setg(errp, "filter '%s' belongs to a different netdev",
-+                        position_id);
-+            g_free(position_id);
-+            return;
-+        }
-+
-+        g_free(position_id);
-+    }
-+
-     nf->netdev = ncs[0];
-     if (nfc->setup) {
-@@ -XXX,XX +XXX,XX @@ static void netfilter_complete(UserCreatable *uc, Error **errp)
-             return;
-         }
-     }
--    QTAILQ_INSERT_TAIL(&nf->netdev->filters, nf, next);
-+
-+    if (position) {
-+        if (nf->insert_before_flag) {
-+            QTAILQ_INSERT_BEFORE(position, nf, next);
-+        } else {
-+            QTAILQ_INSERT_AFTER(&nf->netdev->filters, position, nf, next);
-+        }
-+    } else if (!strcmp(nf->position, "head")) {
-+        QTAILQ_INSERT_HEAD(&nf->netdev->filters, nf, next);
-+    } else if (!strcmp(nf->position, "tail")) {
-+        QTAILQ_INSERT_TAIL(&nf->netdev->filters, nf, next);
-+    }
- }
- static void netfilter_finalize(Object *obj)
-@@ -XXX,XX +XXX,XX @@ static void netfilter_finalize(Object *obj)
-         QTAILQ_REMOVE(&nf->netdev->filters, nf, next);
-     }
-     g_free(nf->netdev_id);
-+    g_free(nf->position);
- }
- static void default_handle_event(NetFilterState *nf, int event, Error **errp)
-diff --git a/qemu-options.hx b/qemu-options.hx
-index XXXXXXX..XXXXXXX 100644
---- a/qemu-options.hx
-+++ b/qemu-options.hx
-@@ -XXX,XX +XXX,XX @@ applications, they can do this through this parameter. Its format is
- a gnutls priority string as described at
- @url{https://gnutls.org/manual/html_node/Priority-Strings.html}.
--@item -object filter-buffer,id=@var{id},netdev=@var{netdevid},interval=@var{t}[,queue=@var{all|rx|tx}][,status=@var{on|off}]
-+@item -object filter-buffer,id=@var{id},netdev=@var{netdevid},interval=@var{t}[,queue=@var{all|rx|tx}][,status=@var{on|off}][,position=@var{head|tail|id=<id>}][,insert=@var{behind|before}]
- Interval @var{t} can't be 0, this filter batches the packet delivery: all
- packets arriving in a given interval on netdev @var{netdevid} are delayed
-@@ -XXX,XX +XXX,XX @@ queue @var{all|rx|tx} is an option that can be applied to any netfilter.
- @option{tx}: the filter is attached to the transmit queue of the netdev,
-              where it will receive packets sent by the netdev.
--@item -object filter-mirror,id=@var{id},netdev=@var{netdevid},outdev=@var{chardevid},queue=@var{all|rx|tx}[,vnet_hdr_support]
-+position @var{head|tail|id=<id>} is an option to specify where the
-+filter should be inserted in the filter list. It can be applied to any
-+netfilter.
-+
-+@option{head}: the filter is inserted at the head of the filter
-+               list, before any existing filters.
-+
-+@option{tail}: the filter is inserted at the tail of the filter
-+               list, behind any existing filters (default).
-+
-+@option{id=<id>}: the filter is inserted before or behind the filter
-+                  specified by <id>, see the insert option below.
-+
-+insert @var{behind|before} is an option to specify where to insert the
-+new filter relative to the one specified with position=id=<id>. It can
-+be applied to any netfilter.
-+
-+@option{before}: insert before the specified filter.
-+
-+@option{behind}: insert behind the specified filter (default).
-+
-+@item -object filter-mirror,id=@var{id},netdev=@var{netdevid},outdev=@var{chardevid},queue=@var{all|rx|tx}[,vnet_hdr_support][,position=@var{head|tail|id=<id>}][,insert=@var{behind|before}]
- filter-mirror on netdev @var{netdevid},mirror net packet to chardev@var{chardevid}, if it has the vnet_hdr_support flag, filter-mirror will mirror packet with vnet_hdr_len.
--@item -object filter-redirector,id=@var{id},netdev=@var{netdevid},indev=@var{chardevid},outdev=@var{chardevid},queue=@var{all|rx|tx}[,vnet_hdr_support]
-+@item -object filter-redirector,id=@var{id},netdev=@var{netdevid},indev=@var{chardevid},outdev=@var{chardevid},queue=@var{all|rx|tx}[,vnet_hdr_support][,position=@var{head|tail|id=<id>}][,insert=@var{behind|before}]
- filter-redirector on netdev @var{netdevid},redirect filter's net packet to chardev
- @var{chardevid},and redirect indev's packet to filter.if it has the vnet_hdr_support flag,
-@@ -XXX,XX +XXX,XX @@ Create a filter-redirector we need to differ outdev id from indev id, id can not
- be the same. we can just use indev or outdev, but at least one of indev or outdev
- need to be specified.
--@item -object filter-rewriter,id=@var{id},netdev=@var{netdevid},queue=@var{all|rx|tx},[vnet_hdr_support]
-+@item -object filter-rewriter,id=@var{id},netdev=@var{netdevid},queue=@var{all|rx|tx},[vnet_hdr_support][,position=@var{head|tail|id=<id>}][,insert=@var{behind|before}]
- Filter-rewriter is a part of COLO project.It will rewrite tcp packet to
- secondary from primary to keep secondary tcp connection,and rewrite
-@@ -XXX,XX +XXX,XX @@ colo secondary:
- -object filter-redirector,id=f2,netdev=hn0,queue=rx,outdev=red1
- -object filter-rewriter,id=rew0,netdev=hn0,queue=all
--@item -object filter-dump,id=@var{id},netdev=@var{dev}[,file=@var{filename}][,maxlen=@var{len}]
-+@item -object filter-dump,id=@var{id},netdev=@var{dev}[,file=@var{filename}][,maxlen=@var{len}][,position=@var{head|tail|id=<id>}][,insert=@var{behind|before}]
- Dump the network traffic on netdev @var{dev} to the file specified by
- @var{filename}. At most @var{len} bytes (64k by default) per packet are stored.
---
-.5.0

-[PULL 22/23] colo: Update Documentation for continuous replication
+Deleted patch
-From: Lukas Straub <lukasstraub2@web.de>
-Document the qemu command-line and qmp commands for continuous replication
-Signed-off-by: Lukas Straub <lukasstraub2@web.de>
-Signed-off-by: Jason Wang <jasowang@redhat.com>
----
- docs/COLO-FT.txt           | 224 +++++++++++++++++++++++++++++++++------------
- docs/block-replication.txt |  28 ++++--
-files changed, 184 insertions(+), 68 deletions(-)
-diff --git a/docs/COLO-FT.txt b/docs/COLO-FT.txt
-index XXXXXXX..XXXXXXX 100644
---- a/docs/COLO-FT.txt
-+++ b/docs/COLO-FT.txt
-@@ -XXX,XX +XXX,XX @@ The diagram just shows the main qmp command, you can get the detail
- in test procedure.
- == Test procedure ==
--1. Startup qemu
--Primary:
--# qemu-system-x86_64 -accel kvm -m 2048 -smp 2 -qmp stdio -name primary \
--  -device piix3-usb-uhci -vnc :7 \
--  -device usb-tablet -netdev tap,id=hn0,vhost=off \
--  -device virtio-net-pci,id=net-pci0,netdev=hn0 \
--  -drive if=virtio,id=primary-disk0,driver=quorum,read-pattern=fifo,vote-threshold=1,\
--         children.0.file.filename=1.raw,\
--         children.0.driver=raw -S
--Secondary:
--# qemu-system-x86_64 -accel kvm -m 2048 -smp 2 -qmp stdio -name secondary \
--  -device piix3-usb-uhci -vnc :7 \
--  -device usb-tablet -netdev tap,id=hn0,vhost=off \
--  -device virtio-net-pci,id=net-pci0,netdev=hn0 \
--  -drive if=none,id=secondary-disk0,file.filename=1.raw,driver=raw,node-name=node0 \
--  -drive if=virtio,id=active-disk0,driver=replication,mode=secondary,\
--         file.driver=qcow2,top-id=active-disk0,\
--         file.file.filename=/mnt/ramfs/active_disk.img,\
--         file.backing.driver=qcow2,\
--         file.backing.file.filename=/mnt/ramfs/hidden_disk.img,\
--         file.backing.backing=secondary-disk0 \
--  -incoming tcp:0:8888
--
--2. On Secondary VM's QEMU monitor, issue command
-+Note: Here we are running both instances on the same host for testing,
-+change the IP Addresses if you want to run it on two hosts. Initally
-+127.0.0.1 is the Primary Host and 127.0.0.2 is the Secondary Host.
-+
-+== Startup qemu ==
-+1. Primary:
-+Note: Initally, $imagefolder/primary.qcow2 needs to be copied to all hosts.
-+You don't need to change any IP's here, because 0.0.0.0 listens on any
-+interface. The chardev's with 127.0.0.1 IP's loopback to the local qemu
-+instance.
-+
-+# imagefolder="/mnt/vms/colo-test-primary"
-+
-+# qemu-system-x86_64 -enable-kvm -cpu qemu64,+kvmclock -m 512 -smp 1 -qmp stdio \
-+   -device piix3-usb-uhci -device usb-tablet -name primary \
-+   -netdev tap,id=hn0,vhost=off,helper=/usr/lib/qemu/qemu-bridge-helper \
-+   -device rtl8139,id=e0,netdev=hn0 \
-+   -chardev socket,id=mirror0,host=0.0.0.0,port=9003,server,nowait \
-+   -chardev socket,id=compare1,host=0.0.0.0,port=9004,server,wait \
-+   -chardev socket,id=compare0,host=127.0.0.1,port=9001,server,nowait \
-+   -chardev socket,id=compare0-0,host=127.0.0.1,port=9001 \
-+   -chardev socket,id=compare_out,host=127.0.0.1,port=9005,server,nowait \
-+   -chardev socket,id=compare_out0,host=127.0.0.1,port=9005 \
-+   -object filter-mirror,id=m0,netdev=hn0,queue=tx,outdev=mirror0 \
-+   -object filter-redirector,netdev=hn0,id=redire0,queue=rx,indev=compare_out \
-+   -object filter-redirector,netdev=hn0,id=redire1,queue=rx,outdev=compare0 \
-+   -object iothread,id=iothread1 \
-+   -object colo-compare,id=comp0,primary_in=compare0-0,secondary_in=compare1,\
-+outdev=compare_out0,iothread=iothread1 \
-+   -drive if=ide,id=colo-disk0,driver=quorum,read-pattern=fifo,vote-threshold=1,\
-+children.0.file.filename=$imagefolder/primary.qcow2,children.0.driver=qcow2 -S
-+
-+2. Secondary:
-+Note: Active and hidden images need to be created only once and the
-+size should be the same as primary.qcow2. Again, you don't need to change
-+any IP's here, except for the $primary_ip variable.
-+
-+# imagefolder="/mnt/vms/colo-test-secondary"
-+# primary_ip=127.0.0.1
-+
-+# qemu-img create -f qcow2 $imagefolder/secondary-active.qcow2 10G
-+
-+# qemu-img create -f qcow2 $imagefolder/secondary-hidden.qcow2 10G
-+
-+# qemu-system-x86_64 -enable-kvm -cpu qemu64,+kvmclock -m 512 -smp 1 -qmp stdio \
-+   -device piix3-usb-uhci -device usb-tablet -name secondary \
-+   -netdev tap,id=hn0,vhost=off,helper=/usr/lib/qemu/qemu-bridge-helper \
-+   -device rtl8139,id=e0,netdev=hn0 \
-+   -chardev socket,id=red0,host=$primary_ip,port=9003,reconnect=1 \
-+   -chardev socket,id=red1,host=$primary_ip,port=9004,reconnect=1 \
-+   -object filter-redirector,id=f1,netdev=hn0,queue=tx,indev=red0 \
-+   -object filter-redirector,id=f2,netdev=hn0,queue=rx,outdev=red1 \
-+   -object filter-rewriter,id=rew0,netdev=hn0,queue=all \
-+   -drive if=none,id=parent0,file.filename=$imagefolder/primary.qcow2,driver=qcow2 \
-+   -drive if=none,id=childs0,driver=replication,mode=secondary,file.driver=qcow2,\
-+top-id=colo-disk0,file.file.filename=$imagefolder/secondary-active.qcow2,\
-+file.backing.driver=qcow2,file.backing.file.filename=$imagefolder/secondary-hidden.qcow2,\
-+file.backing.backing=parent0 \
-+   -drive if=ide,id=colo-disk0,driver=quorum,read-pattern=fifo,vote-threshold=1,\
-+children.0=childs0 \
-+   -incoming tcp:0.0.0.0:9998
-+
-+
-+3. On Secondary VM's QEMU monitor, issue command
- {'execute':'qmp_capabilities'}
--{ 'execute': 'nbd-server-start',
--  'arguments': {'addr': {'type': 'inet', 'data': {'host': 'xx.xx.xx.xx', 'port': '8889'} } }
--}
--{'execute': 'nbd-server-add', 'arguments': {'device': 'secondary-disk0', 'writable': true } }
-+{'execute': 'nbd-server-start', 'arguments': {'addr': {'type': 'inet', 'data': {'host': '0.0.0.0', 'port': '9999'} } } }
-+{'execute': 'nbd-server-add', 'arguments': {'device': 'parent0', 'writable': true } }
- Note:
-   a. The qmp command nbd-server-start and nbd-server-add must be run
-      before running the qmp command migrate on primary QEMU
-   b. Active disk, hidden disk and nbd target's length should be the
-      same.
--  c. It is better to put active disk and hidden disk in ramdisk.
-+  c. It is better to put active disk and hidden disk in ramdisk. They
-+     will be merged into the parent disk on failover.
--3. On Primary VM's QEMU monitor, issue command:
-+4. On Primary VM's QEMU monitor, issue command:
- {'execute':'qmp_capabilities'}
--{ 'execute': 'human-monitor-command',
--  'arguments': {'command-line': 'drive_add -n buddy driver=replication,mode=primary,file.driver=nbd,file.host=xx.xx.xx.xx,file.port=8889,file.export=secondary-disk0,node-name=nbd_client0'}}
--{ 'execute':'x-blockdev-change', 'arguments':{'parent': 'primary-disk0', 'node': 'nbd_client0' } }
--{ 'execute': 'migrate-set-capabilities',
--      'arguments': {'capabilities': [ {'capability': 'x-colo', 'state': true } ] } }
--{ 'execute': 'migrate', 'arguments': {'uri': 'tcp:xx.xx.xx.xx:8888' } }
-+{'execute': 'human-monitor-command', 'arguments': {'command-line': 'drive_add -n buddy driver=replication,mode=primary,file.driver=nbd,file.host=127.0.0.2,file.port=9999,file.export=parent0,node-name=replication0'}}
-+{'execute': 'x-blockdev-change', 'arguments':{'parent': 'colo-disk0', 'node': 'replication0' } }
-+{'execute': 'migrate-set-capabilities', 'arguments': {'capabilities': [ {'capability': 'x-colo', 'state': true } ] } }
-+{'execute': 'migrate', 'arguments': {'uri': 'tcp:127.0.0.2:9998' } }
-   Note:
-   a. There should be only one NBD Client for each primary disk.
--  b. xx.xx.xx.xx is the secondary physical machine's hostname or IP
--  c. The qmp command line must be run after running qmp command line in
-+  b. The qmp command line must be run after running qmp command line in
-      secondary qemu.
--4. After the above steps, you will see, whenever you make changes to PVM, SVM will be synced.
-+5. After the above steps, you will see, whenever you make changes to PVM, SVM will be synced.
- You can issue command '{ "execute": "migrate-set-parameters" , "arguments":{ "x-checkpoint-delay": 2000 } }'
--to change the checkpoint period time
-+to change the idle checkpoint period time
-+
-+6. Failover test
-+You can kill one of the VMs and Failover on the surviving VM:
-+
-+If you killed the Secondary, then follow "Primary Failover". After that,
-+if you want to resume the replication, follow "Primary resume replication"
-+
-+If you killed the Primary, then follow "Secondary Failover". After that,
-+if you want to resume the replication, follow "Secondary resume replication"
-+
-+== Primary Failover ==
-+The Secondary died, resume on the Primary
-+
-+{'execute': 'x-blockdev-change', 'arguments':{ 'parent': 'colo-disk0', 'child': 'children.1'} }
-+{'execute': 'human-monitor-command', 'arguments':{ 'command-line': 'drive_del replication0' } }
-+{'execute': 'object-del', 'arguments':{ 'id': 'comp0' } }
-+{'execute': 'object-del', 'arguments':{ 'id': 'iothread1' } }
-+{'execute': 'object-del', 'arguments':{ 'id': 'm0' } }
-+{'execute': 'object-del', 'arguments':{ 'id': 'redire0' } }
-+{'execute': 'object-del', 'arguments':{ 'id': 'redire1' } }
-+{'execute': 'x-colo-lost-heartbeat' }
-+
-+== Secondary Failover ==
-+The Primary died, resume on the Secondary and prepare to become the new Primary
-+
-+{'execute': 'nbd-server-stop'}
-+{'execute': 'x-colo-lost-heartbeat'}
-+
-+{'execute': 'object-del', 'arguments':{ 'id': 'f2' } }
-+{'execute': 'object-del', 'arguments':{ 'id': 'f1' } }
-+{'execute': 'chardev-remove', 'arguments':{ 'id': 'red1' } }
-+{'execute': 'chardev-remove', 'arguments':{ 'id': 'red0' } }
-+
-+{'execute': 'chardev-add', 'arguments':{ 'id': 'mirror0', 'backend': {'type': 'socket', 'data': {'addr': { 'type': 'inet', 'data': { 'host': '0.0.0.0', 'port': '9003' } }, 'server': true } } } }
-+{'execute': 'chardev-add', 'arguments':{ 'id': 'compare1', 'backend': {'type': 'socket', 'data': {'addr': { 'type': 'inet', 'data': { 'host': '0.0.0.0', 'port': '9004' } }, 'server': true } } } }
-+{'execute': 'chardev-add', 'arguments':{ 'id': 'compare0', 'backend': {'type': 'socket', 'data': {'addr': { 'type': 'inet', 'data': { 'host': '127.0.0.1', 'port': '9001' } }, 'server': true } } } }
-+{'execute': 'chardev-add', 'arguments':{ 'id': 'compare0-0', 'backend': {'type': 'socket', 'data': {'addr': { 'type': 'inet', 'data': { 'host': '127.0.0.1', 'port': '9001' } }, 'server': false } } } }
-+{'execute': 'chardev-add', 'arguments':{ 'id': 'compare_out', 'backend': {'type': 'socket', 'data': {'addr': { 'type': 'inet', 'data': { 'host': '127.0.0.1', 'port': '9005' } }, 'server': true } } } }
-+{'execute': 'chardev-add', 'arguments':{ 'id': 'compare_out0', 'backend': {'type': 'socket', 'data': {'addr': { 'type': 'inet', 'data': { 'host': '127.0.0.1', 'port': '9005' } }, 'server': false } } } }
-+
-+== Primary resume replication ==
-+Resume replication after new Secondary is up.
-+
-+Start the new Secondary (Steps 2 and 3 above), then on the Primary:
-+{'execute': 'drive-mirror', 'arguments':{ 'device': 'colo-disk0', 'job-id': 'resync', 'target': 'nbd://127.0.0.2:9999/parent0', 'mode': 'existing', 'format': 'raw', 'sync': 'full'} }
-+
-+Wait until disk is synced, then:
-+{'execute': 'stop'}
-+{'execute': 'block-job-cancel', 'arguments':{ 'device': 'resync'} }
-+
-+{'execute': 'human-monitor-command', 'arguments':{ 'command-line': 'drive_add -n buddy driver=replication,mode=primary,file.driver=nbd,file.host=127.0.0.2,file.port=9999,file.export=parent0,node-name=replication0'}}
-+{'execute': 'x-blockdev-change', 'arguments':{ 'parent': 'colo-disk0', 'node': 'replication0' } }
-+
-+{'execute': 'object-add', 'arguments':{ 'qom-type': 'filter-mirror', 'id': 'm0', 'props': { 'netdev': 'hn0', 'queue': 'tx', 'outdev': 'mirror0' } } }
-+{'execute': 'object-add', 'arguments':{ 'qom-type': 'filter-redirector', 'id': 'redire0', 'props': { 'netdev': 'hn0', 'queue': 'rx', 'indev': 'compare_out' } } }
-+{'execute': 'object-add', 'arguments':{ 'qom-type': 'filter-redirector', 'id': 'redire1', 'props': { 'netdev': 'hn0', 'queue': 'rx', 'outdev': 'compare0' } } }
-+{'execute': 'object-add', 'arguments':{ 'qom-type': 'iothread', 'id': 'iothread1' } }
-+{'execute': 'object-add', 'arguments':{ 'qom-type': 'colo-compare', 'id': 'comp0', 'props': { 'primary_in': 'compare0-0', 'secondary_in': 'compare1', 'outdev': 'compare_out0', 'iothread': 'iothread1' } } }
-+
-+{'execute': 'migrate-set-capabilities', 'arguments':{ 'capabilities': [ {'capability': 'x-colo', 'state': true } ] } }
-+{'execute': 'migrate', 'arguments':{ 'uri': 'tcp:127.0.0.2:9998' } }
-+
-+Note:
-+If this Primary previously was a Secondary, then we need to insert the
-+filters before the filter-rewriter by using the
-+"'insert': 'before', 'position': 'id=rew0'" Options. See below.
-+
-+== Secondary resume replication ==
-+Become Primary and resume replication after new Secondary is up. Note
-+that now 127.0.0.1 is the Secondary and 127.0.0.2 is the Primary.
-+
-+Start the new Secondary (Steps 2 and 3 above, but with primary_ip=127.0.0.2),
-+then on the old Secondary:
-+{'execute': 'drive-mirror', 'arguments':{ 'device': 'colo-disk0', 'job-id': 'resync', 'target': 'nbd://127.0.0.1:9999/parent0', 'mode': 'existing', 'format': 'raw', 'sync': 'full'} }
-+
-+Wait until disk is synced, then:
-+{'execute': 'stop'}
-+{'execute': 'block-job-cancel', 'arguments':{ 'device': 'resync' } }
--5. Failover test
--You can kill Primary VM and run 'x_colo_lost_heartbeat' in Secondary VM's
--monitor at the same time, then SVM will failover and client will not detect this
--change.
-+{'execute': 'human-monitor-command', 'arguments':{ 'command-line': 'drive_add -n buddy driver=replication,mode=primary,file.driver=nbd,file.host=127.0.0.1,file.port=9999,file.export=parent0,node-name=replication0'}}
-+{'execute': 'x-blockdev-change', 'arguments':{ 'parent': 'colo-disk0', 'node': 'replication0' } }
--Before issuing '{ "execute": "x-colo-lost-heartbeat" }' command, we have to
--issue block related command to stop block replication.
--Primary:
--  Remove the nbd child from the quorum:
--  { 'execute': 'x-blockdev-change', 'arguments': {'parent': 'colo-disk0', 'child': 'children.1'}}
--  { 'execute': 'human-monitor-command','arguments': {'command-line': 'drive_del blk-buddy0'}}
--  Note: there is no qmp command to remove the blockdev now
-+{'execute': 'object-add', 'arguments':{ 'qom-type': 'filter-mirror', 'id': 'm0', 'props': { 'insert': 'before', 'position': 'id=rew0', 'netdev': 'hn0', 'queue': 'tx', 'outdev': 'mirror0' } } }
-+{'execute': 'object-add', 'arguments':{ 'qom-type': 'filter-redirector', 'id': 'redire0', 'props': { 'insert': 'before', 'position': 'id=rew0', 'netdev': 'hn0', 'queue': 'rx', 'indev': 'compare_out' } } }
-+{'execute': 'object-add', 'arguments':{ 'qom-type': 'filter-redirector', 'id': 'redire1', 'props': { 'insert': 'before', 'position': 'id=rew0', 'netdev': 'hn0', 'queue': 'rx', 'outdev': 'compare0' } } }
-+{'execute': 'object-add', 'arguments':{ 'qom-type': 'iothread', 'id': 'iothread1' } }
-+{'execute': 'object-add', 'arguments':{ 'qom-type': 'colo-compare', 'id': 'comp0', 'props': { 'primary_in': 'compare0-0', 'secondary_in': 'compare1', 'outdev': 'compare_out0', 'iothread': 'iothread1' } } }
--Secondary:
--  The primary host is down, so we should do the following thing:
--  { 'execute': 'nbd-server-stop' }
-+{'execute': 'migrate-set-capabilities', 'arguments':{ 'capabilities': [ {'capability': 'x-colo', 'state': true } ] } }
-+{'execute': 'migrate', 'arguments':{ 'uri': 'tcp:127.0.0.1:9998' } }
- == TODO ==
--1. Support continuous VM replication.
--2. Support shared storage.
--3. Develop the heartbeat part.
--4. Reduce checkpoint VM’s downtime while doing checkpoint.
-+1. Support shared storage.
-+2. Develop the heartbeat part.
-+3. Reduce checkpoint VM’s downtime while doing checkpoint.
-diff --git a/docs/block-replication.txt b/docs/block-replication.txt
-index XXXXXXX..XXXXXXX 100644
---- a/docs/block-replication.txt
-+++ b/docs/block-replication.txt
-@@ -XXX,XX +XXX,XX @@ blocks that are already in QEMU.
-              ^            ||                            .----------
-              |            ||                            | Secondary
-Quorum          ||                            '----------
--         /      \         ||
--        /        \        ||
--   Primary    2 filter
--     disk         ^                                                             virtio-blk
--                  |                                                                  ^
--                3 NBD  ------->  3 NBD                                               |
-+         /      \         ||                                                           virtio-blk
-+        /        \        ||                                                               ^
-+   Primary    2 filter                                                                     |
-+     disk         ^                                                                   7 Quorum
-+                  |                                                                    /
-+                3 NBD  ------->  3 NBD                                                /
-                 client    ||     server                                          2 filter
-                           ||        ^                                                ^
- --------.                 ||        |                                                |
-@@ -XXX,XX +XXX,XX @@ any state that would otherwise be lost by the speculative write-through
- of the NBD server into the secondary disk. So before block replication,
- the primary disk and secondary disk should contain the same data.
-+7) The secondary also has a quorum node, so after secondary failover it
-+can become the new primary and continue replication.
-+
-+
- == Failure Handling ==
- There are 7 internal errors when block replication is running:
-. I/O error on primary disk
-@@ -XXX,XX +XXX,XX @@ Primary:
-      leading whitespace.
-. The qmp command line must be run after running qmp command line in
-      secondary qemu.
--  6. After failover we need remove children.1 (replication driver).
-+  6. After primary failover we need remove children.1 (replication driver).
- Secondary:
-   -drive if=none,driver=raw,file.filename=1.raw,id=colo1 \
--  -drive if=xxx,id=topxxx,driver=replication,mode=secondary,top-id=topxxx\
-+  -drive if=none,id=childs1,driver=replication,mode=secondary,top-id=childs1
-          file.file.filename=active_disk.qcow2,\
-          file.driver=qcow2,\
-          file.backing.file.filename=hidden_disk.qcow2,\
-          file.backing.driver=qcow2,\
-          file.backing.backing=colo1
-+  -drive if=xxx,driver=quorum,read-pattern=fifo,id=top-disk1,\
-+         vote-threshold=1,children.0=childs1
-   Then run qmp command in secondary qemu:
-     { 'execute': 'nbd-server-start',
-@@ -XXX,XX +XXX,XX @@ Secondary:
-   The primary host is down, so we should do the following thing:
-   { 'execute': 'nbd-server-stop' }
-+Promote Secondary to Primary:
-+  see COLO-FT.txt
-+
- TODO:
--1. Continuous block replication
--2. Shared disk
-+1. Shared disk
---
-.5.0

-[PULL 23/23] l2tpv3: fix RFC number typo in qemu-options.hx
+[PULL V2 3/3] tap-bsd: Remove special casing for older OpenBSD releases
-From: Stefan Hajnoczi <stefanha@redhat.com>
+From: Brad Smith <brad@comstyle.com>
-The L2TPv3 RFC number is 3931:
+OpenBSD added support for tap(4) 10 releases ago.
 https://tools.ietf.org/html/rfc3931
-Reported-by: Henrik Johansson <henrikjohansson@rocketmail.com>
+Remove the special casing for older releases.
-Reviewed-by: Stefan Weil <sw@weilnetz.de>
-Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
+Signed-off-by: Brad Smith <brad@comstyle.com>
 Signed-off-by: Jason Wang <jasowang@redhat.com>
 ---
- qemu-options.hx | 4 ++--
+ net/tap-bsd.c | 8 --------
-file changed, 2 insertions(+), 2 deletions(-)
+file changed, 8 deletions(-)
-diff --git a/qemu-options.hx b/qemu-options.hx
+diff --git a/net/tap-bsd.c b/net/tap-bsd.c
 index XXXXXXX..XXXXXXX 100644
---- a/qemu-options.hx
+--- a/net/tap-bsd.c
-+++ b/qemu-options.hx
++++ b/net/tap-bsd.c
-@@ -XXX,XX +XXX,XX @@ DEF("netdev", HAS_ARG, QEMU_OPTION_netdev,
+@@ -XXX,XX +XXX,XX @@
-     "                Linux kernel 3.3+ as well as most routers can talk\n"
+ #include <net/if_tap.h>
-     "                L2TPv3. This transport allows connecting a VM to a VM,\n"
+ #endif
-     "                VM to a router and even VM to Host. It is a nearly-universal\n"
--    "                standard (RFC3391). Note - this implementation uses static\n"
+-#if defined(__OpenBSD__)
-+    "                standard (RFC3931). Note - this implementation uses static\n"
+-#include <sys/param.h>
-     "                pre-configured tunnels (same as the Linux kernel).\n"
+-#endif
-     "                use 'src=' to specify source address\n"
+-
-     "                use 'dst=' to specify destination address\n"
+ #ifndef __FreeBSD__
-@@ -XXX,XX +XXX,XX @@ Example (send packets from host's 1.2.3.4):
+ int tap_open(char *ifname, int ifname_size, int *vnet_hdr,
- @end example
+              int vnet_hdr_required, int mq_required, Error **errp)
+@@ -XXX,XX +XXX,XX @@ int tap_open(char *ifname, int ifname_size, int *vnet_hdr,
- @item -netdev l2tpv3,id=@var{id},src=@var{srcaddr},dst=@var{dstaddr}[,srcport=@var{srcport}][,dstport=@var{dstport}],txsession=@var{txsession}[,rxsession=@var{rxsession}][,ipv6][,udp][,cookie64][,counter][,pincounter][,txcookie=@var{txcookie}][,rxcookie=@var{rxcookie}][,offset=@var{offset}]
+         if (*ifname) {
--Configure a L2TPv3 pseudowire host network backend. L2TPv3 (RFC3391) is a
+             snprintf(dname, sizeof dname, "/dev/%s", ifname);
-+Configure a L2TPv3 pseudowire host network backend. L2TPv3 (RFC3931) is a
+         } else {
- popular protocol to transport Ethernet (and other Layer 2) data frames between
+-#if defined(__OpenBSD__) && OpenBSD < 201605
- two systems. It is present in routers, firewalls and the Linux kernel
+-            snprintf(dname, sizeof dname, "/dev/tun%d", i);
- (from version 3.3 onwards).
+-#else
              snprintf(dname, sizeof dname, "/dev/tap%d", i);
 -#endif
          }
          TFR(fd = open(dname, O_RDWR));
          if (fd >= 0) {
 --
-.5.0
+.7.4

The following changes since commit e0175b71638cf4398903c0d25f93fe62e0606389:

Merge remote-tracking branch 'remotes/pmaydell/tags/pull-target-arm-20200228' into staging (2020-02-28 16:39:27 +0000)

are available in the git repository at:

https://github.com/jasowang/qemu.git tags/net-pull-request

for you to fetch changes up to 41aa2e3f9b27fd259a13711545d933a20f1d2f16:

l2tpv3: fix RFC number typo in qemu-options.hx (2020-03-02 15:30:08 +0800)

----------------------------------------------------------------

----------------------------------------------------------------
Bin Meng (1):
      hw: net: cadence_gem: Fix build errors in DB_PRINT()

Finn Thain (14):
      dp8393x: Mask EOL bit from descriptor addresses
      dp8393x: Always use 32-bit accesses
      dp8393x: Clean up endianness hacks
      dp8393x: Have dp8393x_receive() return the packet size
      dp8393x: Update LLFA and CRDA registers from rx descriptor
      dp8393x: Clear RRRA command register bit only when appropriate
      dp8393x: Implement packet size limit and RBAE interrupt
      dp8393x: Don't clobber packet checksum
      dp8393x: Use long-word-aligned RRA pointers in 32-bit mode
      dp8393x: Pad frames to word or long word boundary
      dp8393x: Clear descriptor in_use field to release packet
      dp8393x: Always update RRA pointers and sequence numbers
      dp8393x: Don't reset Silicon Revision register
      dp8393x: Don't stop reception upon RBE interrupt assertion

Lukas Straub (4):
      block/replication.c: Ignore requests after failover
      tests/test-replication.c: Add test for for secondary node continuing replication
      net/filter.c: Add Options to insert filters anywhere in the filter list
      colo: Update Documentation for continuous replication

Stefan Hajnoczi (1):
      l2tpv3: fix RFC number typo in qemu-options.hx

Yuri Benditovich (3):
      e1000e: Avoid hw_error if legacy mode used
      NetRxPkt: Introduce support for additional hash types
      NetRxPkt: fix hash calculation of IPV6 TCP

From: Finn Thain <fthain@telegraphics.com.au>

The Least Significant bit of a descriptor address register is used as
an EOL flag. It has to be masked when the register value is to be used
as an actual address for copying memory around. But when the registers
are to be updated the EOL bit should not be masked.

Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
Tested-by: Laurent Vivier <laurent@vivier.eu>
---
 hw/net/dp8393x.c | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/dp8393x.c
+++ b/hw/net/dp8393x.c
@@ -XXX,XX +XXX,XX @@ do { printf("sonic ERROR: %s: " fmt, __func__ , ## __VA_ARGS__); } while (0)
 #define SONIC_ISR_PINT   0x0800
 #define SONIC_ISR_LCD    0x1000
 
+#define SONIC_DESC_EOL   0x0001
+#define SONIC_DESC_ADDR  0xFFFE
+
 #define TYPE_DP8393X "dp8393x"
 #define DP8393X(obj) OBJECT_CHECK(dp8393xState, (obj), TYPE_DP8393X)
 
@@ -XXX,XX +XXX,XX @@ static uint32_t dp8393x_crba(dp8393xState *s)
 
 static uint32_t dp8393x_crda(dp8393xState *s)
 {
-    return (s->regs[SONIC_URDA] << 16) | s->regs[SONIC_CRDA];
+    return (s->regs[SONIC_URDA] << 16) |
+           (s->regs[SONIC_CRDA] & SONIC_DESC_ADDR);
 }
 
 static uint32_t dp8393x_rbwc(dp8393xState *s)
@@ -XXX,XX +XXX,XX @@ static uint32_t dp8393x_tsa(dp8393xState *s)
 
 static uint32_t dp8393x_ttda(dp8393xState *s)
 {
-    return (s->regs[SONIC_UTDA] << 16) | s->regs[SONIC_TTDA];
+    return (s->regs[SONIC_UTDA] << 16) |
+           (s->regs[SONIC_TTDA] & SONIC_DESC_ADDR);
 }
 
 static uint32_t dp8393x_wt(dp8393xState *s)
@@ -XXX,XX +XXX,XX @@ static void dp8393x_do_transmit_packets(dp8393xState *s)
                                MEMTXATTRS_UNSPECIFIED, s->data,
                                size);
             s->regs[SONIC_CTDA] = dp8393x_get(s, width, 0) & ~0x1;
-            if (dp8393x_get(s, width, 0) & 0x1) {
+            if (dp8393x_get(s, width, 0) & SONIC_DESC_EOL) {
                 /* EOL detected */
                 break;
             }
@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
     /* XXX: Check byte ordering */
 
     /* Check for EOL */
-    if (s->regs[SONIC_LLFA] & 0x1) {
+    if (s->regs[SONIC_LLFA] & SONIC_DESC_EOL) {
         /* Are we still in resource exhaustion? */
         size = sizeof(uint16_t) * 1 * width;
         address = dp8393x_crda(s) + sizeof(uint16_t) * 5 * width;
         address_space_read(&s->as, address, MEMTXATTRS_UNSPECIFIED,
                            s->data, size);
-        if (dp8393x_get(s, width, 0) & 0x1) {
+        if (dp8393x_get(s, width, 0) & SONIC_DESC_EOL) {
             /* Still EOL ; stop reception */
             return -1;
         } else {
@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
                        dp8393x_crda(s) + sizeof(uint16_t) * 5 * width,
                        MEMTXATTRS_UNSPECIFIED, s->data, size);
     s->regs[SONIC_LLFA] = dp8393x_get(s, width, 0);
-    if (s->regs[SONIC_LLFA] & 0x1) {
+    if (s->regs[SONIC_LLFA] & SONIC_DESC_EOL) {
         /* EOL detected */
         s->regs[SONIC_ISR] |= SONIC_ISR_RDE;
     } else {
-- 
2.5.0

From: Finn Thain <fthain@telegraphics.com.au>

The DP83932 and DP83934 have 32 data lines. The datasheet says,

Data Bus: These bidirectional lines are used to transfer data on the
    system bus. When the SONIC is a bus master, 16-bit data is transferred
    on D15-D0 and 32-bit data is transferred on D31-D0. When the SONIC is
    accessed as a slave, register data is driven onto lines D15-D0.
    D31-D16 are held TRI-STATE if SONIC is in 16-bit mode. If SONIC is in
    32-bit mode, they are driven, but invalid.

Always use 32-bit accesses both as bus master and bus slave.

Force the MSW to zero in bus master mode.

This gets the Linux 'jazzsonic' driver working, and avoids the need for
prior hacks to make the NetBSD 'sn' driver work.

Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
Tested-by: Laurent Vivier <laurent@vivier.eu>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 hw/net/dp8393x.c | 47 +++++++++++++++++++++++++++++------------------
 1 file changed, 29 insertions(+), 18 deletions(-)

diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/dp8393x.c
+++ b/hw/net/dp8393x.c
@@ -XXX,XX +XXX,XX @@ static void dp8393x_put(dp8393xState *s, int width, int offset,
                         uint16_t val)
 {
     if (s->big_endian) {
-        s->data[offset * width + width - 1] = cpu_to_be16(val);
+        if (width == 2) {
+            s->data[offset * 2] = 0;
+            s->data[offset * 2 + 1] = cpu_to_be16(val);
+        } else {
+            s->data[offset] = cpu_to_be16(val);
+        }
     } else {
-        s->data[offset * width] = cpu_to_le16(val);
+        if (width == 2) {
+            s->data[offset * 2] = cpu_to_le16(val);
+            s->data[offset * 2 + 1] = 0;
+        } else {
+            s->data[offset] = cpu_to_le16(val);
+        }
     }
 }
 
@@ -XXX,XX +XXX,XX @@ static uint64_t dp8393x_read(void *opaque, hwaddr addr, unsigned int size)
 
     DPRINTF("read 0x%04x from reg %s\n", val, reg_names[reg]);
 
-    return val;
+    return s->big_endian ? val << 16 : val;
 }
 
 static void dp8393x_write(void *opaque, hwaddr addr, uint64_t data,
@@ -XXX,XX +XXX,XX @@ static void dp8393x_write(void *opaque, hwaddr addr, uint64_t data,
 {
     dp8393xState *s = opaque;
     int reg = addr >> s->it_shift;
+    uint32_t val = s->big_endian ? data >> 16 : data;
 
-    DPRINTF("write 0x%04x to reg %s\n", (uint16_t)data, reg_names[reg]);
+    DPRINTF("write 0x%04x to reg %s\n", (uint16_t)val, reg_names[reg]);
 
     switch (reg) {
         /* Command register */
         case SONIC_CR:
-            dp8393x_do_command(s, data);
+            dp8393x_do_command(s, val);
             break;
         /* Prevent write to read-only registers */
         case SONIC_CAP2:
@@ -XXX,XX +XXX,XX @@ static void dp8393x_write(void *opaque, hwaddr addr, uint64_t data,
         /* Accept write to some registers only when in reset mode */
         case SONIC_DCR:
             if (s->regs[SONIC_CR] & SONIC_CR_RST) {
-                s->regs[reg] = data & 0xbfff;
+                s->regs[reg] = val & 0xbfff;
             } else {
                 DPRINTF("writing to DCR invalid\n");
             }
             break;
         case SONIC_DCR2:
             if (s->regs[SONIC_CR] & SONIC_CR_RST) {
-                s->regs[reg] = data & 0xf017;
+                s->regs[reg] = val & 0xf017;
             } else {
                 DPRINTF("writing to DCR2 invalid\n");
             }
             break;
         /* 12 lower bytes are Read Only */
         case SONIC_TCR:
-            s->regs[reg] = data & 0xf000;
+            s->regs[reg] = val & 0xf000;
             break;
         /* 9 lower bytes are Read Only */
         case SONIC_RCR:
-            s->regs[reg] = data & 0xffe0;
+            s->regs[reg] = val & 0xffe0;
             break;
         /* Ignore most significant bit */
         case SONIC_IMR:
-            s->regs[reg] = data & 0x7fff;
+            s->regs[reg] = val & 0x7fff;
             dp8393x_update_irq(s);
             break;
         /* Clear bits by writing 1 to them */
         case SONIC_ISR:
-            data &= s->regs[reg];
-            s->regs[reg] &= ~data;
-            if (data & SONIC_ISR_RBE) {
+            val &= s->regs[reg];
+            s->regs[reg] &= ~val;
+            if (val & SONIC_ISR_RBE) {
                 dp8393x_do_read_rra(s);
             }
             dp8393x_update_irq(s);
@@ -XXX,XX +XXX,XX @@ static void dp8393x_write(void *opaque, hwaddr addr, uint64_t data,
         case SONIC_REA:
         case SONIC_RRP:
         case SONIC_RWP:
-            s->regs[reg] = data & 0xfffe;
+            s->regs[reg] = val & 0xfffe;
             break;
         /* Invert written value for some registers */
         case SONIC_CRCT:
         case SONIC_FAET:
         case SONIC_MPT:
-            s->regs[reg] = data ^ 0xffff;
+            s->regs[reg] = val ^ 0xffff;
             break;
         /* All other registers have no special contrainst */
         default:
-            s->regs[reg] = data;
+            s->regs[reg] = val;
     }
 
     if (reg == SONIC_WT0 || reg == SONIC_WT1) {
@@ -XXX,XX +XXX,XX @@ static void dp8393x_write(void *opaque, hwaddr addr, uint64_t data,
 static const MemoryRegionOps dp8393x_ops = {
     .read = dp8393x_read,
     .write = dp8393x_write,
-    .impl.min_access_size = 2,
-    .impl.max_access_size = 2,
+    .impl.min_access_size = 4,
+    .impl.max_access_size = 4,
     .endianness = DEVICE_NATIVE_ENDIAN,
 };
 
-- 
2.5.0

From: Finn Thain <fthain@telegraphics.com.au>

According to the datasheet, section 3.4.4, "in 32-bit mode ... the SONIC
always writes long words".

Therefore, use the same technique for the 'in_use' field that is used
everywhere else, and write the full long word.

Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
Tested-by: Laurent Vivier <laurent@vivier.eu>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 hw/net/dp8393x.c | 17 ++++++-----------
 1 file changed, 6 insertions(+), 11 deletions(-)

diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/dp8393x.c
+++ b/hw/net/dp8393x.c
@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
         return -1;
     }
 
-    /* XXX: Check byte ordering */
-
     /* Check for EOL */
     if (s->regs[SONIC_LLFA] & SONIC_DESC_EOL) {
         /* Are we still in resource exhaustion? */
@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
         /* EOL detected */
         s->regs[SONIC_ISR] |= SONIC_ISR_RDE;
     } else {
-        /* Clear in_use, but it is always 16bit wide */
-        int offset = dp8393x_crda(s) + sizeof(uint16_t) * 6 * width;
-        if (s->big_endian && width == 2) {
-            /* we need to adjust the offset of the 16bit field */
-            offset += sizeof(uint16_t);
-        }
-        s->data[0] = 0;
-        address_space_write(&s->as, offset, MEMTXATTRS_UNSPECIFIED,
-                            s->data, sizeof(uint16_t));
+        /* Clear in_use */
+        size = sizeof(uint16_t) * width;
+        address = dp8393x_crda(s) + sizeof(uint16_t) * 6 * width;
+        dp8393x_put(s, width, 0, 0);
+        address_space_write(&s->as, address, MEMTXATTRS_UNSPECIFIED,
+                            s->data, size);
         s->regs[SONIC_CRDA] = s->regs[SONIC_LLFA];
         s->regs[SONIC_ISR] |= SONIC_ISR_PKTRX;
         s->regs[SONIC_RSC] = (s->regs[SONIC_RSC] & 0xff00) | (((s->regs[SONIC_RSC] & 0x00ff) + 1) & 0x00ff);
-- 
2.5.0

From: Finn Thain <fthain@telegraphics.com.au>

This function re-uses its 'size' argument as a scratch variable.
Instead, declare a local 'size' variable for that purpose so that the
function result doesn't get messed up.

Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Tested-by: Laurent Vivier <laurent@vivier.eu>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 hw/net/dp8393x.c | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/dp8393x.c
+++ b/hw/net/dp8393x.c
@@ -XXX,XX +XXX,XX @@ static int dp8393x_receive_filter(dp8393xState *s, const uint8_t * buf,
 }
 
 static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
-                               size_t size)
+                               size_t pkt_size)
 {
     dp8393xState *s = qemu_get_nic_opaque(nc);
     int packet_type;
     uint32_t available, address;
-    int width, rx_len = size;
+    int width, rx_len = pkt_size;
     uint32_t checksum;
+    int size;
 
     width = (s->regs[SONIC_DCR] & SONIC_DCR_DW) ? 2 : 1;
 
     s->regs[SONIC_RCR] &= ~(SONIC_RCR_PRX | SONIC_RCR_LBK | SONIC_RCR_FAER |
         SONIC_RCR_CRCR | SONIC_RCR_LPKT | SONIC_RCR_BC | SONIC_RCR_MC);
 
-    packet_type = dp8393x_receive_filter(s, buf, size);
+    packet_type = dp8393x_receive_filter(s, buf, pkt_size);
     if (packet_type < 0) {
         DPRINTF("packet not for netcard\n");
         return -1;
@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
     /* Done */
     dp8393x_update_irq(s);
 
-    return size;
+    return pkt_size;
 }
 
 static void dp8393x_reset(DeviceState *dev)
-- 
2.5.0

From: Finn Thain <fthain@telegraphics.com.au>

Follow the algorithm given in the National Semiconductor DP83932C
datasheet in section 3.4.7:

At the next reception, the SONIC re-reads the last RXpkt.link field,
    and updates its CRDA register to point to the next descriptor.

The chip is designed to allow the host to provide a new list of
descriptors in this way.

Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
Tested-by: Laurent Vivier <laurent@vivier.eu>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 hw/net/dp8393x.c | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/dp8393x.c
+++ b/hw/net/dp8393x.c
@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
         address = dp8393x_crda(s) + sizeof(uint16_t) * 5 * width;
         address_space_read(&s->as, address, MEMTXATTRS_UNSPECIFIED,
                            s->data, size);
-        if (dp8393x_get(s, width, 0) & SONIC_DESC_EOL) {
+        s->regs[SONIC_LLFA] = dp8393x_get(s, width, 0);
+        if (s->regs[SONIC_LLFA] & SONIC_DESC_EOL) {
             /* Still EOL ; stop reception */
             return -1;
-        } else {
-            s->regs[SONIC_CRDA] = s->regs[SONIC_LLFA];
         }
+        /* Link has been updated by host */
+        s->regs[SONIC_CRDA] = s->regs[SONIC_LLFA];
     }
 
     /* Save current position */
@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
                         MEMTXATTRS_UNSPECIFIED,
                         s->data, size);
 
-    /* Move to next descriptor */
+    /* Check link field */
     size = sizeof(uint16_t) * width;
     address_space_read(&s->as,
                        dp8393x_crda(s) + sizeof(uint16_t) * 5 * width,
@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
         dp8393x_put(s, width, 0, 0);
         address_space_write(&s->as, address, MEMTXATTRS_UNSPECIFIED,
                             s->data, size);
+
+        /* Move to next descriptor */
         s->regs[SONIC_CRDA] = s->regs[SONIC_LLFA];
         s->regs[SONIC_ISR] |= SONIC_ISR_PKTRX;
         s->regs[SONIC_RSC] = (s->regs[SONIC_RSC] & 0xff00) | (((s->regs[SONIC_RSC] & 0x00ff) + 1) & 0x00ff);
-- 
2.5.0

From: Finn Thain <fthain@telegraphics.com.au>

It doesn't make sense to clear the command register bit unless the
command was actually issued.

Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Tested-by: Laurent Vivier <laurent@vivier.eu>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 hw/net/dp8393x.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/dp8393x.c
+++ b/hw/net/dp8393x.c
@@ -XXX,XX +XXX,XX @@ static void dp8393x_do_read_rra(dp8393xState *s)
         s->regs[SONIC_ISR] |= SONIC_ISR_RBE;
         dp8393x_update_irq(s);
     }
-
-    /* Done */
-    s->regs[SONIC_CR] &= ~SONIC_CR_RRRA;
 }
 
 static void dp8393x_do_software_reset(dp8393xState *s)
@@ -XXX,XX +XXX,XX @@ static void dp8393x_do_command(dp8393xState *s, uint16_t command)
         dp8393x_do_start_timer(s);
     if (command & SONIC_CR_RST)
         dp8393x_do_software_reset(s);
-    if (command & SONIC_CR_RRRA)
+    if (command & SONIC_CR_RRRA) {
         dp8393x_do_read_rra(s);
+        s->regs[SONIC_CR] &= ~SONIC_CR_RRRA;
+    }
     if (command & SONIC_CR_LCAM)
         dp8393x_do_load_cam(s);
 }
-- 
2.5.0

From: Finn Thain <fthain@telegraphics.com.au>

Add a bounds check to prevent a large packet from causing a buffer
overflow. This is defensive programming -- I haven't actually tried
sending an oversized packet or a jumbo ethernet frame.

The SONIC handles packets that are too big for the buffer by raising
the RBAE interrupt and dropping them. Linux uses that interrupt to
count dropped packets.

From: Finn Thain <fthain@telegraphics.com.au>

A received packet consumes pkt_size bytes in the buffer and the frame
checksum that's appended to it consumes another 4 bytes. The Receive
Buffer Address register takes the former quantity into account but
not the latter. So the next packet written to the buffer overwrites
the frame checksum. Fix this.

diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/dp8393x.c
+++ b/hw/net/dp8393x.c
@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
     address += rx_len;
     address_space_write(&s->as, address, MEMTXATTRS_UNSPECIFIED,
                         &checksum, 4);
+    address += 4;
     rx_len += 4;
     s->regs[SONIC_CRBA1] = address >> 16;
     s->regs[SONIC_CRBA0] = address & 0xffff;
-- 
2.5.0

From: Finn Thain <fthain@telegraphics.com.au>

Section 3.4.1 of the datasheet says,

The alignment of the RRA is confined to either word or long word
    boundaries, depending upon the data width mode. In 16-bit mode,
    the RRA must be aligned to a word boundary (A0 is always zero)
    and in 32-bit mode, the RRA is aligned to a long word boundary
    (A0 and A1 are always zero).

This constraint has been implemented for 16-bit mode; implement it
for 32-bit mode too.

Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
Tested-by: Laurent Vivier <laurent@vivier.eu>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 hw/net/dp8393x.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/dp8393x.c
+++ b/hw/net/dp8393x.c
@@ -XXX,XX +XXX,XX @@ static void dp8393x_write(void *opaque, hwaddr addr, uint64_t data,
                 qemu_flush_queued_packets(qemu_get_queue(s->nic));
             }
             break;
-        /* Ignore least significant bit */
+        /* The guest is required to store aligned pointers here */
         case SONIC_RSA:
         case SONIC_REA:
         case SONIC_RRP:
         case SONIC_RWP:
-            s->regs[reg] = val & 0xfffe;
+            if (s->regs[SONIC_DCR] & SONIC_DCR_DW) {
+                s->regs[reg] = val & 0xfffc;
+            } else {
+                s->regs[reg] = val & 0xfffe;
+            }
             break;
         /* Invert written value for some registers */
         case SONIC_CRCT:
-- 
2.5.0

From: Finn Thain <fthain@telegraphics.com.au>

The existing code has a bug where the Remaining Buffer Word Count (RBWC)
is calculated with a truncating division, which gives the wrong result
for odd-sized packets.

Section 1.4.1 of the datasheet says,

Once the end of the packet has been reached, the serializer will
    fill out the last word (16-bit mode) or long word (32-bit mode)
    if the last byte did not end on a word or long word boundary
    respectively. The fill byte will be 0FFh.

Implement buffer padding so that buffer limits are correctly enforced.

Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
Tested-by: Laurent Vivier <laurent@vivier.eu>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 hw/net/dp8393x.c | 39 ++++++++++++++++++++++++++++-----------
 1 file changed, 28 insertions(+), 11 deletions(-)

diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/dp8393x.c
+++ b/hw/net/dp8393x.c
@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
     dp8393xState *s = qemu_get_nic_opaque(nc);
     int packet_type;
     uint32_t available, address;
-    int width, rx_len = pkt_size;
+    int width, rx_len, padded_len;
     uint32_t checksum;
     int size;
 
-    width = (s->regs[SONIC_DCR] & SONIC_DCR_DW) ? 2 : 1;
-
     s->regs[SONIC_RCR] &= ~(SONIC_RCR_PRX | SONIC_RCR_LBK | SONIC_RCR_FAER |
         SONIC_RCR_CRCR | SONIC_RCR_LPKT | SONIC_RCR_BC | SONIC_RCR_MC);
 
-    if (pkt_size + 4 > dp8393x_rbwc(s) * 2) {
+    rx_len = pkt_size + sizeof(checksum);
+    if (s->regs[SONIC_DCR] & SONIC_DCR_DW) {
+        width = 2;
+        padded_len = ((rx_len - 1) | 3) + 1;
+    } else {
+        width = 1;
+        padded_len = ((rx_len - 1) | 1) + 1;
+    }
+
+    if (padded_len > dp8393x_rbwc(s) * 2) {
         DPRINTF("oversize packet, pkt_size is %d\n", pkt_size);
         s->regs[SONIC_ISR] |= SONIC_ISR_RBAE;
         dp8393x_update_irq(s);
@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
     s->regs[SONIC_TRBA0] = s->regs[SONIC_CRBA0];
 
     /* Calculate the ethernet checksum */
-    checksum = cpu_to_le32(crc32(0, buf, rx_len));
+    checksum = cpu_to_le32(crc32(0, buf, pkt_size));
 
     /* Put packet into RBA */
     DPRINTF("Receive packet at %08x\n", dp8393x_crba(s));
     address = dp8393x_crba(s);
     address_space_write(&s->as, address, MEMTXATTRS_UNSPECIFIED,
-                        buf, rx_len);
-    address += rx_len;
+                        buf, pkt_size);
+    address += pkt_size;
+
+    /* Put frame checksum into RBA */
     address_space_write(&s->as, address, MEMTXATTRS_UNSPECIFIED,
-                        &checksum, 4);
-    address += 4;
-    rx_len += 4;
+                        &checksum, sizeof(checksum));
+    address += sizeof(checksum);
+
+    /* Pad short packets to keep pointers aligned */
+    if (rx_len < padded_len) {
+        size = padded_len - rx_len;
+        address_space_rw(&s->as, address, MEMTXATTRS_UNSPECIFIED,
+            (uint8_t *)"\xFF\xFF\xFF", size, 1);
+        address += size;
+    }
+
     s->regs[SONIC_CRBA1] = address >> 16;
     s->regs[SONIC_CRBA0] = address & 0xffff;
     available = dp8393x_rbwc(s);
-    available -= rx_len / 2;
+    available -= padded_len >> 1;
     s->regs[SONIC_RBWC1] = available >> 16;
     s->regs[SONIC_RBWC0] = available & 0xffff;
 
-- 
2.5.0

From: Finn Thain <fthain@telegraphics.com.au>

When the SONIC receives a packet into the last available descriptor, it
retains ownership of that descriptor for as long as necessary.

Section 3.4.7 of the datasheet says,

When the system appends more descriptors, the SONIC releases ownership
    of the descriptor after writing 0000h to the RXpkt.in_use field.

The packet can now be processed by the host, so raise a PKTRX interrupt,
just like the normal case.

diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/dp8393x.c
+++ b/hw/net/dp8393x.c
@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
             return -1;
         }
         /* Link has been updated by host */
+
+        /* Clear in_use */
+        size = sizeof(uint16_t) * width;
+        address = dp8393x_crda(s) + sizeof(uint16_t) * 6 * width;
+        dp8393x_put(s, width, 0, 0);
+        address_space_rw(&s->as, address, MEMTXATTRS_UNSPECIFIED,
+                         (uint8_t *)s->data, size, 1);
+
+        /* Move to next descriptor */
         s->regs[SONIC_CRDA] = s->regs[SONIC_LLFA];
+        s->regs[SONIC_ISR] |= SONIC_ISR_PKTRX;
     }
 
     /* Save current position */
-- 
2.5.0

From: Finn Thain <fthain@telegraphics.com.au>

These operations need to take place regardless of whether or not
rx descriptors have been used up (that is, EOL flag was observed).

The algorithm is now the same for a packet that was withheld as for
a packet that was not.

Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
Tested-by: Laurent Vivier <laurent@vivier.eu>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 hw/net/dp8393x.c | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/dp8393x.c
+++ b/hw/net/dp8393x.c
@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
         /* Move to next descriptor */
         s->regs[SONIC_CRDA] = s->regs[SONIC_LLFA];
         s->regs[SONIC_ISR] |= SONIC_ISR_PKTRX;
-        s->regs[SONIC_RSC] = (s->regs[SONIC_RSC] & 0xff00) | (((s->regs[SONIC_RSC] & 0x00ff) + 1) & 0x00ff);
+    }
 
-        if (s->regs[SONIC_RCR] & SONIC_RCR_LPKT) {
-            /* Read next RRA */
-            dp8393x_do_read_rra(s);
-        }
+    s->regs[SONIC_RSC] = (s->regs[SONIC_RSC] & 0xff00) |
+                         ((s->regs[SONIC_RSC] + 1) & 0x00ff);
+
+    if (s->regs[SONIC_RCR] & SONIC_RCR_LPKT) {
+        /* Read next RRA */
+        dp8393x_do_read_rra(s);
     }
 
     /* Done */
-- 
2.5.0

From: Finn Thain <fthain@telegraphics.com.au>

The jazzsonic driver in Linux uses the Silicon Revision register value
to probe the chip. The driver fails unless the SR register contains 4.
Unfortunately, reading this register in QEMU usually returns 0 because
the s->regs[] array gets wiped after a software reset.

Fixes: bd8f1ebce4 ("net/dp8393x: fix hardware reset")
Suggested-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 hw/net/dp8393x.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/dp8393x.c
+++ b/hw/net/dp8393x.c
@@ -XXX,XX +XXX,XX @@ static void dp8393x_reset(DeviceState *dev)
     timer_del(s->watchdog);
 
     memset(s->regs, 0, sizeof(s->regs));
+    s->regs[SONIC_SR] = 0x0004; /* only revision recognized by Linux/mips */
     s->regs[SONIC_CR] = SONIC_CR_RST | SONIC_CR_STP | SONIC_CR_RXDIS;
     s->regs[SONIC_DCR] &= ~(SONIC_DCR_EXBUS | SONIC_DCR_LBR);
     s->regs[SONIC_RCR] &= ~(SONIC_RCR_LB0 | SONIC_RCR_LB1 | SONIC_RCR_BRD | SONIC_RCR_RNT);
@@ -XXX,XX +XXX,XX @@ static void dp8393x_realize(DeviceState *dev, Error **errp)
     qemu_format_nic_info_str(qemu_get_queue(s->nic), s->conf.macaddr.a);
 
     s->watchdog = timer_new_ns(QEMU_CLOCK_VIRTUAL, dp8393x_watchdog, s);
-    s->regs[SONIC_SR] = 0x0004; /* only revision recognized by Linux */
 
     memory_region_init_ram(&s->prom, OBJECT(dev),
                            "dp8393x-prom", SONIC_PROM_SIZE, &local_err);
-- 
2.5.0

From: Finn Thain <fthain@telegraphics.com.au>

Section 3.4.7 of the datasheet explains that,

The RBE bit in the Interrupt Status register is set when the
    SONIC finishes using the second to last receive buffer and reads
    the last RRA descriptor. Actually, the SONIC is not truly out of
    resources, but gives the system an early warning of an impending
    out of resources condition.

RBE does not mean actual receive buffer exhaustion, and reception should
not be stopped. This is important because Linux will not check and clear
the RBE interrupt until it receives another packet. But that won't
happen if can_receive returns false. This bug causes the SONIC to become
deaf (until reset).

Fix this with a new flag to indicate actual receive buffer exhaustion.

Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
Tested-by: Laurent Vivier <laurent@vivier.eu>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 hw/net/dp8393x.c | 35 ++++++++++++++++++++++-------------
 1 file changed, 22 insertions(+), 13 deletions(-)

diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/dp8393x.c
+++ b/hw/net/dp8393x.c
@@ -XXX,XX +XXX,XX @@ typedef struct dp8393xState {
     /* Hardware */
     uint8_t it_shift;
     bool big_endian;
+    bool last_rba_is_full;
     qemu_irq irq;
 #ifdef DEBUG_SONIC
     int irq_level;
@@ -XXX,XX +XXX,XX @@ static void dp8393x_do_read_rra(dp8393xState *s)
         s->regs[SONIC_RRP] = s->regs[SONIC_RSA];
     }
 
-    /* Check resource exhaustion */
+    /* Warn the host if CRBA now has the last available resource */
     if (s->regs[SONIC_RRP] == s->regs[SONIC_RWP])
     {
         s->regs[SONIC_ISR] |= SONIC_ISR_RBE;
         dp8393x_update_irq(s);
     }
+
+    /* Allow packet reception */
+    s->last_rba_is_full = false;
 }
 
 static void dp8393x_do_software_reset(dp8393xState *s)
@@ -XXX,XX +XXX,XX @@ static void dp8393x_write(void *opaque, hwaddr addr, uint64_t data,
                 dp8393x_do_read_rra(s);
             }
             dp8393x_update_irq(s);
-            if (dp8393x_can_receive(s->nic->ncs)) {
-                qemu_flush_queued_packets(qemu_get_queue(s->nic));
-            }
             break;
         /* The guest is required to store aligned pointers here */
         case SONIC_RSA:
@@ -XXX,XX +XXX,XX @@ static int dp8393x_can_receive(NetClientState *nc)
 
     if (!(s->regs[SONIC_CR] & SONIC_CR_RXEN))
         return 0;
-    if (s->regs[SONIC_ISR] & SONIC_ISR_RBE)
-        return 0;
     return 1;
 }
 
@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
     s->regs[SONIC_RCR] &= ~(SONIC_RCR_PRX | SONIC_RCR_LBK | SONIC_RCR_FAER |
         SONIC_RCR_CRCR | SONIC_RCR_LPKT | SONIC_RCR_BC | SONIC_RCR_MC);
 
+    if (s->last_rba_is_full) {
+        return pkt_size;
+    }
+
     rx_len = pkt_size + sizeof(checksum);
     if (s->regs[SONIC_DCR] & SONIC_DCR_DW) {
         width = 2;
@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
         DPRINTF("oversize packet, pkt_size is %d\n", pkt_size);
         s->regs[SONIC_ISR] |= SONIC_ISR_RBAE;
         dp8393x_update_irq(s);
-        dp8393x_do_read_rra(s);
-        return pkt_size;
+        s->regs[SONIC_RCR] |= SONIC_RCR_LPKT;
+        goto done;
     }
 
     packet_type = dp8393x_receive_filter(s, buf, pkt_size);
@@ -XXX,XX +XXX,XX @@ static ssize_t dp8393x_receive(NetClientState *nc, const uint8_t * buf,
         s->regs[SONIC_ISR] |= SONIC_ISR_PKTRX;
     }
 
+    dp8393x_update_irq(s);
+
     s->regs[SONIC_RSC] = (s->regs[SONIC_RSC] & 0xff00) |
                          ((s->regs[SONIC_RSC] + 1) & 0x00ff);
 
+done:
+
     if (s->regs[SONIC_RCR] & SONIC_RCR_LPKT) {
-        /* Read next RRA */
-        dp8393x_do_read_rra(s);
+        if (s->regs[SONIC_RRP] == s->regs[SONIC_RWP]) {
+            /* Stop packet reception */
+            s->last_rba_is_full = true;
+        } else {
+            /* Read next resource */
+            dp8393x_do_read_rra(s);
+        }
     }
 
-    /* Done */
-    dp8393x_update_irq(s);
-
     return pkt_size;
 }
 
-- 
2.5.0

From: Yuri Benditovich <yuri.benditovich@daynix.com>

https://bugzilla.redhat.com/show_bug.cgi?id=1787142
The emulation issues hw_error if PSRCTL register
is written, for example, with zero value.
Such configuration does not present any problem when
DTYP bits of RCTL register define legacy format of
transfer descriptors. Current commit discards check
for BSIZE0 and BSIZE1 when legacy mode used.

Acked-by: Dmitry Fleytman <dmitry.fleytman@gmail.com>
Signed-off-by: Yuri Benditovich <yuri.benditovich@daynix.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 hw/net/e1000e_core.c | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/hw/net/e1000e_core.c b/hw/net/e1000e_core.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/e1000e_core.c
+++ b/hw/net/e1000e_core.c
@@ -XXX,XX +XXX,XX @@ e1000e_set_eitr(E1000ECore *core, int index, uint32_t val)
 static void
 e1000e_set_psrctl(E1000ECore *core, int index, uint32_t val)
 {
-    if ((val & E1000_PSRCTL_BSIZE0_MASK) == 0) {
-        hw_error("e1000e: PSRCTL.BSIZE0 cannot be zero");
-    }
+    if (core->mac[RCTL] & E1000_RCTL_DTYP_MASK) {
+
+        if ((val & E1000_PSRCTL_BSIZE0_MASK) == 0) {
+            hw_error("e1000e: PSRCTL.BSIZE0 cannot be zero");
+        }
 
-    if ((val & E1000_PSRCTL_BSIZE1_MASK) == 0) {
-        hw_error("e1000e: PSRCTL.BSIZE1 cannot be zero");
+        if ((val & E1000_PSRCTL_BSIZE1_MASK) == 0) {
+            hw_error("e1000e: PSRCTL.BSIZE1 cannot be zero");
+        }
     }
 
     core->mac[PSRCTL] = val;
-- 
2.5.0

From: Yuri Benditovich <yuri.benditovich@daynix.com>

Add support for following hash types:
IPV6 TCP with extension headers
IPV4 UDP
IPV6 UDP
IPV6 UDP with extension headers

Signed-off-by: Yuri Benditovich <yuri.benditovich@daynix.com>
Acked-by: Dmitry Fleytman <dmitry.fleytman@gmail.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 hw/net/net_rx_pkt.c | 42 ++++++++++++++++++++++++++++++++++++++++++
 hw/net/net_rx_pkt.h |  6 +++++-
 hw/net/trace-events |  4 ++++
 3 files changed, 51 insertions(+), 1 deletion(-)

diff --git a/hw/net/net_rx_pkt.c b/hw/net/net_rx_pkt.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/net_rx_pkt.c
+++ b/hw/net/net_rx_pkt.c
@@ -XXX,XX +XXX,XX @@ _net_rx_rss_prepare_tcp(uint8_t *rss_input,
                           &tcphdr->th_dport, sizeof(uint16_t));
 }
 
+static inline void
+_net_rx_rss_prepare_udp(uint8_t *rss_input,
+                        struct NetRxPkt *pkt,
+                        size_t *bytes_written)
+{
+    struct udp_header *udphdr = &pkt->l4hdr_info.hdr.udp;
+
+    _net_rx_rss_add_chunk(rss_input, bytes_written,
+                          &udphdr->uh_sport, sizeof(uint16_t));
+
+    _net_rx_rss_add_chunk(rss_input, bytes_written,
+                          &udphdr->uh_dport, sizeof(uint16_t));
+}
+
 uint32_t
 net_rx_pkt_calc_rss_hash(struct NetRxPkt *pkt,
                          NetRxPktRssType type,
@@ -XXX,XX +XXX,XX @@ net_rx_pkt_calc_rss_hash(struct NetRxPkt *pkt,
         trace_net_rx_pkt_rss_ip6_ex();
         _net_rx_rss_prepare_ip6(&rss_input[0], pkt, true, &rss_length);
         break;
+    case NetPktRssIpV6TcpEx:
+        assert(pkt->isip6);
+        assert(pkt->istcp);
+        trace_net_rx_pkt_rss_ip6_ex_tcp();
+        _net_rx_rss_prepare_ip6(&rss_input[0], pkt, true, &rss_length);
+        _net_rx_rss_prepare_tcp(&rss_input[0], pkt, &rss_length);
+        break;
+    case NetPktRssIpV4Udp:
+        assert(pkt->isip4);
+        assert(pkt->isudp);
+        trace_net_rx_pkt_rss_ip4_udp();
+        _net_rx_rss_prepare_ip4(&rss_input[0], pkt, &rss_length);
+        _net_rx_rss_prepare_udp(&rss_input[0], pkt, &rss_length);
+        break;
+    case NetPktRssIpV6Udp:
+        assert(pkt->isip6);
+        assert(pkt->isudp);
+        trace_net_rx_pkt_rss_ip6_udp();
+        _net_rx_rss_prepare_ip6(&rss_input[0], pkt, false, &rss_length);
+        _net_rx_rss_prepare_udp(&rss_input[0], pkt, &rss_length);
+        break;
+    case NetPktRssIpV6UdpEx:
+        assert(pkt->isip6);
+        assert(pkt->isudp);
+        trace_net_rx_pkt_rss_ip6_ex_udp();
+        _net_rx_rss_prepare_ip6(&rss_input[0], pkt, true, &rss_length);
+        _net_rx_rss_prepare_udp(&rss_input[0], pkt, &rss_length);
+        break;
     default:
         assert(false);
         break;
diff --git a/hw/net/net_rx_pkt.h b/hw/net/net_rx_pkt.h
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/net_rx_pkt.h
+++ b/hw/net/net_rx_pkt.h
@@ -XXX,XX +XXX,XX @@ typedef enum {
     NetPktRssIpV4Tcp,
     NetPktRssIpV6Tcp,
     NetPktRssIpV6,
-    NetPktRssIpV6Ex
+    NetPktRssIpV6Ex,
+    NetPktRssIpV6TcpEx,
+    NetPktRssIpV4Udp,
+    NetPktRssIpV6Udp,
+    NetPktRssIpV6UdpEx,
 } NetRxPktRssType;
 
 /**
diff --git a/hw/net/trace-events b/hw/net/trace-events
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/trace-events
+++ b/hw/net/trace-events
@@ -XXX,XX +XXX,XX @@ net_rx_pkt_l3_csum_validate_csum(size_t l3hdr_off, uint32_t csl, uint32_t cntr,
 
 net_rx_pkt_rss_ip4(void) "Calculating IPv4 RSS  hash"
 net_rx_pkt_rss_ip4_tcp(void) "Calculating IPv4/TCP RSS  hash"
+net_rx_pkt_rss_ip4_udp(void) "Calculating IPv4/UDP RSS  hash"
 net_rx_pkt_rss_ip6_tcp(void) "Calculating IPv6/TCP RSS  hash"
+net_rx_pkt_rss_ip6_udp(void) "Calculating IPv6/UDP RSS  hash"
 net_rx_pkt_rss_ip6(void) "Calculating IPv6 RSS  hash"
 net_rx_pkt_rss_ip6_ex(void) "Calculating IPv6/EX RSS  hash"
+net_rx_pkt_rss_ip6_ex_tcp(void) "Calculating IPv6/EX/TCP RSS  hash"
+net_rx_pkt_rss_ip6_ex_udp(void) "Calculating IPv6/EX/UDP RSS  hash"
 net_rx_pkt_rss_hash(size_t rss_length, uint32_t rss_hash) "RSS hash for %zu bytes: 0x%X"
 net_rx_pkt_rss_add_chunk(void* ptr, size_t size, size_t input_offset) "Add RSS chunk %p, %zu bytes, RSS input offset %zu bytes"
 
-- 
2.5.0

From: Yuri Benditovich <yuri.benditovich@daynix.com>

When requested to calculate the hash for TCPV6 packet,
ignore overrides of source and destination addresses
in in extension headers.
Use these overrides when new hash type NetPktRssIpV6TcpEx
requested.
Use this type in e1000e hash calculation for IPv6 TCP, which
should take in account overrides of the addresses.

Signed-off-by: Yuri Benditovich <yuri.benditovich@daynix.com>
Acked-by: Dmitry Fleytman <dmitry.fleytman@gmail.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 hw/net/e1000e_core.c | 2 +-
 hw/net/net_rx_pkt.c  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/hw/net/e1000e_core.c b/hw/net/e1000e_core.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/e1000e_core.c
+++ b/hw/net/e1000e_core.c
@@ -XXX,XX +XXX,XX @@ e1000e_rss_calc_hash(E1000ECore *core,
         type = NetPktRssIpV4Tcp;
         break;
     case E1000_MRQ_RSS_TYPE_IPV6TCP:
-        type = NetPktRssIpV6Tcp;
+        type = NetPktRssIpV6TcpEx;
         break;
     case E1000_MRQ_RSS_TYPE_IPV6:
         type = NetPktRssIpV6;
diff --git a/hw/net/net_rx_pkt.c b/hw/net/net_rx_pkt.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/net_rx_pkt.c
+++ b/hw/net/net_rx_pkt.c
@@ -XXX,XX +XXX,XX @@ net_rx_pkt_calc_rss_hash(struct NetRxPkt *pkt,
         assert(pkt->isip6);
         assert(pkt->istcp);
         trace_net_rx_pkt_rss_ip6_tcp();
-        _net_rx_rss_prepare_ip6(&rss_input[0], pkt, true, &rss_length);
+        _net_rx_rss_prepare_ip6(&rss_input[0], pkt, false, &rss_length);
         _net_rx_rss_prepare_tcp(&rss_input[0], pkt, &rss_length);
         break;
     case NetPktRssIpV6:
-- 
2.5.0

From: Bin Meng <bmeng.cn@gmail.com>

When CADENCE_GEM_ERR_DEBUG is turned on, there are several
compilation errors in DB_PRINT(). Fix them.

While we are here, update to use appropriate modifiers in
the same DB_PRINT() call.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 hw/net/cadence_gem.c | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/hw/net/cadence_gem.c b/hw/net/cadence_gem.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/cadence_gem.c
+++ b/hw/net/cadence_gem.c
@@ -XXX,XX +XXX,XX @@ static ssize_t gem_receive(NetClientState *nc, const uint8_t *buf, size_t size)
             return -1;
         }
 
-        DB_PRINT("copy %d bytes to 0x%x\n", MIN(bytes_to_copy, rxbufsize),
-                rx_desc_get_buffer(s->rx_desc[q]));
+        DB_PRINT("copy %u bytes to 0x%" PRIx64 "\n",
+                 MIN(bytes_to_copy, rxbufsize),
+                 rx_desc_get_buffer(s, s->rx_desc[q]));
 
         /* Copy packet data to emulated DMA buffer */
         address_space_write(&s->dma_as, rx_desc_get_buffer(s, s->rx_desc[q]) +
@@ -XXX,XX +XXX,XX @@ static void gem_transmit(CadenceGEMState *s)
 
             if (tx_desc_get_length(desc) > sizeof(tx_packet) -
                                                (p - tx_packet)) {
-                DB_PRINT("TX descriptor @ 0x%x too large: size 0x%x space " \
-                         "0x%x\n", (unsigned)packet_desc_addr,
-                         (unsigned)tx_desc_get_length(desc),
+                DB_PRINT("TX descriptor @ 0x%" HWADDR_PRIx \
+                         " too large: size 0x%x space 0x%zx\n",
+                         packet_desc_addr, tx_desc_get_length(desc),
                          sizeof(tx_packet) - (p - tx_packet));
                 break;
             }
-- 
2.5.0

From: Lukas Straub <lukasstraub2@web.de>

After failover the Secondary side of replication shouldn't change state, because
it now functions as our primary disk.

In replication_start, replication_do_checkpoint, replication_stop, ignore
the request if current state is BLOCK_REPLICATION_DONE (sucessful failover) or
BLOCK_REPLICATION_FAILOVER (failover in progres i.e. currently merging active
and hidden images into the base image).

Signed-off-by: Lukas Straub <lukasstraub2@web.de>
Reviewed-by: Zhang Chen <chen.zhang@intel.com>
Acked-by: Max Reitz <mreitz@redhat.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 block/replication.c | 35 ++++++++++++++++++++++++++++++++++-
 1 file changed, 34 insertions(+), 1 deletion(-)

diff --git a/block/replication.c b/block/replication.c
index XXXXXXX..XXXXXXX 100644
--- a/block/replication.c
+++ b/block/replication.c
@@ -XXX,XX +XXX,XX @@ static void replication_start(ReplicationState *rs, ReplicationMode mode,
     aio_context_acquire(aio_context);
     s = bs->opaque;
 
+    if (s->stage == BLOCK_REPLICATION_DONE ||
+        s->stage == BLOCK_REPLICATION_FAILOVER) {
+        /*
+         * This case happens when a secondary is promoted to primary.
+         * Ignore the request because the secondary side of replication
+         * doesn't have to do anything anymore.
+         */
+        aio_context_release(aio_context);
+        return;
+    }
+
     if (s->stage != BLOCK_REPLICATION_NONE) {
         error_setg(errp, "Block replication is running or done");
         aio_context_release(aio_context);
@@ -XXX,XX +XXX,XX @@ static void replication_do_checkpoint(ReplicationState *rs, Error **errp)
     aio_context_acquire(aio_context);
     s = bs->opaque;
 
+    if (s->stage == BLOCK_REPLICATION_DONE ||
+        s->stage == BLOCK_REPLICATION_FAILOVER) {
+        /*
+         * This case happens when a secondary was promoted to primary.
+         * Ignore the request because the secondary side of replication
+         * doesn't have to do anything anymore.
+         */
+        aio_context_release(aio_context);
+        return;
+    }
+
     if (s->mode == REPLICATION_MODE_SECONDARY) {
         secondary_do_checkpoint(s, errp);
     }
@@ -XXX,XX +XXX,XX @@ static void replication_get_error(ReplicationState *rs, Error **errp)
     aio_context_acquire(aio_context);
     s = bs->opaque;
 
-    if (s->stage != BLOCK_REPLICATION_RUNNING) {
+    if (s->stage == BLOCK_REPLICATION_NONE) {
         error_setg(errp, "Block replication is not running");
         aio_context_release(aio_context);
         return;
@@ -XXX,XX +XXX,XX @@ static void replication_stop(ReplicationState *rs, bool failover, Error **errp)
     aio_context_acquire(aio_context);
     s = bs->opaque;
 
+    if (s->stage == BLOCK_REPLICATION_DONE ||
+        s->stage == BLOCK_REPLICATION_FAILOVER) {
+        /*
+         * This case happens when a secondary was promoted to primary.
+         * Ignore the request because the secondary side of replication
+         * doesn't have to do anything anymore.
+         */
+        aio_context_release(aio_context);
+        return;
+    }
+
     if (s->stage != BLOCK_REPLICATION_RUNNING) {
         error_setg(errp, "Block replication is not running");
         aio_context_release(aio_context);
-- 
2.5.0

From: Lukas Straub <lukasstraub2@web.de>

This simulates the case that happens when we resume COLO after failover.

Signed-off-by: Lukas Straub <lukasstraub2@web.de>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 tests/test-replication.c | 52 ++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 52 insertions(+)

diff --git a/tests/test-replication.c b/tests/test-replication.c
index XXXXXXX..XXXXXXX 100644
--- a/tests/test-replication.c
+++ b/tests/test-replication.c
@@ -XXX,XX +XXX,XX @@ static void test_secondary_stop(void)
     teardown_secondary();
 }
 
+static void test_secondary_continuous_replication(void)
+{
+    BlockBackend *top_blk, *local_blk;
+    Error *local_err = NULL;
+
+    top_blk = start_secondary();
+    replication_start_all(REPLICATION_MODE_SECONDARY, &local_err);
+    g_assert(!local_err);
+
+    /* write 0x22 to s_local_disk (IMG_SIZE / 2, IMG_SIZE) */
+    local_blk = blk_by_name(S_LOCAL_DISK_ID);
+    test_blk_write(local_blk, 0x22, IMG_SIZE / 2, IMG_SIZE / 2, false);
+
+    /* replication will backup s_local_disk to s_hidden_disk */
+    test_blk_read(top_blk, 0x11, IMG_SIZE / 2,
+                  IMG_SIZE / 2, 0, IMG_SIZE, false);
+
+    /* write 0x33 to s_active_disk (0, IMG_SIZE / 2) */
+    test_blk_write(top_blk, 0x33, 0, IMG_SIZE / 2, false);
+
+    /* do failover (active commit) */
+    replication_stop_all(true, &local_err);
+    g_assert(!local_err);
+
+    /* it should ignore all requests from now on */
+
+    /* start after failover */
+    replication_start_all(REPLICATION_MODE_PRIMARY, &local_err);
+    g_assert(!local_err);
+
+    /* checkpoint */
+    replication_do_checkpoint_all(&local_err);
+    g_assert(!local_err);
+
+    /* stop */
+    replication_stop_all(true, &local_err);
+    g_assert(!local_err);
+
+    /* read from s_local_disk (0, IMG_SIZE / 2) */
+    test_blk_read(top_blk, 0x33, 0, IMG_SIZE / 2,
+                  0, IMG_SIZE / 2, false);
+
+
+    /* read from s_local_disk (IMG_SIZE / 2, IMG_SIZE) */
+    test_blk_read(top_blk, 0x22, IMG_SIZE / 2,
+                  IMG_SIZE / 2, 0, IMG_SIZE, false);
+
+    teardown_secondary();
+}
+
 static void test_secondary_do_checkpoint(void)
 {
     BlockBackend *top_blk, *local_blk;
@@ -XXX,XX +XXX,XX @@ int main(int argc, char **argv)
     g_test_add_func("/replication/secondary/write", test_secondary_write);
     g_test_add_func("/replication/secondary/start", test_secondary_start);
     g_test_add_func("/replication/secondary/stop",  test_secondary_stop);
+    g_test_add_func("/replication/secondary/continuous_replication",
+                    test_secondary_continuous_replication);
     g_test_add_func("/replication/secondary/do_checkpoint",
                     test_secondary_do_checkpoint);
     g_test_add_func("/replication/secondary/get_error_all",
-- 
2.5.0

From: Lukas Straub <lukasstraub2@web.de>

To switch the Secondary to Primary, we need to insert new filters
before the filter-rewriter.

Add the options insert= and position= to be able to insert filters
anywhere in the filter list.

position should be "head" or "tail" to insert at the head or
tail of the filter list or it should be "id=<id>" to specify
the id of another filter.
insert should be either "before" or "behind" to specify where to
insert the new filter relative to the one specified with position.

Signed-off-by: Lukas Straub <lukasstraub2@web.de>
Reviewed-by: Zhang Chen <chen.zhang@intel.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 include/net/filter.h |  2 ++
 net/filter.c         | 92 +++++++++++++++++++++++++++++++++++++++++++++++++++-
 qemu-options.hx      | 31 +++++++++++++++---
 3 files changed, 119 insertions(+), 6 deletions(-)

diff --git a/include/net/filter.h b/include/net/filter.h
index XXXXXXX..XXXXXXX 100644
--- a/include/net/filter.h
+++ b/include/net/filter.h
@@ -XXX,XX +XXX,XX @@ struct NetFilterState {
     NetClientState *netdev;
     NetFilterDirection direction;
     bool on;
+    char *position;
+    bool insert_before_flag;
     QTAILQ_ENTRY(NetFilterState) next;
 };
 
diff --git a/net/filter.c b/net/filter.c
index XXXXXXX..XXXXXXX 100644
--- a/net/filter.c
+++ b/net/filter.c
@@ -XXX,XX +XXX,XX @@ static void netfilter_set_status(Object *obj, const char *str, Error **errp)
     }
 }
 
+static char *netfilter_get_position(Object *obj, Error **errp)
+{
+    NetFilterState *nf = NETFILTER(obj);
+
+    return g_strdup(nf->position);
+}
+
+static void netfilter_set_position(Object *obj, const char *str, Error **errp)
+{
+    NetFilterState *nf = NETFILTER(obj);
+
+    nf->position = g_strdup(str);
+}
+
+static char *netfilter_get_insert(Object *obj, Error **errp)
+{
+    NetFilterState *nf = NETFILTER(obj);
+
+    return nf->insert_before_flag ? g_strdup("before") : g_strdup("behind");
+}
+
+static void netfilter_set_insert(Object *obj, const char *str, Error **errp)
+{
+    NetFilterState *nf = NETFILTER(obj);
+
+    if (strcmp(str, "before") && strcmp(str, "behind")) {
+        error_setg(errp, "Invalid value for netfilter insert, "
+                         "should be 'before' or 'behind'");
+        return;
+    }
+
+    nf->insert_before_flag = !strcmp(str, "before");
+}
+
 static void netfilter_init(Object *obj)
 {
     NetFilterState *nf = NETFILTER(obj);
 
     nf->on = true;
+    nf->insert_before_flag = false;
+    nf->position = g_strdup("tail");
 
     object_property_add_str(obj, "netdev",
                             netfilter_get_netdev_id, netfilter_set_netdev_id,
@@ -XXX,XX +XXX,XX @@ static void netfilter_init(Object *obj)
     object_property_add_str(obj, "status",
                             netfilter_get_status, netfilter_set_status,
                             NULL);
+    object_property_add_str(obj, "position",
+                            netfilter_get_position, netfilter_set_position,
+                            NULL);
+    object_property_add_str(obj, "insert",
+                            netfilter_get_insert, netfilter_set_insert,
+                            NULL);
 }
 
 static void netfilter_complete(UserCreatable *uc, Error **errp)
 {
     NetFilterState *nf = NETFILTER(uc);
+    NetFilterState *position = NULL;
     NetClientState *ncs[MAX_QUEUE_NUM];
     NetFilterClass *nfc = NETFILTER_GET_CLASS(uc);
     int queues;
@@ -XXX,XX +XXX,XX @@ static void netfilter_complete(UserCreatable *uc, Error **errp)
         return;
     }
 
+    if (strcmp(nf->position, "head") && strcmp(nf->position, "tail")) {
+        Object *container;
+        Object *obj;
+        char *position_id;
+
+        if (!g_str_has_prefix(nf->position, "id=")) {
+            error_setg(errp, QERR_INVALID_PARAMETER_VALUE, "position",
+                       "'head', 'tail' or 'id=<id>'");
+            return;
+        }
+
+        /* get the id from the string */
+        position_id = g_strndup(nf->position + 3, strlen(nf->position) - 3);
+
+        /* Search for the position to insert before/behind */
+        container = object_get_objects_root();
+        obj = object_resolve_path_component(container, position_id);
+        if (!obj) {
+            error_setg(errp, "filter '%s' not found", position_id);
+            g_free(position_id);
+            return;
+        }
+
+        position = NETFILTER(obj);
+
+        if (position->netdev != ncs[0]) {
+            error_setg(errp, "filter '%s' belongs to a different netdev",
+                        position_id);
+            g_free(position_id);
+            return;
+        }
+
+        g_free(position_id);
+    }
+
     nf->netdev = ncs[0];
 
     if (nfc->setup) {
@@ -XXX,XX +XXX,XX @@ static void netfilter_complete(UserCreatable *uc, Error **errp)
             return;
         }
     }
-    QTAILQ_INSERT_TAIL(&nf->netdev->filters, nf, next);
+
+    if (position) {
+        if (nf->insert_before_flag) {
+            QTAILQ_INSERT_BEFORE(position, nf, next);
+        } else {
+            QTAILQ_INSERT_AFTER(&nf->netdev->filters, position, nf, next);
+        }
+    } else if (!strcmp(nf->position, "head")) {
+        QTAILQ_INSERT_HEAD(&nf->netdev->filters, nf, next);
+    } else if (!strcmp(nf->position, "tail")) {
+        QTAILQ_INSERT_TAIL(&nf->netdev->filters, nf, next);
+    }
 }
 
 static void netfilter_finalize(Object *obj)
@@ -XXX,XX +XXX,XX @@ static void netfilter_finalize(Object *obj)
         QTAILQ_REMOVE(&nf->netdev->filters, nf, next);
     }
     g_free(nf->netdev_id);
+    g_free(nf->position);
 }
 
 static void default_handle_event(NetFilterState *nf, int event, Error **errp)
diff --git a/qemu-options.hx b/qemu-options.hx
index XXXXXXX..XXXXXXX 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -XXX,XX +XXX,XX @@ applications, they can do this through this parameter. Its format is
 a gnutls priority string as described at
 @url{https://gnutls.org/manual/html_node/Priority-Strings.html}.
 
-@item -object filter-buffer,id=@var{id},netdev=@var{netdevid},interval=@var{t}[,queue=@var{all|rx|tx}][,status=@var{on|off}]
+@item -object filter-buffer,id=@var{id},netdev=@var{netdevid},interval=@var{t}[,queue=@var{all|rx|tx}][,status=@var{on|off}][,position=@var{head|tail|id=<id>}][,insert=@var{behind|before}]
 
 Interval @var{t} can't be 0, this filter batches the packet delivery: all
 packets arriving in a given interval on netdev @var{netdevid} are delayed
@@ -XXX,XX +XXX,XX @@ queue @var{all|rx|tx} is an option that can be applied to any netfilter.
 @option{tx}: the filter is attached to the transmit queue of the netdev,
              where it will receive packets sent by the netdev.
 
-@item -object filter-mirror,id=@var{id},netdev=@var{netdevid},outdev=@var{chardevid},queue=@var{all|rx|tx}[,vnet_hdr_support]
+position @var{head|tail|id=<id>} is an option to specify where the
+filter should be inserted in the filter list. It can be applied to any
+netfilter.
+
+@option{head}: the filter is inserted at the head of the filter
+               list, before any existing filters.
+
+@option{tail}: the filter is inserted at the tail of the filter
+               list, behind any existing filters (default).
+
+@option{id=<id>}: the filter is inserted before or behind the filter
+                  specified by <id>, see the insert option below.
+
+insert @var{behind|before} is an option to specify where to insert the
+new filter relative to the one specified with position=id=<id>. It can
+be applied to any netfilter.
+
+@option{before}: insert before the specified filter.
+
+@option{behind}: insert behind the specified filter (default).
+
+@item -object filter-mirror,id=@var{id},netdev=@var{netdevid},outdev=@var{chardevid},queue=@var{all|rx|tx}[,vnet_hdr_support][,position=@var{head|tail|id=<id>}][,insert=@var{behind|before}]
 
 filter-mirror on netdev @var{netdevid},mirror net packet to chardev@var{chardevid}, if it has the vnet_hdr_support flag, filter-mirror will mirror packet with vnet_hdr_len.
 
-@item -object filter-redirector,id=@var{id},netdev=@var{netdevid},indev=@var{chardevid},outdev=@var{chardevid},queue=@var{all|rx|tx}[,vnet_hdr_support]
+@item -object filter-redirector,id=@var{id},netdev=@var{netdevid},indev=@var{chardevid},outdev=@var{chardevid},queue=@var{all|rx|tx}[,vnet_hdr_support][,position=@var{head|tail|id=<id>}][,insert=@var{behind|before}]
 
 filter-redirector on netdev @var{netdevid},redirect filter's net packet to chardev
 @var{chardevid},and redirect indev's packet to filter.if it has the vnet_hdr_support flag,
@@ -XXX,XX +XXX,XX @@ Create a filter-redirector we need to differ outdev id from indev id, id can not
 be the same. we can just use indev or outdev, but at least one of indev or outdev
 need to be specified.
 
-@item -object filter-rewriter,id=@var{id},netdev=@var{netdevid},queue=@var{all|rx|tx},[vnet_hdr_support]
+@item -object filter-rewriter,id=@var{id},netdev=@var{netdevid},queue=@var{all|rx|tx},[vnet_hdr_support][,position=@var{head|tail|id=<id>}][,insert=@var{behind|before}]
 
 Filter-rewriter is a part of COLO project.It will rewrite tcp packet to
 secondary from primary to keep secondary tcp connection,and rewrite
@@ -XXX,XX +XXX,XX @@ colo secondary:
 -object filter-redirector,id=f2,netdev=hn0,queue=rx,outdev=red1
 -object filter-rewriter,id=rew0,netdev=hn0,queue=all
 
-@item -object filter-dump,id=@var{id},netdev=@var{dev}[,file=@var{filename}][,maxlen=@var{len}]
+@item -object filter-dump,id=@var{id},netdev=@var{dev}[,file=@var{filename}][,maxlen=@var{len}][,position=@var{head|tail|id=<id>}][,insert=@var{behind|before}]
 
 Dump the network traffic on netdev @var{dev} to the file specified by
 @var{filename}. At most @var{len} bytes (64k by default) per packet are stored.
-- 
2.5.0

From: Lukas Straub <lukasstraub2@web.de>

Document the qemu command-line and qmp commands for continuous replication

Signed-off-by: Lukas Straub <lukasstraub2@web.de>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 docs/COLO-FT.txt           | 224 +++++++++++++++++++++++++++++++++------------
 docs/block-replication.txt |  28 ++++--
 2 files changed, 184 insertions(+), 68 deletions(-)

diff --git a/docs/COLO-FT.txt b/docs/COLO-FT.txt
index XXXXXXX..XXXXXXX 100644
--- a/docs/COLO-FT.txt
+++ b/docs/COLO-FT.txt
@@ -XXX,XX +XXX,XX @@ The diagram just shows the main qmp command, you can get the detail
 in test procedure.
 
 == Test procedure ==
-1. Startup qemu
-Primary:
-# qemu-system-x86_64 -accel kvm -m 2048 -smp 2 -qmp stdio -name primary \
-  -device piix3-usb-uhci -vnc :7 \
-  -device usb-tablet -netdev tap,id=hn0,vhost=off \
-  -device virtio-net-pci,id=net-pci0,netdev=hn0 \
-  -drive if=virtio,id=primary-disk0,driver=quorum,read-pattern=fifo,vote-threshold=1,\
-         children.0.file.filename=1.raw,\
-         children.0.driver=raw -S
-Secondary:
-# qemu-system-x86_64 -accel kvm -m 2048 -smp 2 -qmp stdio -name secondary \
-  -device piix3-usb-uhci -vnc :7 \
-  -device usb-tablet -netdev tap,id=hn0,vhost=off \
-  -device virtio-net-pci,id=net-pci0,netdev=hn0 \
-  -drive if=none,id=secondary-disk0,file.filename=1.raw,driver=raw,node-name=node0 \
-  -drive if=virtio,id=active-disk0,driver=replication,mode=secondary,\
-         file.driver=qcow2,top-id=active-disk0,\
-         file.file.filename=/mnt/ramfs/active_disk.img,\
-         file.backing.driver=qcow2,\
-         file.backing.file.filename=/mnt/ramfs/hidden_disk.img,\
-         file.backing.backing=secondary-disk0 \
-  -incoming tcp:0:8888
-
-2. On Secondary VM's QEMU monitor, issue command
+Note: Here we are running both instances on the same host for testing,
+change the IP Addresses if you want to run it on two hosts. Initally
+127.0.0.1 is the Primary Host and 127.0.0.2 is the Secondary Host.
+
+== Startup qemu ==
+1. Primary:
+Note: Initally, $imagefolder/primary.qcow2 needs to be copied to all hosts.
+You don't need to change any IP's here, because 0.0.0.0 listens on any
+interface. The chardev's with 127.0.0.1 IP's loopback to the local qemu
+instance.
+
+# imagefolder="/mnt/vms/colo-test-primary"
+
+# qemu-system-x86_64 -enable-kvm -cpu qemu64,+kvmclock -m 512 -smp 1 -qmp stdio \
+   -device piix3-usb-uhci -device usb-tablet -name primary \
+   -netdev tap,id=hn0,vhost=off,helper=/usr/lib/qemu/qemu-bridge-helper \
+   -device rtl8139,id=e0,netdev=hn0 \
+   -chardev socket,id=mirror0,host=0.0.0.0,port=9003,server,nowait \
+   -chardev socket,id=compare1,host=0.0.0.0,port=9004,server,wait \
+   -chardev socket,id=compare0,host=127.0.0.1,port=9001,server,nowait \
+   -chardev socket,id=compare0-0,host=127.0.0.1,port=9001 \
+   -chardev socket,id=compare_out,host=127.0.0.1,port=9005,server,nowait \
+   -chardev socket,id=compare_out0,host=127.0.0.1,port=9005 \
+   -object filter-mirror,id=m0,netdev=hn0,queue=tx,outdev=mirror0 \
+   -object filter-redirector,netdev=hn0,id=redire0,queue=rx,indev=compare_out \
+   -object filter-redirector,netdev=hn0,id=redire1,queue=rx,outdev=compare0 \
+   -object iothread,id=iothread1 \
+   -object colo-compare,id=comp0,primary_in=compare0-0,secondary_in=compare1,\
+outdev=compare_out0,iothread=iothread1 \
+   -drive if=ide,id=colo-disk0,driver=quorum,read-pattern=fifo,vote-threshold=1,\
+children.0.file.filename=$imagefolder/primary.qcow2,children.0.driver=qcow2 -S
+
+2. Secondary:
+Note: Active and hidden images need to be created only once and the
+size should be the same as primary.qcow2. Again, you don't need to change
+any IP's here, except for the $primary_ip variable.
+
+# imagefolder="/mnt/vms/colo-test-secondary"
+# primary_ip=127.0.0.1
+
+# qemu-img create -f qcow2 $imagefolder/secondary-active.qcow2 10G
+
+# qemu-img create -f qcow2 $imagefolder/secondary-hidden.qcow2 10G
+
+# qemu-system-x86_64 -enable-kvm -cpu qemu64,+kvmclock -m 512 -smp 1 -qmp stdio \
+   -device piix3-usb-uhci -device usb-tablet -name secondary \
+   -netdev tap,id=hn0,vhost=off,helper=/usr/lib/qemu/qemu-bridge-helper \
+   -device rtl8139,id=e0,netdev=hn0 \
+   -chardev socket,id=red0,host=$primary_ip,port=9003,reconnect=1 \
+   -chardev socket,id=red1,host=$primary_ip,port=9004,reconnect=1 \
+   -object filter-redirector,id=f1,netdev=hn0,queue=tx,indev=red0 \
+   -object filter-redirector,id=f2,netdev=hn0,queue=rx,outdev=red1 \
+   -object filter-rewriter,id=rew0,netdev=hn0,queue=all \
+   -drive if=none,id=parent0,file.filename=$imagefolder/primary.qcow2,driver=qcow2 \
+   -drive if=none,id=childs0,driver=replication,mode=secondary,file.driver=qcow2,\
+top-id=colo-disk0,file.file.filename=$imagefolder/secondary-active.qcow2,\
+file.backing.driver=qcow2,file.backing.file.filename=$imagefolder/secondary-hidden.qcow2,\
+file.backing.backing=parent0 \
+   -drive if=ide,id=colo-disk0,driver=quorum,read-pattern=fifo,vote-threshold=1,\
+children.0=childs0 \
+   -incoming tcp:0.0.0.0:9998
+
+
+3. On Secondary VM's QEMU monitor, issue command
 {'execute':'qmp_capabilities'}
-{ 'execute': 'nbd-server-start',
-  'arguments': {'addr': {'type': 'inet', 'data': {'host': 'xx.xx.xx.xx', 'port': '8889'} } }
-}
-{'execute': 'nbd-server-add', 'arguments': {'device': 'secondary-disk0', 'writable': true } }
+{'execute': 'nbd-server-start', 'arguments': {'addr': {'type': 'inet', 'data': {'host': '0.0.0.0', 'port': '9999'} } } }
+{'execute': 'nbd-server-add', 'arguments': {'device': 'parent0', 'writable': true } }
 
 Note:
   a. The qmp command nbd-server-start and nbd-server-add must be run
      before running the qmp command migrate on primary QEMU
   b. Active disk, hidden disk and nbd target's length should be the
      same.
-  c. It is better to put active disk and hidden disk in ramdisk.
+  c. It is better to put active disk and hidden disk in ramdisk. They
+     will be merged into the parent disk on failover.
 
-3. On Primary VM's QEMU monitor, issue command:
+4. On Primary VM's QEMU monitor, issue command:
 {'execute':'qmp_capabilities'}
-{ 'execute': 'human-monitor-command',
-  'arguments': {'command-line': 'drive_add -n buddy driver=replication,mode=primary,file.driver=nbd,file.host=xx.xx.xx.xx,file.port=8889,file.export=secondary-disk0,node-name=nbd_client0'}}
-{ 'execute':'x-blockdev-change', 'arguments':{'parent': 'primary-disk0', 'node': 'nbd_client0' } }
-{ 'execute': 'migrate-set-capabilities',
-      'arguments': {'capabilities': [ {'capability': 'x-colo', 'state': true } ] } }
-{ 'execute': 'migrate', 'arguments': {'uri': 'tcp:xx.xx.xx.xx:8888' } }
+{'execute': 'human-monitor-command', 'arguments': {'command-line': 'drive_add -n buddy driver=replication,mode=primary,file.driver=nbd,file.host=127.0.0.2,file.port=9999,file.export=parent0,node-name=replication0'}}
+{'execute': 'x-blockdev-change', 'arguments':{'parent': 'colo-disk0', 'node': 'replication0' } }
+{'execute': 'migrate-set-capabilities', 'arguments': {'capabilities': [ {'capability': 'x-colo', 'state': true } ] } }
+{'execute': 'migrate', 'arguments': {'uri': 'tcp:127.0.0.2:9998' } }
 
   Note:
   a. There should be only one NBD Client for each primary disk.
-  b. xx.xx.xx.xx is the secondary physical machine's hostname or IP
-  c. The qmp command line must be run after running qmp command line in
+  b. The qmp command line must be run after running qmp command line in
      secondary qemu.
 
-4. After the above steps, you will see, whenever you make changes to PVM, SVM will be synced.
+5. After the above steps, you will see, whenever you make changes to PVM, SVM will be synced.
 You can issue command '{ "execute": "migrate-set-parameters" , "arguments":{ "x-checkpoint-delay": 2000 } }'
-to change the checkpoint period time
+to change the idle checkpoint period time
+
+6. Failover test
+You can kill one of the VMs and Failover on the surviving VM:
+
+If you killed the Secondary, then follow "Primary Failover". After that,
+if you want to resume the replication, follow "Primary resume replication"
+
+If you killed the Primary, then follow "Secondary Failover". After that,
+if you want to resume the replication, follow "Secondary resume replication"
+
+== Primary Failover ==
+The Secondary died, resume on the Primary
+
+{'execute': 'x-blockdev-change', 'arguments':{ 'parent': 'colo-disk0', 'child': 'children.1'} }
+{'execute': 'human-monitor-command', 'arguments':{ 'command-line': 'drive_del replication0' } }
+{'execute': 'object-del', 'arguments':{ 'id': 'comp0' } }
+{'execute': 'object-del', 'arguments':{ 'id': 'iothread1' } }
+{'execute': 'object-del', 'arguments':{ 'id': 'm0' } }
+{'execute': 'object-del', 'arguments':{ 'id': 'redire0' } }
+{'execute': 'object-del', 'arguments':{ 'id': 'redire1' } }
+{'execute': 'x-colo-lost-heartbeat' }
+
+== Secondary Failover ==
+The Primary died, resume on the Secondary and prepare to become the new Primary
+
+{'execute': 'nbd-server-stop'}
+{'execute': 'x-colo-lost-heartbeat'}
+
+{'execute': 'object-del', 'arguments':{ 'id': 'f2' } }
+{'execute': 'object-del', 'arguments':{ 'id': 'f1' } }
+{'execute': 'chardev-remove', 'arguments':{ 'id': 'red1' } }
+{'execute': 'chardev-remove', 'arguments':{ 'id': 'red0' } }
+
+{'execute': 'chardev-add', 'arguments':{ 'id': 'mirror0', 'backend': {'type': 'socket', 'data': {'addr': { 'type': 'inet', 'data': { 'host': '0.0.0.0', 'port': '9003' } }, 'server': true } } } }
+{'execute': 'chardev-add', 'arguments':{ 'id': 'compare1', 'backend': {'type': 'socket', 'data': {'addr': { 'type': 'inet', 'data': { 'host': '0.0.0.0', 'port': '9004' } }, 'server': true } } } }
+{'execute': 'chardev-add', 'arguments':{ 'id': 'compare0', 'backend': {'type': 'socket', 'data': {'addr': { 'type': 'inet', 'data': { 'host': '127.0.0.1', 'port': '9001' } }, 'server': true } } } }
+{'execute': 'chardev-add', 'arguments':{ 'id': 'compare0-0', 'backend': {'type': 'socket', 'data': {'addr': { 'type': 'inet', 'data': { 'host': '127.0.0.1', 'port': '9001' } }, 'server': false } } } }
+{'execute': 'chardev-add', 'arguments':{ 'id': 'compare_out', 'backend': {'type': 'socket', 'data': {'addr': { 'type': 'inet', 'data': { 'host': '127.0.0.1', 'port': '9005' } }, 'server': true } } } }
+{'execute': 'chardev-add', 'arguments':{ 'id': 'compare_out0', 'backend': {'type': 'socket', 'data': {'addr': { 'type': 'inet', 'data': { 'host': '127.0.0.1', 'port': '9005' } }, 'server': false } } } }
+
+== Primary resume replication ==
+Resume replication after new Secondary is up.
+
+Start the new Secondary (Steps 2 and 3 above), then on the Primary:
+{'execute': 'drive-mirror', 'arguments':{ 'device': 'colo-disk0', 'job-id': 'resync', 'target': 'nbd://127.0.0.2:9999/parent0', 'mode': 'existing', 'format': 'raw', 'sync': 'full'} }
+
+Wait until disk is synced, then:
+{'execute': 'stop'}
+{'execute': 'block-job-cancel', 'arguments':{ 'device': 'resync'} }
+
+{'execute': 'human-monitor-command', 'arguments':{ 'command-line': 'drive_add -n buddy driver=replication,mode=primary,file.driver=nbd,file.host=127.0.0.2,file.port=9999,file.export=parent0,node-name=replication0'}}
+{'execute': 'x-blockdev-change', 'arguments':{ 'parent': 'colo-disk0', 'node': 'replication0' } }
+
+{'execute': 'object-add', 'arguments':{ 'qom-type': 'filter-mirror', 'id': 'm0', 'props': { 'netdev': 'hn0', 'queue': 'tx', 'outdev': 'mirror0' } } }
+{'execute': 'object-add', 'arguments':{ 'qom-type': 'filter-redirector', 'id': 'redire0', 'props': { 'netdev': 'hn0', 'queue': 'rx', 'indev': 'compare_out' } } }
+{'execute': 'object-add', 'arguments':{ 'qom-type': 'filter-redirector', 'id': 'redire1', 'props': { 'netdev': 'hn0', 'queue': 'rx', 'outdev': 'compare0' } } }
+{'execute': 'object-add', 'arguments':{ 'qom-type': 'iothread', 'id': 'iothread1' } }
+{'execute': 'object-add', 'arguments':{ 'qom-type': 'colo-compare', 'id': 'comp0', 'props': { 'primary_in': 'compare0-0', 'secondary_in': 'compare1', 'outdev': 'compare_out0', 'iothread': 'iothread1' } } }
+
+{'execute': 'migrate-set-capabilities', 'arguments':{ 'capabilities': [ {'capability': 'x-colo', 'state': true } ] } }
+{'execute': 'migrate', 'arguments':{ 'uri': 'tcp:127.0.0.2:9998' } }
+
+Note:
+If this Primary previously was a Secondary, then we need to insert the
+filters before the filter-rewriter by using the
+"'insert': 'before', 'position': 'id=rew0'" Options. See below.
+
+== Secondary resume replication ==
+Become Primary and resume replication after new Secondary is up. Note
+that now 127.0.0.1 is the Secondary and 127.0.0.2 is the Primary.
+
+Start the new Secondary (Steps 2 and 3 above, but with primary_ip=127.0.0.2),
+then on the old Secondary:
+{'execute': 'drive-mirror', 'arguments':{ 'device': 'colo-disk0', 'job-id': 'resync', 'target': 'nbd://127.0.0.1:9999/parent0', 'mode': 'existing', 'format': 'raw', 'sync': 'full'} }
+
+Wait until disk is synced, then:
+{'execute': 'stop'}
+{'execute': 'block-job-cancel', 'arguments':{ 'device': 'resync' } }
 
-5. Failover test
-You can kill Primary VM and run 'x_colo_lost_heartbeat' in Secondary VM's
-monitor at the same time, then SVM will failover and client will not detect this
-change.
+{'execute': 'human-monitor-command', 'arguments':{ 'command-line': 'drive_add -n buddy driver=replication,mode=primary,file.driver=nbd,file.host=127.0.0.1,file.port=9999,file.export=parent0,node-name=replication0'}}
+{'execute': 'x-blockdev-change', 'arguments':{ 'parent': 'colo-disk0', 'node': 'replication0' } }
 
-Before issuing '{ "execute": "x-colo-lost-heartbeat" }' command, we have to
-issue block related command to stop block replication.
-Primary:
-  Remove the nbd child from the quorum:
-  { 'execute': 'x-blockdev-change', 'arguments': {'parent': 'colo-disk0', 'child': 'children.1'}}
-  { 'execute': 'human-monitor-command','arguments': {'command-line': 'drive_del blk-buddy0'}}
-  Note: there is no qmp command to remove the blockdev now
+{'execute': 'object-add', 'arguments':{ 'qom-type': 'filter-mirror', 'id': 'm0', 'props': { 'insert': 'before', 'position': 'id=rew0', 'netdev': 'hn0', 'queue': 'tx', 'outdev': 'mirror0' } } }
+{'execute': 'object-add', 'arguments':{ 'qom-type': 'filter-redirector', 'id': 'redire0', 'props': { 'insert': 'before', 'position': 'id=rew0', 'netdev': 'hn0', 'queue': 'rx', 'indev': 'compare_out' } } }
+{'execute': 'object-add', 'arguments':{ 'qom-type': 'filter-redirector', 'id': 'redire1', 'props': { 'insert': 'before', 'position': 'id=rew0', 'netdev': 'hn0', 'queue': 'rx', 'outdev': 'compare0' } } }
+{'execute': 'object-add', 'arguments':{ 'qom-type': 'iothread', 'id': 'iothread1' } }
+{'execute': 'object-add', 'arguments':{ 'qom-type': 'colo-compare', 'id': 'comp0', 'props': { 'primary_in': 'compare0-0', 'secondary_in': 'compare1', 'outdev': 'compare_out0', 'iothread': 'iothread1' } } }
 
-Secondary:
-  The primary host is down, so we should do the following thing:
-  { 'execute': 'nbd-server-stop' }
+{'execute': 'migrate-set-capabilities', 'arguments':{ 'capabilities': [ {'capability': 'x-colo', 'state': true } ] } }
+{'execute': 'migrate', 'arguments':{ 'uri': 'tcp:127.0.0.1:9998' } }
 
 == TODO ==
-1. Support continuous VM replication.
-2. Support shared storage.
-3. Develop the heartbeat part.
-4. Reduce checkpoint VM’s downtime while doing checkpoint.
+1. Support shared storage.
+2. Develop the heartbeat part.
+3. Reduce checkpoint VM’s downtime while doing checkpoint.
diff --git a/docs/block-replication.txt b/docs/block-replication.txt
index XXXXXXX..XXXXXXX 100644
--- a/docs/block-replication.txt
+++ b/docs/block-replication.txt
@@ -XXX,XX +XXX,XX @@ blocks that are already in QEMU.
              ^            ||                            .----------
              |            ||                            | Secondary
         1 Quorum          ||                            '----------
-         /      \         ||
-        /        \        ||
-   Primary    2 filter
-     disk         ^                                                             virtio-blk
-                  |                                                                  ^
-                3 NBD  ------->  3 NBD                                               |
+         /      \         ||                                                           virtio-blk
+        /        \        ||                                                               ^
+   Primary    2 filter                                                                     |
+     disk         ^                                                                   7 Quorum
+                  |                                                                    /
+                3 NBD  ------->  3 NBD                                                /
                 client    ||     server                                          2 filter
                           ||        ^                                                ^
 --------.                 ||        |                                                |
@@ -XXX,XX +XXX,XX @@ any state that would otherwise be lost by the speculative write-through
 of the NBD server into the secondary disk. So before block replication,
 the primary disk and secondary disk should contain the same data.
 
+7) The secondary also has a quorum node, so after secondary failover it
+can become the new primary and continue replication.
+
+
 == Failure Handling ==
 There are 7 internal errors when block replication is running:
 1. I/O error on primary disk
@@ -XXX,XX +XXX,XX @@ Primary:
      leading whitespace.
   5. The qmp command line must be run after running qmp command line in
      secondary qemu.
-  6. After failover we need remove children.1 (replication driver).
+  6. After primary failover we need remove children.1 (replication driver).
 
 Secondary:
   -drive if=none,driver=raw,file.filename=1.raw,id=colo1 \
-  -drive if=xxx,id=topxxx,driver=replication,mode=secondary,top-id=topxxx\
+  -drive if=none,id=childs1,driver=replication,mode=secondary,top-id=childs1
          file.file.filename=active_disk.qcow2,\
          file.driver=qcow2,\
          file.backing.file.filename=hidden_disk.qcow2,\
          file.backing.driver=qcow2,\
          file.backing.backing=colo1
+  -drive if=xxx,driver=quorum,read-pattern=fifo,id=top-disk1,\
+         vote-threshold=1,children.0=childs1
 
   Then run qmp command in secondary qemu:
     { 'execute': 'nbd-server-start',
@@ -XXX,XX +XXX,XX @@ Secondary:
   The primary host is down, so we should do the following thing:
   { 'execute': 'nbd-server-stop' }
 
+Promote Secondary to Primary:
+  see COLO-FT.txt
+
 TODO:
-1. Continuous block replication
-2. Shared disk
+1. Shared disk
-- 
2.5.0

From: Stefan Hajnoczi <stefanha@redhat.com>

The L2TPv3 RFC number is 3931:
https://tools.ietf.org/html/rfc3931

Reported-by: Henrik Johansson <henrikjohansson@rocketmail.com>
Reviewed-by: Stefan Weil <sw@weilnetz.de>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 qemu-options.hx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/qemu-options.hx b/qemu-options.hx
index XXXXXXX..XXXXXXX 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -XXX,XX +XXX,XX @@ DEF("netdev", HAS_ARG, QEMU_OPTION_netdev,
     "                Linux kernel 3.3+ as well as most routers can talk\n"
     "                L2TPv3. This transport allows connecting a VM to a VM,\n"
     "                VM to a router and even VM to Host. It is a nearly-universal\n"
-    "                standard (RFC3391). Note - this implementation uses static\n"
+    "                standard (RFC3931). Note - this implementation uses static\n"
     "                pre-configured tunnels (same as the Linux kernel).\n"
     "                use 'src=' to specify source address\n"
     "                use 'dst=' to specify destination address\n"
@@ -XXX,XX +XXX,XX @@ Example (send packets from host's 1.2.3.4):
 @end example
 
 @item -netdev l2tpv3,id=@var{id},src=@var{srcaddr},dst=@var{dstaddr}[,srcport=@var{srcport}][,dstport=@var{dstport}],txsession=@var{txsession}[,rxsession=@var{rxsession}][,ipv6][,udp][,cookie64][,counter][,pincounter][,txcookie=@var{txcookie}][,rxcookie=@var{rxcookie}][,offset=@var{offset}]
-Configure a L2TPv3 pseudowire host network backend. L2TPv3 (RFC3391) is a
+Configure a L2TPv3 pseudowire host network backend. L2TPv3 (RFC3931) is a
 popular protocol to transport Ethernet (and other Layer 2) data frames between
 two systems. It is present in routers, firewalls and the Linux kernel
 (from version 3.3 onwards).
-- 
2.5.0

The following changes since commit d90f154867ec0ec22fd719164b88716e8fd48672:

Merge remote-tracking branch 'remotes/dg-gitlab/tags/ppc-for-6.1-20210504' into staging (2021-05-05 20:29:14 +0100)

are available in the git repository at:

https://github.com/jasowang/qemu.git tags/net-pull-request

for you to fetch changes up to 4f8a39494aded9f2026a26b137378ea2ee3d5338:

tap-bsd: Remove special casing for older OpenBSD releases (2021-05-27 11:03:55 +0800)

----------------------------------------------------------------

----------------------------------------------------------------
Brad Smith (1):
      tap-bsd: Remove special casing for older OpenBSD releases

Guenter Roeck (1):
      hw/net/imx_fec: return 0xffff when accessing non-existing PHY

Laurent Vivier (1):
      virtio-net: failover: add missing remove_migration_state_change_notifier()

hw/net/imx_fec.c    | 8 +++-----
 hw/net/trace-events | 2 ++
 hw/net/virtio-net.c | 1 +
 net/tap-bsd.c       | 8 --------
 4 files changed, 6 insertions(+), 13 deletions(-)

From: Guenter Roeck <linux@roeck-us.net>

If a PHY does not exist, attempts to read from it should return 0xffff.
Otherwise the Linux kernel will believe that a PHY is there and select
the non-existing PHY. This in turn will result in network errors later
on since the real PHY is not selected or configured.

Since reading from or writing to a non-existing PHY is not an emulation
error, replace guest error messages with traces.

Fixes: 461c51ad4275 ("Add a phy-num property to the i.MX FEC emulator")
Cc: Jean-Christophe Dubois <jcd@tribudubois.net>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
Tested-by: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 hw/net/imx_fec.c    | 8 +++-----
 hw/net/trace-events | 2 ++
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/hw/net/imx_fec.c b/hw/net/imx_fec.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/imx_fec.c
+++ b/hw/net/imx_fec.c
@@ -XXX,XX +XXX,XX @@ static uint32_t imx_phy_read(IMXFECState *s, int reg)
     uint32_t phy = reg / 32;
 
     if (phy != s->phy_num) {
-        qemu_log_mask(LOG_GUEST_ERROR, "[%s.phy]%s: Bad phy num %u\n",
-                      TYPE_IMX_FEC, __func__, phy);
-        return 0;
+        trace_imx_phy_read_num(phy, s->phy_num);
+        return 0xffff;
     }
 
     reg %= 32;
@@ -XXX,XX +XXX,XX @@ static void imx_phy_write(IMXFECState *s, int reg, uint32_t val)
     uint32_t phy = reg / 32;
 
     if (phy != s->phy_num) {
-        qemu_log_mask(LOG_GUEST_ERROR, "[%s.phy]%s: Bad phy num %u\n",
-                      TYPE_IMX_FEC, __func__, phy);
+        trace_imx_phy_write_num(phy, s->phy_num);
         return;
     }
 
diff --git a/hw/net/trace-events b/hw/net/trace-events
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/trace-events
+++ b/hw/net/trace-events
@@ -XXX,XX +XXX,XX @@ i82596_channel_attention(void *s) "%p: Received CHANNEL ATTENTION"
 
 # imx_fec.c
 imx_phy_read(uint32_t val, int phy, int reg) "0x%04"PRIx32" <= phy[%d].reg[%d]"
+imx_phy_read_num(int phy, int configured) "read request from unconfigured phy %d (configured %d)"
 imx_phy_write(uint32_t val, int phy, int reg) "0x%04"PRIx32" => phy[%d].reg[%d]"
+imx_phy_write_num(int phy, int configured) "write request to unconfigured phy %d (configured %d)"
 imx_phy_update_link(const char *s) "%s"
 imx_phy_reset(void) ""
 imx_fec_read_bd(uint64_t addr, int flags, int len, int data) "tx_bd 0x%"PRIx64" flags 0x%04x len %d data 0x%08x"
-- 
2.7.4

From: Laurent Vivier <lvivier@redhat.com>

In the failover case configuration, virtio_net_device_realize() uses an
add_migration_state_change_notifier() to add a state notifier, but this
notifier is not removed by the unrealize function when the virtio-net
card is unplugged.

If the card is unplugged and a migration is started, the notifier is
called and as it is not valid anymore QEMU crashes.

This patch fixes the problem by adding the
remove_migration_state_change_notifier() in virtio_net_device_unrealize().

The problem can be reproduced with:

$ qemu-system-x86_64 -enable-kvm -m 1g -M q35 \
    -device pcie-root-port,slot=4,id=root1 \
    -device pcie-root-port,slot=5,id=root2 \
    -device virtio-net-pci,id=net1,mac=52:54:00:6f:55:cc,failover=on,bus=root1 \
    -monitor stdio disk.qcow2
  (qemu) device_del net1
  (qemu) migrate "exec:gzip -c > STATEFILE.gz"

Thread 1 "qemu-system-x86" received signal SIGSEGV, Segmentation fault.
  0x0000000000000000 in ?? ()
  (gdb) bt
  #0  0x0000000000000000 in  ()
  #1  0x0000555555d726d7 in notifier_list_notify (...)
      at .../util/notify.c:39
  #2  0x0000555555842c1a in migrate_fd_connect (...)
      at .../migration/migration.c:3975
  #3  0x0000555555950f7d in migration_channel_connect (...)
      error@entry=0x0) at .../migration/channel.c:107
  #4  0x0000555555910922 in exec_start_outgoing_migration (...)
      at .../migration/exec.c:42

Reported-by: Igor Mammedov <imammedo@redhat.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Laurent Vivier <lvivier@redhat.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 hw/net/virtio-net.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/net/virtio-net.c
+++ b/hw/net/virtio-net.c
@@ -XXX,XX +XXX,XX @@ static void virtio_net_device_unrealize(DeviceState *dev)
 
     if (n->failover) {
         device_listener_unregister(&n->primary_listener);
+        remove_migration_state_change_notifier(&n->migration_state);
     }
 
     max_queues = n->multiqueue ? n->max_queues : 1;
-- 
2.7.4

From: Brad Smith <brad@comstyle.com>

OpenBSD added support for tap(4) 10 releases ago.

Remove the special casing for older releases.

Signed-off-by: Brad Smith <brad@comstyle.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 net/tap-bsd.c | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/net/tap-bsd.c b/net/tap-bsd.c
index XXXXXXX..XXXXXXX 100644
--- a/net/tap-bsd.c
+++ b/net/tap-bsd.c
@@ -XXX,XX +XXX,XX @@
 #include <net/if_tap.h>
 #endif
 
-#if defined(__OpenBSD__)
-#include <sys/param.h>
-#endif
-
 #ifndef __FreeBSD__
 int tap_open(char *ifname, int ifname_size, int *vnet_hdr,
              int vnet_hdr_required, int mq_required, Error **errp)
@@ -XXX,XX +XXX,XX @@ int tap_open(char *ifname, int ifname_size, int *vnet_hdr,
         if (*ifname) {
             snprintf(dname, sizeof dname, "/dev/%s", ifname);
         } else {
-#if defined(__OpenBSD__) && OpenBSD < 201605
-            snprintf(dname, sizeof dname, "/dev/tun%d", i);
-#else
             snprintf(dname, sizeof dname, "/dev/tap%d", i);
-#endif
         }
         TFR(fd = open(dname, O_RDWR));
         if (fd >= 0) {
-- 
2.7.4