James Carlson wrote:
> I was referring to the open source version, which is quite different
> from the one in OpenSolaris today. (Particularly in the kernel
> modules, which we had to rewrite substantially to pass ON muster.)
>
> I went back to the external code and had another look. It does set
> dl_addr_length and dl_sap_length both to 4, so it looks like this one
> should "just work." That doesn't match what I thought I remembered
> from testing, though.
OK
> Erik Nordmark writes:
>> The offending code I was thinking of is this code in ip_if.c
>> if ((ipif->ipif_flags & IPIF_POINTOPOINT) == 0) {
>
> There are more offenses here than that. There's no way that
> IFF_POINTOPOINT can be in ipif_flags. It has to be an ill_flags or
> (better still) phyint_flags value because it's a property of the
> underlying interface itself, not a per-logical-interface matter.
Yes, if we change it so IFF_POINTOPOINT, IFF_BROADCAST are only set
based on the dl_info_ack then it makes sense to move those flags from
the ipif to the ill.
Erik
_______________________________________________
networking-discuss mailing list
[email protected]
