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:   e73f0f0ee7541171d89f2e2491130c7771ba58d3
commit: c43ce12328df0770ce899feabdf9c430c54c766a drm/i915: Use per object 
locking in execbuf, v12.
date:   10 months ago
:::::: branch date: 16 hours ago
:::::: commit date: 10 months ago
compiler: gcc-9 (Debian 9.3.0-22) 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

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