[drm:drm-syncobj-sem 3/8] drivers/dma-buf/sync_debug.c:140:2: note: in expansion of macro 'if'

kbuild test robot fengguang.wu at intel.com
Thu Apr 13 05:04:09 UTC 2017


tree:   git://people.freedesktop.org/~airlied/linux.git drm-syncobj-sem
head:   faf022804be516663a33f620536b006a56d59ee4
commit: 8281fe367426f82485f68767c5c8c7106685e9a9 [3/8] sync_file: split out fence_file base class from sync_file.
config: x86_64-randconfig-s3-04131130 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        git checkout 8281fe367426f82485f68767c5c8c7106685e9a9
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

   drivers/dma-buf/sync_debug.c: In function 'sync_print_sync_file':
   drivers/dma-buf/sync_debug.c:138:52: error: 'struct sync_file' has no member named 'fence'
         sync_status_str(dma_fence_get_status(sync_file->fence)));
                                                       ^~
   In file included from include/linux/linkage.h:4:0,
                    from include/linux/fs.h:4,
                    from include/linux/debugfs.h:18,
                    from drivers/dma-buf/sync_debug.c:17:
   drivers/dma-buf/sync_debug.c:140:34: error: 'struct sync_file' has no member named 'fence'
     if (dma_fence_is_array(sync_file->fence)) {
                                     ^
   include/linux/compiler.h:160:30: note: in definition of macro '__trace_if'
     if (__builtin_constant_p(!!(cond)) ? !!(cond) :   \
                                 ^~~~
>> drivers/dma-buf/sync_debug.c:140:2: note: in expansion of macro 'if'
     if (dma_fence_is_array(sync_file->fence)) {
     ^~
   drivers/dma-buf/sync_debug.c:140:34: error: 'struct sync_file' has no member named 'fence'
     if (dma_fence_is_array(sync_file->fence)) {
                                     ^
   include/linux/compiler.h:160:42: note: in definition of macro '__trace_if'
     if (__builtin_constant_p(!!(cond)) ? !!(cond) :   \
                                             ^~~~
>> drivers/dma-buf/sync_debug.c:140:2: note: in expansion of macro 'if'
     if (dma_fence_is_array(sync_file->fence)) {
     ^~
   drivers/dma-buf/sync_debug.c:140:34: error: 'struct sync_file' has no member named 'fence'
     if (dma_fence_is_array(sync_file->fence)) {
                                     ^
   include/linux/compiler.h:171:16: note: in definition of macro '__trace_if'
      ______r = !!(cond);     \
                   ^~~~
>> drivers/dma-buf/sync_debug.c:140:2: note: in expansion of macro 'if'
     if (dma_fence_is_array(sync_file->fence)) {
     ^~
   drivers/dma-buf/sync_debug.c:141:63: error: 'struct sync_file' has no member named 'fence'
      struct dma_fence_array *array = to_dma_fence_array(sync_file->fence);
                                                                  ^~
   drivers/dma-buf/sync_debug.c:146:32: error: 'struct sync_file' has no member named 'fence'
      sync_print_fence(s, sync_file->fence, true);
                                   ^~

vim +/if +140 drivers/dma-buf/sync_debug.c

0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   11   * but WITHOUT ANY WARRANTY; without even the implied warranty of
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   12   * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   13   * GNU General Public License for more details.
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   14   *
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   15   */
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   16  
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  @17  #include <linux/debugfs.h>
1fe82e2e1 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-05-31   18  #include "sync_debug.h"
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   19  
8a0044846 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21   20  static struct dentry *dbgfs;
8a0044846 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21   21  
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   22  static LIST_HEAD(sync_timeline_list_head);
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   23  static DEFINE_SPINLOCK(sync_timeline_list_lock);
d7fdb0ae9 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21   24  static LIST_HEAD(sync_file_list_head);
d7fdb0ae9 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21   25  static DEFINE_SPINLOCK(sync_file_list_lock);
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   26  
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   27  void sync_timeline_debug_add(struct sync_timeline *obj)
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   28  {
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   29  	unsigned long flags;
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   30  
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   31  	spin_lock_irqsave(&sync_timeline_list_lock, flags);
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   32  	list_add_tail(&obj->sync_timeline_list, &sync_timeline_list_head);
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   33  	spin_unlock_irqrestore(&sync_timeline_list_lock, flags);
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   34  }
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   35  
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   36  void sync_timeline_debug_remove(struct sync_timeline *obj)
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   37  {
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   38  	unsigned long flags;
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   39  
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   40  	spin_lock_irqsave(&sync_timeline_list_lock, flags);
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   41  	list_del(&obj->sync_timeline_list);
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   42  	spin_unlock_irqrestore(&sync_timeline_list_lock, flags);
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   43  }
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   44  
d7fdb0ae9 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21   45  void sync_file_debug_add(struct sync_file *sync_file)
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   46  {
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   47  	unsigned long flags;
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   48  
d7fdb0ae9 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21   49  	spin_lock_irqsave(&sync_file_list_lock, flags);
d7fdb0ae9 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21   50  	list_add_tail(&sync_file->sync_file_list, &sync_file_list_head);
d7fdb0ae9 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21   51  	spin_unlock_irqrestore(&sync_file_list_lock, flags);
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   52  }
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   53  
d7fdb0ae9 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21   54  void sync_file_debug_remove(struct sync_file *sync_file)
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   55  {
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   56  	unsigned long flags;
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   57  
d7fdb0ae9 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21   58  	spin_lock_irqsave(&sync_file_list_lock, flags);
d7fdb0ae9 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21   59  	list_del(&sync_file->sync_file_list);
d7fdb0ae9 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21   60  	spin_unlock_irqrestore(&sync_file_list_lock, flags);
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   61  }
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   62  
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   63  static const char *sync_status_str(int status)
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   64  {
d6c99f4bf drivers/dma-buf/sync_debug.c         Chris Wilson         2017-01-04   65  	if (status < 0)
d6c99f4bf drivers/dma-buf/sync_debug.c         Chris Wilson         2017-01-04   66  		return "error";
954513551 drivers/staging/android/sync_debug.c Peter Senna Tschudin 2014-07-12   67  
954513551 drivers/staging/android/sync_debug.c Peter Senna Tschudin 2014-07-12   68  	if (status > 0)
d6c99f4bf drivers/dma-buf/sync_debug.c         Chris Wilson         2017-01-04   69  		return "signaled";
954513551 drivers/staging/android/sync_debug.c Peter Senna Tschudin 2014-07-12   70  
d6c99f4bf drivers/dma-buf/sync_debug.c         Chris Wilson         2017-01-04   71  	return "active";
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   72  }
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   73  
f54d18670 drivers/dma-buf/sync_debug.c         Chris Wilson         2016-10-25   74  static void sync_print_fence(struct seq_file *s,
f54d18670 drivers/dma-buf/sync_debug.c         Chris Wilson         2016-10-25   75  			     struct dma_fence *fence, bool show)
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   76  {
f54d18670 drivers/dma-buf/sync_debug.c         Chris Wilson         2016-10-25   77  	struct sync_timeline *parent = dma_fence_parent(fence);
d6c99f4bf drivers/dma-buf/sync_debug.c         Chris Wilson         2017-01-04   78  	int status;
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   79  
d6c99f4bf drivers/dma-buf/sync_debug.c         Chris Wilson         2017-01-04   80  	status = dma_fence_get_status_locked(fence);
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   81  
b55b54b5d drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21   82  	seq_printf(s, "  %s%sfence %s",
b55b54b5d drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21   83  		   show ? parent->name : "",
b55b54b5d drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21   84  		   show ? "_" : "",
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   85  		   sync_status_str(status));
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   86  
d6c99f4bf drivers/dma-buf/sync_debug.c         Chris Wilson         2017-01-04   87  	if (status) {
0541cdf58 drivers/staging/android/sync_debug.c Steve Pennington     2014-12-24   88  		struct timespec64 ts64 =
b55b54b5d drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21   89  			ktime_to_timespec64(fence->timestamp);
954513551 drivers/staging/android/sync_debug.c Peter Senna Tschudin 2014-07-12   90  
353fdf170 drivers/staging/android/sync_debug.c Tapasweni Pathak     2014-10-26   91  		seq_printf(s, "@%lld.%09ld", (s64)ts64.tv_sec, ts64.tv_nsec);
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   92  	}
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   93  
724812d6a drivers/staging/android/sync_debug.c Gustavo Padovan      2016-05-31   94  	if (fence->ops->timeline_value_str &&
b55b54b5d drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21   95  		fence->ops->fence_value_str) {
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01   96  		char value[64];
73465f1c0 drivers/staging/android/sync_debug.c Maarten Lankhorst    2015-12-11   97  		bool success;
954513551 drivers/staging/android/sync_debug.c Peter Senna Tschudin 2014-07-12   98  
b55b54b5d drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21   99  		fence->ops->fence_value_str(fence, value, sizeof(value));
73465f1c0 drivers/staging/android/sync_debug.c Maarten Lankhorst    2015-12-11  100  		success = strlen(value);
73465f1c0 drivers/staging/android/sync_debug.c Maarten Lankhorst    2015-12-11  101  
724812d6a drivers/staging/android/sync_debug.c Gustavo Padovan      2016-05-31  102  		if (success) {
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  103  			seq_printf(s, ": %s", value);
73465f1c0 drivers/staging/android/sync_debug.c Maarten Lankhorst    2015-12-11  104  
b55b54b5d drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21  105  			fence->ops->timeline_value_str(fence, value,
b55b54b5d drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21  106  						       sizeof(value));
73465f1c0 drivers/staging/android/sync_debug.c Maarten Lankhorst    2015-12-11  107  
73465f1c0 drivers/staging/android/sync_debug.c Maarten Lankhorst    2015-12-11  108  			if (strlen(value))
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  109  				seq_printf(s, " / %s", value);
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  110  		}
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  111  	}
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  112  
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  113  	seq_puts(s, "\n");
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  114  }
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  115  
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  116  static void sync_print_obj(struct seq_file *s, struct sync_timeline *obj)
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  117  {
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  118  	struct list_head *pos;
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  119  	unsigned long flags;
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  120  
b9bc2b7b6 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-05-31  121  	seq_printf(s, "%s: %d\n", obj->name, obj->value);
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  122  
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  123  	spin_lock_irqsave(&obj->child_list_lock, flags);
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  124  	list_for_each(pos, &obj->child_list_head) {
0431b9065 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-05-31  125  		struct sync_pt *pt =
0431b9065 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-05-31  126  			container_of(pos, struct sync_pt, child_list);
0431b9065 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-05-31  127  		sync_print_fence(s, &pt->base, false);
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  128  	}
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  129  	spin_unlock_irqrestore(&obj->child_list_lock, flags);
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  130  }
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  131  
d7fdb0ae9 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21  132  static void sync_print_sync_file(struct seq_file *s,
d7fdb0ae9 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21  133  				  struct sync_file *sync_file)
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  134  {
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  135  	int i;
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  136  
d7fdb0ae9 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-01-21  137  	seq_printf(s, "[%p] %s: %s\n", sync_file, sync_file->name,
d6c99f4bf drivers/dma-buf/sync_debug.c         Chris Wilson         2017-01-04  138  		   sync_status_str(dma_fence_get_status(sync_file->fence)));
0f0d8406f drivers/staging/android/sync_debug.c Maarten Lankhorst    2014-07-01  139  
f54d18670 drivers/dma-buf/sync_debug.c         Chris Wilson         2016-10-25 @140  	if (dma_fence_is_array(sync_file->fence)) {
f54d18670 drivers/dma-buf/sync_debug.c         Chris Wilson         2016-10-25  141  		struct dma_fence_array *array = to_dma_fence_array(sync_file->fence);
a02b9dc90 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-08-05  142  
a02b9dc90 drivers/staging/android/sync_debug.c Gustavo Padovan      2016-08-05  143  		for (i = 0; i < array->num_fences; ++i)

:::::: The code at line 140 was first introduced by commit
:::::: f54d1867005c3323f5d8ad83eed823e84226c429 dma-buf: Rename struct fence to dma_fence

:::::: TO: Chris Wilson <chris at chris-wilson.co.uk>
:::::: CC: Daniel Vetter <daniel.vetter at ffwll.ch>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
-------------- next part --------------
A non-text attachment was scrubbed...
Name: .config.gz
Type: application/gzip
Size: 25395 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20170413/0d40dae7/attachment-0001.gz>


More information about the dri-devel mailing list