All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: [mchinth: sep_socwatch_linux_5_9 19/41] drivers/platform/x86/socwatch/sw_output_buffer.c:299:25: warning: variable 'write_index' set but not used
Date: Thu, 03 Jun 2021 11:46:28 +0800	[thread overview]
Message-ID: <202106031111.twpHNe4o-lkp@intel.com> (raw)

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

tree:   https://github.com/mchinth/linux  sep_socwatch_linux_5_9
head:   7296486889985b9d9895a39f551764a4b0425bea
commit: e65547c5a9bb37e907edecece4da7cb12dadea96 [19/41] platform/x86: Use $(srctree) instead of $(src) to adapt to kbuild regression
config: x86_64-randconfig-a002-20210603 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project d8e0ae9a76a62bdc6117630d59bf9967ac9bb4ea)
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 x86_64 cross compiling tool for clang build
        # apt-get install binutils-x86-64-linux-gnu
        # https://github.com/mchinth/linux/commit/e65547c5a9bb37e907edecece4da7cb12dadea96
        git remote add mchinth https://github.com/mchinth/linux
        git fetch --no-tags mchinth  sep_socwatch_linux_5_9
        git checkout e65547c5a9bb37e907edecece4da7cb12dadea96
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 

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 >>):

>> drivers/platform/x86/socwatch/sw_output_buffer.c:299:25: warning: variable 'write_index' set but not used [-Wunused-but-set-variable]
           size_t read_index = 0, write_index = 0, dst_index = 0;
                                  ^
>> drivers/platform/x86/socwatch/sw_output_buffer.c:301:7: warning: variable 'wrapped' set but not used [-Wunused-but-set-variable]
           bool wrapped = false;
                ^
>> drivers/platform/x86/socwatch/sw_output_buffer.c:398:6: warning: no previous prototype for function 'reset_output_buffers' [-Wmissing-prototypes]
   void reset_output_buffers(void)
        ^
   drivers/platform/x86/socwatch/sw_output_buffer.c:398:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   void reset_output_buffers(void)
   ^
   static 
   3 warnings generated.
--
>> drivers/platform/x86/socwatch/sw_collector.c:86:1: warning: no previous prototype for function 'sw_copy_driver_interface_info_i' [-Wmissing-prototypes]
   sw_copy_driver_interface_info_i(const struct sw_driver_interface_info *info)
   ^
   drivers/platform/x86/socwatch/sw_collector.c:85:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   struct sw_driver_interface_info *
   ^
   static 
>> drivers/platform/x86/socwatch/sw_collector.c:116:5: warning: no previous prototype for function 'sw_init_driver_interface_info_i' [-Wmissing-prototypes]
   int sw_init_driver_interface_info_i(struct sw_driver_interface_info *info)
       ^
   drivers/platform/x86/socwatch/sw_collector.c:116:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   int sw_init_driver_interface_info_i(struct sw_driver_interface_info *info)
   ^
   static 
