> http://www.opensolaris.org/os/project/clearview/docs/linkagg.pdf
General:
* As with the other documents, I would avoid using link names
like "mylink0", as they become confusing when we need to
talk about them at the IP layer (e.g., unplumbing them).
* The document tends to the use the term "interface" where it
would be more accurate and precise to use "link". Similarly,
"IP interface" should be used when talking about an interface at
the IP layer. The bullet at the bottom of page 68 is a good
example of the confusion that case result, since it assumes the
reader understands that "plumbed" means "has an IP interface
created over it".
* I'm strongly opposed to the amount of discussion of the way
previous releases worked. For instance, on two separate
occasions (page 69 and page 72), the text talks about the change
from -l to -L for LACP mode. Given that backward compatibility
has been preserved (that is, we still accept <-l mode>), I don't
see why we are twisting the reader up in all this trivia. The
documentation should describe the current release, and the
current release prefers -L (and -T).
* Somewhere, we should help the administrator choose between
deploying link aggregations and IPMP.
Specifics:
* Page 65, last bullet: Should be "The entire aggregation can be
assigned one IP address."
* Figure 6-2: The legend doesn't seem to align with the figure --
shouldn't there be an aggr1 and aggr2 here somewhere?
* Page 69, second bullet from the top: No longer applies post-UV;
needs to be removed.
* Page 69: "Link Names and the dladm Command" states "Beginning
with the current release, key is marked obsolete". Given that
the current release will continue to change, this doesn't seem
right; we should mention the release that made it obsolete.
* Page 71, step 8D: Replace "/etc/hostname[6].aggr-link" with
"/etc/hostname.<aggr> and/or /etc/hostname6.<aggr>" (with
<aggr> being "aggr" in italics.
* Page 72, top of the page: "agg" should be "aggr". Other
instances of this need to be fixed as well.
* Page 73: "How to Add a Link to an Aggregation" makes it seem
like the steps are entirely done on the local system -- but
there's also likely configuration outside of the system that
needs to be done, such as reconfiguring switches or recabling.
--
meem
