Hi Chris,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on linus/master]
[also build test WARNING on v5.1-rc1 next-20190322]
[if your patch is applied to the wrong git tree, please drop us a note to help 
improve the system]

url:    
https://github.com/0day-ci/linux/commits/Chris-Wilson/dma-fence-Propagate-errors-to-dma-fence-array-container/20190325-074009
config: sparc64-allmodconfig (attached as .config)
compiler: sparc64-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0
reproduce:
        wget 
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O 
~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        GCC_VERSION=7.2.0 make.cross ARCH=sparc64 

All warnings (new ones prefixed by >>):

   In file included from arch/sparc/include/asm/cmpxchg.h:5:0,
                    from arch/sparc/include/asm/atomic_64.h:12,
                    from arch/sparc/include/asm/atomic.h:5,
                    from include/linux/atomic.h:7,
                    from include/asm-generic/bitops/lock.h:5,
                    from arch/sparc/include/asm/bitops_64.h:52,
                    from arch/sparc/include/asm/bitops.h:5,
                    from include/linux/bitops.h:19,
                    from include/linux/kernel.h:12,
                    from include/asm-generic/bug.h:18,
                    from arch/sparc/include/asm/bug.h:25,
                    from include/linux/bug.h:5,
                    from include/linux/mmdebug.h:5,
                    from include/linux/gfp.h:5,
                    from include/linux/slab.h:15,
                    from drivers/dma-buf/dma-fence-array.c:21:
   drivers/dma-buf/dma-fence-array.c: In function 'dma_fence_array_cb_func':
   arch/sparc/include/asm/cmpxchg_64.h:199:3: warning: value computed is not 
used [-Wunused-value]
     ((__typeof__(*(ptr)))__cmpxchg_local((ptr), (unsigned long)(o), \
     ~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
       (unsigned long)(n), sizeof(*(ptr))))
       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/dma-buf/dma-fence-array.c:50:3: note: in expansion of macro 
>> 'cmpxchg_local'
      cmpxchg_local(&array->base.error, 0, f->error);
      ^~~~~~~~~~~~~

vim +/cmpxchg_local +50 drivers/dma-buf/dma-fence-array.c

  > 21  #include <linux/slab.h>
    22  #include <linux/dma-fence-array.h>
    23  
    24  static const char *dma_fence_array_get_driver_name(struct dma_fence 
*fence)
    25  {
    26          return "dma_fence_array";
    27  }
    28  
    29  static const char *dma_fence_array_get_timeline_name(struct dma_fence 
*fence)
    30  {
    31          return "unbound";
    32  }
    33  
    34  static void irq_dma_fence_array_work(struct irq_work *wrk)
    35  {
    36          struct dma_fence_array *array = container_of(wrk, 
typeof(*array), work);
    37  
    38          dma_fence_signal(&array->base);
    39          dma_fence_put(&array->base);
    40  }
    41  
    42  static void dma_fence_array_cb_func(struct dma_fence *f,
    43                                      struct dma_fence_cb *cb)
    44  {
    45          struct dma_fence_array_cb *array_cb =
    46                  container_of(cb, struct dma_fence_array_cb, cb);
    47          struct dma_fence_array *array = array_cb->array;
    48  
    49          if (atomic_read(&array->num_pending) > 0)
  > 50                  cmpxchg_local(&array->base.error, 0, f->error);
    51  
    52          if (atomic_dec_and_test(&array->num_pending))
    53                  irq_work_queue(&array->work);
    54          else
    55                  dma_fence_put(&array->base);
    56  }
    57  

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to