[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]

Minh Diep ([e-mail]) uploaded a new patch: https://review.whamcloud.com/30264
Subject: LU-6210 utils: Use C99 initializer for lfsck_types_names
Project: fs/lustre-release
Branch: b2_10
Current Patch Set: 1
Commit: f77870de3c743b30d2481c3c546f74a875a33af3

[Steve Guminski (Inactive)]

Final patch landed for 2.11.

[Peter Jones]

Dare I ask - are there more such patches still to come?

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in 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:
Commit: 7e58fd07723483517dbe712ee671db416b6d3ac4

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in 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:
Commit: 60c9a2d351d40b5306b586d08847b6c7727b73cf

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/28479/
Subject: LU-6210 ptlrpc: Use C99 initializer in ptlrpc_register_rqbd()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 3af10d510078ddf050fbda2ef80bdc6920d89f8d

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in 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:
Commit: 0568e93539ebc1bbec601182f9faba4d43ed106b

[Gerrit Updater]

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

[Gerrit Updater]

Andreas Dilger ([e-mail]) uploaded a new patch: https://review.whamcloud.com/28592
Subject: LU-6210 utils: clean up optarg definitions
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: c15e377f7b360d73b15c91412ea43e950f64f07a

[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