All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Jing Zhang <jingzhangos@google.com>, KVM <kvm@vger.kernel.org>,
	KVMARM <kvmarm@lists.cs.columbia.edu>,
	LinuxMIPS <linux-mips@vger.kernel.org>,
	KVMPPC <kvm-ppc@vger.kernel.org>,
	LinuxS390 <linux-s390@vger.kernel.org>,
	Linuxkselftest <linux-kselftest@vger.kernel.org>,
	Paolo Bonzini <pbonzini@redhat.com>,
	Marc Zyngier <maz@kernel.org>, James Morse <james.morse@arm.com>,
	Julien Thierry <julien.thierry.kdev@gmail.com>
Cc: kbuild-all@lists.01.org, clang-built-linux@googlegroups.com
Subject: Re: [PATCH v4 2/4] KVM: stats: Add fd-based API to read binary stats data
Date: Fri, 30 Apr 2021 09:26:13 +0800	[thread overview]
Message-ID: <202104300902.XgyYInj5-lkp@intel.com> (raw)
In-Reply-To: <20210429203740.1935629-3-jingzhangos@google.com>

[-- Attachment #1: Type: text/plain, Size: 13711 bytes --]

Hi Jing,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on 9f242010c3b46e63bc62f08fff42cef992d3801b]

url:    https://github.com/0day-ci/linux/commits/Jing-Zhang/KVM-statistics-data-fd-based-binary-interface/20210430-043830
base:   9f242010c3b46e63bc62f08fff42cef992d3801b
config: s390-randconfig-r024-20210429 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8f5a2a5836cc8e4c1def2bdeb022e7b496623439)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install s390 cross compiling tool for clang build
        # apt-get install binutils-s390x-linux-gnu
        # https://github.com/0day-ci/linux/commit/434cb14317623e9908098fc1c3925f2a6dcaa556
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Jing-Zhang/KVM-statistics-data-fd-based-binary-interface/20210430-043830
        git checkout 434cb14317623e9908098fc1c3925f2a6dcaa556
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=s390 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

   In file included from arch/s390/kvm/kvm-s390.c:23:
   In file included from include/linux/kvm_host.h:33:
   In file included from include/linux/kvm_para.h:5:
   In file included from include/uapi/linux/kvm_para.h:37:
   In file included from arch/s390/include/asm/kvm_para.h:25:
   In file included from arch/s390/include/asm/diag.h:12:
   In file included from include/linux/if_ether.h:19:
   In file included from include/linux/skbuff.h:31:
   In file included from include/linux/dma-mapping.h:10:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/s390/include/asm/io.h:80:
   include/asm-generic/io.h:464:31: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __raw_readb(PCI_IOBASE + addr);
                             ~~~~~~~~~~ ^
   include/asm-generic/io.h:477:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __le16_to_cpu((__le16 __force)__raw_readw(PCI_IOBASE + addr));
                                                           ~~~~~~~~~~ ^
   include/uapi/linux/byteorder/big_endian.h:36:59: note: expanded from macro '__le16_to_cpu'
   #define __le16_to_cpu(x) __swab16((__force __u16)(__le16)(x))
                                                             ^
   include/uapi/linux/swab.h:102:54: note: expanded from macro '__swab16'
   #define __swab16(x) (__u16)__builtin_bswap16((__u16)(x))
                                                        ^
   In file included from arch/s390/kvm/kvm-s390.c:23:
   In file included from include/linux/kvm_host.h:33:
   In file included from include/linux/kvm_para.h:5:
   In file included from include/uapi/linux/kvm_para.h:37:
   In file included from arch/s390/include/asm/kvm_para.h:25:
   In file included from arch/s390/include/asm/diag.h:12:
   In file included from include/linux/if_ether.h:19:
   In file included from include/linux/skbuff.h:31:
   In file included from include/linux/dma-mapping.h:10:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/s390/include/asm/io.h:80:
   include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
                                                           ~~~~~~~~~~ ^
   include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
   #define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
                                                             ^
   include/uapi/linux/swab.h:115:54: note: expanded from macro '__swab32'
   #define __swab32(x) (__u32)__builtin_bswap32((__u32)(x))
                                                        ^
   In file included from arch/s390/kvm/kvm-s390.c:23:
   In file included from include/linux/kvm_host.h:33:
   In file included from include/linux/kvm_para.h:5:
   In file included from include/uapi/linux/kvm_para.h:37:
   In file included from arch/s390/include/asm/kvm_para.h:25:
   In file included from arch/s390/include/asm/diag.h:12:
   In file included from include/linux/if_ether.h:19:
   In file included from include/linux/skbuff.h:31:
   In file included from include/linux/dma-mapping.h:10:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/s390/include/asm/io.h:80:
   include/asm-generic/io.h:501:33: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writeb(value, PCI_IOBASE + addr);
                               ~~~~~~~~~~ ^
   include/asm-generic/io.h:511:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writew((u16 __force)cpu_to_le16(value), PCI_IOBASE + addr);
                                                         ~~~~~~~~~~ ^
   include/asm-generic/io.h:521:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writel((u32 __force)cpu_to_le32(value), PCI_IOBASE + addr);
                                                         ~~~~~~~~~~ ^
   include/asm-generic/io.h:609:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           readsb(PCI_IOBASE + addr, buffer, count);
                  ~~~~~~~~~~ ^
   include/asm-generic/io.h:617:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           readsw(PCI_IOBASE + addr, buffer, count);
                  ~~~~~~~~~~ ^
   include/asm-generic/io.h:625:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           readsl(PCI_IOBASE + addr, buffer, count);
                  ~~~~~~~~~~ ^
   include/asm-generic/io.h:634:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           writesb(PCI_IOBASE + addr, buffer, count);
                   ~~~~~~~~~~ ^
   include/asm-generic/io.h:643:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           writesw(PCI_IOBASE + addr, buffer, count);
                   ~~~~~~~~~~ ^
   include/asm-generic/io.h:652:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           writesl(PCI_IOBASE + addr, buffer, count);
                   ~~~~~~~~~~ ^
