Just a quick note to say "thanks" for taking the time to record this change.
-- Garrett
Sebastien Roy wrote:
> I'm sponsoring the following self-reviewed case for Cathy Zhou. I've
> marked the case as closed approved automatic.
>
> ---
>
> Release binding: patch
>
> 1 Introduction
> ===============
>
> This fast-track proposes to change the signature of the following two
> GLDv3 functions, which were introduced in PSARC/2006/249.
>
> - MAC client interface: mac_open()
> - DLS clients interface: dls_create()
>
> 2. Details
> ===========
>
> 2.2 mac_open()
> ---------------
>
> The current signature of mac_open() is:
>
> int mac_open(const char *macname, uint_t ddi_instance,
> mac_handle_t *mhp);
>
> To use this interface, MAC clients have to determine the correct
> "ddi_instance" by comparing the driver name part of the "macname"
> argument with all known pseudo driver names, something like:
>
> ddi_parse(macname, drv, &ddi_instance);
> if ((strcmp(drv, "aggr") == 0) || (strcmp(drv, "vnic") == 0))
> ddi_instance = 0;
> mac_open(macname, ddi_instance, &mh);
>
> In fact, the comparision logic is already in the mac_open() function,
> therefore the "ddi_instance" argument is no longer needed. As a
> result, the function signature will be changed to:
>
> int mac_open(const char *macname, mac_handle_t *mhp);
>
> 2.1 dls_create()
> ----------------
>
> The current signature of dls_create() is:
>
> int dls_create(const char *linkname, const char *macname,
> uint_t ddi_instance);
>
> This fast track proposes to remove the need of the "ddi_instance"
> argument, and change the function signature to be:
>
> int dls_create(const char *linkname, const char *macname);
>
> 3. Interface Table
> ===================
>
> +---------------------------------------------------------+
> | | Interfaces Exported | |
> |-----------------+-----------------------+---------------|
> | Interface | Classification | Comments |
> |-----------------+-----------------------+---------------|
> | mac_open() | Consolidation Private | <sys/mac.h> |
> | dls_create() | Consolidation Private | <sys/dls.h> |
> +---------------------------------------------------------+
