The branch main has been updated by kp:

URL: 
https://cgit.FreeBSD.org/src/commit/?id=9f2e5184173f6af70306678b018270df9a9600f2

commit 9f2e5184173f6af70306678b018270df9a9600f2
Author:     Thomas Kurschel <topi...@gmx.net>
AuthorDate: 2021-03-15 13:28:52 +0000
Commit:     Kristof Provost <k...@freebsd.org>
CommitDate: 2021-03-17 18:18:14 +0000

    pfsync: Unconditionally push packets when requesting state updates
    
    When we request a bulk sync we need to ensure we actually send out that
    request, not just buffer it until we have enough data to send a full
    packet.
    
    PR:             254236
    MFC after:      2 weeks
    Differential Revision:  https://reviews.freebsd.org/D29271
---
 sys/netpfil/pf/if_pfsync.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/sys/netpfil/pf/if_pfsync.c b/sys/netpfil/pf/if_pfsync.c
index cf2ff2ef0926..06bad556e885 100644
--- a/sys/netpfil/pf/if_pfsync.c
+++ b/sys/netpfil/pf/if_pfsync.c
@@ -1960,7 +1960,7 @@ pfsync_request_update(u_int32_t creatorid, u_int64_t id)
                nlen += sizeof(struct pfsync_subheader);
 
        if (b->b_len + nlen > sc->sc_ifp->if_mtu) {
-               pfsync_sendout(1, 0);
+               pfsync_sendout(0, 0);
 
                nlen = sizeof(struct pfsync_subheader) +
                    sizeof(struct pfsync_upd_req);
@@ -1968,6 +1968,8 @@ pfsync_request_update(u_int32_t creatorid, u_int64_t id)
 
        TAILQ_INSERT_TAIL(&b->b_upd_req_list, item, ur_entry);
        b->b_len += nlen;
+
+       pfsync_push(b);
 }
 
 static bool
_______________________________________________
dev-commits-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/dev-commits-src-all
To unsubscribe, send any mail to "dev-commits-src-all-unsubscr...@freebsd.org"

Reply via email to