[LU-6210] change positional struct initializers to C99 initializers

• Type: Task
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.11.0, Lustre 2.10.3
• Labels: easy patch
• Assignee: Steve Guminski (Inactive)
• Reporter: Andreas Dilger

Description

In the libcfs and lnet code there are many places where positional struct initializers are used instead of named C99 initializers:

static struct netstrfns  libcfs_netstrfns[] = {
        {/* .nf_type      */  LOLND,
         /* .nf_name      */  "lo",
         /* .nf_modname   */  "klolnd",
         /* .nf_addr2str  */  libcfs_decnum_addr2str,
         /* .nf_str2addr  */  libcfs_lo_str2addr,
         /* .nf_parse_addr*/  libcfs_num_parse,
         /* .nf_print_addrlist*/  libcfs_num_addr_range_print,
         /* .nf_match_addr*/      libcfs_num_match,
         /* .nf_is_contiguous */  cfs_num_is_contiguous,
         /* .nf_min_max   */      cfs_num_min_max},

instead of

static struct netstrfns  libcfs_netstrfns[] = {
        {
                 .nf_type              =  LOLND,
                 .nf_name              = "lo",
                 .nf_modname           = "klolnd",
                 .nf_addr2str          = libcfs_decnum_addr2str,
                 .nf_str2addr          = libcfs_lo_str2addr,
                 .nf_parse_addr        = libcfs_num_parse,
                 .nf_print_addrlist    = libcfs_num_addr_range_print,
                 .nf_match_addr        = libcfs_num_match,
                 .nf_is_contiguous     = cfs_num_is_contiguous,
                 .nf_min_max           = cfs_num_min_max},

This was done because WinNT didn't have C99 initializer support, but we don't care about that anymore.

All of the positional struct initializers in libcfs, lnet, and lustre should be changed to named initializers, probably in several independent patches for different subdirectories.

Comments

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/28417/
Subject: LU-6210 utils: Use C99 struct initializers in lsnapshot
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 6e27028b08eace5fc9a141d33499940ef00c9761

[Gerrit Updater]

Steve Guminski ([e-mail]) uploaded a new patch: https://review.whamcloud.com/28479
Subject: LU-6210 lnet: Use C99 initializer in ptlrpc_register_rqbd()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: de18309b95d97123dc6fdd932dee1487e378d76c

[Gerrit Updater]

Steve Guminski ([e-mail]) uploaded a new patch: https://review.whamcloud.com/28436
Subject: LU-6210 lnet: Use C99 struct initializer in framework.c
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: e142ee7cc568319d271de821bf1aeb0e34073335

[Gerrit Updater]

Steve Guminski ([e-mail]) uploaded a new patch: https://review.whamcloud.com/28423
Subject: LU-6210 utils: Use C99 struct initializers in lnetctl
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: d0ad876c7011cfc1dc748a4f217206b613b3c539

[Gerrit Updater]

Steve Guminski ([e-mail]) uploaded a new patch: https://review.whamcloud.com/28421
Subject: LU-6210 utils: Use C99 struct initializers in lfs_getdirstripe
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 7f7e46213f488a66b09e9be3b891c04bbdf082db

[Gerrit Updater]

Steve Guminski ([e-mail]) uploaded a new patch: https://review.whamcloud.com/28417
Subject: LU-6210 lnet: Use C99 struct initializers in lsnapshot
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 20aed5ac4bb96ae5e5a2626d9a80fc0a61c335d6

[Steve Guminski (Inactive)]

Yes, it looks like https://review.whamcloud.com/#/c/27522/ is the only committed patch that has not yet been landed.  I just re-ran the Python script attached to this ticket, and discovered that there are still a few occurrences of C89 initializers.  I'll submit patches for them shortly.

[James A Simmons]

Only one patch left maybe? Steve is that the case

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/27883/
Subject: LU-6210 utils: Use C99 initializer in jt_nodemap_del_range()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 2b90c15b72c25711793cc4ea46b964098131c736

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/27881/
Subject: LU-6210 utils: Use C99 initializer in jt_nodemap_test_id()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: dad40c1980cbe67ec60055258f435cc3369745db

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/27825/
Subject: LU-6210 utils: Use C99 struct initializer in mount_utils_zfs.c
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 990890eacf757d73fc87a83bfa703a51b739a05c