吉藤英明 <hideaki.yoshif...@miraclelinux.com> writes: >>> +static int addrconf_ifid_random(u8 *eui, struct net_device *dev) >> +{ >> + get_random_bytes(eui, 8); >> + eui[0] |= 0x02; >> + return 0; >> +} >> + > > Since random identifier is locally assigned, drop the global bit > instead if setting it.
Yes, definitely. Thanks. I'm considering reusing __ipv6_regen_rndid() which already does this correctly, and also avoids some locally assigned addresses with special meanings. Another issue with the initial RFC is that every prefix will have a new random ifid, which isn't necessarily what the users expect. I wonder if it would be acceptable to abuse the rndid field for storing a "permanent" random ifid? Bjørn -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html