CC: [email protected]
CC: [email protected]
TO: Maarten Lankhorst <[email protected]>
CC: Joonas Lahtinen <[email protected]>
CC: "Thomas Hellström" <[email protected]>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 
master
head:   4525c8781ec0701ce824e8bd379ae1b129e26568
commit: c43ce12328df0770ce899feabdf9c430c54c766a drm/i915: Use per object 
locking in execbuf, v12.
date:   7 weeks ago
:::::: branch date: 14 hours ago
:::::: commit date: 7 weeks ago
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0

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


cppcheck possible warnings: (new ones prefixed by >>, may not real problems)

>> drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c:1515:6: warning: Local 
>> variable reloc_gpu shadows outer function [shadowFunction]
    int reloc_gpu = reloc_entry_gpu(eb, vma, offset, target_addr);
        ^
   drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c:1364:13: note: Shadowed 
declaration
   static u32 *reloc_gpu(struct i915_execbuffer *eb,
               ^
   drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c:1515:6: note: Shadow variable
    int reloc_gpu = reloc_entry_gpu(eb, vma, offset, target_addr);
        ^

vim +1515 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c

102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1506  
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1507  static u64
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1508  relocate_entry(struct i915_vma *vma,
e3d291301f99ef9 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Chris Wilson     
 2020-05-04  1509              const struct drm_i915_gem_relocation_entry 
*reloc,
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1510              struct i915_execbuffer *eb,
e3d291301f99ef9 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Chris Wilson     
 2020-05-04  1511              const struct i915_vma *target)
e3d291301f99ef9 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Chris Wilson     
 2020-05-04  1512  {
e3d291301f99ef9 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Chris Wilson     
 2020-05-04  1513       u64 target_addr = relocation_target(reloc, target);
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1514       u64 offset = reloc->offset;
c43ce12328df077 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19 @1515       int reloc_gpu = reloc_entry_gpu(eb, vma, 
offset, target_addr);
c43ce12328df077 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1516  
c43ce12328df077 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1517       if (reloc_gpu < 0)
c43ce12328df077 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1518               return reloc_gpu;
edf4427b8055dc9 drivers/gpu/drm/i915/i915_gem_execbuffer.c     Chris Wilson     
 2015-01-14  1519  
c43ce12328df077 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1520       if (!reloc_gpu) {
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1521               bool wide = 
eb->reloc_cache.use_64bit_reloc;
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1522               void *vaddr;
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1523  
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1524  repeat:
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1525               vaddr = reloc_vaddr(vma->obj,
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1526                                   &eb->reloc_cache,
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1527                                   offset >> 
PAGE_SHIFT);
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1528               if (IS_ERR(vaddr))
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1529                       return PTR_ERR(vaddr);
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1530  
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1531               GEM_BUG_ON(!IS_ALIGNED(offset, 
sizeof(u32)));
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1532               clflush_write32(vaddr + 
offset_in_page(offset),
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1533                               
lower_32_bits(target_addr),
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1534                               eb->reloc_cache.vaddr);
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1535  
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1536               if (wide) {
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1537                       offset += sizeof(u32);
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1538                       target_addr >>= 32;
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1539                       wide = false;
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1540                       goto repeat;
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1541               }
102a0a9051f4bb6 drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c Maarten 
Lankhorst 2020-08-19  1542       }
edf4427b8055dc9 drivers/gpu/drm/i915/i915_gem_execbuffer.c     Chris Wilson     
 2015-01-14  1543  
2889caa9232109a drivers/gpu/drm/i915/i915_gem_execbuffer.c     Chris Wilson     
 2017-06-16  1544       return target->node.start | UPDATE;
edf4427b8055dc9 drivers/gpu/drm/i915/i915_gem_execbuffer.c     Chris Wilson     
 2015-01-14  1545  }
edf4427b8055dc9 drivers/gpu/drm/i915/i915_gem_execbuffer.c     Chris Wilson     
 2015-01-14  1546  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/[email protected]
_______________________________________________
kbuild mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to