[LU-1730] cfs_lockdep_set_class() typo in cl_object.c

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.4.0
• Affects Version/s: Lustre 2.3.0
• Severity: 3
• Assignee: Jinshan Xiong (Inactive)
• Reporter: Ned Bass

Description

Reading through the code I noticed a probable typo in cl_object.c.

  72 /** Lock class of cl_object_header::coh_page_guard */                           
  73 static cfs_lock_class_key_t cl_page_guard_class;                                
  74 /** Lock class of cl_object_header::coh_lock_guard */                           
  75 static cfs_lock_class_key_t cl_lock_guard_class;                                
  76 /** Lock class of cl_object_header::coh_attr_guard */                           
  77 static cfs_lock_class_key_t cl_attr_guard_class;                                
  78                                                                                 
  79 extern __u32 lu_context_tags_default;                                           
  80 extern __u32 lu_session_tags_default;                                           
  81 /**                                                                             
  82  * Initialize cl_object_header.                                                 
  83  */                                                                             
  84 int cl_object_header_init(struct cl_object_header *h)                           
  85 {                                                                               
  86         int result;                                                             
  87                                                                                 
  88         ENTRY;                                                                  
  89         result = lu_object_header_init(&h->coh_lu);                             
  90         if (result == 0) {                                                      
  91                 cfs_spin_lock_init(&h->coh_page_guard);                         
  92                 cfs_spin_lock_init(&h->coh_lock_guard);                         
  93                 cfs_spin_lock_init(&h->coh_attr_guard);                         
  94                 cfs_lockdep_set_class(&h->coh_attr_guard, &cl_page_guard_class);
  95                 cfs_lockdep_set_class(&h->coh_attr_guard, &cl_lock_guard_class);
  96                 cfs_lockdep_set_class(&h->coh_attr_guard, &cl_attr_guard_class);  

Those last three lines have the same first argument, but I think the first- and second-to-last should be &h->coh_page_guard and &h->coh_lock_guard, respecitvely.