The support for the "key" dladm(1M) parameter to refer to an aggregation is there strictly for backward compatibility. The preferred method to refer to an aggregation link is by name, and it's awkward in the documentation to always say that one can either use an integer key or a link name. Such statements are numerous since there are a number of aggregation dladm subcommands.
I'm thinking that it would be appropriate to remove all references to keys from the documentation, and only leave support for it in the command as an undocumented thing to not break existing scripts and trained fingers... Do others see this as problematic? -Seb
