Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 30 Dec 2009 08:50:50 -0800
From:      Xin LI <delphij@delphij.net>
To:        kevin <k@kevinkevin.com>
Cc:        freebsd-pf@freebsd.org
Subject:   Re: carpdev : bad value?
Message-ID:  <4B3B84EA.3060507@delphij.net>
In-Reply-To: <013c01ca896a$1baea710$530bf530$@com>
References:  <012c01ca891e$393e7860$abbb6920$@com>	<a78074950912292313t2b4d64ft38bb53cbf59242ab@mail.gmail.com>	<012f01ca8920$7789aa70$669cff50$@com>	<a78074950912292348j69ced05aj7d4efab58568aa51@mail.gmail.com> <013c01ca896a$1baea710$530bf530$@com>

next in thread | previous in thread | raw e-mail | index | archive | help

[-- Attachment #1 --]
On 2009/12/30 08:06, kevin wrote:
>> I have committed a patch that makes pf w/DSR setup work a week ago but
>> have not yet MFC'ed it, the patch can be directly applied against
>> 8-STABLE, though.
>
> Would you be able to share the patch with me? I am on 7.2-RELEASE, however.

The patch should work cleanly on RELENG_7_2 (sys/net).

However, based on your usage I don't think this is appropriate, it's a 
workaround and I believe it would cause problem for your setup which 
rely on bridge...

Cheers,
-- 
Xin LI <delphij@delphij.net>	http://www.delphij.net/
FreeBSD - The Power to Serve!	       Live free or die

[-- Attachment #2 --]
Index: if_ethersubr.c
===================================================================
--- if_ethersubr.c	(revision 201259)
+++ if_ethersubr.c	(working copy)
@@ -347,8 +347,18 @@
 	* Bridges require special output handling.
 	*/
 	if (ifp->if_bridge) {
+#if defined(INET) || defined(INET6)
+#ifdef DEV_CARP
+		if ((m->m_flags & ~(M_MCAST | M_BCAST)) == m->m_flags) {
+#endif
+#endif
 		BRIDGE_OUTPUT(ifp, m, error);
 		return (error);
+#if defined(INET) || defined(INET6)
+#ifdef DEV_CARP
+		}
+#endif
+#endif
 	}
 
 #ifdef DEV_CARP

Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4B3B84EA.3060507>