Hi,

kernel test robot noticed the following build errors:

[auto build test ERROR on jaegeuk-f2fs/dev-test]
[also build test ERROR on jaegeuk-f2fs/dev linus/master v6.6 next-20231107]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    
https://github.com/intel-lab-lkp/linux/commits/dslab-lzu-edu-cn/f2fs-the-name-of-a-struct-is-wrong-in-a-comment/20231106-201822
base:   https://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git 
dev-test
patch link:    
https://lore.kernel.org/r/20231106121418.3675-1-dslab%40lzu.edu.cn
patch subject: [PATCH] f2fs: the name of a struct is wrong in a comment.
config: parisc-randconfig-001-20231107 
(https://download.01.org/0day-ci/archive/20231107/[email protected]/config)
compiler: hppa-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): 
(https://download.01.org/0day-ci/archive/20231107/[email protected]/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <[email protected]>
| Closes: 
https://lore.kernel.org/oe-kbuild-all/[email protected]/

All error/warnings (new ones prefixed by >>):

   In file included from fs/f2fs/segment.c:9:
>> include/linux/f2fs_fs.h:474:66: error: expected ']' before 'SIT_JOURNA'
     474 |                                 sizeof(struct 
nat_journal_entry))SIT_JOURNA
         |                                                                  
^~~~~~~~~~
   include/linux/f2fs_fs.h:501:23: note: in expansion of macro 
'NAT_JOURNAL_RESERVED'
     501 |         __u8 reserved[NAT_JOURNAL_RESERVED];
         |                       ^~~~~~~~~~~~~~~~~~~~
>> include/linux/f2fs_fs.h:502:1: warning: no semicolon at end of struct or 
>> union
     502 | } __packed;
         | ^


vim +474 include/linux/f2fs_fs.h

   468  
   469  #define SUM_JOURNAL_SIZE        (F2FS_BLKSIZE - SUM_FOOTER_SIZE -\
   470                                  SUM_ENTRY_SIZE)
   471  #define NAT_JOURNAL_ENTRIES     ((SUM_JOURNAL_SIZE - 2) /\
   472                                  sizeof(struct nat_journal_entry))
   473  #define NAT_JOURNAL_RESERVED    ((SUM_JOURNAL_SIZE - 2) %\
 > 474                                  sizeof(struct 
 > nat_journal_entry))SIT_JOURNA
   475  #define SIT_JOURNAL_ENTRIES     ((SUM_JOURNAL_SIZE - 2) /\
   476                                  sizeof(struct sit_journal_entry))
   477  #define SIT_JOURNAL_RESERVED    ((SUM_JOURNAL_SIZE - 2) %\
   478                                  sizeof(struct sit_journal_entry))
   479  
   480  /* Reserved area should make size of f2fs_extra_info equals to
   481   * that of nat_journal and sit_journal.
   482   */
   483  #define EXTRA_INFO_RESERVED     (SUM_JOURNAL_SIZE - 2 - 8)
   484  
   485  /*
   486   * frequently updated NAT/SIT entries can be stored in the spare area in
   487   * summary blocks
   488   */
   489  enum {
   490          NAT_JOURNAL = 0,
   491          SIT_JOURNAL
   492  };
   493  
   494  struct nat_journal_entry {
   495          __le32 nid;
   496          struct f2fs_nat_entry ne;
   497  } __packed;
   498  
   499  struct nat_journal {
   500          struct nat_journal_entry entries[NAT_JOURNAL_ENTRIES];
   501          __u8 reserved[NAT_JOURNAL_RESERVED];
 > 502  } __packed;
   503  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki


_______________________________________________
Linux-f2fs-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to