Sowmini.Varadhan at Sun.COM wrote: > As was discussed in inception review last week (seb-02), I have pulled > in the relevant pieces of mac_maxsdu_update() from Clearview gate to > replace mac_update_reg()- > > see > http://cr.opensolaris.org/~sowmini/macsdu/
Looks good to me. A couple of aesthetic questions: The get function returns both min and max values, while the update function is for max only. Is there never a need to change min? Is the lack of symmetry due to that? Some functions in the same group are called refresh, others update. Is there a subtle difference or just sloppy naming? -Artem
