> I've implemented what is IMO a cleaner fix: > > http://cr.opensolaris.org/~seb/physaddr-fix/ > > DL_NOTE_PHYS_ADDR/DL_CURR_DEST_ADDR is only sent up for datalinks that > have a destination, and libdlpi drops notifications for address types it > doesn't know about (which is everything except for DL_CURR_PHYS_ADDR). > To me, this makes the most sense.
I'm fine with that, provided that libdlpi is enhanced to have some way to report DL_CURR_DEST_ADDR indications. > I'm currently in the process of testing this from the IP tunnel > perspective, and it's fine. This is as expected, as the fix is quite > unobtrusive to the IP tunnel code. I'd be interested in verifying that > this fix addresses your original problem (I'd bet money that it does), > but I don't think we can be completely sure of that until I putback and > you bringover to your child. Any better ideas? That's a fine way to go. Bringing the fix over is easy. -- meem
