public inbox for [email protected]
 help / color / mirror / Atom feed
From: kernel test robot <[email protected]>
To: Ammar Faizi <[email protected]>,
	GNU/Weeb Mailing List <[email protected]>
Cc: [email protected]
Subject: [ammarfaizi2-block:google/android/kernel/common/android-4.14-stable 4872/9999] fs/incfs/format.c:377:23: sparse: sparse: cast truncates bits from constant value (5346434e49 becomes 46434e49)
Date: Thu, 4 Aug 2022 19:16:46 +0800	[thread overview]
Message-ID: <[email protected]> (raw)

tree:   https://github.com/ammarfaizi2/linux-block google/android/kernel/common/android-4.14-stable
head:   c529afa15a69594211793a68cb70c873508061df
commit: 57e7c47daa1cae8cb8c1450ad359027bac68fb9b [4872/9999] ANDROID: Incremental fs: fix magic compatibility again
config: i386-randconfig-s002 (https://download.01.org/0day-ci/archive/20220804/[email protected]/config)
compiler: gcc-7 (Ubuntu 7.5.0-6ubuntu2) 7.5.0
reproduce:
        # apt-get install sparse
        # sparse version: v0.6.4-39-gce1a6720-dirty
        # https://github.com/ammarfaizi2/linux-block/commit/57e7c47daa1cae8cb8c1450ad359027bac68fb9b
        git remote add ammarfaizi2-block https://github.com/ammarfaizi2/linux-block
        git fetch --no-tags ammarfaizi2-block google/android/kernel/common/android-4.14-stable
        git checkout 57e7c47daa1cae8cb8c1450ad359027bac68fb9b
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=i386 SHELL=/bin/bash drivers/md/ fs/incfs/

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <[email protected]>

sparse warnings: (new ones prefixed by >>)
>> fs/incfs/format.c:377:23: sparse: sparse: cast truncates bits from constant value (5346434e49 becomes 46434e49)
   fs/incfs/format.c:559:41: sparse: sparse: cast truncates bits from constant value (5346434e49 becomes 46434e49)
   In file included from include/linux/printk.h:7:0,
                    from include/linux/kernel.h:14,
                    from include/linux/list.h:9,
                    from include/linux/wait.h:7,
                    from include/linux/wait_bit.h:8,
                    from include/linux/fs.h:6,
                    from fs/incfs/format.c:5:
   fs/incfs/format.c: In function 'incfs_read_next_metadata_record':
   include/linux/kern_levels.h:5:18: warning: format '%ld' expects argument of type 'long int', but argument 2 has type 'size_t {aka unsigned int}' [-Wformat=]
    #define KERN_SOH "001"  /* ASCII Start Of Header */
                     ^
   include/linux/kern_levels.h:12:22: note: in expansion of macro 'KERN_SOH'
    #define KERN_WARNING KERN_SOH "4" /* warning conditions */
                         ^~~~~~~~
   include/linux/printk.h:308:9: note: in expansion of macro 'KERN_WARNING'
     printk(KERN_WARNING pr_fmt(fmt), ##__VA_ARGS__)
            ^~~~~~~~~~~~
   include/linux/printk.h:309:17: note: in expansion of macro 'pr_warning'
    #define pr_warn pr_warning
                    ^~~~~~~~~~
   fs/incfs/format.c:619:3: note: in expansion of macro 'pr_warn'
      pr_warn("incfs: The record is too large. Size: %ld",
      ^~~~~~~
   fs/incfs/format.c:619:52: note: format string is defined here
      pr_warn("incfs: The record is too large. Size: %ld",
                                                     ~~^
                                                     %d

vim +377 fs/incfs/format.c

318babce66a62d8 Eugene Zemtsov 2019-11-18  361  
318babce66a62d8 Eugene Zemtsov 2019-11-18  362  /*
318babce66a62d8 Eugene Zemtsov 2019-11-18  363   * Write a backing file header
318babce66a62d8 Eugene Zemtsov 2019-11-18  364   * It should always be called only on empty file.
318babce66a62d8 Eugene Zemtsov 2019-11-18  365   * incfs_super_block.s_first_md_offset is 0 for now, but will be updated
318babce66a62d8 Eugene Zemtsov 2019-11-18  366   * once first metadata record is added.
318babce66a62d8 Eugene Zemtsov 2019-11-18  367   */
318babce66a62d8 Eugene Zemtsov 2019-11-18  368  int incfs_write_fh_to_backing_file(struct backing_file_context *bfc,
318babce66a62d8 Eugene Zemtsov 2019-11-18  369  				   incfs_uuid_t *uuid, u64 file_size)
318babce66a62d8 Eugene Zemtsov 2019-11-18  370  {
318babce66a62d8 Eugene Zemtsov 2019-11-18  371  	struct incfs_file_header fh = {};
318babce66a62d8 Eugene Zemtsov 2019-11-18  372  	loff_t file_pos = 0;
318babce66a62d8 Eugene Zemtsov 2019-11-18  373  
318babce66a62d8 Eugene Zemtsov 2019-11-18  374  	if (!bfc)
318babce66a62d8 Eugene Zemtsov 2019-11-18  375  		return -EFAULT;
318babce66a62d8 Eugene Zemtsov 2019-11-18  376  
318babce66a62d8 Eugene Zemtsov 2019-11-18 @377  	fh.fh_magic = cpu_to_le64(INCFS_MAGIC_NUMBER);
318babce66a62d8 Eugene Zemtsov 2019-11-18  378  	fh.fh_version = cpu_to_le64(INCFS_FORMAT_CURRENT_VER);
318babce66a62d8 Eugene Zemtsov 2019-11-18  379  	fh.fh_header_size = cpu_to_le16(sizeof(fh));
318babce66a62d8 Eugene Zemtsov 2019-11-18  380  	fh.fh_first_md_offset = cpu_to_le64(0);
318babce66a62d8 Eugene Zemtsov 2019-11-18  381  	fh.fh_data_block_size = cpu_to_le16(INCFS_DATA_FILE_BLOCK_SIZE);
318babce66a62d8 Eugene Zemtsov 2019-11-18  382  
318babce66a62d8 Eugene Zemtsov 2019-11-18  383  	fh.fh_file_size = cpu_to_le64(file_size);
318babce66a62d8 Eugene Zemtsov 2019-11-18  384  	fh.fh_uuid = *uuid;
318babce66a62d8 Eugene Zemtsov 2019-11-18  385  
318babce66a62d8 Eugene Zemtsov 2019-11-18  386  	LOCK_REQUIRED(bfc->bc_mutex);
318babce66a62d8 Eugene Zemtsov 2019-11-18  387  
318babce66a62d8 Eugene Zemtsov 2019-11-18  388  	file_pos = incfs_get_end_offset(bfc->bc_file);
318babce66a62d8 Eugene Zemtsov 2019-11-18  389  	if (file_pos != 0)
318babce66a62d8 Eugene Zemtsov 2019-11-18  390  		return -EEXIST;
318babce66a62d8 Eugene Zemtsov 2019-11-18  391  
4f81903f2276fc9 Paul Lawrence  2020-04-07  392  	return write_to_bf(bfc, &fh, sizeof(fh), file_pos);
318babce66a62d8 Eugene Zemtsov 2019-11-18  393  }
318babce66a62d8 Eugene Zemtsov 2019-11-18  394  

:::::: The code at line 377 was first introduced by commit
:::::: 318babce66a62d8bc8d5434909a044b89ab3a53e ANDROID: Initial commit of Incremental FS

:::::: TO: Eugene Zemtsov <[email protected]>
:::::: CC: Paul Lawrence <[email protected]>

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp

                 reply	other threads:[~2022-08-04 11:17 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    [email protected] \
    [email protected] \
    [email protected] \
    [email protected] \
    [email protected] \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox