<div dir="ltr">Thanks, that patch is working perfectly for me. I look forward to seeing this in the next release.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 3, 2015 at 10:11 AM, Ondrej Zajicek <span dir="ltr"><<a href="mailto:santiago@crfreenet.org" target="_blank">santiago@crfreenet.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Mar 03, 2015 at 09:48:41AM -0500, Clint Armstrong wrote:<br>
> So I have briefly tested a change that should allow OSPF broadcast over GRE<br>
> so long as the multicast flag is set on the GRE interface, enabling the<br>
> behavior you were initially expecting. I'd love to hear feedback on this,<br>
> especially if anyone is aware of problems this might cause.<br>
><br>
> diff --git sysdep/linux/netlink.c sysdep/linux/netlink.c<br>
> index 860c860..4b543a0 100644<br>
> --- sysdep/linux/netlink.c<br>
> +++ sysdep/linux/netlink.c<br>
> @@ -431,7 +431,7 @@ nl_parse_link(struct nlmsghdr *h, int scan)<br>
>   f.flags |= IF_MULTIACCESS | IF_LOOPBACK | IF_IGNORE;<br>
>        else if (fl & IFF_POINTOPOINT) /* PtP */<br>
>   f.flags |= IF_MULTICAST;<br>
> -      else if (fl & IFF_BROADCAST) /* Broadcast */<br>
> +      else if (fl & (IFF_BROADCAST || IFF_MULTICAST)) /* Broadcast */<br>
>   f.flags |= IF_MULTIACCESS | IF_BROADCAST | IF_MULTICAST;<br>
>        else<br>
>   f.flags |= IF_MULTIACCESS; /* NBMA */<br>
<br>
</span>This would work most likely without problems.<br>
<br>
I would use just this one:<br>
<br>
diff --git a/sysdep/linux/netlink.c b/sysdep/linux/netlink.c<br>
index 860c860..48dd8ba 100644<br>
--- a/sysdep/linux/netlink.c<br>
+++ b/sysdep/linux/netlink.c<br>
@@ -436,6 +436,9 @@ nl_parse_link(struct nlmsghdr *h, int scan)<br>
<span class="">       else<br>
        f.flags |= IF_MULTIACCESS;      /* NBMA */<br>
<br>
</span>+      if (fl & IFF_MULTICAST)<br>
+       f.flags |= IF_MULTICAST;<br>
+<br>
       ifi = if_update(&f);<br>
<br>
       if (!scan)<br>
<div class="HOEnZb"><div class="h5"><br>
--<br>
Elen sila lumenn' omentielvo<br>
<br>
Ondrej 'Santiago' Zajicek (email: <a href="mailto:santiago@crfreenet.org">santiago@crfreenet.org</a>)<br>
OpenPGP encrypted e-mails preferred (KeyID 0x11DEADC3, <a href="http://wwwkeys.pgp.net" target="_blank">wwwkeys.pgp.net</a>)<br>
"To err is human -- to blame it on a computer is even more so."<br>
</div></div></blockquote></div><br></div>