Hi,

I patched on side of this tandem
and had following setup:

fw1: em0 mtu 9000, pfsync0 mtu 2048
fw2: em0 mtu 9000, pfsync0 mtu 9000

This produced "pfsync: failed to receive bulk update".
If I change back to mtu 2048 states get propagated.

I also changed hardmtu as dlg@ suggested.


//maxim

On 10/23/2011 07:47 AM, David Gwynne wrote:
mike,

might have to tweak hardmtu in attach too. maybe.

dlg

On 23/10/2011, at 6:18 AM, Mike Belopuhov wrote:

On Sat, Oct 22, 2011 at 20:14 +0200, Maxim Bourmistrov wrote:
On both sides I use em(4) with MTU 9000.
Then tried to set the same value to the pfsync with success (ifconfig pfsync0 
mtu 9000), but the actual value I see is 2048.

ugh. i thought you've fixed up the source code.
i'm curious if it'll still work with a smaller mtu on the physical
interface :-)

Index: net/if_pfsync.c
===================================================================
RCS file: /cvs/src/sys/net/if_pfsync.c,v
retrieving revision 1.169
diff -u -p -r1.169 if_pfsync.c
--- net/if_pfsync.c     20 Oct 2011 08:57:26 -0000      1.169
+++ net/if_pfsync.c     22 Oct 2011 18:17:44 -0000
@@ -1294,8 +1294,8 @@ pfsyncioctl(struct ifnet *ifp, u_long cm
                s = splnet();
                if (ifr->ifr_mtu<= PFSYNC_MINPKT)
                        return (EINVAL);
-               if (ifr->ifr_mtu>  MCLBYTES) /* XXX could be bigger */
-                       ifr->ifr_mtu = MCLBYTES;
+               if (ifr->ifr_mtu>  65536)
+                       ifr->ifr_mtu = 65536;
                if (ifr->ifr_mtu<  ifp->if_mtu)
                        pfsync_sendout();
                ifp->if_mtu = ifr->ifr_mtu;

Reply via email to