[LU-15584] ppc64le build failure: llog_reader.c:921:42: error: format '%llx' expects argument of type 'long long unsigned int', but argument 3 has type 'long unsigned int'

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.15.0
• Affects Version/s: Lustre 2.15.0
• Labels: llnl llnlfixready
• Environment: TOSS 4.3-3 4.18.0-348.12.2.1toss.t4.x86_64 ; lustre 2.15.0_0RC2_1.lnll (based on v2_15_0-RC2) ; ppc64le
• Severity: 3
• Assignee: Gian-Carlo Defazio
• Reporter: Gian-Carlo Defazio

Description

While attempting to build a new local branch based on v2_15_0-RC2 with local patches, the ppc64le build failed due to a printf type error.

llog_reader.c: In function 'print_unlink_rec':                                                                                                                                       
llog_reader.c:921:42: error: format '%llx' expects argument of type 'long long unsigned int', but argument 3 has type 'long unsigned int' [-Werror=format=]                          
  printf("unlink record id:0x%x target %llx:%x:%x\n",                                                                                                                                
                                       ~~~^                                                                                                                                          
                                       %lx                                                                                                                                           
llog_reader.c: In function 'print_setattr64_rec':                                                                                                                                    
llog_reader.c:937:9: error: format '%llx' expects argument of type 'long long unsigned int', but argument 6 has type 'long unsigned int' [-Werror=format=]                           
  printf("setattr64 record id:0x%x target "DFID" valid %llx uid %u:%u gid %u:%u\n",                                                                                                  
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~                                                                                                                                          
llog_reader.c:937:58: note: format string is defined here                                                                                                                            
  printf("setattr64 record id:0x%x target "DFID" valid %llx uid %u:%u gid %u:%u\n",                                                                                                  
                                                       ~~~^                                                                                                                          
                                                       %lx                                                                                                                           
llog_reader.c: In function 'print_setattr64_rec_v2':                                                                                                                                 
llog_reader.c:949:9: error: format '%llx' expects argument of type 'long long unsigned int', but argument 6 has type 'long unsigned int' [-Werror=format=]                           
  printf("setattr64 v2 record id:0x%x target "DFID" valid %llx uid %u:%u gid %u:%u prj %u\n",                                                                                        
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~                                                                                                                                       
llog_reader.c:949:61: note: format string is defined here                                                                                                                            
  printf("setattr64 v2 record id:0x%x target "DFID" valid %llx uid %u:%u gid %u:%u prj %u\n",                                                                                        
                                                          ~~~^                                                                                                                       
                                                          %lx    

There are 3 additional places in the file llog_reader.c that attempt to print a value returned by __le64_to_cpu which do not match the %llx used in the format string.

Comments

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48307/
Subject: LU-15584 utils: ppc64le __le64_to_cpu type mismatch
Project: fs/lustre-release
Branch: b2_12
Current Patch Set:
Commit: 30f4f34c3cd17585f5367a875e60abbecc63ab20

[Gerrit Updater]

"Etienne AUJAMES <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/48307
Subject: LU-15584 utils: ppc64le __le64_to_cpu type mismatch
Project: fs/lustre-release
Branch: b2_12
Current Patch Set: 1
Commit: 60eeca873193c1e90a024c8128d4c499c7475375

[Peter Jones]

Landed for 2.15

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/46588/
Subject: LU-15584 utils: ppc64le __le64_to_cpu type mismatch
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 131f559c5a241f15b8a91009968dfbd9a5dcddeb

[Gerrit Updater]

"Gian-Carlo DeFazio <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/46588
Subject: LU-15584 utils: ppc64le __le64_to_cpu type mismatch
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 509f0d56c187260a1e46db7ce18f6421c735e74b

[Olaf Faaland]

This issue was introduced by https://review.whamcloud.com/#/c/44591/,

commit 80447caf980699fd1b0118e8f7c11e48aead04ce
Author: Alexander Zarochentsev <[e-mail]>
Date:   Fri Jul 16 22:16:29 2021 +0300

    LU-14926 utils: print unlink and setattr recs in llog_reader
    
    Enhance llog_reader to print unlink and setattr llog records
    correctly.
    
    HPE-bug-id: LUS-10220

from LU-14926