> >>> c. Since link properties are all initialized by the new mechanism, do > >>> we still need the dladm_init_linkprop() calls in various RCM modules > >>> and libdladm function calls (for example, i_dladm_aggr_up())? > >> > >> Will file a CR. > >> > > Why don't fix it now? > > Because it will require more code reviewers, more testing, delay > integration even further and upset our very patient management. These > calls do not hurt, they are just extra fat that can be removed later.
I disagree. Those calls will lead to confusion. Part of adding a proper persistence model for properties includes removing the old incomplete mechanism. -- meem