>> drivers/platform/x86/socwatch/sw_collector.c:138:5: warning: no previous prototype for function 'sw_init_ops_i' [-Wmissing-prototypes]
   int sw_init_ops_i(const struct sw_hw_ops **ops,
       ^
   drivers/platform/x86/socwatch/sw_collector.c:138:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   int sw_init_ops_i(const struct sw_hw_ops **ops,
   ^
   static 
>> drivers/platform/x86/socwatch/sw_collector.c:157:8: warning: no previous prototype for function 'sw_get_payload_size_i' [-Wmissing-prototypes]
   size_t sw_get_payload_size_i(const struct sw_driver_interface_info *info)
          ^
   drivers/platform/x86/socwatch/sw_collector.c:157:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   size_t sw_get_payload_size_i(const struct sw_driver_interface_info *info)
   ^
   static 
>> drivers/platform/x86/socwatch/sw_collector.c:175:1: warning: no previous prototype for function 'sw_alloc_collector_msg_i' [-Wmissing-prototypes]
   sw_alloc_collector_msg_i(const struct sw_driver_interface_info *info,
   ^
   drivers/platform/x86/socwatch/sw_collector.c:174:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   sw_driver_msg_t *
   ^
   static 
>> drivers/platform/x86/socwatch/sw_collector.c:211:26: warning: no previous prototype for function 'sw_alloc_ops_i' [-Wmissing-prototypes]
   const struct sw_hw_ops **sw_alloc_ops_i(pw_u16_t num_io_descriptors)
                            ^
   drivers/platform/x86/socwatch/sw_collector.c:211:7: note: declare 'static' if the function is not intended to be used outside of this translation unit
   const struct sw_hw_ops **sw_alloc_ops_i(pw_u16_t num_io_descriptors)
         ^
         static 
>> drivers/platform/x86/socwatch/sw_collector.c:323:6: warning: no previous prototype for function 'sw_free_driver_interface_info_i' [-Wmissing-prototypes]
   void sw_free_driver_interface_info_i(struct sw_driver_interface_info *info)
        ^
   drivers/platform/x86/socwatch/sw_collector.c:323:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   void sw_free_driver_interface_info_i(struct sw_driver_interface_info *info)
   ^
   static 
>> drivers/platform/x86/socwatch/sw_collector.c:329:6: warning: no previous prototype for function 'sw_free_ops_i' [-Wmissing-prototypes]
   void sw_free_ops_i(const struct sw_hw_ops **ops)
        ^
   drivers/platform/x86/socwatch/sw_collector.c:329:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   void sw_free_ops_i(const struct sw_hw_ops **ops)
   ^
   static 
>> drivers/platform/x86/socwatch/sw_collector.c:335:5: warning: no previous prototype for function 'sw_reset_driver_interface_info_i' [-Wmissing-prototypes]
   int sw_reset_driver_interface_info_i(struct sw_driver_interface_info *info)
       ^
   drivers/platform/x86/socwatch/sw_collector.c:335:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   int sw_reset_driver_interface_info_i(struct sw_driver_interface_info *info)
   ^
   static 
>> drivers/platform/x86/socwatch/sw_collector.c:453:6: warning: no previous prototype for function 'sw_free_collector_msg_i' [-Wmissing-prototypes]
   void sw_free_collector_msg_i(sw_driver_msg_t *msg)
        ^
   drivers/platform/x86/socwatch/sw_collector.c:453:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   void sw_free_collector_msg_i(sw_driver_msg_t *msg)
   ^
   static 
>> drivers/platform/x86/socwatch/sw_collector.c:459:6: warning: no previous prototype for function 'sw_handle_per_cpu_msg_i' [-Wmissing-prototypes]
   void sw_handle_per_cpu_msg_i(void *info, enum sw_wakeup_action action)
        ^
   drivers/platform/x86/socwatch/sw_collector.c:459:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   void sw_handle_per_cpu_msg_i(void *info, enum sw_wakeup_action action)
   ^
   static 
   11 warnings generated.
--
   drivers/platform/x86/socwatch/sw_telem.c:239:32: error: implicit declaration of function 'ioremap_nocache' [-Werror,-Wimplicit-function-declaration]
           s_punitInterfaceAddr = (u64 *)ioremap_nocache(
                                         ^
   drivers/platform/x86/socwatch/sw_telem.c:239:32: note: did you mean 'ioremap_cache'?
   arch/x86/include/asm/io.h:185:22: note: 'ioremap_cache' declared here
   extern void __iomem *ioremap_cache(resource_size_t offset, unsigned long size);
                        ^
   drivers/platform/x86/socwatch/sw_telem.c:239:25: warning: cast to 'u64 *' (aka 'unsigned long long *') from smaller integer type 'int' [-Wint-to-pointer-cast]
           s_punitInterfaceAddr = (u64 *)ioremap_nocache(
                                  ^~~~~~~~~~~~~~~~~~~~~~~
   drivers/platform/x86/socwatch/sw_telem.c:242:20: warning: cast to 'u64 *' (aka 'unsigned long long *') from smaller integer type 'int' [-Wint-to-pointer-cast]
           s_punitDataAddr = (u64 *)ioremap_nocache(
                             ^~~~~~~~~~~~~~~~~~~~~~~
   drivers/platform/x86/socwatch/sw_telem.c:245:50: warning: cast to 'u64 *' (aka 'unsigned long long *') from smaller integer type 'int' [-Wint-to-pointer-cast]
           s_telemEventInfo[TELEM_PUNIT].ssram_virt_addr = (u64 *)ioremap_nocache(
                                                           ^~~~~~~~~~~~~~~~~~~~~~~
   drivers/platform/x86/socwatch/sw_telem.c:280:27: error: implicit declaration of function 'ioremap_nocache' [-Werror,-Wimplicit-function-declaration]
           s_pmcIPCCmdAddr = (u64 *)ioremap_nocache(
                                    ^
   drivers/platform/x86/socwatch/sw_telem.c:280:20: warning: cast to 'u64 *' (aka 'unsigned long long *') from smaller integer type 'int' [-Wint-to-pointer-cast]
           s_pmcIPCCmdAddr = (u64 *)ioremap_nocache(
                             ^~~~~~~~~~~~~~~~~~~~~~~
   drivers/platform/x86/socwatch/sw_telem.c:283:20: warning: cast to 'u64 *' (aka 'unsigned long long *') from smaller integer type 'int' [-Wint-to-pointer-cast]
           s_pmcIPCStsAddr = (u64 *)ioremap_nocache(
                             ^~~~~~~~~~~~~~~~~~~~~~~
   drivers/platform/x86/socwatch/sw_telem.c:286:21: warning: cast to 'u64 *' (aka 'unsigned long long *') from smaller integer type 'int' [-Wint-to-pointer-cast]
           s_pmcIPCWBufAddr = (u64 *)ioremap_nocache(
                              ^~~~~~~~~~~~~~~~~~~~~~~
   drivers/platform/x86/socwatch/sw_telem.c:289:21: warning: cast to 'u64 *' (aka 'unsigned long long *') from smaller integer type 'int' [-Wint-to-pointer-cast]
           s_pmcIPCRBufAddr = (u64 *)ioremap_nocache(
                              ^~~~~~~~~~~~~~~~~~~~~~~
   drivers/platform/x86/socwatch/sw_telem.c:292:48: warning: cast to 'u64 *' (aka 'unsigned long long *') from smaller integer type 'int' [-Wint-to-pointer-cast]
           s_telemEventInfo[TELEM_PMC].ssram_virt_addr = (u64 *)ioremap_nocache(
                                                         ^~~~~~~~~~~~~~~~~~~~~~~
>> drivers/platform/x86/socwatch/sw_telem.c:671:5: warning: no previous prototype for function 'read_telem' [-Wmissing-prototypes]
   int read_telem(u64 *dst, enum telemetry_unit unit, bool should_retry)
       ^
   drivers/platform/x86/socwatch/sw_telem.c:671:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   int read_telem(u64 *dst, enum telemetry_unit unit, bool should_retry)
   ^
   static 
   9 warnings and 2 errors generated.
--
>> drivers/platform/x86/sepdk/sep/sys_info.c:320:6: warning: variable 'model' set but not used [-Wunused-but-set-variable]
           U32 model = 0;
               ^
   1 warning generated.
--
   drivers/platform/x86/sepdk/sep/pebs.c:96:26: error: variable has incomplete type 'struct flush_tlb_info'
                           struct flush_tlb_info info;
                                                 ^
   arch/x86/include/asm/paravirt_types.h:56:8: note: forward declaration of 'struct flush_tlb_info'
   struct flush_tlb_info;
          ^
>> drivers/platform/x86/sepdk/sep/pebs.c:977:6: warning: variable 'cur_grp' set but not used [-Wunused-but-set-variable]
           U32 cur_grp;
               ^
   drivers/platform/x86/sepdk/sep/pebs.c:1345:8: error: implicit declaration of function '__get_user_pages_fast' [-Werror,-Wimplicit-function-declaration]
                           if (__get_user_pages_fast(lin_addr, 1, 1, &page)) {
                               ^
   drivers/platform/x86/sepdk/sep/pebs.c:1345:8: note: did you mean 'get_user_pages_fast'?
   include/linux/mm.h:1737:5: note: 'get_user_pages_fast' declared here
   int get_user_pages_fast(unsigned long start, int nr_pages,
       ^
   1 warning and 2 errors generated.
--
>> drivers/platform/x86/sepdk/sep/unc_common.c:280:6: warning: variable 'device_id' set but not used [-Wunused-but-set-variable]
           U32 device_id;
               ^
   1 warning generated.
--
>> drivers/platform/x86/sepdk/sep/unc_pci.c:57:6: warning: variable 'device_id' set but not used [-Wunused-but-set-variable]
           U32 device_id;
               ^
   1 warning generated.
--
>> drivers/platform/x86/socwatch/sw_output_buffer.c:301:7: warning: variable 'wrapped' set but not used [-Wunused-but-set-variable]
           bool wrapped = false;
                ^
>> drivers/platform/x86/socwatch/sw_output_buffer.c:299:25: warning: variable 'write_index' set but not used [-Wunused-but-set-variable]
           size_t read_index = 0, write_index = 0, dst_index = 0;
                                  ^
>> drivers/platform/x86/socwatch/sw_output_buffer.c:398:6: warning: no previous prototype for function 'reset_output_buffers' [-Wmissing-prototypes]
   void reset_output_buffers(void)
        ^
   drivers/platform/x86/socwatch/sw_output_buffer.c:398:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   void reset_output_buffers(void)
   ^
   static 
   3 warnings generated.


vim +/write_index +299 drivers/platform/x86/socwatch/sw_output_buffer.c

688e338c753b33 Faycal Benmlih 2019-04-23  292  
688e338c753b33 Faycal Benmlih 2019-04-23  293  /*
688e338c753b33 Faycal Benmlih 2019-04-23  294   * Returns # of bytes successfully consumed on success
688e338c753b33 Faycal Benmlih 2019-04-23  295   * 0 on EOF (no error condition)
688e338c753b33 Faycal Benmlih 2019-04-23  296   */
688e338c753b33 Faycal Benmlih 2019-04-23  297  static size_t consume_buffer(void *dest, size_t bytes_to_read)
688e338c753b33 Faycal Benmlih 2019-04-23  298  {
688e338c753b33 Faycal Benmlih 2019-04-23 @299  	size_t read_index = 0, write_index = 0, dst_index = 0;
688e338c753b33 Faycal Benmlih 2019-04-23  300  	size_t to_read = 0;
688e338c753b33 Faycal Benmlih 2019-04-23 @301  	bool wrapped = false;
688e338c753b33 Faycal Benmlih 2019-04-23  302  	size_t read_size = bytes_to_read;
688e338c753b33 Faycal Benmlih 2019-04-23  303  	unsigned long bytes_not_copied = 0;
688e338c753b33 Faycal Benmlih 2019-04-23  304  	struct sw_driver_continuous_collect data = {0};
688e338c753b33 Faycal Benmlih 2019-04-23  305  
688e338c753b33 Faycal Benmlih 2019-04-23  306  	WRITE_LOCK(sw_continuous_lock);
688e338c753b33 Faycal Benmlih 2019-04-23  307  	smp_mb(); /* memory access ordering */
688e338c753b33 Faycal Benmlih 2019-04-23  308  	read_index = buffer.read_index;
688e338c753b33 Faycal Benmlih 2019-04-23  309  	write_index = buffer.write_index;
688e338c753b33 Faycal Benmlih 2019-04-23  310  	/* EXE sends size as header + payload; we only want payload */
688e338c753b33 Faycal Benmlih 2019-04-23  311  	read_size -= SW_DRIVER_CONTINUOUS_COLLECT_HEADER_SIZE();
688e338c753b33 Faycal Benmlih 2019-04-23  312  	data.collection_size = to_read =
688e338c753b33 Faycal Benmlih 2019-04-23  313  		MIN(read_size, get_data_available(&buffer));
688e338c753b33 Faycal Benmlih 2019-04-23  314  	pw_pr_debug(
688e338c753b33 Faycal Benmlih 2019-04-23  315  		"DEBUG: read = %zu, write = %zu, avail = %zu, to_read = %zu\n",
688e338c753b33 Faycal Benmlih 2019-04-23  316  		read_index, write_index, get_data_available(&buffer), to_read);
688e338c753b33 Faycal Benmlih 2019-04-23  317  	while (to_read) {
688e338c753b33 Faycal Benmlih 2019-04-23  318  		size_t curr_read = to_read;
688e338c753b33 Faycal Benmlih 2019-04-23  319  
688e338c753b33 Faycal Benmlih 2019-04-23  320  		if (read_index + to_read > buffer.size) {
688e338c753b33 Faycal Benmlih 2019-04-23  321  			curr_read = buffer.size - read_index;
688e338c753b33 Faycal Benmlih 2019-04-23  322  			wrapped = true;
688e338c753b33 Faycal Benmlih 2019-04-23  323  			pw_pr_debug(
688e338c753b33 Faycal Benmlih 2019-04-23  324  				"DEBUG: read = %zu, to_read = %zu, curr_read = %zu, buffer.size = %lu, WRAPPED!\n",
688e338c753b33 Faycal Benmlih 2019-04-23  325  				read_index, to_read, curr_read, buffer.size);
688e338c753b33 Faycal Benmlih 2019-04-23  326  		}
688e338c753b33 Faycal Benmlih 2019-04-23  327  		memcpy(&output_buffer[dst_index],
688e338c753b33 Faycal Benmlih 2019-04-23  328  			&buffer.data[read_index], curr_read);
688e338c753b33 Faycal Benmlih 2019-04-23  329  		read_index = (read_index + curr_read) & (buffer.size - 1);
688e338c753b33 Faycal Benmlih 2019-04-23  330  		to_read -= curr_read;
688e338c753b33 Faycal Benmlih 2019-04-23  331  		dst_index += curr_read;
688e338c753b33 Faycal Benmlih 2019-04-23  332  	}
688e338c753b33 Faycal Benmlih 2019-04-23  333  	buffer.read_index = read_index;
688e338c753b33 Faycal Benmlih 2019-04-23  334  	smp_mb(); /* memory access ordering */
688e338c753b33 Faycal Benmlih 2019-04-23  335  	pw_pr_debug("DEBUG: read at end of while = %zu\n", buffer.read_index);
688e338c753b33 Faycal Benmlih 2019-04-23  336  	WRITE_UNLOCK(sw_continuous_lock);
688e338c753b33 Faycal Benmlih 2019-04-23  337  
688e338c753b33 Faycal Benmlih 2019-04-23  338  	/*
688e338c753b33 Faycal Benmlih 2019-04-23  339  	 * Call 'copy_to_user' instead of 'sw_copy_to_user' since
688e338c753b33 Faycal Benmlih 2019-04-23  340  	 * sw_copy_to_user expects to see a 'struct uio' while this
688e338c753b33 Faycal Benmlih 2019-04-23  341  	 * is called from an IOCTL which does NOT have a 'struct uio'
688e338c753b33 Faycal Benmlih 2019-04-23  342  	 */
688e338c753b33 Faycal Benmlih 2019-04-23  343  	bytes_not_copied =
688e338c753b33 Faycal Benmlih 2019-04-23  344  	copy_to_user(dest, (char *)&data,
688e338c753b33 Faycal Benmlih 2019-04-23  345  		SW_DRIVER_CONTINUOUS_COLLECT_HEADER_SIZE());
688e338c753b33 Faycal Benmlih 2019-04-23  346  	if (bytes_not_copied)
688e338c753b33 Faycal Benmlih 2019-04-23  347  		return 0;
688e338c753b33 Faycal Benmlih 2019-04-23  348  
688e338c753b33 Faycal Benmlih 2019-04-23  349  	pw_pr_debug("DEBUG: collection size = %u\n", data.collection_size);
688e338c753b33 Faycal Benmlih 2019-04-23  350  	if (data.collection_size) {
688e338c753b33 Faycal Benmlih 2019-04-23  351  		bytes_not_copied =
688e338c753b33 Faycal Benmlih 2019-04-23  352  			copy_to_user(dest +
688e338c753b33 Faycal Benmlih 2019-04-23  353  				SW_DRIVER_CONTINUOUS_COLLECT_HEADER_SIZE(),
688e338c753b33 Faycal Benmlih 2019-04-23  354  				output_buffer, data.collection_size);
688e338c753b33 Faycal Benmlih 2019-04-23  355  		if (bytes_not_copied)
688e338c753b33 Faycal Benmlih 2019-04-23  356  			return 0;
688e338c753b33 Faycal Benmlih 2019-04-23  357  
688e338c753b33 Faycal Benmlih 2019-04-23  358  	}
688e338c753b33 Faycal Benmlih 2019-04-23  359  	return data.collection_size;
688e338c753b33 Faycal Benmlih 2019-04-23  360  }
688e338c753b33 Faycal Benmlih 2019-04-23  361  
688e338c753b33 Faycal Benmlih 2019-04-23  362  long initialize_circular_buffer(size_t size)
688e338c753b33 Faycal Benmlih 2019-04-23  363  {
688e338c753b33 Faycal Benmlih 2019-04-23  364  	size_t alloc_size = size, read_size = size;
688e338c753b33 Faycal Benmlih 2019-04-23  365  	/*
688e338c753b33 Faycal Benmlih 2019-04-23  366  	 * We require a power of two size
688e338c753b33 Faycal Benmlih 2019-04-23  367  	 */
688e338c753b33 Faycal Benmlih 2019-04-23  368  	pw_pr_debug("DEBUG: old alloc size = %zu\n", alloc_size);
688e338c753b33 Faycal Benmlih 2019-04-23  369  	if ((alloc_size & (alloc_size - 1)) != 0)
688e338c753b33 Faycal Benmlih 2019-04-23  370  		alloc_size = 1 << fls(alloc_size);
688e338c753b33 Faycal Benmlih 2019-04-23  371  
688e338c753b33 Faycal Benmlih 2019-04-23  372  	pw_pr_debug("DEBUG: new alloc size = %zu\n", alloc_size);
688e338c753b33 Faycal Benmlih 2019-04-23  373  	/* Create double-sized buffer */
688e338c753b33 Faycal Benmlih 2019-04-23  374  	alloc_size <<= 1;
688e338c753b33 Faycal Benmlih 2019-04-23  375  	pw_pr_debug("DEBUG: double alloc size = %zu\n", alloc_size);
688e338c753b33 Faycal Benmlih 2019-04-23  376  	memset(&buffer, 0, sizeof(buffer));
688e338c753b33 Faycal Benmlih 2019-04-23  377  	buffer.free_pages =
688e338c753b33 Faycal Benmlih 2019-04-23  378  		sw_allocate_pages(GFP_KERNEL | __GFP_ZERO, alloc_size);
688e338c753b33 Faycal Benmlih 2019-04-23  379  	if (!buffer.free_pages) {
688e338c753b33 Faycal Benmlih 2019-04-23  380  		pw_pr_error("Couldn't allocate space for buffer!\n");
688e338c753b33 Faycal Benmlih 2019-04-23  381  		return -ENOMEM;
688e338c753b33 Faycal Benmlih 2019-04-23  382  	}
688e338c753b33 Faycal Benmlih 2019-04-23  383  	buffer.read_index = buffer.write_index = 0;
688e338c753b33 Faycal Benmlih 2019-04-23  384  	buffer.size = alloc_size;
688e338c753b33 Faycal Benmlih 2019-04-23  385  	SW_INIT_RWLOCK(sw_continuous_lock);
688e338c753b33 Faycal Benmlih 2019-04-23  386  	/*
688e338c753b33 Faycal Benmlih 2019-04-23  387  	 * Create temp output buffer
688e338c753b33 Faycal Benmlih 2019-04-23  388  	 */
688e338c753b33 Faycal Benmlih 2019-04-23  389  	output_buffer = vmalloc(read_size);
688e338c753b33 Faycal Benmlih 2019-04-23  390  	if (!output_buffer) {
688e338c753b33 Faycal Benmlih 2019-04-23  391  		pw_pr_error(
688e338c753b33 Faycal Benmlih 2019-04-23  392  			"Couldn't create temporary buffer for data output!\n");
688e338c753b33 Faycal Benmlih 2019-04-23  393  		return -ENOMEM;
688e338c753b33 Faycal Benmlih 2019-04-23  394  	}
688e338c753b33 Faycal Benmlih 2019-04-23  395  	return 0;
688e338c753b33 Faycal Benmlih 2019-04-23  396  }
688e338c753b33 Faycal Benmlih 2019-04-23  397  
688e338c753b33 Faycal Benmlih 2019-04-23 @398  void reset_output_buffers(void)
688e338c753b33 Faycal Benmlih 2019-04-23  399  {
688e338c753b33 Faycal Benmlih 2019-04-23  400  	buffer.read_index = buffer.write_index = 0;
688e338c753b33 Faycal Benmlih 2019-04-23  401  }
688e338c753b33 Faycal Benmlih 2019-04-23  402  

:::::: The code@line 299 was first introduced by commit
:::::: 688e338c753b33c023de3b774f7347effbf537a4 Platform/x86: Update SoCWatch driver code to 2.10

:::::: TO: Faycal Benmlih <faycal.benmlih@intel.com>
:::::: CC: Faycal Benmlih <faycal.benmlih@intel.com>

---
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: 36062 bytes --]

                 reply	other threads:[~2021-06-03  3:46 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 \
    --in-reply-to=202106031111.twpHNe4o-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=kbuild-all@lists.01.org \
    /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.