>> arch/s390/kvm/kvm-s390.c:154:21: warning: initializer-string for char array is too long [-Wexcess-initializers]
           STATS_DESC_COUNTER("instruction_sigp_stop_store_status"),
                              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/kvm_host.h:1241:24: note: expanded from macro 'STATS_DESC_COUNTER'
           STATS_DESC_CUMULATIVE(name, KVM_STATS_UNIT_NONE,                       \
                                 ^~~~
   include/linux/kvm_host.h:1235:13: note: expanded from macro 'STATS_DESC_CUMULATIVE'
           STATS_DESC(name, KVM_STATS_TYPE_CUMULATIVE, unit, scale, exponent)
                      ^~~~
   include/linux/kvm_host.h:1232:39: note: expanded from macro 'STATS_DESC'
                   {type | unit | scale, exponent, 1}, name,                      \
                                                       ^~~~
   arch/s390/kvm/kvm-s390.c:156:21: warning: initializer-string for char array is too long [-Wexcess-initializers]
           STATS_DESC_COUNTER("instruction_sigp_store_adtl_status"),
                              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/kvm_host.h:1241:24: note: expanded from macro 'STATS_DESC_COUNTER'
           STATS_DESC_CUMULATIVE(name, KVM_STATS_UNIT_NONE,                       \
                                 ^~~~
   include/linux/kvm_host.h:1235:13: note: expanded from macro 'STATS_DESC_CUMULATIVE'
           STATS_DESC(name, KVM_STATS_TYPE_CUMULATIVE, unit, scale, exponent)
                      ^~~~
   include/linux/kvm_host.h:1232:39: note: expanded from macro 'STATS_DESC'
                   {type | unit | scale, exponent, 1}, name,                      \
                                                       ^~~~
   14 warnings generated.


vim +154 arch/s390/kvm/kvm-s390.c

    77	
    78	struct _kvm_stats_desc kvm_vcpu_stats_desc[] = {
    79		STATS_VCPU_COMMON,
    80		STATS_DESC_COUNTER("exit_userspace"),
    81		STATS_DESC_COUNTER("exit_null"),
    82		STATS_DESC_COUNTER("exit_external_request"),
    83		STATS_DESC_COUNTER("exit_io_request"),
    84		STATS_DESC_COUNTER("exit_external_interrupt"),
    85		STATS_DESC_COUNTER("exit_stop_request"),
    86		STATS_DESC_COUNTER("exit_validity"),
    87		STATS_DESC_COUNTER("exit_instruction"),
    88		STATS_DESC_COUNTER("exit_pei"),
    89		STATS_DESC_COUNTER("halt_no_poll_steal"),
    90		STATS_DESC_COUNTER("instruction_lctl"),
    91		STATS_DESC_COUNTER("instruction_lctlg"),
    92		STATS_DESC_COUNTER("instruction_stctl"),
    93		STATS_DESC_COUNTER("instruction_stctg"),
    94		STATS_DESC_COUNTER("exit_program_interruption"),
    95		STATS_DESC_COUNTER("exit_instr_and_program"),
    96		STATS_DESC_COUNTER("exit_operation_exception"),
    97		STATS_DESC_COUNTER("deliver_ckc"),
    98		STATS_DESC_COUNTER("deliver_cputm"),
    99		STATS_DESC_COUNTER("deliver_external_call"),
   100		STATS_DESC_COUNTER("deliver_emergency_signal"),
   101		STATS_DESC_COUNTER("deliver_service_signal"),
   102		STATS_DESC_COUNTER("deliver_virtio"),
   103		STATS_DESC_COUNTER("deliver_stop_signal"),
   104		STATS_DESC_COUNTER("deliver_prefix_signal"),
   105		STATS_DESC_COUNTER("deliver_restart_signal"),
   106		STATS_DESC_COUNTER("deliver_program"),
   107		STATS_DESC_COUNTER("deliver_io"),
   108		STATS_DESC_COUNTER("deliver_machine_check"),
   109		STATS_DESC_COUNTER("exit_wait_state"),
   110		STATS_DESC_COUNTER("inject_ckc"),
   111		STATS_DESC_COUNTER("inject_cputm"),
   112		STATS_DESC_COUNTER("inject_external_call"),
   113		STATS_DESC_COUNTER("inject_emergency_signal"),
   114		STATS_DESC_COUNTER("inject_mchk"),
   115		STATS_DESC_COUNTER("inject_pfault_init"),
   116		STATS_DESC_COUNTER("inject_program"),
   117		STATS_DESC_COUNTER("inject_restart"),
   118		STATS_DESC_COUNTER("inject_set_prefix"),
   119		STATS_DESC_COUNTER("inject_stop_signal"),
   120		STATS_DESC_COUNTER("instruction_epsw"),
   121		STATS_DESC_COUNTER("instruction_gs"),
   122		STATS_DESC_COUNTER("instruction_io_other"),
   123		STATS_DESC_COUNTER("instruction_lpsw"),
   124		STATS_DESC_COUNTER("instruction_lpswe"),
   125		STATS_DESC_COUNTER("instruction_pfmf"),
   126		STATS_DESC_COUNTER("instruction_ptff"),
   127		STATS_DESC_COUNTER("instruction_sck"),
   128		STATS_DESC_COUNTER("instruction_sckpf"),
   129		STATS_DESC_COUNTER("instruction_stidp"),
   130		STATS_DESC_COUNTER("instruction_spx"),
   131		STATS_DESC_COUNTER("instruction_stpx"),
   132		STATS_DESC_COUNTER("instruction_stap"),
   133		STATS_DESC_COUNTER("instruction_iske"),
   134		STATS_DESC_COUNTER("instruction_ri"),
   135		STATS_DESC_COUNTER("instruction_rrbe"),
   136		STATS_DESC_COUNTER("instruction_sske"),
   137		STATS_DESC_COUNTER("instruction_ipte_interlock"),
   138		STATS_DESC_COUNTER("instruction_stsi"),
   139		STATS_DESC_COUNTER("instruction_stfl"),
   140		STATS_DESC_COUNTER("instruction_tb"),
   141		STATS_DESC_COUNTER("instruction_tpi"),
   142		STATS_DESC_COUNTER("instruction_tprot"),
   143		STATS_DESC_COUNTER("instruction_tsch"),
   144		STATS_DESC_COUNTER("instruction_sie"),
   145		STATS_DESC_COUNTER("instruction_essa"),
   146		STATS_DESC_COUNTER("instruction_sthyi"),
   147		STATS_DESC_COUNTER("instruction_sigp_sense"),
   148		STATS_DESC_COUNTER("instruction_sigp_sense_running"),
   149		STATS_DESC_COUNTER("instruction_sigp_external_call"),
   150		STATS_DESC_COUNTER("instruction_sigp_emergency"),
   151		STATS_DESC_COUNTER("instruction_sigp_cond_emergency"),
   152		STATS_DESC_COUNTER("instruction_sigp_start"),
   153		STATS_DESC_COUNTER("instruction_sigp_stop"),
 > 154		STATS_DESC_COUNTER("instruction_sigp_stop_store_status"),
   155		STATS_DESC_COUNTER("instruction_sigp_store_status"),
   156		STATS_DESC_COUNTER("instruction_sigp_store_adtl_status"),
   157		STATS_DESC_COUNTER("instruction_sigp_arch"),
   158		STATS_DESC_COUNTER("instruction_sigp_prefix"),
   159		STATS_DESC_COUNTER("instruction_sigp_restart"),
   160		STATS_DESC_COUNTER("instruction_sigp_init_cpu_reset"),
   161		STATS_DESC_COUNTER("instruction_sigp_cpu_reset"),
   162		STATS_DESC_COUNTER("instruction_sigp_unknown"),
   163		STATS_DESC_COUNTER("diagnose_10"),
   164		STATS_DESC_COUNTER("diagnose_44"),
   165		STATS_DESC_COUNTER("diagnose_9c"),
   166		STATS_DESC_COUNTER("diagnose_9c_ignored"),
   167		STATS_DESC_COUNTER("diagnose_258"),
   168		STATS_DESC_COUNTER("diagnose_308"),
   169		STATS_DESC_COUNTER("diagnose_500"),
   170		STATS_DESC_COUNTER("diagnose_other"),
   171		STATS_DESC_COUNTER("pfault_sync"),
   172	};
   173	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 27158 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: Jing Zhang <jingzhangos@google.com>, KVM <kvm@vger.kernel.org>,
	KVMARM <kvmarm@lists.cs.columbia.edu>,
	LinuxMIPS <linux-mips@vger.kernel.org>,
	KVMPPC <kvm-ppc@vger.kernel.org>,
	LinuxS390 <linux-s390@vger.kernel.org>,
	Linuxkselftest <linux-kselftest@vger.kernel.org>,
	Paolo Bonzini <pbonzini@redhat.com>,
	Marc Zyngier <maz@kernel.org>, James Morse <james.morse@arm.com>,
	Julien Thierry <julien.thierry.kdev@gmail.com>
Cc: clang-built-linux@googlegroups.com, kbuild-all@lists.01.org
Subject: Re: [PATCH v4 2/4] KVM: stats: Add fd-based API to read binary stats data
Date: Fri, 30 Apr 2021 09:26:13 +0800	[thread overview]
Message-ID: <202104300902.XgyYInj5-lkp@intel.com> (raw)
In-Reply-To: <20210429203740.1935629-3-jingzhangos@google.com>

[-- Attachment #1: Type: text/plain, Size: 13711 bytes --]

Hi Jing,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on 9f242010c3b46e63bc62f08fff42cef992d3801b]

url:    https://github.com/0day-ci/linux/commits/Jing-Zhang/KVM-statistics-data-fd-based-binary-interface/20210430-043830
base:   9f242010c3b46e63bc62f08fff42cef992d3801b
config: s390-randconfig-r024-20210429 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8f5a2a5836cc8e4c1def2bdeb022e7b496623439)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install s390 cross compiling tool for clang build
        # apt-get install binutils-s390x-linux-gnu
        # https://github.com/0day-ci/linux/commit/434cb14317623e9908098fc1c3925f2a6dcaa556
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Jing-Zhang/KVM-statistics-data-fd-based-binary-interface/20210430-043830
        git checkout 434cb14317623e9908098fc1c3925f2a6dcaa556
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=s390 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

   In file included from arch/s390/kvm/kvm-s390.c:23:
   In file included from include/linux/kvm_host.h:33:
   In file included from include/linux/kvm_para.h:5:
   In file included from include/uapi/linux/kvm_para.h:37:
   In file included from arch/s390/include/asm/kvm_para.h:25:
   In file included from arch/s390/include/asm/diag.h:12:
   In file included from include/linux/if_ether.h:19:
   In file included from include/linux/skbuff.h:31:
   In file included from include/linux/dma-mapping.h:10:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/s390/include/asm/io.h:80:
   include/asm-generic/io.h:464:31: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __raw_readb(PCI_IOBASE + addr);
                             ~~~~~~~~~~ ^
   include/asm-generic/io.h:477:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __le16_to_cpu((__le16 __force)__raw_readw(PCI_IOBASE + addr));
                                                           ~~~~~~~~~~ ^
   include/uapi/linux/byteorder/big_endian.h:36:59: note: expanded from macro '__le16_to_cpu'
   #define __le16_to_cpu(x) __swab16((__force __u16)(__le16)(x))
                                                             ^
   include/uapi/linux/swab.h:102:54: note: expanded from macro '__swab16'
   #define __swab16(x) (__u16)__builtin_bswap16((__u16)(x))
                                                        ^
   In file included from arch/s390/kvm/kvm-s390.c:23:
   In file included from include/linux/kvm_host.h:33:
   In file included from include/linux/kvm_para.h:5:
   In file included from include/uapi/linux/kvm_para.h:37:
   In file included from arch/s390/include/asm/kvm_para.h:25:
   In file included from arch/s390/include/asm/diag.h:12:
   In file included from include/linux/if_ether.h:19:
   In file included from include/linux/skbuff.h:31:
   In file included from include/linux/dma-mapping.h:10:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/s390/include/asm/io.h:80:
   include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
                                                           ~~~~~~~~~~ ^
   include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
   #define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
                                                             ^
   include/uapi/linux/swab.h:115:54: note: expanded from macro '__swab32'
   #define __swab32(x) (__u32)__builtin_bswap32((__u32)(x))
                                                        ^
   In file included from arch/s390/kvm/kvm-s390.c:23:
   In file included from include/linux/kvm_host.h:33:
   In file included from include/linux/kvm_para.h:5:
   In file included from include/uapi/linux/kvm_para.h:37:
   In file included from arch/s390/include/asm/kvm_para.h:25:
   In file included from arch/s390/include/asm/diag.h:12:
   In file included from include/linux/if_ether.h:19:
   In file included from include/linux/skbuff.h:31:
   In file included from include/linux/dma-mapping.h:10:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/s390/include/asm/io.h:80:
   include/asm-generic/io.h:501:33: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writeb(value, PCI_IOBASE + addr);
                               ~~~~~~~~~~ ^
   include/asm-generic/io.h:511:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writew((u16 __force)cpu_to_le16(value), PCI_IOBASE + addr);
                                                         ~~~~~~~~~~ ^
   include/asm-generic/io.h:521:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writel((u32 __force)cpu_to_le32(value), PCI_IOBASE + addr);
                                                         ~~~~~~~~~~ ^
   include/asm-generic/io.h:609:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           readsb(PCI_IOBASE + addr, buffer, count);
                  ~~~~~~~~~~ ^
   include/asm-generic/io.h:617:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           readsw(PCI_IOBASE + addr, buffer, count);
                  ~~~~~~~~~~ ^
   include/asm-generic/io.h:625:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           readsl(PCI_IOBASE + addr, buffer, count);
                  ~~~~~~~~~~ ^
   include/asm-generic/io.h:634:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           writesb(PCI_IOBASE + addr, buffer, count);
                   ~~~~~~~~~~ ^
   include/asm-generic/io.h:643:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           writesw(PCI_IOBASE + addr, buffer, count);
                   ~~~~~~~~~~ ^
   include/asm-generic/io.h:652:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           writesl(PCI_IOBASE + addr, buffer, count);
                   ~~~~~~~~~~ ^
>> arch/s390/kvm/kvm-s390.c:154:21: warning: initializer-string for char array is too long [-Wexcess-initializers]
           STATS_DESC_COUNTER("instruction_sigp_stop_store_status"),
                              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/kvm_host.h:1241:24: note: expanded from macro 'STATS_DESC_COUNTER'
           STATS_DESC_CUMULATIVE(name, KVM_STATS_UNIT_NONE,                       \
                                 ^~~~
   include/linux/kvm_host.h:1235:13: note: expanded from macro 'STATS_DESC_CUMULATIVE'
           STATS_DESC(name, KVM_STATS_TYPE_CUMULATIVE, unit, scale, exponent)
                      ^~~~
   include/linux/kvm_host.h:1232:39: note: expanded from macro 'STATS_DESC'
                   {type | unit | scale, exponent, 1}, name,                      \
                                                       ^~~~
   arch/s390/kvm/kvm-s390.c:156:21: warning: initializer-string for char array is too long [-Wexcess-initializers]
           STATS_DESC_COUNTER("instruction_sigp_store_adtl_status"),
                              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/kvm_host.h:1241:24: note: expanded from macro 'STATS_DESC_COUNTER'
           STATS_DESC_CUMULATIVE(name, KVM_STATS_UNIT_NONE,                       \
                                 ^~~~
   include/linux/kvm_host.h:1235:13: note: expanded from macro 'STATS_DESC_CUMULATIVE'
           STATS_DESC(name, KVM_STATS_TYPE_CUMULATIVE, unit, scale, exponent)
                      ^~~~
   include/linux/kvm_host.h:1232:39: note: expanded from macro 'STATS_DESC'
                   {type | unit | scale, exponent, 1}, name,                      \
                                                       ^~~~
   14 warnings generated.


vim +154 arch/s390/kvm/kvm-s390.c

    77	
    78	struct _kvm_stats_desc kvm_vcpu_stats_desc[] = {
    79		STATS_VCPU_COMMON,
    80		STATS_DESC_COUNTER("exit_userspace"),
    81		STATS_DESC_COUNTER("exit_null"),
    82		STATS_DESC_COUNTER("exit_external_request"),
    83		STATS_DESC_COUNTER("exit_io_request"),
    84		STATS_DESC_COUNTER("exit_external_interrupt"),
    85		STATS_DESC_COUNTER("exit_stop_request"),
    86		STATS_DESC_COUNTER("exit_validity"),
    87		STATS_DESC_COUNTER("exit_instruction"),
    88		STATS_DESC_COUNTER("exit_pei"),
    89		STATS_DESC_COUNTER("halt_no_poll_steal"),
    90		STATS_DESC_COUNTER("instruction_lctl"),
    91		STATS_DESC_COUNTER("instruction_lctlg"),
    92		STATS_DESC_COUNTER("instruction_stctl"),
    93		STATS_DESC_COUNTER("instruction_stctg"),
    94		STATS_DESC_COUNTER("exit_program_interruption"),
    95		STATS_DESC_COUNTER("exit_instr_and_program"),
    96		STATS_DESC_COUNTER("exit_operation_exception"),
    97		STATS_DESC_COUNTER("deliver_ckc"),
    98		STATS_DESC_COUNTER("deliver_cputm"),
    99		STATS_DESC_COUNTER("deliver_external_call"),
   100		STATS_DESC_COUNTER("deliver_emergency_signal"),
   101		STATS_DESC_COUNTER("deliver_service_signal"),
   102		STATS_DESC_COUNTER("deliver_virtio"),
   103		STATS_DESC_COUNTER("deliver_stop_signal"),
   104		STATS_DESC_COUNTER("deliver_prefix_signal"),
   105		STATS_DESC_COUNTER("deliver_restart_signal"),
   106		STATS_DESC_COUNTER("deliver_program"),
   107		STATS_DESC_COUNTER("deliver_io"),
   108		STATS_DESC_COUNTER("deliver_machine_check"),
   109		STATS_DESC_COUNTER("exit_wait_state"),
   110		STATS_DESC_COUNTER("inject_ckc"),
   111		STATS_DESC_COUNTER("inject_cputm"),
   112		STATS_DESC_COUNTER("inject_external_call"),
   113		STATS_DESC_COUNTER("inject_emergency_signal"),
   114		STATS_DESC_COUNTER("inject_mchk"),
   115		STATS_DESC_COUNTER("inject_pfault_init"),
   116		STATS_DESC_COUNTER("inject_program"),
   117		STATS_DESC_COUNTER("inject_restart"),
   118		STATS_DESC_COUNTER("inject_set_prefix"),
   119		STATS_DESC_COUNTER("inject_stop_signal"),
   120		STATS_DESC_COUNTER("instruction_epsw"),
   121		STATS_DESC_COUNTER("instruction_gs"),
   122		STATS_DESC_COUNTER("instruction_io_other"),
   123		STATS_DESC_COUNTER("instruction_lpsw"),
   124		STATS_DESC_COUNTER("instruction_lpswe"),
   125		STATS_DESC_COUNTER("instruction_pfmf"),
   126		STATS_DESC_COUNTER("instruction_ptff"),
   127		STATS_DESC_COUNTER("instruction_sck"),
   128		STATS_DESC_COUNTER("instruction_sckpf"),
   129		STATS_DESC_COUNTER("instruction_stidp"),
   130		STATS_DESC_COUNTER("instruction_spx"),
   131		STATS_DESC_COUNTER("instruction_stpx"),
   132		STATS_DESC_COUNTER("instruction_stap"),
   133		STATS_DESC_COUNTER("instruction_iske"),
   134		STATS_DESC_COUNTER("instruction_ri"),
   135		STATS_DESC_COUNTER("instruction_rrbe"),
   136		STATS_DESC_COUNTER("instruction_sske"),
   137		STATS_DESC_COUNTER("instruction_ipte_interlock"),
   138		STATS_DESC_COUNTER("instruction_stsi"),
   139		STATS_DESC_COUNTER("instruction_stfl"),
   140		STATS_DESC_COUNTER("instruction_tb"),
   141		STATS_DESC_COUNTER("instruction_tpi"),
   142		STATS_DESC_COUNTER("instruction_tprot"),
   143		STATS_DESC_COUNTER("instruction_tsch"),
   144		STATS_DESC_COUNTER("instruction_sie"),
   145		STATS_DESC_COUNTER("instruction_essa"),
   146		STATS_DESC_COUNTER("instruction_sthyi"),
   147		STATS_DESC_COUNTER("instruction_sigp_sense"),
   148		STATS_DESC_COUNTER("instruction_sigp_sense_running"),
   149		STATS_DESC_COUNTER("instruction_sigp_external_call"),
   150		STATS_DESC_COUNTER("instruction_sigp_emergency"),
   151		STATS_DESC_COUNTER("instruction_sigp_cond_emergency"),
   152		STATS_DESC_COUNTER("instruction_sigp_start"),
   153		STATS_DESC_COUNTER("instruction_sigp_stop"),
 > 154		STATS_DESC_COUNTER("instruction_sigp_stop_store_status"),
   155		STATS_DESC_COUNTER("instruction_sigp_store_status"),
   156		STATS_DESC_COUNTER("instruction_sigp_store_adtl_status"),
   157		STATS_DESC_COUNTER("instruction_sigp_arch"),
   158		STATS_DESC_COUNTER("instruction_sigp_prefix"),
   159		STATS_DESC_COUNTER("instruction_sigp_restart"),
   160		STATS_DESC_COUNTER("instruction_sigp_init_cpu_reset"),
   161		STATS_DESC_COUNTER("instruction_sigp_cpu_reset"),
   162		STATS_DESC_COUNTER("instruction_sigp_unknown"),
   163		STATS_DESC_COUNTER("diagnose_10"),
   164		STATS_DESC_COUNTER("diagnose_44"),
   165		STATS_DESC_COUNTER("diagnose_9c"),
   166		STATS_DESC_COUNTER("diagnose_9c_ignored"),
   167		STATS_DESC_COUNTER("diagnose_258"),
   168		STATS_DESC_COUNTER("diagnose_308"),
   169		STATS_DESC_COUNTER("diagnose_500"),
   170		STATS_DESC_COUNTER("diagnose_other"),
   171		STATS_DESC_COUNTER("pfault_sync"),
   172	};
   173	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 27158 bytes --]

[-- Attachment #3: Type: text/plain, Size: 151 bytes --]

_______________________________________________
kvmarm mailing list
kvmarm@lists.cs.columbia.edu
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm

WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH v4 2/4] KVM: stats: Add fd-based API to read binary stats data
Date: Fri, 30 Apr 2021 09:26:13 +0800	[thread overview]
Message-ID: <202104300902.XgyYInj5-lkp@intel.com> (raw)
In-Reply-To: <20210429203740.1935629-3-jingzhangos@google.com>

[-- Attachment #1: Type: text/plain, Size: 13951 bytes --]

Hi Jing,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on 9f242010c3b46e63bc62f08fff42cef992d3801b]

url:    https://github.com/0day-ci/linux/commits/Jing-Zhang/KVM-statistics-data-fd-based-binary-interface/20210430-043830
base:   9f242010c3b46e63bc62f08fff42cef992d3801b
config: s390-randconfig-r024-20210429 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8f5a2a5836cc8e4c1def2bdeb022e7b496623439)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install s390 cross compiling tool for clang build
        # apt-get install binutils-s390x-linux-gnu
        # https://github.com/0day-ci/linux/commit/434cb14317623e9908098fc1c3925f2a6dcaa556
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Jing-Zhang/KVM-statistics-data-fd-based-binary-interface/20210430-043830
        git checkout 434cb14317623e9908098fc1c3925f2a6dcaa556
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=s390 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

   In file included from arch/s390/kvm/kvm-s390.c:23:
   In file included from include/linux/kvm_host.h:33:
   In file included from include/linux/kvm_para.h:5:
   In file included from include/uapi/linux/kvm_para.h:37:
   In file included from arch/s390/include/asm/kvm_para.h:25:
   In file included from arch/s390/include/asm/diag.h:12:
   In file included from include/linux/if_ether.h:19:
   In file included from include/linux/skbuff.h:31:
   In file included from include/linux/dma-mapping.h:10:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/s390/include/asm/io.h:80:
   include/asm-generic/io.h:464:31: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __raw_readb(PCI_IOBASE + addr);
                             ~~~~~~~~~~ ^
   include/asm-generic/io.h:477:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __le16_to_cpu((__le16 __force)__raw_readw(PCI_IOBASE + addr));
                                                           ~~~~~~~~~~ ^
   include/uapi/linux/byteorder/big_endian.h:36:59: note: expanded from macro '__le16_to_cpu'
   #define __le16_to_cpu(x) __swab16((__force __u16)(__le16)(x))
                                                             ^
   include/uapi/linux/swab.h:102:54: note: expanded from macro '__swab16'
   #define __swab16(x) (__u16)__builtin_bswap16((__u16)(x))
                                                        ^
   In file included from arch/s390/kvm/kvm-s390.c:23:
   In file included from include/linux/kvm_host.h:33:
   In file included from include/linux/kvm_para.h:5:
   In file included from include/uapi/linux/kvm_para.h:37:
   In file included from arch/s390/include/asm/kvm_para.h:25:
   In file included from arch/s390/include/asm/diag.h:12:
   In file included from include/linux/if_ether.h:19:
   In file included from include/linux/skbuff.h:31:
   In file included from include/linux/dma-mapping.h:10:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/s390/include/asm/io.h:80:
   include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
                                                           ~~~~~~~~~~ ^
   include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
   #define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
                                                             ^
   include/uapi/linux/swab.h:115:54: note: expanded from macro '__swab32'
   #define __swab32(x) (__u32)__builtin_bswap32((__u32)(x))
                                                        ^
   In file included from arch/s390/kvm/kvm-s390.c:23:
   In file included from include/linux/kvm_host.h:33:
   In file included from include/linux/kvm_para.h:5:
   In file included from include/uapi/linux/kvm_para.h:37:
   In file included from arch/s390/include/asm/kvm_para.h:25:
   In file included from arch/s390/include/asm/diag.h:12:
   In file included from include/linux/if_ether.h:19:
   In file included from include/linux/skbuff.h:31:
   In file included from include/linux/dma-mapping.h:10:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/s390/include/asm/io.h:80:
   include/asm-generic/io.h:501:33: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writeb(value, PCI_IOBASE + addr);
                               ~~~~~~~~~~ ^
   include/asm-generic/io.h:511:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writew((u16 __force)cpu_to_le16(value), PCI_IOBASE + addr);
                                                         ~~~~~~~~~~ ^
   include/asm-generic/io.h:521:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writel((u32 __force)cpu_to_le32(value), PCI_IOBASE + addr);
                                                         ~~~~~~~~~~ ^
   include/asm-generic/io.h:609:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           readsb(PCI_IOBASE + addr, buffer, count);
                  ~~~~~~~~~~ ^
   include/asm-generic/io.h:617:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           readsw(PCI_IOBASE + addr, buffer, count);
                  ~~~~~~~~~~ ^
   include/asm-generic/io.h:625:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           readsl(PCI_IOBASE + addr, buffer, count);
                  ~~~~~~~~~~ ^
   include/asm-generic/io.h:634:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           writesb(PCI_IOBASE + addr, buffer, count);
                   ~~~~~~~~~~ ^
   include/asm-generic/io.h:643:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           writesw(PCI_IOBASE + addr, buffer, count);
                   ~~~~~~~~~~ ^
   include/asm-generic/io.h:652:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           writesl(PCI_IOBASE + addr, buffer, count);
                   ~~~~~~~~~~ ^
>> arch/s390/kvm/kvm-s390.c:154:21: warning: initializer-string for char array is too long [-Wexcess-initializers]
           STATS_DESC_COUNTER("instruction_sigp_stop_store_status"),
                              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/kvm_host.h:1241:24: note: expanded from macro 'STATS_DESC_COUNTER'
           STATS_DESC_CUMULATIVE(name, KVM_STATS_UNIT_NONE,                       \
                                 ^~~~
   include/linux/kvm_host.h:1235:13: note: expanded from macro 'STATS_DESC_CUMULATIVE'
           STATS_DESC(name, KVM_STATS_TYPE_CUMULATIVE, unit, scale, exponent)
                      ^~~~
   include/linux/kvm_host.h:1232:39: note: expanded from macro 'STATS_DESC'
                   {type | unit | scale, exponent, 1}, name,                      \
                                                       ^~~~
   arch/s390/kvm/kvm-s390.c:156:21: warning: initializer-string for char array is too long [-Wexcess-initializers]
           STATS_DESC_COUNTER("instruction_sigp_store_adtl_status"),
                              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/kvm_host.h:1241:24: note: expanded from macro 'STATS_DESC_COUNTER'
           STATS_DESC_CUMULATIVE(name, KVM_STATS_UNIT_NONE,                       \
                                 ^~~~
   include/linux/kvm_host.h:1235:13: note: expanded from macro 'STATS_DESC_CUMULATIVE'
           STATS_DESC(name, KVM_STATS_TYPE_CUMULATIVE, unit, scale, exponent)
                      ^~~~
   include/linux/kvm_host.h:1232:39: note: expanded from macro 'STATS_DESC'
                   {type | unit | scale, exponent, 1}, name,                      \
                                                       ^~~~
   14 warnings generated.


vim +154 arch/s390/kvm/kvm-s390.c

    77	
    78	struct _kvm_stats_desc kvm_vcpu_stats_desc[] = {
    79		STATS_VCPU_COMMON,
    80		STATS_DESC_COUNTER("exit_userspace"),
    81		STATS_DESC_COUNTER("exit_null"),
    82		STATS_DESC_COUNTER("exit_external_request"),
    83		STATS_DESC_COUNTER("exit_io_request"),
    84		STATS_DESC_COUNTER("exit_external_interrupt"),
    85		STATS_DESC_COUNTER("exit_stop_request"),
    86		STATS_DESC_COUNTER("exit_validity"),
    87		STATS_DESC_COUNTER("exit_instruction"),
    88		STATS_DESC_COUNTER("exit_pei"),
    89		STATS_DESC_COUNTER("halt_no_poll_steal"),
    90		STATS_DESC_COUNTER("instruction_lctl"),
    91		STATS_DESC_COUNTER("instruction_lctlg"),
    92		STATS_DESC_COUNTER("instruction_stctl"),
    93		STATS_DESC_COUNTER("instruction_stctg"),
    94		STATS_DESC_COUNTER("exit_program_interruption"),
    95		STATS_DESC_COUNTER("exit_instr_and_program"),
    96		STATS_DESC_COUNTER("exit_operation_exception"),
    97		STATS_DESC_COUNTER("deliver_ckc"),
    98		STATS_DESC_COUNTER("deliver_cputm"),
    99		STATS_DESC_COUNTER("deliver_external_call"),
   100		STATS_DESC_COUNTER("deliver_emergency_signal"),
   101		STATS_DESC_COUNTER("deliver_service_signal"),
   102		STATS_DESC_COUNTER("deliver_virtio"),
   103		STATS_DESC_COUNTER("deliver_stop_signal"),
   104		STATS_DESC_COUNTER("deliver_prefix_signal"),
   105		STATS_DESC_COUNTER("deliver_restart_signal"),
   106		STATS_DESC_COUNTER("deliver_program"),
   107		STATS_DESC_COUNTER("deliver_io"),
   108		STATS_DESC_COUNTER("deliver_machine_check"),
   109		STATS_DESC_COUNTER("exit_wait_state"),
   110		STATS_DESC_COUNTER("inject_ckc"),
   111		STATS_DESC_COUNTER("inject_cputm"),
   112		STATS_DESC_COUNTER("inject_external_call"),
   113		STATS_DESC_COUNTER("inject_emergency_signal"),
   114		STATS_DESC_COUNTER("inject_mchk"),
   115		STATS_DESC_COUNTER("inject_pfault_init"),
   116		STATS_DESC_COUNTER("inject_program"),
   117		STATS_DESC_COUNTER("inject_restart"),
   118		STATS_DESC_COUNTER("inject_set_prefix"),
   119		STATS_DESC_COUNTER("inject_stop_signal"),
   120		STATS_DESC_COUNTER("instruction_epsw"),
   121		STATS_DESC_COUNTER("instruction_gs"),
   122		STATS_DESC_COUNTER("instruction_io_other"),
   123		STATS_DESC_COUNTER("instruction_lpsw"),
   124		STATS_DESC_COUNTER("instruction_lpswe"),
   125		STATS_DESC_COUNTER("instruction_pfmf"),
   126		STATS_DESC_COUNTER("instruction_ptff"),
   127		STATS_DESC_COUNTER("instruction_sck"),
   128		STATS_DESC_COUNTER("instruction_sckpf"),
   129		STATS_DESC_COUNTER("instruction_stidp"),
   130		STATS_DESC_COUNTER("instruction_spx"),
   131		STATS_DESC_COUNTER("instruction_stpx"),
   132		STATS_DESC_COUNTER("instruction_stap"),
   133		STATS_DESC_COUNTER("instruction_iske"),
   134		STATS_DESC_COUNTER("instruction_ri"),
   135		STATS_DESC_COUNTER("instruction_rrbe"),
   136		STATS_DESC_COUNTER("instruction_sske"),
   137		STATS_DESC_COUNTER("instruction_ipte_interlock"),
   138		STATS_DESC_COUNTER("instruction_stsi"),
   139		STATS_DESC_COUNTER("instruction_stfl"),
   140		STATS_DESC_COUNTER("instruction_tb"),
   141		STATS_DESC_COUNTER("instruction_tpi"),
   142		STATS_DESC_COUNTER("instruction_tprot"),
   143		STATS_DESC_COUNTER("instruction_tsch"),
   144		STATS_DESC_COUNTER("instruction_sie"),
   145		STATS_DESC_COUNTER("instruction_essa"),
   146		STATS_DESC_COUNTER("instruction_sthyi"),
   147		STATS_DESC_COUNTER("instruction_sigp_sense"),
   148		STATS_DESC_COUNTER("instruction_sigp_sense_running"),
   149		STATS_DESC_COUNTER("instruction_sigp_external_call"),
   150		STATS_DESC_COUNTER("instruction_sigp_emergency"),
   151		STATS_DESC_COUNTER("instruction_sigp_cond_emergency"),
   152		STATS_DESC_COUNTER("instruction_sigp_start"),
   153		STATS_DESC_COUNTER("instruction_sigp_stop"),
 > 154		STATS_DESC_COUNTER("instruction_sigp_stop_store_status"),
   155		STATS_DESC_COUNTER("instruction_sigp_store_status"),
   156		STATS_DESC_COUNTER("instruction_sigp_store_adtl_status"),
   157		STATS_DESC_COUNTER("instruction_sigp_arch"),
   158		STATS_DESC_COUNTER("instruction_sigp_prefix"),
   159		STATS_DESC_COUNTER("instruction_sigp_restart"),
   160		STATS_DESC_COUNTER("instruction_sigp_init_cpu_reset"),
   161		STATS_DESC_COUNTER("instruction_sigp_cpu_reset"),
   162		STATS_DESC_COUNTER("instruction_sigp_unknown"),
   163		STATS_DESC_COUNTER("diagnose_10"),
   164		STATS_DESC_COUNTER("diagnose_44"),
   165		STATS_DESC_COUNTER("diagnose_9c"),
   166		STATS_DESC_COUNTER("diagnose_9c_ignored"),
   167		STATS_DESC_COUNTER("diagnose_258"),
   168		STATS_DESC_COUNTER("diagnose_308"),
   169		STATS_DESC_COUNTER("diagnose_500"),
   170		STATS_DESC_COUNTER("diagnose_other"),
   171		STATS_DESC_COUNTER("pfault_sync"),
   172	};
   173	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 27158 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: Jing Zhang <jingzhangos@google.com>, KVM <kvm@vger.kernel.org>,
	KVMARM <kvmarm@lists.cs.columbia.edu>,
	LinuxMIPS <linux-mips@vger.kernel.org>,
	KVMPPC <kvm-ppc@vger.kernel.org>,
	LinuxS390 <linux-s390@vger.kernel.org>,
	Linuxkselftest <linux-kselftest@vger.kernel.org>,
	Paolo Bonzini <pbonzini@redhat.com>,
	Marc Zyngier <maz@kernel.org>, James Morse <james.morse@arm.com>,
	Julien Thierry <julien.thierry.kdev@gmail.com>
Cc: kbuild-all@lists.01.org, clang-built-linux@googlegroups.com
Subject: Re: [PATCH v4 2/4] KVM: stats: Add fd-based API to read binary stats data
Date: Fri, 30 Apr 2021 01:26:13 +0000	[thread overview]
Message-ID: <202104300902.XgyYInj5-lkp@intel.com> (raw)
In-Reply-To: <20210429203740.1935629-3-jingzhangos@google.com>

[-- Attachment #1: Type: text/plain, Size: 13711 bytes --]

Hi Jing,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on 9f242010c3b46e63bc62f08fff42cef992d3801b]

url:    https://github.com/0day-ci/linux/commits/Jing-Zhang/KVM-statistics-data-fd-based-binary-interface/20210430-043830
base:   9f242010c3b46e63bc62f08fff42cef992d3801b
config: s390-randconfig-r024-20210429 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8f5a2a5836cc8e4c1def2bdeb022e7b496623439)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install s390 cross compiling tool for clang build
        # apt-get install binutils-s390x-linux-gnu
        # https://github.com/0day-ci/linux/commit/434cb14317623e9908098fc1c3925f2a6dcaa556
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Jing-Zhang/KVM-statistics-data-fd-based-binary-interface/20210430-043830
        git checkout 434cb14317623e9908098fc1c3925f2a6dcaa556
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=s390 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

   In file included from arch/s390/kvm/kvm-s390.c:23:
   In file included from include/linux/kvm_host.h:33:
   In file included from include/linux/kvm_para.h:5:
   In file included from include/uapi/linux/kvm_para.h:37:
   In file included from arch/s390/include/asm/kvm_para.h:25:
   In file included from arch/s390/include/asm/diag.h:12:
   In file included from include/linux/if_ether.h:19:
   In file included from include/linux/skbuff.h:31:
   In file included from include/linux/dma-mapping.h:10:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/s390/include/asm/io.h:80:
   include/asm-generic/io.h:464:31: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __raw_readb(PCI_IOBASE + addr);
                             ~~~~~~~~~~ ^
   include/asm-generic/io.h:477:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __le16_to_cpu((__le16 __force)__raw_readw(PCI_IOBASE + addr));
                                                           ~~~~~~~~~~ ^
   include/uapi/linux/byteorder/big_endian.h:36:59: note: expanded from macro '__le16_to_cpu'
   #define __le16_to_cpu(x) __swab16((__force __u16)(__le16)(x))
                                                             ^
   include/uapi/linux/swab.h:102:54: note: expanded from macro '__swab16'
   #define __swab16(x) (__u16)__builtin_bswap16((__u16)(x))
                                                        ^
   In file included from arch/s390/kvm/kvm-s390.c:23:
   In file included from include/linux/kvm_host.h:33:
   In file included from include/linux/kvm_para.h:5:
   In file included from include/uapi/linux/kvm_para.h:37:
   In file included from arch/s390/include/asm/kvm_para.h:25:
   In file included from arch/s390/include/asm/diag.h:12:
   In file included from include/linux/if_ether.h:19:
   In file included from include/linux/skbuff.h:31:
   In file included from include/linux/dma-mapping.h:10:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/s390/include/asm/io.h:80:
   include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
                                                           ~~~~~~~~~~ ^
   include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
   #define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
                                                             ^
   include/uapi/linux/swab.h:115:54: note: expanded from macro '__swab32'
   #define __swab32(x) (__u32)__builtin_bswap32((__u32)(x))
                                                        ^
   In file included from arch/s390/kvm/kvm-s390.c:23:
   In file included from include/linux/kvm_host.h:33:
   In file included from include/linux/kvm_para.h:5:
   In file included from include/uapi/linux/kvm_para.h:37:
   In file included from arch/s390/include/asm/kvm_para.h:25:
   In file included from arch/s390/include/asm/diag.h:12:
   In file included from include/linux/if_ether.h:19:
   In file included from include/linux/skbuff.h:31:
   In file included from include/linux/dma-mapping.h:10:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/s390/include/asm/io.h:80:
   include/asm-generic/io.h:501:33: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writeb(value, PCI_IOBASE + addr);
                               ~~~~~~~~~~ ^
   include/asm-generic/io.h:511:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writew((u16 __force)cpu_to_le16(value), PCI_IOBASE + addr);
                                                         ~~~~~~~~~~ ^
   include/asm-generic/io.h:521:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writel((u32 __force)cpu_to_le32(value), PCI_IOBASE + addr);
                                                         ~~~~~~~~~~ ^
   include/asm-generic/io.h:609:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           readsb(PCI_IOBASE + addr, buffer, count);
                  ~~~~~~~~~~ ^
   include/asm-generic/io.h:617:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           readsw(PCI_IOBASE + addr, buffer, count);
                  ~~~~~~~~~~ ^
   include/asm-generic/io.h:625:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           readsl(PCI_IOBASE + addr, buffer, count);
                  ~~~~~~~~~~ ^
   include/asm-generic/io.h:634:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           writesb(PCI_IOBASE + addr, buffer, count);
                   ~~~~~~~~~~ ^
   include/asm-generic/io.h:643:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           writesw(PCI_IOBASE + addr, buffer, count);
                   ~~~~~~~~~~ ^
   include/asm-generic/io.h:652:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           writesl(PCI_IOBASE + addr, buffer, count);
                   ~~~~~~~~~~ ^
>> arch/s390/kvm/kvm-s390.c:154:21: warning: initializer-string for char array is too long [-Wexcess-initializers]
           STATS_DESC_COUNTER("instruction_sigp_stop_store_status"),
                              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/kvm_host.h:1241:24: note: expanded from macro 'STATS_DESC_COUNTER'
           STATS_DESC_CUMULATIVE(name, KVM_STATS_UNIT_NONE,                       \
                                 ^~~~
   include/linux/kvm_host.h:1235:13: note: expanded from macro 'STATS_DESC_CUMULATIVE'
           STATS_DESC(name, KVM_STATS_TYPE_CUMULATIVE, unit, scale, exponent)
                      ^~~~
   include/linux/kvm_host.h:1232:39: note: expanded from macro 'STATS_DESC'
                   {type | unit | scale, exponent, 1}, name,                      \
                                                       ^~~~
   arch/s390/kvm/kvm-s390.c:156:21: warning: initializer-string for char array is too long [-Wexcess-initializers]
           STATS_DESC_COUNTER("instruction_sigp_store_adtl_status"),
                              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/kvm_host.h:1241:24: note: expanded from macro 'STATS_DESC_COUNTER'
           STATS_DESC_CUMULATIVE(name, KVM_STATS_UNIT_NONE,                       \
                                 ^~~~
   include/linux/kvm_host.h:1235:13: note: expanded from macro 'STATS_DESC_CUMULATIVE'
           STATS_DESC(name, KVM_STATS_TYPE_CUMULATIVE, unit, scale, exponent)
                      ^~~~
   include/linux/kvm_host.h:1232:39: note: expanded from macro 'STATS_DESC'
                   {type | unit | scale, exponent, 1}, name,                      \
                                                       ^~~~
   14 warnings generated.


vim +154 arch/s390/kvm/kvm-s390.c

    77	
    78	struct _kvm_stats_desc kvm_vcpu_stats_desc[] = {
    79		STATS_VCPU_COMMON,
    80		STATS_DESC_COUNTER("exit_userspace"),
    81		STATS_DESC_COUNTER("exit_null"),
    82		STATS_DESC_COUNTER("exit_external_request"),
    83		STATS_DESC_COUNTER("exit_io_request"),
    84		STATS_DESC_COUNTER("exit_external_interrupt"),
    85		STATS_DESC_COUNTER("exit_stop_request"),
    86		STATS_DESC_COUNTER("exit_validity"),
    87		STATS_DESC_COUNTER("exit_instruction"),
    88		STATS_DESC_COUNTER("exit_pei"),
    89		STATS_DESC_COUNTER("halt_no_poll_steal"),
    90		STATS_DESC_COUNTER("instruction_lctl"),
    91		STATS_DESC_COUNTER("instruction_lctlg"),
    92		STATS_DESC_COUNTER("instruction_stctl"),
    93		STATS_DESC_COUNTER("instruction_stctg"),
    94		STATS_DESC_COUNTER("exit_program_interruption"),
    95		STATS_DESC_COUNTER("exit_instr_and_program"),
    96		STATS_DESC_COUNTER("exit_operation_exception"),
    97		STATS_DESC_COUNTER("deliver_ckc"),
    98		STATS_DESC_COUNTER("deliver_cputm"),
    99		STATS_DESC_COUNTER("deliver_external_call"),
   100		STATS_DESC_COUNTER("deliver_emergency_signal"),
   101		STATS_DESC_COUNTER("deliver_service_signal"),
   102		STATS_DESC_COUNTER("deliver_virtio"),
   103		STATS_DESC_COUNTER("deliver_stop_signal"),
   104		STATS_DESC_COUNTER("deliver_prefix_signal"),
   105		STATS_DESC_COUNTER("deliver_restart_signal"),
   106		STATS_DESC_COUNTER("deliver_program"),
   107		STATS_DESC_COUNTER("deliver_io"),
   108		STATS_DESC_COUNTER("deliver_machine_check"),
   109		STATS_DESC_COUNTER("exit_wait_state"),
   110		STATS_DESC_COUNTER("inject_ckc"),
   111		STATS_DESC_COUNTER("inject_cputm"),
   112		STATS_DESC_COUNTER("inject_external_call"),
   113		STATS_DESC_COUNTER("inject_emergency_signal"),
   114		STATS_DESC_COUNTER("inject_mchk"),
   115		STATS_DESC_COUNTER("inject_pfault_init"),
   116		STATS_DESC_COUNTER("inject_program"),
   117		STATS_DESC_COUNTER("inject_restart"),
   118		STATS_DESC_COUNTER("inject_set_prefix"),
   119		STATS_DESC_COUNTER("inject_stop_signal"),
   120		STATS_DESC_COUNTER("instruction_epsw"),
   121		STATS_DESC_COUNTER("instruction_gs"),
   122		STATS_DESC_COUNTER("instruction_io_other"),
   123		STATS_DESC_COUNTER("instruction_lpsw"),
   124		STATS_DESC_COUNTER("instruction_lpswe"),
   125		STATS_DESC_COUNTER("instruction_pfmf"),
   126		STATS_DESC_COUNTER("instruction_ptff"),
   127		STATS_DESC_COUNTER("instruction_sck"),
   128		STATS_DESC_COUNTER("instruction_sckpf"),
   129		STATS_DESC_COUNTER("instruction_stidp"),
   130		STATS_DESC_COUNTER("instruction_spx"),
   131		STATS_DESC_COUNTER("instruction_stpx"),
   132		STATS_DESC_COUNTER("instruction_stap"),
   133		STATS_DESC_COUNTER("instruction_iske"),
   134		STATS_DESC_COUNTER("instruction_ri"),
   135		STATS_DESC_COUNTER("instruction_rrbe"),
   136		STATS_DESC_COUNTER("instruction_sske"),
   137		STATS_DESC_COUNTER("instruction_ipte_interlock"),
   138		STATS_DESC_COUNTER("instruction_stsi"),
   139		STATS_DESC_COUNTER("instruction_stfl"),
   140		STATS_DESC_COUNTER("instruction_tb"),
   141		STATS_DESC_COUNTER("instruction_tpi"),
   142		STATS_DESC_COUNTER("instruction_tprot"),
   143		STATS_DESC_COUNTER("instruction_tsch"),
   144		STATS_DESC_COUNTER("instruction_sie"),
   145		STATS_DESC_COUNTER("instruction_essa"),
   146		STATS_DESC_COUNTER("instruction_sthyi"),
   147		STATS_DESC_COUNTER("instruction_sigp_sense"),
   148		STATS_DESC_COUNTER("instruction_sigp_sense_running"),
   149		STATS_DESC_COUNTER("instruction_sigp_external_call"),
   150		STATS_DESC_COUNTER("instruction_sigp_emergency"),
   151		STATS_DESC_COUNTER("instruction_sigp_cond_emergency"),
   152		STATS_DESC_COUNTER("instruction_sigp_start"),
   153		STATS_DESC_COUNTER("instruction_sigp_stop"),
 > 154		STATS_DESC_COUNTER("instruction_sigp_stop_store_status"),
   155		STATS_DESC_COUNTER("instruction_sigp_store_status"),
   156		STATS_DESC_COUNTER("instruction_sigp_store_adtl_status"),
   157		STATS_DESC_COUNTER("instruction_sigp_arch"),
   158		STATS_DESC_COUNTER("instruction_sigp_prefix"),
   159		STATS_DESC_COUNTER("instruction_sigp_restart"),
   160		STATS_DESC_COUNTER("instruction_sigp_init_cpu_reset"),
   161		STATS_DESC_COUNTER("instruction_sigp_cpu_reset"),
   162		STATS_DESC_COUNTER("instruction_sigp_unknown"),
   163		STATS_DESC_COUNTER("diagnose_10"),
   164		STATS_DESC_COUNTER("diagnose_44"),
   165		STATS_DESC_COUNTER("diagnose_9c"),
   166		STATS_DESC_COUNTER("diagnose_9c_ignored"),
   167		STATS_DESC_COUNTER("diagnose_258"),
   168		STATS_DESC_COUNTER("diagnose_308"),
   169		STATS_DESC_COUNTER("diagnose_500"),
   170		STATS_DESC_COUNTER("diagnose_other"),
   171		STATS_DESC_COUNTER("pfault_sync"),
   172	};
   173	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 27158 bytes --]

  reply	other threads:[~2021-04-30  1:27 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-29 20:37 [PATCH v4 0/4] KVM statistics data fd-based binary interface Jing Zhang
2021-04-29 20:37 ` Jing Zhang
2021-04-29 20:37 ` [PATCH v4 1/4] KVM: stats: Separate common stats from architecture specific ones Jing Zhang
2021-04-29 20:37   ` Jing Zhang
2021-04-29 20:37   ` Jing Zhang
2021-04-30 12:07   ` Marc Zyngier
2021-04-30 12:07     ` Marc Zyngier
2021-04-30 12:07     ` Marc Zyngier
2021-04-30 15:34     ` Jing Zhang
2021-04-30 15:34       ` Jing Zhang
2021-04-30 15:34       ` Jing Zhang
2021-04-29 20:37 ` [PATCH v4 2/4] KVM: stats: Add fd-based API to read binary stats data Jing Zhang
2021-04-29 20:37   ` Jing Zhang
2021-04-29 20:37   ` Jing Zhang
2021-04-30  1:26   ` kernel test robot [this message]
2021-04-30  1:26     ` kernel test robot
2021-04-30  1:26     ` kernel test robot
2021-04-30  1:26     ` kernel test robot
2021-04-30  2:26   ` Jing Zhang
2021-04-30  2:26     ` Jing Zhang
2021-04-30  2:26     ` Jing Zhang
2021-04-29 20:37 ` [PATCH v4 3/4] KVM: stats: Add documentation for statistics data binary interface Jing Zhang
2021-04-29 20:37   ` Jing Zhang
2021-04-29 20:37   ` Jing Zhang
2021-04-29 20:37 ` [PATCH v4 4/4] KVM: selftests: Add selftest for KVM " Jing Zhang
2021-04-29 20:37   ` Jing Zhang
2021-04-29 20:37   ` Jing Zhang
2021-05-10 18:57 ` [PATCH v4 0/4] KVM statistics data fd-based " Jing Zhang
2021-05-10 18:57   ` Jing Zhang
2021-05-10 18:57   ` Jing Zhang
2021-05-11  8:38   ` Paolo Bonzini
2021-05-11  8:38     ` Paolo Bonzini
2021-05-11  8:38     ` Paolo Bonzini

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 \
    --in-reply-to=202104300902.XgyYInj5-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=clang-built-linux@googlegroups.com \
    --cc=james.morse@arm.com \
    --cc=jingzhangos@google.com \
    --cc=julien.thierry.kdev@gmail.com \
    --cc=kbuild-all@lists.01.org \
    --cc=kvm-ppc@vger.kernel.org \
    --cc=kvm@vger.kernel.org \
    --cc=kvmarm@lists.cs.columbia.edu \
    --cc=linux-kselftest@vger.kernel.org \
    --cc=linux-mips@vger.kernel.org \
    --cc=linux-s390@vger.kernel.org \
    --cc=maz@kernel.org \
    --cc=pbonzini@redhat.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.