Anurag S. Maskey wrote:
> Hi all,
>
> I've fixed this bug (RFE) "6674402 sort out property management
> related constants and data-structures in {dld,mac}.h" and would like a
> code-review.
>
> Miles, can you also get a sign-off for the NVIDIA driver?
Nvidia has reviewed and approved the nge part of the code.
Please add the following comments in your RTI
The nge part of the code is approved by Nvidia:
SUN CR# 6674402 sort out property management related constants and
data-structures in {dld,mac}.h
Code Reviewed by: Paul Gyugyi(PGyugyi at nvidia.com)
PM Signoff: Brian Liclican
Regards,
Miles Xu
> Since this is my first bug fix, I would appreciate if this code review
> can be done in one to two weeks?
>
> The webrev is at http://cr.opensolaris.org/~sowmini/anurag/.
>
> Fixing this RFE involved renaming constants and variables with "dld"
> to "mac" and also moving the definition of these constants from
> /usr/src/uts/common/sys/dld.h to /usr/src/uts/common/sys/mac.h. Also,
> the following drivers include sys/mac.h rather than sys/dld.h: dld,
> mac, bge, e1000g, nge, nxge.
>
> The following shows how the renaming was done:
>
> OLD_NAME NEW_NAME
>
> dld_prop_id_t mac_prop_id_t
> dld_ioc_prop_t dld_ioc_macprop_t
> DLD_PROBUF_SIZE MAC_PROPBUF_SIZE
> DLD_PROP_DUPLEX MAC_PROP_DUPLEX
> DLD_PROP_SPEED MAC_PROP_SPEED
> DLD_PROP_STATUS MAC_PROP_STATUS
> DLD_PROP_AUTONEG MAC_PROP_AUTONEG
> DLD_PROP_MTU MAC_PROP_MTU
> DLD_PROP_FLOWCTRL MAC_PROP_FLOWCTRL
> DLD_PROP_VERSION MAC_PROP_VERSION
> DLD_PROP_ADV_1000FDX_CAP MAC_PROP_ADV_1000FDX_CAP
> DLD_PROP_EN_1000FDX_CAP MAC_PROP_EN_1000FDX_CAP
> DLD_PROP_ADV_1000HDX_CAP MAC_PROP_ADV_1000HDX_CAP
> DLD_PROP_EN_1000HDX_CAP MAC_PROP_EN_1000HDX_CAP
> DLD_PROP_ADV_100FDX_CAP MAC_PROP_ADV_1000FDX_CAP
> DLD_PROP_EN_100FDX_CAP MAC_PROP_EN_1000FDX_CAP
> DLD_PROP_ADV_100HDX_CAP MAC_PROP_ADV_1000HDX_CAP
> DLD_PROP_EN_100HDX_CAP MAC_PROP_EN_1000HDX_CAP
> DLD_PROP_ADV_10FDX_CAP MAC_PROP_ADV_1000FDX_CAP
> DLD_PROP_EN_10FDX_CAP MAC_PROP_EN_1000FDX_CAP
> DLD_PROP_ADV_10HDX_CAP MAC_PROP_ADV_1000HDX_CAP
> DLD_PROP_EN_10HDX_CAP MAC_PROP_EN_1000HDX_CAP
> DLD_PROP_PRIVATE MAC_PROP_PRIVATE
> DLD_DEFAULT MACPROP_DEFAULT
> DLD_PROP_ADV_100T4_CAP MAC_PROP_ADV_100T4_CAP
> DLD_PROP_EN_100T4_CAP MAC_PROP_EN_100T4_CA
> DLDIOCSETPROP DLDIOCSETMACPROP
> DLDIOCGETPROP DLDIOCGETMACPROP
> dld_ioc_prop_s dld_ioc_macprop_s
> DLD_PROP_VERSION MAC_PROP_VERSION
>
> Thanks,
> Anurag
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: min.xu.vcf
Type: text/x-vcard
Size: 138 bytes
Desc: not available
URL:
<http://mail.opensolaris.org/pipermail/brussels-dev/attachments/20080521/946c4b70/attachment.vcf>
