Whamcloud - gitweb
LU-6210 ldlm: Change positional struct initializers to C99 43/23743/2
authorSteve Guminski <stephenx.guminski@intel.com>
Mon, 14 Nov 2016 14:02:13 +0000 (09:02 -0500)
committerOleg Drokin <oleg.drokin@intel.com>
Sun, 1 Jan 2017 02:00:49 +0000 (02:00 +0000)
This patch makes no functional changes.  Struct initializers in the
ldlm directory that use C89 or GCC-only syntax are updated to C99
syntax.

The C99 syntax prevents incorrect initialization if values are
accidently placed in the wrong position, allows changes in the struct
definition, and clears any members that are not given an explicit
value.

The following struct initializers have been updated:

lustre/ldlm/ldlm_extent.c:
struct interval_node_extent limiter
struct interval_node_extent ext

Test-Parameters: trivial
Signed-off-by: Steve Guminski <stephenx.guminski@intel.com>
Change-Id: I219f5b2bd4b96d92eaf120f7080e762d72eb3105
Reviewed-on: https://review.whamcloud.com/23743
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lustre/ldlm/ldlm_extent.c

index 6975201..374834f 100644 (file)
@@ -130,7 +130,10 @@ static void ldlm_extent_internal_policy_granted(struct ldlm_lock *req,
        __u64 req_start = req->l_req_extent.start;
        __u64 req_end = req->l_req_extent.end;
        struct ldlm_interval_tree *tree;
-       struct interval_node_extent limiter = { new_ex->start, new_ex->end };
+       struct interval_node_extent limiter = {
+               .start  = new_ex->start,
+               .end    = new_ex->end,
+       };
        int conflicting = 0;
        int idx;
        ENTRY;
@@ -139,7 +142,10 @@ static void ldlm_extent_internal_policy_granted(struct ldlm_lock *req,
 
        /* Using interval tree to handle the LDLM extent granted locks. */
         for (idx = 0; idx < LCK_MODE_NUM; idx++) {
-                struct interval_node_extent ext = { req_start, req_end };
+               struct interval_node_extent ext = {
+                       .start  = req_start,
+                       .end    = req_end,
+               };
 
                 tree = &res->lr_itree[idx];
                 if (lockmode_compat(tree->lit_mode, req_mode))